----- Original Message -----
> From: "Wanlong Gao" <[email protected]>
> To: "LTP" <[email protected]>
> Cc: "Cyril Hrubis" <[email protected]>, "Caspar Zhang" 
> <[email protected]>, "Garrett Cooper" <[email protected]>,
> "Mike Frysinger" <[email protected]>, [email protected], "Wanlong Gao" 
> <[email protected]>
> 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 <[email protected]>

Reviewed-by: Jan Stancek <[email protected]>

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
> 
> 

------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
Ltp-list mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ltp-list

Reply via email to