Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Debug stream thread info fixe. Should be backported for 2.11 #9451

Merged
merged 2 commits into from
Sep 11, 2024

Conversation

jsarha
Copy link
Contributor

@jsarha jsarha commented Sep 9, 2024

Couple of loose ends that were left from hasty merging to 2.11.

Jyri Sarha added 2 commits September 9, 2024 22:42
…mments

The cache-line alignment was removed from circular buffer descriptors
on last moments of debug_stream_slot PR's review process. This broke
the circular buffer alignment and its now fixed in this PR. The commit
also contains the documentation update for the alignment removal change
that was also forgotten from the original PR.

Since this only affects the offsets of the circular buffers, that are
explicitly written in the debug window slot, there is no need to
update anything on the host side client. Also the unaligned circular
buffers appeared to work just fine, because of that.

Signed-off-by: Jyri Sarha <jyri.sarha@linux.intel.com>
Signed-off-by: Jyri Sarha <jyri.sarha@linux.intel.com>
@cujomalainey cujomalainey added the backport-to-stable PRs that should be backported to stable branches label Sep 9, 2024
@kv2019i
Copy link
Collaborator

kv2019i commented Sep 10, 2024

@jsarha v2.11 hasn't branched yet, so we have still time to merge this to main and have this in 2.11. Current release status #9360

@lgirdwood
Copy link
Member

A lot of timeouts on one of the DUTS. Rerun.

@lgirdwood
Copy link
Member

SOFCI TEST

@kv2019i kv2019i merged commit 7dd2d7f into thesofproject:main Sep 11, 2024
43 of 47 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-stable PRs that should be backported to stable branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants