Hello Michael, > Unfortunately, it's not quite right. The line of code in question > needs to be > > if ((vstor_packet->vm_srb.scsi_status & 0xFF) == SAM_STAT_CHECK_CONDITION && > The status_byte() helper was doing the masking as well as the right > shift, so the masking will need to be open coded. CHECK_CONDITION is obsolete so no shifting is required for the SAM status. And as far as I can tell vm_srb.scsi_status is a u8: struct vmscsi_request { u16 length; u8 srb_status; u8 scsi_status; [...] -- Martin K. Petersen Oracle Linux Engineering