[border-router] rename RxRaTracker callback for clarity (#12019)

Renames the callback used by `RxRaTracker` to inform `RoutingManager`
to `HandleRxRaTrackerDecisionFactorChanged()`.

This name more accurately reflects the triggering condition, as the
callback is invoked whenever any of the "decision factors" change,
not just when the on-link prefix table is updated.

For consistency, the related method in `OnLinkPrefixManager` is also
renamed to `HandleRxRaTrackerChanged()`.
This commit is contained in:
Abtin Keshavarzian
2025-10-10 12:56:54 -07:00
committed by GitHub
parent 989727feae
commit e07ed85620
3 changed files with 8 additions and 7 deletions
+5 -4
View File
@@ -723,14 +723,15 @@ exit:
return;
}
void RoutingManager::HandleRaPrefixTableChanged(void)
void RoutingManager::HandleRxRaTrackerDecisionFactorChanged(void)
{
// This is a callback from `RxRaTracker` indicating that
// there has been a change in the table.
// there has been a change impacting one of the decision
// factors.
VerifyOrExit(mIsRunning);
mOnLinkPrefixManager.HandleRaPrefixTableChanged();
mOnLinkPrefixManager.HandleRxRaTrackerChanged();
mRoutePublisher.Evaluate();
#if OPENTHREAD_CONFIG_BORDER_ROUTING_MULTI_AIL_DETECTION_ENABLE
mMultiAilDetector.Evaluate();
@@ -1578,7 +1579,7 @@ bool RoutingManager::OnLinkPrefixManager::IsInitalEvaluationDone(void) const
return (mAilPrefix.GetLength() != 0 || IsPublishingOrAdvertising());
}
void RoutingManager::OnLinkPrefixManager::HandleRaPrefixTableChanged(void)
void RoutingManager::OnLinkPrefixManager::HandleRxRaTrackerChanged(void)
{
// This is a callback from `RxRaTracker` indicating that
// there has been a change in the table. If the favored on-link
+2 -2
View File
@@ -767,7 +767,7 @@ private:
const Ip6::Prefix &GetFavoredPrefix(void) const { return mFavoredPrefix; }
bool AddressMatchesLocalPrefix(const Ip6::Address &aAddress) const;
bool IsInitalEvaluationDone(void) const;
void HandleRaPrefixTableChanged(void);
void HandleRxRaTrackerChanged(void);
bool ShouldPublishUlaRoute(void) const;
Error AppendAsPiosTo(RouterAdvert::TxMessage &aRaMessage);
void HandleNetDataChange(void);
@@ -1157,7 +1157,7 @@ private:
void HandleNeighborAdvertisement(const InfraIf::Icmp6Packet &aPacket);
bool NetworkDataContainsUlaRoute(void) const;
void HandleRaPrefixTableChanged(void);
void HandleRxRaTrackerDecisionFactorChanged(void);
void HandleLocalOnLinkPrefixChanged(void);
static bool IsValidBrUlaPrefix(const Ip6::Prefix &aBrUlaPrefix);
+1 -1
View File
@@ -812,7 +812,7 @@ exit:
void RxRaTracker::HandleExpirationTimer(void) { Evaluate(); }
void RxRaTracker::HandleSignalTask(void) { Get<RoutingManager>().HandleRaPrefixTableChanged(); }
void RxRaTracker::HandleSignalTask(void) { Get<RoutingManager>().HandleRxRaTrackerDecisionFactorChanged(); }
void RxRaTracker::HandleRdnssAddrTask(void) { mRdnssCallback.InvokeIfSet(); }