Page MenuHomeFreeBSD

bhyve nvme: Fix Set Features
ClosedPublic

Authored by chuck on Dec 19 2021, 4:17 PM.
Tags
None
Referenced Files
F157835123: D33571.id100300.diff
Mon, May 25, 5:41 PM
Unknown Object (File)
Fri, May 22, 4:21 PM
Unknown Object (File)
Wed, May 20, 9:05 AM
Unknown Object (File)
Tue, May 19, 10:39 PM
Unknown Object (File)
Sun, May 17, 12:37 PM
Unknown Object (File)
Wed, May 13, 7:16 PM
Unknown Object (File)
Wed, May 13, 7:12 PM
Unknown Object (File)
Wed, May 13, 7:09 PM

Details

Summary

Be more conservative and only support the Features mandatory for an I/O
Controller.

Avoids a "hang" in UNH test 1.2.10 associated with Predictable Latency
Mode Configuration and Host Behavior Support features.

Tested by:
MFC after: 2 weeks

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 43484
Build 40372: arc lint + arc unit

Event Timeline

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

nvme_feature_obj has a cdw11 field to store values for set command. but some features are not only take one uint32_t input, but also other data types. such as timestamp, lba range type etc.
for further implements, can we have a void * here? let each feature decide what data structure it should be.

struct nvme_feature_obj {
	void * data;
	nvme_feature_cb	set;
	nvme_feature_cb	get;
	bool namespace_specific;
};
This revision was not accepted when it landed; it landed in state Needs Review.Jan 29 2022, 11:17 PM
This revision was automatically updated to reflect the committed changes.