Re: [PATCH] sched/cpuset/pm: Fix cpuset vs suspend-resume

2017-09-07 Thread Tejun Heo
Hello, On Thu, Sep 07, 2017 at 11:26:16AM +0200, Peter Zijlstra wrote: > TJ, I _think_ it was commit: > > deb7aa308ea2 ("cpuset: reorganize CPU / memory hotplug handling") Heh, that's a while ago. > That wrecked things, but there's been so much changes in this area it is > really hard to tell

Re: [PATCH] sched/cpuset/pm: Fix cpuset vs suspend-resume

2017-09-07 Thread Rafael J. Wysocki
On Thursday, September 7, 2017 11:26:16 AM CEST Peter Zijlstra wrote: > On Thu, Sep 07, 2017 at 11:13:38AM +0200, Peter Zijlstra wrote: > > Subject: sched/cpuset/pm: Fix cpuset vs suspend-resume > > > > Cpusets vs suspend-resume is _completely_ broken. And it got noticed > > because it now resulte

Re: [PATCH] sched/cpuset/pm: Fix cpuset vs suspend-resume

2017-09-07 Thread Peter Zijlstra
On Thu, Sep 07, 2017 at 11:13:38AM +0200, Peter Zijlstra wrote: > Subject: sched/cpuset/pm: Fix cpuset vs suspend-resume > > Cpusets vs suspend-resume is _completely_ broken. And it got noticed > because it now resulted in non-cpuset usage breaking too. > > On suspend cpuset_cpu_inactive() doesn'

[PATCH] sched/cpuset/pm: Fix cpuset vs suspend-resume

2017-09-07 Thread Peter Zijlstra
On Thu, Sep 07, 2017 at 08:15:29AM +0200, Mike Galbraith wrote: > > I'd be suspicious of these here commits: > > > > 77d1dfda0e79 ("sched/topology, cpuset: Avoid spurious/wrong domain > > rebuilds") > > Seems to be that one.  After suspend/resume, cpu7 (i4790+SMT) is online > with NULL domain