[LTP] [PATCH] aio_cancel/3-1: call aio_read before aio_cancel

[LTP] [PATCH] aio_cancel/3-1: call aio_read before aio_cancel

[Kang Kai]

The case calls aio_error() with an invalid aiocbp, but it should do
some asynchronous I/O operations first, so call aio_read() and then
test the aio_error().

Signed-off-by: Kang Kai <kai.kang@windriver.com>
---
 .../conformance/interfaces/aio_error/3-1.c         |    5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

diff --git a/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c b/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
index 8b6c2f5..6665da1 100644
--- a/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
+++ b/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
@@ -68,6 +68,9 @@ int main()
 	aiocb.aio_reqprio = -1;
 	aiocb.aio_nbytes = BUF_SIZE;
 
+	/* Just do aio_read() or aio_write */
+	aio_read(&aiocb);
+
 	ret = aio_error(&aiocb);
 
 	if (ret != EINVAL)
@@ -80,4 +83,4 @@ int main()
 	close(fd);
 	printf ("Test PASSED\n");
 	return PTS_PASS;
-}
\ No newline at end of file
+}
-- 
1.7.5.4


------------------------------------------------------------------------------
Virtualization & Cloud Management Using Capacity Planning
Cloud computing makes use of virtualization - but cloud computing 
also focuses on allowing computing to be delivered as a service.
http://www.accelacomm.com/jaw/sfnl/114/51521223/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Re: [LTP] [PATCH] aio_cancel/3-1: call aio_read before aio_cancel

[Wanlong Gao]

On 02/16/2012 10:41 AM, Kang Kai wrote:

> The case calls aio_error() with an invalid aiocbp, but it should do
> some asynchronous I/O operations first, so call aio_read() and then
> test the aio_error().


NACK. If so, the error is from aio_read but not from aio_error.
This was discussed long time ago, please refer to the mail archive.

This test case is right, but this FAIL is not hard in POSIX,
so we changed the result from FAIL to UNRESOLVED now.

So, just let it UNRESOLVED. ;)

Thanks
-Wanlong Gao

> 
> Signed-off-by: Kang Kai <kai.kang@windriver.com>
> ---
>  .../conformance/interfaces/aio_error/3-1.c         |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c b/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
> index 8b6c2f5..6665da1 100644
> --- a/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
> +++ b/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
> @@ -68,6 +68,9 @@ int main()
>  	aiocb.aio_reqprio = -1;
>  	aiocb.aio_nbytes = BUF_SIZE;
>  
> +	/* Just do aio_read() or aio_write */
> +	aio_read(&aiocb);
> +
>  	ret = aio_error(&aiocb);
>  
>  	if (ret != EINVAL)
> @@ -80,4 +83,4 @@ int main()
>  	close(fd);
>  	printf ("Test PASSED\n");
>  	return PTS_PASS;
> -}
> \ No newline at end of file
> +}



------------------------------------------------------------------------------
Virtualization & Cloud Management Using Capacity Planning
Cloud computing makes use of virtualization - but cloud computing 
also focuses on allowing computing to be delivered as a service.
http://www.accelacomm.com/jaw/sfnl/114/51521223/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

Re: [LTP] [PATCH] aio_cancel/3-1: call aio_read before aio_cancel

[Kang Kai]

On 2012年02月16日 11:25, Wanlong Gao wrote:

Hi WanLong,

> On 02/16/2012 10:41 AM, Kang Kai wrote:
>
>> The case calls aio_error() with an invalid aiocbp, but it should do
>> some asynchronous I/O operations first, so call aio_read() and then
>> test the aio_error().
>
> NACK. If so, the error is from aio_read but not from aio_error.
> This was discussed long time ago, please refer to the mail archive.
>
> This test case is right, but this FAIL is not hard in POSIX,
> so we changed the result from FAIL to UNRESOLVED now.
OK, I got it.
Thanks.

Kai

>
> So, just let it UNRESOLVED. ;)
>
> Thanks
> -Wanlong Gao
>
>> Signed-off-by: Kang Kai<kai.kang@windriver.com>
>> ---
>>   .../conformance/interfaces/aio_error/3-1.c         |    5 ++++-
>>   1 files changed, 4 insertions(+), 1 deletions(-)
>>
>> diff --git a/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c b/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
>> index 8b6c2f5..6665da1 100644
>> --- a/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
>> +++ b/testcases/open_posix_testsuite/conformance/interfaces/aio_error/3-1.c
>> @@ -68,6 +68,9 @@ int main()
>>   	aiocb.aio_reqprio = -1;
>>   	aiocb.aio_nbytes = BUF_SIZE;
>>
>> +	/* Just do aio_read() or aio_write */
>> +	aio_read(&aiocb);
>> +
>>   	ret = aio_error(&aiocb);
>>
>>   	if (ret != EINVAL)
>> @@ -80,4 +83,4 @@ int main()
>>   	close(fd);
>>   	printf ("Test PASSED\n");
>>   	return PTS_PASS;
>> -}
>> \ No newline at end of file
>> +}
>


------------------------------------------------------------------------------
Virtualization & Cloud Management Using Capacity Planning
Cloud computing makes use of virtualization - but cloud computing 
also focuses on allowing computing to be delivered as a service.
http://www.accelacomm.com/jaw/sfnl/114/51521223/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list