Hey Ben,<br><br>What you are saying makes perfect sense. <br><br>I have sent out a new patch which removes the opcode check altogether. <br>   <a href="http://openvswitch.org/pipermail/dev/2012-October/022194.html" target="_blank">http://openvswitch.org/pipermail/dev/2012-October/022194.html</a><br>
<br>thanx!<br>mehak <br><br><div class="gmail_quote">On Fri, Oct 19, 2012 at 2:44 PM, 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>On Fri, Oct 19, 2012 at 01:39:59PM -0700, Mehak Mahajan wrote:<br>
&gt; With this commit, the datapath will process the ARP header for<br>
&gt; RARP packets, both request and reply.<br>
&gt;<br>
&gt; Signed-off-by: Mehak Mahajan &lt;<a href="mailto:mmahajan@nicira.com" target="_blank">mmahajan@nicira.com</a>&gt;<br>
<br>
</div>It looks very much to me like all defined ARP opcodes (at least the<br>
ones listed here:<br>
<a href="http://www.networksorcery.com/enp/protocol/arp.htm#Opcode" target="_blank">http://www.networksorcery.com/enp/protocol/arp.htm#Opcode</a>) use the<br>
same packet format.  Would it be better to simply remove the protocol<br>
check, rather than to whitelist more values?<br>
</blockquote></div><br>