Re: [RFC PATCH 16/35] ovl: readd lsattr/chattr support
From: Miklos Szeredi
Date: Mon Apr 23 2018 - 06:28:53 EST
On Mon, Apr 23, 2018 at 12:21 PM, Miklos Szeredi <miklos@xxxxxxxxxx> wrote:
> We could fix it to use file_inode()->i_sb instead of
> f_path.dentry->d_sb after reverting the overlay specific hack, and
> that would fix the freeze bypass bug and would be correct for all
> filesystems. But I wonder how many such issues we have where
> discrepancy between f_path.dentry and file_inode() matters.
OTOH, any such issues should already have surfaced. Just need to be
careful when reverting VFS patches that they don't regress in the face
of f_path containing the overlay path.
So we are probably better to keep using overlay path in real file
universally, just to keep the code simpler.
Thanks,
Miklos