linux-kernel - Re: [tip: perf/core] perf: Use EXPORT_SYMBOL_FOR

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: <aUVn7tPw8EaJS-d7@google.com>
Date: Fri, 19 Dec 2025 06:57:50 -0800
From: Sean Christopherson <seanjc@...gle.com>
To: Peter Zijlstra <peterz@...radead.org>
Cc: linux-kernel@...r.kernel.org, sfr@...b.auug.org.au, 
	linux-tip-commits@...r.kernel.org, x86@...nel.org, pbonzini@...hat.com, 
	kvm@...r.kernel.org
Subject: Re: [tip: perf/core] perf: Use EXPORT_SYMBOL_FOR_KVM() for the
 mediated APIs

On Fri, Dec 19, 2025, Peter Zijlstra wrote:
> On Thu, Dec 18, 2025 at 10:40:51AM -0800, Sean Christopherson wrote:
> 
> 
> > Include the arch-defined asm/kvm_types.h if and only if the kernel is
> > being compiled for an architecture that supports KVM so that kvm_types.h
> > can be included in generic code without having to guard _those_ includes,
> > and without having to add "generic-y += kvm_types.h" for all architectures
> > that don't support KVM.
> 
> Something jogged my brain and the below seems to work for the few
> architectures I've tried. Let me update the patch and see if the build
> robot still finds fail.

Nice!  Works on my end as well.  Just when I think I've learned most of the
build system's tricks...

> ---
> diff --git a/include/asm-generic/Kbuild b/include/asm-generic/Kbuild
> index 295c94a3ccc1..9aff61e7b8f2 100644
> --- a/include/asm-generic/Kbuild
> +++ b/include/asm-generic/Kbuild
> @@ -32,6 +32,7 @@ mandatory-y += irq_work.h
>  mandatory-y += kdebug.h
>  mandatory-y += kmap_size.h
>  mandatory-y += kprobes.h
> +mandatory-y += kvm_types.h
>  mandatory-y += linkage.h
>  mandatory-y += local.h
>  mandatory-y += local64.h