Re: [PATCH V2] clockevents: Fix cpu down race for hrtimer based broadcasting

From: Peter Zijlstra
Date: Thu Apr 02 2015 - 08:58:36 EST


On Thu, Apr 02, 2015 at 02:12:47PM +0200, Ingo Molnar wrote:
> Okay. I'll resurrect the fix with a hotplug_cpu__tick_pull() name -
> agreed?

Sure.

> > > Also, I improved the changelog (attached below), but decided
> > > against applying it until these questions are cleared - please use
> > > that for future versions of this patch.
> >
> >
> > > Fixes: http://linuxppc.10917.n7.nabble.com/offlining-cpus-breakage-td88619.html
> >
> > You forgot to fix the Fixes line ;-)
> >
> > My copy has:
> >
> > Fixes: 5d1638acb9f6 ("tick: Introduce hrtimer based broadcast")
>
> Hm, not sure how that got lost - my git-log of the patch ported on top
> of timers/core still has it:


> Suggested-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> Tested-by: Nicolas Pitre <nico@xxxxxxxxxx>
> Signed-off-by: Preeti U. Murthy <preeti@xxxxxxxxxxxxxxxxxx>
> Cc: linuxppc-dev@xxxxxxxxxxxxxxxx
> Cc: mpe@xxxxxxxxxxxxxx
> Cc: nicolas.pitre@xxxxxxxxxx
> Cc: peterz@xxxxxxxxxxxxx
> Cc: rjw@xxxxxxxxxxxxx
> Fixes: http://linuxppc.10917.n7.nabble.com/offlining-cpus-breakage-td88619.html
> Link: http://lkml.kernel.org/r/20150330092410.24979.59887.stgit@xxxxxxxxxxxxxxxxx
> [ Merged it to the latest timer tree, tidied up the changelog. ]
> Signed-off-by: Ingo Molnar <mingo@xxxxxxxxxx>

That doesn't have my Sob in it, so you took it from the list yourself,
not from my queue. I also (concurrently it seems) fixed up the Changelog
some :-)
--
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/