[nvmewin] Question on SntiTranslateWrite6()

[Luse, Paul E]

No problem.  Sorry, I meant to add a note to that email too - either my thought transcriber is not working or I'm doing too many things at once J

From: Neal Galbo (ngalbo) [mailto:ngalbo at micron.com]
Sent: Tuesday, October 02, 2012 8:11 AM
To: Luse, Paul E; Freyensee, James P; nvmewin at lists.openfabrics.org
Subject: RE: Question on SntiTranslateWrite6()

Sorry, I stand corrected.

Thank you,
Neal


From: Luse, Paul E [mailto:paul.e.luse at intel.com]<mailto:[mailto:paul.e.luse at intel.com]>
Sent: Tuesday, October 02, 2012 10:12 AM
To: Neal Galbo (ngalbo); Freyensee, James P; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: RE: Question on SntiTranslateWrite6()



[cid:image001.png at 01CDA075.A41E2830]

From: Neal Galbo (ngalbo) [mailto:ngalbo at micron.com]<mailto:[mailto:ngalbo at micron.com]>
Sent: Tuesday, October 02, 2012 7:01 AM
To: Luse, Paul E; Freyensee, James P; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: RE: Question on SntiTranslateWrite6()

FYI,

6-byte SCSI READ/WRITE CDB's have a Big Endian 16-bit LBA.
Shouldn't you be "getting" a U16? (GET_U16_FROM_CDB)

Regards,
Neal Galbo

From: nvmewin-bounces at lists.openfabrics.org<mailto:nvmewin-bounces at lists.openfabrics.org> [mailto:nvmewin-bounces at lists.openfabrics.org]<mailto:[mailto:nvmewin-bounces at lists.openfabrics.org]> On Behalf Of Luse, Paul E
Sent: Monday, October 01, 2012 11:16 PM
To: Freyensee, James P; nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: Re: [nvmewin] Question on SntiTranslateWrite6()

Yup, there's a few copy n paste errors here.  In SntiTranslateWrite6() we should be using the 24 macro and in SntiTranslateRead6() we should be masking with READ_6_CDB_LBA_MASK for readability (it's the same as the write mask define).  If you'd like some practice submitting a patch you can send one out with this and any other cleanup you might find after my patch in flight is pushed.  I'd offer to throw these in mine but I've already tested with 2 HW platforms and QEMU and don't want to re-test :)

Thanks
Paul

From: nvmewin-bounces at lists.openfabrics.org<mailto:nvmewin-bounces at lists.openfabrics.org> [mailto:nvmewin-bounces at lists.openfabrics.org]<mailto:[mailto:nvmewin-bounces at lists.openfabrics.org]> On Behalf Of Freyensee, James P
Sent: Monday, October 01, 2012 7:04 PM
To: nvmewin at lists.openfabrics.org<mailto:nvmewin at lists.openfabrics.org>
Subject: [nvmewin] Question on SntiTranslateWrite6()

I have a question in SntiTranslateWrite6().  Should this line:

lba = GET_U32_FROM_CDB(pSrb, WRITE_6_CDB_LBA_OFFSET);

actually be using the following MACRO mask, found in SntiTranslateRead6():

    lba = GET_U24_FROM_CDB(pSrb, READ_6_CDB_LBA_OFFSET);

I'm not totally clear on the different byte shifting and the different offsets being used between the two.  However from the SCSI spec, the LBA bits are in the same spot in CDB[] for Write6 and Read6.  So I would have assumed the byte shifting and offset algorithm and values would be the same.

Thanks,
Jay

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20121002/c842d8b0/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image001.png
Type: image/png
Size: 20878 bytes
Desc: image001.png
URL: <http://lists.openfabrics.org/pipermail/nvmewin/attachments/20121002/c842d8b0/attachment.png>