[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1347032550.18408.92.camel@twins>
Date: Fri, 07 Sep 2012 17:42:30 +0200
From: Peter Zijlstra <peterz@...radead.org>
To: Stephane Eranian <eranian@...gle.com>
Cc: linux-kernel@...r.kernel.org, mingo@...e.hu, ak@...ux.intel.com,
zheng.z.yan@...el.com, robert.richter@....com
Subject: Re: [PATCH 1/3] perf: use hrtimer for event multiplexing
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 majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists