On Tue, Jan 26, 2010 at 1:34 AM, Shi Weihua <[email protected]> wrote:
> 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) ;-)

    Yes, I did; I tried to apply the patch, almost all of the hunks
failed to apply.

Liubo,
    Please generate a new patch and attach it someone can apply it and test.
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

Reply via email to