at 2010-1-26 16:25, Garrett Cooper wrote: > On Mon, Jan 25, 2010 at 11:54 PM, Garrett Cooper <[email protected]> wrote: >> On Mon, Jan 25, 2010 at 9:45 PM, Shi Weihua <[email protected]> wrote: >>> Hi, Cooper >>> >>> I noticed that you added a check of platform which not defined >>> sa_sigaction in struct sigaction. >>> http://ltp.cvs.sourceforge.net/viewvc/ltp/ltp/testcases/kernel/syscalls/rt_sigaction/rt_sigaction01.c?r1=1.6&r2=1.7 >>> >>> As your expectted, the following message will output on x86_64. >>> ------------ >>> Your architecture doesn't support this test (no sa_sigaction field in >>> struct sigaction). >>> ------------ >>> But unfortunately, it still failed on my x86_64. >>> >>> On my x86_64, signal.h like this: >>> ------------ >>> #ifndef __ASSEMBLY__ >>> >>> #ifdef __i386__ >>> (sa_sigaction defined here) <-- >>> #else /* __i386__ */ >>> (no sa_sigaction defined) <-- >>> #endif /* !__i386__ */ >>> >>> #endif /* __ASSEMBLY__ */ >>> ------------ >>> your check still work ? Sorry, I don't know the check process in >>> LTP configure clearly. Could you give me some explanation. >> >> Shi, >> I'm just going to test out the other proposed rt_sigaction, and >> revert this change if it doesn't work back to a semi-functioning state >> (which means that x86_64 will once again be broken). > > Files reverted because liubo's patch didn't stick.
Do you will pick out some useful code from liubo's patch ? Previously, you said you would do that. (2009-12-22) ;-) > Thanks, > -Garrett > > ------------------------------------------------------------------------------ The Planet: dedicated and managed hosting, cloud storage, colocation Stay online with enterprise data centers and the best network in the business Choose flexible plans and management services without long-term contracts Personal 24x7 support from experience hosting pros just a phone call away. http://p.sf.net/sfu/theplanet-com _______________________________________________ Ltp-list mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ltp-list
