diff options
author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2015-03-27 22:39:49 +0100 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2015-03-30 19:36:31 +0200 |
commit | d631c8cceb1d1d06f372878935949d421585186b (patch) | |
tree | 7a213ad0e5ede27961a63eb97bbc361309c4a599 /kernel/trace/trace_events_trigger.c | |
parent | trace: Don't use __weak in header files (diff) | |
download | linux-d631c8cceb1d1d06f372878935949d421585186b.tar.xz linux-d631c8cceb1d1d06f372878935949d421585186b.zip |
ring-buffer: Remove duplicate use of '&' in recursive code
A clean up of the recursive protection code changed
val = this_cpu_read(current_context);
val--;
val &= this_cpu_read(current_context);
to
val = this_cpu_read(current_context);
val &= val & (val - 1);
Which has a duplicate use of '&' as the above is the same as
val = val & (val - 1);
Actually, it would be best to remove that line altogether and
just add it to where it is used.
And Christoph even mentioned that it can be further compacted to
just a single line:
__this_cpu_and(current_context, __this_cpu_read(current_context) - 1);
Link: http://lkml.kernel.org/alpine.DEB.2.11.1503271423580.23114@gentwo.org
Suggested-by: Christoph Lameter <cl@linux.com>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace_events_trigger.c')
0 files changed, 0 insertions, 0 deletions