Re: [storm] iSER draft

[Tom Talpey <[e-mail]>]

Changing the name doesn't address the core issues here. There are no processing rules defined in the draft to support the arithmetic you describe below, or even to use the Steering/Virtual Address.

For example, section 7.3.5 states the target MUST use the Buffer Offset of the Data-In PDU as the Tagged Offset of an RDMA Write. Where does the new Steering Address in the control PDU header get folded into the Tagged Offset? And what protocol state would trigger the target to do so, including perform the arithmetic? Same question for several other sections.



From: Michael Ko [mailto:[e-mail]]
Sent: Thursday, July 14, 2011 2:39 PM
To: Caitlin Bestler; Tom Talpey
Cc: Alexander Nezhinsky; [e-mail]
Subject: Re: [storm] iSER draft

When "Virtual Address" is not used, as in iWARP, the initiator side needs to translate the "Tagged Offset" which is just the offset into the buffer, into a usable address by adding it to the starting base address of the buffer.  If the starting base address is communicated to the target side, as is done in some RCaPs, then the "Tagged Offset" is the usable address itself at the initiator where data is to be fetched or stored.  Alex suggested that perhaps we can change the name "Virtual Address" to "Steering Address".  Then it can be defined in the glossary without tripping over the common term "virtual address".

Mike
----- Original Message -----
From: Caitlin Bestler<mailto:[e-mail]>
To: Tom Talpey<mailto:[e-mail]>
Cc: Alexander Nezhinsky<mailto:[e-mail]> ; Michael Ko<mailto:[e-mail]> ; [e-mail]<mailto:[e-mail]>
Sent: Wednesday, July 13, 2011 9:49 PM
Subject: Re: [storm] iSER draft


On Jul 13, 2011, at 8:12 AM, Tom Talpey wrote:

> Yes, understood on the ZBVA/Infiniband issue. Regarding the VA term's presence in an earlier draft, I did not see it in RFC5046 and I did not review the expired draft, so consider these comments to be on the overall change, not the last revision.
>
> Let me try a different approach to perhaps make myself clearer.
>
> First, it seems to me that a "virtual address" has no real meaning to the network protocol. It begins, perhaps, as some value in an address space on the initiator, but it's not meaningful on the target in this way, it's only used to request that the remote RNIC perform a transfer to that originally registered remote address. So calling it a Virtual Address as an iSER protocol object seems, to me, an artificial and somewhat leading convention.
>
> Second, the Virtual Address goes out from the initiator to the target in a Control PDU, but where does it come back? The RDMA Read or Write as depicted in (xx) shows only a Tagged Offset. So, it's not clear what its protocol meaning is.
>
> Third, I don't ever see a Tagged Buffer described by a fully qualified four-tuple. I see it appearing as either { Stag, TO, length } or { Stag, VA, length }, depending on the addressing mode.
>
> I think the non-ZBVA mode is really just a special case of the existing one, but where the meaning of TO has changed from a small offset to some other token, generated and managed only at the initiator. So, it seems artificial to define it as distinct, and document it as possessing some new properties. Isn't it just a Target Offset, still?
>
> Tom.
>
>

I agree, so far we have not seen a protocol justification for the need to add "Virtual Address" to the glossary as something distinct from Target Offset
for the purpose of defining an IETF protocol.

I am sympathetic to the interoperability issues raised, but I don't think those can justify something that has *no* justification in the protocol.

If someone could site a class of implementation where there is a real need for this distinction in an iSER adapter, but as far as I can see the adapters have
to be able to translate to TO *anyway* in order to use an RDMA Write or RDMA Read.

Local Interface compatibility with IB can make a *lot* of sense, but why does it have to make its way into the *wire* protocol?

--
Caitlin Bestler
[e-mail]<mailto:[e-mail]>
http://www.asomi.com/CaitlinBestlerResume.html