On Thu, Aug 11, 2016 at 10:08:12AM -0700, Ben Pfaff wrote:
> On Fri, Aug 05, 2016 at 03:25:47PM +0200, Christian Ehrhardt wrote:
> > Hi,
> > while checking latest master we found that there were several tests that
> > failed sometimes.
> >   15: bfd - bfd decay
> > 1149: ofproto-dpif - in place modification (vlan)
> > But one of them seems to reliably fail in i686 mode while working just fine
> > in a similar amd64 build.
> > 
> > Steps to reproduce:
> > make check TESTSUITEFLAGS='2214'
> > 
> > I usually use the Debian build using the debian/ subdir packaged in
> > openvswitch:
> > dh build --with autoreconf,python2,python3,systemd --parallel
> > 
> > But to make sure I also ran a "normal" clean build with:
> > ./boot.sh && ./configure && make
> > Built that way it fails just as well.
> > 
> > It seems that the packets were expected twice but only received once:
> > checking packets in hv1/vif1-tx.pcap against 1.expected:
> > expected 6 packets, only received 3
> > ../../tests/ovn.at:1326: sort $rcv_text
> > --- expout 2016-08-05 12:13:09.661688314 +0000
> > +++ /<<PKGBUILDDIR>>/_dpdk/tests/testsuite.dir/at-groups/2214/stdout
> > 2016-08-05 12:13:09.661688314 +0000
> > @@ -1,6 +1,3 @@
> >  010000000000f0000000000202ff
> > -010000000000f0000000000303ff
> >  f00000000001f000000000020021
> > -f00000000001f000000000030031
> >  fffffffffffff0000000000202ff
> > -fffffffffffff0000000000303ff
> > 
> > Full log(s) attached
> 
> I'd like to take a look at this but the logs don't seem to have been
> attached.
> 
> (As a data point, I develop on with 32-bit userspace but I don't see
> this consistent failure.)

Did the "master" that you tested include the following commit?  It fixed
a number of races in practice.

commit 49d7c759b82e78acd03288c123b1bc0c8904a52b
Author: Ben Pfaff <b...@ovn.org>
Date:   Tue Jul 26 23:50:06 2016 -0700

    tests: Remove most packet-forwarding related "sleep"s from OVN tests.
    
    These arbitrary sleeps are often longer than necessary and can be too short
    in corner cases.  This commit replaces them by a common macro that waits
    only as long as necessary.
    
    Signed-off-by: Ben Pfaff <b...@ovn.org>
    Acked-by: Ryan Moats <rmo...@us.ibm.com>
    Acked-by: Flavio Fernandes <fla...@flaviof.com>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to