linux-kernel - ./include/asm-generic/tlb.h:629:10: error: parameter 'ptep' set but not used

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

Date: Tue, 16 Apr 2024 12:26:57 +0200
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: david@...hat.com, Arnd Bergmann <arnd@...db.de>, Huacai Chen <chenhuacai@...nel.org>, 
	WANG Xuerui <kernel@...0n.name>
Cc: Andrew Morton <akpm@...ux-foundation.org>, Ryan Roberts <ryan.roberts@....com>, 
	linux-arch <linux-arch@...r.kernel.org>, Linux-MM <linux-mm@...ck.org>, 
	linux-kernel <linux-kernel@...r.kernel.org>, loongarch@...ts.linux.dev, 
	clang-built-linux <llvm@...ts.linux.dev>
Subject: ./include/asm-generic/tlb.h:629:10: error: parameter 'ptep' set but
 not used

Hi David, Arnd, LoongArch,

In a linux-next defconfig LLVM=1 build today I got:

    ./include/asm-generic/tlb.h:629:10: error: parameter 'ptep' set
but not used [-Werror,-Wunused-but-set-parameter]
      629 |                 pte_t *ptep, unsigned int nr, unsigned long address)
          |                        ^

Indeed, in loongarch, `__tlb_remove_tlb_entry` does not do anything.
This seems the same that Arnd reported for arm64:

    https://lore.kernel.org/all/20240221154549.2026073-1-arnd@kernel.org/

So perhaps the loongarch's one should also be changed into an static inline?

I hope that helps!

Cheers,
Miguel