Re: [storm] On Virtual Address, Steering Address, Base Offset, Tagged Base Offset and more

Tom Talpey <ttalpey@microsoft.com> Fri, 15 July 2011 19:14 UTC

From: Tom Talpey <ttalpey@microsoft.com>
To: Michael Ko <Michael@huaweisymantec.com>, "storm@ietf.org" <storm@ietf.org>
Thread-Topic: [storm] On Virtual Address, Steering Address, Base Offset, Tagged Base Offset and more
Thread-Index: AQHMQyJAOFlwK00ElE28ipXGIqoEp5Ttvxrw
Date: Fri, 15 Jul 2011 19:14:32 +0000
Message-ID: <F83812DF4B59B9499C1BC978336D91745EE0405B@TK5EX14MBXC111.redmond.corp.microsoft.com>
References: <ED870C9971DC4A6EBA7FC6CC9A9A4E68@china.huawei.com>
In-Reply-To: <ED870C9971DC4A6EBA7FC6CC9A9A4E68@china.huawei.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: multipart/alternative; boundary="_000_F83812DF4B59B9499C1BC978336D91745EE0405BTK5EX14MBXC111r_"
MIME-Version: 1.0
Subject: Re: [storm] On Virtual Address, Steering Address, Base Offset, Tagged Base Offset and more
Precedence: list

Sounds good. I assume you'll scan the document for other instances of the term "address" etc. I also spot a few uses of the term "points to" in 6.9 and 7.3.1. The one in 7.3.1 is associated with some questionable text indicating that the arithmetic is not needed (Base Offset == Tagged Offset). I assume you will investigate those as part of tightening up the TO calculation rules..

Tom.

From: storm-bounces@ietf.org [mailto:storm-bounces@ietf.org] On Behalf Of Michael Ko
Sent: Friday, July 15, 2011 3:06 PM
To: storm@ietf.org
Subject: [storm] On Virtual Address, Steering Address, Base Offset, Tagged Base Offset and more

I will combine my responses to Mallikarjun and Tom in one e-mail.

I am fine with using the term "Base Offset" instead of "Virtual Address".  It will be defined in the Definition section as "This represents a value to be added to the Buffer Offset in order to obtain the Tagged Offset."  So in sec. 2.4.1, the text "The base Tagged Offset is not explicitly specified, but the target must always assume it as zero" will be deleted.

In sec. 7.3.5 on SCSI Data-in, the original text "It MUST use the Buffer Offset from the SCSI Data-in PDU as the Data Sink Tagged Offset of the RDMA Write Message" will be replaced with:

"It MUST add the Buffer Offset from the SCSI Data-in PDU to the Base Offset as the Data Sink Tagged Offset of the RDMA Write Message."

Similarly, in sec. 7.3.6 on Ready To Transfer, the original text "MUST use the Buffer Offset from the R2T PDU as the Data
Source Tagged Offset of the RDMA Read Request Message" will be replaced with:

"MUST add the Buffer Offset from the R2T PDU to the Base Offset as the Data Source Tagged Offset of the RDMA Read Request Message."

Mike

[storm] On Virtual Address, Steering Address, Bas… Michael Ko
Re: [storm] On Virtual Address, Steering Address,… Tom Talpey
Re: [storm] On Virtual Address, Steering Address,… Michael Ko