Whamcloud - gitweb
LU-15644 llog: don't report warning in no error case
authorMikhail Pershin <mpershin@whamcloud.com>
Thu, 6 Jun 2024 11:12:00 +0000 (14:12 +0300)
committerAndreas Dilger <adilger@whamcloud.com>
Fri, 7 Jun 2024 07:27:05 +0000 (07:27 +0000)
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>
lustre/obdclass/llog_cat.c

index 89d49b4..148b919 100644 (file)
@@ -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);