* the ->aio_pending store is seen after the ->aio_events store
*/
sd->aio_events[sd->aio_pending] = io_u;
+ write_barrier();
sd->aio_pending++;
sd->nr--;
}
-static void fio_solarisaio_sigio(int sig)
-{
- wait_for_event(NULL);
-}
-
static int fio_solarisaio_getevents(struct thread_data *td, unsigned int min,
unsigned int max, struct timespec *t)
{
wait_for_event(&tv);
/*
- * Needs locking here for SIGIO
+ * should be OK without locking, as int operations should be atomic
*/
ret = sd->aio_pending;
- sd->aio_pending = 0;
+ sd->aio_pending -= ret;
return ret;
}
}
/*
- * Set USE_SIGNAL_COMPLETIONS to use SIGIO as completion events. Needs
- * locking around ->aio_pending and ->aio_events, see comment
+ * Set USE_SIGNAL_COMPLETIONS to use SIGIO as completion events.
*/
+#ifdef USE_SIGNAL_COMPLETIONS
+static void fio_solarisaio_sigio(int sig)
+{
+ wait_for_event(NULL);
+}
+
static void fio_solarisaio_init_sigio(void)
{
-#ifdef USE_SIGNAL_COMPLETIONS
struct sigaction act;
memset(&act, 0, sizeof(act));
act.sa_handler = fio_solarisaio_sigio;
act.sa_flags = SA_RESTART;
sigaction(SIGIO, &act, NULL);
-#endif
}
+#endif
static int fio_solarisaio_init(struct thread_data *td)
{
memset(sd->aio_events, 0, max_depth * sizeof(struct io_u *));
sd->max_depth = max_depth;
+#ifdef USE_SIGNAL_COMPLETIONS
fio_solarisaio_init_sigio();
+#endif
td->io_ops->data = sd;
return 0;
.cleanup = fio_solarisaio_cleanup,
.open_file = generic_open_file,
.close_file = generic_close_file,
+ .get_file_size = generic_get_file_size,
};
#else /* FIO_HAVE_SOLARISAIO */