Re: Patch "iommu/arm-smmu: Drop if with an always false condition" has been added to the 6.3-stable tree

Re: Patch "iommu/arm-smmu: Drop if with an always false condition" has been added to the 6.3-stable tree

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 1082 bytes --]

Hello,

On Fri, May 19, 2023 at 09:49:37PM -0400, Sasha Levin wrote:
> This is a note to let you know that I've just added the patch titled
> 
>     iommu/arm-smmu: Drop if with an always false condition
> 
> to the 6.3-stable tree which can be found at:
>     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> 
> The filename of the patch is:
>      iommu-arm-smmu-drop-if-with-an-always-false-conditio.patch
> and it can be found in the queue-6.3 subdirectory.
> 
> If you, or anyone else, feels it should not be added to the stable tree,
> please let <stable@vger.kernel.org> know about it.

I'd not add that patch to stable. It's just about dropping an

	if (false) {
		something();
	}

The compiler probably isn't able to see that the condition is always
false, so the only benefit is that the patch makes the compiled code a
bit smaller.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: Patch "iommu/arm-smmu: Drop if with an always false condition" has been added to the 6.3-stable tree

[Sasha Levin]

On Sat, May 20, 2023 at 04:40:18PM +0200, Uwe Kleine-König wrote:
>Hello,
>
>On Fri, May 19, 2023 at 09:49:37PM -0400, Sasha Levin wrote:
>> This is a note to let you know that I've just added the patch titled
>>
>>     iommu/arm-smmu: Drop if with an always false condition
>>
>> to the 6.3-stable tree which can be found at:
>>     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
>>
>> The filename of the patch is:
>>      iommu-arm-smmu-drop-if-with-an-always-false-conditio.patch
>> and it can be found in the queue-6.3 subdirectory.
>>
>> If you, or anyone else, feels it should not be added to the stable tree,
>> please let <stable@vger.kernel.org> know about it.
>
>I'd not add that patch to stable. It's just about dropping an
>
>	if (false) {
>		something();
>	}
>
>The compiler probably isn't able to see that the condition is always
>false, so the only benefit is that the patch makes the compiled code a
>bit smaller.

Dropped, thanks!

-- 
Thanks,
Sasha