Commit 88c80bee authored by Jakub Kicinski's avatar Jakub Kicinski Committed by David S. Miller

net/tls: remove set but not used variables

Commit 4504ab0e ("net/tls: Inform user space about send buffer availability")
made us report write_space regardless whether partial record
push was successful or not.  Remove the now unused return value
to clean up the following W=1 warning:

net/tls/tls_device.c: In function ‘tls_device_write_space’:
net/tls/tls_device.c:546:6: warning: variable ‘rc’ set but not used [-Wunused-but-set-variable]
  int rc = 0;
      ^~

CC: Vakul Garg <vakul.garg@nxp.com>
CC: Boris Pismenny <borisp@mellanox.com>
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: default avatarDirk van der Merwe <dirk.vandermerwe@netronome.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 15192f25
...@@ -541,14 +541,11 @@ static int tls_device_push_pending_record(struct sock *sk, int flags) ...@@ -541,14 +541,11 @@ static int tls_device_push_pending_record(struct sock *sk, int flags)
void tls_device_write_space(struct sock *sk, struct tls_context *ctx) void tls_device_write_space(struct sock *sk, struct tls_context *ctx)
{ {
int rc = 0;
if (!sk->sk_write_pending && tls_is_partially_sent_record(ctx)) { if (!sk->sk_write_pending && tls_is_partially_sent_record(ctx)) {
gfp_t sk_allocation = sk->sk_allocation; gfp_t sk_allocation = sk->sk_allocation;
sk->sk_allocation = GFP_ATOMIC; sk->sk_allocation = GFP_ATOMIC;
rc = tls_push_partial_record(sk, ctx, tls_push_partial_record(sk, ctx, MSG_DONTWAIT | MSG_NOSIGNAL);
MSG_DONTWAIT | MSG_NOSIGNAL);
sk->sk_allocation = sk_allocation; sk->sk_allocation = sk_allocation;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment