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