From: Olaf Faaland <faaland1@llnl.gov>
Date: Thu, 25 Mar 2021 01:35:10 +0000 (-0700)
Subject: LU-14553 changelog: eliminate mdd_changelog_clear warning
X-Git-Tag: 2.14.52~88
X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=commitdiff_plain;h=6b183927e19715d093c80a35ebc42a1cda5e70e2

LU-14553 changelog: eliminate mdd_changelog_clear warning

When handling a changelog_clear request, the user may specify a
range of indices which do not exist.  Similarly, the user may
specify a changelog user which does not exist.  Neither indicates
a problem within Lustre that justifies a a console warning.

Change those cases to CDEBUG.

Test-Parameters: trivial
Signed-off-by: Olaf Faaland <faaland1@llnl.gov>
Change-Id: I64bab12ef4978c4bf7139f5f36a39f9b109616fb
Reviewed-on: https://review.whamcloud.com/43125
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
---

diff --git a/lustre/mdd/mdd_device.c b/lustre/mdd/mdd_device.c
index a782db3..9323424 100644
--- a/lustre/mdd/mdd_device.c
+++ b/lustre/mdd/mdd_device.c
@@ -1817,7 +1817,11 @@ static int mdd_changelog_clear(const struct lu_env *env,
 			      mdd_changelog_clear_cb, (void *)&mcuc,
 			      0, 0);
 
-	if (rc < 0) {
+	if (rc == -EINVAL) {
+		CDEBUG(D_IOCTL, "%s: No changelog recnum <= %llu to clear\n",
+		       mdd2obd_dev(mdd)->obd_name, (unsigned long long) endrec);
+		RETURN(-EINVAL);
+	} else if (rc < 0) {
 		CWARN("%s: Failure to clear the changelog for user %d: %d\n",
 		      mdd2obd_dev(mdd)->obd_name, id, rc);
 	} else if (mcuc.mcuc_flush) {
@@ -1832,7 +1836,7 @@ static int mdd_changelog_clear(const struct lu_env *env,
 						      mcuc.mcuc_minrec);
 		}
 	} else {
-		CWARN("%s: No entry for user %d\n",
+		CDEBUG(D_IOCTL, "%s: No entry for user %d\n",
 		      mdd2obd_dev(mdd)->obd_name, id);
 		rc = -ENOENT;
 	}