about summary refs log tree commit diff
diff options
context:
space:
mode:
authorSiddhesh Poyarekar <siddhesh@redhat.com>2015-02-18 11:06:05 +0530
committerSiddhesh Poyarekar <siddhesh@redhat.com>2015-02-18 11:06:06 +0530
commit94c5a52a841f807a23dbdd19a5ddeb505cc1d543 (patch)
tree82235a60b85b44b18bae60a31069a3bfa7d9c8d2
parent8a35c3fe122d49ba76dff815b3537affb5a50b45 (diff)
downloadglibc-94c5a52a841f807a23dbdd19a5ddeb505cc1d543.tar.gz
glibc-94c5a52a841f807a23dbdd19a5ddeb505cc1d543.tar.xz
glibc-94c5a52a841f807a23dbdd19a5ddeb505cc1d543.zip
Consolidate arena_lookup and arena_lock into a single arena_get
This seems to have been left behind as an artifact of some old changes
and can now be merged.  Verified that the only generated code change
on x86_64 is that of line numbers in asserts, like so:

@@ -27253,7 +27253,7 @@ Disassembly of section .text:
   416f09:      48 89 42 20             mov    %rax,0x20(%rdx)
   416f0d:      e9 7e f6 ff ff          jmpq   416590 <_int_free+0x230>
   416f12:      b9 3f 9f 4a 00          mov    $0x4a9f3f,%ecx
-  416f17:      ba d5 0f 00 00          mov    $0xfd5,%edx
+  416f17:      ba d6 0f 00 00          mov    $0xfd6,%edx
   416f1c:      be a8 9b 4a 00          mov    $0x4a9ba8,%esi
   416f21:      bf 6a 9c 4a 00          mov    $0x4a9c6a,%edi
   416f26:      e8 45 e8 ff ff          callq  415770 <__malloc_assert>
-rw-r--r--ChangeLog5
-rw-r--r--malloc/malloc.c3
2 files changed, 6 insertions, 2 deletions
diff --git a/ChangeLog b/ChangeLog
index cdf49aaa7e..3ada4f6ef0 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2015-02-18  Siddhesh Poyarekar  <siddhesh@redhat.com>
+
+	* malloc/malloc.c (__libc_malloc): Consolidate arena_lookup and
+	arena_lock into a single arena_get.
+
 2015-02-17  Carlos O'Donell  <carlos@redhat.com>
 
 	* dl-reloc.c: Inlucde libc-internal.h.
diff --git a/malloc/malloc.c b/malloc/malloc.c
index ad9487e5e6..f361bad636 100644
--- a/malloc/malloc.c
+++ b/malloc/malloc.c
@@ -2886,9 +2886,8 @@ __libc_malloc (size_t bytes)
   if (__builtin_expect (hook != NULL, 0))
     return (*hook)(bytes, RETURN_ADDRESS (0));
 
-  arena_lookup (ar_ptr);
+  arena_get (ar_ptr, bytes);
 
-  arena_lock (ar_ptr, bytes);
   if (!ar_ptr)
     return 0;