> > @@ -1154,6 +1154,7 @@ static void storvsc_on_io_completion(struct
> > storvsc_device *stor_device,
> >
> > if ((stor_pkt->vm_srb.cdb[0] == INQUIRY) ||
> > (stor_pkt->vm_srb.cdb[0] == MODE_SENSE) ||
> > + (stor_pkt->vm_srb.cdb[0] == MODE_SENSE_10) ||
> > (stor_pkt->vm_srb.cdb[0] == MAINTENANCE_IN &&
> > hv_dev_is_fc(device))) {
> > vstor_packet->vm_srb.scsi_status = 0;
>
> There's a code comment above this "if" statement that describes the situation.
> The comment specifically lists INQUIRY, MODE_SENSE, and
> MAINTENANCE_IN. For consistency, it should be updated to include
> MODE_SENSE_10.
>
> With the comment updated,
>
> Reviewed-by: Michael Kelley <[email protected]>
Will send v2, thank you.
Long
