Re: [PATCH v2 5/5] hw/block/nvme: report non-mdts command size limit for dsm

2021-03-01 Thread Keith Busch
On Mon, Mar 01, 2021 at 12:15:26PM +0100, Klaus Jensen wrote: > On Feb 22 22:12, Klaus Jensen wrote: > > On Feb 23 05:55, Keith Busch wrote: > > > On Mon, Feb 22, 2021 at 07:47:59PM +0100, Klaus Jensen wrote: > > > > +typedef struct NvmeIdCtrlNvm { > > > > +uint8_t vsl; > > > > +uint8_t

Re: [PATCH v2 5/5] hw/block/nvme: report non-mdts command size limit for dsm

2021-03-01 Thread Klaus Jensen
On Feb 22 22:12, Klaus Jensen wrote: > On Feb 23 05:55, Keith Busch wrote: > > On Mon, Feb 22, 2021 at 07:47:59PM +0100, Klaus Jensen wrote: > > > +typedef struct NvmeIdCtrlNvm { > > > +uint8_t vsl; > > > +uint8_t wzsl; > > > +uint8_t wusl; > > > +uint8_t dmrl; > > >

Re: [PATCH v2 5/5] hw/block/nvme: report non-mdts command size limit for dsm

2021-02-22 Thread Klaus Jensen
On Feb 23 05:55, Keith Busch wrote: > On Mon, Feb 22, 2021 at 07:47:59PM +0100, Klaus Jensen wrote: > > +typedef struct NvmeIdCtrlNvm { > > +uint8_t vsl; > > +uint8_t wzsl; > > +uint8_t wusl; > > +uint8_t dmrl; > > +uint32_tdmrsl; > > +uint64_tdmsl; >

Re: [PATCH v2 5/5] hw/block/nvme: report non-mdts command size limit for dsm

2021-02-22 Thread Keith Busch
On Mon, Feb 22, 2021 at 07:47:59PM +0100, Klaus Jensen wrote: > +typedef struct NvmeIdCtrlNvm { > +uint8_t vsl; > +uint8_t wzsl; > +uint8_t wusl; > +uint8_t dmrl; > +uint32_tdmrsl; > +uint64_tdmsl; > +uint8_t rsvd16[4080]; > +} NvmeIdCtrlNvm;

[PATCH v2 5/5] hw/block/nvme: report non-mdts command size limit for dsm

2021-02-22 Thread Klaus Jensen
From: Gollu Appalanaidu Dataset Management is not subject to MDTS, but exceeded a certain size per range causes internal looping. Report this limit (DMRSL) in the NVM command set specific identify controller data structure. Signed-off-by: Gollu Appalanaidu Signed-off-by: Klaus Jensen --- hw/b