Messages in this thread | | | Subject | Re: [Ecryptfs-devel] [PATCH] ecryptfs: some inode attrs, and a question | From | Dave Kleikamp <> | Date | Fri, 16 Jan 2009 10:59:29 -0600 |
| |
On Fri, 2009-01-16 at 16:36 +0900, hooanon05@yahoo.co.jp wrote: > Tyler Hicks:
> I guess the problem was related to the i_count including my fixes for > ecryptfs_unlink() and ecryptfs_link(). Current ecryptfs_link() calls > ecryptfs_interpose(), but obviously the inode is not I_NEW and the > incremented i_count is decremented again (finally unchanged). > The current unnecessary d_drop()s may help hiding the problem, but I am > not sure.
I think you're hitting on something here. I never understood the need for the d_drop()s, but taking them out broke things. They probably are just papering over bugs where the ecryptfs inode is not being properly updated after changes are made to the lower inode.
Shaggy -- David Kleikamp IBM Linux Technology Center
| |