[ovs-dev] [PATCH] Accept the traffic marked with RARP eth_type.

Mehak Mahajan mmahajan at nicira.com
Wed Oct 31 17:41:40 UTC 2012


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 <blp at nicira.com> 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 <mmahajan at nicira.com>
> > ---
> >  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?
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20121031/1ce1b32c/attachment-0003.html>


More information about the dev mailing list