[PATCH] ipv6: Remove extra counter pull before gc

Tanmay Bhushan posted 1 patch 2 years, 8 months ago
net/ipv6/route.c | 4 ----
1 file changed, 4 deletions(-)
[PATCH] ipv6: Remove extra counter pull before gc
Posted by Tanmay Bhushan 2 years, 8 months ago
Per cpu entries are no longer used in consideration
for doing gc or not. Remove the extra per cpu entries
pull to directly check for time and perform gc.

Signed-off-by: Tanmay Bhushan <007047221b@gmail.com>
---
 net/ipv6/route.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/net/ipv6/route.c b/net/ipv6/route.c
index b643dda68d31..76889ceeead9 100644
--- a/net/ipv6/route.c
+++ b/net/ipv6/route.c
@@ -3294,10 +3294,6 @@ static void ip6_dst_gc(struct dst_ops *ops)
 	unsigned int val;
 	int entries;
 
-	entries = dst_entries_get_fast(ops);
-	if (entries > ops->gc_thresh)
-		entries = dst_entries_get_slow(ops);
-
 	if (time_after(rt_last_gc + rt_min_interval, jiffies))
 		goto out;
 
-- 
2.34.1
Re: [PATCH] ipv6: Remove extra counter pull before gc
Posted by patchwork-bot+netdevbpf@kernel.org 2 years, 7 months ago
Hello:

This patch was applied to netdev/net-next.git (master)
by David S. Miller <davem@davemloft.net>:

On Mon, 16 Jan 2023 15:55:00 +0100 you wrote:
> Per cpu entries are no longer used in consideration
> for doing gc or not. Remove the extra per cpu entries
> pull to directly check for time and perform gc.
> 
> Signed-off-by: Tanmay Bhushan <007047221b@gmail.com>
> ---
>  net/ipv6/route.c | 4 ----
>  1 file changed, 4 deletions(-)

Here is the summary with links:
  - ipv6: Remove extra counter pull before gc
    https://git.kernel.org/netdev/net-next/c/9259f6b573cf

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
Re: [PATCH] ipv6: Remove extra counter pull before gc
Posted by Eric Dumazet 2 years, 8 months ago
On Mon, Jan 16, 2023 at 3:55 PM Tanmay Bhushan <007047221b@gmail.com> wrote:
>
> Per cpu entries are no longer used in consideration
> for doing gc or not. Remove the extra per cpu entries
> pull to directly check for time and perform gc.
>
> Signed-off-by: Tanmay Bhushan <007047221b@gmail.com>
> ---
>  net/ipv6/route.c | 4 ----
>  1 file changed, 4 deletions(-)
>

OK, please next time specify which tree your patch is targeting (this
is for net-next tree here)

Reviewed-by: Eric Dumazet <edumazet@google.com>

Thanks.