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

Reply via email to