struct fio_libnfs_options {
struct nfs_context *context;
char *nfs_url;
- unsigned int queue_depth; /* nfs_callback needs this info, but doesn't have fio td structure to pull it from */
+ /* nfs_callback needs this info, but doesn't have fio td structure to
+ * pull it from
+ */
+ unsigned int queue_depth;
+
/* the following implement a circular queue of outstanding IOs */
- int outstanding_events; /* IOs issued to libnfs, that have not returned yet */
- int prev_requested_event_index; /* event last returned via fio_libnfs_event */
+
+ /* IOs issued to libnfs, that have not returned yet */
+ int outstanding_events;
+ /* event last returned via fio_libnfs_event */
+ int prev_requested_event_index;
int next_buffered_event; /* round robin-pointer within events[] */
int buffered_event_count; /* IOs completed by libnfs, waiting for FIO */
int free_event_buffer_index; /* next free buffer */
static struct fio_option options[] = {
{
- .name = "nfs_url",
- .lname = "nfs_url",
- .type = FIO_OPT_STR_STORE,
- .help = "URL in libnfs format, eg nfs://<server|ipv4|ipv6>/path[?arg=val[&arg=val]*]",
- .off1 = offsetof(struct fio_libnfs_options, nfs_url),
+ .name = "nfs_url",
+ .lname = "nfs_url",
+ .type = FIO_OPT_STR_STORE,
+ .help = "URL in libnfs format, eg nfs://<server|ipv4|"
+ "ipv6>/path[?arg=val[&arg=val]*]",
+ .off1 = offsetof(struct fio_libnfs_options, nfs_url),
.category = FIO_OPT_C_ENGINE,
.group = __FIO_OPT_G_NFS,
},
{
struct fio_libnfs_options *o = td->eo;
struct io_u *io_u = o->events[o->next_buffered_event];
+
assert(o->events[o->next_buffered_event]);
o->events[o->next_buffered_event] = NULL;
o->next_buffered_event = (o->next_buffered_event + 1) % td->o.iodepth;
+
/* validate our state machine */
assert(o->buffered_event_count);
o->buffered_event_count--;
assert(io_u);
+
/* assert that fio_libnfs_event is being called in sequential fashion */
assert(event == 0 || o->prev_requested_event_index + 1 == event);
- if (o->buffered_event_count == 0) {
+ if (o->buffered_event_count == 0)
o->prev_requested_event_index = -1;
- } else {
+ else
o->prev_requested_event_index = event;
- }
return io_u;
}
-static int nfs_event_loop(struct thread_data *td, bool flush) {
+/*
+ * fio core logic seems to stop calling this event-loop if we ever return with
+ * 0 events
+ */
+#define SHOULD_WAIT(td, o, flush) \
+ ((o)->outstanding_events == (td)->o.iodepth || \
+ (flush && (o)->outstanding_events))
+
+static int nfs_event_loop(struct thread_data *td, bool flush)
+{
struct fio_libnfs_options *o = td->eo;
struct pollfd pfds[1]; /* nfs:0 */
+
/* we already have stuff queued for fio, no need to waste cpu on poll() */
if (o->buffered_event_count)
return o->buffered_event_count;
- /* fio core logic seems to stop calling this event-loop if we ever return with 0 events */
- #define SHOULD_WAIT() (o->outstanding_events == td->o.iodepth || (flush && o->outstanding_events))
do {
- int timeout = SHOULD_WAIT() ? -1 : 0;
+ int timeout = SHOULD_WAIT(td, o, flush) ? -1 : 0;
int ret = 0;
+
pfds[0].fd = nfs_get_fd(o->context);
pfds[0].events = nfs_which_events(o->context);
ret = poll(&pfds[0], 1, timeout);
if (ret < 0) {
- if (errno == EINTR || errno == EAGAIN) {
+ if (errno == EINTR || errno == EAGAIN)
continue;
- }
- log_err("nfs: failed to poll events: %s.\n",
- strerror(errno));
+ log_err("nfs: failed to poll events: %s\n", strerror(errno));
break;
}
log_err("nfs: socket is in an unrecoverable error state.\n");
break;
}
- } while (SHOULD_WAIT());
+ } while (SHOULD_WAIT(td, o, flush));
+
return o->buffered_event_count;
-#undef SHOULD_WAIT
}
static int fio_libnfs_getevents(struct thread_data *td, unsigned int min,
- unsigned int max, const struct timespec *t)
+ unsigned int max, const struct timespec *t)
{
return nfs_event_loop(td, false);
}
static void nfs_callback(int res, struct nfs_context *nfs, void *data,
- void *private_data)
+ void *private_data)
{
struct io_u *io_u = private_data;
struct nfs_data *nfs_data = io_u->file->engine_data;
struct fio_libnfs_options *o = nfs_data->options;
if (res < 0) {
- log_err("Failed NFS operation(code:%d): %s\n", res, nfs_get_error(o->context));
+ log_err("Failed NFS operation(code:%d): %s\n", res,
+ nfs_get_error(o->context));
io_u->error = -res;
- /* res is used for read math below, don't wanna pass negative there */
+ /* res is used for read math below, don't want to pass negative
+ * there
+ */
res = 0;
} else if (io_u->ddir == DDIR_READ) {
memcpy(io_u->buf, data, res);
o->buffered_event_count++;
}
-static int queue_write(struct fio_libnfs_options *o, struct io_u *io_u) {
+static int queue_write(struct fio_libnfs_options *o, struct io_u *io_u)
+{
struct nfs_data *nfs_data = io_u->engine_data;
- return nfs_pwrite_async(o->context, nfs_data->nfsfh,
- io_u->offset, io_u->buflen, io_u->buf, nfs_callback,
- io_u);
+
+ return nfs_pwrite_async(o->context, nfs_data->nfsfh, io_u->offset,
+ io_u->buflen, io_u->buf, nfs_callback, io_u);
}
-static int queue_read(struct fio_libnfs_options *o, struct io_u *io_u) {
+static int queue_read(struct fio_libnfs_options *o, struct io_u *io_u)
+{
struct nfs_data *nfs_data = io_u->engine_data;
- return nfs_pread_async(o->context, nfs_data->nfsfh, io_u->offset, io_u->buflen, nfs_callback, io_u);
+
+ return nfs_pread_async(o->context, nfs_data->nfsfh, io_u->offset,
+ io_u->buflen, nfs_callback, io_u);
}
static enum fio_q_status fio_libnfs_queue(struct thread_data *td,
- struct io_u *io_u)
+ struct io_u *io_u)
{
struct nfs_data *nfs_data = io_u->file->engine_data;
struct fio_libnfs_options *o = nfs_data->options;
struct nfs_context *nfs = o->context;
- int err;
enum fio_q_status ret = FIO_Q_QUEUED;
+ int err;
io_u->engine_data = nfs_data;
- switch(io_u->ddir) {
- case DDIR_WRITE:
- err = queue_write(o, io_u);
- break;
- case DDIR_READ:
- err = queue_read(o, io_u);
- break;
- case DDIR_TRIM:
- log_err("nfs: trim is not supported");
- err = -1;
- break;
- default:
- log_err("nfs: unhandled io %d\n", io_u->ddir);
- err = -1;
+ switch (io_u->ddir) {
+ case DDIR_WRITE:
+ err = queue_write(o, io_u);
+ break;
+ case DDIR_READ:
+ err = queue_read(o, io_u);
+ break;
+ case DDIR_TRIM:
+ log_err("nfs: trim is not supported");
+ err = -1;
+ break;
+ default:
+ log_err("nfs: unhandled io %d\n", io_u->ddir);
+ err = -1;
}
if (err) {
log_err("nfs: Failed to queue nfs op: %s\n", nfs_get_error(nfs));
return 0;
options->context = nfs_init_context();
- if (options->context == NULL) {
+ if (!options->context) {
log_err("nfs: failed to init nfs context\n");
return -1;
}
static int fio_libnfs_setup(struct thread_data *td)
{
- /* Using threads with libnfs causes fio to hang on exit, lower performance */
+ /* Using threads with libnfs causes fio to hang on exit, lower
+ * performance
+ */
td->o.use_thread = 0;
return 0;
}
static void fio_libnfs_cleanup(struct thread_data *td)
{
struct fio_libnfs_options *o = td->eo;
+
nfs_umount(o->context);
nfs_destroy_context(o->context);
free(o->events);
static int fio_libnfs_open(struct thread_data *td, struct fio_file *f)
{
- int ret;
struct fio_libnfs_options *options = td->eo;
struct nfs_data *nfs_data = NULL;
int flags = 0;
+ int ret;
if (!options->nfs_url) {
log_err("nfs: nfs_url is a required parameter\n");
ret = do_mount(td, options->nfs_url);
- if (ret != 0) {
- log_err("nfs: Failed to mount %s with code %d: %s\n", options->nfs_url, ret, nfs_get_error(options->context));
+ if (ret) {
+ log_err("nfs: Failed to mount %s with code %d: %s\n",
+ options->nfs_url, ret, nfs_get_error(options->context));
return ret;
}
nfs_data = malloc(sizeof(struct nfs_data));
memset(nfs_data, 0, sizeof(struct nfs_data));
nfs_data->options = options;
- if (td->o.td_ddir == TD_DDIR_WRITE) {
+ if (td->o.td_ddir == TD_DDIR_WRITE)
flags |= O_CREAT | O_RDWR;
- } else {
+ else
flags |= O_RDWR;
- }
+
ret = nfs_open(options->context, f->file_name, flags, &nfs_data->nfsfh);
- if (ret != 0)
- log_err("Failed to open %s: %s\n", f->file_name, nfs_get_error(options->context));
+ if (ret)
+ log_err("Failed to open %s: %s\n", f->file_name,
+ nfs_get_error(options->context));
f->engine_data = nfs_data;
return ret;
}
struct nfs_data *nfs_data = f->engine_data;
struct fio_libnfs_options *o = nfs_data->options;
int ret = 0;
+
if (nfs_data->nfsfh)
ret = nfs_close(o->context, nfs_data->nfsfh);
+
free(nfs_data);
f->engine_data = NULL;
return ret;
.cleanup = fio_libnfs_cleanup,
.open_file = fio_libnfs_open,
.close_file = fio_libnfs_close,
- .flags = FIO_DISKLESSIO | FIO_NOEXTEND | FIO_NODISKUTIL,
+ .flags = FIO_DISKLESSIO | FIO_NOEXTEND | FIO_NODISKUTIL,
.options = options,
.option_struct_size = sizeof(struct fio_libnfs_options),
};