Skip to content

asn1: add rtcmem wrapper#279

Closed
buraksenkus wants to merge 1 commit intoriebl:masterfrom
VeNIT-Lab:rtcmem
Closed

asn1: add rtcmem wrapper#279
buraksenkus wants to merge 1 commit intoriebl:masterfrom
VeNIT-Lab:rtcmem

Conversation

@buraksenkus
Copy link
Copy Markdown
Contributor

I've added missing asn1c wrapper for RTCMEM

@riebl
Copy link
Copy Markdown
Owner

riebl commented Mar 25, 2026

Hi @buraksenkus,

I will add the RTCMEM wrapper. However, I am not a fan of adding the recursive mutexes because they will affect everyone including those running Vanetza single-threaded.
My usual pattern when integrating Vanetza in a multi-threaded application: All Vanetza entities (including security, DCC, access interface etc.) are running on a single thread and thread-safe queues are used for sharing incoming/outgoing packets with other application code potentially running on other threads.
If you have an idea for another (optional) synchronization mechanism, please create a separate pull request. Thanks!

@buraksenkus
Copy link
Copy Markdown
Contributor Author

Actually those commits shouldn't be in this branch, they were added by mistake. We're working on multi-thread operations in another branch and they are not related with this PR. Sorry for the mistake. I'll fix the PR

@sonarqubecloud
Copy link
Copy Markdown

@buraksenkus
Copy link
Copy Markdown
Contributor Author

Now this is the intended PR :)

@riebl
Copy link
Copy Markdown
Owner

riebl commented Mar 30, 2026

Merged as 361409c

@riebl riebl closed this Mar 30, 2026
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