Skip to content

[TO BE DROPPED] pipeline: limit the effect of a recent CPC work-around #9304

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.

Sign up for GitHub

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

Jump to bottom
Closed
lyakh wants to merge 1 commit into from
Closed

[TO BE DROPPED] pipeline: limit the effect of a recent CPC work-around #9304

lyakh wants to merge 1 commit into from

Conversation

@lyakh
Copy link
Collaborator

@lyakh lyakh commented Jul 16, 2024

Limit the scope of a recent dynamic KCPS change to only cases with 0 module CPC.

@lyakh
pipeline: limit the effect of a recent CPC work-around
5964dab 
Limit the scope of a recent dynamic KCPS change to only cases with 0
module CPC.

Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
@marc-hb
Copy link
Collaborator

marc-hb commented Jul 18, 2024

IPC timeout in https://sof-ci.01.org/sofpr/PR9304/build6537/devicetest/index.html?model=MTLP_SDW_AIOC&testcase=multiple-pipeline-all

Re-running

@marc-hb
Copy link
Collaborator

marc-hb commented Jul 18, 2024

SOFCI TEST

@marc-hb
Copy link
Collaborator

marc-hb commented Jul 18, 2024 •
edited
Loading

Same IPC timeout again in MTL https://sof-ci.01.org/sofpr/PR9304/build6580/devicetest/index.html?model=MTLP_SDW_AIOC&testcase=multiple-pipeline-all so this PR really has a problem.

@lyakh lyakh marked this pull request as draft August 13, 2024 15:32
@lyakh
Copy link
Collaborator Author

lyakh commented Aug 13, 2024 •
edited
Loading

#9319 should be merged instead

@lgirdwood
Copy link
Member

lgirdwood commented Aug 14, 2024

#9319 should be merged instead

close this one ?

@lyakh lyakh changed the title pipeline: limit the effect of a recent CPC work-around [TO BE DROPPED] pipeline: limit the effect of a recent CPC work-around Aug 15, 2024
@lyakh
Copy link
Collaborator Author

lyakh commented Aug 15, 2024

#9319 should be merged instead

close this one ?

Yes, I've converted it to a draft and marked "to be dropped" to wait until #9319 is merged

@lyakh
Copy link
Collaborator Author

lyakh commented Aug 30, 2024

#9319 merged

@lyakh lyakh closed this Aug 30, 2024
@lyakh lyakh deleted the smatest-mtl branch August 30, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lgirdwood lgirdwood Awaiting requested review from lgirdwood lgirdwood is a code owner

@plbossart plbossart Awaiting requested review from plbossart plbossart is a code owner

@mmaka1 mmaka1 Awaiting requested review from mmaka1 mmaka1 is a code owner

@lbetlej lbetlej Awaiting requested review from lbetlej lbetlej is a code owner

@dbaluta dbaluta Awaiting requested review from dbaluta dbaluta is a code owner

@kv2019i kv2019i Awaiting requested review from kv2019i kv2019i is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lyakh @marc-hb @lgirdwood