Re: [LTP] [PATCH] aio02: remove test for aio_fsync() and aio_fdsync()

[Jan S. <jst...@re...>]

----- Original Message -----
> From: "Wanlong Gao" <gao...@cn...>
> To: "LTP" <ltp...@li...>
> Cc: "Cyril Hrubis" <ch...@su...>, "Caspar Zhang" <ca...@ca...>, "Garrett Cooper" <yan...@gm...>,
> "Mike Frysinger" <va...@ge...>, jst...@re..., "Wanlong Gao" <gao...@cn...>
> Sent: Thursday, 27 June, 2013 10:38:10 AM
> Subject: [PATCH] aio02: remove test for aio_fsync() and aio_fdsync()
> 
> Linux filesystems doesn't support aio_fsync() and aio_fdsync()
> operations, so just remove test for these two.
> 
> Signed-off-by: Wanlong Gao <gao...@cn...>

Reviewed-by: Jan Stancek <jst...@re...>

Looks good to me.

Regards,
Jan


> ---
>  testcases/kernel/io/aio/aio02/aio_tio.c | 34
>  ---------------------------------
>  1 file changed, 34 deletions(-)
> 
> diff --git a/testcases/kernel/io/aio/aio02/aio_tio.c
> b/testcases/kernel/io/aio/aio02/aio_tio.c
> index 9dda462..34fa9ed 100644
> --- a/testcases/kernel/io/aio/aio02/aio_tio.c
> +++ b/testcases/kernel/io/aio/aio02/aio_tio.c
> @@ -111,8 +111,6 @@ int io_tio(char *pathname, int flag, int n, int
> operation)
>  	for (i = 0; i < AIO_MAXIO; i++) {
>  
>  		switch (operation) {
> -		case IO_CMD_FSYNC:
> -		case IO_CMD_FDSYNC:
>  		case IO_CMD_PWRITE:
>  			if (posix_memalign(&bufptr, alignment, AIO_BLKSIZE)) {
>  				perror(" posix_memalign failed ");
> @@ -179,22 +177,6 @@ int io_tio(char *pathname, int flag, int n, int
> operation)
>  			}
>  		}
>  		break;
> -	case IO_CMD_FSYNC:
> -		for (i = 0; i < AIO_MAXIO; i++) {
> -			res = io_fsync(myctx, iocbps[i], work_done, fd);
> -			if (res < 0) {
> -				io_error("io_fsync write", res);
> -			}
> -		}
> -		break;
> -	case IO_CMD_FDSYNC:
> -		for (i = 0; i < AIO_MAXIO; i++) {
> -			res = io_fdsync(myctx, iocbps[i], work_done, fd);
> -			if (res < 0) {
> -				io_error("io_fsync write", res);
> -			}
> -		}
> -		break;
>  	}
>  
>  	close(fd);
> @@ -253,22 +235,6 @@ int test_main(void)
>  		return status;
>  	}
>  
> -	tst_resm(TINFO, "Running test 7 \n");
> -	status = io_tio("file2",
> -			O_TRUNC | O_DIRECT | O_WRONLY | O_CREAT | O_LARGEFILE,
> -			AIO_MAXIO, IO_CMD_FSYNC);
> -	if (status) {
> -		return status;
> -	}
> -
> -	tst_resm(TINFO, "Running test 8 \n");
> -	status = io_tio("file2",
> -			O_TRUNC | O_DIRECT | O_WRONLY | O_CREAT | O_LARGEFILE,
> -			AIO_MAXIO, IO_CMD_FDSYNC);
> -	if (status) {
> -		return status;
> -	}
> -
>  	return status;
>  }
>  
> --
> 1.8.3.1.448.gfb7dfaa
> 
>