[PATCH V2 0/2] nvme: Support for fused NVME_IOCTL_SUBMIT_IO

Clay Mayers Clay.Mayers at kioxia.com
Tue Feb 9 10:53:38 EST 2021


> From: Christoph Hellwig <hch at lst.de>
> Sent: Monday, February 8, 2021 11:55 PM
> To: Clay Mayers <Clay.Mayers at kioxia.com>
> Cc: linux-nvme at lists.infradead.org; Keith Busch <kbusch at kernel.org>; Jens
> Axboe <axboe at fb.com>; Chaitanya Kulkarni
> <Chaitanya.Kulkarni at wdc.com>; Christoph Hellwig <hch at lst.de>; Sagi
> Grimberg <sagi at grimberg.me>
> Subject: Re: [PATCH V2 0/2] nvme: Support for fused
> NVME_IOCTL_SUBMIT_IO
> 
> On Tue, Feb 09, 2021 at 12:53:17AM +0000, Clay Mayers wrote:
> > > From: Clay Mayers
> > > Sent: Tuesday, January 26, 2021 1:14 PM
> > > To: linux-nvme at lists.infradead.org
> > > Cc: Keith Busch <kbusch at kernel.org>; Jens Axboe <axboe at fb.com>;
> > > 'Chaitanya Kulkarni' <Chaitanya.Kulkarni at wdc.com>; Christoph Hellwig
> > > <hch at lst.de>; Sagi Grimberg <sagi at grimberg.me>
> > > Subject: RE: [PATCH V2 0/2] nvme: Support for fused
> > > NVME_IOCTL_SUBMIT_IO
> > >
> > Is there any other feedback on V2?
> 
> My feedback is that we should not add code to the kernel driver and our fast
> path for fused command unless we have a mainstream in-kernel users, that
> is not at all for the foreseeable future.

Thanks - that makes sense.  I know the patch isn't enough to be very useful on
its own.  I shared it at this stage because of my concern over the implementation.
Thanks to the feedback from the list, it is better than it was.



More information about the Linux-nvme mailing list