Page MenuHomeFreeBSD
Differential D28454

cam: Don't permit crashdumps on non-pollable devices.
ClosedPublic
Actions

Authored by jhb on Feb 1 2021, 11:32 PM.
Tags
None
Referenced Files
F81573234: D28454.id.diff
Thu, Apr 18, 7:44 AM
Unknown Object (File)
Tue, Apr 9, 8:56 PM
Unknown Object (File)
Sun, Mar 31, 12:17 PM
Unknown Object (File)
Feb 29 2024, 6:02 PM
Unknown Object (File)
Feb 15 2024, 6:44 AM
Unknown Object (File)
Feb 13 2024, 9:10 AM
Unknown Object (File)
Feb 12 2024, 3:22 PM
Unknown Object (File)
Feb 1 2024, 1:07 AM
View All 32 Files
Subscribers
dab
cam

Details

Reviewers
mav
imp
scottl
Group Reviewers
cam
Commits
rGe07ac3f2fd73: cam: Don't permit crashdumps on non-pollable devices.
Summary

If a disk's SIM doesn't support polling, then it can't be used to
store crashdumps. Leave d_dump NULL in that case so that dumpon(8)
fails gracefully rather than having dumps fail at crash time.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jhb created this revision.Feb 1 2021, 11:32 PM
Herald added a reviewer: cam. · View Herald TranscriptFeb 1 2021, 11:32 PM
Herald added a subscriber: dab. · View Herald Transcript
Harbormaster completed remote builds in B36663: Diff 83250.Feb 1 2021, 11:32 PM
jhb requested review of this revision.Feb 1 2021, 11:32 PM
jhb added a parent revision: D28453: cam: Permit non-pollable sims..
jhb added a child revision: D28455: iscsi: Mark iSCSI CAM sims as non-pollable..
imp accepted this revision.Feb 1 2021, 11:35 PM

I'm happy with this, but all nvme SIMs support polling... But the change isn't wrong, we'll just never fail to take the if into setting d_dump.

sys/cam/nvme/nvme_da.c
901–902

All nvme sims support polling...

This revision is now accepted and ready to land.Feb 1 2021, 11:35 PM
mav accepted this revision.Feb 1 2021, 11:42 PM
scottl accepted this revision.Feb 2 2021, 4:39 AM
Closed by commit rGe07ac3f2fd73: cam: Don't permit crashdumps on non-pollable devices. (authored by jhb). · Explain WhyFeb 11 2021, 9:53 PM
This revision was automatically updated to reflect the committed changes.
jhb added a commit: rGe07ac3f2fd73: cam: Don't permit crashdumps on non-pollable devices..

Revision Contents
Changeset List

PathSize
sys/
cam/
ata/
3 lines
nvme/
3 lines
scsi/
5 lines

Diff 83712

View Options

sys/cam/ata/ata_da.c

Loading...
View Options

sys/cam/nvme/nvme_da.c

Loading...
View Options

sys/cam/scsi/scsi_da.c

Loading...