Re: [patch-rt] hotplug, hrtimer: Migrate expired/deferred timers during cpu offline

2017-08-17 Thread Mike Galbraith
On Thu, 2017-08-17 at 19:17 +0200, Sebastian Andrzej Siewior wrote: > On 2017-08-17 18:50:42 [+0200], To Mike Galbraith wrote: > > __remove_hrtimer() shouldn't be required because it has been done > > already. It should be enough to just list_splice() one list to the > > other and raise the softirq

Re: [patch-rt] hotplug, hrtimer: Migrate expired/deferred timers during cpu offline

2017-08-17 Thread Mike Galbraith
On Thu, 2017-08-17 at 19:26 +0200, Mike Galbraith wrote: > On Thu, 2017-08-17 at 19:17 +0200, Sebastian Andrzej Siewior wrote: > > On 2017-08-17 18:50:42 [+0200], To Mike Galbraith wrote: > > > __remove_hrtimer() shouldn't be required because it has been done > > > already. It should be enough to j

Re: [patch-rt] hotplug, hrtimer: Migrate expired/deferred timers during cpu offline

2017-08-17 Thread Mike Galbraith
On Thu, 2017-08-17 at 19:17 +0200, Sebastian Andrzej Siewior wrote: > On 2017-08-17 18:50:42 [+0200], To Mike Galbraith wrote: > > __remove_hrtimer() shouldn't be required because it has been done > > already. It should be enough to just list_splice() one list to the > > other and raise the softirq

Re: [patch-rt] hotplug, hrtimer: Migrate expired/deferred timers during cpu offline

2017-08-17 Thread Sebastian Andrzej Siewior
On 2017-08-17 18:50:42 [+0200], To Mike Galbraith wrote: > __remove_hrtimer() shouldn't be required because it has been done > already. It should be enough to just list_splice() one list to the > other and raise the softirq afterwards. Mike, can you check this please: diff --git a/kernel/time/hrt

Re: [patch-rt] hotplug, hrtimer: Migrate expired/deferred timers during cpu offline

2017-08-17 Thread Sebastian Andrzej Siewior
On 2017-08-14 09:59:48 [+0200], Mike Galbraith wrote: > On Fri, 2017-08-11 at 10:15 +0200, Mike Galbraith wrote: > > On Fri, 2017-08-11 at 09:55 +0200, Mike Galbraith wrote: > > > The below fixes the list debug explosion up. > > > > > > If we do not migrate expired/deferred timers during cpu offli

Re: [patch-rt] hotplug, hrtimer: Migrate expired/deferred timers during cpu offline

2017-08-14 Thread Mike Galbraith
On Fri, 2017-08-11 at 10:15 +0200, Mike Galbraith wrote: > On Fri, 2017-08-11 at 09:55 +0200, Mike Galbraith wrote: > > The below fixes the list debug explosion up. > > > > If we do not migrate expired/deferred timers during cpu offline, ->cb_entry > > will be corrupted by online initialization of

Re: [patch-rt] hotplug, hrtimer: Migrate expired/deferred timers during cpu offline

2017-08-11 Thread Mike Galbraith
On Fri, 2017-08-11 at 09:55 +0200, Mike Galbraith wrote: > The below fixes the list debug explosion up. > > If we do not migrate expired/deferred timers during cpu offline, ->cb_entry > will be corrupted by online initialization of base->expired, leading to a > loud list debug complaint should som

[patch-rt] hotplug, hrtimer: Migrate expired/deferred timers during cpu offline

2017-08-11 Thread Mike Galbraith
The below fixes the list debug explosion up. If we do not migrate expired/deferred timers during cpu offline, ->cb_entry will be corrupted by online initialization of base->expired, leading to a loud list debug complaint should someone call __remove_hrtimer() thereafter. Signed-off-by: Mike Galvr