Skip to content

fix a DSP hang issue - #3140

Closed
keyonjie wants to merge 1 commit into
thesofproject:tgl-007-drop-stablefrom
keyonjie:tgl-007-drop-stable
Closed

keyonjie wants to merge 1 commit into
thesofproject:tgl-007-drop-stablefrom
keyonjie:tgl-007-drop-stable

Conversation

@keyonjie

@keyonjie keyonjie commented Jul 7, 2020

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/ipc/handler.c Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

A similar issue is being solved by #3113 wrt HPRO requests in order to deal with unbalanced requests. There is a bitmap of clients requesting override. Similar thing would help here (dedicated bit for host_pg_override and another one for fw_pg_override). Then multiple unbalanced req/de-req does not cause issues.

@lgirdwood lgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please provide context in the PR title, it also looks like one patch is not related to a hang ?

@keyonjie
keyonjie force-pushed the tgl-007-drop-stable branch from 15f3423 to b6f34d2 Compare July 8, 2020 06:27
We should decrease the ref_cnt only when the DSP is active (dsp_d0_sref
> 0), otherwise, consective PM_GATE IPC asking for enter PM GATE with
different flags will lead to minus ref_cnt and the DSP won't be abled to
activate from the PM_GATE with SOF_PM_PPG flag.

Signed-off-by: Keyon Jie <yang.jie@linux.intel.com>
@jajanusz
jajanusz force-pushed the tgl-007-drop-stable branch 3 times, most recently from be6c347 to fb801c8 Compare July 14, 2020 20:06
@keyonjie

Copy link
Copy Markdown
Contributor Author

TGL-007 release was done so close this.

@keyonjie keyonjie closed this Jul 24, 2020
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