Paths

Table of Contentst

cam: Implement ascq ranges correctly
AbandonedPublic
Actions

Authored by imp on Jan 25 2025, 8:10 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Oct 10, 2:59 PM

	Unknown Object (File)
	Thu, Oct 2, 3:33 AM

	Unknown Object (File)
	Thu, Sep 25, 7:56 PM

	Unknown Object (File)
	Thu, Sep 18, 8:28 PM

	Unknown Object (File)
	Sep 3 2025, 8:44 PM

	Unknown Object (File)
	Sep 3 2025, 10:32 AM

	Unknown Object (File)
	Sep 1 2025, 3:35 PM

	Unknown Object (File)
	Aug 28 2025, 6:34 AM

View All 35 Files

Subscribers

None

Details

Reviewers

mav
chs
ken

Group Reviewers

cam

Summary

Ensure that the range is between the first and second entries. At the
moment, all the SSQ_RANGE entries are tagged with 0xFF, no change is
expeccted.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 62029
Build 58913: arc lint + arc unit

Event Timeline

imp created this revision.Jan 25 2025, 8:10 PM

Herald added a reviewer: cam. · View Herald TranscriptJan 25 2025, 8:10 PM

imp requested review of this revision.Jan 25 2025, 8:10 PM

imp added a parent revision: D48685: cam: Fix off by one error in ASC/ASCQ lookup.

Harbormaster completed remote builds in B62029: Diff 149973.Jan 25 2025, 8:10 PM

imp added a child revision: D48687: cam/da: Document what an 'invalid pack' means..Jan 25 2025, 8:10 PM

This change is unneeded. ascq <= table_entry->ascq is already checked few lines above.

This revision now requires changes to proceed.Jan 27 2025, 4:08 PM

Bah! You're right. Punting on this one.

Revision Contents
Changeset List

Path

Size

sys/

cam/

scsi/

scsi_all.c

7 lines

Diff 149973

View Options

cam: Implement ascq ranges correctlyAbandonedPublicActions