Re: [PATCH 15/13] firmware: arm_sdei: move the frozen flag under the spinlock

2017-11-08 Thread James Morse
On 01/11/17 15:59, James Morse wrote: > dpm_suspend() calls the freeze/thaw callbacks for hibernate before > disable_non_bootcpus() takes down secondaries. > > This leads to a fun race where the freeze/thaw callbacks reset the > SDEI interface (as we may be restoring a kernel with a different > la

[PATCH 15/13] firmware: arm_sdei: move the frozen flag under the spinlock

2017-11-01 Thread James Morse
dpm_suspend() calls the freeze/thaw callbacks for hibernate before disable_non_bootcpus() takes down secondaries. This leads to a fun race where the freeze/thaw callbacks reset the SDEI interface (as we may be restoring a kernel with a different layout due to KASLR), then the cpu-hotplug callbacks