Do not iterate through device list doing 'sync'
if NOSYNC reboot method is explicitly chosen.
Calling sync may cause additional panic on dodump if the panic
was first originated from USB/SCSI layer.
Differential D31549
Don't sync DAs if reboot type is RB_NOSYNC wma on Aug 16 2021, 5:38 AM. Authored by Tags None Referenced Files
Subscribers None
Details Do not iterate through device list doing 'sync' if NOSYNC reboot method is explicitly chosen. Calling sync may cause additional panic on dodump if the panic
Diff Detail
Event TimelineComment Actions The description is wrong, or at least ambiguous. This has nothing to do with sync(2), but instead avoids sending any SCSI commands to the drive, specifically, the SCSI SYNCHRONIZE CACHE command for the open devices. Since there's two different kinds of sync here, the ambiguity is confusing. I'd recommend: cam: don't send scsi commands on shutdown when reboot method RB_NOSYNC Don't send the SCSI comand SYNCHRONIZE CACHE on devices that are still open when RB_NOSYNC is the reboot method. This may avoid recursive panics when doadump is called due to a SCSI/CAM/USB error/bug. Though it would be better to know which of the devices would cause the Comment Actions Thanks for comment, yes, the description might have been misleading. @all Any objections to commit this patch? |