bird2: Fix bus error on OSPF on IPQ806X 713/head
authorMatt Reeve <matt@mreeve.com>
Tue, 10 Aug 2021 11:41:04 +0000 (12:41 +0100)
committerMatt Reeve <matt@mreeve.com>
Thu, 12 Aug 2021 10:42:37 +0000 (11:42 +0100)
On a Netgear R7800, if ospf v2 or v3 is configured in bird.conf, it fails to start with this error:

Fri Jun 11 14:41:11 2021 daemon.info bird: Started
Fri Jun 11 14:41:11 2021 kern.err kernel: [ 3500.853248] Alignment trap: not handling instruction f44c0a1f at [<00035848>] Fri Jun 11 14:41:11 2021 kern.alert kernel: [ 3500.853283] 8<--- cut here ---
Fri Jun 11 14:41:11 2021 kern.alert kernel: [ 3500.859363] Unhandled fault: alignment exception (0x801) at 0x007e0624
Fri Jun 11 14:41:11 2021 kern.alert kernel: [ 3500.862443] pgd = 0bbef4fd
Fri Jun 11 14:41:11 2021 kern.alert kernel: [ 3500.868821] [007e0624] *pgd=5d6ca835, *pte=5c40b75f, *ppte=5c40bc7f

The problem is due to a struct not being properly aligned on the ARMv7 architecture.

This patch fixes the problem by adding the "PACKED" macro to the affected struct. Note
that upstream may later fix this in another way, in which case this patch will not be required.

Signed-off-by: Matt Reeve <matt@mreeve.com>
(cherry picked from commit 166b6ea77b30f82eb880ce43a9e9ff343b79fe2e)

bird2/Makefile
bird2/patches/000-ospf-bus-error.patch [new file with mode: 0644]

index cdc6ec1237e65d0d6d80e6390118cc13de1ce6d0..9fee377639e9ae34c3da4087498761851fa293e4 100644 (file)
@@ -8,7 +8,7 @@ include $(TOPDIR)/rules.mk
 
 PKG_NAME:=bird2
 PKG_VERSION:=2.0.8
-PKG_RELEASE:=1
+PKG_RELEASE:=2
 
 PKG_SOURCE:=bird-$(PKG_VERSION).tar.gz
 PKG_SOURCE_URL:=ftp://bird.network.cz/pub/bird
diff --git a/bird2/patches/000-ospf-bus-error.patch b/bird2/patches/000-ospf-bus-error.patch
new file mode 100644 (file)
index 0000000..085965e
--- /dev/null
@@ -0,0 +1,11 @@
+--- a/proto/ospf/topology.h
++++ b/proto/ospf/topology.h
+@@ -41,7 +41,7 @@ struct top_hash_entry
+   u8 mode;                    /* LSA generated during RT calculation (LSA_RTCALC or LSA_STALE)*/
+   u8 nhs_reuse;                       /* Whether nhs nodes can be reused during merging.
+                                  See a note in rt.c:add_cand() */
+-};
++} PACKED;
+ /* Prevents ospf_hash_find() to ignore the entry, for p->lsrqh and p->lsrth */