netdev - Re: Build regressions/improvements in v6.1-rc7

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>] [day] [month] [year] [list]

Date:   Mon, 28 Nov 2022 14:52:03 +0100 (CET)
From:   Geert Uytterhoeven <geert@...ux-m68k.org>
To:     linux-kernel@...r.kernel.org
cc:     linux-um@...ts.infradead.org, Taras Chornyi <tchornyi@...vell.com>,
        netdev@...r.kernel.org, linux-spi@...r.kernel.org
Subject: Re: Build regressions/improvements in v6.1-rc7

On Mon, 28 Nov 2022, Geert Uytterhoeven wrote:
> JFYI, when comparing v6.1-rc7[1] to v6.1-rc6[3], the summaries are:
>  - build errors: +6/-0

   + /kisskb/src/arch/um/include/asm/processor-generic.h: error: called object is not a function or function pointer: 94:18 => 94:19, 94:18

um-x86_64-gcc12/um-all{mod,yes}config (in kfd_cpumask_to_apic_id(), seen
before with um-x86_64/um-all{mod,yes}config)

   + /kisskb/src/drivers/infiniband/sw/rdmavt/qp.c: error: 'struct cpuinfo_um' has no member named 'x86_cache_size':  => 88:22
   + /kisskb/src/drivers/infiniband/sw/rdmavt/qp.c: error: control reaches end of non-void function [-Werror=return-type]:  => 89:1
   + /kisskb/src/drivers/infiniband/sw/rdmavt/qp.c: error: implicit declaration of function '__copy_user_nocache' [-Werror=implicit-function-declaration]:  => 100:2

um-x86_64/um-all{mod,yes}config (seen before on v6.1-rc4)

   + /kisskb/src/drivers/net/ethernet/marvell/prestera/prestera_flower.c: error: 'rule' is used uninitialized [-Werror=uninitialized]:  => 480:34
   + /kisskb/src/drivers/spi/spi-stm32-qspi.c: error: 'op' is used uninitialized [-Werror=uninitialized]:  => 523:27, 564:27

um-x86_64-gcc12/um-all{mod,yes}config

The latter looks completely bogus to me, unless there's a bug in the UML
implementation of memcpy() and memset(), which are used to initialize
"op".

> [1] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/b7b275e60bcd5f89771e865a8239325f86d9927d/ (all 152 configs)
> [3] http://kisskb.ellerman.id.au/kisskb/branch/linus/head/eb7081409f94a9a8608593d0fb63a1aa3d6f95d8/ (149 out of 152 configs)

Gr{oetje,eeting}s,

 						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
 							    -- Linus Torvalds