Re: [PATCH 4.19 00/97] 4.19.300-rc1 review

From: Greg Kroah-Hartman
Date: Sat Nov 25 2023 - 10:44:05 EST


On Fri, Nov 24, 2023 at 01:13:33PM -0600, Daniel Díaz wrote:
> Hello!
>
> On 24/11/23 11:49 a. m., Greg Kroah-Hartman wrote:
> > This is the start of the stable review cycle for the 4.19.300 release.
> > There are 97 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 Sun, 26 Nov 2023 17:19:17 +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/v4.x/stable-review/patch-4.19.300-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-4.19.y
> > and the diffstat can be found below.
> >
> > thanks,
> >
> > greg k-h
>
> We see this failure on Arm32:
> -----8<-----
> /builds/linux/drivers/tty/serial/meson_uart.c: In function 'meson_uart_probe':
> /builds/linux/drivers/tty/serial/meson_uart.c:728:13: error: 'struct uart_port' has no member named 'has_sysrq'
> 728 | port->has_sysrq = IS_ENABLED(CONFIG_SERIAL_MESON_CONSOLE);
> | ^~
> make[4]: *** [/builds/linux/scripts/Makefile.build:303: drivers/tty/serial/meson_uart.o] Error 1
> ----->8-----

Thanks, now fixed.

> And this one on Arm64:
> -----8<-----
> /builds/linux/kernel/profile.c: In function 'profile_dead_cpu':
> /builds/linux/kernel/profile.c:346:27: warning: the comparison will always evaluate as 'true' for the address of 'prof_cpu_mask' will never be NULL [-Waddress]
> 346 | if (prof_cpu_mask != NULL)
> | ^~
> /builds/linux/kernel/profile.c:49:22: note: 'prof_cpu_mask' declared here
> 49 | static cpumask_var_t prof_cpu_mask;
> | ^~~~~~~~~~~~~
> /builds/linux/kernel/profile.c: In function 'profile_online_cpu':
> /builds/linux/kernel/profile.c:383:27: warning: the comparison will always evaluate as 'true' for the address of 'prof_cpu_mask' will never be NULL [-Waddress]
> 383 | if (prof_cpu_mask != NULL)
> | ^~
> /builds/linux/kernel/profile.c:49:22: note: 'prof_cpu_mask' declared here
> 49 | static cpumask_var_t prof_cpu_mask;
> | ^~~~~~~~~~~~~
> /builds/linux/kernel/profile.c: In function 'profile_tick':
> /builds/linux/kernel/profile.c:413:47: warning: the comparison will always evaluate as 'true' for the address of 'prof_cpu_mask' will never be NULL [-Waddress]
> 413 | if (!user_mode(regs) && prof_cpu_mask != NULL &&
> | ^~
> /builds/linux/kernel/profile.c:49:22: note: 'prof_cpu_mask' declared here
> 49 | static cpumask_var_t prof_cpu_mask;
> | ^~~~~~~~~~~~~
> ----->8-----

Is this new? No code in this -rc touches this file, or prof_cpu_mask
that I can tell. Hints for what went wrong would be appreciated...

thanks,

greg k-h