diff options
| author | David Howells <dhowells@redhat.com> | 2023-11-08 16:01:11 +0300 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2024-01-01 19:37:27 +0300 |
| commit | 445f9b6952869586990ec3140dcd87c86d795d2e (patch) | |
| tree | ae2a5365ed9a1b9d0831c0a81fa4bd877a003b97 /fs/afs/vl_alias.c | |
| parent | ca0e79a46097d54e4af46c67c852479d97af35bb (diff) | |
| download | linux-445f9b6952869586990ec3140dcd87c86d795d2e.tar.xz | |
afs: Defer volume record destruction to a workqueue
Defer volume record destruction to a workqueue so that afs_put_volume()
isn't going to run the destruction process in the callback workqueue whilst
the server is holding up other clients whilst waiting for us to reply to a
CB.CallBack notification RPC.
Signed-off-by: David Howells <dhowells@redhat.com>
cc: Marc Dionne <marc.dionne@auristor.com>
cc: linux-afs@lists.infradead.org
Diffstat (limited to 'fs/afs/vl_alias.c')
| -rw-r--r-- | fs/afs/vl_alias.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/afs/vl_alias.c b/fs/afs/vl_alias.c index 63e7ed324af9..9f36e14f1c2d 100644 --- a/fs/afs/vl_alias.c +++ b/fs/afs/vl_alias.c @@ -156,7 +156,7 @@ static int afs_query_for_alias_one(struct afs_cell *cell, struct key *key, /* And see if it's in the new cell. */ volume = afs_sample_volume(cell, key, pvol->name, pvol->name_len); if (IS_ERR(volume)) { - afs_put_volume(cell->net, pvol, afs_volume_trace_put_query_alias); + afs_put_volume(pvol, afs_volume_trace_put_query_alias); if (PTR_ERR(volume) != -ENOMEDIUM) return PTR_ERR(volume); /* That volume is not in the new cell, so not an alias */ @@ -174,8 +174,8 @@ static int afs_query_for_alias_one(struct afs_cell *cell, struct key *key, rcu_read_unlock(); } - afs_put_volume(cell->net, volume, afs_volume_trace_put_query_alias); - afs_put_volume(cell->net, pvol, afs_volume_trace_put_query_alias); + afs_put_volume(volume, afs_volume_trace_put_query_alias); + afs_put_volume(pvol, afs_volume_trace_put_query_alias); return ret; } |