diff options
author | David Howells <dhowells@redhat.com> | 2023-10-26 10:54:07 +0200 |
---|---|---|
committer | David Howells <dhowells@redhat.com> | 2023-12-24 16:22:53 +0100 |
commit | 6f2ff7e89bd05677f4c08fccafcf625ca3e09c1c (patch) | |
tree | d1e9ddf29f02e24ad17379b67238c319c44baf45 /fs/afs/fsclient.c | |
parent | afs: Wrap most op->error accesses with inline funcs (diff) | |
download | linux-6f2ff7e89bd05677f4c08fccafcf625ca3e09c1c.tar.xz linux-6f2ff7e89bd05677f4c08fccafcf625ca3e09c1c.zip |
afs: Don't put afs_call in afs_wait_for_call_to_complete()
Don't put the afs_call struct in afs_wait_for_call_to_complete() but rather
have the caller do it. This will allow the caller to fish stuff out of the
afs_call struct rather than the afs_addr_cursor struct, thereby allowing a
subsequent patch to subsume it.
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/fsclient.c')
-rw-r--r-- | fs/afs/fsclient.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/fs/afs/fsclient.c b/fs/afs/fsclient.c index 6821ce0f9d63..020073387111 100644 --- a/fs/afs/fsclient.c +++ b/fs/afs/fsclient.c @@ -1612,6 +1612,7 @@ int afs_fs_give_up_all_callbacks(struct afs_net *net, { struct afs_call *call; __be32 *bp; + int ret; _enter(""); @@ -1627,7 +1628,9 @@ int afs_fs_give_up_all_callbacks(struct afs_net *net, call->server = afs_use_server(server, afs_server_trace_give_up_cb); afs_make_call(ac, call, GFP_NOFS); - return afs_wait_for_call_to_complete(call, ac); + afs_wait_for_call_to_complete(call, ac); + afs_put_call(call); + return ret; } /* |