From: Mikhail Pershin <mpershin@whamcloud.com>
Date: Thu, 6 Jun 2024 11:12:00 +0000 (+0300)
Subject: LU-15644 llog: don't report warning in no error case
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=8a60d7adc2f453c4be85a044a9071a1ab741d8fd;p=fs%2Flustre-release.git

LU-15644 llog: don't report warning in no error case

Fix wrong check which includes rc == 0 valid case wronly

Fixes: 53d946a1222 (LU-15644 llog: don't replace llog error with -ENOTDIR)
Test-Parameters: trivial
Signed-off-by: Mikhail Pershin <mpershin@whamcloud.com>
Change-Id: Id6e7b2cd42b4769765c67d418552a13f048ea050
Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/55337
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
---

diff --git a/lustre/obdclass/llog_cat.c b/lustre/obdclass/llog_cat.c
index 89d49b4..148b919 100644
--- a/lustre/obdclass/llog_cat.c
+++ b/lustre/obdclass/llog_cat.c
@@ -733,7 +733,7 @@ int llog_cat_cancel_arr_rec(const struct lu_env *env,
 
 	}
 	llog_handle_put(env, loghandle);
-	if (rc != -ENOENT && rc != -ESTALE && rc != -EIO)
+	if (rc && rc != -ENOENT && rc != -ESTALE && rc != -EIO)
 		CWARN("%s: fail to cancel %d records in "DFID": rc = %d\n",
 		      loghandle2name(cathandle), count, PLOGID(lgl), rc);
 	RETURN(rc);