diff options
| author | David Howells <dhowells@redhat.com> | 2019-05-09 23:22:50 +0200 |
|---|---|---|
| committer | David Howells <dhowells@redhat.com> | 2019-05-16 17:25:21 +0200 |
| commit | ffba718e935402e7f42b8cd5d1e00e4a3907d361 (patch) | |
| tree | a77dadcb55b3cc56cbb1f8a583448944214ba163 /fs/afs/rxrpc.c | |
| parent | afs: Don't pass the vnode pointer through into the inline bulk status op (diff) | |
| download | linux-ffba718e935402e7f42b8cd5d1e00e4a3907d361.tar.xz linux-ffba718e935402e7f42b8cd5d1e00e4a3907d361.zip | |
afs: Get rid of afs_call::reply[]
Replace the afs_call::reply[] array with a bunch of typed members so that
the compiler can use type-checking on them. It's also easier for the eye
to see what's going on.
Signed-off-by: David Howells <dhowells@redhat.com>
Diffstat (limited to 'fs/afs/rxrpc.c')
| -rw-r--r-- | fs/afs/rxrpc.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/fs/afs/rxrpc.c b/fs/afs/rxrpc.c index 02a39e6adf63..d63e0c9c09e7 100644 --- a/fs/afs/rxrpc.c +++ b/fs/afs/rxrpc.c @@ -188,7 +188,7 @@ void afs_put_call(struct afs_call *call) if (call->type->destructor) call->type->destructor(call); - afs_put_server(call->net, call->cm_server); + afs_put_server(call->net, call->server); afs_put_cb_interest(call->net, call->cbi); afs_put_addrlist(call->alist); kfree(call->request); @@ -696,10 +696,9 @@ long afs_wait_for_call_to_complete(struct afs_call *call, ret = ac->error; switch (ret) { case 0: - if (call->ret_reply0) { - ret = (long)call->reply[0]; - call->reply[0] = NULL; - } + ret = call->ret0; + call->ret0 = 0; + /* Fall through */ case -ECONNABORTED: ac->responded = true; |