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

	F102912657: D32952.diff
	Mon, Nov 18, 4:19 PM

	Unknown Object (File)
	Oct 19 2024, 5:31 AM

	Unknown Object (File)
	Oct 6 2024, 7:56 PM

	Unknown Object (File)
	Oct 3 2024, 4:59 AM

	Unknown Object (File)
	Oct 3 2024, 2:01 AM

	Unknown Object (File)
	Oct 2 2024, 7:17 PM

	Unknown Object (File)
	Oct 1 2024, 12:47 PM

	Unknown Object (File)
	Sep 28 2024, 4:58 PM

View All 59 Files

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...