From e5dadcd9cf9f1cae606b6383db7991f6c9803f60 Mon Sep 17 00:00:00 2001 From: Andreas Dilger Date: Fri, 22 Mar 2024 16:20:38 -0700 Subject: [PATCH] LU-930 ptlrpc: quiet idle import logging Don't log a debug message for every idle import every 25s, as this pushes out other more important messages from the logs. Fixes: 5a6ceb664f ("LU-7236 ptlrpc: idle connections can disconnect") Test-Parameters: trivial Signed-off-by: Andreas Dilger Change-Id: Id98c2acad07cec62af0d705a437a4d2915ce9f62 Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/54540 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Alex Zhuravlev Reviewed-by: Timothy Day Reviewed-by: Oleg Drokin --- lustre/ptlrpc/pinger.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/lustre/ptlrpc/pinger.c b/lustre/ptlrpc/pinger.c index b122c42..0abc772 100644 --- a/lustre/ptlrpc/pinger.c +++ b/lustre/ptlrpc/pinger.c @@ -221,7 +221,7 @@ EXPORT_SYMBOL(ptlrpc_pinger_ir_down); static void ptlrpc_pinger_process_import(struct obd_import *imp, time64_t this_ping) { - int level; + enum lustre_imp_state level; int force; int force_next; int suppress; @@ -245,7 +245,8 @@ static void ptlrpc_pinger_process_import(struct obd_import *imp, imp->imp_force_next_verify = 0; - CDEBUG(level == LUSTRE_IMP_FULL ? D_INFO : D_HA, + CDEBUG((level == LUSTRE_IMP_FULL || level == LUSTRE_IMP_IDLE) ? + D_INFO : D_HA, "%s->%s: level %s/%u force %u force_next %u deactive %u pingable %u suppress %u\n", imp->imp_obd->obd_uuid.uuid, obd2cli_tgt(imp->imp_obd), ptlrpc_import_state_name(level), level, force, force_next, @@ -264,7 +265,7 @@ static void ptlrpc_pinger_process_import(struct obd_import *imp, } } else if (level != LUSTRE_IMP_FULL || imp->imp_obd->obd_no_recov || imp_is_deactive(imp)) { - CDEBUG(D_HA, + CDEBUG(level == LUSTRE_IMP_IDLE ? D_INFO : D_HA, "%s->%s: not pinging (in recovery or recovery disabled: %s)\n", imp->imp_obd->obd_uuid.uuid, obd2cli_tgt(imp->imp_obd), ptlrpc_import_state_name(level)); -- 1.8.3.1