Differential D19579

Fix bhyve's NVMe Identify Namespace data
ClosedPublic
Actions

Authored by chuck on Mar 14 2019, 9:48 AM.

Tags

None

Referenced Files

	F142356494: D19579.id55048.diff
	Mon, Jan 19, 12:13 AM

	Unknown Object (File)
	Fri, Jan 9, 9:56 AM

	Unknown Object (File)
	Sat, Dec 27, 7:09 PM

	Unknown Object (File)
	Dec 11 2025, 10:40 AM

	Unknown Object (File)
	Dec 10 2025, 7:31 AM

	Unknown Object (File)
	Dec 4 2025, 7:03 AM

	Unknown Object (File)
	Dec 4 2025, 7:03 AM

	Unknown Object (File)
	Nov 27 2025, 9:34 AM

View All Files

Subscribers

imp

rgrimes

Details

Reviewers

imp
ken
araujo
jhb
rgrimes

Group Reviewers

bhyve

Commits

rS345170: Fix bhyve's NVMe Identify Namespace data

Summary

The NVMe Identify Namespace data structure's Number of LBA Formats
(NLBAF) field is a 0's based value (i.e. 0x0 means 1). Since the
emulation only supports a single format, set NLBAF to 0x0, not 1.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chuck created this revision.Mar 14 2019, 9:48 AM

Herald added a reviewer: bhyve. · View Herald TranscriptMar 14 2019, 9:48 AM

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

Harbormaster completed remote builds in B23079: Diff 55048.Mar 14 2019, 9:48 AM

style(9)

Harbormaster completed remote builds in B23080: Diff 55049.Mar 14 2019, 9:49 AM

chuck added reviewers: imp, ken, araujo.Mar 14 2019, 9:50 AM

rgrimes accepted this revision as: rgrimes.Mar 14 2019, 11:29 AM

rgrimes added a reviewer: jhb.

This revision is now accepted and ready to land.Mar 14 2019, 11:30 AM

imp accepted this revision.Mar 14 2019, 11:18 PM

LGTM!

Closed by commit rS345170: Fix bhyve's NVMe Identify Namespace data (authored by chuck). · Explain WhyMar 15 2019, 2:11 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

usr.sbin/

bhyve/

pci_nvme.c

2 lines

Diff 55086

View Options

head/usr.sbin/bhyve/pci_nvme.c

Fix bhyve's NVMe Identify Namespace dataClosedPublicActions