This is the mail archive of the
binutils-cvs@sourceware.org
mailing list for the binutils project.
[binutils-gdb] Initialize pagesize_m1 in bfd_cache_init
- From: H.J.Lu <hjl at sourceware dot org>
- To: bfd-cvs at sourceware dot org
- Date: 11 Apr 2015 22:14:09 -0000
- Subject: [binutils-gdb] Initialize pagesize_m1 in bfd_cache_init
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=2c008f9a5b65fe8d9379e761b056fabeb34a5da6
commit 2c008f9a5b65fe8d9379e761b056fabeb34a5da6
Author: H.J. Lu <hjl.tools@gmail.com>
Date: Sat Apr 11 15:12:08 2015 -0700
Initialize pagesize_m1 in bfd_cache_init
There is no need to check and initialize pagesize_m1 in cache_bmmap.
* cache.c (cache_bmmap): Move pagesize_m1 ... to
(pagesize_m1): Here.
(bfd_cache_init): Initialize pagesize_m1.
Diff:
---
bfd/ChangeLog | 6 ++++++
bfd/cache.c | 11 +++++++----
2 files changed, 13 insertions(+), 4 deletions(-)
diff --git a/bfd/ChangeLog b/bfd/ChangeLog
index 207884a..8935574 100644
--- a/bfd/ChangeLog
+++ b/bfd/ChangeLog
@@ -1,5 +1,11 @@
2015-04-11 H.J. Lu <hongjiu.lu@intel.com>
+ * cache.c (cache_bmmap): Move pagesize_m1 ... to
+ (pagesize_m1): Here.
+ (bfd_cache_init): Initialize pagesize_m1.
+
+2015-04-11 H.J. Lu <hongjiu.lu@intel.com>
+
* elf32-i386.c (elf_i386_relocate_section): Replace SYMBOLIC_BIND
with SYMBOL_REFERENCES_LOCAL when checking R_386_GOTOFF against
protected data symbol when building shared library.
diff --git a/bfd/cache.c b/bfd/cache.c
index 94a82da..67d3d17 100644
--- a/bfd/cache.c
+++ b/bfd/cache.c
@@ -418,6 +418,10 @@ cache_bstat (struct bfd *abfd, struct stat *sb)
return sts;
}
+#ifdef HAVE_MMAP
+static uintptr_t pagesize_m1;
+#endif
+
static void *
cache_bmmap (struct bfd *abfd ATTRIBUTE_UNUSED,
void *addr ATTRIBUTE_UNUSED,
@@ -435,7 +439,6 @@ cache_bmmap (struct bfd *abfd ATTRIBUTE_UNUSED,
#ifdef HAVE_MMAP
else
{
- static uintptr_t pagesize_m1;
FILE *f;
file_ptr pg_offset;
bfd_size_type pg_len;
@@ -444,9 +447,6 @@ cache_bmmap (struct bfd *abfd ATTRIBUTE_UNUSED,
if (f == NULL)
return ret;
- if (pagesize_m1 == 0)
- pagesize_m1 = getpagesize () - 1;
-
/* Handle archive members. */
if (abfd->my_archive != NULL)
offset += abfd->origin;
@@ -496,6 +496,9 @@ bfd_cache_init (bfd *abfd)
if (! close_one ())
return FALSE;
}
+#ifdef HAVE_MMAP
+ pagesize_m1 = getpagesize () - 1;
+#endif
abfd->iovec = &cache_iovec;
insert (abfd);
++open_files;