Whamcloud - gitweb
LU-6496 ptlrpc: Fix wrong code indentation in plain_authorize
[fs/lustre-release.git] / lustre / ptlrpc / sec_null.c
index f884574..d661ee3 100644 (file)
@@ -27,7 +27,7 @@
  * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2011, Whamcloud, Inc.
+ * Copyright (c) 2011, 2014, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -40,9 +40,6 @@
 
 #define DEBUG_SUBSYSTEM S_SEC
 
-#ifndef __KERNEL__
-#include <liblustre.h>
-#endif
 
 #include <obd_support.h>
 #include <obd_cksum.h>
@@ -50,6 +47,8 @@
 #include <lustre_net.h>
 #include <lustre_sec.h>
 
+#include "ptlrpc_internal.h"
+
 static struct ptlrpc_sec_policy null_policy;
 static struct ptlrpc_sec        null_sec;
 static struct ptlrpc_cli_ctx    null_cli_ctx;
@@ -104,15 +103,14 @@ int null_ctx_verify(struct ptlrpc_cli_ctx *ctx, struct ptlrpc_request *req)
 
         if (req->rq_early) {
                 cksums = lustre_msg_get_cksum(req->rq_repdata);
-#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 7, 50, 0)
-                if (lustre_msghdr_get_flags(req->rq_reqmsg) &
-                    MSGHDR_CKSUM_INCOMPAT18)
-                        cksumc = lustre_msg_calc_cksum(req->rq_repmsg, 0);
-                else
-                        cksumc = lustre_msg_calc_cksum(req->rq_repmsg, 1);
+#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 7, 53, 0)
+               if (lustre_msghdr_get_flags(req->rq_reqmsg) &
+                   MSGHDR_CKSUM_INCOMPAT18)
+                       cksumc = lustre_msg_calc_cksum(req->rq_repmsg, 0);
+               else
+                       cksumc = lustre_msg_calc_cksum(req->rq_repmsg, 1);
 #else
-# warning "remove checksum compatibility support for b1_8"
-                cksumc = lustre_msg_calc_cksum(req->rq_repmsg);
+               cksumc = lustre_msg_calc_cksum(req->rq_repmsg);
 #endif
                 if (cksumc != cksums) {
                         CDEBUG(D_SEC,
@@ -150,8 +148,8 @@ struct ptlrpc_cli_ctx *null_lookup_ctx(struct ptlrpc_sec *sec,
                                        struct vfs_cred *vcred,
                                        int create, int remove_dead)
 {
-        cfs_atomic_inc(&null_cli_ctx.cc_refcount);
-        return &null_cli_ctx;
+       atomic_inc(&null_cli_ctx.cc_refcount);
+       return &null_cli_ctx;
 }
 
 static
@@ -263,11 +261,22 @@ int null_enlarge_reqbuf(struct ptlrpc_sec *sec,
                 if (newbuf == NULL)
                         return -ENOMEM;
 
+               /* Must lock this, so that otherwise unprotected change of
+                * rq_reqmsg is not racing with parallel processing of
+                * imp_replay_list traversing threads. See LU-3333
+                * This is a bandaid at best, we really need to deal with this
+                * in request enlarging code before unpacking that's already
+                * there */
+               if (req->rq_import)
+                       spin_lock(&req->rq_import->imp_lock);
                 memcpy(newbuf, req->rq_reqbuf, req->rq_reqlen);
 
                 OBD_FREE_LARGE(req->rq_reqbuf, req->rq_reqbuf_len);
                 req->rq_reqbuf = req->rq_reqmsg = newbuf;
                 req->rq_reqbuf_len = alloc_size;
+
+               if (req->rq_import)
+                       spin_unlock(&req->rq_import->imp_lock);
         }
 
         _sptlrpc_enlarge_msg_inplace(req->rq_reqmsg, segment, newsize);
@@ -277,30 +286,30 @@ int null_enlarge_reqbuf(struct ptlrpc_sec *sec,
 }
 
 static struct ptlrpc_svc_ctx null_svc_ctx = {
-        .sc_refcount    = CFS_ATOMIC_INIT(1),
+       .sc_refcount    = ATOMIC_INIT(1),
         .sc_policy      = &null_policy,
 };
 
 static
 int null_accept(struct ptlrpc_request *req)
 {
-        LASSERT(SPTLRPC_FLVR_POLICY(req->rq_flvr.sf_rpc) ==
-                SPTLRPC_POLICY_NULL);
+       LASSERT(SPTLRPC_FLVR_POLICY(req->rq_flvr.sf_rpc) ==
+               SPTLRPC_POLICY_NULL);
 
-        if (req->rq_flvr.sf_rpc != SPTLRPC_FLVR_NULL) {
-                CERROR("Invalid rpc flavor 0x%x\n", req->rq_flvr.sf_rpc);
-                return SECSVC_DROP;
-        }
+       if (req->rq_flvr.sf_rpc != SPTLRPC_FLVR_NULL) {
+               CERROR("Invalid rpc flavor 0x%x\n", req->rq_flvr.sf_rpc);
+               return SECSVC_DROP;
+       }
 
-        req->rq_sp_from = null_decode_sec_part(req->rq_reqbuf);
+       req->rq_sp_from = null_decode_sec_part(req->rq_reqbuf);
 
-        req->rq_reqmsg = req->rq_reqbuf;
-        req->rq_reqlen = req->rq_reqdata_len;
+       req->rq_reqmsg = req->rq_reqbuf;
+       req->rq_reqlen = req->rq_reqdata_len;
 
-        req->rq_svc_ctx = &null_svc_ctx;
-        cfs_atomic_inc(&req->rq_svc_ctx->sc_refcount);
+       req->rq_svc_ctx = &null_svc_ctx;
+       atomic_inc(&req->rq_svc_ctx->sc_refcount);
 
-        return SECSVC_OK;
+       return SECSVC_OK;
 }
 
 static
@@ -324,25 +333,25 @@ int null_alloc_rs(struct ptlrpc_request *req, int msgsize)
                 rs->rs_size = rs_size;
         }
 
-        rs->rs_svc_ctx = req->rq_svc_ctx;
-        cfs_atomic_inc(&req->rq_svc_ctx->sc_refcount);
+       rs->rs_svc_ctx = req->rq_svc_ctx;
+       atomic_inc(&req->rq_svc_ctx->sc_refcount);
 
-        rs->rs_repbuf = (struct lustre_msg *) (rs + 1);
-        rs->rs_repbuf_len = rs_size - sizeof(*rs);
-        rs->rs_msg = rs->rs_repbuf;
+       rs->rs_repbuf = (struct lustre_msg *) (rs + 1);
+       rs->rs_repbuf_len = rs_size - sizeof(*rs);
+       rs->rs_msg = rs->rs_repbuf;
 
-        req->rq_reply_state = rs;
-        return 0;
+       req->rq_reply_state = rs;
+       return 0;
 }
 
 static
 void null_free_rs(struct ptlrpc_reply_state *rs)
 {
-        LASSERT_ATOMIC_GT(&rs->rs_svc_ctx->sc_refcount, 1);
-        cfs_atomic_dec(&rs->rs_svc_ctx->sc_refcount);
+       LASSERT_ATOMIC_GT(&rs->rs_svc_ctx->sc_refcount, 1);
+       atomic_dec(&rs->rs_svc_ctx->sc_refcount);
 
-        if (!rs->rs_prealloc)
-                OBD_FREE_LARGE(rs, rs->rs_size);
+       if (!rs->rs_prealloc)
+               OBD_FREE_LARGE(rs, rs->rs_size);
 }
 
 static
@@ -363,21 +372,20 @@ int null_authorize(struct ptlrpc_request *req)
         } else {
                 __u32 cksum;
 
-#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 7, 50, 0)
-                if (lustre_msghdr_get_flags(req->rq_reqmsg) &
-                    MSGHDR_CKSUM_INCOMPAT18)
-                        cksum = lustre_msg_calc_cksum(rs->rs_repbuf, 0);
-                else
-                        cksum = lustre_msg_calc_cksum(rs->rs_repbuf, 1);
+#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 7, 53, 0)
+               if (lustre_msghdr_get_flags(req->rq_reqmsg) &
+                   MSGHDR_CKSUM_INCOMPAT18)
+                       cksum = lustre_msg_calc_cksum(rs->rs_repbuf, 0);
+               else
+                       cksum = lustre_msg_calc_cksum(rs->rs_repbuf, 1);
 #else
-# warning "remove checksum compatibility support for b1_8"
-                cksum = lustre_msg_calc_cksum(rs->rs_repbuf);
+               cksum = lustre_msg_calc_cksum(rs->rs_repbuf);
 #endif
-                lustre_msg_set_cksum(rs->rs_repbuf, cksum);
-                req->rq_reply_off = 0;
-        }
+               lustre_msg_set_cksum(rs->rs_repbuf, cksum);
+               req->rq_reply_off = 0;
+       }
 
-        return 0;
+       return 0;
 }
 
 static struct ptlrpc_ctx_ops null_ctx_ops = {
@@ -415,33 +423,33 @@ static struct ptlrpc_sec_policy null_policy = {
 
 static void null_init_internal(void)
 {
-        static CFS_HLIST_HEAD(__list);
-
-        null_sec.ps_policy = &null_policy;
-        cfs_atomic_set(&null_sec.ps_refcount, 1);     /* always busy */
-        null_sec.ps_id = -1;
-        null_sec.ps_import = NULL;
-        null_sec.ps_flvr.sf_rpc = SPTLRPC_FLVR_NULL;
-        null_sec.ps_flvr.sf_flags = 0;
-        null_sec.ps_part = LUSTRE_SP_ANY;
-        null_sec.ps_dying = 0;
-        cfs_spin_lock_init(&null_sec.ps_lock);
-        cfs_atomic_set(&null_sec.ps_nctx, 1);         /* for "null_cli_ctx" */
-        CFS_INIT_LIST_HEAD(&null_sec.ps_gc_list);
-        null_sec.ps_gc_interval = 0;
-        null_sec.ps_gc_next = 0;
-
-        cfs_hlist_add_head(&null_cli_ctx.cc_cache, &__list);
-        cfs_atomic_set(&null_cli_ctx.cc_refcount, 1);    /* for hash */
-        null_cli_ctx.cc_sec = &null_sec;
-        null_cli_ctx.cc_ops = &null_ctx_ops;
-        null_cli_ctx.cc_expire = 0;
-        null_cli_ctx.cc_flags = PTLRPC_CTX_CACHED | PTLRPC_CTX_ETERNAL |
-                                PTLRPC_CTX_UPTODATE;
-        null_cli_ctx.cc_vcred.vc_uid = 0;
-        cfs_spin_lock_init(&null_cli_ctx.cc_lock);
-        CFS_INIT_LIST_HEAD(&null_cli_ctx.cc_req_list);
-        CFS_INIT_LIST_HEAD(&null_cli_ctx.cc_gc_chain);
+       static HLIST_HEAD(__list);
+
+       null_sec.ps_policy = &null_policy;
+       atomic_set(&null_sec.ps_refcount, 1);   /* always busy */
+       null_sec.ps_id = -1;
+       null_sec.ps_import = NULL;
+       null_sec.ps_flvr.sf_rpc = SPTLRPC_FLVR_NULL;
+       null_sec.ps_flvr.sf_flags = 0;
+       null_sec.ps_part = LUSTRE_SP_ANY;
+       null_sec.ps_dying = 0;
+       spin_lock_init(&null_sec.ps_lock);
+       atomic_set(&null_sec.ps_nctx, 1);       /* for "null_cli_ctx" */
+       INIT_LIST_HEAD(&null_sec.ps_gc_list);
+       null_sec.ps_gc_interval = 0;
+       null_sec.ps_gc_next = 0;
+
+       hlist_add_head(&null_cli_ctx.cc_cache, &__list);
+       atomic_set(&null_cli_ctx.cc_refcount, 1);       /* for hash */
+       null_cli_ctx.cc_sec = &null_sec;
+       null_cli_ctx.cc_ops = &null_ctx_ops;
+       null_cli_ctx.cc_expire = 0;
+       null_cli_ctx.cc_flags = PTLRPC_CTX_CACHED | PTLRPC_CTX_ETERNAL |
+                               PTLRPC_CTX_UPTODATE;
+       null_cli_ctx.cc_vcred.vc_uid = 0;
+       spin_lock_init(&null_cli_ctx.cc_lock);
+       INIT_LIST_HEAD(&null_cli_ctx.cc_req_list);
+       INIT_LIST_HEAD(&null_cli_ctx.cc_gc_chain);
 }
 
 int sptlrpc_null_init(void)