bhyve nvme: Add missing Admin opcodes
ClosedPublic
Actions

Authored by chuck on Dec 19 2021, 4:16 PM.

Tags

None

Referenced Files

	F149838961: D33569.diff
	Fri, Mar 27, 12:09 PM

	Unknown Object (File)
	Wed, Mar 25, 8:56 AM

	Unknown Object (File)
	Fri, Mar 20, 3:29 AM

	Unknown Object (File)
	Thu, Mar 19, 5:32 PM

	Unknown Object (File)
	Wed, Mar 18, 11:23 PM

	Unknown Object (File)
	Thu, Mar 12, 8:10 AM

	Unknown Object (File)
	Thu, Mar 12, 8:04 AM

	Unknown Object (File)
	Thu, Mar 5, 4:07 PM

Subscribers

Details

Reviewers

grehan
jhb
imp
rpokala
allanjude

Group Reviewers

bhyve

Commits

rG785b5da318bf: bhyve nvme: Add missing Admin opcodes

Summary

Don't treat unsupported Admin commands as Invalid Opcode. Instead return
the proper Invalid Field in Command.

Fixes UNH IOL test 1.17.2

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chuck created this revision.Dec 19 2021, 4:16 PM

Herald added a reviewer: bhyve. · View Herald TranscriptDec 19 2021, 4:16 PM

Herald added subscribers: dab, bcran, rgrimes. · View Herald Transcript

chuck requested review of this revision.Dec 19 2021, 4:16 PM

Harbormaster completed remote builds in B43482: Diff 100298.Dec 19 2021, 4:16 PM

chuck added a parent revision: D33568: bhyve nvme: Implement Log Page Offset.Dec 19 2021, 4:16 PM

chuck added a child revision: D33570: bhyve nvme: Remove redundant AER Limit checks.

Reviewed By: allanjude

This revision is now accepted and ready to land.Dec 23 2021, 5:16 PM

imp accepted this revision.Dec 23 2021, 7:24 PM

Closed by commit rG785b5da318bf: bhyve nvme: Add missing Admin opcodes (authored by chuck). · Explain WhyJan 29 2022, 11:17 PM

This revision was automatically updated to reflect the committed changes.

chuck added a commit: rG785b5da318bf: bhyve nvme: Add missing Admin opcodes.

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

pci_nvme.c

7 lines

Diff 102092

View Options

bhyve nvme: Add missing Admin opcodesClosedPublicActions