[PATCH] ipv6: delete useless dst check in ip6_dst_lookup_tail

Subsystems: networking [general], networking [ipv4/ipv6], the rest

STALE1871d

2 messages, 2 authors, 2021-06-24 · open the first message on its own page

[PATCH] ipv6: delete useless dst check in ip6_dst_lookup_tail

From: zhang kai <hidden>
Date: 2021-06-24 03:43:19

parameter dst always points to null.

Signed-off-by: zhang kai <redacted>
---
 net/ipv6/ip6_output.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/net/ipv6/ip6_output.c b/net/ipv6/ip6_output.c
index ff4f9ebcf..3b632e469 100644
--- a/net/ipv6/ip6_output.c
+++ b/net/ipv6/ip6_output.c
@@ -1055,13 +1055,11 @@ static int ip6_dst_lookup_tail(struct net *net, const struct sock *sk,
 	 * ip6_route_output will fail given src=any saddr, though, so
 	 * that's why we try it again later.
 	 */
-	if (ipv6_addr_any(&fl6->saddr) && (!*dst || !(*dst)->error)) {
+	if (ipv6_addr_any(&fl6->saddr)) {
 		struct fib6_info *from;
 		struct rt6_info *rt;
-		bool had_dst = *dst != NULL;
 
-		if (!had_dst)
-			*dst = ip6_route_output(net, sk, fl6);
+		*dst = ip6_route_output(net, sk, fl6);
 		rt = (*dst)->error ? NULL : (struct rt6_info *)*dst;
 
 		rcu_read_lock();
@@ -1078,7 +1076,7 @@ static int ip6_dst_lookup_tail(struct net *net, const struct sock *sk,
 		 * never existed and let the SA-enabled version take
 		 * over.
 		 */
-		if (!had_dst && (*dst)->error) {
+		if ((*dst)->error) {
 			dst_release(*dst);
 			*dst = NULL;
 		}
-- 
2.17.1

Re: [PATCH] ipv6: delete useless dst check in ip6_dst_lookup_tail

From: patchwork-bot+netdevbpf@kernel.org
Date: 2021-06-24 22:50:44

Hello:

This patch was applied to netdev/net-next.git (refs/heads/master):

On Thu, 24 Jun 2021 11:09:14 +0800 you wrote:
parameter dst always points to null.

Signed-off-by: zhang kai <redacted>
---
 net/ipv6/ip6_output.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
Here is the summary with links:
  - ipv6: delete useless dst check in ip6_dst_lookup_tail
    https://git.kernel.org/netdev/net-next/c/c305b9e6d553

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

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help