On Fri, 2012-09-07 at 16:29 +0200, Stephane Eranian wrote: Style nit:
> + if (h->active) > + list_for_each_entry_safe(cpuctx, tmp, head, rotation_list) > + rotations += perf_rotate_context(cpuctx); > + if (!hrtimer_callback_running(hr)) > + __hrtimer_start_range_ns(hr, ns_to_ktime(PERF_CPU_HRTIMER), > + 0, HRTIMER_MODE_REL_PINNED, 0); All multi-line statements should be a stmt block, even if not strictly required. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/