Hi Lorenzo, Catalin,
On 14/01/18 12:20, Lorenzo Pieralisi wrote:
> On Sat, Jan 13, 2018 at 12:00:31PM +0000, Catalin Marinas wrote:
>> On Mon, Jan 08, 2018 at 05:22:26PM +0000, Lorenzo Pieralisi wrote:
>>> On Mon, Jan 08, 2018 at 03:38:13PM +0000, James Morse wrote:
>>>> +/* When entering idle, mask/unmask events for this cpu */
>>>
>>> Just mentioning (since I know you know), this notifier is called also
>>> through syscore_ops so it is not just idle (CPUidle or Suspend-to-idle).
>>>
>>>> +static int sdei_pm_notifier(struct notifier_block *nb, unsigned long action,
>>>> + void *data)
>>>> +{
>>>> + int rv;
>>>> +
>>>> + switch (action) {
>>>> + case CPU_PM_ENTER:
>>>> + rv = sdei_mask_local_cpu();
>>>> + break;
>>>> + case CPU_PM_EXIT:
>>>> + rv = sdei_unmask_local_cpu();
>>>
>>> You should handle CPU_PM_ENTER_FAILED here for correctness,
>>> in case the notifier chain fails.
(not seen this before, I mirrored what KVM does here ... which may have the same
bug)
>> So, just to confirm, the CPU_PM_ENTER_FAILED case goes together with
>> CPU_PM_EXIT so that we unmask the SDE again for the CPU.
>
> Yes, that's correct (I do not think that's a problem with current
> mainline but that ought to be handled correctly anyway).
I'll send a fix..,
... oh wait Catalin already did this.
Thanks!
James
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
