2022-02-11 18:35:58

by Kalash Nainwal

[permalink] [raw]
Subject: [PATCH v2] Generate netlink notification when default IPv6 route preference changes

Generate RTM_NEWROUTE netlink notification when the route preference
changes on an existing kernel generated default route in response to
RA messages. Currently netlink notifications are generated only when
this route is added or deleted but not when the route preference
changes, which can cause userspace routing application state to go
out of sync with kernel.

Signed-off-by: Kalash Nainwal <[email protected]>
---
net/ipv6/ndisc.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/net/ipv6/ndisc.c b/net/ipv6/ndisc.c
index f03b597e4121..1c06d0cd02f7 100644
--- a/net/ipv6/ndisc.c
+++ b/net/ipv6/ndisc.c
@@ -1337,8 +1337,12 @@ static void ndisc_router_discovery(struct sk_buff *skb)
return;
}
neigh->flags |= NTF_ROUTER;
- } else if (rt) {
+ } else if (rt && IPV6_EXTRACT_PREF(rt->fib6_flags) != pref) {
+ struct nl_info nlinfo = {
+ .nl_net = net,
+ };
rt->fib6_flags = (rt->fib6_flags & ~RTF_PREF_MASK) | RTF_PREF(pref);
+ inet6_rt_notify(RTM_NEWROUTE, rt, &nlinfo, NLM_F_REPLACE);
}

if (rt)
--
2.30.1 (Apple Git-130)


2022-02-14 04:27:13

by David Ahern

[permalink] [raw]
Subject: Re: [PATCH v2] Generate netlink notification when default IPv6 route preference changes

On 2/10/22 2:09 PM, Kalash Nainwal wrote:
> Generate RTM_NEWROUTE netlink notification when the route preference
> changes on an existing kernel generated default route in response to
> RA messages. Currently netlink notifications are generated only when
> this route is added or deleted but not when the route preference
> changes, which can cause userspace routing application state to go
> out of sync with kernel.
>
> Signed-off-by: Kalash Nainwal <[email protected]>
> ---
> net/ipv6/ndisc.c | 6 +++++-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>

Reviewed-by: David Ahern <[email protected]>


2022-02-14 21:01:57

by patchwork-bot+netdevbpf

[permalink] [raw]
Subject: Re: [PATCH v2] Generate netlink notification when default IPv6 route preference changes

Hello:

This patch was applied to netdev/net-next.git (master)
by David S. Miller <[email protected]>:

On Thu, 10 Feb 2022 14:09:35 -0800 you wrote:
> Generate RTM_NEWROUTE netlink notification when the route preference
> changes on an existing kernel generated default route in response to
> RA messages. Currently netlink notifications are generated only when
> this route is added or deleted but not when the route preference
> changes, which can cause userspace routing application state to go
> out of sync with kernel.
>
> [...]

Here is the summary with links:
- [v2] Generate netlink notification when default IPv6 route preference changes
https://git.kernel.org/netdev/net-next/c/806c37ddcf28

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html