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.
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
