Differential D32952

bhyve nvme: Add AEN support to NVMe emulation
ClosedPublic
Actions

Authored by chuck on Nov 12 2021, 12:43 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Tue, Oct 21, 7:07 AM

	Unknown Object (File)
	Sat, Oct 18, 4:57 AM

	Unknown Object (File)
	Sep 14 2025, 1:05 AM

	Unknown Object (File)
	Sep 12 2025, 2:19 PM

	Unknown Object (File)
	Sep 11 2025, 12:11 PM

	Unknown Object (File)
	Sep 11 2025, 11:29 AM

	Unknown Object (File)
	Sep 11 2025, 3:17 AM

	Unknown Object (File)
	Sep 10 2025, 10:33 AM

Subscribers

Details

Reviewers

imp
rpokala
jhb
grehan

Group Reviewers

Commits

rG9f1fa1a46117: bhyve nvme: Add AEN support to NVMe emulation

Summary

Add Asynchronous Event Notification infrastructure to the NVMe
emulation.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 42742
Build 39630: arc lint + arc unit

Event Timeline

chuck created this revision.Nov 12 2021, 12:43 AM

Herald added a reviewer: bhyve. · View Herald TranscriptNov 12 2021, 12:43 AM

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

chuck requested review of this revision.Nov 12 2021, 12:43 AM

Harbormaster completed remote builds in B42742: Diff 98388.Nov 12 2021, 12:43 AM

chuck added a child revision: D32953: bhyve nvme: Inform guests of namespace resize.Nov 12 2021, 12:44 AM

chuck added reviewers: imp, rpokala, jhb, grehan.Nov 12 2021, 12:46 AM

Maybe a note to fix the spelling later in the commit :)

This revision is now accepted and ready to land.Nov 12 2021, 9:47 PM

A few updates after passing the compliance tests

Enhanced debug logging
Enable all AENs by default
Wrap waking of AEN processing thread in helper function
Also wake AEN processing as part of some Set Features and Async Requests

This revision now requires review to proceed.Dec 10 2021, 8:04 PM

Generally, this looks good. I've not looked at every bit of it, though, with an eye towards pedantry.
And I don't know bhyve well, but the NVMe stuff looks generally good.

This revision is now accepted and ready to land.Dec 10 2021, 9:06 PM

Closed by commit rG9f1fa1a46117: bhyve nvme: Add AEN support to NVMe emulation (authored by chuck). · Explain WhyDec 14 2021, 11:31 PM

This revision was automatically updated to reflect the committed changes.

chuck added a commit: rG9f1fa1a46117: bhyve nvme: Add AEN support to NVMe emulation.

Revision Contents
Changeset List

Path

Size

usr.sbin/

bhyve/

248 lines

Diff 98388

usr.sbin/bhyve/pci_nvme.c

Loading...