Re: [PATCH v3 2/2] perf stat: add interval printing

From: Stephane Eranian
Date: Fri Jan 18 2013 - 18:02:53 EST


On Fri, Jan 18, 2013 at 7:30 PM, Andi Kleen <ak@xxxxxxxxxxxxxxx> wrote:
>> if (forks) {
>> close(go_pipe[1]);
>> + if (interval) {
>> + while (!waitpid(child_pid, &status, WNOHANG)) {
>> + ret = nanosleep(&ts, NULL);
>> + if (ret == 0 && interval)
>> + print_interval();
>
> It would be good to make sure that if someone Ctrl-Cs
> you still print the latest interval(even if shorter)
>
Ok, I can remove the test. The interval is redundant here anyway.
Will post V4 shortly.
--
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/