Skip to content

Conversation

@ikegami-t
Copy link
Contributor

Added for NVMe 2.3 commands support.

@ikegami-t ikegami-t force-pushed the nvme-2.3-def branch 3 times, most recently from acd7603 to f691b99 Compare December 30, 2025 12:11
@igaw
Copy link
Collaborator

igaw commented Jan 2, 2026

Please update the formatting, the rest looks good.

Added for NVMe 2.3 commands support.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
Supported by NVMe 2.3.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
Since the status code is duplicated with the fw-commit status code.

Signed-off-by: Tokunori Ikegami <ikegami.t@gmail.com>
* or a standard errno string if status is < 0.
*/
static inline const char *
nvme_opcode_status_to_string(int status, bool admin, __u8 opcode)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ideally, we would have only one function which handles the negative error code and the postive status code. Though this should be handled in nvme-experiments#98

So let's go with this for now. It will be addressed later :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see this. Thank you.

@igaw igaw merged commit 26d1137 into linux-nvme:master Jan 5, 2026
19 of 20 checks passed
@igaw
Copy link
Collaborator

igaw commented Jan 5, 2026

Thanks a lot. Having the library in the same repo seems to be a bliss :)

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.

2 participants