|
From: | Amar Tumballi |
Subject: | Re: [Gluster-devel] no dentry for non-root inode |
Date: | Tue, 08 Nov 2011 10:56:14 +0530 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:7.0) Gecko/20110927 Thunderbird/7.0 |
Hello Emmanuel,
[2011-11-05 03:12:47.856612] W [inode.c:1044:inode_path] 0-/export/wd3e/inode: no dentry for non-root inode -9091625530591748852: d968c71c-9c3f-471e-81d4-0ebfda34dd0c
This log is not a 'error', but warning. What that means is, this condition need not be causing the error seen by the user.
One of the case why this warning would come up is from below set of operations:
machine 1 --> user creates a file (inode created, fd on inode is created, dentry with filename is linked to parent inode).
machine 2 --> just do a 'cat' on file (inode is looked up, updated in inode_table).
machine 3 --> delete the file (dentry is unlinked from inode, inode still valid as there is fd open on it)
machine 2 --> try to perform some operation on the file (inode is tried to looked up, inode is valid on server as there is a 'fd' open on it, but there is no 'PATH' for it, as dentry is unlinked), user gets error ENOENT. (this is when you can see such logs in server).
machine 1 --> perform operations on 'fd' --> succeeds as fd is still valid.machine 1 --> close the fd.. (inode gets destroyed on server, no more valid inode for that gfid).
Hope you can now relate this type of pattern in your testing.
Jeff, Thanks for answering this part. Hope my answer gives some idea on why inode_path is used (internal terms wise, just to populate 'loc_t *' on server, as storage/posix works based on paths).The error message seems a bit misleading: what is missing is not a true dentry, but a GlusterFS-internal simulacrum of one, which seems to be gone because the parent inode was forgotten. I also see that the message comes from __inode_path, which is only called from three of the performance translators in non-essential code to dump fd contexts. The first thing I'd do is try to figure out which code path is actually involved, and why anyone's dumping that context in the first place. I'll bet there's a platform-specific reason, requiring a platform-specific tweak to avoid the underlying issue.
Regards, Amar
[Prev in Thread] | Current Thread | [Next in Thread] |