Re: [PATCH v2 1/2] perf record: Propagate exit status of a command line workload

2014-04-23 Thread Stephane Eranian
On Wed, Apr 23, 2014 at 4:38 PM, Namhyung Kim wrote: > Hi Stephane, > > 2014-04-23 (수), 14:11 +0200, Stephane Eranian: >> On Wed, Apr 23, 2014 at 8:40 AM, Namhyung Kim wrote: >> > +out_child: >> > + if (forks) { >> > + int exit_status; >> > >> > - return 0; >> > +

Re: [PATCH v2 1/2] perf record: Propagate exit status of a command line workload

2014-04-23 Thread Namhyung Kim
Hi Stephane, 2014-04-23 (수), 14:11 +0200, Stephane Eranian: > On Wed, Apr 23, 2014 at 8:40 AM, Namhyung Kim wrote: > > +out_child: > > + if (forks) { > > + int exit_status; > > > > - return 0; > > + if (!child_finished) > > + kill(rec-

Re: [PATCH v2 1/2] perf record: Propagate exit status of a command line workload

2014-04-23 Thread Stephane Eranian
On Wed, Apr 23, 2014 at 8:40 AM, Namhyung Kim wrote: > Currently perf record doesn't propagate the exit status of a workload > given by the command line. But sometimes it'd useful if it's > propagated so that a monitoring script can handle errors > appropriately. > > To do that, it got rid of exi