Whamcloud - gitweb
LU-9127 target: tgt_cb_last_committed is too noisy 69/25469/4
authorAndrew Perepechko <andrew.perepechko@seagate.com>
Wed, 15 Feb 2017 09:02:20 +0000 (12:02 +0300)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 1 Mar 2017 05:12:24 +0000 (05:12 +0000)
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 <andrew.perepechko@seagate.com>
Reviewed-on: https://review.whamcloud.com/25469
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Alexander Boyko <alexander.boyko@seagate.com>
Reviewed-by: Alexander Zarochentsev <alexander.zarochentsev@seagate.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>

index 6ee01de..ea70037 100644 (file)
@@ -804,11 +804,12 @@ static void tgt_cb_last_committed(struct lu_env *env, struct thandle *th,
        } else {
+       CDEBUG(D_HA, "%s: transno %lld is committed\n",
+              ccb->llcc_tgt->lut_obd->obd_name, ccb->llcc_transno);
-       if (ccb->llcc_transno)
-               CDEBUG(D_HA, "%s: transno %lld is committed\n",
-                      ccb->llcc_tgt->lut_obd->obd_name, ccb->llcc_transno);