Skip to content

Conversation

@ikegami-t
Copy link
Contributor

Since added the NVMe 2.1 log page.

@ikegami-t
Copy link
Contributor Author

The changes depended on the libnvme PR linux-nvme/libnvme#950 changes.

@keithbusch
Copy link
Contributor

does anyone actually implement this log? Linux doesn't support the feature, so even if a device does support it, I don't think you get consistent enumeration in Linux.

@hreinecke
Copy link
Collaborator

does anyone actually implement this log? Linux doesn't support the feature, so even if a device does support it, I don't think you get consistent enumeration in Linux.

I have been frequently told that this is the case; most of the major storage vendors (claim to) support it. Not that we've actually ever tested this, being as it is a quite contentious topic with Linux.

@ikegami-t
Copy link
Contributor Author

Should we not added the log command for now as not supported the feature by Linux?

@igaw
Copy link
Collaborator

igaw commented Feb 7, 2025

I don't see a problem by adding this command. There are many features which the Linux implementation doesn't support but nvme-cli is showing the output from the target.

@ikegami-t
Copy link
Contributor Author

Just rebased and added a documentation and completions.

@ikegami-t ikegami-t force-pushed the dns-psub-log branch 3 times, most recently from fe56948 to e606057 Compare February 9, 2025 14:24
Since added the NVMe 2.1 log page.

Signed-off-by: Tokunori Ikegami <[email protected]>
Added the nvme-dispersed-ns-participating-nss-log.txt file.

Signed-off-by: Tokunori Ikegami <[email protected]>
Both bash and zsh completions updated for the command.

Signed-off-by: Tokunori Ikegami <[email protected]>
Also fixed get-reg and set-reg commands completions.

Signed-off-by: Tokunori Ikegami <[email protected]>
@igaw igaw merged commit 741d2e5 into linux-nvme:master Feb 10, 2025
17 of 18 checks passed
@igaw
Copy link
Collaborator

igaw commented Feb 10, 2025

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants