Then we can get rid of the NULLIO flags hack
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
.version = FIO_IOOPS_VERSION,
.init = fio_cpuio_init,
.setup = fio_cpuio_setup,
.version = FIO_IOOPS_VERSION,
.init = fio_cpuio_init,
.setup = fio_cpuio_setup,
- .flags = FIO_CPUIO | FIO_NULLIO,
};
static void fio_init fio_cpuio_register(void)
};
static void fio_init fio_cpuio_register(void)
return FIO_Q_COMPLETED;
}
return FIO_Q_COMPLETED;
}
+static int fio_null_setup(struct thread_data *td)
+{
+ struct fio_file *f;
+ int i;
+
+ if (!td->total_file_size) {
+ log_err("fio: need size= set\n");
+ return 1;
+ }
+
+ td->io_size = td->total_file_size;
+ td->total_io_size = td->io_size;
+
+ for_each_file(td, f, i) {
+ f->fd = dup(STDOUT_FILENO);
+ f->real_file_size = td->total_io_size / td->nr_files;
+ f->file_size = f->real_file_size;
+ }
+
+ td->nr_open_files = td->nr_files;
+ return 0;
+}
+
static struct ioengine_ops ioengine = {
.name = "null",
.version = FIO_IOOPS_VERSION,
static struct ioengine_ops ioengine = {
.name = "null",
.version = FIO_IOOPS_VERSION,
+ .setup = fio_null_setup,
- .flags = FIO_SYNCIO | FIO_NULLIO | FIO_DISKLESSIO,
+ .flags = FIO_SYNCIO | FIO_DISKLESSIO | FIO_SELFOPEN,
};
static void fio_init fio_null_register(void)
};
static void fio_init fio_null_register(void)
- if (td->filetype != FIO_TYPE_FILE || (td->io_ops->flags & FIO_NULLIO))
+ if (td->filetype != FIO_TYPE_FILE)
return 0;
if (lstat(f->file_name, &st) == -1)
return 0;
if (lstat(f->file_name, &st) == -1)
- /*
- * if we are not doing real io, just pretend the file is as large
- * as the size= given. this works fine with nrfiles > 1 as well,
- * we only really care about it being at least as big as size=
- */
- if (td->io_ops->flags & FIO_NULLIO) {
- f->real_file_size = f->file_size = td->total_file_size;
- return 0;
- }
-
if (td->overwrite) {
if (fstat(f->fd, &st) == -1) {
td_verror(td, errno, "fstat");
if (td->overwrite) {
if (fstat(f->fd, &st) == -1) {
td_verror(td, errno, "fstat");
if (td->io_ops->flags & FIO_SELFOPEN)
return 0;
if (td->io_ops->flags & FIO_SELFOPEN)
return 0;
- /*
- * we need a valid file descriptor, but don't create a real file.
- * lets just dup stdout, seems like a sensible approach.
- */
- if (td->io_ops->flags & FIO_NULLIO)
- f->fd = dup(STDOUT_FILENO);
- else {
- if (td->odirect)
- flags |= OS_O_DIRECT;
- if (td->sync_io)
- flags |= O_SYNC;
-
- if (td_write(td) || td_rw(td)) {
- flags |= O_RDWR;
-
- if (td->filetype == FIO_TYPE_FILE) {
- if (!td->overwrite)
- flags |= O_TRUNC;
+ if (td->odirect)
+ flags |= OS_O_DIRECT;
+ if (td->sync_io)
+ flags |= O_SYNC;
+ if (td_write(td) || td_rw(td)) {
+ flags |= O_RDWR;
- open_file(td, f, flags, 0600);
- } else {
- if (td->filetype == FIO_TYPE_CHAR)
- flags |= O_RDWR;
- else
- flags |= O_RDONLY;
+ if (td->filetype == FIO_TYPE_FILE) {
+ if (!td->overwrite)
+ flags |= O_TRUNC;
- open_file(td, f, flags, 0);
+
+ open_file(td, f, flags, 0600);
+ } else {
+ if (td->filetype == FIO_TYPE_CHAR)
+ flags |= O_RDWR;
+ else
+ flags |= O_RDONLY;
+
+ open_file(td, f, flags, 0);
FIO_RAWIO = 1 << 3, /* some sort of direct/raw io */
FIO_DISKLESSIO = 1 << 4, /* no disk involved */
FIO_SELFOPEN = 1 << 5, /* opens its own devices */
FIO_RAWIO = 1 << 3, /* some sort of direct/raw io */
FIO_DISKLESSIO = 1 << 4, /* no disk involved */
FIO_SELFOPEN = 1 << 5, /* opens its own devices */
- FIO_NULLIO = 1 << 6, /* no real data transfer (cpu/null) */