Whamcloud - gitweb
LU-980 llog: cleanup return value in llog_client_create
[fs/lustre-release.git] / lustre / ptlrpc / llog_client.c
index 395d756..0a5bb91 100644 (file)
@@ -1,29 +1,43 @@
 /* -*- mode: c; c-basic-offset: 8; indent-tabs-mode: nil; -*-
  * vim:expandtab:shiftwidth=8:tabstop=8:
  *
- *  Copyright (C) 2001-2004 Cluster File Systems, Inc.
- *   Author: Andreas Dilger <adilger@clusterfs.com>
+ * GPL HEADER START
  *
- *   This file is part of the Lustre file system, http://www.lustre.org
- *   Lustre is a trademark of Cluster File Systems, Inc.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
- *   You may have signed or agreed to another license before downloading
- *   this software.  If so, you are bound by the terms and conditions
- *   of that agreement, and the following does not apply to you.  See the
- *   LICENSE file included with this distribution for more information.
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 only,
+ * as published by the Free Software Foundation.
  *
- *   If you did not agree to a different license, then this copy of Lustre
- *   is open source software; you can redistribute it and/or modify it
- *   under the terms of version 2 of the GNU General Public License as
- *   published by the Free Software Foundation.
+ * This program is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * General Public License version 2 for more details (a copy is included
+ * in the LICENSE file that accompanied this code).
  *
- *   In either case, Lustre is distributed in the hope that it will be
- *   useful, but WITHOUT ANY WARRANTY; without even the implied warranty
- *   of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *   license text for more details.
+ * You should have received a copy of the GNU General Public License
+ * version 2 along with this program; If not, see
+ * http://www.sun.com/software/products/lustre/docs/GPLv2.pdf
  *
- *  remote api for llog - client side
+ * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
+ * CA 95054 USA or visit www.sun.com if you need additional information or
+ * have any questions.
  *
+ * GPL HEADER END
+ */
+/*
+ * Copyright (c) 2003, 2010, Oracle and/or its affiliates. All rights reserved.
+ * Use is subject to license terms.
+ */
+/*
+ * This file is part of Lustre, http://www.lustre.org/
+ * Lustre is a trademark of Sun Microsystems, Inc.
+ *
+ * lustre/ptlrpc/llog_client.c
+ *
+ * remote api for llog - client side
+ *
+ * Author: Andreas Dilger <adilger@clusterfs.com>
  */
 
 #define DEBUG_SUBSYSTEM S_LOG
 #include <lustre_net.h>
 #include <libcfs/list.h>
 
+#define LLOG_CLIENT_ENTRY(ctxt, imp) do {                             \
+        cfs_mutex_down(&ctxt->loc_sem);                               \
+        if (ctxt->loc_imp) {                                          \
+                imp = class_import_get(ctxt->loc_imp);                \
+        } else {                                                      \
+                CERROR("ctxt->loc_imp == NULL for context idx %d."    \
+                       "Unable to complete MDS/OSS recovery,"         \
+                       "but I'll try again next time.  Not fatal.\n", \
+                       ctxt->loc_idx);                                \
+                imp = NULL;                                           \
+                cfs_mutex_up(&ctxt->loc_sem);                         \
+                return (-EINVAL);                                     \
+        }                                                             \
+        cfs_mutex_up(&ctxt->loc_sem);                                 \
+} while(0)
+
+#define LLOG_CLIENT_EXIT(ctxt, imp) do {                              \
+        cfs_mutex_down(&ctxt->loc_sem);                               \
+        if (ctxt->loc_imp != imp)                                     \
+                CWARN("loc_imp has changed from %p to %p\n",          \
+                       ctxt->loc_imp, imp);                           \
+        class_import_put(imp);                                        \
+        cfs_mutex_up(&ctxt->loc_sem);                                 \
+} while(0)
+
 /* This is a callback from the llog_* functions.
  * Assumes caller has already pushed us into the kernel context. */
 static int llog_client_create(struct llog_ctxt *ctxt, struct llog_handle **res,
@@ -55,14 +94,7 @@ static int llog_client_create(struct llog_ctxt *ctxt, struct llog_handle **res,
         int                    rc;
         ENTRY;
 
-        if (ctxt->loc_imp == NULL) {
-                /* This used to be an assert; bug 6200 */
-                CERROR("ctxt->loc_imp == NULL for context idx %d.  Unable to "
-                       "complete MDS/OSS recovery, but I'll try again next "
-                       "time.  Not fatal.\n", ctxt->loc_idx);
-                RETURN(-EINVAL);
-        }
-        imp = ctxt->loc_imp;
+        LLOG_CLIENT_ENTRY(ctxt, imp);
 
         handle = llog_alloc_handle();
         if (handle == NULL)
@@ -81,6 +113,7 @@ static int llog_client_create(struct llog_ctxt *ctxt, struct llog_handle **res,
                                  LLOG_ORIGIN_HANDLE_CREATE);
         if (rc) {
                 ptlrpc_request_free(req);
+                req = NULL;
                 GOTO(err_free, rc);
         }
         ptlrpc_request_set_replen(req);
@@ -110,26 +143,29 @@ static int llog_client_create(struct llog_ctxt *ctxt, struct llog_handle **res,
         handle->lgh_ctxt = ctxt;
         EXIT;
 out:
+        LLOG_CLIENT_EXIT(ctxt, imp);
         ptlrpc_req_finished(req);
         return rc;
 err_free:
+        *res = NULL;
         llog_free_handle(handle);
         goto out;
 }
 
 static int llog_client_destroy(struct llog_handle *loghandle)
 {
+        struct obd_import     *imp;
         struct ptlrpc_request *req = NULL;
         struct llogd_body     *body;
         int                    rc;
         ENTRY;
 
-        req = ptlrpc_request_alloc_pack(loghandle->lgh_ctxt->loc_imp,
-                                        &RQF_LLOG_ORIGIN_HANDLE_DESTROY,
+        LLOG_CLIENT_ENTRY(loghandle->lgh_ctxt, imp);
+        req = ptlrpc_request_alloc_pack(imp, &RQF_LLOG_ORIGIN_HANDLE_DESTROY,
                                         LUSTRE_LOG_VERSION,
                                         LLOG_ORIGIN_HANDLE_DESTROY);
         if (req == NULL)
-                RETURN(-ENOMEM);
+                GOTO(err_exit, rc =-ENOMEM);
 
         body = req_capsule_client_get(&req->rq_pill, &RMF_LLOGD_BODY);
         body->lgd_logid = loghandle->lgh_id;
@@ -139,6 +175,8 @@ static int llog_client_destroy(struct llog_handle *loghandle)
         rc = ptlrpc_queue_wait(req);
         
         ptlrpc_req_finished(req);
+err_exit:
+        LLOG_CLIENT_EXIT(loghandle->lgh_ctxt, imp);
         RETURN(rc);
 }
 
@@ -147,19 +185,20 @@ static int llog_client_next_block(struct llog_handle *loghandle,
                                   int *cur_idx, int next_idx,
                                   __u64 *cur_offset, void *buf, int len)
 {
+        struct obd_import     *imp;
         struct ptlrpc_request *req = NULL;
         struct llogd_body     *body;
         void                  *ptr;
         int                    rc;
         ENTRY;
 
-        req = ptlrpc_request_alloc_pack(loghandle->lgh_ctxt->loc_imp,
-                                        &RQF_LLOG_ORIGIN_HANDLE_NEXT_BLOCK,
+        LLOG_CLIENT_ENTRY(loghandle->lgh_ctxt, imp);
+        req = ptlrpc_request_alloc_pack(imp, &RQF_LLOG_ORIGIN_HANDLE_NEXT_BLOCK,
                                         LUSTRE_LOG_VERSION,
                                         LLOG_ORIGIN_HANDLE_NEXT_BLOCK);
         if (req == NULL)
-                RETURN(-ENOMEM);
-
+                GOTO(err_exit, rc =-ENOMEM);
+                
         body = req_capsule_client_get(&req->rq_pill, &RMF_LLOGD_BODY);
         body->lgd_logid = loghandle->lgh_id;
         body->lgd_ctxt_idx = loghandle->lgh_ctxt->loc_idx - 1;
@@ -191,24 +230,27 @@ static int llog_client_next_block(struct llog_handle *loghandle,
         EXIT;
 out:
         ptlrpc_req_finished(req);
+err_exit:
+        LLOG_CLIENT_EXIT(loghandle->lgh_ctxt, imp);
         return rc;
 }
 
 static int llog_client_prev_block(struct llog_handle *loghandle,
                                   int prev_idx, void *buf, int len)
 {
+        struct obd_import     *imp;
         struct ptlrpc_request *req = NULL;
         struct llogd_body     *body;
         void                  *ptr;
         int                    rc;
         ENTRY;
 
-        req = ptlrpc_request_alloc_pack(loghandle->lgh_ctxt->loc_imp,
-                                        &RQF_LLOG_ORIGIN_HANDLE_PREV_BLOCK,
+        LLOG_CLIENT_ENTRY(loghandle->lgh_ctxt, imp);
+        req = ptlrpc_request_alloc_pack(imp, &RQF_LLOG_ORIGIN_HANDLE_PREV_BLOCK,
                                         LUSTRE_LOG_VERSION,
                                         LLOG_ORIGIN_HANDLE_PREV_BLOCK);
         if (req == NULL)
-                RETURN(-ENOMEM);
+                GOTO(err_exit, rc = -ENOMEM);
 
         body = req_capsule_client_get(&req->rq_pill, &RMF_LLOGD_BODY);
         body->lgd_logid = loghandle->lgh_id;
@@ -236,11 +278,14 @@ static int llog_client_prev_block(struct llog_handle *loghandle,
         EXIT;
 out:
         ptlrpc_req_finished(req);
+err_exit:
+        LLOG_CLIENT_EXIT(loghandle->lgh_ctxt, imp);
         return rc;
 }
 
 static int llog_client_read_header(struct llog_handle *handle)
 {
+        struct obd_import     *imp;
         struct ptlrpc_request *req = NULL;
         struct llogd_body     *body;
         struct llog_log_hdr   *hdr;
@@ -248,12 +293,12 @@ static int llog_client_read_header(struct llog_handle *handle)
         int                    rc;
         ENTRY;
 
-        req = ptlrpc_request_alloc_pack(handle->lgh_ctxt->loc_imp,
-                                        &RQF_LLOG_ORIGIN_HANDLE_READ_HEADER,
+        LLOG_CLIENT_ENTRY(handle->lgh_ctxt, imp);
+        req = ptlrpc_request_alloc_pack(imp,&RQF_LLOG_ORIGIN_HANDLE_READ_HEADER,
                                         LUSTRE_LOG_VERSION,
                                         LLOG_ORIGIN_HANDLE_READ_HEADER);
         if (req == NULL)
-                RETURN(-ENOMEM);
+                GOTO(err_exit, rc = -ENOMEM);
 
         body = req_capsule_client_get(&req->rq_pill, &RMF_LLOGD_BODY);
         body->lgd_logid = handle->lgh_id;
@@ -288,6 +333,8 @@ static int llog_client_read_header(struct llog_handle *handle)
         EXIT;
 out:
         ptlrpc_req_finished(req);
+err_exit:
+        LLOG_CLIENT_EXIT(handle->lgh_ctxt, imp);
         return rc;
 }