* [PATCH] NFS: Fix an rcu deadlock in nfs_delegation_find_inode()
@ 2018-06-14 13:39 schumaker.anna
2018-06-14 16:26 ` Trond Myklebust
0 siblings, 1 reply; 2+ messages in thread
From: schumaker.anna @ 2018-06-14 13:39 UTC (permalink / raw)
To: Trond.Myklebust, linux-nfs; +Cc: ross.zwisler
From: Anna Schumaker <Anna.Schumaker@Netapp.com>
I was able to reproduce this pretty regularily using xfstests
generic/013 on NFS v4.0.
Reported-by: Ross Zwisler <Ross.Zwisler@linux.intel.com>
Fixes: a337fe91 (NFSv4: Return NFS4ERR_DELAY when a delegation recall fails due to igrab())
Signed-off-by: Anna Schumaker <Anna.Schumaker@Netapp.com>
---
fs/nfs/delegation.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/fs/nfs/delegation.c b/fs/nfs/delegation.c
index bbd0465535eb..f033f3a69a3b 100644
--- a/fs/nfs/delegation.c
+++ b/fs/nfs/delegation.c
@@ -883,8 +883,10 @@ struct inode *nfs_delegation_find_inode(struct nfs_client *clp,
rcu_read_lock();
list_for_each_entry_rcu(server, &clp->cl_superblocks, client_link) {
res = nfs_delegation_find_inode_server(server, fhandle);
- if (res != ERR_PTR(-ENOENT))
+ if (res != ERR_PTR(-ENOENT)) {
+ rcu_read_unlock();
return res;
+ }
}
rcu_read_unlock();
return ERR_PTR(-ENOENT);
--
2.17.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] NFS: Fix an rcu deadlock in nfs_delegation_find_inode()
2018-06-14 13:39 [PATCH] NFS: Fix an rcu deadlock in nfs_delegation_find_inode() schumaker.anna
@ 2018-06-14 16:26 ` Trond Myklebust
0 siblings, 0 replies; 2+ messages in thread
From: Trond Myklebust @ 2018-06-14 16:26 UTC (permalink / raw)
To: linux-nfs, schumaker.anna; +Cc: ross.zwisler
T24gVGh1LCAyMDE4LTA2LTE0IGF0IDA5OjM5IC0wNDAwLCBzY2h1bWFrZXIuYW5uYUBnbWFpbC5j
b20gd3JvdGU6DQo+IEZyb206IEFubmEgU2NodW1ha2VyIDxBbm5hLlNjaHVtYWtlckBOZXRhcHAu
Y29tPg0KPiANCj4gSSB3YXMgYWJsZSB0byByZXByb2R1Y2UgdGhpcyBwcmV0dHkgcmVndWxhcmls
eSB1c2luZyB4ZnN0ZXN0cw0KPiBnZW5lcmljLzAxMyBvbiBORlMgdjQuMC4NCj4gDQo+IFJlcG9y
dGVkLWJ5OiBSb3NzIFp3aXNsZXIgPFJvc3MuWndpc2xlckBsaW51eC5pbnRlbC5jb20+DQo+IEZp
eGVzOiBhMzM3ZmU5MSAoTkZTdjQ6IFJldHVybiBORlM0RVJSX0RFTEFZIHdoZW4gYSBkZWxlZ2F0
aW9uIHJlY2FsbA0KPiBmYWlscyBkdWUgdG8gaWdyYWIoKSkNCj4gU2lnbmVkLW9mZi1ieTogQW5u
YSBTY2h1bWFrZXIgPEFubmEuU2NodW1ha2VyQE5ldGFwcC5jb20+DQo+IC0tLQ0KPiAgZnMvbmZz
L2RlbGVnYXRpb24uYyB8IDQgKysrLQ0KPiAgMSBmaWxlIGNoYW5nZWQsIDMgaW5zZXJ0aW9ucygr
KSwgMSBkZWxldGlvbigtKQ0KPiANCj4gZGlmZiAtLWdpdCBhL2ZzL25mcy9kZWxlZ2F0aW9uLmMg
Yi9mcy9uZnMvZGVsZWdhdGlvbi5jDQo+IGluZGV4IGJiZDA0NjU1MzVlYi4uZjAzM2YzYTY5YTNi
IDEwMDY0NA0KPiAtLS0gYS9mcy9uZnMvZGVsZWdhdGlvbi5jDQo+ICsrKyBiL2ZzL25mcy9kZWxl
Z2F0aW9uLmMNCj4gQEAgLTg4Myw4ICs4ODMsMTAgQEAgc3RydWN0IGlub2RlICpuZnNfZGVsZWdh
dGlvbl9maW5kX2lub2RlKHN0cnVjdA0KPiBuZnNfY2xpZW50ICpjbHAsDQo+ICAJcmN1X3JlYWRf
bG9jaygpOw0KPiAgCWxpc3RfZm9yX2VhY2hfZW50cnlfcmN1KHNlcnZlciwgJmNscC0+Y2xfc3Vw
ZXJibG9ja3MsDQo+IGNsaWVudF9saW5rKSB7DQo+ICAJCXJlcyA9IG5mc19kZWxlZ2F0aW9uX2Zp
bmRfaW5vZGVfc2VydmVyKHNlcnZlciwNCj4gZmhhbmRsZSk7DQo+IC0JCWlmIChyZXMgIT0gRVJS
X1BUUigtRU5PRU5UKSkNCj4gKwkJaWYgKHJlcyAhPSBFUlJfUFRSKC1FTk9FTlQpKSB7DQo+ICsJ
CQlyY3VfcmVhZF91bmxvY2soKTsNCj4gIAkJCXJldHVybiByZXM7DQo+ICsJCX0NCj4gIAl9DQo+
ICAJcmN1X3JlYWRfdW5sb2NrKCk7DQo+ICAJcmV0dXJuIEVSUl9QVFIoLUVOT0VOVCk7DQoNCk9o
LCBkdWguLi4gVGhhbmtzIGZvciBzcG90dGluZyB0aGF0IQ0KDQotLSANClRyb25kIE15a2xlYnVz
dA0KTGludXggTkZTIGNsaWVudCBtYWludGFpbmVyLCBIYW1tZXJzcGFjZQ0KdHJvbmQubXlrbGVi
dXN0QGhhbW1lcnNwYWNlLmNvbQ0KDQo=
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-06-14 16:26 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-06-14 13:39 [PATCH] NFS: Fix an rcu deadlock in nfs_delegation_find_inode() schumaker.anna
2018-06-14 16:26 ` Trond Myklebust
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).