RE: [PATCH] ASoC: soc-compress: Fix race condition in soc_compr_open_fe

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



I am sending it again after changing it to plain text mail.

Thanks

From: 이승빈 <seungbin.lee@xxxxxxxxxxx> 
Sent: Monday, June 23, 2025 10:31 AM
To: 'alsa-devel@xxxxxxxxxxxxxxxx' <alsa-devel@xxxxxxxxxxxxxxxx>; 'linux-sound@xxxxxxxxxxxxxxx' <linux-sound@xxxxxxxxxxxxxxx>; 'gregkh@xxxxxxxxxxxxxxxxxxx' <gregkh@xxxxxxxxxxxxxxxxxxx>; 'pierre-louis.bossart@xxxxxxxxxxxxxxx' <pierre-louis.bossart@xxxxxxxxxxxxxxx>; 'perex@xxxxxxxx' <perex@xxxxxxxx>; 'tiwai@xxxxxxx' <tiwai@xxxxxxx>
Cc: 'pilsun.jang@xxxxxxxxxxx' <pilsun.jang@xxxxxxxxxxx>; 's47.kang@xxxxxxxxxxx' <s47.kang@xxxxxxxxxxx>; 'donghee.moon@xxxxxxxxxxx' <donghee.moon@xxxxxxxxxxx>
Subject: FW: [PATCH] ASoC: soc-compress: Fix race condition in soc_compr_open_fe

Add recipient addition and modification patch to the body of the transmission.

From: 이승빈 <mailto:seungbin.lee@xxxxxxxxxxx> 
Sent: Thursday, June 19, 2025 10:25 AM
To: 'tiwai@xxxxxxx' <mailto:tiwai@xxxxxxx>
Subject: [PATCH] ASoC: soc-compress: Fix race condition in soc_compr_open_fe

Hi

I am sending this email to report an issue found in sound/soc/soc-compress.c.

Scenario: Playing a mix of MP3 and WAV files repeatedly from YouTube Music by pressing "next".

Issue occurs when snd_soc_dpcm_runtime_update is called and soc_compr_trigger_fe is invoked, causing the value of “fe->dpcm[stream].runtime_update” to change mid-process. This results in a problem where, during the closing of the BE connected to the FE in snd_soc_dpcm_runtime_update, only the entry is removed from the list without actually being closed.

The attached patch resolves this issue as confirmed through testing.

Additionally, while performing DPCM related operations in soc-compress.c, snd_soc_dpcm_mutex_lock is used; however, soc_compr_trigger_fe does not have this lock. Please review the attached patch regarding this matter.

Thank you.


[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux