Re: [d-kernel] [PATCH 1/2] Revert "ASoC: Intel: sof_es8336: add ES8326 component support"

[Vasiliy Kovalev <kovalev@altlinux.org>]

07.03.2024 12:44, Vitaly Chikunov wrote:
> On Thu, Mar 07, 2024 at 11:50:41AM +0300, kovalev@altlinux.org wrote:
>> From: Vasiliy Kovalev <kovalev@altlinux.org>
>>
>> This reverts commit 15942fcf7b5fa138e8e7b597f327d252f7895d2c.
>>
>> Remove an outdated ALT patch that leads to null pointer dereference:
>> --
>> [    5.784881] RIP: 0010:sof_es8316_exit+0x1b/0x30 [snd_soc_sof_es8336]
>> [    5.785397] Code: 08 5b c3 cc cc cc cc 0f 1f 84 00 00 00 00 00 0f 1f
>> 		     44 00 00 48 8b 47 10 31 f6 8b 50 18 48 8b 87 18 06
>> 		     00 00 48 8b 04 d0 31 d2 <48> 8b 78 58 e9 5c ec e6
>> 		     ff 66 66 2e 0f 1f 84 00 00 00 00 00 90 0f
>> --
>> [    5.794869]  ? asm_exc_page_fault+0x22/0x30
>> [    5.795424]  ? sof_es8316_exit+0x1b/0x30 [snd_soc_sof_es8336]
>> [    5.796205]  snd_soc_remove_pcm_runtime+0x16/0x30 [snd_soc_core]
>> [    5.797141]  snd_soc_add_pcm_runtime.cold+0x29/0x10a [snd_soc_core]
>> [    5.798096]  snd_soc_bind_card+0x34c/0xb70 [snd_soc_core]
>> [    5.798912]  ? __kmalloc_node_track_caller+0x4c/0x130
>> [    5.799592]  ? devm_snd_soc_register_card+0x90/0x90 [snd_soc_core]
>> [    5.800287]  devm_snd_soc_register_card+0x43/0x90 [snd_soc_core]
>> [    5.800984]  sof_es8336_probe.cold+0x968/0x9f1 [snd_soc_sof_es8336]
>> [    5.801665]  platform_probe+0x3c/0xa0
>> --
>> [    5.820537]  </TASK>
> 
> Отревертить патч, который крашит наше ядро это хорошо. Но тут вопрос -
> исходный коммит - кем он был рецензирован? Была ли попытка послать его в
> upstream на review?

Коммит 15942fcf7b5f "ASoC: Intel: sof_es8336: add ES8326 component 
support" изначально разрабатывался для ядра 5.15 и был портирован в 6.1 
вместе с другими патчами с целью пофиксить те же проблемы совместимости, 
но из-за отсутствия возможности протестировать на то время, этот патч 
оказался лишним и ломающим систему.
Тогда же этот патч в апстрим не отправлялся.

> Прошу далее слать подобные коммиты на review в upstream. Upstream first!

Эти патчи только для альтовых 6.1 un-def. А в целом суть ясна.

> После приема или review - людей, которые хорошо понимают ту подсистему,
> которую он меняет - или ждем его прохождения в stable, (но понимая, что
> это может не произойти так как Грег может не принимать патчи по не
> понятным причинам и без объяснений с фиксами известных багов которые
> даже есть в mainline), или принимаем сами - со ссылкой на
> lore.kernel.org с обсуждением патча или на sha1 в mainline.
> 
> Но важно, чтоб коммит, как и со stable ядрами, сначала должен попасть в
> mainline - или очень желательно, если объективные причины не
> препятствуют его попаданию, или если он действительно не ALT
> специфичный, например как AltHa.
> 
> Иначе, наши ядра перестанут быть стабильными, а превратятся в
> экспериментальный сборник нагугленных патчей из инета. Мне кажется, что
> это было бы не очень ответственно по отношению к пользователям. Хотелось
> бы, чтоб наши ядра были качеством не ниже чем upstream stable/longterm
> ядра.

-- 
Regards,
Vasiliy Kovalev