scsi: target: transport should handle st FM/EOM/ILI reads
authorLee Duncan <lduncan@suse.com>
Wed, 16 May 2018 01:25:24 +0000 (18:25 -0700)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 18 May 2018 16:22:48 +0000 (12:22 -0400)
commitbd81372065fa467e262150c70be885f47f9535df
treedfa782e8a1a792f76ca14dffb4b224c6174abebd
parent51b910c3c70986a5a0a84eea11cb8e904e37ba8b
scsi: target: transport should handle st FM/EOM/ILI reads

When a tape drive is exported via LIO using the pscsi module, a read
that requests more bytes per block than the tape can supply returns an
empty buffer. This is because the pscsi pass-through target module sees
the "ILI" illegal length bit set and thinks there is no reason to return
the data.

This is a long-standing transport issue, since it assumes that no data
need be returned under a check condition, which isn't always the case
for tape.

Add in a check for tape reads with the ILI, EOM, or FM bits set, with a
sense code of NO_SENSE, treating such cases as if the read
succeeded. The layered tape driver then "does the right thing" when it
gets such a response.

Signed-off-by: Bodo Stroesser <bstroesser@ts.fujitsu.com>
Signed-off-by: Lee Duncan <lduncan@suse.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/target/target_core_pscsi.c
drivers/target/target_core_transport.c
include/target/target_core_base.h