On 01/27/2010 12:43 AM, Garrett Cooper wrote:
> 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
>
>
>   
  Hi, Garrett,
  Sorry for replying so late.
  Sure, I'll generate a new patch based on the newest LTP version and
send it then.

  Thanks,
  Liubo

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to