Skip to content

Conversation

@JDuchniewicz
Copy link

@JDuchniewicz JDuchniewicz commented Dec 18, 2025

Looking at

sel4utils_create_word_args(producer_args_strings, producer_argv, N_PRODUCER_CONSUMER_ARGS,
and
SYNC_PRODUCER_CONSUMER_FUNC_EP(consumer_func_ep, (*fifo_head == 0), sync_cv_wait, (*fifo_head)--, consumer_to_producer)

The results are accidentally swapped for the EP version of benchmark. This PR fixes that.

The results before the change (run for odroid_c2):

Producer to consumer: 1328(137)
Producer to consumer (early processing): 1302(7)
Consumer to producer: 1310(9)
Consumer to producer (early processing): 1322(136)

One can clearly see that the early processing labels are swapped.
We still have a large spike from the last sample in the Producer to consumer case but I don't really know how to fix it easily.

JDuchniewicz and others added 2 commits December 18, 2025 15:02
Copy link
Member

@lsf37 lsf37 left a comment

Choose a reason for hiding this comment

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

Thanks for catching that. Looks good from my side.

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