Hi Peter,
I add error case like below. Is this error case same to you said. Thank you very much!
diff --git a/tests/qemucapabilitiesdata/caps_7.1.0.x86_64.replies b/tests/qemucapabilitiesdata/caps_7.1.0.x86_64.replies
index 8444825cb7..c52b7917e2 100644
--- a/tests/qemucapabilitiesdata/caps_7.1.0.x86_64.replies
+++ b/tests/qemucapabilitiesdata/caps_7.1.0.x86_64.replies
@@ -33484,6 +33484,19 @@
}
}
+{
+ "execute": "query-sgx-capabilities",
+ "id": "libvirt-51"
+}
+
+{
+ "id": "libvirt-51",
+ "error": {
+ "class": "GenericError",
+ "desc": "SGX is not enabled in KVM"
+ }
+}
+
> -----Original Message-----
> From: Huang, Haibin
> Sent: Tuesday, May 17, 2022 10:16 AM
> To: Peter Krempa <pkrempa@xxxxxxxxxx>
> Cc: Yang, Lin A <lin.a.yang@xxxxxxxxx>; libvir-list@xxxxxxxxxx; Ding, Jian-
> feng <jian-feng.ding@xxxxxxxxx>; Zhong, Yang <yang.zhong@xxxxxxxxx>
> Subject: RE: [PATCH v11 1/4] qemu: provide support to query the SGX
> capability
>
>
>
> > -----Original Message-----
> > From: Peter Krempa <pkrempa@xxxxxxxxxx>
> > Sent: Monday, May 16, 2022 3:12 PM
> > To: Huang, Haibin <haibin.huang@xxxxxxxxx>
> > Cc: Yang, Lin A <lin.a.yang@xxxxxxxxx>; libvir-list@xxxxxxxxxx; Ding,
> > Jian- feng <jian-feng.ding@xxxxxxxxx>; Zhong, Yang
> > <yang.zhong@xxxxxxxxx>
> > Subject: Re: [PATCH v11 1/4] qemu: provide support to query
> > the SGX capability
> >
> > On Mon, May 16, 2022 at 01:47:35 +0000, Huang, Haibin wrote:
> > >
> > >
> > > > -----Original Message-----
> > > > From: Peter Krempa <pkrempa@xxxxxxxxxx>
> > > > Sent: Thursday, May 12, 2022 12:05 AM
> > > > To: Yang, Lin A <lin.a.yang@xxxxxxxxx>
> > > > Cc: libvir-list@xxxxxxxxxx; Huang, Haibin
> > > > <haibin.huang@xxxxxxxxx>; Ding, Jian-feng
> > > > <jian-feng.ding@xxxxxxxxx>; Zhong, Yang <yang.zhong@xxxxxxxxx>
> > > > Subject: Re: [PATCH v11 1/4] qemu: provide support to
> > > > query the SGX capability
> > > >
> > > > On Tue, May 10, 2022 at 23:11:09 -0700, Lin Yang wrote:
> > > > > From: Haibin Huang <haibin.huang@xxxxxxxxx>
> > > > >
> > > > > QEMU version >= 6.2.0 provides support for creating enclave on
> > > > > SGX
> > > > > x86 platform using Software Guard Extensions (SGX) feature.
> > > > > This patch adds support to query the SGX capability from the qemu.
> > > > >
> > > > > Signed-off-by: Haibin Huang <haibin.huang@xxxxxxxxx>
> > > > > ---
> > > > > src/conf/domain_capabilities.c | 10 ++
> > > > > src/conf/domain_capabilities.h | 13 ++
> > > > > src/libvirt_private.syms | 1 +
> > > > > src/qemu/qemu_capabilities.c | 119 ++++++++++++++++++
> > > > > src/qemu/qemu_capabilities.h | 6 +
> > > > > src/qemu/qemu_capspriv.h | 4 +
> > > > > src/qemu/qemu_monitor.c | 10 ++
> > > > > src/qemu/qemu_monitor.h | 3 +
> > > > > src/qemu/qemu_monitor_json.c | 104 +++++++++++++--
> > > > > src/qemu/qemu_monitor_json.h | 9 ++
> > > > > .../caps_6.2.0.x86_64.replies | 22 +++-
> > > > > .../caps_6.2.0.x86_64.xml | 5 +
> > > > > .../caps_7.0.0.x86_64.replies | 22 +++-
> > > > > .../caps_7.0.0.x86_64.xml | 5 +
> > > > > 14 files changed, 318 insertions(+), 15 deletions(-)
> > > >
> > > > This is not a full review. Couple of points:
> > > >
> > > > 1) Do not mix other changes with adding QEMU_CAPS* stuff
> > > > Basically theres waaay too much going on in this patch and it
> > > > definitely can be separated into smaller chunks. The QEMU_CAPS is
> > > > just one of them.
> > > > Separate at least:
> > > > - qemu monitor command introduction
> > > > - domain capabilities data structs for sgx
> > > > - parsing and formatting of the XML
> > > > - adding of the QEMU_CAPS_ flag
> > > [Haibin] may be "domain capabilities structs" should be put in "qemu
> > monitor command", because the virSGXCapability will be used by qemu
> > monitor command.
> > > > 2) caps for qemu-7.1 were added very recently
> > > > You'll need to fix that one too since you added an extra query. Make
> > > > sure that you _don't_ add the faking of SXG into that file, but
> > > > rather the error case. My box doesn't support SGX so it will be
> > > > overwritten in my next refresh anyways.
> > > [Haibin] Is this advice just for qemu-7.1 or all qemu version?
> >
> > For any non-released qemu, those capabilities are being re-generated,
> > thus any modifications will be overwritten.
> >
> > > This is just for unit test, why not add the faking of SGX into that
> > > file. If don't
> > add faking of SGX into that file, the unit can not pass.
> >
> > You can add fake caps into any capabilities for already released qemu.
> > Those are not being re-generated and thus any faked data will not be
> > deleted.
> [Haibin] ok, I got it, but I not familiar with the error case of the QEMU
> capabilities, could you give me a sample example. Thank you very much!
