Thanks Simon for the information, Yes, I think I have fixed this bug. I have tried in on both physical machine and vm
On Sun, Oct 13, 2013 at 12:43 AM, Simon Horman <[email protected]> wrote: > To clarify, > > I was running the test-suite with a concurrency level of 2 > and the -e flag set. The first two tests failed and then the > test-run exited. > > I am happy to run test suite to answer your question about the > other tests but a subsequent email from you seems to indicate > that you may have found the cause of the problem I saw. > > On Thu, Oct 10, 2013 at 10:13:38PM -0700, Alex Wang wrote: > > Hi Simon, > > > > Just to confirm with you, > > > > Are these two tests broken? > > > > 1: bfd - basic config on different bridges > > 2: bfd - Verify tunnel down detection > > > > All other tests passed? > > > > Thanks, > > Alex Wang > > > > > > On Thu, Oct 10, 2013 at 9:47 PM, Simon Horman <[email protected]> > wrote: > > > > > Hi Ethan, > > > > > > I have investigated slightly further and it seems that the problem > > > does not manifest when compiling with clang but does manifest > > > when compiling with gcc. > > > > > > # clang --version > > > Debian clang version 3.4-1~exp1 (trunk) (based on LLVM 3.4) > > > Target: x86_64-pc-linux-gnu > > > Thread model: posix > > > > > > # cc --version > > > gcc (Debian 4.8.1-9) 4.8.1 > > > Copyright (C) 2013 Free Software Foundation, Inc. > > > This is free software; see the source for copying conditions. There > is NO > > > warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR > PURPOSE. > > > > > > On Thu, Oct 10, 2013 at 09:07:59PM -0700, Ethan Jackson wrote: > > > > Looking into it, should have checked before merging. > > > > > > > > Ethan > > > > > > > > On Thu, Oct 10, 2013 at 9:05 PM, Simon Horman <[email protected]> > > > wrote: > > > > > Curiously, making the stub non-static seems to break > > > > > at least the first two tests of the test-suite. > > > > > I assume its exposing a bug but I haven't investigated further. > > > > > > > > > > On Thu, Oct 10, 2013 at 06:18:17PM -0700, Ethan Jackson wrote: > > > > >> Oops didn't notice that it was. I'll change it before merging. > > > > >> > > > > >> Ethan > > > > >> > > > > >> On Thu, Oct 10, 2013 at 6:02 PM, Ben Pfaff <[email protected]> > wrote: > > > > >> > On Thu, Oct 10, 2013 at 05:57:51PM -0700, Ethan Jackson wrote: > > > > >> >> Stubs must be uninitialized when used in case they allocate > memory. > > > > >> >> > > > > >> >> Signed-off-by: Ethan Jackson <[email protected]> > > > > >> > > > > > >> > Also, stubs shouldn't ordinarily be static, any particular > reason > > > the > > > > >> > one here is? > > > > >> > > > > > >> > Acked-by: Ben Pfaff <[email protected]> > > > > >> > > > > > >> _______________________________________________ > > > > >> dev mailing list > > > > >> [email protected] > > > > >> http://openvswitch.org/mailman/listinfo/dev > > > > >> > > > > > > > _______________________________________________ > > > dev mailing list > > > [email protected] > > > http://openvswitch.org/mailman/listinfo/dev > > > >
_______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
