On 2012年08月10日 18:22, Jan Stancek wrote:
> Hi,
Hi Jan,
Thanks for your reply.
> Here's a different proposal: don't count how many pthread_cond_signals it
> took,
> if any fails SIGALRM will trigger testcase failure.
That is a more simple solution, and I agree with you.
Regards,
Kai
>
> diff --git
> a/testcases/open_posix_testsuite/conformance/interfaces/pthread_cond_signal/1-1.c
>
> b/testcases/open_posix_testsuite/conformance/interfaces/pthread_cond_signal/1-1.c
> index bf55b31..266d42f 100644
> ---
> a/testcases/open_posix_testsuite/conformance/interfaces/pthread_cond_signal/1-1.c
> +++
> b/testcases/open_posix_testsuite/conformance/interfaces/pthread_cond_signal/1-1.c
> @@ -134,9 +134,7 @@ int main()
> alarm(5);
>
> /* loop to wake up the rest threads */
> - i=0;
> - while (waken_num< THREAD_NUM) {
> - ++i;
> + for (i=1; i<THREAD_NUM; i++) {
> fprintf(stderr,"[Main thread] signals to wake up the next
> thread\n");
> if (pthread_cond_signal(&td.cond) != 0) {
> fprintf(stderr,"Main failed to signal the condition\n");
> @@ -145,12 +143,6 @@ int main()
> usleep(100);
> }
>
> - if (i>= THREAD_NUM) {
> - fprintf(stderr,"[Main thread] had to signal the condition %i
> times\n", i+1);
> - fprintf(stderr,"[Main thread] to wake up %i threads\n. Test
> FAILED.\n", THREAD_NUM);
> - exit(PTS_FAIL);
> - }
> -
> /* join all secondary threads */
> for (i=0; i<THREAD_NUM; i++) {
> if (pthread_join(thread[i], NULL) != 0) {
>
> Regards,
> Jan
>
> On 08/09/2012 12:30 PM, Kang Kai wrote:
>> This case fails on mips board routerstation randomly. The root cause is
>> that when main thread call usleep(100) after signal the child threads,
>> no child thread is scheduled to run. So the case fails.
>>
>> I update it to set main thread with a lower priority and child threads
>> with a higher one, then call sched_yield and usleep in main thread will
>> make sure that one of the child threads will be scheduled.
>>
>> Signed-off-by: Kang Kai<[email protected]>
>> ---
>> .../interfaces/pthread_cond_signal/1-1.c | 44
>> +++++++++++++++++++-
>> 1 files changed, 42 insertions(+), 2 deletions(-)
>>
>> diff --git
>> a/testcases/open_posix_testsuite/conformance/interfaces/pthread_cond_signal/1-1.c
>>
>> b/testcases/open_posix_testsuite/conformance/interfaces/pthread_cond_signal/1-1.c
>> index bf55b31..77374bf 100644
>> ---
>> a/testcases/open_posix_testsuite/conformance/interfaces/pthread_cond_signal/1-1.c
>> +++
>> b/testcases/open_posix_testsuite/conformance/interfaces/pthread_cond_signal/1-1.c
>> @@ -12,6 +12,7 @@
>>
>> #define _XOPEN_SOURCE 600
>>
>> +#include<errno.h>
>> #include<pthread.h>
>> #include<stdio.h>
>> #include<stdlib.h>
>> @@ -20,6 +21,7 @@
>> #include "posixtest.h"
>>
>> #define THREAD_NUM 3
>> +#define SCHED_POLICY SCHED_FIFO
>>
>> struct testdata
>> {
>> @@ -77,6 +79,10 @@ int main()
>> {
>> int i, rc;
>> struct sigaction act;
>> + int pid;
>> + int policy;
>> + struct sched_param param;
>> + pthread_attr_t attr;
>>
>> if (pthread_mutex_init(&td.mutex, NULL) != 0) {
>> fprintf(stderr,"Fail to initialize mutex\n");
>> @@ -87,8 +93,35 @@ int main()
>> return PTS_UNRESOLVED;
>> }
>>
>> + pid = getpid();
>> + param.sched_priority = sched_get_priority_min(SCHED_POLICY);
>> + if (sched_setscheduler(pid, SCHED_POLICY,¶m) != 0) {
>> + fprintf(stderr, "Fail to set main thread scheduler.\n");
>> + if (errno == EPERM)
>> + fprintf(stderr, "Failed with EPERM: are you root?\n");
>> + return PTS_UNRESOLVED;
>> + }
>> +
>> + if (pthread_attr_init(&attr) != 0) {
>> + fprintf(stderr, "Fail to init child thread attribute.\n");
>> + return PTS_UNRESOLVED;
>> + }
>> + if (pthread_attr_setschedpolicy(&attr, SCHED_POLICY) != 0) {
>> + fprintf(stderr, "Fail to set child thread schedule policy
>> attribute.\n");
>> + return PTS_UNRESOLVED;
>> + }
>> + param.sched_priority += 10;
>> + if (pthread_attr_setschedparam(&attr,¶m) != 0) {
>> + fprintf(stderr, "Fail to set child thread schedule priority
>> attribute.\n");
>> + return PTS_UNRESOLVED;
>> + }
>> + if (pthread_attr_setinheritsched(&attr, PTHREAD_EXPLICIT_SCHED) != 0) {
>> + fprintf(stderr, "Fail to set child thread inheritace
>> attribute.\n");
>> + return PTS_UNRESOLVED;
>> + }
>> +
>> for (i=0; i<THREAD_NUM; i++) { /* create THREAD_NUM threads */
>> - if (pthread_create(&thread[i], NULL, thr_func, NULL) != 0) {
>> + if (pthread_create(&thread[i],&attr, thr_func, NULL) != 0) {
>> fprintf(stderr,"Fail to create thread[%d]\n", i);
>> exit(PTS_UNRESOLVED);
>> }
>> @@ -96,6 +129,8 @@ int main()
>> while (start_num< THREAD_NUM) /* waiting for all threads started */
>> usleep(100);
>>
>> + pthread_attr_destroy(&attr);
>> +
>> /* Acquire the mutex to make sure that all waiters are currently
>> blocked on pthread_cond_wait */
>> if (pthread_mutex_lock(&td.mutex) != 0) {
>> @@ -142,6 +177,7 @@ int main()
>> fprintf(stderr,"Main failed to signal the condition\n");
>> exit(PTS_UNRESOLVED);
>> }
>> + sched_yield();
>> usleep(100);
>> }
>>
>> @@ -158,6 +194,10 @@ int main()
>> exit(PTS_UNRESOLVED);
>> }
>> }
>> +
>> + pthread_mutex_destroy(&td.mutex);
>> + pthread_cond_destroy(&td.cond);
>> +
>> printf("Test PASSED\n");
>> return PTS_PASS;
>> -}
>> \ No newline at end of file
>> +}
>
> ------------------------------------------------------------------------------
> Live Security Virtual Conference
> Exclusive live event will cover all the ways today's security and
> threat landscape has changed and how IT managers can respond. Discussions
> will include endpoint security, mobile security and the latest in malware
> threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
> _______________________________________________
> Ltp-list mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ltp-list
>
------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and
threat landscape has changed and how IT managers can respond. Discussions
will include endpoint security, mobile security and the latest in malware
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list
