[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]>