From f761c86c47d56ff42770336488559e09385c5268 Mon Sep 17 00:00:00 2001 From: hxing Date: Fri, 28 Jul 2023 15:36:12 +0800 Subject: [PATCH] LU-16911 sec: quiet messages from identity upcall retry mech Do not use CERROR to print messages about failed identity acquire upcalls. And make a difference between initial attempt before retry, and final failure. Lustre-change: https://review.whamcloud.com/51355 Lustre-commit: 765e59d37dc2b4c85e9bdaea2765c9023226778b Fixes: 61c3b3a9bb ("LU-16165 sec: retry mechanism for identity cache") Signed-off-by: Sebastien Buisson Signed-off-by: Xing Huang Change-Id: I35e04ca31b623d6037bb49e4ded4ea96d653f074 Reviewed-by: Andreas Dilger Reviewed-by: James Simmons Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/51797 Tested-by: jenkins Tested-by: Maloo --- lustre/obdclass/upcall_cache.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/lustre/obdclass/upcall_cache.c b/lustre/obdclass/upcall_cache.c index 911a5ea..1cdc85b 100644 --- a/lustre/obdclass/upcall_cache.c +++ b/lustre/obdclass/upcall_cache.c @@ -346,15 +346,18 @@ find_again: if (UC_CACHE_IS_ACQUIRING(entry)) { /* we're interrupted or upcall failed in the middle */ rc = left > 0 ? -EINTR : -ETIMEDOUT; - CERROR("acquire for key %llu: error %d\n", - entry->ue_key, rc); put_entry(cache, entry); if (!failedacquiring) { spin_unlock(&cache->uc_lock); failedacquiring = true; new = NULL; + CDEBUG(D_OTHER, + "retry acquire for key %llu (got %d)\n", + entry->ue_key, rc); goto find_again; } + CERROR("acquire for key %llu: error %d\n", + entry->ue_key, rc); GOTO(out, entry = ERR_PTR(rc)); } } -- 1.8.3.1