[prev in list] [next in list] [prev in thread] [next in thread] 

List:       glibc-alpha
Subject:    Re: [PATCH 0/8] nptl: Fix Race conditions in pthread cancellation (BZ#12683)
From:       Adhemerval Zanella <adhemerval.zanella () linaro ! org>
Date:       2015-06-28 20:14:29
Message-ID: 559055A5.5020006 () linaro ! org
[Download RAW message or body]



On 26-06-2015 18:08, Joseph Myers wrote:
> Are any of the patches in this set intended as separate commits, or are 
> they just posted separately for convenience of review?  (Bisectability 
> means it's a bad idea to have a commit that breaks builds for an 
> architecture and then a separate commit that fixes them.)
> 

I see it more logically to divide this implementation for each
port instead of pack everything in a large patch, although I am
not against to push it in only one part.

Also, for arm/aarch64 it shows the minimum required changes to adequate
the port, which make review somewhat easier.

However as I stated the patchset as is only works for the mentioned
architectures and it break on the not supported ones.  My idea is
also only push the patchset when all architectures have been ported.
[prev in list] [next in list] [prev in thread] [next in thread]