Skip to content

Conversation

@sylvain-pierrot
Copy link
Collaborator

@sylvain-pierrot sylvain-pierrot commented Sep 3, 2025

  • Implement SO_RCVTIMEO timeout support
  • Replace separate SAP and ADU registries with unified endpoint_ctx structure
  • New workflow: Reception thread is launched during bind operation
  • Simplify netlink communication by removing confirmation/failure callbacks
  • Remove netlink comunication during bp_recvmsg
  • Add thread-safe endpoint lifecycle management with atomic operations
  • Improve bundle reception with continuous polling loop
  • Remove nl_utils.c/h and sap_registry.c/h files
  • Update BP socket protocol to use ENQUEUE_BUNDLE command

…tures (new netlink workflow)

- Replace separate SAP and ADU registries with unified endpoint_ctx structure
- New workflow: Reception thread is launched during bind operation
- Simplify netlink communication by removing confirmation/failure callbacks
- Remove netlink comunication during bp_recvmsg
- Add thread-safe endpoint lifecycle management with atomic operations
- Improve bundle reception with continuous polling loop
- Remove nl_utils.c/h and sap_registry.c/h files
- Update BP socket protocol to use ENQUEUE_BUNDLE command

Signed-off-by: Sylvain Pierrot <[email protected]>
- Fix race condition in send threads by passing node_id/service_id directly
- Ensure proper thread cleanup in ion_close_endpoint with bp_interrupt before join
- Rename thread to recv_thread for consistency
- Fix corrupted node_id/service_id values in logs

Signed-off-by: Sylvain Pierrot <[email protected]>
@sylvain-pierrot sylvain-pierrot changed the title refactor: unified endpoint context management + datagram oriented fea… feat: unified endpoint context management + datagram oriented sockets Sep 3, 2025
@sylvain-pierrot sylvain-pierrot merged commit 6866a40 into main Sep 3, 2025
2 checks passed
@sylvain-pierrot sylvain-pierrot deleted the wip/datagram-oriented branch September 3, 2025 15:09
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.

2 participants