<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 19 May 2016 at 02:08, Black, David <span dir="ltr">&lt;<a href="mailto:david.black@emc.com" target="_blank">david.black@emc.com</a>&gt;</span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="#0563C1" vlink="#954F72">
<p class="MsoNormal"><span style="color:#1f497d">As for the original question:<u></u><u></u></span></p><span class="">
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p><span style="color:#1f497d">&gt; </span>When the spec says &quot;number of logical blocks specified to be [...] written&quot; is this only in reference to the NUMBER OF LOGICAL BLOCKS<u></u><u></u></p>
<p>&gt; passed in rather than the computed number of blocks to be written or is there a choice over which is used (i.e. the target can pick<u></u><u></u></p>
<p>&gt; whether it errors or not if the computed blocks exceed the MAXIMUM WRITE SAME LENGTH)?<u></u><u></u></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
</span><p class="MsoNormal"><span style="color:#1f497d">Neither - my reading of the current SBC-4 text is that the limit always applies to the number of blocks written by the command, but a careful reading of the text is involved to reach that conclusion.  Excerpts
 that follow are from 5.47 WRITE SAME (10) in SBC-4 rev 10: <u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal" style="margin-left:.5in;text-autospace:none"><span style="font-size:10.0pt;font-family:ArialMT">If the
</span><span style="font-size:8.0pt;font-family:ArialMT">WSNZ </span><span style="font-size:10.0pt;font-family:ArialMT">bit is set to zero, then a
</span><span style="font-size:8.0pt;font-family:ArialMT">NUMBER OF LOGICAL BLOCKS
</span><span style="font-size:10.0pt;font-family:ArialMT">field set to zero specifies that the number of<u></u><u></u></span></p>
<p class="MsoNormal" style="margin-left:.5in;text-autospace:none"><span style="font-size:10.0pt;font-family:ArialMT">contiguous logical blocks that are requested to be unmapped or written includes all of the logical blocks<u></u><u></u></span></p>
<p class="MsoNormal" style="margin-left:.5in"><span style="font-size:10.0pt;font-family:ArialMT">starting with the LBA specified in the
</span><span style="font-size:8.0pt;font-family:ArialMT">LOGICAL BLOCK ADDRESS </span>
<span style="font-size:10.0pt;font-family:ArialMT">field to the last logical block on the medium.</span><span style="color:#1f497d"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d">Note the use of “specifies” in the first line.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal" style="margin-left:.5in;text-autospace:none"><span style="font-size:10.0pt;font-family:ArialMT;color:black">If the number of logical blocks specified to be unmapped or written exceeds the value indicated in the<u></u><u></u></span></p>
<p class="MsoNormal" style="margin-left:.5in;text-autospace:none"><span style="font-size:8.0pt;font-family:ArialMT;color:black">MAXIMUM WRITE SAME LENGTH
</span><span style="font-size:10.0pt;font-family:ArialMT;color:black">field</span><span style="font-size:10.0pt;font-family:ArialMT;color:red">
</span><span style="font-size:10.0pt;font-family:ArialMT;color:black">(see 6.6.4), then the device server shall<u></u><u></u></span></p>
<p class="MsoNormal" style="margin-left:.5in;text-autospace:none"><span style="font-size:10.0pt;font-family:ArialMT;color:black">terminate the command with CHECK CONDITION status with the sense key set to ILLEGAL REQUEST and<u></u><u></u></span></p>
<p class="MsoNormal" style="margin-left:.5in"><span style="font-size:10.0pt;font-family:ArialMT;color:black">the additional sense code set to INVALID FIELD IN CDB.</span><span style="color:#1f497d"><u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d">Note the use of “specified” in the first line.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d">So, a NUMBER OF LOGICAL BLOCKS field set to zero specifies that all of the logical blocks up to the end of the medium are to be unmapped or written, and if that (computed) number of logical blocks exceeds the
 MAXIMUM WRITE SAME LENGTH value, the required result is CHECK CONDITION, ILLEGAL REQUEST, INVALID FIELD IN CDB.<u></u><u></u></span></p><span class="">
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p><span style="color:#1f497d">&gt; </span>For example, if I have an *SBC-3* 1GByte target with a WSNZ of 0 and a MAXIMUM WRITE SAME LENGTH of 131072 blocks<u></u><u></u></p>
<p>&gt; (64Mbytes with a sector size of 512), then I issue a WRITE SAME with NUMBER OF LOGICAL BLOCKS of 0 and a<u></u><u></u></p>
<p>&gt; LOGICAL BLOCK ADDRESS of 0 is it legal for the SBC-3 target to produce an error because the computed number of<u></u><u></u></p>
<p>&gt; blocks to be written will be greater than 131072?<u></u><u></u></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
</span><p class="MsoNormal"><span style="color:#1f497d">That is not only “legal”, it is also the behavior required by the standard.<u></u><u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d"><u></u> <u></u></span></p>
<p class="MsoNormal"><span style="color:#1f497d">I believe the standard is unambiguous as written, but I’d support an editorial clarification to reduce the degree of careful reading required to reach that conclusion.</span></p></div></blockquote><div><br></div><div>The minor clarifications (where you added the word &quot;computed&quot; in brackets) were helpful and I&#39;d say it would be helpful in the spec. Whatever happens thanks for clearing up the only acceptable behaviour!</div></div><div><br></div>-- <br><div class="gmail_signature">Sitsofe | <a href="http://sucs.org/~sits/" target="_blank">http://sucs.org/~sits/</a></div>
</div></div>