This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH][BZ 18743] PowerPC: Fix a race condition when eliding a lock
- From: "Tulio Magno Quites Machado Filho" <tuliom at linux dot vnet dot ibm dot com>
- To: "Carlos O'Donell" <carlos at redhat dot com>
- Cc: libc-alpha at sourceware dot org, Torvald Riegel <triegel at redhat dot com>, munroesj at linux dot vnet dot ibm dot com
- Cc:
- Date: Fri, 31 Jul 2015 12:14:25 -0300
- Subject: Re: [PATCH][BZ 18743] PowerPC: Fix a race condition when eliding a lock
- Authentication-results: sourceware.org; auth=none
- References: <1438274936-26493-1-git-send-email-tuliom at linux dot vnet dot ibm dot com> <55BAEE77 dot 9000501 at redhat dot com>
"Carlos O'Donell" <carlos@redhat.com> writes:
> On 07/30/2015 12:48 PM, Tulio Magno Quites Machado Filho wrote:
>> Note: this patch is updating the NEWS section of glibc 2.22 because that's the
>> only one available right now. Whether this patch will end up in glibc 2.23
>> or 2.22 has to be defined.
>
> No for 2.22. Please work on 2.23.
Ack.
> You will need to describe in much more detail exactly what synchronization
> you are using to ensure that this is race-free now.
In fact, I'm using the same mechanism already available. I'm just adding
another step inside the critical path.
But I'll improve my commit message and the comments in the code.
> If there is any synchronization going on with HTM we need a way to describe
> that synchronization and document the concurrency aspect of the change. It
> is the only way we'll remember what we did when we come back to look at this
> code in 1, 2, 5, 10 years.
Agreed.
--
Tulio Magno