From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 3 Aug 2018 08:11:43 +0800 From: Ming Lei To: Bart Van Assche Cc: "hch@lst.de" , "linux-block@vger.kernel.org" , "jthumshirn@suse.de" , "stern@rowland.harvard.edu" , "axboe@kernel.dk" , "jianchao.w.wang@oracle.com" Subject: Re: [PATCH v3 8/9] blk-mq: Insert blk_pm_{add,put}_request() calls Message-ID: <20180803001142.GD16047@ming.t460p> References: <20180802182944.14442-1-bart.vanassche@wdc.com> <20180802182944.14442-9-bart.vanassche@wdc.com> <20180802235338.GB16047@ming.t460p> <51cfe7ab9e54013e278072e91b59b04432488dc7.camel@wdc.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <51cfe7ab9e54013e278072e91b59b04432488dc7.camel@wdc.com> List-ID: On Fri, Aug 03, 2018 at 12:08:54AM +0000, Bart Van Assche wrote: > On Fri, 2018-08-03 at 07:53 +0800, Ming Lei wrote: > > blk_pm_add_request() calls pm_request_resume() for waking up device, but > > it is wrong because it is async request, which can't guarantee device > > will be ready before calling .queue_rq(). > > That's a longstanding issue and not something that has been introduced by this > patch series. Hence patch 6/9 that issues a warning if pm_request_resume() would > be called. It is introduced by your patch, please see blk_pm_allow_request(). Thanks, Ming