Hey Ben,

I seem to have missed out the lib/flow.c file in the diff.

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.

thanx!
mehak

On Wed, Oct 31, 2012 at 8:07 AM, Ben Pfaff <[email protected]> wrote:

> On Tue, Oct 30, 2012 at 09:44:52PM -0700, Mehak Mahajan wrote:
> > With this commit, OVS will accept the traffic with eth_type RARP.
> >
> > Signed-off-by: Mehak Mahajan <[email protected]>
> > ---
> >  datapath/flow.c |    3 ++-
> >  1 files changed, 2 insertions(+), 1 deletions(-)
>
> Don't we need a matching change to userspace?  Also, there are several
> references to ETH_TYPE_ARP throughout the tree.  Did you review them to
> see whether some of those should be generalized to include RARP?
>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to