Fix file unlinking
authorJens Axboe <jens.axboe@oracle.com>
Thu, 19 Apr 2007 14:31:16 +0000 (16:31 +0200)
committerJens Axboe <jens.axboe@oracle.com>
Thu, 19 Apr 2007 14:31:16 +0000 (16:31 +0200)
We never set FIO_FILE_UNLINK. Just unlink if it's a regular
file and unlink=1 was given.

Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
filesetup.c
fio.h

index faeefaf308fa3e32107375fed6ac68faf65fe594..d79fc61081278c934fd1c4aecbfe344666c75811 100644 (file)
@@ -474,8 +474,7 @@ void close_files(struct thread_data *td)
        unsigned int i;
 
        for_each_file(td, f, i) {
-               if ((f->flags & FIO_FILE_UNLINK) &&
-                   f->filetype == FIO_TYPE_FILE)
+               if (td->o.unlink && f->filetype == FIO_TYPE_FILE)
                        unlink(f->file_name);
 
                td_io_close_file(td, f);
diff --git a/fio.h b/fio.h
index 92264d998752900730db96bb870af2b591b4afe3..f9f2a577836554c52ac7b9e816acea6aef1384b2 100644 (file)
--- a/fio.h
+++ b/fio.h
@@ -238,13 +238,12 @@ enum fio_ioengine_flags {
 
 enum fio_file_flags {
        FIO_FILE_OPEN           = 1 << 0,       /* file is open */
-       FIO_FILE_UNLINK         = 1 << 1,       /* unlink on close */
-       FIO_FILE_CLOSING        = 1 << 2,       /* file being closed */
-       FIO_FILE_EXISTS         = 1 << 3,       /* file there */
-       FIO_FILE_EXTEND         = 1 << 4,       /* needs extend */
-       FIO_FILE_NOSORT         = 1 << 5,       /* don't sort verify blocks */
-       FIO_FILE_DONE           = 1 << 6,       /* io completed to this file */
-       FIO_SIZE_KNOWN          = 1 << 7,       /* size has been set */
+       FIO_FILE_CLOSING        = 1 << 1,       /* file being closed */
+       FIO_FILE_EXISTS         = 1 << 2,       /* file there */
+       FIO_FILE_EXTEND         = 1 << 3,       /* needs extend */
+       FIO_FILE_NOSORT         = 1 << 4,       /* don't sort verify blocks */
+       FIO_FILE_DONE           = 1 << 5,       /* io completed to this file */
+       FIO_SIZE_KNOWN          = 1 << 6,       /* size has been set */
 };
 
 /*