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: [PATCH siddhesh/wait_bitset] Use FUTEX_WAIT_BITSET forpthread_cond_timedwait for non-x86: ppc, s390

From: Siddhesh Poyarekar <siddhesh at redhat dot com>
To: Roland McGrath <roland at hack dot frob dot com>
Cc: libc-alpha at sourceware dot org
Date: Wed, 24 Oct 2012 13:26:27 +0530
Subject: Re: [PATCH siddhesh/wait_bitset] Use FUTEX_WAIT_BITSET forpthread_cond_timedwait for non-x86: ppc, s390
References: <20121022182330.4f774b71@spoyarek><20121024000039.5492B2C061@topped-with-meat.com>

On Tue, 23 Oct 2012 17:00:39 -0700 (PDT), Roland wrote:
> Space before paren, or omit the parens after 'defined' entirely.
> They are superfluous here.
> 
> The substance of the change looks fine to me, but it needs powerpc
> and s390 folks to look at it.

Thanks, I've updated my branch with fixes based on your comments.

Siddhesh

References:
- [PATCH siddhesh/wait_bitset] Use FUTEX_WAIT_BITSET forpthread_cond_timedwait for non-x86: ppc, s390
  - From: Siddhesh Poyarekar
- Re: [PATCH siddhesh/wait_bitset] Use FUTEX_WAIT_BITSET forpthread_cond_timedwait for non-x86: ppc, s390
  - From: Roland McGrath

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