[stgt] [PATCH] UNMAP Change order of no-media and thin-provisioning tests

ronnie sahlberg ronniesahlberg at gmail.com
Wed Jul 18 00:39:21 CEST 2012


Tomo,

Please find attached a trivial patch to the UNMAP command.
It only changes in which order we perform two error checks.

Currently TGTD checks the UNMAP command using the sequence
1, if not-ThinProvisioned  then error
2, if NoMedia  then error


But since ThinProvisioning is more an attribute of the media itself
and not of the LUN/DiskDrive
it makes it more logical to do the checks in the reverse order for
LUNs that support BOTH removable media and thin-provisioning like this
:
1, if NoMedia then error
2, if not ThinProvisioned then error


regards
ronnie sahlberg
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Check-for-MediaPresent-before-checking-for-ThinProvi.patch.gz
Type: application/x-gzip
Size: 778 bytes
Desc: not available
URL: <http://lists.wpkg.org/pipermail/stgt/attachments/20120718/7a949037/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Check-for-MediaPresent-before-checking-for-ThinProvi.patch
Type: application/octet-stream
Size: 1449 bytes
Desc: not available
URL: <http://lists.wpkg.org/pipermail/stgt/attachments/20120718/7a949037/attachment-0001.obj>


More information about the stgt mailing list