Re: [PATCH 5.12 000/363] 5.12.5-rc1 review

[Ronald Warsow <rwarsow@gmx.de>]

On 5/17/21 7:57 AM, Greg Kroah-Hartman wrote:
 > This is the start of the stable review cycle for the 5.12.5 release.
 > There are 363 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 Wed, 19 May 2021 14:02:12 +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/v5.x/stable-review/patch-5.12.5-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-5.12.y
 > and the diffstat can be found below.
 >
 > thanks,
 >
 > greg k-h
 >

Thanks

apart from some warnings all is fine here.

Fedora 34,
gcc (GCC) 11.1.1 20210428 (Red Hat 11.1.1-1)



..
CC      arch/x86/platform/efi/efi_64.o
In file included from ./include/linux/string.h:269,
                  from ./include/linux/bitmap.h:9,
                  from ./include/linux/cpumask.h:12,
                  from ./arch/x86/include/asm/cpumask.h:5,
                  from ./arch/x86/include/asm/msr.h:11,
                  from ./arch/x86/include/asm/processor.h:22,
                  from ./arch/x86/include/asm/cpufeature.h:5,
                  from ./arch/x86/include/asm/thread_info.h:53,
                  from ./include/linux/thread_info.h:59,
                  from ./arch/x86/include/asm/preempt.h:7,
                  from ./include/linux/preempt.h:78,
                  from ./include/linux/rcupdate.h:27,
                  from ./include/linux/rbtree.h:22,
                  from ./include/linux/iova.h:14,
                  from ./include/linux/intel-iommu.h:14,
                  from arch/x86/kernel/tboot.c:9:
In function ‘memcmp’,
     inlined from ‘tboot_probe’ at arch/x86/kernel/tboot.c:70:6:
./include/linux/fortify-string.h:19:33: warning: ‘__builtin_memcmp_eq’
specified bound 16 exceeds source size 0 [-Wstringop-overread]
    19 | #define __underlying_memcmp     __builtin_memcmp
       |                                 ^
./include/linux/fortify-string.h:235:16: note: in expansion of macro
‘__underlying_memcmp’
   235 |         return __underlying_memcmp(p, q, size);
       |                ^~~~~~~~~~~~~~~~~~~
   CC      crypto/cmac.o
...



CC      net/netfilter/nft_osf.o
arch/x86/lib/msr-smp.c:255:51: warning: argument 2 of type ‘u32 *’ {aka
‘unsigned int *’} declared as a pointer [-Warray-parameter=]
   255 | int rdmsr_safe_regs_on_cpu(unsigned int cpu, u32 *regs)
       |                                              ~~~~~^~~~
In file included from ./arch/x86/include/asm/processor.h:22,
                  from ./arch/x86/include/asm/cpufeature.h:5,
                  from ./arch/x86/include/asm/thread_info.h:53,
                  from ./include/linux/thread_info.h:59,
                  from ./arch/x86/include/asm/preempt.h:7,
                  from ./include/linux/preempt.h:78,
                  from arch/x86/lib/msr-smp.c:3:
./arch/x86/include/asm/msr.h:347:50: note: previously declared as an
array ‘u32[8]’ {aka ‘unsigned int[8]’}
   347 | int rdmsr_safe_regs_on_cpu(unsigned int cpu, u32 regs[8]);
       |                                              ~~~~^~~~~~~
arch/x86/lib/msr-smp.c:268:51: warning: argument 2 of type ‘u32 *’ {aka
‘unsigned int *’} declared as a pointer [-Warray-parameter=]
   268 | int wrmsr_safe_regs_on_cpu(unsigned int cpu, u32 *regs)
       |                                              ~~~~~^~~~
In file included from ./arch/x86/include/asm/processor.h:22,
                  from ./arch/x86/include/asm/cpufeature.h:5,
                  from ./arch/x86/include/asm/thread_info.h:53,
                  from ./include/linux/thread_info.h:59,
                  from ./arch/x86/include/asm/preempt.h:7,
                  from ./include/linux/preempt.h:78,
                  from arch/x86/lib/msr-smp.c:3:
./arch/x86/include/asm/msr.h:348:50: note: previously declared as an
array ‘u32[8]’ {aka ‘unsigned int[8]’}
   348 | int wrmsr_safe_regs_on_cpu(unsigned int cpu, u32 regs[8]);
       |                                              ~~~~^~~~~~~
   CC      net/bridge/netfilter/nf_conntrack_bridge.o
...



In function ‘intel_dp_check_mst_status’,
     inlined from ‘intel_dp_hpd_pulse’ at
drivers/gpu/drm/i915/display/intel_dp.c:5852:8:
drivers/gpu/drm/i915/display/intel_dp.c:4505:22: warning:
‘drm_dp_channel_eq_ok’ reading 6 bytes from a region of size 4
[-Wstringop-overread]
  4505 |                     !drm_dp_channel_eq_ok(&esi[10],
intel_dp->lane_count)) {
       |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/display/intel_dp.c: In function ‘intel_dp_hpd_pulse’:
drivers/gpu/drm/i915/display/intel_dp.c:4505:22: note: referencing
argument 1 of type ‘const u8 *’ {aka ‘const unsigned char *’}
In file included from drivers/gpu/drm/i915/display/intel_dp.c:38:
./include/drm/drm_dp_helper.h:1459:6: note: in a call to function
‘drm_dp_channel_eq_ok’
  1459 | bool drm_dp_channel_eq_ok(const u8
link_status[DP_LINK_STATUS_SIZE],
       |      ^~~~~~~~~~~~~~~~~~~~
   CC      drivers/gpu/drm/drm_blend.o




--
regards

Ronald