linux-kernel - Re: [PATCH 2/2] ASoC: hdmi-codec: dump ELD through procfs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdWjc1rE54KZ39XmVbjvGa4rYeaSjAMgMQfs6gFKpRbhZg@mail.gmail.com>
Date: Mon, 7 Apr 2025 15:09:45 +0200
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
Cc: Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>, Liam Girdwood <lgirdwood@...il.com>, 
	Mark Brown <broonie@...nel.org>, linux-sound@...r.kernel.org, 
	linux-kernel@...r.kernel.org, 
	Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>, 
	Linux-Renesas <linux-renesas-soc@...r.kernel.org>
Subject: Re: [PATCH 2/2] ASoC: hdmi-codec: dump ELD through procfs

Hi Dmitry,

On Fri, 24 Jan 2025 at 22:14, Dmitry Baryshkov
<dmitry.baryshkov@...aro.org> wrote:
> Use freshly added API and add eld#n files to procfs for the ASoC cards
> utilizing HDMI codec. This simplifies debugging of the possible ASoC /
> HDMI / DisplayPort audio issues.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>

Thanks for your patch, which is now commit 0ecd24a6d8b251eb ("ASoC:
hdmi-codec: dump ELD through procfs") in v6.15-rc1.

This causes the following failure on Salvator-XS:

    ------------[ cut here ]------------
    proc_dir_entry 'card0/eld#0' already registered
    WARNING: CPU: 3 PID: 93 at fs/proc/generic.c:377 proc_register+0x12c/0x1b8
    CPU: 3 UID: 0 PID: 93 Comm: kworker/u33:5 Not tainted
6.14.0-rc1-arm64-renesas-00004-g0ecd24a6d8b2 #2923
    Hardware name: Renesas Salvator-X 2nd version board based on r8a77951 (DT)
    Workqueue: events_unbound deferred_probe_work_func
    pstate: 60000005 (nZCv daif -PAN -UAO -TCO -DIT -SSBS BTYPE=--)
    pc : proc_register+0x12c/0x1b8
    lr : proc_register+0x12c/0x1b8
    sp : ffff800081ac38d0
    x29: ffff800081ac38d0 x28: ffff0004c2f7b398 x27: ffff8000813fca80
    x26: ffff0004c2f3b8c0 x25: ffff8000814b3660 x24: ffff0004c2f3b840
    x23: 0000000000000005 x22: ffff0004c2ce7b2c x21: 0000000000000005
    x20: ffff0004c2ce7a80 x19: ffff0004c2ce7a48 x18: 0000000000000006
    x17: 0000000000000000 x16: 0000000000000068 x15: 0769076707650772
    x14: 0720077907640761 x13: ffff800081380a30 x12: 00000000000005ca
    x11: 00000000000001ee x10: ffff8000813d8a30 x9 : ffff800081380a30
    x8 : 00000000ffffefff x7 : ffff8000813d8a30 x6 : 80000000fffff000
    x5 : 00000000000001ef x4 : 0000000000000000 x3 : 0000000000000000
    x2 : 0000000000000000 x1 : 0000000000000000 x0 : ffff0004c0870000
    Call trace:
     proc_register+0x12c/0x1b8 (P)
     proc_create_data+0x3c/0x60
     snd_info_register+0xcc/0x12c
     snd_info_register+0x30/0x12c
     snd_info_card_register+0x1c/0x94
     snd_card_register+0x1a4/0x1e4
     snd_soc_bind_card+0x7dc/0xab4
     snd_soc_register_card+0xec/0x100
     devm_snd_soc_register_card+0x48/0x98
     audio_graph_parse_of+0x1c4/0x1f8
     graph_probe+0x6c/0x80
     platform_probe+0x64/0xbc
     really_probe+0xb8/0x294
     __driver_probe_device+0x74/0x124
     driver_probe_device+0xd4/0x158
     __device_attach_driver+0xd4/0x154
     bus_for_each_drv+0x84/0xe0
     __device_attach+0x9c/0x188
     device_initial_probe+0x10/0x18
     bus_probe_device+0xa0/0xa4
     deferred_probe_work_func+0x80/0xb4
     process_one_work+0x144/0x280
     worker_thread+0x2c4/0x3cc
     kthread+0x128/0x1e0
     ret_from_fork+0x10/0x20
    ---[ end trace 0000000000000000 ]---
    asoc-audio-graph-card sound: ASoC: failed to register soundcard -12
    asoc-audio-graph-card sound: probe with driver
asoc-audio-graph-card failed with error -12

And:

     ALSA device list:
    -  #0: rcar-sound
    +  No soundcards found.

So the card it tried to register is the first one?

Reverting the commit fixes the issue.

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds