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.

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

Support non-directed address generation offload #381

Merged
merged 1 commit into from
Jan 28, 2024
Merged

Conversation

zxzxwu
Copy link
Collaborator

@zxzxwu zxzxwu commented Dec 24, 2023

Address generation offloading uses peer address as an entry of IRK, even in non-directed usage. By adding an empty entry containing local IRK to controller resolving list, controllers are able to generate RPA without pairing.

@zxzxwu
Copy link
Collaborator Author

zxzxwu commented Dec 25, 2023

@benquike @uael @hchataing It seems rootcanal doesn't handle address resolution offloading properly?

@zxzxwu
Copy link
Collaborator Author

zxzxwu commented Dec 26, 2023

Maybe the rootcanal is fine, but Bumble tries to add resolving list when there is active connection...

@zxzxwu
Copy link
Collaborator Author

zxzxwu commented Dec 26, 2023

root-canal D 12-26 15:45:46.731 link_layer_controller.cc:3221         7  Legacy advertising ignored by initiator because the advertising address da:4c:10:de:17:01[PUBLIC_IDENTITY_ADDRESS] does not match the peer address da:4c:10:de:17:01[PUBLIC_DEVICE_ADDRESS]

@zxzxwu zxzxwu closed this Dec 26, 2023
@zxzxwu zxzxwu reopened this Dec 26, 2023
@zxzxwu zxzxwu force-pushed the offload branch 2 times, most recently from e8335b0 to fffe24d Compare December 30, 2023 09:07
@zxzxwu zxzxwu closed this Jan 23, 2024
@zxzxwu zxzxwu reopened this Jan 23, 2024
@zxzxwu zxzxwu requested a review from barbibulle January 25, 2024 06:41
@zxzxwu zxzxwu marked this pull request as ready for review January 25, 2024 06:42
@hchataing
Copy link
Collaborator

I just updated v1.6.0 of RootCanal to pypi.

@zxzxwu zxzxwu force-pushed the offload branch 3 times, most recently from 136b131 to 527bc32 Compare January 26, 2024 07:53
@zxzxwu zxzxwu merged commit ef4ea86 into google:main Jan 28, 2024
51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants