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
Unknown Object (File)
Sat, Jan 17, 9:07 AM
Unknown Object (File)
Sat, Jan 10, 3:39 AM
Unknown Object (File)
Wed, Jan 7, 9:06 PM
Unknown Object (File)
Tue, Jan 6, 12:37 PM
Unknown Object (File)
Mon, Jan 5, 7:19 AM
Unknown Object (File)
Sat, Jan 3, 10:10 AM
Unknown Object (File)
Wed, Dec 31, 5:13 PM
Unknown Object (File)
Tue, Dec 30, 1:03 AM
View All 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
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 36663
Build 33552: arc lint + arc unit

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

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 83250

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...