Page MenuHomeFreeBSD
Differential D27853

cam: Remove Giant handling from cam_sim_alloc()
ClosedPublic
Actions

Authored by markj on Dec 30 2020, 10:34 PM.
Tags
None
Referenced Files
F168081426: D27853.diff
Wed, Aug 26, 7:13 AM
Unknown Object (File)
Wed, Aug 19, 11:34 PM
Unknown Object (File)
Thu, Aug 13, 1:19 AM
Unknown Object (File)
Tue, Aug 11, 5:38 AM
Unknown Object (File)
Mon, Aug 10, 7:55 PM
Unknown Object (File)
Sat, Aug 8, 11:25 PM
Unknown Object (File)
Fri, Aug 7, 6:29 AM
Unknown Object (File)
Thu, Aug 6, 1:54 PM
View All Files
Subscribers
imp
scottl

Details

Reviewers
imp
scottl
Group Reviewers
cam
Commits
rG2ce5eef6e23a: cam: Remove Giant handling from cam_sim_alloc()
Summary

There are no non-MPSAFE SIM drivers left in the tree, verified with
coccinelle. I just noticed this while reviewing non-MPSAFE callouts.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 35804
Build 32693: arc lint + arc unit

Event Timeline

markj created this revision.Dec 30 2020, 10:34 PM
Herald added a reviewer: cam. · View Herald TranscriptDec 30 2020, 10:34 PM
Herald added a subscriber: imp. · View Herald Transcript
markj requested review of this revision.Dec 30 2020, 10:34 PM
Harbormaster completed remote builds in B35804: Diff 81388.Dec 30 2020, 10:34 PM
imp accepted this revision.Dec 30 2020, 11:15 PM
This revision is now accepted and ready to land.Dec 30 2020, 11:15 PM
scottl accepted this revision.Dec 31 2020, 9:32 PM
scottl added a subscriber: scottl.

I'm good with this, actually thought that Alexander had already done it. This should go into a release not in case there are people maintaining out-of-tree drivers.

Closed by commit rG2ce5eef6e23a: cam: Remove Giant handling from cam_sim_alloc() (authored by markj). · Explain WhyJan 3 2021, 4:51 PM
This revision was automatically updated to reflect the committed changes.
markj added a commit: rG2ce5eef6e23a: cam: Remove Giant handling from cam_sim_alloc().

Revision Contents
Changeset List

PathSize
sys/
cam/
1 line
12 lines

Diff 81388

View Options

sys/cam/cam_sim.h

Loading...
View Options

sys/cam/cam_sim.c

Loading...