Re: [PATCH] vfs: check dentry is still valid in get_link()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Ian Kent <raven@xxxxxxxxxx>
Subject: Re: [PATCH] vfs: check dentry is still valid in get_link()
From: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
Date: Tue, 18 Jan 2022 03:03:52 +0000
Cc: Brian Foster <bfoster@xxxxxxxxxx>, "Darrick J. Wong" <djwong@xxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, Miklos Szeredi <miklos@xxxxxxxxxx>, David Howells <dhowells@xxxxxxxxxx>, Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, linux-fsdevel <linux-fsdevel@xxxxxxxxxxxxxxx>, xfs <linux-xfs@xxxxxxxxxxxxxxx>, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
In-reply-to: <0f6c2348dae2c47ea46a986884a75fc7d44bb6fb.camel@themaw.net>

On Tue, Jan 18, 2022 at 10:31:53AM +0800, Ian Kent wrote:

> Wouldn't that case be caught by the unlazy call since ->get_link()
> needs to return -ECHILD for the rcu case now (in xfs anyway)?

*shrug*

that'd solve the problem, all right, but it's a serious overkill in
several respects.

References:
- [PATCH] vfs: check dentry is still valid in get_link()
  - From: Ian Kent
- Re: [PATCH] vfs: check dentry is still valid in get_link()
  - From: Al Viro
- Re: [PATCH] vfs: check dentry is still valid in get_link()
  - From: Ian Kent
- Re: [PATCH] vfs: check dentry is still valid in get_link()
  - From: Brian Foster
- Re: [PATCH] vfs: check dentry is still valid in get_link()
  - From: Al Viro
- Re: [PATCH] vfs: check dentry is still valid in get_link()
  - From: Al Viro
- Re: [PATCH] vfs: check dentry is still valid in get_link()
  - From: Al Viro
- Re: [PATCH] vfs: check dentry is still valid in get_link()
  - From: Al Viro
- Re: [PATCH] vfs: check dentry is still valid in get_link()
  - From: Ian Kent

Prev by Date: Re: [PATCH] vfs: check dentry is still valid in get_link()
Next by Date: Re: [PATCH] vfs: check dentry is still valid in get_link()
Previous by thread: Re: [PATCH] vfs: check dentry is still valid in get_link()
Next by thread: Re: [PATCH] vfs: check dentry is still valid in get_link()
Index(es):
- Date
- Thread

[Index of Archives] [Linux Ext4 Filesystem] [Union Filesystem] [Filesystem Testing] [Ceph Users] [Ecryptfs] [NTFS 3] [AutoFS] [Kernel Newbies] [Share Photos] [Security] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux Cachefs] [Reiser Filesystem] [Linux RAID] [NTFS 3] [Samba] [Device Mapper] [CEPH Development]