Normally we'd just pass the scsi_sense_hdr structure in from thecaler if we care about sense data. Is this something you considered?Otherwise this looks fine to me.