Re: Backporting *ip: Treat IPv4 segment's lowest address as unicast* to Linux 5.10.y?

[Seth David Schoen <schoen@loyalty.org>]

Paul Menzel writes:

> Dear Linux folks,
> 
> 
> Seth backported commit 94c821c74bf5fe0c25e09df5334a16f98608db90 in OpenWrt
> [1]. Could we also add to the Linux LTS 5.10 series?

Thanks for suggesting this.  This would be something like the attached
version (which is against the 5.10 stable tree), perhaps with different
naming/commit message documentations for backporting.

I understand if this turns out to be too much like a functionality
change rather than a bugfix for 5.10; in that case, we could just
continue making it available on our own repo.


From 323c87d4c2de7598ac810632450816732056b111 Mon Sep 17 00:00:00 2001
From: Seth Schoen <schoen@loyalty.org>
Date: Sun, 22 Jan 2023 14:39:24 -0800
Subject: [PATCH] Backport 5.14: lowest-address is not broadcast

Allow the lowest address in a network segment to be used as an
ordinary unicast address, not a duplicate broadcast address.

Signed-off-by: Seth David Schoen <schoen@loyalty.org>
Suggested-by: John Gilmore <gnu@toad.com>
---
 net/ipv4/fib_frontend.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/net/ipv4/fib_frontend.c b/net/ipv4/fib_frontend.c
index 5f786ef662ea..0b3fadc002d6 100644
--- a/net/ipv4/fib_frontend.c
+++ b/net/ipv4/fib_frontend.c
@@ -1129,10 +1129,8 @@ void fib_add_ifaddr(struct in_ifaddr *ifa)
 				  prefix, ifa->ifa_prefixlen, prim,
 				  ifa->ifa_rt_priority);
 
-		/* Add network specific broadcasts, when it takes a sense */
+		/* Add the network broadcast address, when it makes sense */
 		if (ifa->ifa_prefixlen < 31) {
-			fib_magic(RTM_NEWROUTE, RTN_BROADCAST, prefix, 32,
-				  prim, 0);
 			fib_magic(RTM_NEWROUTE, RTN_BROADCAST, prefix | ~mask,
 				  32, prim, 0);
 			arp_invalidate(dev, prefix | ~mask, false);
-- 
2.25.1