Page MenuHomeFreeBSD

bhyve nvme: Support minimal Controller list
ClosedPublic

Authored by chuck on Aug 13 2022, 3:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, May 22, 12:22 PM
Unknown Object (File)
Wed, May 8, 4:57 AM
Unknown Object (File)
Apr 24 2024, 1:09 AM
Unknown Object (File)
Apr 7 2024, 4:23 PM
Unknown Object (File)
Apr 6 2024, 6:14 PM
Unknown Object (File)
Mar 12 2024, 4:55 PM
Unknown Object (File)
Mar 10 2024, 6:06 PM
Unknown Object (File)
Mar 10 2024, 6:05 PM
Subscribers

Details

Summary

Controllers must support the Identify Controller list if they support
Namespace Management. But the UNH NVMe tests use this command regardless
of whether the device under test supports Namespace Management.

This implementation returns an empty Controller list (i.e., Number of
Identifiers is zero).

Fixes UNH Test 1.1.2

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 46955
Build 43844: arc lint + arc unit

Event Timeline

chuck requested review of this revision.Aug 13 2022, 3:39 PM
This revision is now accepted and ready to land.Aug 15 2022, 6:52 PM
usr.sbin/bhyve/pci_nvme.c
1579

For new code I prefer memset since it is POSIX unlike bzero.

This revision now requires review to proceed.Aug 16 2022, 3:34 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 16 2022, 4:17 PM
This revision was automatically updated to reflect the committed changes.