Skip to content

Middleware concurrency interface changes#462

Merged
rolandreichweinbmw merged 2 commits into
eclipse-openbsw:mainfrom
lcastela:ApplyMiddlewareConcurrencyPatch
May 27, 2026
Merged

Middleware concurrency interface changes#462
rolandreichweinbmw merged 2 commits into
eclipse-openbsw:mainfrom
lcastela:ApplyMiddlewareConcurrencyPatch

Conversation

@lcastela
Copy link
Copy Markdown
Contributor

No description provided.

@rolandreichweinbmw rolandreichweinbmw marked this pull request as draft May 26, 2026 08:05
@lcastela lcastela force-pushed the ApplyMiddlewareConcurrencyPatch branch from bbe6cbc to ff224a7 Compare May 26, 2026 08:11
@lcastela lcastela marked this pull request as ready for review May 26, 2026 08:19
@rolandreichweinbmw
Copy link
Copy Markdown
Contributor

Content-wise, looks fine to me. Can you please craft the commit message according to
https://github.com/eclipse-openbsw/openbsw/blob/main/doc/dev/guidelines/commit_message.rst
?

@lcastela lcastela changed the title update middleware concurrency integration Middleware concurrency interface changes May 26, 2026
@lcastela lcastela force-pushed the ApplyMiddlewareConcurrencyPatch branch 2 times, most recently from 07cc8fb to e0b3a0d Compare May 26, 2026 15:59
lcastela added 2 commits May 27, 2026 10:57
Move concrete lock type definitions out of LockStrategies.h and into
a separate lock_types.h header. This allows platform integrations to
provide ScopedCoreLock and ScopedECULock at build time.
The call to `suspendAllInterrupts()` in `ProxyBase` and `SkeletonBase`
was redundant. After logging the cross-thread violation, both call
sites unconditionally hit an assert, which is not recoverable.
@lcastela lcastela force-pushed the ApplyMiddlewareConcurrencyPatch branch from 143b221 to 1794d86 Compare May 27, 2026 09:57
@lcastela lcastela marked this pull request as ready for review May 27, 2026 10:06
@rolandreichweinbmw rolandreichweinbmw merged commit 00a8a02 into eclipse-openbsw:main May 27, 2026
127 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.

2 participants