linux-kernel - Re: [PATCH v2 05/24] media: iris: Allow substate transition to load resources during output streaming

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: <11acb8fd-9d81-460e-ac2c-38f370324ea5@kernel.org>
Date: Thu, 14 Aug 2025 08:13:42 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Konrad Dybcio <konrad.dybcio@....qualcomm.com>,
 Bryan O'Donoghue <bod.linux@...w.ie>,
 Dikshita Agarwal <quic_dikshita@...cinc.com>,
 Vikash Garodia <quic_vgarodia@...cinc.com>,
 Abhinav Kumar <abhinav.kumar@...ux.dev>,
 Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
 Mauro Carvalho Chehab <mchehab@...nel.org>, Hans Verkuil
 <hverkuil@...all.nl>, Stefan Schmidt <stefan.schmidt@...aro.org>,
 Vedang Nagar <quic_vnagar@...cinc.com>
Cc: linux-media@...r.kernel.org, linux-arm-msm@...r.kernel.org,
 linux-kernel@...r.kernel.org, Renjiang Han <quic_renjiang@...cinc.com>,
 Wangao Wang <quic_wangaow@...cinc.com>
Subject: Re: [PATCH v2 05/24] media: iris: Allow substate transition to load
 resources during output streaming

On 14/08/2025 00:55, Konrad Dybcio wrote:
> On 8/13/25 11:51 PM, Bryan O'Donoghue wrote:
>> On 13/08/2025 10:37, Dikshita Agarwal wrote:
>>> However, now after removing that restriction, the instance state can be
>>> OUTPUT_STREAMING when firmware start is triggered and substate needs to
>>> be moved to LOAD_RESOURCES.
>>>
>>> Fixes: 547f7b8c5090 ("media: iris: add check to allow sub states transitions")
>>
>> If a restriction has been removed, has that restriction been marked as a 
>> Fixes: ?
>>
>> If not then we can't actually backport this fix as the dependency - the 
>> restriction removal has not been marked for backport.
> 
> Please stop confusing fixes and backports..
> 
> If you're really paranoid about this commit getting autoselected, see
> 'noautosel' under this section:
> 
> https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html#option-1

Commit still needs to explain what is the bug, what is observable issue
being fixed here. Lack of feature is not a bug.

Best regards,
Krzysztof