Re: [LTP] [PATCH] pthread_detach/4-3: workaround for segment fault

[Wanlong G. <gao...@cn...>]

On 08/20/2012 09:39 AM, Kang Kai wrote:
> On 2012年08月19日 06:58, Wanlong Gao wrote:
>> On 08/14/2012 05:00 PM, Kang Kai wrote:
>>> This case fails randomly with "Segmentation fault". Reset VERBOSE to 6
>>> can avoid this problem.
>> Can you explain that why "reset VERBOSE" can avoid this "Seg Fault" problem?
> It is a multi-thread issue. You can run it for less than 10 times, and it will fail with "Segmentation fault".

I didn't see this on my side.

> It is hard to debug and I didn't find the root cause, but open the VERBOSE will avoid this situation. So it is a workaround.

I'd like to send a patch to fix this typo first.

Thanks,
Wanlong Gao

> 
> Regards,
> Kai
> 
>>
>>> Fix typo at same time.
>> The typo fix is OK.
>>
>> Thanks,
>> Wanlong Gao
>>
>>> Signed-off-by: Kang Kai<kai...@wi...>
>>> ---
>>>   .../conformance/interfaces/pthread_detach/4-3.c    |    4 ++--
>>>   1 files changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/testcases/open_posix_testsuite/conformance/interfaces/pthread_detach/4-3.c b/testcases/open_posix_testsuite/conformance/interfaces/pthread_detach/4-3.c
>>> index 5c15e93..23738dc 100644
>>> --- a/testcases/open_posix_testsuite/conformance/interfaces/pthread_detach/4-3.c
>>> +++ b/testcases/open_posix_testsuite/conformance/interfaces/pthread_detach/4-3.c
>>> @@ -49,7 +49,7 @@
>>>   #include "../testfrmw/testfrmw.c"
>>>
>>>   #ifndef VERBOSE
>>> -#define VERBOSE 1
>>> +#define VERBOSE 6
>>>   #endif
>>>
>>>   #define WITH_SYNCHRO
>>> @@ -281,7 +281,7 @@ int main(int argc, char *argv[])
>>>
>>>       do {
>>>           do_it2 = 0;
>>> -    } while (do_it2 = 0);
>>> +    } while (do_it2);
>>>
>>>       ret = pthread_join(th_sig1, NULL);
>>>       if (ret)
>>>
>>
> 
>