Skip to content

FW boot failed due to no memory window info #695

Closed

Description

@markyang

Summary:
Can't set params for DMA on APL with PCM512x.
This issue cannot be reproduced with the same kernel & topology, and the firmware on Dec. 7, so it seems to be a firmware regression.
sof-logger failed: Unable to open in file /sys/kernel/debug/sof/etrace
sof-logger -t is empty
dmesg:


[    4.064974] sof-audio sof-audio: booting DSP firmware
[    4.065069] sof-audio sof-audio: period_bytes:0x0
[    4.065071] sof-audio sof-audio: periods:1
[    4.086876] sof-audio sof-audio: unstall/run core: core_mask = 1
[    4.086881] sof-audio sof-audio: DSP core(s) enabled? 1 : core_mask 1
[    4.127936] sof-audio sof-audio: ipc rx: 0x70000000: FW_READY
[    4.127939] sof-audio sof-audio: ipc: DSP is ready 0x70000000 offset 0x81000
[    4.127996] sof-audio sof-audio:  Firmware info: version 1.1-8ef28 build 32 on Dec  9 2018:21:52:09
[    4.128007] sof-audio sof-audio: error: have no window info
[    4.128008] sof-audio sof-audio: booting DSP firmware completed
[    4.128010] sof-audio sof-audio: ipc rx done: 0x70000000: FW_READY
[    4.131881] sof-audio sof-audio: Firmware download successful, booting...
[    4.131884] sof-audio sof-audio: firmware boot complete
[    4.131891] sof-audio sof-audio: Debug PCIR: 00000000 at  00000048
[    4.131894] sof-audio sof-audio: Debug PCIW: 00000002 at  00000048
[    4.131898] sof-audio sof-audio: Debug PCIR: 00000000 at  00000044
[    4.131901] sof-audio sof-audio: Debug PCIW: 00000000 at  00000044
[    4.131950] sof-audio sof-audio: generating page table for 000000006c525691 size 0x10000 pages 16
[    4.131952] sof-audio sof-audio: dma_trace_pages: 16
[    4.131984] sof-audio sof-audio: period_bytes:0x0
[    4.131985] sof-audio sof-audio: periods:1
[    4.131991] sof-audio sof-audio: stream_tag: 1
[    4.132007] sof-audio sof-audio: ipc tx: 0x90010000: GLB_TRACE_MSG
[    4.403586] random: crng init done
[    4.403590] random: 7 urandom warning(s) missed due to ratelimiting
[    4.439898] sof-audio sof-audio: error: ipc timed out for 0x90010000 size 0x1c
[    4.439957] sof-audio sof-audio: unknown ROM status value ffffffff
[    4.440072] sof-audio sof-audio: error: unexpected fault 0xffffffff trace 0xffffffff
[    4.440118] sof-audio sof-audio: error: can't set params for DMA for trace -110
[    4.440169] sof-audio sof-audio: warning: failed to initialize trace -110
[    4.440253] sof-audio sof-audio: created machine bxt-pcm512x
[    4.467487] HDMI HDA Codec ehdaudio0D2: Max dais supported: 3
[    4.474193] bxt-pcm512x bxt-pcm512x: info: override FE DAI link SSP5-Codec
[    4.474196] bxt-pcm512x bxt-pcm512x: info: override FE DAI link iDisp1
[    4.474198] bxt-pcm512x bxt-pcm512x: info: override FE DAI link iDisp2
[    4.474199] bxt-pcm512x bxt-pcm512x: info: override FE DAI link iDisp3
[    4.474264] sof-audio sof-audio: loading topology:intel/sof-apl-pcm512x.tplg
[    4.492076] sof-audio sof-audio: tplg: ready widget id 0 pipe 1 type 23 name : PCM0P stream Passthrough Playback 0
[    4.492081] sof-audio sof-audio: loaded host PCM0P
[    4.492083] sof-audio sof-audio:  config: periods snk 2 src 0 fmt 0
[    4.492105] sof-audio sof-audio: ipc tx: 0x30010000: GLB_TPLG_MSG: COMP_NEW
[    4.799878] sof-audio sof-audio: error: ipc timed out for 0x30010000 size 0x44
[    4.799938] sof-audio sof-audio: unknown ROM status value ffffffff
[    4.800054] sof-audio sof-audio: error: unexpected fault 0xffffffff trace 0xffffffff
[    4.800102] sof-audio sof-audio: error: DSP failed to add widget id 0 type 23 name : PCM0P stream Passthrough Playback 0 reply 0
[    4.800166] sof-audio sof-audio: ASoC: failed to load widget PCM0P
[    4.800200] sof-audio sof-audio: error: tplg component load failed -110
[    4.800246] sof-audio sof-audio: error: failed to load DSP topology -22
[    4.800281] sof-audio sof-audio: ASoC: failed to probe component -22
[    4.800330] bxt-pcm512x bxt-pcm512x: ASoC: failed to instantiate card -22
[    4.800396] bxt-pcm512x bxt-pcm512x: snd_soc_register_card failed -22
[    4.800442] bxt-pcm512x: probe of bxt-pcm512x failed with error -22

Test steps:

  1. $ aplay -l
    aplay: device_list:270: no soundcards found...

Test env:
sof master: 8ef2894
kernel sof-dev: 8cf9a6a
tplg: sof-apl-pcm512x.tplg

Log:
dmesg-apl.txt

Activity

  1. added
    bugSomething isn't working as expected
    APLApplies to Apollolake platform
    P1Blocker bugs or important features
    blockedprogress blocked by something else, applies to either feature or bug
    on Dec 10, 2018
  2. markyang commented on Dec 11, 2018

    @markyang
    Author

    This issue can be reproduced if sof-acce7428 was used, but sof-be45dacd worked fine.
    So it seems that sof-acce7428 caused this firmware regression.

  3. xiulipan commented on Dec 11, 2018

    @xiulipan
    Contributor

    @markyang
    From the log here
    [ 4.128007] sof-audio sof-audio: error: have no window info
    The FW boot is failed due to no mem window info. Changed the issue title.

    Root caused to be #657 changed IPC ready structure size make mismatch between kernel and fw.

  4. changed the title [-]Can't set params for DMA on APL with PCM512x[/-] [+]FW boot failed due to no memory window info[/+] on Dec 11, 2018
  5. added
    BYTApplies to Baytrail platform
    GLKApplies to Gemini Lake platform
    on Dec 11, 2018
  6. keqiaozhang commented on Dec 11, 2018

    @keqiaozhang
    Collaborator

    This issue also can be reproduced on GLK.
    cros-sof-v4.14-rebase:676c0362
    sof-master:8ef289

  7. mrajwa commented on Dec 11, 2018

    @mrajwa
    Contributor

    @markyang, what is the evidence that suggest you this commit is the reason for the issue?
    I had a short teleconference today with Jocelyn and one of her engineer. During this meeting we agreed that the issue doesn't mach in any way to the changes done in commit #657 for two main reasons. The issue is only observed when a lot of IPC messeges is being exchenged. Second it was observed at least a week before that commit has been merged.

  8. jocelyn-li commented on Dec 11, 2018

    @jocelyn-li

    @mrajwa some misunderstanding here :( the issue I talked in the meeting is thesofproject/linux#370.

  9. mengdonglin commented on Dec 12, 2018

    @mengdonglin
    Collaborator

    @mrajwa,

    Your commit acce742
    introduced an ABI chnage. It's recommended to use 1 of the reserved 4 uint32_t for the new flag "gdb_enabled".

    In src/include/uapi/ipc/info.h

    /* FW ready Message - sent by firmware when boot has completed */
    struct sof_ipc_fw_ready { struct sof_ipc_fw_ready {
    ...
    uint32_t gdb_enabled;

    /* reserved for future use */		/* reserved for future use */
    uint32_t reserved[4];		uint32_t reserved[4];
    

    } attribute((packed)); } attribute((packed));

  10. markyang commented on Dec 12, 2018

    @markyang
    Author

    @mrajwa, I've tested both firmwares built by the two commits(be45dacd and acce7428) below with the same kernel and the same topology. The former(be45dacd) worked fine, but the latter(acce7428) did not work. BTW, This is the first time to test your commit acce7428, which was merged on Dec 7.

    $ git log --oneline
        8ef2894a (HEAD -> master, origin/master, origin/HEAD) hda-dma: traces: added host hd-dma traces
        acce7428 GDB: Added IPC trigger for GDB debug.
        be45dacd ipc: apply ipc change on BDW & HSW
        ...
    
  11. mengdonglin commented on Dec 14, 2018

    @mengdonglin
    Collaborator

    @markyang SOF has merged the fix f28d3fb

    Could you help to verify this solution for this issue and its duplicate #694?

  12. removed
    GLKApplies to Gemini Lake platform
    on Dec 17, 2018
  13. keqiaozhang commented on Dec 17, 2018

    @keqiaozhang
    Collaborator

    This issue is gone on GLK, logger trace works well on GLK.

  14. markyang commented on Dec 17, 2018

    @markyang
    Author

    Summary:
    The issue cannot be reproduced on APL with PCM512x.

    Test env:
    sof master: 0dad013
    kernel sof-dev: 2d3fcf1
    tplg: sof-apl-pcm512x.tplg

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

Metadata

Metadata

Labels

APLApplies to Apollolake platformBYTApplies to Baytrail platformP1Blocker bugs or important featuresblockedprogress blocked by something else, applies to either feature or bugbugSomething isn't working as expectedverifiedQA/Bug reporter verifies solution worked.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions