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

List:       ltp-list
Subject:    Re: [LTP] [PATCH v5 2/2] open_posix_testsuite/../mq_timedsend/12-1: fix race
From:       Jan Stancek <jstancek () redhat ! com>
Date:       2013-08-27 12:09:29
Message-ID: 1493995038.1942917.1377605369570.JavaMail.root () redhat ! com
[Download RAW message or body]



----- Original Message -----
> From: chrubis@suse.cz
> To: "Jan Stancek" <jstancek@redhat.com>
> Cc: ltp-list@lists.sourceforge.net
> Sent: Tuesday, 27 August, 2013 1:51:12 PM
> Subject: Re: [LTP] [PATCH v5 2/2] open_posix_testsuite/../mq_timedsend/12-1: fix race
> 
> Hi!
> > > Test can send SIGUSR1 signal after thread has already completed
> > > and report failure from main() and success from a_thread_func():
> > > 
> > >   conformance/interfaces/mq_timedsend/12-1: execution: UNRESOLVED:
> > >   Output:
> > >   Error: in pthread_kill
> > >   : Success
> > >   thread: mq_timedsend interrupted by signal and correctly set errno to
> > >   EINTR
> > > 
> > > Synchronize exit of a_thread_func(), so that thread can exit only
> > > after loop sending SIGUSR1 completed.
> > > 
> > > Signed-off-by: Jan Stancek <jstancek@redhat.com>
> > > ---
> > > Changes in v5:
> > > - return value from pthread_exit is not used, always return NULL
> > 
> > Looks good to me. Acked.
> > 
> 
> Did you forget to commit this patch?

Pushed now, thanks for reminder,
Jan

> 
> --
> Cyril Hrubis
> chrubis@suse.cz
> 

------------------------------------------------------------------------------
Introducing Performance Central, a new site from SourceForge and 
AppDynamics. Performance Central is your source for news, insights, 
analysis and resources for efficient Application Performance Management. 
Visit us today!
http://pubads.g.doubleclick.net/gampad/clk?id=48897511&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
[prev in list] [next in list] [prev in thread] [next in thread]