Whamcloud - gitweb
LU-930 ptlrpc: quiet idle import logging 40/54540/2
authorAndreas Dilger <adilger@whamcloud.com>
Fri, 22 Mar 2024 23:20:38 +0000 (16:20 -0700)
committerOleg Drokin <green@whamcloud.com>
Tue, 30 Apr 2024 06:55:36 +0000 (06:55 +0000)
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 <adilger@whamcloud.com>
Change-Id: Id98c2acad07cec62af0d705a437a4d2915ce9f62
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/54540
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com>
Reviewed-by: Timothy Day <timday@amazon.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/ptlrpc/pinger.c

index b122c42..0abc772 100644 (file)
@@ -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));