Re: [PATCH rfcv4 01/13] tools: Secure guest check for Intel in virt-host-validate

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 06, 2024 at 10:27:39AM +0000, Duan, Zhenzhong wrote:
> 
> 
> >-----Original Message-----
> >From: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> >Subject: Re: [PATCH rfcv4 01/13] tools: Secure guest check for Intel in virt-
> >host-validate
> >
> >On Fri, May 24, 2024 at 02:21:16PM +0800, Zhenzhong Duan wrote:
> >> Add check in virt-host-validate for secure guest support
> >> on x86 for Intel Trust Domain Extentions.
> >>
> >> Suggested-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> >> Signed-off-by: Zhenzhong Duan <zhenzhong.duan@xxxxxxxxx>
> >> ---
> >>  tools/virt-host-validate-common.c | 22 +++++++++++++++++++++-
> >>  tools/virt-host-validate-common.h |  1 +
> >>  2 files changed, 22 insertions(+), 1 deletion(-)
> >>
> >> diff --git a/tools/virt-host-validate-common.c b/tools/virt-host-validate-
> >common.c
> >> index c8a23e2e99..4f0698b8ce 100644
> >> --- a/tools/virt-host-validate-common.c
> >> +++ b/tools/virt-host-validate-common.c
> >> @@ -43,7 +43,8 @@ VIR_ENUM_IMPL(virHostValidateCPUFlag,
> >>                "svm",
> >>                "sie",
> >>                "158",
> >> -              "sev");
> >> +              "sev",
> >> +              "tdx_host_platform");
> >
> >I don't think we need to be quiet so verbose here. I think it is
> >sufficient to just call it 'tdx'.
> 
> This string is used to compare with output of /proc/cpuinfo.
> So we can't use 'tdx', or else string compare will fail.

Ah yes, nevermind then.

Reviewed-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>


With regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux