Skip to content

Conversation

ckhardin
Copy link
Contributor

Use this PR to test - but, the original PR should be used for reference #89624 and merged when that works as desired.

@ckhardin ckhardin force-pushed the ckhardin/event-object-phantoms branch 6 times, most recently from b1b32f5 to dcfc6ed Compare August 30, 2025 22:28
This is variation of the PR to handle phantom events and hopefully
this get merged into the PR to land.

See-also: zephyrproject-rtos#89624
Signed-off-by: Charles Hardin <ckhardin@gmail.com>
@ckhardin ckhardin force-pushed the ckhardin/event-object-phantoms branch from dcfc6ed to 2f02eec Compare August 31, 2025 22:36
Copy link

Copy link
Contributor

@peter-mitsis peter-mitsis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nicely done! :)

@ckhardin
Copy link
Contributor Author

ckhardin commented Sep 4, 2025

@cfriedt - you are the original author - we've been using this commit (well functional part of it) for a bit now and so just let me know if you want to take over this PR and I can close out or if folks want to proceed with this along with any cleanup needed. The only thing i modified was the synchronization domain to "getting on the waitq and events" and "on the waitq and events" - once we are off the waitq we cann't touch the events because that is in the past and we just have to consume everything prior to that barrier.

I switched our team to this commit to just make sure we didn't miss something and everything seems to be working as expected.

Copy link
Member

@cfriedt cfriedt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thanks for picking this up 👍

@cfriedt cfriedt merged commit 81283c6 into zephyrproject-rtos:main Sep 5, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants