Commit bdef16d6 authored by Lai Siyao's avatar Lai Siyao Committed by Oleg Drokin
Browse files

LU-13922 osd-ldiskfs: no need to add OI cache in readdir

It's a waste of time to call osd_add_oi_cache() in osd_it_ea_rec(),
because each dirent read will override it.

Lustre-change: https://review.whamcloud.com/39782
Lustre-commit: bc593463

Signed-off-by: default avatarLai Siyao <lai.siyao@whamcloud.com>
Change-Id: Iec701bf66153fdf2ba7a3f3b89565381215abf33
Reviewed-by: default avatarAndreas Dilger <adilger@whamcloud.com>
Reviewed-by: default avatarStephane Thiell <sthiell@stanford.edu>
Signed-off-by: default avatarMinh Diep <mdiep@whamcloud.com>
Reviewed-on: https://review.whamcloud.com/40135

Tested-by: default avatarjenkins <devops@whamcloud.com>
Tested-by: default avatarMaloo <maloo@whamcloud.com>
Reviewed-by: default avatarOleg Drokin <green@whamcloud.com>
parent ddbfc253
......@@ -7239,8 +7239,6 @@ static inline int osd_it_ea_rec(const struct lu_env *env,
rc = osd_ea_fid_get(env, obj, ino, fid, id);
}
} else {
osd_id_gen(id, ino, OSD_OII_NOGEN);
}
}
......@@ -7250,15 +7248,6 @@ static inline int osd_it_ea_rec(const struct lu_env *env,
it->oie_dirent->oied_namelen,
it->oie_dirent->oied_type, attr);
if (rc < 0)
RETURN(rc);
if (osd_remote_fid(env, dev, fid))
RETURN(0);
if (likely(!(attr & (LUDA_IGNORE | LUDA_UNKNOWN)) && rc == 0))
osd_add_oi_cache(oti, dev, id, fid);
RETURN(rc > 0 ? 0 : rc);
}
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment