Re: [LTP] [PATCH] clone/clone08.c: check whether clone supports 7 arguments

[chrubis@suse.cz]

Hi!
> > > exactly.  why does a clone() test have any bearing at all on ia64
> > > behavior ? the code that checks the define won't get used in the
> > > __ia64__ case, so having
> > > a define here is pointless.
> > 
> > I thought the check applies to whole ltp_clone, since that is what is
> > called by testcases checking this define.
> 
> there's debate as to whether that's how we want to take things :).  we'll sort 
> it out there.

I would go with the __clone2() in the #else branch because the same
ifdef is used in the clone08 testcase and not having positive result in
__ia64__ case would wrongly disable the testcase.

It's shame that ia64 has different clone syscall interface but that is
price for the way how our system is evolving...

-- 
Cyril Hrubis
chrubis@suse.cz

------------------------------------------------------------------------------
CenturyLink Cloud: The Leader in Enterprise Cloud Services.
Learn Why More Businesses Are Choosing CenturyLink Cloud For
Critical Workloads, Development Environments & Everything In Between.
Get a Quote or Start a Free Trial Today. 
http://pubads.g.doubleclick.net/gampad/clk?id=119420431&iu=/4140/ostg.clktrk
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list