Received-SPF: pass (google.com: domain of linux-kernel+bounces-58155-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Precedence: bulk
MIME-Version: 1.0
References: <cover.1705965634.git.isaku.yamahata@intel.com> <88bcf53760c42dafb14cd9a92bf4f9243f597bbe.1705965634.git.isaku.yamahata@intel.com>
In-Reply-To: <88bcf53760c42dafb14cd9a92bf4f9243f597bbe.1705965634.git.isaku.yamahata@intel.com>
From: Paolo Bonzini <pbonzini@redhat.com>
Date: Thu, 8 Feb 2024 14:30:40 +0100
Message-ID: <CABgObfYo0OaSXUYjQbn188y8JOAZGzD56TabdyENXAW6_Ca0Hw@mail.gmail.com>
Subject: Re: [PATCH v18 002/121] x86/virt/tdx: Export SEAMCALL functions
To: isaku.yamahata@intel.com
Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, 
	isaku.yamahata@gmail.com, erdemaktas@google.com, 
	Sean Christopherson <seanjc@google.com>, Sagi Shahar <sagis@google.com>, Kai Huang <kai.huang@intel.com>, 
	chen.bo@intel.com, hang.yuan@intel.com, tina.zhang@intel.com
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Jan 23, 2024 at 12:54=E2=80=AFAM <isaku.yamahata@intel.com> wrote:
>
> From: Kai Huang <kai.huang@intel.com>
>
> KVM will need to make SEAMCALLs to create and run TDX guests.  Export
> SEAMCALL functions for KVM to use.
>
> Also add declaration of SEAMCALL functions to <asm/asm-prototypes.h> to
> support CONFIG_MODVERSIONS=3Dy.
>
> Signed-off-by: Kai Huang <kai.huang@intel.com>

I thought you were going to introduce builtin functions for SEAMCALLs
needed by KVM, instead?

In any case, missing Signed-off-by.

Paolo

> ---
>  arch/x86/include/asm/asm-prototypes.h | 1 +
>  arch/x86/virt/vmx/tdx/seamcall.S      | 4 ++++
>  2 files changed, 5 insertions(+)
>
> diff --git a/arch/x86/include/asm/asm-prototypes.h b/arch/x86/include/asm=
/asm-prototypes.h
> index b1a98fa38828..0ec572ad75f1 100644
> --- a/arch/x86/include/asm/asm-prototypes.h
> +++ b/arch/x86/include/asm/asm-prototypes.h
> @@ -13,6 +13,7 @@
>  #include <asm/preempt.h>
>  #include <asm/asm.h>
>  #include <asm/gsseg.h>
> +#include <asm/tdx.h>
>
>  #ifndef CONFIG_X86_CMPXCHG64
>  extern void cmpxchg8b_emu(void);
> diff --git a/arch/x86/virt/vmx/tdx/seamcall.S b/arch/x86/virt/vmx/tdx/sea=
mcall.S
> index 5b1f2286aea9..e32cf82ed47e 100644
> --- a/arch/x86/virt/vmx/tdx/seamcall.S
> +++ b/arch/x86/virt/vmx/tdx/seamcall.S
> @@ -1,5 +1,6 @@
>  /* SPDX-License-Identifier: GPL-2.0 */
>  #include <linux/linkage.h>
> +#include <linux/export.h>
>  #include <asm/frame.h>
>
>  #include "tdxcall.S"
> @@ -21,6 +22,7 @@
>  SYM_FUNC_START(__seamcall)
>         TDX_MODULE_CALL host=3D1
>  SYM_FUNC_END(__seamcall)
> +EXPORT_SYMBOL_GPL(__seamcall);
>
>  /*
>   * __seamcall_ret() - Host-side interface functions to SEAM software
> @@ -40,6 +42,7 @@ SYM_FUNC_END(__seamcall)
>  SYM_FUNC_START(__seamcall_ret)
>         TDX_MODULE_CALL host=3D1 ret=3D1
>  SYM_FUNC_END(__seamcall_ret)
> +EXPORT_SYMBOL_GPL(__seamcall_ret);
>
>  /*
>   * __seamcall_saved_ret() - Host-side interface functions to SEAM softwa=
re
> @@ -59,3 +62,4 @@ SYM_FUNC_END(__seamcall_ret)
>  SYM_FUNC_START(__seamcall_saved_ret)
>         TDX_MODULE_CALL host=3D1 ret=3D1 saved=3D1
>  SYM_FUNC_END(__seamcall_saved_ret)
> +EXPORT_SYMBOL_GPL(__seamcall_saved_ret);
> --
> 2.25.1
>