all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH frr] fix regression with evpn MH introduce in 7.5
@ 2021-08-10  7:45 Alexandre Derumier
  0 siblings, 0 replies; only message in thread
From: Alexandre Derumier @ 2021-08-10  7:45 UTC (permalink / raw)
  To: pve-devel

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
---
 Makefile                                      |  2 +-
 debian/changelog                              |  6 ++++
 .../pve/0003-bugfix-local_inactive.patch      | 33 +++++++++++++++++++
 debian/patches/series                         |  1 +
 4 files changed, 41 insertions(+), 1 deletion(-)
 create mode 100644 debian/patches/pve/0003-bugfix-local_inactive.patch

diff --git a/Makefile b/Makefile
index 39a5d10..7c875d0 100644
--- a/Makefile
+++ b/Makefile
@@ -1,6 +1,6 @@
 PACKAGE=frr
 VER=7.5.1
-PKGREL=1+pve
+PKGREL=2+pve
 
 SRCDIR=frr
 BUILDDIR=${SRCDIR}.tmp
diff --git a/debian/changelog b/debian/changelog
index 244f82b..9b5a912 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+frr (7.5.1-2+pve) bullseye; urgency=medium
+
+  * Fix evpn MH local_inactive bug
+
+ -- Proxmox Support Team <support@proxmox.com>  Tue, 10 Aug 2021 09:35:20 +0200
+
 frr (7.5.1-1+pve) bullseye; urgency=medium
 
   * update submodule to latest 7.5.1 stable release
diff --git a/debian/patches/pve/0003-bugfix-local_inactive.patch b/debian/patches/pve/0003-bugfix-local_inactive.patch
new file mode 100644
index 0000000..8c0fe63
--- /dev/null
+++ b/debian/patches/pve/0003-bugfix-local_inactive.patch
@@ -0,0 +1,33 @@
+From 1e564145bb43c31dae7106c86a9b52a138fbfa86 Mon Sep 17 00:00:00 2001
+From: Alexandre Derumier <aderumier@odiso.com>
+Date: Tue, 10 Aug 2021 09:00:54 +0200
+Subject: [PATCH 2/2] bugfix: fix evpn MH local_inactive
+
+frr 7.5 is missing this patch
+https://github.com/FRRouting/frr/commit/c7bfd085680bf94ea5dbdccc875f7e0257a9a9c8.patch
+
+it's difficult to backport, but we don't use evpn MH, so until it's fixed,
+we can disable local_inactive for now
+
+https://github.com/FRRouting/frr/issues/9347
+
+---
+ zebra/rt_netlink.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/zebra/rt_netlink.c b/zebra/rt_netlink.c
+index 770feb52b..a7c9412a3 100644
+--- a/zebra/rt_netlink.c
++++ b/zebra/rt_netlink.c
+@@ -3370,7 +3370,7 @@ static int netlink_ipneigh_change(struct nlmsghdr *h, int len, ns_id_t ns_id)
+ 		 * in re-adding the neighbor if it is a valid "remote" neighbor.
+ 		 */
+ 		if (ndm->ndm_state & NUD_VALID) {
+-			local_inactive = !(ndm->ndm_state & NUD_LOCAL_ACTIVE);
++			local_inactive = false;
+ 
+ 			/* XXX - populate dp-static based on the sync flags
+ 			 * in the kernel
+-- 
+2.30.2
+
diff --git a/debian/patches/series b/debian/patches/series
index 50b22cc..64c0303 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,2 +1,3 @@
 pve/0001-enable-bgp-daemon.patch
 pve/0002-bgpd-add-an-option-for-RT-auto-derivation-to-force-A.patch
+pve/0003-bugfix-local_inactive.patch
\ No newline at end of file
-- 
2.30.2




^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-08-10  7:45 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-10  7:45 [pve-devel] [PATCH frr] fix regression with evpn MH introduce in 7.5 Alexandre Derumier

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal