<div>Да, заметил опечатку. Направлю версию 2.<br /><br />Прошу уточнить, при ее исправлении будет ли зачтен бэкпорт по указанному срабатыванию, так как время до конца программы осталось немного, а для полноценного внедрения патча понадобится проводить дополнительный анализ, поскольку произошло изменение кодовой базы.</div><div> </div><div>----------------</div><div>Кому: Aleksandr Burakov (a.burakov@rosa.ru);</div><div>Копия: lvc-project@linuxtesting.org;</div><div>Тема: [lvc-project] [PATCH 6.1] drm/amd/display: Check null pointers before multiple uses;</div><div>14.05.2025, 23:06, "Fedor Pchelkin" <pchelkin@ispras.ru>:</div><blockquote><p>On Wed, 14. May 21:14, Aleksandr Burakov wrote:</p><blockquote> From: Alex Hung <<a href="mailto:alex.hung@amd.com" rel="noopener noreferrer">alex.hung@amd.com</a>><br /> <br /> commit fdd5ecbbff751c3b9061d8ebb08e5c96119915b4 upstream.</blockquote><p><br />[...]<br /> </p><blockquote> [ Aleksandr Burakov: Introducing only the NULL check in<br />   dcn32_enable_phantom_plane() resolves the bug while minimizing risk<br />   and preserving stability in the 6.1.y codebase. ]</blockquote><p><br />Подобные патчи от AMD с разовым исправлением множества предупреждений<br />используемого ими стат. анализатора лучше переносить полностью во всех<br />частях, что актуальны для целевой стабильной ветки (возможно с некоторой<br />тривиальной адаптацией, если потребуется). Вдруг на следующей итерации<br />будет размечена как Confirmed ещё какая-то одна сработка Свейса, которая<br />также исправлена этим же патчем? Поэтому лучше затаскивать полностью, не<br />дробя.<br /><br />Добавление дополнительных проверок на NULL, если уж это было проведено в<br />upstream, в общем случае не является чем-то слишком *рискованным*, и<br />обычно достаточно неплохо ложится на stable окромя отдельных случаев<br />(отсутствие в stable затрагиваемого кода как такового, серьёзное<br />изменение кодовой базы между версиями и т.п.).<br /><br />Однако, как и всегда, проводимые подобными патчами изменения необходимо<br />конечно обдумывать на свежую голову в контексте целевой стабильной ветки.<br /><br />Для внутренних пересылок патчей заведён список рассылки<br /><a href="mailto:lvc-patches@linuxtesting.org" rel="noopener noreferrer">lvc-patches@linuxtesting.org</a>.<br /><br /><a href="mailto:lvc-project@linuxtesting.org" rel="noopener noreferrer">lvc-project@linuxtesting.org</a> только когда идёт взаимодействие с межд.<br />сообществом.<br /><a href="https://gitlab.linuxtesting.ru/lvc/guides#%D1%81%D0%BF%D0%B8%D1%81%D0%BA%D0%B8-%D1%80%D0%B0%D1%81%D1%81%D1%8B%D0%BB%D0%BA%D0%B8-%D1%82%D0%B5%D1%85%D0%BD%D0%BE%D0%BB%D0%BE%D0%B3%D0%B8%D1%87%D0%B5%D1%81%D0%BA%D0%BE%D0%B3%D0%BE-%D1%86%D0%B5%D0%BD%D1%82%D1%80%D0%B0" rel="noopener noreferrer">https://gitlab.linuxtesting.ru/lvc/guides#списки-рассылки-технологического-центра</a><br /> </p><blockquote> @@ -1664,6 +1664,9 @@ static void dcn32_enable_phantom_plane(struct dc *dc,<br />                  else<br />                          phantom_plane = dc_create_plane_state(dc);<br />  <br /> + if !(phantom_plane)<br /> + continue;<br /> +</blockquote><p><br />По очевидным причинам это не соберётся:<br /><br />Wed May 14 18:54:56 UTC 2025 Using -Oline -j 16 W=1 C=1 ARCH=x86_64 CROSS_COMPILE= redirect to /tmp/tmp.94I55qD0lO and /tmp/tmp.V9IImWxUmS<br />Wed May 14 18:54:56 UTC 2025 CC=ccache gcc ccache-dir=/builds/pchelkin/linux-stable-rc-ci/ccache<br />Wed May 14 18:54:56 UTC 2025 gcc (Debian 14.2.0-19) 14.2.0<br />Wed May 14 18:54:56 UTC 2025 Tree base: c4e76361d8d9 ("net: qrtr: use xa_destroy to free internal radix tree memory")<br />Wed May 14 18:54:56 UTC 2025 Baseline building the tree: 729b42915af7 ("drm/amd/display: Check null pointers before multiple uses")<br />../drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource.c: In function 'dcn32_enable_phantom_plane':<br />../drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource.c:1667:20: error: expected '(' before '!' token<br /> 1667 | if !(phantom_plane)<br />      | ^<br />      | (<br />../drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource.c:1667:20: warning: statement with no effect [-Wunused-value]<br /> 1667 | if !(phantom_plane)<br />      | ^~~~~~~~~~~~~~~~<br />make[6]: *** [../scripts/Makefile.build:250: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn32/dcn32_resource.o] Error 1<br />make[5]: *** [../scripts/Makefile.build:503: drivers/gpu/drm/amd/amdgpu] Error 2<br />make[5]: *** Waiting for unfinished jobs....<br />make[4]: *** [../scripts/Makefile.build:503: drivers/gpu/drm] Error 2<br />make[3]: *** [../scripts/Makefile.build:503: drivers/gpu] Error 2<br />make[3]: *** Waiting for unfinished jobs....<br />make[2]: *** [../scripts/Makefile.build:503: drivers] Error 2<br />make[1]: *** [/builds/pchelkin/linux-stable-rc-ci/linux-linux-6.1-lvc-next/Makefile:2013: .] Error 2<br />make: *** [Makefile:238: __sub-make] Error 2<br />Wed May 14 19:34:31 UTC 2025 Exit 1<br /> </p><blockquote>                  memcpy(&phantom_plane->address, &curr_pipe->plane_state->address, sizeof(phantom_plane->address));<br />                  memcpy(&phantom_plane->scaling_quality, &curr_pipe->plane_state->scaling_quality,<br />                                  sizeof(phantom_plane->scaling_quality));<br /> --<br /> 2.40.1</blockquote></blockquote><div> </div><div> </div><div>-- </div><div>С уважением,<br />Бураков Александр,<br />Специалист отдела Информационной безопасности<br />АО "НТЦ ИТ РОСА"<br /><br />email: <a href="mailto:a.burakov@rosa.ru" rel="noopener noreferrer">a.burakov@rosa.ru</a><br />rosa.ru | tg/ntcitrosa | vk/rosaofficial</div><div> </div>