Skip to content

[BUG] TriggerStart IPC timeout in bit depth conversion stress test. #2213

Closed

Description

@jdrzaszcz

Describe the bug
After creating topology with playback and capture pipelines with bit depth conversion on DAI, when creating and deleting PcmStreams multiple times (topology is not destroyed), when sending IPC TrigStart to playback pipelines FW doesn't reply.

Topology

pipe_plb
+-------------------------------+
| +------+   +------+   +-----+ |  +-----+
| | Host |-->|Buffer|-->| Dai |--->| SSP |
| +------+   +------+   +-----+ |  +-----+
+-------------------------------+    | |
                                     | |
pipe_cap                             | |
+-------------------------------+    | |
| +------+   +------+   +-----+ |  +-----+
| | Host |<--|Buffer|<--| Dai |<---| SSP |
| +------+   +------+   +-----+ |  +-----+
+-------------------------------+

To Reproduce
Slim Driver python test:
On APL, WHL, ICL:
test_13_playback_bitdepth_conversion with --playback_iterations>=1000

Example:
02_13_TestPtpPlaybackBitDepthConversion16000Hz16b16b4ch
02_13_TestPtpPlaybackBitDepthConversion48000Hz32b32b4ch

Reproduction Rate
20 %

Environment
Sof master branch 62dbfcf commit

Example log from APL:

2019_12_16_12_16_50.zip
2019_12_16_11_20_23.zip
2019_12_16_11_48_15.zip

Activity

  1. added
    bugSomething isn't working as expected
    APLApplies to Apollolake platform
    WHLApplies to WhiskeyLake platform
    ICLApplies to IceLake platform
    P2Critical bugs or normal features
    on Dec 16, 2019
  2. slawblauciak commented on Jan 27, 2020

    @slawblauciak
    Collaborator

    Does this still occur on master?

  3. added this to the v1.5 milestone on Jan 27, 2020
  4. jdrzaszcz commented on Jan 28, 2020

    @jdrzaszcz
    Author

    Yes, still occur, but the reproduction rate is lower and playback_iterations has to be over 1000 for all platforms.
    I checked it with 0e83f7d commit.

  5. lgirdwood commented on Jan 28, 2020

    @lgirdwood
    Member

    @slawblauciak @jdrzaszcz fwiw, @mrajwa made an interesting observation #2329 that may impact this use case too.

  6. lgirdwood commented on Feb 21, 2020

    @lgirdwood
    Member

    @jdrzaszcz any update, are we still seeing this ? @tlauda it looks from the tail of the last log that the IPC logic get in a muddle prior to the timeout ?

    �[0m    0      2          SSP          33416300.677083         3.489583 src/drivers/intel/cavs/ssp.c:877 	ssp_trigger() cmd 1�[0m
    �[0m    0      2          SSP          33416304.791667         4.114583 src/drivers/intel/cavs/ssp.c:806 	ssp_start()�[0m
    �[0m    0      2      unknown          33416355.156250        50.364582 src/schedule/timer_domain.c:47  	timer_domain_register()�[0m
    �[31m    0      1           SA          33417142.343750       787.187500      src/lib/agent.c:55  	validate(), ll drift detected, delta = 20277�[0m
    �[31m    0      1          IPC          34046050.677083    628908.312500    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34088050.625000     41999.949219    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34130050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34172050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34256050.625000     84000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34298050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34340050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34382050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34466050.625000     84000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34508050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34550050.729167     42000.105469    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34634050.625000     83999.898438    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34676050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34718050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34802050.677083     84000.054688    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34844050.625000     41999.949219    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34886050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34928050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          34995142.708333     67092.085938    src/ipc/handler.c:1275	ipc: msg hdr for 0x90020000 not found replace 1�[0m
    �[31m    0      1          IPC          35012050.625000     16907.916016    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35054050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35096050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35138050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35180050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35264050.625000     84000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35306050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35348050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35432050.625000     84000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35474050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35511106.822917     37056.199219    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0004 not found replace 0�[0m
    �[31m    0      1          IPC          35516050.572917      4943.750000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35558050.625000     42000.050781    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35642050.625000     84000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35684050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35726050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35768050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35852050.625000     84000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35894050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35936050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          35978050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          36015106.614583     37055.988281    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0004 not found replace 0�[0m
    �[31m    0      1          IPC          36062050.625000     46944.011719    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          36104050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          36146050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          36188050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          36272050.625000     84000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          36314050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          36356050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[31m    0      1          IPC          36398050.625000     42000.000000    src/ipc/handler.c:1275	ipc: msg hdr for 0x600a0000 not found replace 0�[0m
    �[0m    0      2          IPC          36443781.145833     45730.519531    src/ipc/handler.c:457 	ipc: comp 0 -> trigger cmd 0x60000�[0m
    �[0m    0      2         PIPE 1.3      36443785.572917         4.427083 src/audio/pipeline.c:711 	pipeline_trigger()�[0m
    �[0m    0      2         HOST 1.0      36443789.270833         3.697917     src/audio/host.c:299 	host_trigger()�[0m
    �[0m    0      2          DAI 1.1      36443799.062500         9.791667      src/audio/dai.c:536 	dai_comp_trigger(), command = 2�[0m
    �[0m    0      2          DAI 1.1      36443806.614583         7.552083      src/audio/dai.c:600 	dai_comp_trigger(), PAUSE/STOP�[0m
    �[0m    0      2          DMA          36443810.208333         3.593750 src/drivers/dw/dma.c:366 	dw_dma_stop(): dma 0 channel 0 stop�[0m
    �[0m    0      2          SSP          36443815.000000         4.791667 src/drivers/intel/cavs/ssp.c:877 	ssp_trigger() cmd 0�[0m
    �[0m    0      2          SSP          36443862.031250        47.031250 src/drivers/intel/cavs/ssp.c:858 	ssp_stop(), TX stop�[0m
    �[0m    0      2      unknown          36443868.072917         6.041667 src/schedule/timer_domain.c:64  	timer_domain_unregister()�[0m
    
  7. jdrzaszcz commented on Feb 26, 2020

    @jdrzaszcz
    Author

    Can't reproduse on master branch 1d273c3 commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    APLApplies to Apollolake platformICLApplies to IceLake platformP2Critical bugs or normal featuresWHLApplies to WhiskeyLake platformbugSomething isn't working as expected

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions