Re: [patch 01/15] ecryptfs: clean up (un)lock_parent

From: Christoph Hellwig
Date: Mon May 05 2008 - 07:00:40 EST


On Mon, May 05, 2008 at 11:54:42AM +0200, Miklos Szeredi wrote:
> From: Miklos Szeredi <mszeredi@xxxxxxx>
>
> dget(dentry->d_parent) --> dget_parent(dentry)
>
> unlock_parent() is racy and unnecessary. Replace single caller with
> unlock_dir().
>
> There are several other suspect uses of ->d_parent in ecryptfs...

Looks good an the first hunk is an obvious fix. Should probably go into
.26

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/