X-Git-Url: https://git.kernel.dk/?a=blobdiff_plain;f=engines%2Fsyslet-rw.c;h=15e4c2578fe9ee978af4f4a1847ecf571871718e;hb=716050f2a0b858835d7836ad74e8b680973e08e8;hp=2e23fc937fee6fca810c694f0dfa7f96563bbb4d;hpb=05f6a172967321798f63869d330f88a1bc2c8ed9;p=fio.git

diff --git a/engines/syslet-rw.c b/engines/syslet-rw.c
index 2e23fc93..15e4c257 100644
--- a/engines/syslet-rw.c
+++ b/engines/syslet-rw.c
@@ -14,8 +14,7 @@
 #include <asm/unistd.h>
 
 #include "../fio.h"
-#include "../indirect.h"
-#include "../syslet.h"
+#include "../lib/fls.h"
 
 #ifdef FIO_HAVE_SYSLET
 
@@ -32,10 +31,11 @@ struct syslet_data {
 	unsigned int nr_events;
 	
 	struct syslet_ring *ring;
+	unsigned int ring_mask;
 	void *stack;
 };
 
-static void fio_syslet_complete(struct thread_data *td, struct io_u *io_u)
+static void fio_syslet_add_event(struct thread_data *td, struct io_u *io_u)
 {
 	struct syslet_data *sd = td->io_ops->data;
 
@@ -43,29 +43,38 @@ static void fio_syslet_complete(struct thread_data *td, struct io_u *io_u)
 	sd->events[sd->nr_events++] = io_u;
 }
 
-static void syslet_complete_nr(struct thread_data *td, unsigned int nr)
+static void fio_syslet_add_events(struct thread_data *td, unsigned int nr)
 {
 	struct syslet_data *sd = td->io_ops->data;
-	unsigned int i;
+	unsigned int i, uidx;
+
+	uidx = sd->ring->user_tail;
+	read_barrier();
 
 	for (i = 0; i < nr; i++) {
-		unsigned int idx = (i + sd->ring->user_tail) % td->o.iodepth;
+		unsigned int idx = (i + uidx) & sd->ring_mask;
 		struct syslet_completion *comp = &sd->ring->comp[idx];
 		struct io_u *io_u = (struct io_u *) (long) comp->caller_data;
+		long ret;
+
+		ret = comp->status;
+		if (ret <= 0) {
+			io_u->resid = io_u->xfer_buflen;
+			io_u->error = -ret;
+		} else {
+			io_u->resid = io_u->xfer_buflen - ret;
+			io_u->error = 0;
+		}
 
-		io_u->resid = io_u->xfer_buflen - comp->status;
-		fio_syslet_complete(td, io_u);
+		fio_syslet_add_event(td, io_u);
 	}
 }
 
-
 static void fio_syslet_wait_for_events(struct thread_data *td)
 {
 	struct syslet_data *sd = td->io_ops->data;
 	struct syslet_ring *ring = sd->ring;
-	unsigned int events;
 
-	events = 0;
 	do {
 		unsigned int kh = ring->kernel_head;
 		int ret;
@@ -76,10 +85,9 @@ static void fio_syslet_wait_for_events(struct thread_data *td)
 		if (ring->user_tail != kh) {
 			unsigned int nr = kh - ring->user_tail;
 
-			syslet_complete_nr(td, nr);
-			events += nr;
+			fio_syslet_add_events(td, nr);
 			ring->user_tail = kh;
-			continue;
+			break;
 		}
 
 		/*
@@ -87,25 +95,23 @@ static void fio_syslet_wait_for_events(struct thread_data *td)
 		 */
 		ret = syscall(__NR_syslet_ring_wait, ring, ring->user_tail);
 		assert(!ret);
-	} while (!events);
+	} while (1);
 }
 
-static int fio_syslet_getevents(struct thread_data *td, int min,
-				int fio_unused max,
+static int fio_syslet_getevents(struct thread_data *td, unsigned int min,
+				unsigned int fio_unused max,
 				struct timespec fio_unused *t)
 {
 	struct syslet_data *sd = td->io_ops->data;
 	long ret;
 
-	do {
-		/*
-		 * do we have enough immediate completions?
-		 */
-		if (sd->nr_events >= (unsigned int) min)
-			break;
-
+	/*
+	 * While we have less events than requested, block waiting for them
+	 * (if we have to, there may already be more completed events ready
+	 * for us - see fio_syslet_wait_for_events()
+	 */
+	while (sd->nr_events < min)
 		fio_syslet_wait_for_events(td);
-	} while (1);
 
 	ret = sd->nr_events;
 	sd->nr_events = 0;
@@ -125,6 +131,12 @@ static void fio_syslet_prep_sync(struct fio_file *f,
 	FILL_IN(*regs, __NR_fsync, (long) f->fd);
 }
 
+static void fio_syslet_prep_datasync(struct fio_file *f,
+				     struct indirect_registers *regs)
+{
+	FILL_IN(*regs, __NR_fdatasync, (long) f->fd);
+}
+
 static void fio_syslet_prep_rw(struct io_u *io_u, struct fio_file *f,
 			       struct indirect_registers *regs)
 {
@@ -148,6 +160,8 @@ static void fio_syslet_prep(struct io_u *io_u, struct indirect_registers *regs)
 
 	if (io_u->ddir == DDIR_SYNC)
 		fio_syslet_prep_sync(f, regs);
+	else if (io_u->ddir == DDIR_DATASYNC)
+		fio_syslet_prep_datasync(f, regs);
 	else
 		fio_syslet_prep_rw(io_u, f, regs);
 }
@@ -215,9 +229,7 @@ static void fio_syslet_cleanup(struct thread_data *td)
 	if (sd) {
 		free(sd->events);
 		free(sd->ring);
-		free(sd->stack);
 		free(sd);
-		td->io_ops->data = NULL;
 	}
 }
 
@@ -237,14 +249,8 @@ static int fio_syslet_init(struct thread_data *td)
 	 * The ring needs to be a power-of-2, so round it up if we have to
 	 */
 	ring_nr = td->o.iodepth;
-	if (ring_nr & (ring_nr - 1)) {
-		int bits = 1;
-
-		while (ring_nr >>= 1)
-			bits++;
-
-		ring_nr = 1 << bits;
-	}
+	if (ring_nr & (ring_nr - 1))
+		ring_nr = 1 << __fls(ring_nr);
 
 	ring_size = sizeof(struct syslet_ring) +
 			ring_nr * sizeof(struct syslet_completion);
@@ -254,6 +260,7 @@ static int fio_syslet_init(struct thread_data *td)
 		goto err_mem;
 
 	sd->ring = ring;
+	sd->ring_mask = ring_nr - 1;
 	sd->stack = stack;
 
 	memset(sd->ring, 0, ring_size);
@@ -285,6 +292,7 @@ static struct ioengine_ops ioengine = {
 	.cleanup	= fio_syslet_cleanup,
 	.open_file	= generic_open_file,
 	.close_file	= generic_close_file,
+	.get_file_size	= generic_get_file_size,
 };
 
 #else /* FIO_HAVE_SYSLET */
@@ -296,7 +304,7 @@ static struct ioengine_ops ioengine = {
  */
 static int fio_syslet_init(struct thread_data fio_unused *td)
 {
-	fprintf(stderr, "fio: syslet not available\n");
+	log_err("fio: syslet not available\n");
 	return 1;
 }