On Mon, 2018-08-06 at 10:46 +0800, jianchao.wang wrote:
> On 08/04/2018 08:03 AM, Bart Van Assche wrote:
> > Instead of allowing requests that are not power management requests
> > to enter the queue in runtime suspended status (RPM_SUSPENDED), make
> > the blk_get_request() caller block. 
> 
> Who will resume the runtime suspended device ?
> 
> In original blk-legacy, when a request is added, blk_pm_add_request will call
> pm_request_resume could do that. The request will be issued after the q is 
> resumed
> successfully.
> 
> After this patch, non-pm request will be blocked and pm_request_resume is 
> removed from
> blk_pm_add_request, who does resume the runtime suspended q and device ?

Hello Jianchao,

I will double check that all contexts from which requests are submitted
protect submission of non-RQF_PM requests with scsi_autopm_get_device() /
scsi_autopm_put_device(). scsi_autopm_get_device() namely resumes suspended
devices.

Bart.

Reply via email to