Whamcloud - gitweb
LU-15934 lod: clear up the message
authorYang Sheng <ys@whamcloud.com>
Thu, 29 Dec 2022 17:46:56 +0000 (01:46 +0800)
committerAndreas Dilger <adilger@whamcloud.com>
Thu, 6 Jul 2023 04:14:18 +0000 (04:14 +0000)
Print out the precise info while llog context error.

Lustre-change: https://review.whamcloud.com/49528
Lustre-commit: 9882d4e933fd8cdbc4a9bc8bf6b29655009f7e03

Signed-off-by: Yang Sheng <ys@whamcloud.com>
Change-Id: I492201cd3ae5eb39ad34f3a873d7bb346b52430f
Reviewed-by: Lai Siyao <lai.siyao@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/51555
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
lustre/lod/lod_dev.c

index 7aa24d2..7ecd2da 100644 (file)
@@ -2438,25 +2438,18 @@ static int lod_obd_get_info(const struct lu_env *env, struct obd_export *exp,
                lod_getref(&d->lod_mdt_descs);
                lod_foreach_mdt(d, tgt) {
                        struct llog_ctxt *ctxt;
+                       struct obd_device *ld = tgt->ltd_tgt->dd_lu_dev.ld_obd;
 
                        if (!tgt->ltd_active)
                                continue;
 
-                       ctxt = llog_get_context(tgt->ltd_tgt->dd_lu_dev.ld_obd,
-                                               LLOG_UPDATELOG_ORIG_CTXT);
-                       if (!ctxt) {
-                               CDEBUG(D_INFO, "%s: %s is not ready.\n",
-                                      obd->obd_name,
-                                     tgt->ltd_tgt->dd_lu_dev.ld_obd->obd_name);
-                               rc = -EAGAIN;
-                               break;
-                       }
+                       ctxt = llog_get_context(ld, LLOG_UPDATELOG_ORIG_CTXT);
+                       LASSERT(ctxt != NULL);
                        if (!ctxt->loc_handle) {
-                               CDEBUG(D_INFO, "%s: %s is not ready.\n",
-                                      obd->obd_name,
-                                     tgt->ltd_tgt->dd_lu_dev.ld_obd->obd_name);
-                               rc = -EAGAIN;
+                               CDEBUG(D_INFO, "%s: %s is not ready(%p).\n",
+                                      obd->obd_name, ld->obd_name, ctxt);
                                llog_ctxt_put(ctxt);
+                               rc = -EAGAIN;
                                break;
                        }
                        llog_ctxt_put(ctxt);