[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2024010420-staple-purify-f5d0@gregkh>
Date: Thu, 4 Jan 2024 13:58:37 +0100
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Naresh Kamboju <naresh.kamboju@...aro.org>
Cc: Daniel Díaz <daniel.diaz@...aro.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, pavel@...x.de,
jonathanh@...dia.com, f.fainelli@...il.com,
sudipm.mukherjee@...il.com, srw@...dewatkins.net, rwarsow@....de,
conor@...nel.org, allen.lkml@...il.com, leo@...lam.fr,
johannes.berg@...el.com, Arnd Bergmann <arnd@...db.de>,
Kees Cook <keescook@...omium.org>
Subject: Re: [PATCH 6.6 00/49] 6.6.10-rc1 review
On Thu, Jan 04, 2024 at 06:09:53PM +0530, Naresh Kamboju wrote:
> On Thu, 4 Jan 2024 at 13:29, Greg Kroah-Hartman
> <gregkh@...uxfoundation.org> wrote:
> >
> > On Thu, Jan 04, 2024 at 01:15:45AM -0600, Daniel Díaz wrote:
> > > Hello!
> > >
> > > On 03/01/24 10:10 p. m., Daniel Díaz wrote:
> > > > Hello!
> > > >
> > > > On 03/01/24 10:55 a. m., Greg Kroah-Hartman wrote:
> > > > > This is the start of the stable review cycle for the 6.6.10 release.
> > > > > There are 49 patches in this series, all will be posted as a response
> > > > > to this one. If anyone has any issues with these being applied, please
> > > > > let me know.
> > > > >
> > > > > Responses should be made by Fri, 05 Jan 2024 16:47:49 +0000.
> > > > > Anything received after that time might be too late.
> > > > >
> > > > > The whole patch series can be found in one patch at:
> > > > > https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.6.10-rc1.gz
> > > > > or in the git tree and branch at:
> > > > > git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-6.6.y
> > > > > and the diffstat can be found below.
> > > > >
> > > > > thanks,
> > > > >
> > > > > greg k-h
> > > >
> > > > We're seeing a build regression with x86/GCC-8 and allmodconfig:
> > > >
> > > > -----8<-----
> > > > In file included from /builds/linux/include/linux/string.h:294,
> > > > from /builds/linux/include/linux/bitmap.h:11,
> > > > from /builds/linux/include/linux/cpumask.h:12,
> > > > from /builds/linux/arch/x86/include/asm/paravirt.h:17,
> > > > from /builds/linux/arch/x86/include/asm/cpuid.h:62,
> > > > from /builds/linux/arch/x86/include/asm/processor.h:19,
> > > > from /builds/linux/arch/x86/include/asm/cpufeature.h:5,
> > > > from /builds/linux/arch/x86/include/asm/thread_info.h:53,
> > > > from /builds/linux/include/linux/thread_info.h:60,
> > > > from /builds/linux/include/linux/uio.h:9,
> > > > from /builds/linux/include/linux/socket.h:8,
> > > > from /builds/linux/include/uapi/linux/if.h:25,
> > > > from /builds/linux/net/wireless/nl80211.c:11:
> > > > In function 'nl80211_set_cqm_rssi.isra.44',
> > > > inlined from 'nl80211_set_cqm' at /builds/linux/net/wireless/nl80211.c:13000:10:
> > > > /builds/linux/include/linux/fortify-string.h:57:29: error: '__builtin_memcpy' pointer overflow between offset 36 and size [-1, 9223372036854775807] [-Werror=array-bounds]
> > > > #define __underlying_memcpy __builtin_memcpy
> > > > ^
> > > > /builds/linux/include/linux/fortify-string.h:648:2: note: in expansion of macro '__underlying_memcpy'
> > > > __underlying_##op(p, q, __fortify_size); \
> > > > ^~~~~~~~~~~~~
> > > > /builds/linux/include/linux/fortify-string.h:693:26: note: in expansion of macro '__fortify_memcpy_chk'
> > > > #define memcpy(p, q, s) __fortify_memcpy_chk(p, q, s, \
> > > > ^~~~~~~~~~~~~~~~~~~~
> > > > /builds/linux/net/wireless/nl80211.c:12939:3: note: in expansion of macro 'memcpy'
> > > > memcpy(cqm_config->rssi_thresholds, thresholds,
> > > > ^~~~~~
> > > > cc1: all warnings being treated as errors
> > > > make[5]: *** [/builds/linux/scripts/Makefile.build:243: net/wireless/nl80211.o] Error 1
> > > > ----->8-----
> > > >
> > > > This is currently being bisected, so there is more to follow.
> > >
> > > Bisection pointed to:
> > >
> > > commit 92045aab1bd9bfd73d816e907ea07739c4550b41
> > > Author: Johannes Berg <johannes.berg@...el.com>
> > > Date: Sat Dec 16 05:47:15 2023 +0000
> > >
> > > wifi: cfg80211: fix CQM for non-range use
> > > commit 7e7efdda6adb385fbdfd6f819d76bc68c923c394 upstream.
> > >
> > > Reverting that was not possible, so I reverted 47f3694a4259 first ("wifi: nl80211: fix deadlock in nl80211_set_cqm_rssi (6.6.x)"), but that was not enough. After reverting both 47f3694a4259 and 92045aab1bd9, the build passed again.
> > >
> > > Reproducer:
> > >
> > > tuxmake --runtime podman --target-arch x86_64 --toolchain gcc-8 --kconfig allmodconfig
> >
> > Very odd, 6.1 works fine with this same compiler?
>
> It works on 6.1 with the same compiler.
>
> > These changes were in
> > the 6.1.70 release (and the 6.1.66 release) before.
> >
> > It was also in 6.6.5, before being reverted in 6.6.6, so why is this
> > causing a problem now?
>
> It was also noticed on 6.6.5-rc1
> git_describe: v6.6.4-135-gb0b05ccdd77d
>
> I have already reported this on a stable mailing list [1] on 6.6.5-rc1.
>
> Please find more details and comments from Arnd [2] and Kees [3].
>
> [1] https://lore.kernel.org/stable/CA+G9fYuL_-Q67t+Y7ST5taYv1XkkoJegH2zBvw_ZUOhF9QRiOg@mail.gmail.com/
> [2] https://lore.kernel.org/stable/ce99918f-eb6a-4ad7-aa44-9d27c27b6b00@app.fastmail.com/
> [3] https://lore.kernel.org/all/202311301016.84D0010@keescook/
>
> Test results comparison,
> https://qa-reports.linaro.org/lkft/linux-stable-rc-linux-6.6.y/build/v6.6.9-50-g5fd1c89851c4/testrun/21921607/suite/build/test/gcc-8-allmodconfig/history/
Ah, ok, thanks, I forgot about that. All should be good then,
greg k-h
Powered by blists - more mailing lists