Whamcloud - gitweb
LU-15644 llog: don't report warning in no error case 35/55335/2
authorMikhail Pershin <mpershin@whamcloud.com>
Thu, 6 Jun 2024 11:12:00 +0000 (14:12 +0300)
committerOleg Drokin <green@whamcloud.com>
Wed, 19 Jun 2024 01:15:15 +0000 (01:15 +0000)
Fix wrong check which includes rc == 0 valid case wronly

Fixes: bd9839f7db (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/fs/lustre-release/+/55335
Reviewed-by: Alex Zhuravlev <bzzz@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
lustre/obdclass/llog_cat.c

index 85ebfba..334de1b 100644 (file)
@@ -732,7 +732,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);