On Tue, Feb 14, 2017 at 08:34:16AM +1100, Dave Chinner wrote:
> >
> > - if (pip && XFS_INHERIT_GID(pip)) {
> > - ip->i_d.di_gid = pip->i_d.di_gid;
> > - if ((VFS_I(pip)->i_mode & S_ISGID) && S_ISDIR(mode))
> > - inode->i_mode |= S_ISGID;
> > - }
> > -
>
> Doesn't this hunk break the "nogrpid" mount option?
It does.
