<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
<div class="moz-cite-prefix">Добрый вечер!<br>
<br>
On 27.02.2023 14:50, Данила Чернецов wrote:<br>
</div>
<blockquote type="cite"
cite="mid:b1229019-61b1-0c69-309b-51f4aa35ee83@ivk.ru">
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<p> </p>
<p class="MsoNormal">Добрый день! <br>
В рамках работ по статистическому анализу кода ядра <span
style="mso-ansi-language:EN-US" lang="EN-US">linux</span>-5.10.<span
style="mso-ansi-language:EN-US" lang="EN-US">y</span><span
lang="EN-US"> </span>перед отправкой в <span
style="mso-ansi-language:EN-US" lang="EN-US">upstream</span>,
согласно рекомендациям, направляем вам на согласование патч по
срабатыванию <a
href="https://lvc-svacer.ispras.ru/mode/review/project/83b681dd-c2b5-459d-80cc-359ff2a64935/branch/4739abc2-5b35-494c-9748-062208797cd3/snapshot/952a1137-9415-42e4-8f91-83c6169ede53/marker/eyJtYXJrZXJJRCI6ICIxOTlkMTk0NS0xMTgzLTQyMmItOWY0Ni1iM2QxOTgwMjQyZmYiLCAiZmlsZSI6ICIvZHJpdmVycy9ncHUvZHJtL2FtZC9kaXNwbGF5L2RjL2NvcmUvZGMuYyJ9"
target="_blank" moz-do-not-send="true"><span
style="font-size:13.5pt;line-height:107%;
mso-ansi-language:EN-US" lang="EN-US">NULL</span><span
style="font-size:13.5pt;line-height: 107%">_</span><span
style="font-size:13.5pt;line-height:107%;
mso-ansi-language:EN-US" lang="EN-US">AFTER</span><span
style="font-size:13.5pt;line-height: 107%">_</span><span
style="font-size:13.5pt;line-height:107%;
mso-ansi-language:EN-US" lang="EN-US">DEREF</span><span
style="font-size:13.5pt;line-height: 107%">: </span><span
style="font-size:13.5pt;line-height:107%;
mso-ansi-language:EN-US" lang="EN-US">drivers</span><span
style="font-size:13.5pt; line-height:107%">/</span><span
style="font-size:13.5pt;line-height:
107%;mso-ansi-language:EN-US" lang="EN-US">gpu</span><span
style="font-size:13.5pt; line-height:107%">/</span><span
style="font-size:13.5pt;line-height:
107%;mso-ansi-language:EN-US" lang="EN-US">drm</span><span
style="font-size:13.5pt; line-height:107%">/</span><span
style="font-size:13.5pt;line-height:
107%;mso-ansi-language:EN-US" lang="EN-US">amd</span><span
style="font-size:13.5pt; line-height:107%">/</span><span
style="font-size:13.5pt;line-height:
107%;mso-ansi-language:EN-US" lang="EN-US">display</span><span
style="font-size:13.5pt; line-height:107%">/</span><span
style="font-size:13.5pt;line-height:
107%;mso-ansi-language:EN-US" lang="EN-US">dc</span><span
style="font-size:13.5pt; line-height:107%">/</span><span
style="font-size:13.5pt;line-height:
107%;mso-ansi-language:EN-US" lang="EN-US">core</span><span
style="font-size:13.5pt; line-height:107%">/</span><span
style="font-size:13.5pt;line-height:
107%;mso-ansi-language:EN-US" lang="EN-US">dc</span><span
style="font-size:13.5pt; line-height:107%">.</span><span
style="font-size:13.5pt;line-height:
107%;mso-ansi-language:EN-US" lang="EN-US">c</span><span
style="font-size:13.5pt;line-height: 107%">:1411</span></a><span
class="MsoHyperlink"><span
style="font-size:13.5pt;line-height:107%" lang="EN-US"> </span></span>Просим
вас провести ревью и указать возможные недочеты. <br>
Во вложении патчи для <span style="mso-ansi-language:EN-US"
lang="EN-US">upstream </span>(0002-drm-amd-display-remove-an-unnecessary-NULL-check.patch)
и <span style="mso-ansi-language:EN-US" lang="EN-US">linux</span>-5.10.<span
style="mso-ansi-language:EN-US" lang="EN-US">y</span>
(0003-drm-amd-display-remove-an-unnecessary-NULL-check.patch).</p>
</blockquote>
1. Патч для linux-5.10.y потребуется после того как будет принят
патч в <span style="mso-ansi-language:EN-US" lang="EN-US">upstream.<br>
2. <br>
Subject: [PATCH 2/2] drm/amd/display: remove an unnecessary NULL
check<br>
--></span><br>
<span style="mso-ansi-language:EN-US" lang="EN-US"><span
style="mso-ansi-language:EN-US" lang="EN-US">Subject: [PATCH]
drm/amd/display: remove an unnecessary NULL check<br>
</span><br>
3.<br>
The "pipe" pointer can't be NULL because it's dereferenced on the
line before.<br>
не совсем корректно, поскольку на предыдущей строке не
разыменование указателя,<br>
а взятие указателя на элемент массива, являющегося полем
структуры.<br>
Скорее:<br>
</span><span style="mso-ansi-language:EN-US" lang="EN-US"><span
style="mso-ansi-language:EN-US" lang="EN-US">The 'pipe' can't be
NULL because it points to an element of array.<br>
<br>
</span>4. И хотя некоторые мейнтейнеры не соглашаются с Fixes: при
удалении лишнего кода, предлагаю добавить <br>
<br>
Fixes: 7cf2c840c6b5 ("drm/amd/display: Commit validation set from
state")<br>
<br>
<br>
</span>Алексей<br>
</body>
</html>