On Thu, Jan 29, 2015 at 09:45:24AM +0100, Hannes Reinecke wrote: > Errm. > > We save the contents first: > > > struct scsi_device_handler *scsi_dh = scsi_dh_data->scsi_dh; > > Then set the pointer to NULL: > > > sdev->scsi_dh_data = NULL; > > and then call 'detach': > > > scsi_dh->detach(sdev); > > So scsi_dh is _not_ NULL, hence it shouldn't oops. The actual ->detach method can, and usually does access it. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html