Page MenuHomeFreeBSD
Differential D26550

Revert most of r360179.
ClosedPublic
Actions

Authored by jhb on Sep 24 2020, 11:10 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 25, 2:36 PM
Unknown Object (File)
Nov 20 2024, 10:48 PM
Unknown Object (File)
Nov 11 2024, 12:12 AM
Unknown Object (File)
Oct 27 2024, 11:29 AM
Unknown Object (File)
Oct 10 2024, 6:03 PM
Unknown Object (File)
Oct 7 2024, 2:55 PM
Unknown Object (File)
Oct 3 2024, 2:01 AM
Unknown Object (File)
Sep 30 2024, 1:45 AM
View All 57 Files
Subscribers
imp

Details

Reviewers
scottl
Group Reviewers
cam
Commits
rS366175: Revert most of r360179.
Summary

I had failed to notice that sgsendccb() was using cam_periph_mapmem()
and thus was not passing down user pointers directly to drivers. In
practice this broke requests submitted from userland.

PR: 249395
Tested by: Trenton Schulz <trueos@norwegianrockcat.com>

Test Plan
  • submitter tested this patch, more details in the bug

Diff Detail

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

Event Timeline

jhb created this revision.Sep 24 2020, 11:10 PM
Herald added a reviewer: cam. · View Herald TranscriptSep 24 2020, 11:10 PM
Herald added a subscriber: imp. · View Herald Transcript
jhb requested review of this revision.Sep 24 2020, 11:10 PM
Harbormaster completed remote builds in B33805: Diff 77500.Sep 24 2020, 11:10 PM
scottl accepted this revision.Sep 25 2020, 5:28 AM
This revision is now accepted and ready to land.Sep 25 2020, 5:28 AM
Closed by commit rS366175: Revert most of r360179. (authored by jhb). · Explain WhySep 25 2020, 9:20 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rS366175: Revert most of r360179..

Revision Contents
Changeset List

PathSize
sys/
cam/
scsi/
25 lines

Diff 77500

View Options

sys/cam/scsi/scsi_sg.c

Loading...