linux-kernel - Re: [PATCH 5.10 000/162] 5.10.157-rc1 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: <Y4nmNb46aqbm7JuS@duo.ucw.cz>
Date:   Fri, 2 Dec 2022 12:49:09 +0100
From:   Pavel Machek <pavel@...x.de>
To:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        mlevitsk@...hat.com, samuel.thibault@...-lyon.org,
        pawell@...ence.com
Cc:     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, pavel@...x.de,
        jonathanh@...dia.com, f.fainelli@...il.com,
        sudipm.mukherjee@...il.com, srw@...dewatkins.net, rwarsow@....de
Subject: Re: [PATCH 5.10 000/162] 5.10.157-rc1 review

Hi!

[If I cc-ed you, you are author of one of patches below. Please take a
look.]

> Responses should be made by Fri, 02 Dec 2022 18:05:05 +0000.
> Anything received after that time might be too late.

I hope to make it :-). I

> Pawel Laszczak <pawell@...ence.com>
>     usb: cdnsp: Device side header file for CDNSP driver
> 
> Pawel Laszczak <pawell@...ence.com>
>     usb: cdns3: Add support for DRD CDNSP

These two together are 1500+ lines, and are marked as Stable-dep-of:
9d5333c93134 ("usb: cdns3: host: fix endless superspeed hub port
reset") . But we don't have that one in 5.10 tree. Likely we should
not have these either.

> Maxim Levitsky <mlevitsk@...hat.com>
>     KVM: x86: emulator: update the emulation mode after rsm

No. The patch does not do anything. Mainline commit this referenced
changed the return value, this changes just a comment. Wrong backport?

> Samuel Thibault <samuel.thibault@...-lyon.org>
>     speakup: Generate speakupmap.h automatically

Ok, so this one rewrites some header generation and is buggy. 500+ lines.

> Đoàn Trần Công Danh <congdanhqx@...il.com>
>     speakup: replace utils' u_char with unsigned char

With this patch fixing it. The rewrite is marked as stable depedncency
of the fix, but fix would not be needed if we did not apply the
rewrite. We should not have these two in stable.

Best regards,
								Pavel

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Download attachment "signature.asc" of type "application/pgp-signature" (196 bytes)