On Thu, 2019-08-22 at 02:28 +0200, Christoph Hellwig wrote: > On Wed, Aug 07, 2019 at 05:51:20PM +1000, Benjamin Herrenschmidt > wrote: > > +#define NVME_NVM_ADMSQES 6 > > #define NVME_NVM_IOSQES 6 > > #define NVME_NVM_IOCQES 4 > > The NVM in the two defines here stands for the NVM command set, > so this should just be named NVME_ADM_SQES or so. But except for > this > the patch looks good: > > Reviewed-by: Christoph Hellwig <h...@lst.de> > > So maybe Sagi can just fix this up in the tree.
Ah ok I missed the meaning. Thanks. Sagi, can you fix that up or do you need me to resubmit ? Cheers, Ben.