From 4f4ff2c5725e60a99aa8d1df83c1574f1ade99e3 Mon Sep 17 00:00:00 2001 From: Landen Date: Tue, 9 Feb 2010 13:23:49 +0800 Subject: [PATCH] b=16909 Simplify MDT/OST service start message i=nathan i=adilger --- lustre/obdclass/obd_mount.c | 5 ----- lustre/obdfilter/filter.c | 27 ++++++++++++--------------- 2 files changed, 12 insertions(+), 20 deletions(-) diff --git a/lustre/obdclass/obd_mount.c b/lustre/obdclass/obd_mount.c index 61d934b..b8c7e7d 100644 --- a/lustre/obdclass/obd_mount.c +++ b/lustre/obdclass/obd_mount.c @@ -1677,11 +1677,6 @@ static int server_fill_super(struct super_block *sb) if (rc) GOTO(out_mnt, rc); - LCONSOLE_WARN("Server %s on device %s has started\n", - ((lsi->lsi_lmd->lmd_flags & LMD_FLG_NOSVC) && - (IS_MDT(lsi->lsi_ldd))) ? "MGS" : lsi->lsi_ldd->ldd_svname, - lsi->lsi_lmd->lmd_dev); - RETURN(0); out_mnt: /* We jump here in case of failure while starting targets or MGS. diff --git a/lustre/obdfilter/filter.c b/lustre/obdfilter/filter.c index 711746a..4aed2ae 100644 --- a/lustre/obdfilter/filter.c +++ b/lustre/obdfilter/filter.c @@ -2138,24 +2138,21 @@ int filter_common_setup(struct obd_device *obd, struct lustre_cfg* lcfg, } label = fsfilt_get_label(obd, obd->u.obt.obt_sb); - - if (obd->obd_recovering) { - LCONSOLE_WARN("OST %s now serving %s (%s%s%s), but will be in " - "recovery for at least %d:%.02d, or until %d " - "client%s reconnect%s.\n", - obd->obd_name, lustre_cfg_string(lcfg, 1), - label ?: "", label ? "/" : "", str, + LCONSOLE_INFO("%s: Now serving %s %s%s with recovery %s\n", + obd->obd_name, label ?: str, lmi ? "on " : "", + lmi ? s2lsi(lmi->lmi_sb)->lsi_lmd->lmd_dev : "", + obd->obd_replayable ? "enabled" : "disabled"); + + if (obd->obd_recovering) + LCONSOLE_WARN("%s: Will be in recovery for at least %d:%.02d, " + "or until %d client%s reconnect%s\n", + obd->obd_name, obd->obd_recovery_timeout / 60, obd->obd_recovery_timeout % 60, obd->obd_max_recoverable_clients, - (obd->obd_max_recoverable_clients == 1) ? "":"s", - (obd->obd_max_recoverable_clients == 1) ? "s":""); - } else { - LCONSOLE_INFO("OST %s now serving %s (%s%s%s) with recovery " - "%s\n", obd->obd_name, lustre_cfg_string(lcfg, 1), - label ?: "", label ? "/" : "", str, - obd->obd_replayable ? "enabled" : "disabled"); - } + (obd->obd_max_recoverable_clients == 1) ? "" : "s", + (obd->obd_max_recoverable_clients == 1) ? "s": ""); + RETURN(0); -- 1.8.3.1