The branch, master has been updated
       via  b12409a... wait_for_aio_completion() should return 0 on non-aio 
compiled case.
       via  be5133f... Rename req -> smbreq.
      from  e0acee4... Change smbd_aio_complete_mid() -> 
smbd_aio_complete_aio_ex(). Simplifies the code and eliminates find_aio_ex().

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit b12409a480410116b88f8c6a4522a79056ccbaff
Author: Jeremy Allison <j...@samba.org>
Date:   Fri Jun 4 11:41:57 2010 -0700

    wait_for_aio_completion() should return 0 on non-aio compiled case.

commit be5133f1e07599862ae732056161be6c124a0664
Author: Jeremy Allison <j...@samba.org>
Date:   Fri Jun 4 11:41:38 2010 -0700

    Rename req -> smbreq.

-----------------------------------------------------------------------

Summary of changes:
 source3/smbd/aio.c |   54 ++++++++++++++++++++++++++--------------------------
 1 files changed, 27 insertions(+), 27 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/aio.c b/source3/smbd/aio.c
index c7a70b3..ab88b12 100644
--- a/source3/smbd/aio.c
+++ b/source3/smbd/aio.c
@@ -43,7 +43,7 @@ struct aio_extra {
        struct aio_extra *next, *prev;
        SMB_STRUCT_AIOCB acb;
        files_struct *fsp;
-       struct smb_request *req;
+       struct smb_request *smbreq;
        DATA_BLOB outbuf;
        struct lock_struct lock;
        int (*handle_completion)(struct aio_extra *ex, int errcode);
@@ -127,7 +127,7 @@ static struct aio_extra *create_aio_extra(files_struct 
*fsp, size_t buflen)
 *****************************************************************************/
 
 NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
-                            struct smb_request *req,
+                            struct smb_request *smbreq,
                             files_struct *fsp, SMB_OFF_T startpos,
                             size_t smb_maxcnt)
 {
@@ -158,7 +158,7 @@ NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
 
        /* Only do this on non-chained and non-chaining reads not using the
         * write cache. */
-        if (req_is_in_chain(req) || (lp_write_cache_size(SNUM(conn)) != 0)) {
+        if (req_is_in_chain(smbreq) || (lp_write_cache_size(SNUM(conn)) != 0)) 
{
                return NT_STATUS_RETRY;
        }
 
@@ -180,11 +180,11 @@ NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
        }
        aio_ex->handle_completion = handle_aio_read_complete;
 
-       construct_reply_common_req(req, (char *)aio_ex->outbuf.data);
+       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)req->smbpid,
+       init_strict_lock_struct(fsp, (uint64_t)smbreq->smbpid,
                (uint64_t)startpos, (uint64_t)smb_maxcnt, READ_LOCK,
                &aio_ex->lock);
 
@@ -216,12 +216,12 @@ NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
        }
 
        outstanding_aio_calls++;
-       aio_ex->req = talloc_move(aio_ex, &req);
+       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_str_dbg(fsp), (double)startpos, (unsigned int)smb_maxcnt,
-                 (unsigned int)aio_ex->req->mid ));
+                 (unsigned int)aio_ex->smbreq->mid ));
 
        return NT_STATUS_OK;
 }
@@ -231,7 +231,7 @@ NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
 *****************************************************************************/
 
 NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
-                             struct smb_request *req,
+                             struct smb_request *smbreq,
                              files_struct *fsp, char *data,
                              SMB_OFF_T startpos,
                              size_t numtowrite)
@@ -239,7 +239,7 @@ NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
        struct aio_extra *aio_ex;
        SMB_STRUCT_AIOCB *a;
        size_t bufsize;
-       bool write_through = BITSETW(req->vwv+7,0);
+       bool write_through = BITSETW(smbreq->vwv+7,0);
        size_t min_aio_write_size = lp_aio_write_size(SNUM(conn));
        int ret;
 
@@ -264,7 +264,7 @@ NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
 
        /* Only do this on non-chained and non-chaining reads not using the
         * write cache. */
-        if (req_is_in_chain(req) || (lp_write_cache_size(SNUM(conn)) != 0)) {
+        if (req_is_in_chain(smbreq) || (lp_write_cache_size(SNUM(conn)) != 0)) 
{
                return NT_STATUS_RETRY;
        }
 
@@ -277,7 +277,7 @@ NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
                          "(mid = %u)\n",
                          fsp_str_dbg(fsp), (double)startpos,
                          (unsigned int)numtowrite,
-                         (unsigned int)req->mid ));
+                         (unsigned int)smbreq->mid ));
                return NT_STATUS_RETRY;
        }
 
@@ -289,11 +289,11 @@ NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
        }
        aio_ex->handle_completion = handle_aio_write_complete;
 
-       construct_reply_common_req(req, (char *)aio_ex->outbuf.data);
+       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. */
 
-       init_strict_lock_struct(fsp, (uint64_t)req->smbpid,
+       init_strict_lock_struct(fsp, (uint64_t)smbreq->smbpid,
                (uint64_t)startpos, (uint64_t)numtowrite, WRITE_LOCK,
                &aio_ex->lock);
 
@@ -325,7 +325,7 @@ NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
        }
 
        outstanding_aio_calls++;
-       aio_ex->req = talloc_move(aio_ex, &req);
+       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);
@@ -339,9 +339,9 @@ NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
                 SSVAL(aio_ex->outbuf.data,smb_vwv4,(numtowrite>>16)&1);
                show_msg((char *)aio_ex->outbuf.data);
                if (!srv_send_smb(smbd_server_fd(),(char *)aio_ex->outbuf.data,
-                               true, aio_ex->req->seqnum+1,
+                               true, aio_ex->smbreq->seqnum+1,
                                IS_CONN_ENCRYPTED(fsp->conn),
-                               &aio_ex->req->pcd)) {
+                               &aio_ex->smbreq->pcd)) {
                        exit_server_cleanly("handle_aio_write: srv_send_smb "
                                            "failed.");
                }
@@ -353,7 +353,7 @@ NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
                  "%s, offset %.0f, len = %u (mid = %u) "
                  "outstanding_aio_calls = %d\n",
                  fsp_str_dbg(fsp), (double)startpos, (unsigned int)numtowrite,
-                 (unsigned int)aio_ex->req->mid, outstanding_aio_calls ));
+                 (unsigned int)aio_ex->smbreq->mid, outstanding_aio_calls ));
 
        return NT_STATUS_OK;
 }
@@ -402,7 +402,7 @@ static int handle_aio_read_complete(struct aio_extra 
*aio_ex, int errcode)
        smb_setlen(outbuf,outsize - 4);
        show_msg(outbuf);
        if (!srv_send_smb(smbd_server_fd(),outbuf,
-                       true, aio_ex->req->seqnum+1,
+                       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.");
@@ -465,7 +465,7 @@ static int handle_aio_write_complete(struct aio_extra 
*aio_ex, int errcode)
                ERROR_NT(map_nt_error_from_unix(errcode));
                srv_set_message(outbuf,0,0,true);
         } else {
-               bool write_through = BITSETW(aio_ex->req->vwv+7,0);
+               bool write_through = BITSETW(aio_ex->smbreq->vwv+7,0);
                NTSTATUS status;
 
                SSVAL(outbuf,smb_vwv2,nwritten);
@@ -492,7 +492,7 @@ static int handle_aio_write_complete(struct aio_extra 
*aio_ex, int errcode)
 
        show_msg(outbuf);
        if (!srv_send_smb(smbd_server_fd(),outbuf,
-                         true, aio_ex->req->seqnum+1,
+                         true, aio_ex->smbreq->seqnum+1,
                          IS_CONN_ENCRYPTED(fsp->conn),
                          NULL)) {
                exit_server_cleanly("handle_aio_write: srv_send_smb failed.");
@@ -528,7 +528,7 @@ static bool handle_aio_completed(struct aio_extra *aio_ex, 
int *perr)
        if (err == EINPROGRESS) {
                DEBUG(10,( "handle_aio_completed: operation mid %llu still in "
                        "process for file %s\n",
-                       (unsigned long long)aio_ex->req->mid,
+                       (unsigned long long)aio_ex->smbreq->mid,
                        fsp_str_dbg(aio_ex->fsp)));
                return False;
        }
@@ -541,7 +541,7 @@ static bool handle_aio_completed(struct aio_extra *aio_ex, 
int *perr)
                 * client. */
                DEBUG(10,( "handle_aio_completed: operation mid %llu"
                        " canceled\n",
-                       (unsigned long long)aio_ex->req->mid));
+                       (unsigned long long)aio_ex->smbreq->mid));
                return True;
         }
 
@@ -565,7 +565,7 @@ void smbd_aio_complete_aio_ex(struct aio_extra *aio_ex)
        outstanding_aio_calls--;
 
        DEBUG(10,("smbd_aio_complete_mid: mid[%llu]\n",
-               (unsigned long long)aio_ex->req->mid));
+               (unsigned long long)aio_ex->smbreq->mid));
 
        fsp = aio_ex->fsp;
        if (fsp == NULL) {
@@ -573,7 +573,7 @@ void smbd_aio_complete_aio_ex(struct aio_extra *aio_ex)
                 * ignore. */
                DEBUG( 3,( "smbd_aio_complete_mid: file closed whilst "
                        "aio outstanding (mid[%llu]).\n",
-                       (unsigned long long)aio_ex->req->mid));
+                       (unsigned long long)aio_ex->smbreq->mid));
                return;
        }
 
@@ -709,7 +709,7 @@ void cancel_aio_by_fsp(files_struct *fsp)
 
 #else
 NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
-                            struct smb_request *req,
+                            struct smb_request *smbreq,
                             files_struct *fsp, SMB_OFF_T startpos,
                             size_t smb_maxcnt)
 {
@@ -717,7 +717,7 @@ NTSTATUS schedule_aio_read_and_X(connection_struct *conn,
 }
 
 NTSTATUS schedule_aio_write_and_X(connection_struct *conn,
-                             struct smb_request *req,
+                             struct smb_request *smbreq,
                              files_struct *fsp, char *data,
                              SMB_OFF_T startpos,
                              size_t numtowrite)
@@ -731,7 +731,7 @@ 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);


-- 
Samba Shared Repository

Reply via email to