From 3e3395aeab6b99720fd70f942def89eb94f1e26a Mon Sep 17 00:00:00 2001
From: bobijam <bobijam>
Date: Thu, 3 Apr 2008 02:38:58 +0000
Subject: [PATCH] Branch HEAD b=14134 i=shadow, nathan
change non-fatal error messages about contacting the MGS.
---
lustre/mgc/mgc_request.c | 5 +++--
lustre/obdclass/obd_mount.c | 9 +++++----
2 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/lustre/mgc/mgc_request.c b/lustre/mgc/mgc_request.c
index 150fc0d2ce..18ad5c374b 100644
--- a/lustre/mgc/mgc_request.c
+++ b/lustre/mgc/mgc_request.c
@@ -1129,8 +1129,9 @@ static int mgc_process_log(struct obd_device *mgc,
"\n", cld->cld_logname);
GOTO(out_pop, rc = -ENOTCONN);
}
- LCONSOLE_WARN("Failed to get MGS log %s, using "
- "local copy.\n", cld->cld_logname);
+ CDEBUG(D_MGC, "Failed to get MGS log %s, using local "
+ "copy for now, will try to update later.\n",
+ cld->cld_logname);
}
/* Now, whether we copied or not, start using the local llog.
If we failed to copy, we'll start using whatever the old
diff --git a/lustre/obdclass/obd_mount.c b/lustre/obdclass/obd_mount.c
index 9104459d51..649617ad35 100644
--- a/lustre/obdclass/obd_mount.c
+++ b/lustre/obdclass/obd_mount.c
@@ -721,7 +721,7 @@ static int lustre_start_mgc(struct super_block *sb)
sizeof(recov_bk), &recov_bk, NULL);
if (rc)
/* nonfatal */
- CERROR("can't set %s %d\n", KEY_INIT_RECOV_BACKUP, rc);
+ CWARN("can't set %s %d\n", KEY_INIT_RECOV_BACKUP, rc);
/* We connect to the MGS at setup, and don't disconnect until cleanup */
ocd.ocd_connect_flags = OBD_CONNECT_VERSION | OBD_CONNECT_FID;
@@ -977,10 +977,8 @@ int server_register_target(struct super_block *sb)
rc = obd_set_info_async(mgc->u.cli.cl_mgc_mgsexp,
strlen("register_target"), "register_target",
sizeof(*mti), mti, NULL);
- if (rc) {
- CERROR("registration with the MGS failed (%d)\n", rc);
+ if (rc)
GOTO(out, rc);
- }
/* Always update our flags */
ldd->ldd_flags = mti->mti_flags & ~LDD_F_REWRITE_LDD;
@@ -1093,6 +1091,9 @@ static int server_start_targets(struct super_block *sb, struct vfsmount *mnt)
lsi->lsi_ldd->ldd_svname);
GOTO(out_mgc, rc);
}
+ /* non-fatal error of registeration with MGS */
+ if (rc)
+ CDEBUG(D_MOUNT, "Cannot register with MGS: %d\n", rc);
/* Let the target look up the mount using the target's name
(we can't pass the sb or mnt through class_process_config.) */
--
GitLab