Re: [PATCH 1/2] nvme: make independent ns identify default

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09-10-2024 09:46, Christoph Hellwig wrote:
On Tue, Oct 08, 2024 at 04:55:02PM +0200, Matias Bjørling wrote:
However, the independent namespace data structure
is mandatory for devices that implement features from the 2.0+
specification. Therefore, we can check this data structure first. If
unavailable, retrieve the generic attributes from the NVM command set
identify namespace data structure.

I'm not a huge fan of this.  For pre-2.0 controllers this means
we'll now send a command that will fail most of them time.  And for
all the cheap low-end consumer device I'm actually worried that they'll
get it wrong and break something.


It's a good point. Damien, Keith, and I were debating it during ALPSS. They preferred the "send command and see if it fails" approach over writing specific conditions where it would apply. Note that Keith did suggest to avoid the command on 1.0 and 1.1 devices, and they were known to fail with unsupported CNS ids.

If making the check conditional, I think checking if the device follows 2.0 specification isn't sufficient, as some devices may implement a subset of the 2.0 features (for example the independent ns data struct), while reporting as a 1.4 device.

Is there maybe better way, that isn't dependent on some feature being implemented (such as CRIMS capability)?








[Index of Archives]     [Linux RAID]     [Linux SCSI]     [Linux ATA RAID]     [IDE]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Device Mapper]

  Powered by Linux