[PATCH batadv] batman-adv: bla: only update backbone lasttime on changed claim backbone
Sven Eckelmann <[email protected]> Wed, 29 Jul 2026 23:04:15 +0200
| Newsgroups | org.open-mesh.lists.batman |
|---|---|
| Message-ID | <20260729-bla-lasttime-old-behavior-v1-1-0be875af0456@narfation.org> |
The old behavior before commit d88d7ea0d0be ("batman-adv: bla: avoid CRC
corruption due to parallel claim add") was to only update the
backbone_gw->lasttime when the backbone_gw of the claim changed. But afte
the CRC fix, the backbone_gw->lasttime was updated for each handled claim.
Return back to the old behavior and only update it when the backbone_gw was
changed.
Fixes: d88d7ea0d0be ("batman-adv: bla: avoid CRC corruption due to parallel claim add")
Signed-off-by: Sven Eckelmann <[email protected]>
---
net/batman-adv/bridge_loop_avoidance.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/net/batman-adv/bridge_loop_avoidance.c b/net/batman-adv/bridge_loop_avoidance.c
index f231afbe..275fdf63 100644
--- a/net/batman-adv/bridge_loop_avoidance.c
+++ b/net/batman-adv/bridge_loop_avoidance.c
@@ -766,6 +766,8 @@ static void batadv_bla_add_claim(struct batadv_priv *bat_priv,
spin_lock_bh(&backbone_gw->crc_lock);
backbone_gw->crc ^= claim_crc;
spin_unlock_bh(&backbone_gw->crc_lock);
+
+ WRITE_ONCE(backbone_gw->lasttime, jiffies);
}
if (old_backbone_gw) {
@@ -777,7 +779,6 @@ static void batadv_bla_add_claim(struct batadv_priv *bat_priv,
batadv_backbone_gw_put(old_backbone_gw);
}
- WRITE_ONCE(backbone_gw->lasttime, jiffies);
batadv_claim_put(claim);
}
---
base-commit: 95b6f66b2659f617ce1937e790f48dc578ee014c
change-id: 20260729-bla-lasttime-old-behavior-f9227e6dbd9d
Best regards,
--
Sven Eckelmann <[email protected]>