Re: [PATCH V2 2/2] hrtimer: Iterate only over active clock-bases

From: Peter Zijlstra
Date: Wed Apr 08 2015 - 08:10:26 EST


On Tue, Apr 07, 2015 at 07:40:53AM +0530, Viresh Kumar wrote:
> At several instances we iterate over all possible clock-bases for a
> particular cpu-base. Whereas, we only need to iterate over active bases.
>
> We already have per cpu-base 'active_bases' field, which is updated on
> addition/removal of hrtimers.
>
> This patch creates for_each_active_base(), which uses 'active_bases' to
> iterate only over active bases.
>
> This also updates code which iterates over clock-bases.

This Changelog is very thin on compelling reasons to do this. Not to
mention you did no analysis on the code generated.

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