Hey Ben,<br><br>I seem to have missed out the lib/flow.c file in the diff. <br><br>However, as you mentioned, I had not addressed the RARP related changes at all the references to ETH_TYPE_ARP. I will add those changes as well and send out a revised patch. <br>
<br>thanx!<br>mehak<br><br><div class="gmail_quote">On Wed, Oct 31, 2012 at 8:07 AM, Ben Pfaff <span dir="ltr"><<a href="mailto:blp@nicira.com" target="_blank">blp@nicira.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Tue, Oct 30, 2012 at 09:44:52PM -0700, Mehak Mahajan wrote:<br>
> With this commit, OVS will accept the traffic with eth_type RARP.<br>
><br>
> Signed-off-by: Mehak Mahajan <<a href="mailto:mmahajan@nicira.com">mmahajan@nicira.com</a>><br>
> ---<br>
> datapath/flow.c | 3 ++-<br>
> 1 files changed, 2 insertions(+), 1 deletions(-)<br>
<br>
</div>Don't we need a matching change to userspace? Also, there are several<br>
references to ETH_TYPE_ARP throughout the tree. Did you review them to<br>
see whether some of those should be generalized to include RARP?<br>
</blockquote></div><br>