From: Shiming Cheng <[email protected]>
When rule policy is changed, ipv6 socket cache is not refreshed.
The sock's skb still uses a outdated route cache and was sent to
a wrong interface.
To avoid this error we should update fib node's version when
rule is changed. Then skb's route will be reroute checked as
route cache version is already different with fib node version.
The route cache is refreshed to match the latest rule.
Fixes: 101367c2f8c4 ("[IPV6]: Policy Routing Rules")
Signed-off-by: Shiming Cheng <[email protected]>
Signed-off-by: Lena Wang <[email protected]>
---
v4:
- add "Fixes:" tag.
- update subject as requested.
v3:
- update patch description and name format in commit message.
v2:
- modify flush function same way as ipv4 flush cache.
- use tabs to aligh with existing code.
---
---
net/ipv6/fib6_rules.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/net/ipv6/fib6_rules.c b/net/ipv6/fib6_rules.c
index 7523c4baef35..52c04f0ac498 100644
--- a/net/ipv6/fib6_rules.c
+++ b/net/ipv6/fib6_rules.c
@@ -449,6 +449,11 @@ static size_t fib6_rule_nlmsg_payload(struct
fib_rule *rule)
+ nla_total_size(16); /* src */
}
+static void fib6_rule_flush_cache(struct fib_rules_ops *ops)
+{
+ rt_genid_bump_ipv6(ops->fro_net);
+}
+
static const struct fib_rules_ops __net_initconst
fib6_rules_ops_template = {
.family = AF_INET6,
.rule_size = sizeof(struct fib6_rule),
@@ -461,6 +466,7 @@ static const struct fib_rules_ops __net_initconst
fib6_rules_ops_template = {
.compare = fib6_rule_compare,
.fill = fib6_rule_fill,
.nlmsg_payload = fib6_rule_nlmsg_payload,
+ .flush_cache = fib6_rule_flush_cache,
.nlgroup = RTNLGRP_IPV6_RULE,
.owner = THIS_MODULE,
.fro_net = &init_net,
--
2.18.0
On 3/1/24 7:39 AM, Lena Wang (王娜) wrote:
> From: Shiming Cheng <[email protected]>
>
> When rule policy is changed, ipv6 socket cache is not refreshed.
> The sock's skb still uses a outdated route cache and was sent to
> a wrong interface.
>
> To avoid this error we should update fib node's version when
> rule is changed. Then skb's route will be reroute checked as
> route cache version is already different with fib node version.
> The route cache is refreshed to match the latest rule.
>
> Fixes: 101367c2f8c4 ("[IPV6]: Policy Routing Rules")
> Signed-off-by: Shiming Cheng <[email protected]>
> Signed-off-by: Lena Wang <[email protected]>
> ---
> v4:
> - add "Fixes:" tag.
> - update subject as requested.
> v3:
> - update patch description and name format in commit message.
> v2:
> - modify flush function same way as ipv4 flush cache.
> - use tabs to aligh with existing code.
> ---
> ---
> net/ipv6/fib6_rules.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
Reviewed-by: David Ahern <[email protected]>
Fri, Mar 01, 2024 at 03:39:46PM CET, [email protected] wrote:
>From: Shiming Cheng <[email protected]>
>
>When rule policy is changed, ipv6 socket cache is not refreshed.
>The sock's skb still uses a outdated route cache and was sent to
>a wrong interface.
>
>To avoid this error we should update fib node's version when
>rule is changed. Then skb's route will be reroute checked as
>route cache version is already different with fib node version.
>The route cache is refreshed to match the latest rule.
>
>Fixes: 101367c2f8c4 ("[IPV6]: Policy Routing Rules")
>Signed-off-by: Shiming Cheng <[email protected]>
>Signed-off-by: Lena Wang <[email protected]>
Reviewed-by: Jiri Pirko <[email protected]>
On Fri, 1 Mar 2024 14:39:46 +0000 Lena Wang (王娜) wrote:
> From: Shiming Cheng <[email protected]>
>
> When rule policy is changed, ipv6 socket cache is not refreshed.
> The sock's skb still uses a outdated route cache and was sent to
> a wrong interface.
>
> To avoid this error we should update fib node's version when
> rule is changed. Then skb's route will be reroute checked as
> route cache version is already different with fib node version.
> The route cache is refreshed to match the latest rule.
Doesn't apply, please rebase on top of latest net/main.
On Mon, 2024-03-04 at 20:45 -0800, Jakub Kicinski wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> On Fri, 1 Mar 2024 14:39:46 +0000 Lena Wang (王娜) wrote:
> > From: Shiming Cheng <[email protected]>
> >
> > When rule policy is changed, ipv6 socket cache is not refreshed.
> > The sock's skb still uses a outdated route cache and was sent to
> > a wrong interface.
> >
> > To avoid this error we should update fib node's version when
> > rule is changed. Then skb's route will be reroute checked as
> > route cache version is already different with fib node version.
> > The route cache is refreshed to match the latest rule.
>
> Doesn't apply, please rebase on top of latest net/main.
Hi Jakub,
I use master branch to make this patch. And it seems same with
main branch of kernel/git/netdev/net.git.
Could you tell me which branch should be used?
Thanks
Lena
On Tue, Mar 05, 2024 at 01:01:24PM +0000, Lena Wang (王娜) wrote:
> On Mon, 2024-03-04 at 20:45 -0800, Jakub Kicinski wrote:
> >
> > External email : Please do not click links or open attachments until
> > you have verified the sender or the content.
> > On Fri, 1 Mar 2024 14:39:46 +0000 Lena Wang (王娜) wrote:
> > > From: Shiming Cheng <[email protected]>
> > >
> > > When rule policy is changed, ipv6 socket cache is not refreshed.
> > > The sock's skb still uses a outdated route cache and was sent to
> > > a wrong interface.
> > >
> > > To avoid this error we should update fib node's version when
> > > rule is changed. Then skb's route will be reroute checked as
> > > route cache version is already different with fib node version.
> > > The route cache is refreshed to match the latest rule.
> >
> > Doesn't apply, please rebase on top of latest net/main.
> Hi Jakub,
> I use master branch to make this patch. And it seems same with
> main branch of kernel/git/netdev/net.git.
>
> Could you tell me which branch should be used?
>
> Thanks
> Lena
Hi Lena,
The primary branch is main these days.
If you are using master than it may well be stale.
For reference, the current HEAD commit is.
4daa873133d3 ("Merge tag 'mlx5-fixes-2024-03-01' of git://git.kernel.org/pub/scm/linux/kernel/git/saeed/linux")
On Tue, 2024-03-05 at 15:23 +0000, Simon Horman wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> On Tue, Mar 05, 2024 at 01:01:24PM +0000, Lena Wang (王娜) wrote:
> > On Mon, 2024-03-04 at 20:45 -0800, Jakub Kicinski wrote:
> > >
> > > External email : Please do not click links or open attachments
> until
> > > you have verified the sender or the content.
> > > On Fri, 1 Mar 2024 14:39:46 +0000 Lena Wang (王娜) wrote:
> > > > From: Shiming Cheng <[email protected]>
> > > >
> > > > When rule policy is changed, ipv6 socket cache is not
> refreshed.
> > > > The sock's skb still uses a outdated route cache and was sent
> to
> > > > a wrong interface.
> > > >
> > > > To avoid this error we should update fib node's version when
> > > > rule is changed. Then skb's route will be reroute checked as
> > > > route cache version is already different with fib node version.
> > > > The route cache is refreshed to match the latest rule.
> > >
> > > Doesn't apply, please rebase on top of latest net/main.
> > Hi Jakub,
> > I use master branch to make this patch. And it seems same with
> > main branch of kernel/git/netdev/net.git.
> >
> > Could you tell me which branch should be used?
> >
> > Thanks
> > Lena
>
> Hi Lena,
>
> The primary branch is main these days.
> If you are using master than it may well be stale.
>
> For reference, the current HEAD commit is.
>
> 4daa873133d3 ("Merge tag 'mlx5-fixes-2024-03-01' of
> git://git.kernel.org/pub/scm/linux/kernel/git/saeed/linux")
Hi Simon,
I clone a latest netdev main branch (on the 4daa873133d3) and make a
new patch diff. However it is totally same with PATCH v4.
Could you please help to check where the stale code is?
Thanks
Lena
On Wed, 6 Mar 2024 07:28:39 +0000 Lena Wang (王娜) wrote:
> > The primary branch is main these days.
> > If you are using master than it may well be stale.
> >
> > For reference, the current HEAD commit is.
> >
> > 4daa873133d3 ("Merge tag 'mlx5-fixes-2024-03-01' of
> > git://git.kernel.org/pub/scm/linux/kernel/git/saeed/linux")
>
> Hi Simon,
> I clone a latest netdev main branch (on the 4daa873133d3) and make a
> new patch diff. However it is totally same with PATCH v4.
>
> Could you please help to check where the stale code is?
It's not the diff, your email setup corrupted the patch.
Please try resending with git send-email ?