linux-kernel - Re: [PATCH] arch/sparc: remove unused varible paddrbase in function leon

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

Message-ID: <59d604b6-8672-4c90-ab10-b3b74e79f9b1@gaisler.com>
Date: Fri, 13 Sep 2024 20:00:03 +0200
From: Andreas Larsson <andreas@...sler.com>
To: alexs@...nel.org, "David S . Miller" <davem@...emloft.net>,
 Christian Brauner <brauner@...nel.org>, sparclinux@...r.kernel.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arch/sparc: remove unused varible paddrbase in function
 leon_swprobe()

On 2024-07-29 08:49, alexs@...nel.org wrote:
> From: Alex Shi <alexs@...nel.org>
> 
> commit f22ed71cd602 ("sparc32,leon: SRMMU MMU Table probe fix") change
> return value from paddrbase to 'pte', but left the varible here.
> That causes a build warning for this varible, so we may remove it.
> 
> make --keep-going CROSS_COMPILE=/home/alexs/0day/gcc-14.1.0-nolibc/sparc-linux/bin/sparc-linux- --jobs=16 KCFLAGS= -Wtautological-compare -Wno-error=return-type -Wreturn-type -Wcast-function-type -funsigned-char -Wundef -fstrict-flex-arrays=3 -Wformat-overflow -Wformat-truncation -Wrestrict -Wenum-conversion W=1 O=sparc ARCH=sparc defconfig SHELL=/bin/bash arch/sparc/mm/ mm/ -s
> <stdin>:1519:2: warning: #warning syscall clone3 not implemented [-Wcpp]
> ../arch/sparc/mm/leon_mm.c: In function 'leon_swprobe':
> ../arch/sparc/mm/leon_mm.c:42:32: warning: variable 'paddrbase' set but not used [-Wunused-but-set-variable]
>    42 |         unsigned int lvl, pte, paddrbase;
>       |                                ^~~~~~~~~
> 
> Signed-off-by: Alex Shi <alexs@...nel.org>
> To: linux-kernel@...r.kernel.org
> To: sparclinux@...r.kernel.org
> To: Christian Brauner <brauner@...nel.org>
> To: Andreas Larsson <andreas@...sler.com>
> To: David S. Miller <davem@...emloft.net>
> ---
>  arch/sparc/mm/leon_mm.c | 8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)

Reviewed-by: Andreas Larsson <andreas@...sler.com>
Tested-by: Andreas Larsson <andreas@...sler.com>

Picking this up to my for-next.

Thanks,
Andreas