From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com ([209.132.183.28]:40628 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752550AbdLLPK7 (ORCPT ); Tue, 12 Dec 2017 10:10:59 -0500 Date: Tue, 12 Dec 2017 23:10:33 +0800 From: Ming Lei To: Christoph Hellwig Cc: linux-nvme@lists.infradead.org, Jens Axboe , linux-block@vger.kernel.org, Bart Van Assche , Keith Busch , Sagi Grimberg , Yi Zhang , Johannes Thumshirn Subject: Re: [PATCH 6/6] nvme: remove .init_request callback Message-ID: <20171212151032.GA15078@ming.t460p> References: <20171212110232.12495-1-ming.lei@redhat.com> <20171212110232.12495-7-ming.lei@redhat.com> <20171212141358.GB7849@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20171212141358.GB7849@lst.de> Sender: linux-block-owner@vger.kernel.org List-Id: linux-block@vger.kernel.org On Tue, Dec 12, 2017 at 03:13:58PM +0100, Christoph Hellwig wrote: > On Tue, Dec 12, 2017 at 07:02:32PM +0800, Ming Lei wrote: > > It may cause race by setting 'nvmeq' in nvme_init_request() > > because .init_request is called inside switching io scheduler, which > > may happen when the NVMe device is being resetted and its nvme queues > > are being freed and created. We don't have any sync between the two > > pathes. > > > > This patch removes the .init_request callback and sets the nvmeq runtime, > > and fixes the following bug: > > If ->init_request doesn't work for NVMe it won't work for any other > driver either, so we need to remove it entirely if we can't fix it. > > I'd much rather try to fix it first, though. Just checked all this uses, and it is a NVMe specific issue, because only NVMe's .init_request() associates reference of one variable to the request, and the variable itself may change during the request's lifetime. And all the following .init_request() have this issue: nvme_fc_init_request() nvme_rdma_init_request() nvme_loop_init_request() Thanks, Ming