> -----Original Message----- > From: Daniel P. Berrangé <berrange@xxxxxxxxxx> > Sent: Tuesday, September 28, 2021 10:15 PM > To: Huang, Haibin <haibin.huang@xxxxxxxxx> > Cc: libvir-list@xxxxxxxxxx; Ding, Jian-feng <jian-feng.ding@xxxxxxxxx>; Yang, > Lin A <lin.a.yang@xxxxxxxxx>; Lu, Lianhao <lianhao.lu@xxxxxxxxx>; > pbonzini@xxxxxxxxxx; pkrempa@xxxxxxxxxx; twiederh@xxxxxxxxxx; > phrdina@xxxxxxxxxx; mprivozn@xxxxxxxxxx > Subject: Re: [PATCH v7 5/5] Add get domaincaps unit test > > On Wed, Sep 08, 2021 at 09:15:58AM +0800, Haibin Huang wrote: > > Signed-off-by: Haibin Huang <haibin.huang@xxxxxxxxx> > > --- > > tests/domaincapsdata/bhyve_basic.x86_64.xml | 1 + > > tests/domaincapsdata/bhyve_fbuf.x86_64.xml | 1 + > > tests/domaincapsdata/bhyve_uefi.x86_64.xml | 1 + > > tests/domaincapsdata/empty.xml | 1 + > > tests/domaincapsdata/libxl-xenfv.xml | 1 + > > tests/domaincapsdata/libxl-xenpv.xml | 1 + > > tests/domaincapsdata/qemu_2.11.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.11.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.11.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_2.11.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.12.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.12.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.12.0-virt.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_2.12.0.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_2.12.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_2.12.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_2.12.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.4.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.4.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.4.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.5.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.5.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.5.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.6.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.6.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.6.0-virt.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_2.6.0.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_2.6.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_2.6.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.7.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.7.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.7.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_2.7.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.8.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.8.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.8.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_2.8.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.9.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.9.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_2.9.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_2.9.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_2.9.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_3.0.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_3.0.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_3.0.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_3.0.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_3.0.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_3.1.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_3.1.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_3.1.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_3.1.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.0.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.0.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.0.0-virt.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_4.0.0.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_4.0.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_4.0.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_4.0.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.1.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.1.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.1.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.2.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.2.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_4.2.0-virt.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_4.2.0.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_4.2.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_4.2.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_4.2.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.0.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.0.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.0.0-virt.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_5.0.0.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_5.0.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_5.0.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.1.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.1.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.1.0.sparc.xml | 1 + > > tests/domaincapsdata/qemu_5.1.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.2.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.2.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_5.2.0-virt.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_5.2.0.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_5.2.0.ppc64.xml | 1 + > > tests/domaincapsdata/qemu_5.2.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_5.2.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_6.0.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_6.0.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_6.0.0-virt.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_6.0.0.aarch64.xml | 1 + > > tests/domaincapsdata/qemu_6.0.0.s390x.xml | 1 + > > tests/domaincapsdata/qemu_6.0.0.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_6.1.0-q35.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_6.1.0-tcg.x86_64.xml | 1 + > > tests/domaincapsdata/qemu_6.1.0.x86_64.xml | 1 + > > 94 files changed, 94 insertions(+) > > I'm pretty sure this would be needed in the same patch that wires up domain > capabilities to report SGX. > > I'd expect to also see an update of tests/qemucapabilitiesdata in the patch that > probes the QEMU feature in qemu_capabilities.c > I totally agree to sperate two patches, one for probe the qemu capabilities, the other for adding domain capability and adding corresponding unit test. just double confirm the unit test should be put in the same patch with function implementation? > > 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 :|