diff options
| author | Amar Tumballi <amar@gluster.com> | 2011-06-10 03:04:18 +0000 |
|---|---|---|
| committer | Anand Avati <avati@gluster.com> | 2011-06-14 00:13:43 -0700 |
| commit | 5e5aede7873c7deb696efae679d870134ddbc9a5 (patch) | |
| tree | 17bdd3396c2fe78a0fab95ff4caea6be8dcfed91 /libglusterfs | |
| parent | 25da481bc5b06d671e41e5a70b2c145777154bf1 (diff) | |
loc_t: add 'gfid' and 'pargfid' fields
these fields are used mainly in case of selfheal path, where
'inode->gfid'||'parent->gfid' is not yet set.
These fields in 'loc' will have lower precedence than 'inode->gfid'
in client protocol.
Signed-off-by: Amar Tumballi <amar@gluster.com>
Signed-off-by: Anand Avati <avati@gluster.com>
BUG: 2346 (Log message enhancements in GlusterFS - phase 1)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=2346
Diffstat (limited to 'libglusterfs')
| -rw-r--r-- | libglusterfs/src/xlator.c | 3 | ||||
| -rw-r--r-- | libglusterfs/src/xlator.h | 13 |
2 files changed, 15 insertions, 1 deletions
diff --git a/libglusterfs/src/xlator.c b/libglusterfs/src/xlator.c index 01a902d4c9a..d38754ecdd4 100644 --- a/libglusterfs/src/xlator.c +++ b/libglusterfs/src/xlator.c @@ -1046,6 +1046,9 @@ loc_copy (loc_t *dst, loc_t *src) dst->ino = src->ino; + uuid_copy (dst->gfid, src->gfid); + uuid_copy (dst->pargfid, src->pargfid); + if (src->inode) dst->inode = inode_ref (src->inode); diff --git a/libglusterfs/src/xlator.h b/libglusterfs/src/xlator.h index e583e8c1308..0a2dcbafd76 100644 --- a/libglusterfs/src/xlator.h +++ b/libglusterfs/src/xlator.h @@ -78,9 +78,20 @@ typedef int32_t (*event_notify_fn_t) (xlator_t *this, int32_t event, void *data, struct _loc { const char *path; const char *name; - ino_t ino; inode_t *inode; inode_t *parent; + + /* Currently all location based operations are through 'gfid' of inode. + * But the 'inode->gfid' only gets set in higher most layer (as in, + * 'fuse', 'protocol/server', or 'nfs/server'). So if translators want + * to send fops on a inode before the 'inode->gfid' is set, they have to + * make use of below 'gfid' fields + */ + uuid_t gfid; + uuid_t pargfid; + + /* ideally, should not be used */ + ino_t ino; }; |