s3-talloc Change TALLOC_ZERO_P() to talloc_zero()
[nivanova/samba-autobuild/.git] / source3 / smbd / aio.c
index f13393b7648d2063c2ad0e1ecce2209429ac073b..db2926b4a4f3a41ff579d2152820d276a2f7aaff 100644 (file)
 */
 
 #include "includes.h"
+#include "smbd/smbd.h"
+#include "smbd/globals.h"
+#include "../lib/util/tevent_ntstatus.h"
 
 #if defined(WITH_AIO)
 
 /* The signal we'll use to signify aio done. */
 #ifndef RT_SIGNAL_AIO
-#define RT_SIGNAL_AIO (SIGRTMIN+3)
+#define RT_SIGNAL_AIO  (SIGRTMIN+3)
+#endif
+
+#ifndef HAVE_STRUCT_SIGEVENT_SIGEV_VALUE_SIVAL_PTR
+#ifdef HAVE_STRUCT_SIGEVENT_SIGEV_VALUE_SIGVAL_PTR
+#define sival_int      sigval_int
+#define sival_ptr      sigval_ptr
+#endif
 #endif
 
 /****************************************************************************
@@ -35,165 +45,106 @@ struct aio_extra {
        struct aio_extra *next, *prev;
        SMB_STRUCT_AIOCB acb;
        files_struct *fsp;
-       bool read_req;
-       uint16 mid;
-       char *inbuf;
-       char *outbuf;
+       struct smb_request *smbreq;
+       DATA_BLOB outbuf;
+       struct lock_struct lock;
+       bool write_through;
+       int (*handle_completion)(struct aio_extra *ex, int errcode);
 };
 
-static struct aio_extra *aio_list_head;
-
 /****************************************************************************
- Create the extended aio struct we must keep around for the lifetime
- of the aio_read call.
+ Initialize the signal handler for aio read/write.
 *****************************************************************************/
 
-static struct aio_extra *create_aio_ex_read(files_struct *fsp, size_t buflen,
-                                           uint16 mid)
+static void smbd_aio_signal_handler(struct tevent_context *ev_ctx,
+                                   struct tevent_signal *se,
+                                   int signum, int count,
+                                   void *_info, void *private_data)
 {
-       struct aio_extra *aio_ex = SMB_MALLOC_P(struct aio_extra);
+       siginfo_t *info = (siginfo_t *)_info;
+       struct aio_extra *aio_ex = (struct aio_extra *)
+                               info->si_value.sival_ptr;
 
-       if (!aio_ex) {
-               return NULL;
-       }
-       ZERO_STRUCTP(aio_ex);
-       /* The output buffer stored in the aio_ex is the start of
-          the smb return buffer. The buffer used in the acb
-          is the start of the reply data portion of that buffer. */
-       aio_ex->outbuf = SMB_MALLOC_ARRAY(char, buflen);
-       if (!aio_ex->outbuf) {
-               SAFE_FREE(aio_ex);
-               return NULL;
-       }
-       DLIST_ADD(aio_list_head, aio_ex);
-       aio_ex->fsp = fsp;
-       aio_ex->read_req = True;
-       aio_ex->mid = mid;
-       return aio_ex;
+       smbd_aio_complete_aio_ex(aio_ex);
 }
 
-/****************************************************************************
- Create the extended aio struct we must keep around for the lifetime
- of the aio_write call.
-*****************************************************************************/
 
-static struct aio_extra *create_aio_ex_write(files_struct *fsp,
-                                            size_t inbuflen,
-                                            size_t outbuflen,
-                                            uint16 mid)
+static bool initialize_async_io_handler(void)
 {
-       struct aio_extra *aio_ex = SMB_MALLOC_P(struct aio_extra);
+       static bool tried_signal_setup = false;
 
-       if (!aio_ex) {
-               return NULL;
+       if (aio_signal_event) {
+               return true;
        }
-       ZERO_STRUCTP(aio_ex);
-
-       /* We need space for an output reply of outbuflen bytes. */
-       aio_ex->outbuf = SMB_MALLOC_ARRAY(char, outbuflen);
-       if (!aio_ex->outbuf) {
-               SAFE_FREE(aio_ex);
-               return NULL;
+       if (tried_signal_setup) {
+               return false;
        }
-
-       if (!(aio_ex->inbuf = SMB_MALLOC_ARRAY(char, inbuflen))) {
-               SAFE_FREE(aio_ex->outbuf);
-               SAFE_FREE(aio_ex);
-               return NULL;
+       tried_signal_setup = true;
+
+       aio_signal_event = tevent_add_signal(server_event_context(),
+                                            server_event_context(),
+                                            RT_SIGNAL_AIO, SA_SIGINFO,
+                                            smbd_aio_signal_handler,
+                                            NULL);
+       if (!aio_signal_event) {
+               DEBUG(10, ("Failed to setup RT_SIGNAL_AIO handler\n"));
+               return false;
        }
 
-       DLIST_ADD(aio_list_head, aio_ex);
-       aio_ex->fsp = fsp;
-       aio_ex->read_req = False;
-       aio_ex->mid = mid;
-       return aio_ex;
+       /* tevent supports 100 signal with SA_SIGINFO */
+       aio_pending_size = 100;
+       return true;
 }
 
-/****************************************************************************
- Delete the extended aio struct.
-*****************************************************************************/
+static int handle_aio_read_complete(struct aio_extra *aio_ex, int errcode);
+static int handle_aio_write_complete(struct aio_extra *aio_ex, int errcode);
+static int handle_aio_smb2_read_complete(struct aio_extra *aio_ex, int errcode);
+static int handle_aio_smb2_write_complete(struct aio_extra *aio_ex, int errcode);
 
-static void delete_aio_ex(struct aio_extra *aio_ex)
+static int aio_extra_destructor(struct aio_extra *aio_ex)
 {
        DLIST_REMOVE(aio_list_head, aio_ex);
-       SAFE_FREE(aio_ex->inbuf);
-       SAFE_FREE(aio_ex->outbuf);
-       SAFE_FREE(aio_ex);
+       return 0;
 }
 
 /****************************************************************************
- Given the aiocb struct find the extended aio struct containing it.
+ Create the extended aio struct we must keep around for the lifetime
+ of the aio call.
 *****************************************************************************/
 
-static struct aio_extra *find_aio_ex(uint16 mid)
+static struct aio_extra *create_aio_extra(TALLOC_CTX *mem_ctx,
+                                       files_struct *fsp,
+                                       size_t buflen)
 {
-       struct aio_extra *p;
+       struct aio_extra *aio_ex = talloc_zero(mem_ctx, struct aio_extra);
 
-       for( p = aio_list_head; p; p = p->next) {
-               if (mid == p->mid) {
-                       return p;
-               }
+       if (!aio_ex) {
+               return NULL;
        }
-       return NULL;
-}
-
-/****************************************************************************
- We can have these many aio buffers in flight.
-*****************************************************************************/
-
-#define AIO_PENDING_SIZE 10
-static sig_atomic_t signals_received;
-static int outstanding_aio_calls;
-static uint16 aio_pending_array[AIO_PENDING_SIZE];
-
-/****************************************************************************
- Signal handler when an aio request completes.
-*****************************************************************************/
-
-static void signal_handler(int sig, siginfo_t *info, void *unused)
-{
-       if (signals_received < AIO_PENDING_SIZE) {
-               aio_pending_array[signals_received] = info->si_value.sival_int;
-               signals_received++;
-       } /* Else signal is lost. */
-       sys_select_signal(RT_SIGNAL_AIO);
-}
-
-/****************************************************************************
- Is there a signal waiting ?
-*****************************************************************************/
-
-bool aio_finished(void)
-{
-       return (signals_received != 0);
-}
-
-/****************************************************************************
- Initialize the signal handler for aio read/write.
-*****************************************************************************/
 
-void initialize_async_io_handler(void)
-{
-       struct sigaction act;
-
-       ZERO_STRUCT(act);
-       act.sa_sigaction = signal_handler;
-       act.sa_flags = SA_SIGINFO;
-       sigemptyset( &act.sa_mask );
-       if (sigaction(RT_SIGNAL_AIO, &act, NULL) != 0) {
-                DEBUG(0,("Failed to setup RT_SIGNAL_AIO handler\n"));
-        }
+       /* The output buffer stored in the aio_ex is the start of
+          the smb return buffer. The buffer used in the acb
+          is the start of the reply data portion of that buffer. */
 
-       /* the signal can start off blocked due to a bug in bash */
-       BlockSignals(False, RT_SIGNAL_AIO);
+       if (buflen) {
+               aio_ex->outbuf = data_blob_talloc(aio_ex, NULL, buflen);
+               if (!aio_ex->outbuf.data) {
+                       TALLOC_FREE(aio_ex);
+                       return NULL;
+               }
+       }
+       DLIST_ADD(aio_list_head, aio_ex);
+       talloc_set_destructor(aio_ex, aio_extra_destructor);
+       aio_ex->fsp = fsp;
+       return aio_ex;
 }
 
 /****************************************************************************
  Set up an aio request from a SMBreadX call.
 *****************************************************************************/
 
-bool schedule_aio_read_and_X(connection_struct *conn,
-                            struct smb_request *req,
+NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
+                            struct smb_request *smbreq,
                             files_struct *fsp, SMB_OFF_T startpos,
                             size_t smb_maxcnt)
 {
@@ -201,28 +152,40 @@ bool schedule_aio_read_and_X(connection_struct *conn,
        SMB_STRUCT_AIOCB *a;
        size_t bufsize;
        size_t min_aio_read_size = lp_aio_read_size(SNUM(conn));
+       int ret;
+
+       /* Ensure aio is initialized. */
+       if (!initialize_async_io_handler()) {
+               return NT_STATUS_RETRY;
+       }
 
-       if (!min_aio_read_size || (smb_maxcnt < min_aio_read_size)) {
+       if (fsp->base_fsp != NULL) {
+               /* No AIO on streams yet */
+               DEBUG(10, ("AIO on streams not yet supported\n"));
+               return NT_STATUS_RETRY;
+       }
+
+       if ((!min_aio_read_size || (smb_maxcnt < min_aio_read_size))
+           && !SMB_VFS_AIO_FORCE(fsp)) {
                /* Too small a read for aio request. */
                DEBUG(10,("schedule_aio_read_and_X: read size (%u) too small "
                          "for minimum aio_read of %u\n",
                          (unsigned int)smb_maxcnt,
                          (unsigned int)min_aio_read_size ));
-               return False;
+               return NT_STATUS_RETRY;
        }
 
        /* Only do this on non-chained and non-chaining reads not using the
         * write cache. */
-        if (chain_size !=0 || (CVAL(req->inbuf,smb_vwv0) != 0xFF)
-           || (lp_write_cache_size(SNUM(conn)) != 0) ) {
-               return False;
+        if (req_is_in_chain(smbreq) || (lp_write_cache_size(SNUM(conn)) != 0)) {
+               return NT_STATUS_RETRY;
        }
 
-       if (outstanding_aio_calls >= AIO_PENDING_SIZE) {
+       if (outstanding_aio_calls >= aio_pending_size) {
                DEBUG(10,("schedule_aio_read_and_X: Already have %d aio "
                          "activities outstanding.\n",
                          outstanding_aio_calls ));
-               return False;
+               return NT_STATUS_RETRY;
        }
 
        /* The following is safe from integer wrap as we've already checked
@@ -230,164 +193,421 @@ bool schedule_aio_read_and_X(connection_struct *conn,
 
        bufsize = smb_size + 12 * 2 + smb_maxcnt;
 
-       if ((aio_ex = create_aio_ex_read(fsp, bufsize, req->mid)) == NULL) {
+       if ((aio_ex = create_aio_extra(NULL, fsp, bufsize)) == NULL) {
                DEBUG(10,("schedule_aio_read_and_X: malloc fail.\n"));
-               return False;
+               return NT_STATUS_NO_MEMORY;
        }
+       aio_ex->handle_completion = handle_aio_read_complete;
+
+       construct_reply_common_req(smbreq, (char *)aio_ex->outbuf.data);
+       srv_set_message((char *)aio_ex->outbuf.data, 12, 0, True);
+       SCVAL(aio_ex->outbuf.data,smb_vwv0,0xFF); /* Never a chained reply. */
+
+       init_strict_lock_struct(fsp, (uint64_t)smbreq->smbpid,
+               (uint64_t)startpos, (uint64_t)smb_maxcnt, READ_LOCK,
+               &aio_ex->lock);
 
-       construct_reply_common((char *)req->inbuf, aio_ex->outbuf);
-       srv_set_message((const char *)req->inbuf, aio_ex->outbuf, 12, 0, True);
-       SCVAL(aio_ex->outbuf,smb_vwv0,0xFF); /* Never a chained reply. */
+       /* Take the lock until the AIO completes. */
+       if (!SMB_VFS_STRICT_LOCK(conn, fsp, &aio_ex->lock)) {
+               TALLOC_FREE(aio_ex);
+               return NT_STATUS_FILE_LOCK_CONFLICT;
+       }
 
        a = &aio_ex->acb;
 
        /* Now set up the aio record for the read call. */
-       
+
        a->aio_fildes = fsp->fh->fd;
-       a->aio_buf = smb_buf(aio_ex->outbuf);
+       a->aio_buf = smb_buf(aio_ex->outbuf.data);
        a->aio_nbytes = smb_maxcnt;
        a->aio_offset = startpos;
        a->aio_sigevent.sigev_notify = SIGEV_SIGNAL;
        a->aio_sigevent.sigev_signo  = RT_SIGNAL_AIO;
-       a->aio_sigevent.sigev_value.sival_int = aio_ex->mid;
+       a->aio_sigevent.sigev_value.sival_ptr = aio_ex;
 
-       if (SMB_VFS_AIO_READ(fsp,a) == -1) {
+       ret = SMB_VFS_AIO_READ(fsp, a);
+       if (ret == -1) {
                DEBUG(0,("schedule_aio_read_and_X: aio_read failed. "
                         "Error %s\n", strerror(errno) ));
-               delete_aio_ex(aio_ex);
-               return False;
+               SMB_VFS_STRICT_UNLOCK(conn, fsp, &aio_ex->lock);
+               TALLOC_FREE(aio_ex);
+               return NT_STATUS_RETRY;
        }
 
+       outstanding_aio_calls++;
+       aio_ex->smbreq = talloc_move(aio_ex, &smbreq);
+
        DEBUG(10,("schedule_aio_read_and_X: scheduled aio_read for file %s, "
                  "offset %.0f, len = %u (mid = %u)\n",
-                 fsp->fsp_name, (double)startpos, (unsigned int)smb_maxcnt,
-                 (unsigned int)aio_ex->mid ));
+                 fsp_str_dbg(fsp), (double)startpos, (unsigned int)smb_maxcnt,
+                 (unsigned int)aio_ex->smbreq->mid ));
 
-       srv_defer_sign_response(aio_ex->mid);
-       outstanding_aio_calls++;
-       return True;
+       return NT_STATUS_OK;
 }
 
 /****************************************************************************
  Set up an aio request from a SMBwriteX call.
 *****************************************************************************/
 
-bool schedule_aio_write_and_X(connection_struct *conn,
-                             struct smb_request *req,
-                             files_struct *fsp, char *data,
+NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
+                             struct smb_request *smbreq,
+                             files_struct *fsp, const char *data,
                              SMB_OFF_T startpos,
                              size_t numtowrite)
 {
        struct aio_extra *aio_ex;
        SMB_STRUCT_AIOCB *a;
-       size_t inbufsize, outbufsize;
-       bool write_through = BITSETW(req->inbuf+smb_vwv7,0);
+       size_t bufsize;
        size_t min_aio_write_size = lp_aio_write_size(SNUM(conn));
+       int ret;
+
+       /* Ensure aio is initialized. */
+       if (!initialize_async_io_handler()) {
+               return NT_STATUS_RETRY;
+       }
+
+       if (fsp->base_fsp != NULL) {
+               /* No AIO on streams yet */
+               DEBUG(10, ("AIO on streams not yet supported\n"));
+               return NT_STATUS_RETRY;
+       }
 
-       if (!min_aio_write_size || (numtowrite < min_aio_write_size)) {
+       if ((!min_aio_write_size || (numtowrite < min_aio_write_size))
+           && !SMB_VFS_AIO_FORCE(fsp)) {
                /* Too small a write for aio request. */
                DEBUG(10,("schedule_aio_write_and_X: write size (%u) too "
                          "small for minimum aio_write of %u\n",
                          (unsigned int)numtowrite,
                          (unsigned int)min_aio_write_size ));
-               return False;
+               return NT_STATUS_RETRY;
        }
 
-       /* Only do this on non-chained and non-chaining reads not using the
+       /* Only do this on non-chained and non-chaining writes not using the
         * write cache. */
-        if (chain_size !=0 || (CVAL(req->inbuf,smb_vwv0) != 0xFF)
-           || (lp_write_cache_size(SNUM(conn)) != 0) ) {
-               return False;
+        if (req_is_in_chain(smbreq) || (lp_write_cache_size(SNUM(conn)) != 0)) {
+               return NT_STATUS_RETRY;
        }
 
-       if (outstanding_aio_calls >= AIO_PENDING_SIZE) {
+       if (outstanding_aio_calls >= aio_pending_size) {
                DEBUG(3,("schedule_aio_write_and_X: Already have %d aio "
                         "activities outstanding.\n",
                          outstanding_aio_calls ));
                DEBUG(10,("schedule_aio_write_and_X: failed to schedule "
                          "aio_write for file %s, offset %.0f, len = %u "
                          "(mid = %u)\n",
-                         fsp->fsp_name, (double)startpos,
+                         fsp_str_dbg(fsp), (double)startpos,
                          (unsigned int)numtowrite,
-                         (unsigned int)req->mid ));
-               return False;
+                         (unsigned int)smbreq->mid ));
+               return NT_STATUS_RETRY;
        }
 
-       inbufsize =  smb_len(req->inbuf) + 4;
-       reply_outbuf(req, 6, 0);
-       outbufsize = smb_len(req->outbuf) + 4;
-       if (!(aio_ex = create_aio_ex_write(fsp, inbufsize, outbufsize,
-                                          req->mid))) {
+       bufsize = smb_size + 6*2;
+
+       if (!(aio_ex = create_aio_extra(NULL, fsp, bufsize))) {
                DEBUG(0,("schedule_aio_write_and_X: malloc fail.\n"));
-               return False;
+               return NT_STATUS_NO_MEMORY;
        }
+       aio_ex->handle_completion = handle_aio_write_complete;
+       aio_ex->write_through = BITSETW(smbreq->vwv+7,0);
 
-       /* Copy the SMB header already setup in outbuf. */
-       memcpy(aio_ex->inbuf, req->inbuf, inbufsize);
+       construct_reply_common_req(smbreq, (char *)aio_ex->outbuf.data);
+       srv_set_message((char *)aio_ex->outbuf.data, 6, 0, True);
+       SCVAL(aio_ex->outbuf.data,smb_vwv0,0xFF); /* Never a chained reply. */
 
-       /* Copy the SMB header already setup in outbuf. */
-       memcpy(aio_ex->outbuf, req->outbuf, outbufsize);
-       TALLOC_FREE(req->outbuf);
-       SCVAL(aio_ex->outbuf,smb_vwv0,0xFF); /* Never a chained reply. */
+       init_strict_lock_struct(fsp, (uint64_t)smbreq->smbpid,
+               (uint64_t)startpos, (uint64_t)numtowrite, WRITE_LOCK,
+               &aio_ex->lock);
+
+       /* Take the lock until the AIO completes. */
+       if (!SMB_VFS_STRICT_LOCK(conn, fsp, &aio_ex->lock)) {
+               TALLOC_FREE(aio_ex);
+               return NT_STATUS_FILE_LOCK_CONFLICT;
+       }
 
        a = &aio_ex->acb;
 
        /* Now set up the aio record for the write call. */
-       
+
        a->aio_fildes = fsp->fh->fd;
-       a->aio_buf = aio_ex->inbuf + (PTR_DIFF(data, req->inbuf));
+       a->aio_buf = discard_const_p(char, data);
        a->aio_nbytes = numtowrite;
        a->aio_offset = startpos;
        a->aio_sigevent.sigev_notify = SIGEV_SIGNAL;
        a->aio_sigevent.sigev_signo  = RT_SIGNAL_AIO;
-       a->aio_sigevent.sigev_value.sival_int = aio_ex->mid;
+       a->aio_sigevent.sigev_value.sival_ptr = aio_ex;
 
-       if (SMB_VFS_AIO_WRITE(fsp,a) == -1) {
+       ret = SMB_VFS_AIO_WRITE(fsp, a);
+       if (ret == -1) {
                DEBUG(3,("schedule_aio_wrote_and_X: aio_write failed. "
                         "Error %s\n", strerror(errno) ));
-               delete_aio_ex(aio_ex);
-               return False;
+               SMB_VFS_STRICT_UNLOCK(conn, fsp, &aio_ex->lock);
+               TALLOC_FREE(aio_ex);
+               return NT_STATUS_RETRY;
        }
 
-       if (!write_through && !lp_syncalways(SNUM(fsp->conn))
+       outstanding_aio_calls++;
+       aio_ex->smbreq = talloc_move(aio_ex, &smbreq);
+
+       /* This should actually be improved to span the write. */
+       contend_level2_oplocks_begin(fsp, LEVEL2_CONTEND_WRITE);
+       contend_level2_oplocks_end(fsp, LEVEL2_CONTEND_WRITE);
+
+       if (!aio_ex->write_through && !lp_syncalways(SNUM(fsp->conn))
            && fsp->aio_write_behind) {
                /* Lie to the client and immediately claim we finished the
                 * write. */
-               SSVAL(aio_ex->outbuf,smb_vwv2,numtowrite);
-                SSVAL(aio_ex->outbuf,smb_vwv4,(numtowrite>>16)&1);
-               show_msg(aio_ex->outbuf);
-               if (!send_smb(smbd_server_fd(),aio_ex->outbuf)) {
-                       exit_server_cleanly("handle_aio_write: send_smb "
-                                           "failed.");
+               SSVAL(aio_ex->outbuf.data,smb_vwv2,numtowrite);
+                SSVAL(aio_ex->outbuf.data,smb_vwv4,(numtowrite>>16)&1);
+               show_msg((char *)aio_ex->outbuf.data);
+               if (!srv_send_smb(aio_ex->smbreq->sconn,
+                               (char *)aio_ex->outbuf.data,
+                               true, aio_ex->smbreq->seqnum+1,
+                               IS_CONN_ENCRYPTED(fsp->conn),
+                               &aio_ex->smbreq->pcd)) {
+                       exit_server_cleanly("schedule_aio_write_and_X: "
+                                           "srv_send_smb failed.");
                }
                DEBUG(10,("schedule_aio_write_and_X: scheduled aio_write "
-                         "behind for file %s\n", fsp->fsp_name ));
-       } else {
-               srv_defer_sign_response(aio_ex->mid);
+                         "behind for file %s\n", fsp_str_dbg(fsp)));
        }
-       outstanding_aio_calls++;
 
        DEBUG(10,("schedule_aio_write_and_X: scheduled aio_write for file "
                  "%s, offset %.0f, len = %u (mid = %u) "
                  "outstanding_aio_calls = %d\n",
-                 fsp->fsp_name, (double)startpos, (unsigned int)numtowrite,
-                 (unsigned int)aio_ex->mid, outstanding_aio_calls ));
+                 fsp_str_dbg(fsp), (double)startpos, (unsigned int)numtowrite,
+                 (unsigned int)aio_ex->smbreq->mid, outstanding_aio_calls ));
 
-       return True;
+       return NT_STATUS_OK;
 }
 
+/****************************************************************************
+ Set up an aio request from a SMB2 read call.
+*****************************************************************************/
+
+NTSTATUS schedule_smb2_aio_read(connection_struct *conn,
+                               struct smb_request *smbreq,
+                               files_struct *fsp,
+                               TALLOC_CTX *ctx,
+                               DATA_BLOB *preadbuf,
+                               SMB_OFF_T startpos,
+                               size_t smb_maxcnt)
+{
+       struct aio_extra *aio_ex;
+       SMB_STRUCT_AIOCB *a;
+       size_t min_aio_read_size = lp_aio_read_size(SNUM(conn));
+       int ret;
+
+       /* Ensure aio is initialized. */
+       if (!initialize_async_io_handler()) {
+               return NT_STATUS_RETRY;
+       }
+
+       if (fsp->base_fsp != NULL) {
+               /* No AIO on streams yet */
+               DEBUG(10, ("AIO on streams not yet supported\n"));
+               return NT_STATUS_RETRY;
+       }
+
+       if ((!min_aio_read_size || (smb_maxcnt < min_aio_read_size))
+           && !SMB_VFS_AIO_FORCE(fsp)) {
+               /* Too small a read for aio request. */
+               DEBUG(10,("smb2: read size (%u) too small "
+                       "for minimum aio_read of %u\n",
+                       (unsigned int)smb_maxcnt,
+                       (unsigned int)min_aio_read_size ));
+               return NT_STATUS_RETRY;
+       }
+
+       /* Only do this on reads not using the write cache. */
+       if (lp_write_cache_size(SNUM(conn)) != 0) {
+               return NT_STATUS_RETRY;
+       }
+
+       if (outstanding_aio_calls >= aio_pending_size) {
+               DEBUG(10,("smb2: Already have %d aio "
+                       "activities outstanding.\n",
+                       outstanding_aio_calls ));
+               return NT_STATUS_RETRY;
+       }
+
+       /* Create the out buffer. */
+       *preadbuf = data_blob_talloc(ctx, NULL, smb_maxcnt);
+       if (preadbuf->data == NULL) {
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       if (!(aio_ex = create_aio_extra(smbreq->smb2req, fsp, 0))) {
+               return NT_STATUS_NO_MEMORY;
+       }
+       aio_ex->handle_completion = handle_aio_smb2_read_complete;
+
+       init_strict_lock_struct(fsp, (uint64_t)smbreq->smbpid,
+               (uint64_t)startpos, (uint64_t)smb_maxcnt, READ_LOCK,
+               &aio_ex->lock);
+
+       /* Take the lock until the AIO completes. */
+       if (!SMB_VFS_STRICT_LOCK(conn, fsp, &aio_ex->lock)) {
+               TALLOC_FREE(aio_ex);
+               return NT_STATUS_FILE_LOCK_CONFLICT;
+       }
+
+       a = &aio_ex->acb;
+
+       /* Now set up the aio record for the read call. */
+
+       a->aio_fildes = fsp->fh->fd;
+       a->aio_buf = preadbuf->data;
+       a->aio_nbytes = smb_maxcnt;
+       a->aio_offset = startpos;
+       a->aio_sigevent.sigev_notify = SIGEV_SIGNAL;
+       a->aio_sigevent.sigev_signo  = RT_SIGNAL_AIO;
+       a->aio_sigevent.sigev_value.sival_ptr = aio_ex;
+
+       ret = SMB_VFS_AIO_READ(fsp, a);
+       if (ret == -1) {
+               DEBUG(0,("smb2: aio_read failed. "
+                       "Error %s\n", strerror(errno) ));
+               SMB_VFS_STRICT_UNLOCK(conn, fsp, &aio_ex->lock);
+               TALLOC_FREE(aio_ex);
+               return NT_STATUS_RETRY;
+       }
+
+       outstanding_aio_calls++;
+       /* We don't need talloc_move here as both aio_ex and
+        * smbreq are children of smbreq->smb2req. */
+       aio_ex->smbreq = smbreq;
+
+       DEBUG(10,("smb2: scheduled aio_read for file %s, "
+               "offset %.0f, len = %u (mid = %u)\n",
+               fsp_str_dbg(fsp), (double)startpos, (unsigned int)smb_maxcnt,
+               (unsigned int)aio_ex->smbreq->mid ));
+
+       return NT_STATUS_OK;
+}
+
+/****************************************************************************
+ Set up an aio request from a SMB2write call.
+*****************************************************************************/
+
+NTSTATUS schedule_aio_smb2_write(connection_struct *conn,
+                               struct smb_request *smbreq,
+                               files_struct *fsp,
+                               uint64_t in_offset,
+                               DATA_BLOB in_data,
+                               bool write_through)
+{
+       struct aio_extra *aio_ex = NULL;
+       SMB_STRUCT_AIOCB *a = NULL;
+       size_t min_aio_write_size = lp_aio_write_size(SNUM(conn));
+       int ret;
+
+       /* Ensure aio is initialized. */
+       if (!initialize_async_io_handler()) {
+               return NT_STATUS_RETRY;
+       }
+
+       if (fsp->base_fsp != NULL) {
+               /* No AIO on streams yet */
+               DEBUG(10, ("AIO on streams not yet supported\n"));
+               return NT_STATUS_RETRY;
+       }
+
+       if ((!min_aio_write_size || (in_data.length < min_aio_write_size))
+           && !SMB_VFS_AIO_FORCE(fsp)) {
+               /* Too small a write for aio request. */
+               DEBUG(10,("smb2: write size (%u) too "
+                       "small for minimum aio_write of %u\n",
+                       (unsigned int)in_data.length,
+                       (unsigned int)min_aio_write_size ));
+               return NT_STATUS_RETRY;
+       }
+
+       /* Only do this on writes not using the write cache. */
+       if (lp_write_cache_size(SNUM(conn)) != 0) {
+               return NT_STATUS_RETRY;
+       }
+
+       if (outstanding_aio_calls >= aio_pending_size) {
+               DEBUG(3,("smb2: Already have %d aio "
+                       "activities outstanding.\n",
+                       outstanding_aio_calls ));
+               return NT_STATUS_RETRY;
+       }
+
+       if (!(aio_ex = create_aio_extra(smbreq->smb2req, fsp, 0))) {
+               return NT_STATUS_NO_MEMORY;
+       }
+
+       aio_ex->handle_completion = handle_aio_smb2_write_complete;
+       aio_ex->write_through = write_through;
+
+       init_strict_lock_struct(fsp, (uint64_t)smbreq->smbpid,
+               in_offset, (uint64_t)in_data.length, WRITE_LOCK,
+               &aio_ex->lock);
+
+       /* Take the lock until the AIO completes. */
+       if (!SMB_VFS_STRICT_LOCK(conn, fsp, &aio_ex->lock)) {
+               TALLOC_FREE(aio_ex);
+               return NT_STATUS_FILE_LOCK_CONFLICT;
+       }
+
+       a = &aio_ex->acb;
+
+       /* Now set up the aio record for the write call. */
+
+       a->aio_fildes = fsp->fh->fd;
+       a->aio_buf = in_data.data;
+       a->aio_nbytes = in_data.length;
+       a->aio_offset = in_offset;
+       a->aio_sigevent.sigev_notify = SIGEV_SIGNAL;
+       a->aio_sigevent.sigev_signo  = RT_SIGNAL_AIO;
+       a->aio_sigevent.sigev_value.sival_ptr = aio_ex;
+
+       ret = SMB_VFS_AIO_WRITE(fsp, a);
+       if (ret == -1) {
+               DEBUG(3,("smb2: aio_write failed. "
+                       "Error %s\n", strerror(errno) ));
+               SMB_VFS_STRICT_UNLOCK(conn, fsp, &aio_ex->lock);
+               TALLOC_FREE(aio_ex);
+               return NT_STATUS_RETRY;
+       }
+
+       outstanding_aio_calls++;
+       /* We don't need talloc_move here as both aio_ex and
+       * smbreq are children of smbreq->smb2req. */
+       aio_ex->smbreq = smbreq;
+
+       /* This should actually be improved to span the write. */
+       contend_level2_oplocks_begin(fsp, LEVEL2_CONTEND_WRITE);
+       contend_level2_oplocks_end(fsp, LEVEL2_CONTEND_WRITE);
+
+       /*
+        * We don't want to do write behind due to ownership
+        * issues of the request structs. Maybe add it if I
+        * figure those out. JRA.
+        */
+
+       DEBUG(10,("smb2: scheduled aio_write for file "
+               "%s, offset %.0f, len = %u (mid = %u) "
+               "outstanding_aio_calls = %d\n",
+               fsp_str_dbg(fsp),
+               (double)in_offset,
+               (unsigned int)in_data.length,
+               (unsigned int)aio_ex->smbreq->mid,
+               outstanding_aio_calls ));
+
+       return NT_STATUS_OK;
+}
 
 /****************************************************************************
  Complete the read and return the data or error back to the client.
  Returns errno or zero if all ok.
 *****************************************************************************/
 
-static int handle_aio_read_complete(struct aio_extra *aio_ex)
+static int handle_aio_read_complete(struct aio_extra *aio_ex, int errcode)
 {
-       int ret = 0;
        int outsize;
-       char *outbuf = aio_ex->outbuf;
-       const char *inbuf = aio_ex->inbuf;
+       char *outbuf = (char *)aio_ex->outbuf.data;
        char *data = smb_buf(outbuf);
        ssize_t nread = SMB_VFS_AIO_RETURN(aio_ex->fsp,&aio_ex->acb);
 
@@ -397,60 +617,55 @@ static int handle_aio_read_complete(struct aio_extra *aio_ex)
                   will return an error. Hopefully this is
                   true.... JRA. */
 
-               /* If errno is ECANCELED then don't return anything to the
-                * client. */
-               if (errno == ECANCELED) {
-                       srv_cancel_sign_response(aio_ex->mid);
-                       return 0;
-               }
-
-               DEBUG( 3,( "handle_aio_read_complete: file %s nread == -1. "
+               DEBUG( 3,( "handle_aio_read_complete: file %s nread == %d. "
                           "Error = %s\n",
-                          aio_ex->fsp->fsp_name, strerror(errno) ));
+                          fsp_str_dbg(aio_ex->fsp), (int)nread, strerror(errcode)));
 
-               ret = errno;
-               ERROR_NT(map_nt_error_from_unix(ret));
-               outsize = srv_set_message(inbuf,outbuf,0,0,true);
+               ERROR_NT(map_nt_error_from_unix(errcode));
+               outsize = srv_set_message(outbuf,0,0,true);
        } else {
-               outsize = srv_set_message(inbuf, outbuf,12,nread,False);
+               outsize = srv_set_message(outbuf,12,nread,False);
                SSVAL(outbuf,smb_vwv2,0xFFFF); /* Remaining - must be * -1. */
                SSVAL(outbuf,smb_vwv5,nread);
                SSVAL(outbuf,smb_vwv6,smb_offset(data,outbuf));
                SSVAL(outbuf,smb_vwv7,((nread >> 16) & 1));
                SSVAL(smb_buf(outbuf),-2,nread);
 
+               aio_ex->fsp->fh->pos = aio_ex->acb.aio_offset + nread;
+               aio_ex->fsp->fh->position_information = aio_ex->fsp->fh->pos;
+
                DEBUG( 3, ( "handle_aio_read_complete file %s max=%d "
                            "nread=%d\n",
-                           aio_ex->fsp->fsp_name,
+                           fsp_str_dbg(aio_ex->fsp),
                            (int)aio_ex->acb.aio_nbytes, (int)nread ) );
 
        }
-       _smb_setlen(outbuf,outsize - 4);
+       smb_setlen(outbuf,outsize - 4);
        show_msg(outbuf);
-       if (!send_smb(smbd_server_fd(),outbuf)) {
-               exit_server_cleanly("handle_aio_read_complete: send_smb "
+       if (!srv_send_smb(aio_ex->smbreq->sconn, outbuf,
+                       true, aio_ex->smbreq->seqnum+1,
+                       IS_CONN_ENCRYPTED(aio_ex->fsp->conn), NULL)) {
+               exit_server_cleanly("handle_aio_read_complete: srv_send_smb "
                                    "failed.");
        }
 
        DEBUG(10,("handle_aio_read_complete: scheduled aio_read completed "
                  "for file %s, offset %.0f, len = %u\n",
-                 aio_ex->fsp->fsp_name, (double)aio_ex->acb.aio_offset,
+                 fsp_str_dbg(aio_ex->fsp), (double)aio_ex->acb.aio_offset,
                  (unsigned int)nread ));
 
-       return ret;
+       return errcode;
 }
 
 /****************************************************************************
  Complete the write and return the data or error back to the client.
- Returns errno or zero if all ok.
+ Returns error code or zero if all ok.
 *****************************************************************************/
 
-static int handle_aio_write_complete(struct aio_extra *aio_ex)
+static int handle_aio_write_complete(struct aio_extra *aio_ex, int errcode)
 {
-       int ret = 0;
        files_struct *fsp = aio_ex->fsp;
-       char *outbuf = aio_ex->outbuf;
-       const char *inbuf = aio_ex->inbuf;
+       char *outbuf = (char *)aio_ex->outbuf.data;
        ssize_t numtowrite = aio_ex->acb.aio_nbytes;
        ssize_t nwritten = SMB_VFS_AIO_RETURN(fsp,&aio_ex->acb);
 
@@ -460,22 +675,22 @@ static int handle_aio_write_complete(struct aio_extra *aio_ex)
                                DEBUG(5,("handle_aio_write_complete: "
                                         "aio_write_behind failed ! File %s "
                                         "is corrupt ! Error %s\n",
-                                        fsp->fsp_name, strerror(errno) ));
-                               ret = errno;
+                                        fsp_str_dbg(fsp), strerror(errcode)));
                        } else {
                                DEBUG(0,("handle_aio_write_complete: "
                                         "aio_write_behind failed ! File %s "
                                         "is corrupt ! Wanted %u bytes but "
-                                        "only wrote %d\n", fsp->fsp_name,
+                                        "only wrote %d\n", fsp_str_dbg(fsp),
                                         (unsigned int)numtowrite,
                                         (int)nwritten ));
-                               ret = EIO;
+                               errcode = EIO;
                        }
                } else {
                        DEBUG(10,("handle_aio_write_complete: "
                                  "aio_write_behind completed for file %s\n",
-                                 fsp->fsp_name ));
+                                 fsp_str_dbg(fsp)));
                }
+               /* TODO: should no return 0 in case of an error !!! */
                return 0;
        }
 
@@ -485,21 +700,12 @@ static int handle_aio_write_complete(struct aio_extra *aio_ex)
        if(nwritten == -1) {
                DEBUG( 3,( "handle_aio_write: file %s wanted %u bytes. "
                           "nwritten == %d. Error = %s\n",
-                          fsp->fsp_name, (unsigned int)numtowrite,
-                          (int)nwritten, strerror(errno) ));
-
-               /* If errno is ECANCELED then don't return anything to the
-                * client. */
-               if (errno == ECANCELED) {
-                       srv_cancel_sign_response(aio_ex->mid);
-                       return 0;
-               }
+                          fsp_str_dbg(fsp), (unsigned int)numtowrite,
+                          (int)nwritten, strerror(errcode) ));
 
-               ret = errno;
-               ERROR_BOTH(ERRHRD, ERRdiskfull, map_nt_error_from_unix(ret));
-               srv_set_message(inbuf,outbuf,0,0,true);
+               ERROR_NT(map_nt_error_from_unix(errcode));
+               srv_set_message(outbuf,0,0,true);
         } else {
-               bool write_through = BITSETW(aio_ex->inbuf+smb_vwv7,0);
                NTSTATUS status;
 
                SSVAL(outbuf,smb_vwv2,nwritten);
@@ -511,27 +717,105 @@ static int handle_aio_write_complete(struct aio_extra *aio_ex)
 
                DEBUG(3,("handle_aio_write: fnum=%d num=%d wrote=%d\n",
                         fsp->fnum, (int)numtowrite, (int)nwritten));
-               status = sync_file(fsp->conn,fsp, write_through);
+               status = sync_file(fsp->conn,fsp, aio_ex->write_through);
                if (!NT_STATUS_IS_OK(status)) {
-                       ret = errno;
-                       ERROR_BOTH(ERRHRD, ERRdiskfull, map_nt_error_from_unix(ret));
-                       srv_set_message(inbuf,outbuf,0,0,true);
+                       errcode = errno;
+                       ERROR_BOTH(map_nt_error_from_unix(errcode),
+                                  ERRHRD, ERRdiskfull);
+                       srv_set_message(outbuf,0,0,true);
                        DEBUG(5,("handle_aio_write: sync_file for %s returned %s\n",
-                               fsp->fsp_name, nt_errstr(status) ));
+                                fsp_str_dbg(fsp), nt_errstr(status)));
                }
+
+               aio_ex->fsp->fh->pos = aio_ex->acb.aio_offset + nwritten;
        }
 
        show_msg(outbuf);
-       if (!send_smb(smbd_server_fd(),outbuf)) {
-               exit_server_cleanly("handle_aio_write: send_smb failed.");
+       if (!srv_send_smb(aio_ex->smbreq->sconn, outbuf,
+                         true, aio_ex->smbreq->seqnum+1,
+                         IS_CONN_ENCRYPTED(fsp->conn),
+                         NULL)) {
+               exit_server_cleanly("handle_aio_write_complete: "
+                                   "srv_send_smb failed.");
        }
 
        DEBUG(10,("handle_aio_write_complete: scheduled aio_write completed "
                  "for file %s, offset %.0f, requested %u, written = %u\n",
-                 fsp->fsp_name, (double)aio_ex->acb.aio_offset,
+                 fsp_str_dbg(fsp), (double)aio_ex->acb.aio_offset,
                  (unsigned int)numtowrite, (unsigned int)nwritten ));
 
-       return ret;
+       return errcode;
+}
+
+/****************************************************************************
+ Complete the read and return the data or error back to the client.
+ Returns errno or zero if all ok.
+*****************************************************************************/
+
+static int handle_aio_smb2_read_complete(struct aio_extra *aio_ex, int errcode)
+{
+       NTSTATUS status;
+       struct tevent_req *subreq = aio_ex->smbreq->smb2req->subreq;
+       ssize_t nread = SMB_VFS_AIO_RETURN(aio_ex->fsp,&aio_ex->acb);
+
+       /* Common error or success code processing for async or sync
+          read returns. */
+
+       status = smb2_read_complete(subreq, nread, errcode);
+
+       if (nread > 0) {
+               aio_ex->fsp->fh->pos = aio_ex->acb.aio_offset + nread;
+               aio_ex->fsp->fh->position_information = aio_ex->fsp->fh->pos;
+       }
+
+       DEBUG(10,("smb2: scheduled aio_read completed "
+               "for file %s, offset %.0f, len = %u "
+               "(errcode = %d, NTSTATUS = %s)\n",
+               fsp_str_dbg(aio_ex->fsp),
+               (double)aio_ex->acb.aio_offset,
+               (unsigned int)nread,
+               errcode,
+               nt_errstr(status) ));
+
+       if (!NT_STATUS_IS_OK(status)) {
+               tevent_req_nterror(subreq, status);
+       }
+
+       tevent_req_done(subreq);
+       return errcode;
+}
+
+/****************************************************************************
+ Complete the SMB2 write and return the data or error back to the client.
+ Returns error code or zero if all ok.
+*****************************************************************************/
+
+static int handle_aio_smb2_write_complete(struct aio_extra *aio_ex, int errcode)
+{
+       files_struct *fsp = aio_ex->fsp;
+       ssize_t numtowrite = aio_ex->acb.aio_nbytes;
+       ssize_t nwritten = SMB_VFS_AIO_RETURN(fsp,&aio_ex->acb);
+       struct tevent_req *subreq = aio_ex->smbreq->smb2req->subreq;
+       NTSTATUS status;
+
+       status = smb2_write_complete(subreq, nwritten, errcode);
+
+       DEBUG(10,("smb2: scheduled aio_write completed "
+               "for file %s, offset %.0f, requested %u, "
+               "written = %u (errcode = %d, NTSTATUS = %s)\n",
+               fsp_str_dbg(fsp),
+               (double)aio_ex->acb.aio_offset,
+               (unsigned int)numtowrite,
+               (unsigned int)nwritten,
+               errcode,
+               nt_errstr(status) ));
+
+       if (!NT_STATUS_IS_OK(status)) {
+               tevent_req_nterror(subreq, status);
+       }
+
+       tevent_req_done(subreq);
+       return errcode;
 }
 
 /****************************************************************************
@@ -541,22 +825,39 @@ static int handle_aio_write_complete(struct aio_extra *aio_ex)
 
 static bool handle_aio_completed(struct aio_extra *aio_ex, int *perr)
 {
+       files_struct *fsp = NULL;
        int err;
 
+       if(!aio_ex) {
+               DEBUG(3, ("handle_aio_completed: Non-existing aio_ex passed\n"));
+               return false;
+       }
+
+       fsp = aio_ex->fsp;
+
        /* Ensure the operation has really completed. */
-       if (SMB_VFS_AIO_ERROR(aio_ex->fsp, &aio_ex->acb) == EINPROGRESS) {
-               DEBUG(10,( "handle_aio_completed: operation mid %u still in "
-                          "process for file %s\n",
-                          aio_ex->mid, aio_ex->fsp->fsp_name ));
+       err = SMB_VFS_AIO_ERROR(fsp, &aio_ex->acb);
+       if (err == EINPROGRESS) {
+               DEBUG(10,( "handle_aio_completed: operation mid %llu still in "
+                       "process for file %s\n",
+                       (unsigned long long)aio_ex->smbreq->mid,
+                       fsp_str_dbg(aio_ex->fsp)));
                return False;
        }
 
-       if (aio_ex->read_req) {
-               err = handle_aio_read_complete(aio_ex);
-       } else {
-               err = handle_aio_write_complete(aio_ex);
-       }
+       /* Unlock now we're done. */
+       SMB_VFS_STRICT_UNLOCK(fsp->conn, fsp, &aio_ex->lock);
+
+       if (err == ECANCELED) {
+               /* If error is ECANCELED then don't return anything to the
+                * client. */
+               DEBUG(10,( "handle_aio_completed: operation mid %llu"
+                       " canceled\n",
+                       (unsigned long long)aio_ex->smbreq->mid));
+               return True;
+        }
 
+       err = aio_ex->handle_completion(aio_ex, err);
        if (err) {
                *perr = err; /* Only save non-zero errors. */
        }
@@ -566,60 +867,33 @@ static bool handle_aio_completed(struct aio_extra *aio_ex, int *perr)
 
 /****************************************************************************
  Handle any aio completion inline.
- Returns non-zero errno if fail or zero if all ok.
 *****************************************************************************/
 
-int process_aio_queue(void)
+void smbd_aio_complete_aio_ex(struct aio_extra *aio_ex)
 {
-       int i;
+       files_struct *fsp = NULL;
        int ret = 0;
 
-       BlockSignals(True, RT_SIGNAL_AIO);
+       outstanding_aio_calls--;
 
-       DEBUG(10,("process_aio_queue: signals_received = %d\n",
-                 (int)signals_received));
-       DEBUG(10,("process_aio_queue: outstanding_aio_calls = %d\n",
-                 outstanding_aio_calls));
+       DEBUG(10,("smbd_aio_complete_mid: mid[%llu]\n",
+               (unsigned long long)aio_ex->smbreq->mid));
 
-       if (!signals_received) {
-               BlockSignals(False, RT_SIGNAL_AIO);
-               return 0;
+       fsp = aio_ex->fsp;
+       if (fsp == NULL) {
+               /* file was closed whilst I/O was outstanding. Just
+                * ignore. */
+               DEBUG( 3,( "smbd_aio_complete_mid: file closed whilst "
+                       "aio outstanding (mid[%llu]).\n",
+                       (unsigned long long)aio_ex->smbreq->mid));
+               return;
        }
 
-       /* Drain all the complete aio_reads. */
-       for (i = 0; i < signals_received; i++) {
-               uint16 mid = aio_pending_array[i];
-               files_struct *fsp = NULL;
-               struct aio_extra *aio_ex = find_aio_ex(mid);
-
-               if (!aio_ex) {
-                       DEBUG(3,("process_aio_queue: Can't find record to "
-                                "match mid %u.\n", (unsigned int)mid));
-                       srv_cancel_sign_response(mid);
-                       continue;
-               }
-
-               fsp = aio_ex->fsp;
-               if (fsp == NULL) {
-                       /* file was closed whilst I/O was outstanding. Just
-                        * ignore. */
-                       DEBUG( 3,( "process_aio_queue: file closed whilst "
-                                  "aio outstanding.\n"));
-                       srv_cancel_sign_response(mid);
-                       continue;
-               }
-
-               if (!handle_aio_completed(aio_ex, &ret)) {
-                       continue;
-               }
-
-               delete_aio_ex(aio_ex);
+       if (!handle_aio_completed(aio_ex, &ret)) {
+               return;
        }
 
-       outstanding_aio_calls -= signals_received;
-       signals_received = 0;
-       BlockSignals(False, RT_SIGNAL_AIO);
-       return ret;
+       TALLOC_FREE(aio_ex);
 }
 
 /****************************************************************************
@@ -635,7 +909,7 @@ int wait_for_aio_completion(files_struct *fsp)
        struct aio_extra *aio_ex;
        const SMB_STRUCT_AIOCB **aiocb_list;
        int aio_completion_count = 0;
-       time_t start_time = time(NULL);
+       time_t start_time = time_mono(NULL);
        int seconds_left;
 
        for (seconds_left = SMB_TIME_FOR_AIO_COMPLETE_WAIT;
@@ -685,7 +959,7 @@ int wait_for_aio_completion(files_struct *fsp)
 
                DEBUG(10,("wait_for_aio_completion: returned err = %d, "
                          "errno = %s\n", err, strerror(errno) ));
-               
+
                if (err == -1 && errno == EAGAIN) {
                        DEBUG(0,("wait_for_aio_completion: aio_suspend timed "
                                 "out waiting for %d events after a wait of "
@@ -700,26 +974,17 @@ int wait_for_aio_completion(files_struct *fsp)
                /* One or more events might have completed - process them if
                 * so. */
                for( i = 0; i < aio_completion_count; i++) {
-                       uint16 mid = aiocb_list[i]->aio_sigevent.sigev_value.sival_int;
-
-                       aio_ex = find_aio_ex(mid);
-
-                       if (!aio_ex) {
-                               DEBUG(0, ("wait_for_aio_completion: mid %u "
-                                         "doesn't match an aio record\n",
-                                         (unsigned int)mid ));
-                               continue;
-                       }
+                       aio_ex = (struct aio_extra *)aiocb_list[i]->aio_sigevent.sigev_value.sival_ptr;
 
                        if (!handle_aio_completed(aio_ex, &err)) {
                                continue;
                        }
-                       delete_aio_ex(aio_ex);
+                       TALLOC_FREE(aio_ex);
                }
 
                SAFE_FREE(aiocb_list);
                seconds_left = SMB_TIME_FOR_AIO_COMPLETE_WAIT
-                       - (time(NULL) - start_time);
+                       - (time_mono(NULL) - start_time);
        }
 
        /* We timed out - we don't know why. Return ret if already an error,
@@ -741,10 +1006,13 @@ void cancel_aio_by_fsp(files_struct *fsp)
 
        for( aio_ex = aio_list_head; aio_ex; aio_ex = aio_ex->next) {
                if (aio_ex->fsp == fsp) {
+                       /* Unlock now we're done. */
+                       SMB_VFS_STRICT_UNLOCK(fsp->conn, fsp, &aio_ex->lock);
+
                        /* Don't delete the aio_extra record as we may have
                           completed and don't yet know it. Just do the
                           aio_cancel call and return. */
-                       SMB_VFS_AIO_CANCEL(fsp,fsp->fh->fd, &aio_ex->acb);
+                       SMB_VFS_AIO_CANCEL(fsp, &aio_ex->acb);
                        aio_ex->fsp = NULL; /* fsp will be closed when we
                                             * return. */
                }
@@ -752,35 +1020,42 @@ void cancel_aio_by_fsp(files_struct *fsp)
 }
 
 #else
-bool aio_finished(void)
-{
-       return False;
-}
-
-void initialize_async_io_handler(void)
+NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
+                            struct smb_request *smbreq,
+                            files_struct *fsp, SMB_OFF_T startpos,
+                            size_t smb_maxcnt)
 {
+       return NT_STATUS_RETRY;
 }
 
-int process_aio_queue(void)
+NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
+                             struct smb_request *smbreq,
+                             files_struct *fsp, const char *data,
+                             SMB_OFF_T startpos,
+                             size_t numtowrite)
 {
-       return False;
+       return NT_STATUS_RETRY;
 }
 
-bool schedule_aio_read_and_X(connection_struct *conn,
-                            struct smb_request *req,
-                            files_struct *fsp, SMB_OFF_T startpos,
-                            size_t smb_maxcnt)
+NTSTATUS schedule_smb2_aio_read(connection_struct *conn,
+                                struct smb_request *smbreq,
+                                files_struct *fsp,
+                               TALLOC_CTX *ctx,
+                               DATA_BLOB *preadbuf,
+                                SMB_OFF_T startpos,
+                                size_t smb_maxcnt)
 {
-       return False;
+       return NT_STATUS_RETRY;
 }
 
-bool schedule_aio_write_and_X(connection_struct *conn,
-                             struct smb_request *req,
-                             files_struct *fsp, char *data,
-                             SMB_OFF_T startpos,
-                             size_t numtowrite)
+NTSTATUS schedule_aio_smb2_write(connection_struct *conn,
+                               struct smb_request *smbreq,
+                               files_struct *fsp,
+                               uint64_t in_offset,
+                               DATA_BLOB in_data,
+                               bool write_through)
 {
-       return False;
+       return NT_STATUS_RETRY;
 }
 
 void cancel_aio_by_fsp(files_struct *fsp)
@@ -789,6 +1064,9 @@ void cancel_aio_by_fsp(files_struct *fsp)
 
 int wait_for_aio_completion(files_struct *fsp)
 {
-       return ENOSYS;
+       return 0;
 }
+
+void smbd_aio_complete_mid(uint64_t mid);
+
 #endif