[BUG][cml-008-patch-backport] switch recording format on wov pipeline while speaker playback causes dsp panic #2298

Activity

1. aiChaoSONG

   added

   bugSomething isn't working as expectedSomething isn't working as expected

   on Jan 17, 2020
2. aiChaoSONG

   changed the title [-][BUG][cml-008-patch-backport] switch recording format from s16_le to s24_le on wov pipeline while speaker playback causes dsp panic[/-] [+][BUG][cml-008-patch-backport] switch recording format on wov pipeline while speaker playback causes dsp panic[/+] on Jan 17, 2020
3. lgirdwood commented on Jan 17, 2020

   

   lgirdwood

   on Jan 17, 2020

   Last edited by lgirdwood

   Member

   More actions

   @mrajwa @keyonjie not sure if it's related, but we do get a KPB allocation error

    CORE  LEVEL      COMP_ID                TIMESTAMP            DELTA                FILE_NAME	CONTENT
       0      1          MEM          81683566.718750  81683568.000000      src/lib/alloc.c:273 	error: 155 blocks needed for allocation but only 154 blocks are remaining
       0      2      KEYWORD         147849475.104167  66165908.000000 /audio/detect_test.c:128 	notify_host()
       0      2      KEYWORD         147849519.635417        44.531250 /audio/detect_test.c:105 	notify_kpb(), preamble: 33600
       0      2          KPB         147849526.354167         6.718750      src/audio/kpb.c:792 	kpb_event_handler(): received event with ID: 2 
       0      2          KPB         147849531.562500         5.208333      src/audio/kpb.c:891 	kpb_init_draining(): requested draining of 2100 [ms] from history buffer
       0      2          KPB         147849536.614583         5.052083      src/audio/kpb.c:984 	kpb_init_draining: unlimited draining speed selected.
       0      2          KPB         147849540.937500         4.322917      src/audio/kpb.c:987 	kpb_init_draining(), schedule draining task
       0      2          KPB         147849556.197917        15.260417      src/audio/kpb.c:1044	kpb_draining_task(), start.
       0      2        POWER         147849561.770833         5.572917 avs/lib/pm_runtime.c:81  	pm_runtime_enable_dsp dsp_d0_sref 2
   
4. mrajwa commented on Jan 17, 2020

   

   mrajwa

   on Jan 17, 2020

   Last edited by mrajwa

   Contributor

   More actions

   @lgirdwood, allocation error log is just a misleading warning which really says "there is not enough memory for one of the requested memory pool" and since we don't see consecutive kpb_prepare(0 error message saying sth like "allocation of histor buffer failed" it means that the next memory pool was sufficient to fullfill the request.... I have to go back to #2058 and pick some approach to hide this error message which is in fact a warning.
5. lgirdwood commented on Jan 17, 2020

   

   lgirdwood

   on Jan 17, 2020

   Member

   More actions

   @mrajwa fwiw, @mmaka1 is thinking over changes to future trace APIs
6. mrajwa

   self-assigned this

   on Jan 27, 2020
7. mrajwa commented on Jan 27, 2020

   

   mrajwa

   on Jan 27, 2020

   Last edited by mrajwa

   Contributor

   More actions

   @aiChaoSONG, I have tested cml-008-patch-bakcport as well as master branches and I admit this bug is on both of them. This is very rare case but after debugging it I came up with a fix. Therefore could you please test below patch and check if it fixes the isssue:

   #2329
8. aiChaoSONG commented on Feb 12, 2020

   

   aiChaoSONG

   on Feb 12, 2020

   CollaboratorAuthor

   More actions

   @mrajwa Are you sure #2329 fix this issue？ since #2329 cannot directly applied to 008-backport, I port it myself, but only get ipc error when run wov pipeline. if I am porting it wrong, can you provide the fw you compiled?
9. tlauda

   added

   NOT ON MASTERIssue is not seen on masterIssue is not seen on master

   on Mar 12, 2020
10. mengdonglin commented on Mar 17, 2020

    

    mengdonglin

    on Mar 17, 2020

    Collaborator

    More actions

    @aiChaoSONG @mrajwa Since #2329 is merged by master, customer can get the fix from master later. Closed this issue on cml-008-patch-backport as won't fix.
11. mengdonglin

    closed this as completedon Mar 17, 2020
12. mengdonglin

    added

    CMLApplies to Comet Lake platformApplies to Comet Lake platform

    won't fixThis will not be worked on atm (e.g. a bug closed for lack of user request, hardware etc)This will not be worked on atm (e.g. a bug closed for lack of user request, hardware etc)

    on Mar 17, 2020