Fix Windows build warnings.
[fio.git] / engines / windowsaio.c
index 299acc47a5f3651d6c772d5d02fbef0569b4b5d3..ee89fa128108e5346f031a750de82ee7a4eb7410 100644 (file)
@@ -1,6 +1,7 @@
 /*
- * Native Windows async IO engine
- * Copyright (C) 2011 Bruce Cran <bruce@cran.org.uk>
+ * windowsaio engine
+ *
+ * IO engine using Windows IO Completion Ports.
  */
 
 #include <stdio.h>
@@ -8,27 +9,25 @@
 #include <unistd.h>
 #include <signal.h>
 #include <errno.h>
-#include <windows.h>
 
 #include "../fio.h"
 
 typedef BOOL (WINAPI *CANCELIOEX)(HANDLE hFile, LPOVERLAPPED lpOverlapped);
 
+int geterrno_from_win_error (DWORD code, int deferrno);
+
 struct fio_overlapped {
        OVERLAPPED o;
        struct io_u *io_u;
        BOOL io_complete;
-    BOOL io_free;
 };
 
 struct windowsaio_data {
-       struct fio_overlapped *ovls;
        struct io_u **aio_events;
+       HANDLE iocp;
        HANDLE iothread;
        HANDLE iocomplete_event;
-       CANCELIOEX pCancelIoEx;
        BOOL iothread_running;
-       BOOL use_iocp;
 };
 
 struct thread_ctx {
@@ -36,99 +35,48 @@ struct thread_ctx {
        struct windowsaio_data *wd;
 };
 
-static void PrintError(LPCSTR lpszFunction);
-static int fio_windowsaio_cancel(struct thread_data *td,
-                              struct io_u *io_u);
 static BOOL timeout_expired(DWORD start_count, DWORD end_count);
 static int fio_windowsaio_getevents(struct thread_data *td, unsigned int min,
-                                   unsigned int max, struct timespec *t);
+                                       unsigned int max, struct timespec *t);
 static struct io_u *fio_windowsaio_event(struct thread_data *td, int event);
 static int fio_windowsaio_queue(struct thread_data *td,
-                             struct io_u *io_u);
+                                 struct io_u *io_u);
 static void fio_windowsaio_cleanup(struct thread_data *td);
 static DWORD WINAPI IoCompletionRoutine(LPVOID lpParameter);
 static int fio_windowsaio_init(struct thread_data *td);
 static int fio_windowsaio_open_file(struct thread_data *td, struct fio_file *f);
 static int fio_windowsaio_close_file(struct thread_data fio_unused *td, struct fio_file *f);
 
-int sync_file_range(int fd, off64_t offset, off64_t nbytes,
-                          unsigned int flags)
-{
-       errno = ENOSYS;
-       return -1;
-}
-
-static void PrintError(LPCSTR lpszFunction)
-{
-       // Retrieve the system error message for the last-error code
-
-       LPSTR lpMsgBuf;
-       DWORD dw = GetLastError();
-
-       FormatMessage(
-               FORMAT_MESSAGE_ALLOCATE_BUFFER |
-               FORMAT_MESSAGE_FROM_SYSTEM |
-               FORMAT_MESSAGE_IGNORE_INSERTS,
-               NULL,
-               dw,
-               MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),
-               (LPTSTR)&lpMsgBuf,
-               0, NULL );
-
-       log_err("%s - %s", lpszFunction, lpMsgBuf);
-       LocalFree(lpMsgBuf);
-}
-
 static int fio_windowsaio_init(struct thread_data *td)
 {
        struct windowsaio_data *wd;
-       HANDLE hKernel32Dll;
        int rc = 0;
-       int i;
 
-       wd = malloc(sizeof(struct windowsaio_data));
-       if (wd != NULL)
-               ZeroMemory(wd, sizeof(struct windowsaio_data));
-       else
+       wd = calloc(1, sizeof(struct windowsaio_data));
+       if (wd == NULL) {
+                log_err("windowsaio: failed to allocate memory for engine data\n");
                rc = 1;
-
-       if (!rc) {
-               wd->aio_events = malloc(td->o.iodepth * sizeof(struct io_u*));
-               if (wd->aio_events == NULL)
-                       rc = 1;
        }
 
        if (!rc) {
-               wd->ovls = malloc(td->o.iodepth * sizeof(struct fio_overlapped));
-               if (wd->ovls == NULL)
+               wd->aio_events = malloc(td->o.iodepth * sizeof(struct io_u*));
+               if (wd->aio_events == NULL) {
+                       log_err("windowsaio: failed to allocate memory for aio events list\n");
                        rc = 1;
-       }
-
-       if (!rc) {
-           for (i = 0; i < td->o.iodepth; i++) {
-               wd->ovls[i].io_free = TRUE;
-               wd->ovls[i].io_complete = FALSE;
-
-                       wd->ovls[i].o.hEvent = CreateEvent(NULL, TRUE, FALSE, NULL);
-                       if (wd->ovls[i].o.hEvent == NULL) {
-                               rc = 1;
-                               break;
-                       }
-           }
+               }
        }
 
        if (!rc) {
                /* Create an auto-reset event */
                wd->iocomplete_event = CreateEvent(NULL, FALSE, FALSE, NULL);
-               if (wd->iocomplete_event == NULL)
+               if (wd->iocomplete_event == NULL) {
+                       log_err("windowsaio: failed to create io complete event handle\n");
                        rc = 1;
+               }
        }
 
        if (rc) {
-               PrintError(__func__);
                if (wd != NULL) {
-                       if (wd->ovls != NULL)
-                               free(wd->ovls);
                        if (wd->aio_events != NULL)
                                free(wd->aio_events);
 
@@ -136,33 +84,63 @@ static int fio_windowsaio_init(struct thread_data *td)
                }
        }
 
-       hKernel32Dll = GetModuleHandle("kernel32.dll");
-       wd->pCancelIoEx = GetProcAddress(hKernel32Dll, "CancelIoEx");
-
        td->io_ops->data = wd;
-       return 0;
+
+       if (!rc) {
+               struct thread_ctx *ctx;
+               struct windowsaio_data *wd;
+               HANDLE hFile;
+
+               hFile = CreateIoCompletionPort(INVALID_HANDLE_VALUE, NULL, 0, 0);
+               if (hFile == INVALID_HANDLE_VALUE) {
+                       log_err("windowsaio: failed to create io completion port\n");
+                       rc = 1;
+               }
+
+               wd = td->io_ops->data;
+               wd->iothread_running = TRUE;
+               wd->iocp = hFile;
+
+               if (!rc)
+                       ctx = malloc(sizeof(struct thread_ctx));
+
+               if (!rc && ctx == NULL)
+               {
+                       log_err("windowsaio: failed to allocate memory for thread context structure\n");
+                       CloseHandle(hFile);
+                       rc = 1;
+               }
+
+               if (!rc)
+               {
+                       ctx->iocp = hFile;
+                       ctx->wd = wd;
+                       wd->iothread = CreateThread(NULL, 0, IoCompletionRoutine, ctx, 0, NULL);
+                       if (wd->iothread == NULL)
+                               log_err("windowsaio: failed to create io completion thread\n");
+               }
+
+               if (rc || wd->iothread == NULL)
+                       rc = 1;
+       }
+
+       return rc;
 }
 
 static void fio_windowsaio_cleanup(struct thread_data *td)
 {
-       int i;
        struct windowsaio_data *wd;
 
        wd = td->io_ops->data;
 
        if (wd != NULL) {
-           wd->iothread_running = FALSE;
-       WaitForSingleObject(wd->iothread, INFINITE);
+               wd->iothread_running = FALSE;
+               WaitForSingleObject(wd->iothread, INFINITE);
 
                CloseHandle(wd->iothread);
                CloseHandle(wd->iocomplete_event);
 
-               for (i = 0; i < td->o.iodepth; i++) {
-                       CloseHandle(wd->ovls[i].o.hEvent);
-               }
-
                free(wd->aio_events);
-               free(wd->ovls);
                free(wd);
 
                td->io_ops->data = NULL;
@@ -173,8 +151,7 @@ static void fio_windowsaio_cleanup(struct thread_data *td)
 static int fio_windowsaio_open_file(struct thread_data *td, struct fio_file *f)
 {
        int rc = 0;
-       HANDLE hFile;
-       DWORD flags = FILE_FLAG_POSIX_SEMANTICS;
+       DWORD flags = FILE_FLAG_POSIX_SEMANTICS | FILE_FLAG_OVERLAPPED;
        DWORD sharemode = FILE_SHARE_READ | FILE_SHARE_WRITE;
        DWORD openmode = OPEN_ALWAYS;
        DWORD access;
@@ -182,24 +159,24 @@ static int fio_windowsaio_open_file(struct thread_data *td, struct fio_file *f)
        dprint(FD_FILE, "fd open %s\n", f->file_name);
 
        if (f->filetype == FIO_TYPE_PIPE) {
-               log_err("fio: windowsaio doesn't support pipes\n");
+               log_err("windowsaio: pipes are not supported\n");
                return 1;
        }
 
        if (!strcmp(f->file_name, "-")) {
-               log_err("fio: can't read/write to stdin/out\n");
+               log_err("windowsaio: can't read/write to stdin/out\n");
                return 1;
        }
 
-    if (!td->o.odirect && !td->o.sync_io && td->io_ops->data != NULL)
-           flags |= FILE_FLAG_OVERLAPPED;
-
        if (td->o.odirect)
                flags |= FILE_FLAG_NO_BUFFERING;
        if (td->o.sync_io)
                flags |= FILE_FLAG_WRITE_THROUGH;
 
-
+       /*
+        * Inform Windows whether we're going to be doing sequential or
+        * random io so it can tune the Cache Manager
+        */
        if (td->o.td_ddir == TD_DDIR_READ  ||
                td->o.td_ddir == TD_DDIR_WRITE)
                flags |= FILE_FLAG_SEQUENTIAL_SCAN;
@@ -211,7 +188,7 @@ static int fio_windowsaio_open_file(struct thread_data *td, struct fio_file *f)
        else
                access = (GENERIC_READ | GENERIC_WRITE);
 
-       if (td->o.create_on_open > 0)
+       if (td->o.create_on_open)
                openmode = OPEN_ALWAYS;
        else
                openmode = OPEN_EXISTING;
@@ -220,36 +197,19 @@ static int fio_windowsaio_open_file(struct thread_data *td, struct fio_file *f)
                NULL, openmode, flags, NULL);
 
        if (f->hFile == INVALID_HANDLE_VALUE) {
-               PrintError(__func__);
+               log_err("windowsaio: failed to open file \"%s\"\n", f->file_name);
                rc = 1;
        }
 
-       /* Only set up the competion port and thread if we're not just
+       /* Only set up the completion port and thread if we're not just
         * querying the device size */
-    if (!rc && td->io_ops->data != NULL && !td->o.odirect && !td->o.sync_io) {
-               struct thread_ctx *ctx;
-        struct windowsaio_data *wd;
-               hFile = CreateIoCompletionPort(f->hFile, NULL, 0, 0);
-
-        wd = td->io_ops->data;
-
-        if (!td->o.odirect && !td->o.sync_io)
-            wd->use_iocp = 1;
-        else
-            wd->use_iocp = 0;
-
-               wd->iothread_running = TRUE;
+       if (!rc && td->io_ops->data != NULL) {
+               struct windowsaio_data *wd;
 
-               if (!rc) {
-                       ctx = malloc(sizeof(struct thread_ctx));
-                       ctx->iocp = hFile;
-                       ctx->wd = wd;
+               wd = td->io_ops->data;
 
-                       wd->iothread = CreateThread(NULL, 0, IoCompletionRoutine, ctx, 0, NULL);
-               }
-
-               if (rc || wd->iothread == NULL) {
-                       PrintError(__func__);
+               if (CreateIoCompletionPort(f->hFile, wd->iocp, 0, 0) == NULL) {
+                       log_err("windowsaio: failed to create io completion port\n");
                        rc = 1;
                }
        }
@@ -259,15 +219,19 @@ static int fio_windowsaio_open_file(struct thread_data *td, struct fio_file *f)
 
 static int fio_windowsaio_close_file(struct thread_data fio_unused *td, struct fio_file *f)
 {
+       int rc = 0;
+
        dprint(FD_FILE, "fd close %s\n", f->file_name);
 
        if (f->hFile != INVALID_HANDLE_VALUE) {
-               if (!CloseHandle(f->hFile))
-                       PrintError(__func__);
+               if (!CloseHandle(f->hFile)) {
+                       log_info("windowsaio: failed to close file handle for \"%s\"\n", f->file_name);
+                       rc = 1;
+               }
        }
 
        f->hFile = INVALID_HANDLE_VALUE;
-       return 0;
+       return rc;
 }
 
 static BOOL timeout_expired(DWORD start_count, DWORD end_count)
@@ -292,7 +256,7 @@ static struct io_u* fio_windowsaio_event(struct thread_data *td, int event)
 }
 
 static int fio_windowsaio_getevents(struct thread_data *td, unsigned int min,
-                                   unsigned int max, struct timespec *t)
+                                       unsigned int max, struct timespec *t)
 {
        struct windowsaio_data *wd = td->io_ops->data;
        struct flist_head *entry;
@@ -316,8 +280,8 @@ static int fio_windowsaio_getevents(struct thread_data *td, unsigned int min,
                        fov = (struct fio_overlapped*)io_u->engine_data;
 
                        if (fov->io_complete) {
-                fov->io_complete = FALSE;
-                               fov->io_free  = TRUE;
+                               fov->io_complete = FALSE;
+                               ResetEvent(fov->o.hEvent);
                                wd->aio_events[dequeued] = io_u;
                                dequeued++;
                        }
@@ -326,10 +290,10 @@ static int fio_windowsaio_getevents(struct thread_data *td, unsigned int min,
                                break;
                }
 
-        if (dequeued < min) {
+               if (dequeued < min) {
                        status = WaitForSingleObject(wd->iocomplete_event, mswait);
-                       if (status != WAIT_OBJECT_0 && dequeued > 0)
-                           break;
+                       if (status != WAIT_OBJECT_0 && dequeued >= min)
+                               break;
                }
 
                if (dequeued >= min || (t != NULL && timeout_expired(start_count, end_count)))
@@ -339,43 +303,23 @@ static int fio_windowsaio_getevents(struct thread_data *td, unsigned int min,
        return dequeued;
 }
 
-static int fio_windowsaio_queue(struct thread_data *td,
-                             struct io_u *io_u)
+static int fio_windowsaio_queue(struct thread_data *td, struct io_u *io_u)
 {
-    LPOVERLAPPED lpOvl = NULL;
-       struct windowsaio_data *wd;
+       struct fio_overlapped *o = io_u->engine_data;
+       LPOVERLAPPED lpOvl = &o->o;
        DWORD iobytes;
-       BOOL success;
-       int index;
+       BOOL success = FALSE;
        int rc = FIO_Q_COMPLETED;
 
        fio_ro_check(td, io_u);
 
-       wd = td->io_ops->data;
-
-       if (wd->use_iocp) {
-           for (index = 0; index < td->o.iodepth; index++) {
-               if (wd->ovls[index].io_free) {
-                wd->ovls[index].io_free = FALSE;
-                   ResetEvent(wd->ovls[index].o.hEvent);
-                   break;
-               }
-        }
-
-        assert(index < td->o.iodepth);
-
-        lpOvl = &wd->ovls[index].o;
-           wd->ovls[index].io_u = io_u;
-       lpOvl->Internal = STATUS_PENDING;
-       lpOvl->InternalHigh = 0;
-       lpOvl->Offset = io_u->offset & 0xFFFFFFFF;
-       lpOvl->OffsetHigh = io_u->offset >> 32;
-       lpOvl->Pointer = NULL;
-        io_u->engine_data = &wd->ovls[index];
-       }
+       lpOvl->Internal = STATUS_PENDING;
+       lpOvl->InternalHigh = 0;
+       lpOvl->Offset = io_u->offset & 0xFFFFFFFF;
+       lpOvl->OffsetHigh = io_u->offset >> 32;
 
        switch (io_u->ddir) {
-    case DDIR_WRITE:
+       case DDIR_WRITE:
                success = WriteFile(io_u->file->hFile, io_u->xfer_buf, io_u->xfer_buflen, &iobytes, lpOvl);
                break;
        case DDIR_READ:
@@ -385,29 +329,28 @@ static int fio_windowsaio_queue(struct thread_data *td,
        case DDIR_DATASYNC:
        case DDIR_SYNC_FILE_RANGE:
                success = FlushFileBuffers(io_u->file->hFile);
-               if (!success)
-                   io_u->error = GetLastError();
+               if (!success) {
+                       log_err("windowsaio: failed to flush file buffers\n");
+                       io_u->error = win_to_posix_error(GetLastError());
+               }
 
                return FIO_Q_COMPLETED;
                break;
        case DDIR_TRIM:
-               log_err("manual TRIM isn't supported on Windows");
+               log_err("windowsaio: manual TRIM isn't supported on Windows\n");
                io_u->error = 1;
                io_u->resid = io_u->xfer_buflen;
                return FIO_Q_COMPLETED;
                break;
        default:
                assert(0);
+               break;
        }
 
-    if (wd->use_iocp && (success || GetLastError() == ERROR_IO_PENDING)) {
+       if (success || GetLastError() == ERROR_IO_PENDING)
                rc = FIO_Q_QUEUED;
-       } else if (success && !wd->use_iocp) {
-               io_u->resid = io_u->xfer_buflen - iobytes;
-               io_u->error = 0;
-       } else {
-               PrintError(__func__);
-               io_u->error = GetLastError();
+       else {
+               io_u->error = win_to_posix_error(GetLastError());
                io_u->resid = io_u->xfer_buflen;
        }
 
@@ -429,7 +372,7 @@ static DWORD WINAPI IoCompletionRoutine(LPVOID lpParameter)
        wd = ctx->wd;
 
        do {
-               if (!GetQueuedCompletionStatus(ctx->iocp, &bytes, &ulKey, &ovl, 250))
+               if (!GetQueuedCompletionStatus(ctx->iocp, &bytes, &ulKey, &ovl, 250) && ovl == NULL)
                        continue;
 
                fov = CONTAINING_RECORD(ovl, struct fio_overlapped, o);
@@ -440,10 +383,10 @@ static DWORD WINAPI IoCompletionRoutine(LPVOID lpParameter)
                        io_u->error = 0;
                } else {
                        io_u->resid = io_u->xfer_buflen;
-                       io_u->error = ovl->Internal;
+                       io_u->error = win_to_posix_error(GetLastError());
                }
 
-        fov->io_complete = TRUE;
+               fov->io_complete = TRUE;
                SetEvent(wd->iocomplete_event);
        } while (ctx->wd->iothread_running);
 
@@ -452,22 +395,33 @@ static DWORD WINAPI IoCompletionRoutine(LPVOID lpParameter)
        return 0;
 }
 
-static int fio_windowsaio_cancel(struct thread_data *td,
-                              struct io_u *io_u)
+static void fio_windowsaio_io_u_free(struct thread_data *td, struct io_u *io_u)
 {
-       int rc = 0;
+       struct fio_overlapped *o = io_u->engine_data;
 
-       struct windowsaio_data *wd = td->io_ops->data;
+       if (o) {
+               CloseHandle(o->o.hEvent);
+               io_u->engine_data = NULL;
+               free(o);
+       }
+}
 
-       /* If we're running on Vista or newer, we can cancel individual IO requests */
-       if (wd->pCancelIoEx != NULL) {
-               struct fio_overlapped *ovl = io_u->engine_data;
-               if (!wd->pCancelIoEx(io_u->file->hFile, &ovl->o))
-                       rc = 1;
-       } else
-               rc = 1;
+static int fio_windowsaio_io_u_init(struct thread_data *td, struct io_u *io_u)
+{
+       struct fio_overlapped *o;
+
+       o = malloc(sizeof(*o));
+       o->io_complete = FALSE;
+       o->io_u = io_u;
+       o->o.hEvent = CreateEvent(NULL, TRUE, FALSE, NULL);
+       if (o->o.hEvent == NULL) {
+               log_err("windowsaio: failed to create event handle\n");
+               free(o);
+               return 1;
+       }
 
-       return rc;
+       io_u->engine_data = o;
+       return 0;
 }
 
 static struct ioengine_ops ioengine = {
@@ -475,21 +429,22 @@ static struct ioengine_ops ioengine = {
        .version        = FIO_IOOPS_VERSION,
        .init           = fio_windowsaio_init,
        .queue          = fio_windowsaio_queue,
-       .cancel         = fio_windowsaio_cancel,
        .getevents      = fio_windowsaio_getevents,
        .event          = fio_windowsaio_event,
        .cleanup        = fio_windowsaio_cleanup,
        .open_file      = fio_windowsaio_open_file,
        .close_file     = fio_windowsaio_close_file,
-       .get_file_size  = generic_get_file_size
+       .get_file_size  = generic_get_file_size,
+       .io_u_init      = fio_windowsaio_io_u_init,
+       .io_u_free      = fio_windowsaio_io_u_free,
 };
 
-static void fio_init fio_posixaio_register(void)
+static void fio_init fio_windowsaio_register(void)
 {
        register_ioengine(&ioengine);
 }
 
-static void fio_exit fio_posixaio_unregister(void)
+static void fio_exit fio_windowsaio_unregister(void)
 {
        unregister_ioengine(&ioengine);
 }