Date:   Tue, 28 Feb 2023 16:45:39 -0800
From:   Isaku Yamahata <isaku.yamahata@gmail.com>
To:     Xiaoyao Li <xiaoyao.li@intel.com>
Cc:     isaku.yamahata@intel.com, kvm@vger.kernel.org,
        linux-kernel@vger.kernel.org, isaku.yamahata@gmail.com,
        Paolo Bonzini <pbonzini@redhat.com>, erdemaktas@google.com,
        Sean Christopherson <seanjc@google.com>,
        Sagi Shahar <sagis@google.com>,
        David Matlack <dmatlack@google.com>,
        Kai Huang <kai.huang@intel.com>,
        Zhi Wang <zhi.wang.linux@gmail.com>
Subject: Re: [PATCH v12 000/106] KVM TDX basic feature support
Message-ID: <20230301004539.GC3852341@ls.amr.corp.intel.com>
References: <cover.1677484918.git.isaku.yamahata@intel.com>
 <7ec69b6b-540c-95d1-079d-8527fabd440b@intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <7ec69b6b-540c-95d1-079d-8527fabd440b@intel.com>
Precedence: bulk

On Tue, Feb 28, 2023 at 04:03:12PM +0800,
Xiaoyao Li <xiaoyao.li@intel.com> wrote:

> On 2/27/2023 4:21 PM, isaku.yamahata@intel.com wrote:
> > KVM TDX basic feature support
> > 
> > Hello.  This is v12 the patch series vof KVM TDX support.  This is based on
> > v6.2-rc7 + the following patch series.
> > 
> > Related patch series This patch is based on:
> > - fd-based approach for supporing KVM v10 and
> >    https://lore.kernel.org/lkml/20221202061347.1070246-1-chao.p.peng@linux.intel.com/
> >    git@github.com:sean-jc/linux.git  x86/upm_base_support
> > - TDX host kernel support v9
> >    https://lore.kernel.org/lkml/e1c717b36033caf8de9d7a448e06eff88ed0c9ae.camel@intel.com/T/#m2562c7f18eabae171793ed7ea81b3544d7af09c4
> 
> 
>  9fdabaaf0346 KVM: TDX: Make TDX VM type supported
>  fa6b8135913c KVM: x86: Introduce vm_type to differentiate default VMs from
> confidential VMs
>  6dbc1fdfb61f KVM: TDX: Add placeholders for TDX VM/vcpu structure
>  64a9c956f265 KVM: TDX: Initialize the TDX module when loading the KVM intel
> kernel module
>  bacb6554fcf5 KVM: x86/vmx: Refactor KVM VMX module init/exit functions
>  b17eee6a7d53 KVM: VMX: Move out vmx_x86_ops to 'main.c' to wrap VMX and TDX
>  5a8b8d63a2e8 x86/mtrr: Mask out keyid bits from variable mtrr mask register
>  29017e2c04cc KVM: arm64: Mark kvm_arm_init() and its unique descendants as
> __init
> 
> 
> I find above 8 patches not included in this patch series after TDX host
> kernel support series. Some of them were included in v11 before, and I think
> they are important to make the patch series intact. What's the reason for
> not including them?

Thanks for pointing it out. My bad. I wrongly specified the start of patch
series.  Those should be included.
-- 
Isaku Yamahata <isaku.yamahata@gmail.com>