Re: [patch 9/9] Scheduler profiling - Use conditional calls

From: Andi Kleen
Date: Thu May 31 2007 - 08:43:19 EST


Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxx> writes:
> }
> - profile_hit(SCHED_PROFILING, __builtin_return_address(0));
> + cond_call(profile_on,
> + profile_hit(SCHED_PROFILING, __builtin_return_address(0)));

Would it be possible to use a syntax like

if (unlikely_cond_call(variable)) { (or better name)
...
}

instead? I think that would be much nicer to read than having
code in a macro argument

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