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

Added NBFT table support #1

Closed
wants to merge 2 commits into from

Conversation

johnmeneghini
Copy link
Collaborator

Added support for parsing and printing the contents of the NBFT table (per NVMe-oF boot specification v1.0).

Signed-off-by: Stuart Hayes stuart_hayes@dell.com
Signed-off-by: John Meneghini jmeneghi@redhat.com
Signed-off-by: Martin Wilck mwilck@suse.com

The Timberland-sig project is using a copy
of the nvme-cli and libnvme repositories until release.

This change sets the libnvme submodule to timberland-sig
and updates the README.md file with work flow instructions.

The default/feature branch is now name timberlandmaster.
This applies to both nvme-cli and libnvme.

Add instructions how to override system libnvme

Update libnvme.wrap to use timberlandmaster

Note: when merging upstream, remove this sha.

Signed-off-by: John Meneghini <jmeneghi@redhat.com>
@mwilck
Copy link
Collaborator

mwilck commented Jan 18, 2023

Why is the target branch timberlandmaster_v2.2.1 ? Shouldn't it simply be master ?

@igaw
Copy link

igaw commented Feb 1, 2023

Please do an official PR against upstream. I'd like to get more of the upstream folks involved.

Added support for parsing and printing the contents
of the NBFT table (per NVMe-oF boot specification v1.0).

Signed-off-by: Stuart Hayes <stuart_hayes@dell.com>
Signed-off-by: John Meneghini <jmeneghi@redhat.com>
Signed-off-by: Martin Wilck <mwilck@suse.com>
Signed-off-by: Martin Belanger <martin.belanger@dell.com>
@johnmeneghini
Copy link
Collaborator Author

Closing this request. Please see the upstream request:

linux-nvme#1791

Note: I have updated this timberland sig repository and rebased this branch on to v2.3.

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.

None yet

4 participants