> On 19 May 2014 19:53, Lori Jakab <loja...@cisco.com> wrote: > >> On 5/19/14, 10:13 AM, YAMAMOTO Takashi wrote: >> >>> Signed-off-by: Joe Stringer <joestrin...@nicira.com> >>>> --- >>>> utilities/ovs-dev.py | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/utilities/ovs-dev.py b/utilities/ovs-dev.py >>>> index 97fab6c..d979afd 100755 >>>> --- a/utilities/ovs-dev.py >>>> +++ b/utilities/ovs-dev.py >>>> @@ -144,7 +144,7 @@ commands.append(make) >>>> def check(): >>>> - flags = "" >>>> + flags = "-j8 " >>>> if options.tests: >>>> flags += options.tests >>>> ENV["TESTSUITEFLAGS"] = flags >>>> -- >>>> 1.7.10.4 >>>> >>> -j8 seems too big to hardcode as the memory requirement of >>> some tests is rather high. >> >> > Out of curiousity, do you know what the memory requirement is? I couldn't > tell the difference on my machine.
i don't know the exact requirement. last time i tried, -j8 didn't work on my VM with 256MB memory. YAMAMOTO Takashi > > > >> Additionally, it used to trigger some race conditions for me in the past, >> casing test failures that didn't occur without parallelism. > > > I think this has gotten a lot better (I've been running tests like this for > several months), although I agree that it occasionally presents race > conditions that otherwise don't show. > > > Thanks for the feedback. It's probably easier to make it a commandline > option, so I'll resend. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
