(correction - error is in SBC-3) Code error in SBC-3 Annex-C

Sheffield, Robert L robert.l.sheffield at intel.com
Tue Jan 3 17:19:07 PST 2006


* From the T10 Reflector (t10 at t10.org), posted by:
* "Sheffield, Robert L" <robert.l.sheffield at intel.com>
*
This is a multi-part message in MIME format.

------_=_NextPart_001_01C610CC.E0F45F9A
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

=20

  _____ =20

From: owner-t10 at t10.org [mailto:owner-t10 at t10.org] On Behalf Of
Sheffield, Robert L
Sent: Tuesday, January 03, 2006 5:39 PM
To: T10 Reflector
Cc: George Penokie; pat.thaler at avagotech.com
Subject: Code error in SBC-3 Annex-C



One of our engineers found a code error in the CRC example code in =
SBC-3
(Annex-C):=20
1) Page 161, C code line 1: This comment should contain SBC-3 instead =
of
SBC-2.=20
2) Page 161, C code line 17: The C code frame[i+1] will fetch garbage
data and generate an incorrect CRC when the data length is odd.

I suspect the normative CRC definition in 4.16.4.2 deals with the
odd-byte condition. But I haven't checked to be certain.


------_=_NextPart_001_01C610CC.E0F45F9A
Content-Type: text/html;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">

Code error in SPC-3 Annex-C  

 From: owner-t10 at t10.org=20 [mailto:owner-t10 at t10.org] On Behalf Of Sheffield, Robert=20 L
Sent: Tuesday, January 03, 2006 5:39 PM
To: T10=20 Reflector
Cc: George Penokie;=20 pat.thaler at avagotech.com
Subject: Code error in SBC-3=20 Annex-C


 
One of our engineers found a code error = in the CRC=20 example code in SBC-3 (Annex-C): 
1) Page 161,=20 C code line 1: This comment should contain SBC-3 instead of SBC-2.=20 
2) Page 161, C code line 17: The = C code=20 frame[i+1] will fetch garbage data and generate an incorrect CRC when = the data=20 length is odd. I suspect the normative CRC definition = in 4.16.4.2=20 deals with the odd-byte condition. But I haven't checked to be=20 certain.

------_=_NextPart_001_01C610CC.E0F45F9A--





More information about the T10 mailing list