diff options
| author | Amar Tumballi <amar@gluster.com> | 2011-06-10 02:50:18 +0000 |
|---|---|---|
| committer | Anand Avati <avati@gluster.com> | 2011-06-14 00:13:51 -0700 |
| commit | 5522ac20241436be6a005058b6360f4339eb678c (patch) | |
| tree | cbf9e466418d97e8f46cfdd1121a8ad86fc70aca /libglusterfs/src | |
| parent | c5321286e540153ff04d189b20c2957776e2b7d5 (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.
also contains 'Pranith <pranithk@gluster.com>'s patch to set proper
loc->gfid during afr selfheal
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/src')
| -rw-r--r-- | libglusterfs/src/xlator.c | 3 | ||||
| -rw-r--r-- | libglusterfs/src/xlator.h | 12 |
2 files changed, 14 insertions, 1 deletions
diff --git a/libglusterfs/src/xlator.c b/libglusterfs/src/xlator.c index b8f800c70..16020c0d2 100644 --- a/libglusterfs/src/xlator.c +++ b/libglusterfs/src/xlator.c @@ -1663,6 +1663,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 e587ae578..60fd77761 100644 --- a/libglusterfs/src/xlator.h +++ b/libglusterfs/src/xlator.h @@ -78,9 +78,19 @@ 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; }; |