Re: [LTP] [POSIX][PATCH]Some building error at pi_test cases.

Garrett Cooper
2010-11-02T02:01:13Z

Will be committed momentarily.
Thanks!
-Garrett

On Mon, Nov 1, 2010 at 6:09 PM, Bian Naimeng <biannm@cn.fujitsu.com> wrote:
> Hi Garrett,
>
>  Would you like to pick up this patch?  ^_^
>
> Best Regards
>  Bian
>
>> Commit 5f2ba92bdbef089bad74906fd8a86b54617f7878 make some building error.
>>
>> Signed-off-by: Bian Naimeng <biannm@cn.fujitsu.com>
>>
>> ---
>>  .../functional/threads/pi_test/pitest-2.c          |    2 +-
>>  .../functional/threads/pi_test/pitest-3.c          |    4 ++--
>>  .../functional/threads/pi_test/pitest-4.c          |    4 ++--
>>  .../functional/threads/pi_test/pitest-6.c          |    4 ++--
>>  4 files changed, 7 insertions(+), 7 deletions(-)
>>
>> diff --git a/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-2.c b/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-2.c
>> index 62f1f45..0a96377 100644
>> --- a/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-2.c
>> +++ b/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-2.c
>> @@ -235,7 +235,7 @@ void *thread_tb2(void *arg)
>>       DPRINTF(stdout, "#EVENT %f TB2 Thread Started\n",
>>               seconds_read() - base_time);
>>
>> -     boost_time.tv_sec = time(NULL) + *(time_t*)timeoutsec;
>> +     boost_time.tv_sec = time(NULL) + *(time_t*)arg;
>>       boost_time.tv_nsec = 0;
>>
>>       t0 = seconds_read();
>> diff --git a/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-3.c b/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-3.c
>> index fe0c886..58d7a4a 100644
>> --- a/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-3.c
>> +++ b/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-3.c
>> @@ -219,7 +219,7 @@ void *thread_tb1(void *arg)
>>       DPRINTF(stdout, "#EVENT %f Thread TB1 Started\n",
>>               seconds_read() - base_time);
>>
>> -     boost_time.tv_sec = time(NULL) + *(time_t *)timeoutsec;
>> +     boost_time.tv_sec = time(NULL) + *(time_t *)arg;
>>       boost_time.tv_nsec = 0;
>>
>>       t0 = seconds_read();
>> @@ -247,7 +247,7 @@ void *thread_tb2(void *arg)
>>       DPRINTF(stdout, "#EVENT %f Thread TB2 Started\n",
>>               seconds_read() - base_time);
>>
>> -     boost_time.tv_sec = time(NULL) + *(time_t *)timeoutsec;
>> +     boost_time.tv_sec = time(NULL) + *(time_t *)arg;
>>       boost_time.tv_nsec = 0;
>>       t0 = seconds_read();
>>       rc = pthread_mutex_timedlock(&mutex2, &boost_time);
>> diff --git a/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-4.c b/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-4.c
>> index a042dbf..077f966 100644
>> --- a/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-4.c
>> +++ b/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-4.c
>> @@ -187,7 +187,7 @@ void *thread_tb1(void *arg)
>>
>>       pthread_mutex_lock(&mutex2);
>>
>> -     boost_time.tv_sec = time(NULL) + *(time_t *)timeoutsec;
>> +     boost_time.tv_sec = time(NULL) + *(time_t *)arg;
>>       boost_time.tv_nsec = 0;
>>       t0 = seconds_read();
>>       rc = pthread_mutex_timedlock(&mutex1, &boost_time);
>> @@ -218,7 +218,7 @@ void *thread_tb2(void *arg)
>>       DPRINTF(stdout, "#EVENT %f Thread TB2 Started\n",
>>               seconds_read() - base_time);
>>
>> -     boost_time.tv_sec = time(NULL) + *(time_t *)timeoutsec;
>> +     boost_time.tv_sec = time(NULL) + *(time_t *)arg;
>>       boost_time.tv_nsec = 0;
>>
>>       t0 = seconds_read();
>> diff --git a/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-6.c b/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-6.c
>> index 6ebb003..2f2272b 100644
>> --- a/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-6.c
>> +++ b/testcases/open_posix_testsuite/functional/threads/pi_test/pitest-6.c
>> @@ -204,9 +204,9 @@ void *thread_tb(void *arg)
>>       DPRINTF(stderr,"Thread TB: started\n");
>>
>>       DPRINTF(stdout, "#EVENT %f TB Started, waiting for mutex for %lu s\n",
>> -             seconds_read() - base_time, timeoutsec);
>> +             seconds_read() - base_time, *(unsigned long*)arg);
>>
>> -     boost_time.tv_sec = time(NULL) + *(time_t *)timeoutsec;
>> +     boost_time.tv_sec = time(NULL) + *(time_t *)arg;
>>       boost_time.tv_nsec = 0;
>>
>>       t0 = seconds_read();
>
> --
> Regards
> Bian Naimeng
>
>