projects
/
fio.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Style fixups
[fio.git]
/
ioengines.c
diff --git
a/ioengines.c
b/ioengines.c
index 9aacc335d906b21b4b5c397c8d26049338bc1b42..e44fce08b51065cf4d6ad9feaf87f6878ee4ff25 100644
(file)
--- a/
ioengines.c
+++ b/
ioengines.c
@@
-23,7
+23,8
@@
static LIST_HEAD(engine_list);
static int check_engine_ops(struct ioengine_ops *ops)
{
if (ops->version != FIO_IOOPS_VERSION) {
static int check_engine_ops(struct ioengine_ops *ops)
{
if (ops->version != FIO_IOOPS_VERSION) {
- log_err("bad ioops version %d (want %d)\n", ops->version, FIO_IOOPS_VERSION);
+ log_err("bad ioops version %d (want %d)\n", ops->version,
+ FIO_IOOPS_VERSION);
return 1;
}
return 1;
}
@@
-37,7
+38,7
@@
static int check_engine_ops(struct ioengine_ops *ops)
*/
if (ops->flags & FIO_SYNCIO)
return 0;
*/
if (ops->flags & FIO_SYNCIO)
return 0;
-
+
if (!ops->event) {
log_err("%s: no event handler\n", ops->name);
return 1;
if (!ops->event) {
log_err("%s: no event handler\n", ops->name);
return 1;
@@
-50,18
+51,20
@@
static int check_engine_ops(struct ioengine_ops *ops)
log_err("%s: no queue handler\n", ops->name);
return 1;
}
log_err("%s: no queue handler\n", ops->name);
return 1;
}
-
+
return 0;
}
void unregister_ioengine(struct ioengine_ops *ops)
{
return 0;
}
void unregister_ioengine(struct ioengine_ops *ops)
{
+ dprint(FD_IO, "ioengine %s unregistered\n", ops->name);
list_del(&ops->list);
INIT_LIST_HEAD(&ops->list);
}
void register_ioengine(struct ioengine_ops *ops)
{
list_del(&ops->list);
INIT_LIST_HEAD(&ops->list);
}
void register_ioengine(struct ioengine_ops *ops)
{
+ dprint(FD_IO, "ioengine %s registered\n", ops->name);
INIT_LIST_HEAD(&ops->list);
list_add_tail(&ops->list, &engine_list);
}
INIT_LIST_HEAD(&ops->list);
list_add_tail(&ops->list, &engine_list);
}
@@
-86,6
+89,8
@@
static struct ioengine_ops *dlopen_ioengine(struct thread_data *td,
struct ioengine_ops *ops;
void *dlhandle;
struct ioengine_ops *ops;
void *dlhandle;
+ dprint(FD_IO, "dload engine %s\n", engine_lib);
+
dlerror();
dlhandle = dlopen(engine_lib, RTLD_LAZY);
if (!dlhandle) {
dlerror();
dlhandle = dlopen(engine_lib, RTLD_LAZY);
if (!dlhandle) {
@@
-113,6
+118,8
@@
struct ioengine_ops *load_ioengine(struct thread_data *td, const char *name)
struct ioengine_ops *ops, *ret;
char engine[16];
struct ioengine_ops *ops, *ret;
char engine[16];
+ dprint(FD_IO, "load ioengine %s\n", name);
+
strncpy(engine, name, sizeof(engine) - 1);
/*
strncpy(engine, name, sizeof(engine) - 1);
/*
@@
-145,6
+152,8
@@
struct ioengine_ops *load_ioengine(struct thread_data *td, const char *name)
void close_ioengine(struct thread_data *td)
{
void close_ioengine(struct thread_data *td)
{
+ dprint(FD_IO, "close ioengine %s\n", td->io_ops->name);
+
if (td->io_ops->cleanup)
td->io_ops->cleanup(td);
if (td->io_ops->cleanup)
td->io_ops->cleanup(td);
@@
-157,10
+166,19
@@
void close_ioengine(struct thread_data *td)
int td_io_prep(struct thread_data *td, struct io_u *io_u)
{
int td_io_prep(struct thread_data *td, struct io_u *io_u)
{
+ dprint_io_u(io_u, "prep");
fio_ro_check(td, io_u);
fio_ro_check(td, io_u);
- if (td->io_ops->prep)
- return td->io_ops->prep(td, io_u);
+ lock_file(td, io_u->file, io_u->ddir);
+
+ if (td->io_ops->prep) {
+ int ret = td->io_ops->prep(td, io_u);
+
+ dprint(FD_IO, "->prep(%p)=%d\n", io_u, ret);
+ if (ret)
+ unlock_file(td, io_u->file);
+ return ret;
+ }
return 0;
}
return 0;
}
@@
-168,22
+186,27
@@
int td_io_prep(struct thread_data *td, struct io_u *io_u)
int td_io_getevents(struct thread_data *td, unsigned int min, unsigned int max,
struct timespec *t)
{
int td_io_getevents(struct thread_data *td, unsigned int min, unsigned int max,
struct timespec *t)
{
- if (min > 0 && td->io_ops->commit) {
- int r = td->io_ops->commit(td);
+ int r = 0;
+ if (min > 0 && td->io_ops->commit) {
+ r = td->io_ops->commit(td);
if (r < 0)
if (r < 0)
-
return r
;
+
goto out
;
}
}
- if (td->io_ops->getevents)
- return td->io_ops->getevents(td, min, max, t);
- return 0;
+ r = 0;
+ if (td->io_ops->getevents)
+ r = td->io_ops->getevents(td, min, max, t);
+out:
+ dprint(FD_IO, "getevents: %d\n", r);
+ return r;
}
int td_io_queue(struct thread_data *td, struct io_u *io_u)
{
int ret;
}
int td_io_queue(struct thread_data *td, struct io_u *io_u)
{
int ret;
+ dprint_io_u(io_u, "queue");
fio_ro_check(td, io_u);
assert((io_u->flags & IO_U_F_FLIGHT) == 0);
fio_ro_check(td, io_u);
assert((io_u->flags & IO_U_F_FLIGHT) == 0);
@@
-196,22
+219,27
@@
int td_io_queue(struct thread_data *td, struct io_u *io_u)
if (td->io_ops->flags & FIO_SYNCIO) {
fio_gettime(&io_u->issue_time, NULL);
if (td->io_ops->flags & FIO_SYNCIO) {
fio_gettime(&io_u->issue_time, NULL);
- memcpy(&td->last_issue, &io_u->issue_time, sizeof(struct timeval));
+ memcpy(&td->last_issue, &io_u->issue_time,
+ sizeof(struct timeval));
/*
* for a sync engine, set the timeout upfront
*/
/*
* for a sync engine, set the timeout upfront
*/
- if (mtime_since(&td->timeout_end, &io_u->issue_time) < IO_U_TIMEOUT)
+ if (mtime_since(&td->timeout_end, &io_u->issue_time)
+ < IO_U_TIMEOUT)
io_u_set_timeout(td);
}
if (io_u->ddir != DDIR_SYNC)
td->io_issues[io_u->ddir]++;
io_u_set_timeout(td);
}
if (io_u->ddir != DDIR_SYNC)
td->io_issues[io_u->ddir]++;
- io_u_mark_depth(td, io_u);
-
ret = td->io_ops->queue(td, io_u);
ret = td->io_ops->queue(td, io_u);
+ unlock_file(td, io_u->file);
+
+ if (ret != FIO_Q_BUSY)
+ io_u_mark_depth(td, io_u);
+
if (ret == FIO_Q_QUEUED) {
int r;
if (ret == FIO_Q_QUEUED) {
int r;
@@
-225,14
+253,17
@@
int td_io_queue(struct thread_data *td, struct io_u *io_u)
if ((td->io_ops->flags & FIO_SYNCIO) == 0) {
fio_gettime(&io_u->issue_time, NULL);
if ((td->io_ops->flags & FIO_SYNCIO) == 0) {
fio_gettime(&io_u->issue_time, NULL);
- memcpy(&td->last_issue, &io_u->issue_time, sizeof(struct timeval));
+ memcpy(&td->last_issue, &io_u->issue_time,
+ sizeof(struct timeval));
/*
* async engine, set the timeout here
*/
if (ret == FIO_Q_QUEUED &&
/*
* async engine, set the timeout here
*/
if (ret == FIO_Q_QUEUED &&
- mtime_since(&td->timeout_end, &io_u->issue_time) < IO_U_TIMEOUT)
+ (mtime_since(&td->timeout_end, &io_u->issue_time)
+ < IO_U_TIMEOUT)) {
io_u_set_timeout(td);
io_u_set_timeout(td);
+ }
}
return ret;
}
return ret;
@@
-244,8
+275,10
@@
int td_io_init(struct thread_data *td)
if (td->io_ops->init) {
ret = td->io_ops->init(td);
if (td->io_ops->init) {
ret = td->io_ops->init(td);
- if (ret && td->o.iodepth > 1)
- log_err("fio: io engine init failed. Perhaps try reducing io depth?\n");
+ if (ret && td->o.iodepth > 1) {
+ log_err("fio: io engine init failed. Perhaps try"
+ " reducing io depth?\n");
+ }
}
return ret;
}
return ret;
@@
-253,6
+286,8
@@
int td_io_init(struct thread_data *td)
int td_io_commit(struct thread_data *td)
{
int td_io_commit(struct thread_data *td)
{
+ dprint(FD_IO, "calling ->commit(), depth %d\n", td->cur_depth);
+
if (!td->cur_depth)
return 0;
if (!td->cur_depth)
return 0;
@@
-268,8
+303,11
@@
int td_io_open_file(struct thread_data *td, struct fio_file *f)
if (td->io_ops->open_file(td, f)) {
if (td->error == EINVAL && td->o.odirect)
log_err("fio: destination does not support O_DIRECT\n");
if (td->io_ops->open_file(td, f)) {
if (td->error == EINVAL && td->o.odirect)
log_err("fio: destination does not support O_DIRECT\n");
- if (td->error == EMFILE)
- log_err("fio: try reducing/setting openfiles (failed at %u of %u)\n", td->nr_open_files, td->o.nr_files);
+ if (td->error == EMFILE) {
+ log_err("fio: try reducing/setting openfiles (failed"
+ " at %u of %u)\n", td->nr_open_files,
+ td->o.nr_files);
+ }
return 1;
}
return 1;
}
@@
-282,7
+320,6
@@
int td_io_open_file(struct thread_data *td, struct fio_file *f)
}
f->last_free_lookup = 0;
}
f->last_free_lookup = 0;
- f->last_completed_pos = 0;
f->last_pos = f->file_offset;
f->flags |= FIO_FILE_OPEN;
f->flags &= ~FIO_FILE_CLOSING;
f->last_pos = f->file_offset;
f->flags |= FIO_FILE_OPEN;
f->flags &= ~FIO_FILE_CLOSING;
@@
-295,7
+332,6
@@
int td_io_open_file(struct thread_data *td, struct fio_file *f)
if (td->o.fadvise_hint &&
(f->filetype == FIO_TYPE_BD || f->filetype == FIO_TYPE_FILE)) {
if (td->o.fadvise_hint &&
(f->filetype == FIO_TYPE_BD || f->filetype == FIO_TYPE_FILE)) {
-
int flags;
if (td_random(td))
int flags;
if (td_random(td))
@@
-323,7
+359,7
@@
err:
return 1;
}
return 1;
}
-
void
td_io_close_file(struct thread_data *td, struct fio_file *f)
+
int
td_io_close_file(struct thread_data *td, struct fio_file *f)
{
if (!(f->flags & FIO_FILE_CLOSING))
log_file(td, f, FIO_LOG_CLOSE_FILE);
{
if (!(f->flags & FIO_FILE_CLOSING))
log_file(td, f, FIO_LOG_CLOSE_FILE);
@@
-333,5
+369,7
@@
void td_io_close_file(struct thread_data *td, struct fio_file *f)
*/
f->flags |= FIO_FILE_CLOSING;
*/
f->flags |= FIO_FILE_CLOSING;
- put_file(td, f);
+ unlock_file_all(td, f);
+
+ return put_file(td, f);
}
}