diff options
author | Jakub Kicinski <jakub.kicinski@netronome.com> | 2019-09-03 00:31:06 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2019-09-05 03:49:49 -0400 |
commit | 6e3d02b670eafce69aa64457b87c3b17ff216d23 (patch) | |
tree | 87b421fee34bd7c9238ac50bebda2b27883c87f0 | |
parent | be2fbc155fc8c0ff6e499753354d965cd9cf1bb0 (diff) |
net/tls: dedup the record cleanup
If retransmit record hint fall into the cleanup window we will
free it by just walking the list. No need to duplicate the code.
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: John Hurley <john.hurley@netronome.com>
Reviewed-by: Dirk van der Merwe <dirk.vandermerwe@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/tls/tls_device.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/net/tls/tls_device.c b/net/tls/tls_device.c index 9e1bec1a0a28..41c106e45f01 100644 --- a/net/tls/tls_device.c +++ b/net/tls/tls_device.c | |||
@@ -159,12 +159,8 @@ static void tls_icsk_clean_acked(struct sock *sk, u32 acked_seq) | |||
159 | 159 | ||
160 | spin_lock_irqsave(&ctx->lock, flags); | 160 | spin_lock_irqsave(&ctx->lock, flags); |
161 | info = ctx->retransmit_hint; | 161 | info = ctx->retransmit_hint; |
162 | if (info && !before(acked_seq, info->end_seq)) { | 162 | if (info && !before(acked_seq, info->end_seq)) |
163 | ctx->retransmit_hint = NULL; | 163 | ctx->retransmit_hint = NULL; |
164 | list_del(&info->list); | ||
165 | destroy_record(info); | ||
166 | deleted_records++; | ||
167 | } | ||
168 | 164 | ||
169 | list_for_each_entry_safe(info, temp, &ctx->records_list, list) { | 165 | list_for_each_entry_safe(info, temp, &ctx->records_list, list) { |
170 | if (before(acked_seq, info->end_seq)) | 166 | if (before(acked_seq, info->end_seq)) |