From: Alexandre Derumier <aderumier@odiso.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH frr] fix regression with evpn MH introduce in 7.5
Date: Tue, 10 Aug 2021 09:45:03 +0200 [thread overview]
Message-ID: <20210810074503.392030-1-aderumier@odiso.com> (raw)
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
reply other threads:[~2021-08-10 7:45 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210810074503.392030-1-aderumier@odiso.com \
--to=aderumier@odiso.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox