Whamcloud - gitweb
LU-6316 lfsck: skip dot name entry 23/13923/2
authorFan Yong <fan.yong@intel.com>
Thu, 4 Dec 2014 15:19:57 +0000 (23:19 +0800)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 25 Mar 2015 13:03:31 +0000 (13:03 +0000)
It is unnecessary for the namespace LFSCK to verify the dot
entry since it is always on the local MDT and has no linkEA.

Signed-off-by: Fan Yong <fan.yong@intel.com>
Change-Id: I01289b04c8807e930c6f777007f1e1fb3295431d
Reviewed-on: http://review.whamcloud.com/13923
Tested-by: Jenkins
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Lai Siyao <lai.siyao@intel.com>
Reviewed-by: Alex Zhuravlev <alexey.zhuravlev@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/lfsck/lfsck_engine.c
lustre/lfsck/lfsck_namespace.c

index 8a05d84..2de9537 100644 (file)
@@ -783,6 +783,10 @@ static int lfsck_master_dir_engine(const struct lu_env *env,
                    strcmp(ent->lde_name, dotdot) != 0)
                        goto checkpoint;
 
+               /* skip dot entry. */
+               if (ent->lde_namelen == 1 && ent->lde_name[0] == '.')
+                       goto checkpoint;
+
                /* The type in the @ent structure may has been overwritten,
                 * so we need to pass the @type parameter independently. */
                rc = lfsck_exec_dir(env, lfsck, ent, type);
index 7755e33..bf8dd2f 100644 (file)
@@ -5066,8 +5066,7 @@ static int lfsck_namespace_assistant_handler_p1(const struct lu_env *env,
                GOTO(put_dir, rc);
        }
 
-       if (lnr->lnr_name[0] == '.' &&
-           (lnr->lnr_namelen == 1 || fid_seq_is_dot(fid_seq(&lnr->lnr_fid))))
+       if (fid_seq_is_dot(fid_seq(&lnr->lnr_fid)))
                GOTO(out, rc = 0);
 
        if (lnr->lnr_lmv != NULL && lnr->lnr_lmv->ll_lmv_master) {