fs/9p: fix create-unlink-getattr idiom
authorEric Van Hensbergen <ericvh@gmail.com>
Wed, 23 Sep 2020 14:11:43 +0000 (22:11 +0800)
committerDominique Martinet <asmadeus@codewreck.org>
Tue, 3 Nov 2020 08:29:31 +0000 (09:29 +0100)
commit154372e67d4053e56591245eb413686621941333
treee89780e1785ebaebfd068566c549bbd6e3230642
parent3cea11cd5e3b00d91caf0b4730194039b45c5891
fs/9p: fix create-unlink-getattr idiom

Fixes several outstanding bug reports of not being able to getattr from an
open file after an unlink.  This patch cleans up transient fids on an unlink
and will search open fids on a client if it detects a dentry that appears to
have been unlinked.  This search is necessary because fstat does not pass fd
information through the VFS API to the filesystem, only the dentry which for
9p has an imperfect match to fids.

Inherent in this patch is also a fix for the qid handling on create/open
which apparently wasn't being set correctly and was necessary for the search
to succeed.

A possible optimization over this fix is to include accounting of open fids
with the inode in the private data (in a similar fashion to the way we track
transient fids with dentries).  This would allow a much quicker search for
a matching open fid.

(changed v9fs_fid_find_global to v9fs_fid_find_inode in comment)

Link: http://lkml.kernel.org/r/20200923141146.90046-2-jianyong.wu@arm.com
Signed-off-by: Eric Van Hensbergen <ericvh@gmail.com>
Signed-off-by: Greg Kurz <gkurz@linux.vnet.ibm.com>
Signed-off-by: Jianyong Wu <jianyong.wu@arm.com>
Signed-off-by: Dominique Martinet <asmadeus@codewreck.org>
fs/9p/fid.c
fs/9p/vfs_inode.c
net/9p/client.c