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">&lt;<a href="mailto:blp@nicira.com" target="_blank">blp@nicira.com</a>&gt;</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>
&gt; With this commit, OVS will accept the traffic with eth_type RARP.<br>
&gt;<br>
&gt; Signed-off-by: Mehak Mahajan &lt;<a href="mailto:mmahajan@nicira.com">mmahajan@nicira.com</a>&gt;<br>
&gt; ---<br>
&gt;  datapath/flow.c |    3 ++-<br>
&gt;  1 files changed, 2 insertions(+), 1 deletions(-)<br>
<br>
</div>Don&#39;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>