[PATCH] nvme: implement non-mdts command limits

Klaus Jensen its at irrelevant.dk
Thu Mar 11 15:59:12 GMT 2021


On Mar 12 00:49, Keith Busch wrote:
> On Thu, Mar 11, 2021 at 06:51:26AM +0100, Klaus Jensen wrote:
> > On Mar 10 11:57, Keith Busch wrote:
> > > +	if (id->dmrsl)
> > > +		ctrl->max_discard_sectors = le32_to_cpu(id->dmrsl);
> > 
> > Since DMRSL is in terms of LBAs, should this use nvme_lba_to_sect?
> 
> We need a namespace for that, but we only have a controller for this
> Identify.

Ah yes.

I remember now that I also complained loudly about this mistake in the
spec when we implemented it in QEMU.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-nvme/attachments/20210311/68a294f5/attachment.sig>


More information about the Linux-nvme mailing list