From 11eda8742fa4505bc62c6113b067a75687f171cc Mon Sep 17 00:00:00 2001 From: Andrew Perepechko Date: Wed, 15 Feb 2017 12:02:20 +0300 Subject: [PATCH] LU-9127 target: tgt_cb_last_committed is too noisy tgt_cb_last_committed() prints a D_HA message even if last_committed was not updated. We can only print a message when last_committed was updated, so we have mostly the same information and save some debug log space and cpu resource. Change-Id: Ic2784e6a3652ca1851cde1313d6985ed2f90e36b Signed-off-by: Andrew Perepechko Reviewed-on: https://review.whamcloud.com/25469 Reviewed-by: Andreas Dilger Reviewed-by: Alexander Boyko Reviewed-by: Alexander Zarochentsev Reviewed-by: Alex Zhuravlev Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Mike Pershin --- lustre/target/tgt_lastrcvd.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lustre/target/tgt_lastrcvd.c b/lustre/target/tgt_lastrcvd.c index 6ee01de..ea70037 100644 --- a/lustre/target/tgt_lastrcvd.c +++ b/lustre/target/tgt_lastrcvd.c @@ -804,11 +804,12 @@ static void tgt_cb_last_committed(struct lu_env *env, struct thandle *th, } else { spin_unlock(&ccb->llcc_tgt->lut_translock); } + + CDEBUG(D_HA, "%s: transno %lld is committed\n", + ccb->llcc_tgt->lut_obd->obd_name, ccb->llcc_transno); + out: class_export_cb_put(ccb->llcc_exp); - if (ccb->llcc_transno) - CDEBUG(D_HA, "%s: transno %lld is committed\n", - ccb->llcc_tgt->lut_obd->obd_name, ccb->llcc_transno); OBD_FREE_PTR(ccb); } -- 1.8.3.1