lkml.org 
[lkml]   [2019]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH ghak100 V2 2/2] audit: ignore fcaps on umount
On Mon, Jan 28, 2019 at 6:25 PM Paul Moore <paul@paul-moore.com> wrote:
> On Fri, Jan 25, 2019 at 5:27 PM Richard Guy Briggs <rgb@redhat.com> wrote:
> > On 2019-01-25 16:45, Paul Moore wrote:
> > > On Wed, Jan 23, 2019 at 1:35 PM Richard Guy Briggs <rgb@redhat.com> wrote:
> > > > Don't fetch fcaps when umount2 is called to avoid a process hang while
> > > > it waits for the missing resource to (possibly never) re-appear.
> > > >
> > > > Note the comment above user_path_mountpoint_at():
> > > > * A umount is a special case for path walking. We're not actually interested
> > > > * in the inode in this situation, and ESTALE errors can be a problem. We
> > > > * simply want track down the dentry and vfsmount attached at the mountpoint
> > > > * and avoid revalidating the last component.
> > > >
> > > > This can happen on ceph, cifs, 9p, lustre, fuse (gluster) or NFS.
> > > >
> > > > Please see the github issue tracker
> > > > https://github.com/linux-audit/audit-kernel/issues/100
> > > >
> > > > Signed-off-by: Richard Guy Briggs <rgb@redhat.com>
> > > > ---
> > > > fs/namei.c | 2 +-
> > > > fs/namespace.c | 2 ++
> > > > include/linux/audit.h | 15 ++++++++++-----
> > > > include/linux/namei.h | 3 +++
> > > > kernel/audit.c | 10 +++++++++-
> > > > kernel/audit.h | 2 +-
> > > > kernel/auditsc.c | 6 +++---
> > > > 7 files changed, 29 insertions(+), 11 deletions(-)

...

> You posted this patch on the 23rd (last Wednesday), let's give the FS
> folks another day or two to comment since it does touch their code.
> If we don't see any objections by later this week I'll merge it into
> audit/next.

I'm not seeing any objections from the fs folks, so I'm merging this
into audit/next.

--
paul moore
www.paul-moore.com

\
 
 \ /
  Last update: 2019-01-31 03:21    [W:0.036 / U:33.104 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site