linux-kernel - Re: [PATCH AUTOSEL 5.7 229/388] usb/ohci-platform: Fix a warning when hibernating

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Sun, 21 Jun 2020 19:46:21 -0400
From:   Sasha Levin <sashal@...nel.org>
To:     Qais Yousef <qais.yousef@....com>
Cc:     linux-kernel@...r.kernel.org, stable@...r.kernel.org,
        Alan Stern <stern@...land.harvard.edu>,
        Tony Prisk <linux@...sktech.co.nz>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Mathias Nyman <mathias.nyman@...el.com>,
        Oliver Neukum <oneukum@...e.de>,
        linux-arm-kernel@...ts.infradead.org, linux-usb@...r.kernel.org
Subject: Re: [PATCH AUTOSEL 5.7 229/388] usb/ohci-platform: Fix a warning
 when hibernating

On Thu, Jun 18, 2020 at 10:56:01AM +0100, Qais Yousef wrote:
>Hi Sasha/Alan
>
>On 06/17/20 21:05, Sasha Levin wrote:
>
>[...]
>
>> diff --git a/drivers/usb/host/ohci-platform.c b/drivers/usb/host/ohci-platform.c
>> index 7addfc2cbadc..4a8456f12a73 100644
>> --- a/drivers/usb/host/ohci-platform.c
>> +++ b/drivers/usb/host/ohci-platform.c
>> @@ -299,6 +299,11 @@ static int ohci_platform_resume(struct device *dev)
>>  	}
>>
>>  	ohci_resume(hcd, false);
>> +
>> +	pm_runtime_disable(dev);
>> +	pm_runtime_set_active(dev);
>> +	pm_runtime_enable(dev);
>> +
>
>I am not sure what's the protocol here, but the series was fixing similar
>problems to this driver but I didn't myself directly observe a warning because
>of that.
>
>https://lore.kernel.org/lkml/20200518154931.6144-2-qais.yousef@arm.com/
>https://lore.kernel.org/lkml/20200518154931.6144-3-qais.yousef@arm.com/
>
>So just pointing it out in case they're worth backporting to stable too.

I'll grab them too, thanks!

-- 
Thanks,
Sasha