Whamcloud - gitweb
LU-3098 mdt: continue on LFSCK start error in mdt_prepare()
authorJohn L. Hammond <john.hammond@intel.com>
Wed, 3 Apr 2013 15:09:15 +0000 (10:09 -0500)
committerOleg Drokin <oleg.drokin@intel.com>
Tue, 9 Apr 2013 03:11:53 +0000 (23:11 -0400)
In mdt_prepare() do not allow failure of the OBD_IOC_START_LFSCK
ioctl() to prevent mount from succeeding on slave MDTs.

Signed-off-by: John L. Hammond <john.hammond@intel.com>
Change-Id: Ie3ab2e61037739e3324f6ec28e5f73be861b58f5
Reviewed-on: http://review.whamcloud.com/5931
Tested-by: Hudson
Reviewed-by: wangdi <di.wang@intel.com>
Reviewed-by: Fan Yong <fan.yong@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/mdt/mdt_handler.c

index 7610285..b9e0c52 100644 (file)
@@ -5034,8 +5034,11 @@ static int mdt_prepare(const struct lu_env *env,
        rc = mdt->mdt_child->md_ops->mdo_iocontrol(env, mdt->mdt_child,
                                                   OBD_IOC_START_LFSCK,
                                                   0, NULL);
        rc = mdt->mdt_child->md_ops->mdo_iocontrol(env, mdt->mdt_child,
                                                   OBD_IOC_START_LFSCK,
                                                   0, NULL);
-       if (rc != 0)
-               CWARN("Fail to auto trigger paused LFSCK.\n");
+       if (rc != 0) {
+               CWARN("%s: auto trigger paused LFSCK failed: rc = %d\n",
+                     mdt_obd_name(mdt), rc);
+               rc = 0;
+       }
 
        if (mdt->mdt_seq_site.ss_node_id == 0) {
                rc = mdt->mdt_child->md_ops->mdo_root_get(env, mdt->mdt_child,
 
        if (mdt->mdt_seq_site.ss_node_id == 0) {
                rc = mdt->mdt_child->md_ops->mdo_root_get(env, mdt->mdt_child,