From f4251ccf69ad70b46c060be62eb39ebc36c47ac6 Mon Sep 17 00:00:00 2001 From: Matt Reeve Date: Tue, 10 Aug 2021 12:41:04 +0100 Subject: [PATCH] bird2: Fix bus error on OSPF on IPQ806X 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 (cherry picked from commit 166b6ea77b30f82eb880ce43a9e9ff343b79fe2e) --- bird2/Makefile | 2 +- bird2/patches/000-ospf-bus-error.patch | 11 +++++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) create mode 100644 bird2/patches/000-ospf-bus-error.patch diff --git a/bird2/Makefile b/bird2/Makefile index cdc6ec1..9fee377 100644 --- a/bird2/Makefile +++ b/bird2/Makefile @@ -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 index 0000000..085965e --- /dev/null +++ b/bird2/patches/000-ospf-bus-error.patch @@ -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 */ -- 2.30.2