[RFC,PATCH 14/14] utrace core
Ingo Molnar
mingo at elte.hu
Wed Nov 25 16:00:32 UTC 2009
* Oleg Nesterov <oleg at redhat.com> wrote:
> Much better. But in this case please note that most of tracehooks just
> do:
>
> if (unlikely(task_utrace_flags(current) & SOME_EVENT))
> utrace_report_some_event();
>
> I really don't understand why we shouldn't have (trivial!) helpers for
> this. (As for naming - personally I do not care at all ;)
We prefer helpers in most such cases - especially when in the normal
case the helper has no side effects - as here. Then we want to compress
all such reporting/callback as much as possible.
Using helpers to abstract away functionality is one of the basic
elements of writing clean kernel code.
Ingo
More information about the utrace-devel
mailing list