On Tue, 2012-11-06 at 17:48 -0500, Martin K. Petersen wrote: > >>>>> "Ewan" == Ewan D Milne <emilne@xxxxxxxxxx> writes: > > Ewan> sd_prep_fn will allocate a larger CDB for the command via > Ewan> mempool_alloc for devices using DIF type 2 protection. This CDB > Ewan> was being freed in sd_done, which results in a kernel crash if the > Ewan> command is retried due to a UNIT ATTENTION. This change moves the > Ewan> code to free the larger CDB into sd_unprep_fn instead, which is > Ewan> invoked after the request is complete. > > The 32-byte CDB support predates unprep. This fix is obviously the > correct approach. > > Acked-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx> > I don't see that this change ever made it in. Can this go in for the 3.12 merge window? http://marc.info/?l=linux-scsi&m=135186352200668&w=2 -Ewan -- 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
