recurse_dir(): Move . and .. check earlier

[fio.git] / filesetup.c

diff --git a/filesetup.c b/filesetup.c
index 0a4e77ef33e3aef63b597cc9af118d19b54a0fa9..0f1a3402237a265d2480b5e865837938959ad6c1 100644 (file)
--- a/filesetup.c
+++ b/filesetup.c
@@ -101,14 +101,25 @@ static int create_files(struct thread_data *td)
 {
        struct fio_file *f;
        int err, need_create, can_extend;
+       unsigned long long total_file_size;
        unsigned int i;
 
+       total_file_size = td->total_file_size;
        for_each_file(td, f, i) {
+               unsigned long long s;
+
+               f->file_offset = td->start_offset;
+
                if (f->filetype != FIO_TYPE_FILE)
                        continue;
 
-               f->file_size = td->total_file_size / td->nr_normal_files;
-               f->file_offset = td->start_offset;
+               if (f->flags & FIO_FILE_EXISTS) {
+                       s = f->file_size;
+                       if (s > total_file_size)
+                               s = total_file_size;
+
+                       total_file_size -= s;
+               }
        }
 
        /*
@@ -124,6 +135,10 @@ static int create_files(struct thread_data *td)
 
                if (f->filetype != FIO_TYPE_FILE)
                        continue;
+               if (f->flags & FIO_FILE_EXISTS)
+                       continue;
+
+               f->file_size = total_file_size / td->nr_normal_files;
 
                file_there = !file_ok(td, f);
 
@@ -217,9 +232,10 @@ static int get_file_size(struct thread_data *td, struct fio_file *f)
 {
        int ret = 0;
 
-       if (f->filetype == FIO_TYPE_FILE)
-               ret = file_size(td, f);
-       else if (f->filetype == FIO_TYPE_BD)
+       if (f->filetype == FIO_TYPE_FILE) {
+               if (!(f->flags & FIO_FILE_EXISTS))
+                       ret = file_size(td, f);
+       } else if (f->filetype == FIO_TYPE_BD)
                ret = bdev_size(td, f);
        else
                f->real_file_size = -1;
@@ -299,6 +315,8 @@ int generic_open_file(struct thread_data *td, struct fio_file *f)
                td_verror(td, __e, "open");
                if (__e == EINVAL && td->odirect)
                        log_err("fio: destination does not support O_DIRECT\n");
+               if (__e == EMFILE)
+                       log_err("fio: try reducing/setting openfiles (failed at %u of %u)\n", td->nr_open_files, td->nr_files);
                return 1;
        }
 
@@ -427,10 +445,19 @@ static void get_file_type(struct fio_file *f)
        f->filetype = FIO_TYPE_FILE;
 
        if (!lstat(f->file_name, &sb)) {
+               f->flags |= FIO_FILE_EXISTS;
+
                if (S_ISBLK(sb.st_mode))
                        f->filetype = FIO_TYPE_BD;
                else if (S_ISCHR(sb.st_mode))
                        f->filetype = FIO_TYPE_CHAR;
+               else {
+                       /*
+                        * might as well do this here, and save a stat later on
+                        */
+                       f->real_file_size = sb.st_size;
+                       f->file_size = f->real_file_size;
+               }
        }
 }
 
@@ -492,6 +519,9 @@ static int recurse_dir(struct thread_data *td, const char *dirname)
                char full_path[PATH_MAX];
                struct stat sb;
 
+               if (!strcmp(dir->d_name, ".") || !strcmp(dir->d_name, ".."))
+                       continue;
+
                sprintf(full_path, "%s/%s", dirname, dir->d_name);
 
                if (lstat(full_path, &sb) == -1) {
@@ -507,9 +537,6 @@ static int recurse_dir(struct thread_data *td, const char *dirname)
                        continue;
                }
 
-               if (!strcmp(dir->d_name, ".") || !strcmp(dir->d_name, ".."))
-                       continue;
-
                if ((ret = recurse_dir(td, full_path)) != 0)
                        break;
        }