Re: [PATCH] perf_counter: cleanup for __perf_event_sched_in()

From: Xiao Guangrong
Date: Tue Sep 22 2009 - 05:28:45 EST




Paul Mackerras wrote:
> Xiao Guangrong writes:
>
>> It must be a group leader if event->attr.pinned is "1"
>
> True, but you shouldn't use BUG_ON unless there is no sensible way for
> the kernel to continue executing, and that's not the case here. Make
> it WARN_ON, or better still, WARN_ON_ONCE.
>

Yeah, thanks for you point out, I'll fix it soon.

Thanks,
Xiao

--
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/