[dhcwg] Some questions on Relay Agent Encapsulation for DHCPv4

[Bharat Joshi <[e-mail]>]

Hi Ted,

     I finally got some time to go through it. I have come up with following questions and suggestions:

• Whole of abstract is one single sentence. Can we divide them into multiple sentences?

• In abstract, 'more than one relay agent can insert relay agent suboptions into the forwarding chain' -> 'each relay agent can insert or remove required relay agent information while forwarding the DHCP packets'

• Layer 2 address suboption is defined in section 2.3 but its not mentioned that this sub-option is defined for which option. Or can we define global sub-option as well?

• Should we use BOOTRELAYFORWARD and BOOTRELAYREPLY instead of RELAYFORWARD and RELAYREPLY?

• 'ep' and padlen will be added only by the first relay agent. Will it make sense to add these details in relay-segment as a sub-option. This way, this information need not be available in other relay-forward or relay-reply messages. It will also save in some memory if there are multiple relay agents.

• In point 3 of section 3.3, it mention 'End packet' which I think should be 'End option'.

• Why do we need to store pad-len and end-option information? Is it for security purposes? 

• If answer to above is yes, do we need to take care of any data after the 'End option' which is included in the IP header length? If yes, how do we do that?

• Last paragraph of section 3.3 mention sname and file field only. Why is that? Actually speaking, it should not touch any field in DHCP header. Right?

• Something is missing in the last sentence of last paragraph of section 4. 'message from the DHCP server is in response such a message, since it'

• In para 3 of section 4.1, it is not clear what to do with these packets?

• When a new RELAYFORWARD packet is created, what should be done if it exceeds the MTU of the interface on which it needs to be sent?

• In section 4.1.3, first sentence of second para, should not we mention that it must be configurable on per interface basis?

• Can we move the section 4.2.2 after 4.2.4?

• Should we be looking at IP TTL when we create RELAYFORWARD or RELAYREPLY message?

• In section 4.2.1.1, total-length in the IP header will also change. So we will need to recalculate IP header checksum as well.

• In section 4.4.1.2, it does not talk about link selection sub-option. Should we add few lines for the same?

• In section 4.5.1, first para, it says that RA must forward the decapsulated message to IP address in aiaddr. But what if the IP address is a braodcast address. Also should it really do routing while returning the DHCP packets or should it identify the outgoing interface using information available in Relay Segment?

• In section 4.5.1, second para mention 'ARP' which means we are assuming that layer 2 is ethernet while link layer address sub-option can be there for other layer 2 protocols as well. This is mentioned in somewhere else also in this document. Please fix it there as well.

• Second para of section 5.1.1 is one single sentence. It would be great if we can divide this into multiple sentence.

• In section 5.1.2, first para, 'decapsulate relay suboptions' -> 'decapsulate RAIO suboptions'

• Second para of section 5.1.2 defines inner encapsulation. Can we move this to terminology?

• Second last para of section 5.1.2, it uses 'must' for using all suboptions or only one. Is not this too strict? Some DHCP server may want to use first two relay agent options while some other DHCP server may want to use first and last only. So should not we provide more flexibility here?

• Last para of section 5.1.2 is not clear. What are we trying to suggest here?

• In section 5.2, can we remove last part of the sentence "add sends it to the next hop on the way to client'? We anyway explains this below.

• With this relay chaining, do we take care of L2RAs sitting between two L3 RAs?

• In section 5.2.1.1, in second para, DHCP will now add something in Relay Agent suboptions instead of echoing back what it received. Are not we breaking away from current standard here? Do we want to give more emphasis on this here?

• In section 5.3, 'an incoming RAIO from a' -> 'an incoming packet with RAIO from a'

• In section 7, last para, how will relay agent authentication work if L2 RA is available in middle.

Best regards,
Bharat
**************** CAUTION - Disclaimer *****************
This e-mail contains PRIVILEGED AND CONFIDENTIAL INFORMATION intended solely 
for the use of the addressee(s). If you are not the intended recipient, please 
notify the sender by e-mail and delete the original message. Further, you are not 
to copy, disclose, or distribute this e-mail or its contents to any other person and 
any such actions are unlawful. This e-mail may contain viruses. Infosys has taken 
every reasonable precaution to minimize this risk, but is not liable for any damage 
you may sustain as a result of any virus in this e-mail. You should carry out your 
own virus checks before opening the e-mail or attachment. Infosys reserves the 
right to monitor and review the content of all messages sent to or from this e-mail 
address. Messages sent to or from this e-mail address may be stored on the 
Infosys e-mail system.
***INFOSYS******** End of Disclaimer ********INFOSYS***