diff options
| author | Paolo Abeni <pabeni@redhat.com> | 2017-10-19 17:07:10 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-10-21 03:39:10 +0300 |
| commit | b886d5f2f2906a866e33212734d204dfe35d50d9 (patch) | |
| tree | 911b97bd3e6644635452914a71405f3aca1055e6 /net | |
| parent | 7f9ad2ace17a3521a80831208d431170ef71591f (diff) | |
| download | linux-b886d5f2f2906a866e33212734d204dfe35d50d9.tar.xz | |
ipv6: start fib6 gc on RTF_CACHE dst creation
After the commit 2b760fcf5cfb ("ipv6: hook up exception table
to store dst cache"), the fib6 gc is not started after the
creation of a RTF_CACHE via a redirect or pmtu update, since
fib6_add() isn't invoked anymore for such dsts.
We need the fib6 gc to run periodically to clean the RTF_CACHE,
or the dst will stay there forever.
Fix it by explicitly calling fib6_force_start_gc() on successful
exception creation. gc_args->more accounting will ensure that
the gc timer will run for whatever time needed to properly
clean the table.
v2 -> v3:
- clarified the commit message
Fixes: 2b760fcf5cfb ("ipv6: hook up exception table to store dst cache")
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Acked-by: Wei Wang <weiwan@google.com>
Acked-by: Martin KaFai Lau <kafai@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/ipv6/route.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/ipv6/route.c b/net/ipv6/route.c index 2e8842fa6450..efecdcff5055 100644 --- a/net/ipv6/route.c +++ b/net/ipv6/route.c @@ -1340,8 +1340,10 @@ out: spin_unlock_bh(&rt6_exception_lock); /* Update fn->fn_sernum to invalidate all cached dst */ - if (!err) + if (!err) { fib6_update_sernum(ort); + fib6_force_start_gc(net); + } return err; } |