On 12-03-07 03:09 PM, Eric Sandeen wrote:
Add LBPRZ support to scsi_debug; i.e. return zero for
unmapped blocks.
Rather than checking for unmapped blocks at
read time, this just zeroes them on the backing store
at unmap time so it behaves the same way.
This also adds a module parameter to disable it, since
some SSDs have this behavior.
unmap_zeroes, "unmapped blocks return 0 on read (def=1)"
Signed-off-by: Eric Sandeen<sandeen@xxxxxxxxxx>
---
Note: This was sent long ago as "TPRZ" support, but lost, I guess.
Note2: dgilbert preferred "zeros" to "zeroes" at the time,
but since we have "discard_zeroes_data" in sysfs it seems like
we should be consistent with the kernel precedent, rather than
the spec spelling.
Eric,
I checked the latest drafts of SPC-4 and SBC-3 and they
contain both "zeros" and "zeroes". Take your pick!
More seriously the LBPRZ flag now appears in the Logical
Block Provisioning VPD page and the READ CAPACITY (16)
response. Your patch sets the latter, could you add the
LBPRZ flag setting in the inquiry_evpd_b2() function as
well. Perhaps:
if (scsi_debug_unmap_zeroes)
arr[1] |= 1 << 2;
And if your are editing that function in the comment introducing
that function:
s/Thin/Logical block/
to reflect the renaming done by t10.org .
Ah, and inquiry_evpd_b2() should return 4 (not 8).
Otherwise it looks good.
Doug Gilbert
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
