This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Should glibc be fully reentrant? -- No. (Roland was right).


On 12/15/2014 11:38 AM, Torvald Riegel wrote:
> On Mon, 2014-12-15 at 09:52 +0100, Florian Weimer wrote:
>> On 12/12/2014 05:17 PM, OndÅej BÃlka wrote:
>>> On Fri, Dec 12, 2014 at 10:50:19AM +0100, Florian Weimer wrote:
>>>> On 12/11/2014 03:11 PM, OndÅej BÃlka wrote:
>>>>
>>>>> Yes, I wrote that from head so I forgot volatile/asm barrier. One could
>>>>> add requirement like needs to be compiled by gcc4-6+ instead pure C as
>>>>> just using signals is not part of C standard.
>>>>
>>>> GCC emulates atomics with locks on some platforms, or some lock-free
>>>> instruction sequences may not be reentrant.  This begins to look
>>>> like a can of worms, unfortunately.
>>>>
>>> It uses only thread local variable. If they are not reentrant its
>>> gigantic hole, you could not for example use sigaction as it could
>>> set errno which is thread local variable.
>>
>> Sorry, what I'm trying to say is that atomics are not specified as 
>> async-signal-safe
> 
> That's not correct in general.  My TLDR reply would be that all atomic
> operations that claim to be lock-free are also async-signal-safe (see
> GCC's __atomic_always_lock_free and the related C11/C++11 functions).  

That's not what C11 says.  I have reviewed this area of the standard
several times, and I do believe it's unspecified in the sense that the
standard intends to say something about it, but doesn't actually do so.

> Using this rule of thumb would allow us to assume atomics to be
> async-signal-safe in glibc.
> 
> Outside of glibc, I agree that some atomics (notably those not claiming
> to be lock-free) to be actually not async-signal safe.

Okay, that makes sense.

-- 
Florian Weimer / Red Hat Product Security


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]