From: Lai Siyao <lai.siyao@whamcloud.com>
Date: Fri, 3 May 2024 00:27:04 +0000 (-0400)
Subject: LU-15988 osp: don't print nid on -ESTALE
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=6bb473bfad9cd3b4b9ee6f2f57a47e101d929d80;p=fs%2Flustre-release.git

LU-15988 osp: don't print nid on -ESTALE

Osp_send_update_req() should not access import upon -ESTALE, because
this MDT may be in umount.

Lustre-change: https://review.whamcloud.com/55049
Lustre-commit: ae26dbc3387a17b763cbc901fa256d894a1f88fb

Signed-off-by: Lai Siyao <lai.siyao@whamcloud.com>
Change-Id: Ibd869e4e8da4f90ffd608a36d866264d5d552d0e
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Jian Yu <yujian@whamcloud.com>
Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/55288
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
---

diff --git a/lustre/osp/osp_trans.c b/lustre/osp/osp_trans.c
index 6eecfe3..cdfb34e 100644
--- a/lustre/osp/osp_trans.c
+++ b/lustre/osp/osp_trans.c
@@ -1124,13 +1124,11 @@ static int osp_send_update_req(const struct lu_env *env,
 	LASSERT(oth != NULL);
 
 	if (ou && ou->ou_generation != our->our_generation) {
-		const struct lnet_process_id *peer =
-			&osp->opd_obd->u.cli.cl_import->imp_connection->c_peer;
 		rc = -ESTALE;
 		osp_trans_callback(env, oth, rc);
-		CDEBUG(D_HA, "%s: stale tx to %s: gen %llu != %llu: rc = %d\n",
-		       osp->opd_obd->obd_name, libcfs_nid2str(peer->nid),
-		       osp->opd_update->ou_generation, our->our_generation, rc);
+		CDEBUG(D_HA, "%s: stale tx: gen %llu != %llu: rc = %d\n",
+		       osp->opd_obd->obd_name, ou->ou_generation,
+		       our->our_generation, rc);
 		RETURN(rc);
 	}