From 13e23af76e64efc055103344bd56d553ceee8512 Mon Sep 17 00:00:00 2001 From: Carlos O'Donell Date: Wed, 22 May 2013 23:22:36 -0400 Subject: [PATCH] Update comments in ldconfig.c and dl-hwcaps.c. In dl-hwcaps.c the comment read that rounding was done to ElfW(Addr), but it's actually rounded to ElfW(Word). In ldconfig.c we make each comment a sentence and mention that the "tls" pseudo-hwcap is just for legacy installations where TLS was optional. --- 2013-05-22 Carlos O'Donell * elf/ldconfig.c (is_hwcap_platform): Make comments full setences. (main): Mention "tls" pseudo-hwcap is legacy. * elf/dl-hwcaps.c (_dl_important_hwcaps): Correct rounding comment. --- ChangeLog | 6 ++++++ elf/dl-hwcaps.c | 4 ++-- elf/ldconfig.c | 14 ++++++++------ 3 files changed, 16 insertions(+), 8 deletions(-) diff --git a/ChangeLog b/ChangeLog index 04dc3afe24..665b0a092d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2013-05-22 Carlos O'Donell + + * elf/ldconfig.c (is_hwcap_platform): Make comments full setences. + (main): Mention "tls" pseudo-hwcap is legacy. + * elf/dl-hwcaps.c (_dl_important_hwcaps): Correct rounding comment. + 2013-05-22 Joseph Myers * math/gen-libm-test.pl (parse_args): Output only string of diff --git a/elf/dl-hwcaps.c b/elf/dl-hwcaps.c index 1b7fe52a6a..94cbf6cc2d 100644 --- a/elf/dl-hwcaps.c +++ b/elf/dl-hwcaps.c @@ -68,9 +68,9 @@ _dl_important_hwcaps (const char *platform, size_t platform_len, size_t *sz, + GLRO(dl_sysinfo_map)->l_addr); /* The standard ELF note layout is exactly as the anonymous struct. The next element is a variable length vendor name of length - VENDORLEN (with a real length rounded to ElfW(Addr)), followed + VENDORLEN (with a real length rounded to ElfW(Word)), followed by the data of length DATALEN (with a real length rounded to - ElfW(Addr)). */ + ElfW(Word)). */ const struct { ElfW(Word) vendorlen; diff --git a/elf/ldconfig.c b/elf/ldconfig.c index 13f262d65c..c7b9eb92fe 100644 --- a/elf/ldconfig.c +++ b/elf/ldconfig.c @@ -174,17 +174,17 @@ is_hwcap_platform (const char *name) { int hwcap_idx = _dl_string_hwcap (name); - /* Is this a normal hwcap for the machine e.g. fpu? */ + /* Is this a normal hwcap for the machine like "fpu?" */ if (hwcap_idx != -1 && ((1 << hwcap_idx) & hwcap_mask)) return 1; - /* ... Or is it a platform pseudo-hwcap e.g. i686? */ + /* Is this a platform pseudo-hwcap like "i686?" */ hwcap_idx = _dl_string_platform (name); if (hwcap_idx != -1) return 1; - /* ... Or is this one of the extra pseudo-hwcaps that we map beyond - _DL_FIRST_EXTRA e.g. tls, or nosegneg? */ + /* Is this one of the extra pseudo-hwcaps that we map beyond + _DL_FIRST_EXTRA like "tls", or "nosegneg?" */ for (hwcap_idx = _DL_FIRST_EXTRA; hwcap_idx < 64; ++hwcap_idx) if (hwcap_extra[hwcap_idx - _DL_FIRST_EXTRA] != NULL && !strcmp (name, hwcap_extra[hwcap_idx - _DL_FIRST_EXTRA])) @@ -1270,8 +1270,10 @@ main (int argc, char **argv) add_dir (argv[i]); } - /* The last entry in hwcap_extra is reserved for the "tls" - pseudo-hwcap which indicates support for TLS. */ + /* The last entry in hwcap_extra is reserved for the "tls" pseudo-hwcap which + indicates support for TLS. This pseudo-hwcap is only used by old versions + under which TLS support was optional. The entry is no longer needed, but + must remain for compatibility. */ hwcap_extra[63 - _DL_FIRST_EXTRA] = "tls"; set_hwcap (); -- 2.43.5