Re: [PATCH] Bind Mount Extensions 0.04 (linux-2.4.25)

From: viro
Date: Thu Mar 18 2004 - 07:34:09 EST


On Mon, Mar 15, 2004 at 04:55:59AM +0100, Herbert Poetzl wrote:
> if (count != 0) {
> - UPDATE_ATIME(file->f_dentry->d_inode);
> + UPDATE_ATIME(file->f_dentry->d_inode, file->f_vfsmnt);

For crying out loud... Make that touch_file(file) and be done with that.
There's a lot of places where we do just that (touch atime of opened file)
and passing pair of vfsmount and inode (not even vfsmount and dentry) is
just plain wrong.
-
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/