This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH v3] PowerPC: libc single-thread lock optimization
- From: "Tulio Magno Quites Machado Filho" <tuliom at linux dot vnet dot ibm dot com>
- To: Florian Weimer <fweimer at redhat dot com>
- Cc: libc-alpha at sourceware dot org, adhemerval dot zanella at linaro dot org, munroesj at linux dot vnet dot ibm dot com
- Cc:
- Date: Fri, 11 Mar 2016 18:12:02 -0300
- Subject: Re: [PATCH v3] PowerPC: libc single-thread lock optimization
- Authentication-results: sourceware.org; auth=none
- References: <540080DF dot 6030205 at linux dot vnet dot ibm dot com> <1457721337-30897-1-git-send-email-tuliom at linux dot vnet dot ibm dot com> <56E310BF dot 303 at redhat dot com>
Florian Weimer <fweimer@redhat.com> writes:
> On 03/11/2016 07:35 PM, Tulio Magno Quites Machado Filho wrote:
>> diff --git a/malloc/malloc.c b/malloc/malloc.c
>> index b8a43bf..1eed794 100644
>> --- a/malloc/malloc.c
>> +++ b/malloc/malloc.c
>> @@ -4150,7 +4150,7 @@ static void malloc_consolidate(mstate av)
>> maxfb = &fastbin (av, NFASTBINS - 1);
>> fb = &fastbin (av, 0);
>> do {
>> - p = atomic_exchange_acq (fb, 0);
>> + p = atomic_exchange_acq (fb, NULL);
>> if (p != 0) {
>> do {
>> check_inuse_chunk(av, p);
>
> This should go in immediately and separately, it is independent of the
> rest of the patch.
Makes sense.
I pushed this hunk as commit b43f552a.
Thanks!
--
Tulio Magno