Re: [PATCH 5.11 000/252] 5.11.11-rc2 review

[Ronald Warsow]

On 29.03.21 16:46, Greg KH wrote:
> On Mon, Mar 29, 2021 at 04:40:00PM +0200, Ronald Warsow wrote:
> > hello
> >
> > 5.11.11-rc2 compiles, boots and run fine here (on an i7-6700 with Fedora
> > 34 Beta)
> >
> > Thanks.
> >
> > but I see some error's/warning's with the new gcc-11.0.1 compared to the
> > elder gcc-10.2.1-11
>
> Do you also see those gcc-11 warnings on Linus's tree?  I don't think
> I've started to backport any gcc-11 fixes like this to older kernels
> just yet.
>
> thanks,
>
> greg k-h

it seems so (I'm not an developer, just a home user compiling kernels).

AFAIK, there is a new gcc 11.x.y release with fixes for Fedora 34 in the
pipe, but not yet released, so no need to invest time.
just an report what I see here.


with linux-5.12-rc5 from https://www.kernel.org/

I see:

...

  CC      mm/gup.o
  AR      arch/x86/kernel/acpi/built-in.a
  CC      fs/proc/inode.o
In function ‘poly1305_simd_init’,
    inlined from ‘crypto_poly1305_setdctxkey’ at
arch/x86/crypto/poly1305_glue.c:150:4,
    inlined from ‘crypto_poly1305_setdctxkey’ at
arch/x86/crypto/poly1305_glue.c:144:21,
    inlined from ‘poly1305_update_arch’ at
arch/x86/crypto/poly1305_glue.c:181:8:
arch/x86/crypto/poly1305_glue.c:86:9: warning: ‘poly1305_init_x86_64’
reading 32 bytes from a region of size 16 [-Wstringop-overread]
   86 |         poly1305_init_x86_64(ctx, key);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
arch/x86/crypto/poly1305_glue.c: In function ‘poly1305_update_arch’:
arch/x86/crypto/poly1305_glue.c:86:9: note: referencing argument 2 of
type ‘const u8 *’ {aka ‘const unsigned char *’}
arch/x86/crypto/poly1305_glue.c:18:17: note: in a call to function
‘poly1305_init_x86_64’
   18 | asmlinkage void poly1305_init_x86_64(void *ctx,
      |                 ^~~~~~~~~~~~~~~~~~~~
  AS      arch/x86/crypto/nh-sse2-x86_64.o
...

  CC      kernel/cgroup/cpuset.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      kernel/time/tick-common.o
...

  CC      drivers/acpi/acpica/utglobal.o
lib/crypto/poly1305-donna64.c:15:67: warning: argument 2 of type ‘const
u8[16]’ {aka ‘const unsigned char[16]’} with mismatched bound
[-Warray-parameter=]
   15 | void poly1305_core_setkey(struct poly1305_core_key *key, const
u8 raw_key[16])
      |
~~~~~~~~~^~~~~~~~~~~
In file included from lib/crypto/poly1305-donna64.c:11:
./include/crypto/internal/poly1305.h:21:68: note: previously declared as
‘const u8 *’ {aka ‘const unsigned char *’}
   21 | void poly1305_core_setkey(struct poly1305_core_key *key, const
u8 *raw_key);
      |
~~~~~~~~~~^~~~~~~
  CC      fs/btrfs/raid56.o
..

  CC      lib/decompress_unlzma.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      arch/x86/lib/cache-smp.o
...


  CC      drivers/base/core.o
In function ‘snb_wm_latency_quirk’,
    inlined from ‘ilk_setup_wm_latency’ at
drivers/gpu/drm/i915/intel_pm.c:3108:3,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3057:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3057 |         intel_print_wm_latency(dev_priv, "Primary",
dev_priv->wm.pri_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3057:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
In function ‘snb_wm_latency_quirk’,
    inlined from ‘ilk_setup_wm_latency’ at
drivers/gpu/drm/i915/intel_pm.c:3108:3,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3058:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3058 |         intel_print_wm_latency(dev_priv, "Sprite",
dev_priv->wm.spr_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3058:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
In function ‘snb_wm_latency_quirk’,
    inlined from ‘ilk_setup_wm_latency’ at
drivers/gpu/drm/i915/intel_pm.c:3108:3,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3059:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3059 |         intel_print_wm_latency(dev_priv, "Cursor",
dev_priv->wm.cur_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3059:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
In function ‘snb_wm_lp3_irq_quirk’,
    inlined from ‘ilk_setup_wm_latency’ at
drivers/gpu/drm/i915/intel_pm.c:3109:3,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3086:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3086 |         intel_print_wm_latency(dev_priv, "Primary",
dev_priv->wm.pri_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3086:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
In function ‘snb_wm_lp3_irq_quirk’,
    inlined from ‘ilk_setup_wm_latency’ at
drivers/gpu/drm/i915/intel_pm.c:3109:3,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3087:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3087 |         intel_print_wm_latency(dev_priv, "Sprite",
dev_priv->wm.spr_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3087:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
In function ‘snb_wm_lp3_irq_quirk’,
    inlined from ‘ilk_setup_wm_latency’ at
drivers/gpu/drm/i915/intel_pm.c:3109:3,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3088:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3088 |         intel_print_wm_latency(dev_priv, "Cursor",
dev_priv->wm.cur_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3088:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
In function ‘ilk_setup_wm_latency’,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3103:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3103 |         intel_print_wm_latency(dev_priv, "Primary",
dev_priv->wm.pri_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3103:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
In function ‘ilk_setup_wm_latency’,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3104:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3104 |         intel_print_wm_latency(dev_priv, "Sprite",
dev_priv->wm.spr_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3104:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
In function ‘ilk_setup_wm_latency’,
    inlined from ‘intel_init_pm’ at drivers/gpu/drm/i915/intel_pm.c:7628:3:
drivers/gpu/drm/i915/intel_pm.c:3105:9: warning:
‘intel_print_wm_latency’ reading 16 bytes from a region of size 10
[-Wstringop-overread]
 3105 |         intel_print_wm_latency(dev_priv, "Cursor",
dev_priv->wm.cur_latency);
      |
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i915/intel_pm.c: In function ‘intel_init_pm’:
drivers/gpu/drm/i915/intel_pm.c:3105:9: note: referencing argument 3 of
type ‘const u16 *’ {aka ‘const short unsigned int *’}
drivers/gpu/drm/i915/intel_pm.c:2994:13: note: in a call to function
‘intel_print_wm_latency’
 2994 | static void intel_print_wm_latency(struct drm_i915_private
*dev_priv,
      |             ^~~~~~~~~~~~~~~~~~~~~~
  CC [M]  drivers/gpu/drm/i915/intel_runtime_pm.o
..

  CC      net/openvswitch/vport-gre.o
In function ‘intel_dp_check_mst_status’,
    inlined from ‘intel_dp_hpd_pulse’ at
drivers/gpu/drm/i915/display/intel_dp.c:5901:8:
drivers/gpu/drm/i915/display/intel_dp.c:4554:22: warning:
‘drm_dp_channel_eq_ok’ reading 6 bytes from a region of size 4
[-Wstringop-overread]
 4554 |                     !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:4554: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],
      |      ^~~~~~~~~~~~~~~~~~~~
  AR      net/xdp/built-in.a

...



--
regards

Ronald