[go: nahoru, domu]
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[routing-manager] indicate peer BRs in discovered infra-if routers #10448

Merged
merged 1 commit into from
Jul 10, 2024
Merged

[routing-manager] indicate peer BRs in discovered infra-if routers #10448

merged 1 commit into from
Jul 10, 2024

Conversation

abtink
Copy link
Member
@abtink abtink commented Jun 27, 2024 •
edited
Loading

This commit updates the public API otBorderRoutingGetNextRouterEntry and otBorderRoutingRouterEntry structure to indicate whether a discovered router on an infrastructure link is likely a peer Thread Border Router (BR) connected to the same Thread mesh. The related CLI commands are also updated. Additionally, this commit adds new tests to validate the discovery and tracking of peer BRs.

This PR currently contains the commit from #10445. Please check and review the last commit. Thanks.

This PR is rebased on #10453. Please check and review the last commit. Thanks.

@size-report size-report
Copy link
size-report bot commented Jun 27, 2024 •
edited
Loading

Size Report of OpenThread

Merging #10448 into main(32fe4c7).

name branch text data bss total
ot-cli-ftd main 466656 856 66364 533876
#10448 466656 856 66364 533876
+/- 0 0 0 0
ot-ncp-ftd main 435860 760 61592 498212
#10448 435860 760 61592 498212
+/- 0 0 0 0
libopenthread-ftd.a main 236002 95 40302 276399
#10448 236002 95 40302 276399
+/- 0 0 0 0
libopenthread-cli-ftd.a main 57524 0 8075 65599
#10448 57524 0 8075 65599
+/- 0 0 0 0
libopenthread-ncp-ftd.a main 32127 0 5924 38051
#10448 32127 0 5924 38051
+/- 0 0 0 0
ot-cli-mtd main 365280 760 51236 417276
#10448 365280 760 51236 417276
+/- 0 0 0 0
ot-ncp-mtd main 348108 760 46480 395348
#10448 348108 760 46480 395348
+/- 0 0 0 0
libopenthread-mtd.a main 158731 0 25190 183921
#10448 158731 0 25190 183921
+/- 0 0 0 0
libopenthread-cli-mtd.a main 39739 0 8059 47798
#10448 39739 0 8059 47798
+/- 0 0 0 0
libopenthread-ncp-mtd.a main 25007 0 5924 30931
#10448 25007 0 5924 30931
+/- 0 0 0 0
ot-cli-ftd-br main 550568 864 131228 682660
#10448 550632 864 131228 682724
+/- +64 0 0 +64
libopenthread-ftd-br.a main 324283 100 105142 429525
#10448 324323 100 105142 429565
+/- +40 0 0 +40
libopenthread-cli-ftd-br.a main 71437 0 8099 79536
#10448 71468 0 8099 79567
+/- +31 0 0 +31
ot-rcp main 62312 564 20636 83512
#10448 62312 564 20636 83512
+/- 0 0 0 0
libopenthread-rcp.a main 9734 0 5060 14794
#10448 9734 0 5060 14794
+/- 0 0 0 0
libopenthread-radio.a main 18991 0 222 19213
#10448 18991 0 222 19213
+/- 0 0 0 0

@abtink abtink marked this pull request as ready for review July 1, 2024 19:43
@abtink abtink mentioned this pull request Jul 1, 2024
Open
@jwhui jwhui force-pushed the rm/api-tbr-track branch 2 times, most recently from 39033fc to 006cb01 Compare July 3, 2024 19:57
@jwhui jwhui requested a review from Vyrastas July 3, 2024 22:52
@abtink abtink mentioned this pull request Jul 8, 2024
Merged
Vyrastas
Vyrastas requested changes Jul 8, 2024
View reviewed changes
src/cli/README_BR.md Outdated Show resolved Hide resolved
src/cli/cli_br.cpp Outdated Show resolved Hide resolved
include/openthread/border_routing.h Outdated Show resolved Hide resolved
Vyrastas
Vyrastas reviewed Jul 8, 2024
View reviewed changes
src/cli/cli_br.cpp Outdated Show resolved Hide resolved
@jwhui jwhui requested a review from Vyrastas July 8, 2024 22:01
@abtink abtink force-pushed the rm/api-tbr-track branch 3 times, most recently from e9ab741 to 9bee6a4 Compare July 9, 2024 17:24
@Vyrastas Vyrastas requested a review from jwhui July 9, 2024 19:16
@abtink
[routing-manager] indicate peer BRs in discovered infra-if routers
2ddd790 
This commit updates the public API `otBorderRoutingGetNextRouterEntry`
and `otBorderRoutingRouterEntry` structure to indicate whether a
discovered router on an infrastructure link is likely a peer Thread
Border Router (BR) connected to the same Thread mesh. The related CLI
commands are also updated.

Additionally, this commit adds new tests to validate the discovery and
tracking of peer BRs.
@jwhui jwhui merged commit 4c84e4d into openthread:main Jul 10, 2024
103 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Vyrastas Vyrastas Vyrastas approved these changes

@jwhui jwhui jwhui approved these changes

Assignees
No one assigned
Labels
api: user comp: border router comp: cli enhancement P4
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@abtink @Vyrastas @jwhui