]> git.karo-electronics.de Git - linux-beck.git/commit
sd: disable discard_zeroes_data for UNMAP
authorMartin K. Petersen <martin.petersen@oracle.com>
Fri, 7 Nov 2014 05:08:13 +0000 (00:08 -0500)
committerChristoph Hellwig <hch@lst.de>
Wed, 12 Nov 2014 10:19:14 +0000 (11:19 +0100)
commit7985090aa0201fa7760583f9f8e6ba41a8d4c392
treebb1c70e1cd918f56068482fcc8fca298e1c7cbda
parente9afccc5245a35468f52bc3f53ed162caabf275d
sd: disable discard_zeroes_data for UNMAP

The T10 SBC UNMAP command does not provide any hard guarantees that
blocks will return zeroes on a subsequent READ. This is due to the fact
that the device server is free to silently ignore all or parts of the
request.

The only way to ensure that a block consistently returns zeroes after
being unmapped is to use WRITE SAME with the UNMAP bit set. Should the
device be unable to unmap one or more blocks described by the command it
is required to manually write zeroes to them.

Until now we have preferred UNMAP over the WRITE SAME variants to
accommodate thinly provisioned devices that predated the final SBC-3
spec. This patch changes the heuristic so that we favor WRITE SAME(16)
or (10) over UNMAP if these commands are marked as supported in the
Logical Block Provisioning VPD page.

The patch also disables discard_zeroes_data for devices operating in
UNMAP mode.

Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
drivers/scsi/sd.c