Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nvme: add dispersed-ns-participating-nss-log command #2691

Merged
merged 4 commits into from
Feb 10, 2025

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 <ikegami.t@gmail.com>
Added the nvme-dispersed-ns-participating-nss-log.txt file.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
Both bash and zsh completions updated for the command.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
Also fixed get-reg and set-reg commands completions.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
@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