linux-kernel - Re: [PATCH 5.10 000/187] 5.10.202-rc3 review

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: <ZWWPJtXZ12WuTtd1@codewreck.org>
Date:   Tue, 28 Nov 2023 15:56:38 +0900
From:   Dominique Martinet <asmadeus@...ewreck.org>
To:     Pavel Machek <pavel@...x.de>, youwan Wang <wangyouwan@....com>,
        Luiz Augusto von Dentz <luiz.von.dentz@...el.com>
Cc:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        stable@...r.kernel.org, patches@...ts.linux.dev,
        linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
        akpm@...ux-foundation.org, linux@...ck-us.net, shuah@...nel.org,
        patches@...nelci.org, lkft-triage@...ts.linaro.org,
        jonathanh@...dia.com, f.fainelli@...il.com,
        sudipm.mukherjee@...il.com, srw@...dewatkins.net, rwarsow@....de,
        conor@...nel.org, allen.lkml@...il.com, maz@...nel.org,
        andy.shevchenko@...il.com, brgl@...ev.pl, wangyouwan@....com,
        jani.nikula@...el.com, rf@...nsource.cirrus.com,
        ilpo.jarvinen@...ux.intel.com, dan.carpenter@...aro.org
Subject: Re: [PATCH 5.10 000/187] 5.10.202-rc3 review

Pavel Machek wrote on Mon, Nov 27, 2023 at 10:25:46PM +0100:
> > Marc Zyngier <maz@...nel.org>
> >     gpio: Don't fiddle with irqchips marked as immutable
>
> This is attempt to move people to new API, which will cause warning
> for existing users. "chip_warn(gc, "not an immutable chip, please
> consider fixing it!\n");". It is marked as dependency of another
> patch, but I'm not sure we should be doing this in stable.

And this actually did make some tests fail on tegra (other branch on
this thread), it's probably safer to drop the gpio patches unless we're
prepared to backport all the "make immutable" patches as well.

(I don't see why it'd be required for dc3115e6c5d9 either but I didn't
look all that close)


> > youwan Wang <wangyouwan@....com>
> >     Bluetooth: btusb: Add date->evt_skb is NULL check
> 
> Could someone double check this? If we hit the null check, we'll be
> returning success, but it sounds like an error case.

I agree with your analysis; added Youwan and Luiz in recipients.

btusb_mtk_hci_wmt_sync will return 0 if the skb is null here -- should
this be setting err = -EINVAL or something?

> > Jani Nikula <jani.nikula@...el.com>
> >     drm/msm/dp: skip validity check for DP CTS EDID checksum
> 
> This is preparation for future cleanup, do we need it?

(For cleanup patches I'd say if it makes future backports easier it
doesn't hurt to take them)

-- 
Dominique Martinet | Asmadeus