Differential D30383

cam: remove sim callout
ClosedPublic
Actions

Authored by imp on May 21 2021, 3:22 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Dec 19, 1:39 PM

	Unknown Object (File)
	Fri, Dec 6, 6:06 AM

	Unknown Object (File)
	Oct 24 2024, 2:53 AM

	Unknown Object (File)
	Sep 12 2024, 12:37 PM

	Unknown Object (File)
	Sep 8 2024, 10:28 AM

	Unknown Object (File)
	Sep 8 2024, 3:37 AM

	Unknown Object (File)
	Sep 7 2024, 3:05 PM

	Unknown Object (File)
	Sep 2 2024, 9:50 AM

View All 46 Files

Subscribers

None

Details

Reviewers

scottl
ken
mav

Group Reviewers

Commits

rG28027f28e607: cam: remove sim callout

Summary

Nothing is using the sim callout to unfreeze the queue. Remove it to
simplify the SIM.

Sponsored by: Netflix

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 39339
Build 36228: arc lint + arc unit

Event Timeline

imp created this revision.May 21 2021, 3:22 PM

Herald added a reviewer: cam. · View Herald TranscriptMay 21 2021, 3:22 PM

imp requested review of this revision.May 21 2021, 3:22 PM

Harbormaster completed remote builds in B39339: Diff 89600.May 21 2021, 3:22 PM

imp added reviewers: scottl, ken, mav.May 21 2021, 3:24 PM

imp added inline comments.

sys/cam/cam_sim.h
104	I know flags is now unused after this change, but I'm about to use it for other reasons which is why I noticed this feature was unused.

imp added inline comments.May 21 2021, 3:47 PM

sys/cam/cam_sim.h
105	This was introduced by Justin in the initial CAM commit. It was used to freeze the queue on a bus reset. Now we just note the time of the reset. It's not been set in the CAM code since sometime before stable/3 (the oldest tree I have around). It looks like it was removed in 1999 when Justin re-did the bug reset logic. 6d039cdb75977fabc63ee268a96c990f08ea68ed has the MFC to stable/3, though I've not trolled to see where it changed on main yet given how old it is.

R10:87cfaf0e1fbd38cf7f9843ec462b5401e914f4d1 is the commit in main that removed setting this flag.

Also, the free code doesn't cancel the callout and there's no locking around it at all to keep multiple threads from frobbing it.

scottl accepted this revision.May 21 2021, 7:30 PM

This revision is now accepted and ready to land.May 21 2021, 7:30 PM

mav accepted this revision.May 21 2021, 8:32 PM

Closed by commit rG28027f28e607: cam: remove sim callout (authored by imp). · Explain WhyMay 25 2021, 3:20 PM

This revision was automatically updated to reflect the committed changes.

imp added a commit: rG28027f28e607: cam: remove sim callout.

Revision Contents
Changeset List

Path

Size

sys/

cam/

2 lines

1 line

9 lines

Diff 89600

sys/cam/cam_sim.h

Loading...

sys/cam/cam_sim.c

Loading...

sys/cam/cam_xpt.c

Loading...