Differential D53920

cam/scsi: Support well known logical unit
ClosedPublic
Actions

Authored by jaeyoon on Nov 26 2025, 9:38 AM.

Tags

None

Referenced Files

	F167228897: D53920.id167116.diff
	Thu, Aug 20, 3:44 AM

	Unknown Object (File)
	Wed, Aug 19, 7:20 AM

	Unknown Object (File)
	Mon, Aug 17, 9:07 AM

	Unknown Object (File)
	Sun, Aug 16, 8:57 AM

	Unknown Object (File)
	Thu, Aug 13, 1:01 AM

	Unknown Object (File)
	Mon, Aug 10, 12:27 PM

	Unknown Object (File)
	Sun, Aug 9, 7:53 AM

	Unknown Object (File)
	Sat, Aug 8, 7:49 PM

View All 73 Files

Subscribers

Details

Reviewers

Group Reviewers

Commits

rGba963776199f: cam/scsi: Support well known logical unit

Summary

This patch adds an additional state to probe well-known logical units
before probing normal logical units.

Sponsored by: Samsung Electronics

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 68861
Build 65744: arc lint + arc unit

Event Timeline

jaeyoon created this revision.Nov 26 2025, 9:38 AM

Herald added a reviewer: cam. · View Herald TranscriptNov 26 2025, 9:38 AM

Herald added a subscriber: imp. · View Herald Transcript

jaeyoon requested review of this revision.Nov 26 2025, 9:38 AM

Harbormaster completed remote builds in B68861: Diff 167116.Nov 26 2025, 9:38 AM

jaeyoon mentioned this in D53375: cam/scsi, ufshci: Add Well-known LUN Probing.Nov 26 2025, 9:50 AM

jaeyoon added a reviewer: imp.Nov 26 2025, 9:53 AM

Since only the SIMs that support WLUN reporting call this, that limits the damage.
It may be the case we have to probe LUNs first and not probe WLUNs at all if that fails (some older, reduced command CDB sets don't have REPORT LUNS at all). For the moment, it's fine.

sys/cam/scsi/scsi_all.h
3060	are there other formats we don't support?

This revision is now accepted and ready to land.Nov 28 2025, 7:16 AM

jaeyoon added inline comments.Dec 1 2025, 6:15 AM

sys/cam/scsi/scsi_all.h
3060	There are several formats for the LUN address method (Peripheral, Flat, Extended flat, Long). And we don't support them.

Closed by commit rGba963776199f: cam/scsi: Support well known logical unit (authored by jaeyoon). · Explain WhyDec 1 2025, 6:23 AM

This revision was automatically updated to reflect the committed changes.

jaeyoon added a commit: rGba963776199f: cam/scsi: Support well known logical unit.

Revision Contents
Changeset List

Path

Size

sys/

cam/

1 line

1 line

cam_xpt_internal.h

1 line

scsi/

2 lines

239 lines

Diff 167116

sys/cam/cam_ccb.h

Loading...

sys/cam/cam_xpt.c

Loading...

sys/cam/cam_xpt_internal.h

Loading...

sys/cam/scsi/scsi_all.h

Loading...

sys/cam/scsi/scsi_xpt.c

Loading...