ufshci: report the highest LUN number in the path inquiry
ClosedPublic
Actions

Authored by jaeyoon on Thu, Aug 13, 8:43 AM.

Tags

None

Referenced Files

	F169156603: D58820.diff
	Mon, Aug 31, 3:09 PM

	F168994402: D58820.diff
	Mon, Aug 31, 2:17 AM

	Unknown Object (File)
	Sun, Aug 30, 2:49 AM

	Unknown Object (File)
	Sat, Aug 29, 5:14 AM

	Unknown Object (File)
	Fri, Aug 28, 7:17 AM

	Unknown Object (File)
	Thu, Aug 27, 7:15 PM

	Unknown Object (File)
	Sun, Aug 23, 6:14 AM

	Unknown Object (File)
	Sat, Aug 22, 8:54 AM

View All 27 Files

Subscribers

None

Details

Reviewers

imp

Commits

rG68091ebf0a72: ufshci: report the highest LUN number in the path inquiry

Summary

cpi->max_lun is an inclusive upper bound, but the driver reported the
LUN count (8 or 32), so CAM probed one nonexistent LUN past the end.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 75556
Build 72439: arc lint + arc unit

Event Timeline

jaeyoon requested review of this revision.Thu, Aug 13, 8:43 AM

jaeyoon created this revision.

Harbormaster completed remote builds in B75556: Diff 183973.Thu, Aug 13, 8:43 AM

jaeyoon added a parent revision: D58819: ufshci: correct the crypto/config register offsets and HCMID fields.Thu, Aug 13, 8:44 AM

imp accepted this revision.Fri, Aug 14, 9:10 PM

This revision is now accepted and ready to land.Fri, Aug 14, 9:10 PM

Closed by commit rG68091ebf0a72: ufshci: report the highest LUN number in the path inquiry (authored by jaeyoon). · Explain WhyTue, Aug 18, 5:26 AM

This revision was automatically updated to reflect the committed changes.

jaeyoon added a commit: rG68091ebf0a72: ufshci: report the highest LUN number in the path inquiry.

Revision Contents
Changeset List

Path

Size

sys/

dev/

ufshci/

ufshci_sim.c

2 lines

Diff 183973

View Options

sys/dev/ufshci/ufshci_sim.c

ufshci: report the highest LUN number in the path inquiryClosedPublicActions