Page MenuHomeFreeBSD
Differential D26550

Revert most of r360179.
ClosedPublic
Actions

Authored by jhb on Sep 24 2020, 11:10 PM.
Tags
None
Referenced Files
F132981454: D26550.id77500.diff
Tue, Oct 21, 7:25 PM
F132978491: D26550.id77554.diff
Tue, Oct 21, 6:40 PM
F132977965: D26550.id.diff
Tue, Oct 21, 6:32 PM
F132928733: D26550.diff
Tue, Oct 21, 6:27 AM
Unknown Object (File)
Mon, Oct 20, 4:30 AM
Unknown Object (File)
Sat, Oct 18, 6:24 PM
Unknown Object (File)
Sat, Oct 18, 4:39 AM
Unknown Object (File)
Tue, Oct 14, 12:07 PM
View All 100 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 Not Applicable
Unit
Tests Not Applicable

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
head/
sys/
cam/
scsi/
25 lines

Diff 77554

View Options

head/sys/cam/scsi/scsi_sg.c

Loading...