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

	Unknown Object (File)
	Sat, Nov 30, 3:26 AM

	Unknown Object (File)
	Nov 23 2024, 2:09 AM

	Unknown Object (File)
	Nov 22 2024, 11:28 PM

	Unknown Object (File)
	Nov 20 2024, 11:13 AM

	Unknown Object (File)
	Nov 17 2024, 1:52 PM

	Unknown Object (File)
	Nov 13 2024, 7:17 PM

	Unknown Object (File)
	Nov 12 2024, 5:50 PM

	Unknown Object (File)
	Nov 11 2024, 2:59 PM

View All 66 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