That is the following commit: commit 4edb9ae90e4092f5f56b9d914d2b88783c49860d Author: Pravin B Shelar <pshe...@nicira.com> 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 <pshe...@nicira.com> Acked-by: Jesse Gross <je...@nicira.com> 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 <sundell.softw...@gmail.com> > 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 <sundell.softw...@gmail.com> > > 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 <b...@nicira.com> 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 <b...@nicira.com> 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." <vora...@kth.se> > >>>> > Reported-by: Jari Sundell <sundell.softw...@gmail.com> > >>>> > --- > >>>> > ?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 ? ? ? ? ? ?hen...@nicira.com > >>>> > ?Jad Naous ? ? ? ? ? ? ? jna...@gmail.com > >>>> > ?Jan Medved ? ? ? ? ? ? ?jmed...@juniper.net > >>>> > ?Janis Hamme ? ? ? ? ? ? janis.ha...@student.kit.edu > >>>> > +Jari Sundell ? ? ? ? ? ?sundell.softw...@gmail.com > >>>> > ?Jed Daniels ? ? ? ? ? ? openvswi...@jeddaniels.com > >>>> > ?Jeongkeun Lee ? ? ? ? ? jk...@hp.com > >>>> > ?Joan Cirer ? ? ? ? ? ? ?j...@ev0.net > >>>> > @@ -105,6 +106,7 @@ Takayuki HAMA ? ? ? ? ? t-h...@cb.jp.nec.com > >>>> > ?Teemu Koponen ? ? ? ? ? kopo...@nicira.com > >>>> > ?Tyler Coumbes ? ? ? ? ? coum...@gmail.com > >>>> > ?Vishal Swarankar ? ? ? ?vishal.swarn...@gmail.com > >>>> > +Voravit T. ? ? ? ? ? ? ?vora...@kth.se > >>>> > ?YAMAMOTO Takashi ? ? ? ?yamam...@valinux.co.jp > >>>> > ?Yongqiang Liu ? ? ? ? ? liuyq7...@gmail.com > >>>> > ?kk yap ? ? ? ? ? ? ? ? ?yap...@stanford.edu > >>>> > 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 > >>>> > dev@openvswitch.org > >>>> > http://openvswitch.org/mailman/listinfo/dev > >>>> > > >>> > >> > > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev