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 3/4] nptl: Fix testcases for new pthread cancellation, mechanism


Hi Joseph, thanks for the review.


On 19-09-2014 14:02, Joseph S. Myers wrote:
> On Fri, 19 Sep 2014, Adhemerval Zanella wrote:
>
>> This patches fixes the NPTL testcase that assumes the old behavior and
> Do the fixes result in tests that work both with and without the main 
> changes?  If so, they should go first.  (In general, for bisectability 
> either you fix tests first to work both with and without the main changes, 
> or the test fixes need to go in the same commit as the main changes even 
> if posted separately for convenience.)

The debug/tst-backtrace5.c is adjusted to take in consideration the new stack chain
call for syscalls. The tst-cancelX modification work on both schemes (the thread
in cancelled in the syscall itself).

>
>> also remove the tst-cancel-wrappers.sh test (which checks for symbols
>> that does not exist anymore).
> Is there a better way of achieving the substance of this test (i.e. 
> verifying that all those functions are cancellable) that still works after 
> the patch?
>
I will double check, but later time I checked all the cancelable entry point in 
GLIBC are already being fairly tested by other nptl/tst-cancelX cases.


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