RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-01-03 09:32:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

FileStore: FDCache lookup is rearranged

Browse Source 
In lfn_open() there is no point of building the Index if the
cache lookup is successful and caller is not asking for Index.

Signed-off-by: Somnath Roy <somnath.roy@sandisk.com>
This commit is contained in:
Somnath Roy 2014-06-30 01:54:36 -07:00
parent 78d70daff4
commit cfff9f6ac3
1 changed files with 16 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/os/FileStore.cc
View File

@ -225,8 +225,23 @@ int FileStore::lfn_open(coll_t cid,
( oid.shard_id == shard_id_t::NO_SHARD &&
oid.generation == ghobject_t::NO_GEN ));
assert(outfd);
int r = 0;
bool need_lock = true;
if (!replaying) {
*outfd = fdcache.lookup(oid);
if (*outfd) {
if (!index) {
return 0;
} else {
if (!((*index).index)) {
r = get_index(cid, index);
return r;
}
}
}
}
int flags = O_RDWR;
if (create)
flags |= O_CREAT;
@ -235,7 +250,7 @@ int FileStore::lfn_open(coll_t cid,
if (!index) {
index = &index2;
}
int r = 0;
if (!((*index).index)) {
r = get_index(cid, index);
} else {
@ -243,11 +258,6 @@ int FileStore::lfn_open(coll_t cid,
}
int fd, exist;
if (!replaying) {
*outfd = fdcache.lookup(oid);
if (*outfd)
return 0;
}
assert(NULL != (*index).index);