I'm still having a really hard time spotting the problem. Can you pass along an "ovs-dpctl dump-flows" output that includes a set(eth(...)) action? An "ofproto/trace" output that includes such an action might also be helpful.
On Sun, Nov 06, 2011 at 12:00:41PM +0900, Jari Sundell wrote: > Packets captured as posted in the earlier mail; > > http://rakshasa.no/downloads/mod_dl_dst.off.ps > http://rakshasa.no/downloads/mod_dl_dst.on.ps > > It uses the openflow 1.0 protocol, and mod_dl_dst action is the only > thing different between the two afaics. > > Jari > > On Sun, Nov 6, 2011 at 8:26 AM, Ben Pfaff <b...@nicira.com> wrote: > > Does the problem only appear for flows that contain a mod_dl_dst action? > > > > On Sat, Nov 05, 2011 at 12:27:52PM +0900, Jari Sundell wrote: > >> Yes, everything was recompiled and rebooted several times to narrow > >> down the point at which the issue appeared. > >> > >> I suspect this could be related to the issue with packet out not > >> working when mod_dl_dst action is specified (earlier mail in > >> ovs-discuss), rather than the in-band issue. > >> > >> Jari Sundell > >> > >> On Sat, Nov 5, 2011 at 1:49 AM, Ben Pfaff <b...@nicira.com> wrote: > >> > 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
