Re: perf: report loops endlessly if record was killed via SIGTERM

From: Mike Galbraith
Date: Mon Apr 29 2013 - 07:43:32 EST


On Mon, 2013-04-29 at 07:27 -0400, David Ahern wrote:
> On 4/29/13 4:00 AM, Mike Galbraith wrote:
> > Say SIGINT instead,
> > and it works as expected. SIGTERM leaves perf.data in a bad state that
> > report can't deal with, but it'll happily keep trying forever.
>
> header is usually updated on_exit which requires a clean exit. perhaps
> we should add SIGTERM to the signal list.

Yeah. Graceful exit, and idiot proofing for free :)

-Mike

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/