Skip to content

Conversation

@jsarha
Copy link
Contributor

@jsarha jsarha commented Feb 28, 2023

This was not the correct fix for the ll timer free crash issue. Now that the correct fix[1] is found and merged, this workaround can be removed.

[1] b3a808a "chain-dma: fix scheduling exception"

This reverts commit 60e9e97.

This should not be merged before #7138 is in.

@jsarha jsarha marked this pull request as ready for review March 2, 2023 13:13
@jsarha jsarha requested a review from makarukp as a code owner March 2, 2023 13:13
@lyakh
Copy link
Collaborator

lyakh commented Mar 2, 2023

SOFCI TEST

@jsarha jsarha force-pushed the revert_chain_dma_cancel_fix branch from fe521d1 to cebd788 Compare March 2, 2023 13:16
This was not the correct fix for the ll timer free crash issue. Now
that the correct fix[1] is found and merged, this workaround can be
removed.

[1] b3a808a "chain-dma: fix scheduling exception"

This reverts commit 60e9e97.

Signed-off-by: Jyri Sarha <jyri.sarha@intel.com>
@jsarha
Copy link
Contributor Author

jsarha commented Mar 2, 2023

The CI was successful: https://sof-ci.01.org/sofpr/PR7189/build4252/devicetest/index.html

I'll fix the checkpatch issue (re-wrap the commit message).

@jsarha jsarha force-pushed the revert_chain_dma_cancel_fix branch from cebd788 to 769e588 Compare March 2, 2023 17:50
@jsarha
Copy link
Contributor Author

jsarha commented Mar 2, 2023

Only the commit message was changed.

@lgirdwood
Copy link
Member

SOFCI TEST

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.

6 participants