Erratum on RDDP Architecture RFC

[email protected] Mon, 1 Dec 2008 16:49:10 -0500
Newsgroups gmane.ietf.rddp
Message-ID <9FA859626025B64FBC2AF149D97C944A010749F7@CORPUSMX80A.corp.emc.com>
A pair of sharp eyes caught a couple of related issues
in the RDDP Architecture RFC:

http://www.rfc-editor.org/errata_search.php?eid=136

In summary, the first prototype for rdma_read is:

  rdma_read(socket_t s, ddp_addr_t s, ddp_addr_t d);

and the second prototype doesn't match:

  rdma_read(socket_t s, ddp_addr_t s, length_t l, ddp_addr_t d)

plus the second prototype has two arguments that are both
named "s".  Oops ...

I believe that the correct fix is that the length argument
from the second prototype needs to be in both prototypes,
and the DDP buffer argument "s" (source) should be renamed
to "rs" (remote source) in both prototypes in order to
avoid a name conflict with the socket argument.

If anyone disagrees, please respond.

Assuming this approach is ok, I propose to carry it out
as follows: 
- I will file a new erratum with the above two changes
- I will then ask the RFC Editor to reject the original
	erratum, as it contains a correction for only one
	of the two problems that it identifies.

Thanks,
--David
----------------------------------------------------
David L. Black, Distinguished Engineer
EMC Corporation, 176 South St., Hopkinton, MA  01748
+1 (508) 293-7953             FAX: +1 (508) 293-7786
[email protected]        Mobile: +1 (978) 394-7754
----------------------------------------------------