From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 19 Oct 2017 15:54:48 +0200 From: Christoph Hellwig To: Sagi Grimberg Cc: Christoph Hellwig , Jens Axboe , Keith Busch , Hannes Reinecke , Johannes Thumshirn , linux-nvme@lists.infradead.org, linux-block@vger.kernel.org Subject: Re: [PATCH 10/17] nvme: switch controller refcounting to use struct device Message-ID: <20171019135448.GE21452@lst.de> References: <20171018165258.23212-1-hch@lst.de> <20171018165258.23212-11-hch@lst.de> <20171019072053.GA12168@lst.de> <05edcff6-4c56-39d9-83c8-e3eae03b824d@grimberg.me> <20171019073741.GA12348@lst.de> <827bd4b9-4228-32e4-e8dd-17679f2dfeba@grimberg.me> <20171019101855.GA17326@lst.de> <58a30ee0-f268-2d92-eccd-9d241402a460@grimberg.me> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <58a30ee0-f268-2d92-eccd-9d241402a460@grimberg.me> List-ID: On Thu, Oct 19, 2017 at 01:33:55PM +0300, Sagi Grimberg wrote: >> Well, the ctrl device integration is what enables us to do this. >> Before that the ctrl refcount could have reached null by the time >> we call the delete_ctrl method. > > OK, maybe a change log mention then? I thought I did that, but I can be more detailed. From mboxrd@z Thu Jan 1 00:00:00 1970 From: hch@lst.de (Christoph Hellwig) Date: Thu, 19 Oct 2017 15:54:48 +0200 Subject: [PATCH 10/17] nvme: switch controller refcounting to use struct device In-Reply-To: <58a30ee0-f268-2d92-eccd-9d241402a460@grimberg.me> References: <20171018165258.23212-1-hch@lst.de> <20171018165258.23212-11-hch@lst.de> <20171019072053.GA12168@lst.de> <05edcff6-4c56-39d9-83c8-e3eae03b824d@grimberg.me> <20171019073741.GA12348@lst.de> <827bd4b9-4228-32e4-e8dd-17679f2dfeba@grimberg.me> <20171019101855.GA17326@lst.de> <58a30ee0-f268-2d92-eccd-9d241402a460@grimberg.me> Message-ID: <20171019135448.GE21452@lst.de> On Thu, Oct 19, 2017@01:33:55PM +0300, Sagi Grimberg wrote: >> Well, the ctrl device integration is what enables us to do this. >> Before that the ctrl refcount could have reached null by the time >> we call the delete_ctrl method. > > OK, maybe a change log mention then? I thought I did that, but I can be more detailed.