bhyve nvme: Add missing Admin opcodes
ClosedPublic
Actions

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

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

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 43482
Build 40370: arc lint + arc unit

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 100298

View Options

bhyve nvme: Add missing Admin opcodesClosedPublicActions