That is the following commit:
commit 4edb9ae90e4092f5f56b9d914d2b88783c49860d
Author: Pravin B Shelar <[email protected]>
Date: Fri Oct 21 14:38:54 2011 -0700
datapath: Refactor actions in terms of match fields.
Almost all current actions can be expressed in the form of
push/pop/set <field>, where field is one of the match fields. We can
create three base actions and take a field. This has both a nice
symmetry and avoids inconsistencies where we can match on the vlan
TPID but not set it.
Following patch converts all actions to this new format.
Signed-off-by: Pravin B Shelar <[email protected]>
Acked-by: Jesse Gross <[email protected]>
Bug #7115
This commit drastically changes the user/kernel interface. After you
recompiled and installed the new version, did you make sure to unload
the old kernel module and install and load the new kernel module?
Without doing that, I'd expect that no packets get through.
On Fri, Nov 04, 2011 at 07:51:21PM +0900, Jari Sundell wrote:
> The specific change was:
>
> git diff
> a0003c0c359bc3ffe8a6683dbd0121877a3ce700..4edb9ae90e4092f5f56b9d914d2b88783c49860d
>
> Jari Sundell
>
> On Fri, Nov 4, 2011 at 4:53 PM, Jari Sundell <[email protected]>
> wrote:
> > After more testing it turns out it isn't the patch itself, rather
> > something that was committed since 65c3058c22.
> >
> > Jari Sundell
> >
> > On Fri, Nov 4, 2011 at 4:30 PM, Jari Sundell <[email protected]>
> > wrote:
> >> Seems things are not yet done... Without this patch (git master)
> >> packets are properly being passed to local port when disable-in-band
> >> is true.
> >>
> >> With the patch no packets get through, with or without disable-in-band set.
> >>
> >> Jari Sundell
> >>
> >> On Thu, Nov 3, 2011 at 5:10 AM, Ben Pfaff <[email protected]> wrote:
> >>> Thanks, pushed to master, branch-1.3, branch-1.2.
> >>>
> >>> On Wed, Nov 02, 2011 at 01:03:56PM -0700, Ethan Jackson wrote:
> >>>> Looks good.
> >>>>
> >>>> Ethan
> >>>>
> >>>> On Wed, Nov 2, 2011 at 12:59, Ben Pfaff <[email protected]> wrote:
> >>>> > stream_init() didn't initialize the remote_ip, remote_port, local_ip,
> >>>> > or
> >>>> > local_port members of the stream, so "unix" streams that don't have
> >>>> > any of
> >>>> > those would get random values instead.
> >>>> >
> >>>> > Reported-by: "Voravit T." <[email protected]>
> >>>> > Reported-by: Jari Sundell <[email protected]>
> >>>> > ---
> >>>> > ?AUTHORS ? ? ?| ? ?2 ++
> >>>> > ?lib/stream.c | ? ?1 +
> >>>> > ?2 files changed, 3 insertions(+), 0 deletions(-)
> >>>> >
> >>>> > diff --git a/AUTHORS b/AUTHORS
> >>>> > index d19d665..0d4cc95 100644
> >>>> > --- a/AUTHORS
> >>>> > +++ b/AUTHORS
> >>>> > @@ -77,6 +77,7 @@ Henrik Amren ? ? ? ? ? [email protected]
> >>>> > ?Jad Naous ? ? ? ? ? ? ? [email protected]
> >>>> > ?Jan Medved ? ? ? ? ? ? [email protected]
> >>>> > ?Janis Hamme ? ? ? ? ? ? [email protected]
> >>>> > +Jari Sundell ? ? ? ? ? [email protected]
> >>>> > ?Jed Daniels ? ? ? ? ? ? [email protected]
> >>>> > ?Jeongkeun Lee ? ? ? ? ? [email protected]
> >>>> > ?Joan Cirer ? ? ? ? ? ? [email protected]
> >>>> > @@ -105,6 +106,7 @@ Takayuki HAMA ? ? ? ? ? [email protected]
> >>>> > ?Teemu Koponen ? ? ? ? ? [email protected]
> >>>> > ?Tyler Coumbes ? ? ? ? ? [email protected]
> >>>> > ?Vishal Swarankar ? ? ? [email protected]
> >>>> > +Voravit T. ? ? ? ? ? ? [email protected]
> >>>> > ?YAMAMOTO Takashi ? ? ? [email protected]
> >>>> > ?Yongqiang Liu ? ? ? ? ? [email protected]
> >>>> > ?kk yap ? ? ? ? ? ? ? ? [email protected]
> >>>> > diff --git a/lib/stream.c b/lib/stream.c
> >>>> > index 37b6110..8f567ca 100644
> >>>> > --- a/lib/stream.c
> >>>> > +++ b/lib/stream.c
> >>>> > @@ -616,6 +616,7 @@ void
> >>>> > ?stream_init(struct stream *stream, struct stream_class *class,
> >>>> > ? ? ? ? ? ? int connect_status, const char *name)
> >>>> > ?{
> >>>> > + ? ?memset(stream, 0, sizeof *stream);
> >>>> > ? ? stream->class = class;
> >>>> > ? ? stream->state = (connect_status == EAGAIN ? SCS_CONNECTING
> >>>> > ? ? ? ? ? ? ? ? ? ? : !connect_status ? SCS_CONNECTED
> >>>> > --
> >>>> > 1.7.4.4
> >>>> >
> >>>> > _______________________________________________
> >>>> > dev mailing list
> >>>> > [email protected]
> >>>> > http://openvswitch.org/mailman/listinfo/dev
> >>>> >
> >>>
> >>
> >
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev
