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 8, 4:57 AM
Unknown Object (File)
Wed, Apr 24, 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
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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.