| Message ID | 20191205170439.11607-1-thuth@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [kvm-unit-tests] travis.yml: Run 32-bit tests with KVM, too | expand |
On 05/12/19 18:04, Thomas Huth wrote: > KVM works on Travis in 32-bit, too, so we can enable more tests there. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > .travis.yml | 10 +++++++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/.travis.yml b/.travis.yml > index 4162366..75bcf08 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -34,15 +34,19 @@ matrix: > env: > - CONFIG="--arch=i386" > - BUILD_DIR="." > - - TESTS="eventinj port80 sieve tsc taskswitch umip vmexit_ple_round_robin" > + - TESTS="asyncpf hyperv_stimer hyperv_synic kvmclock_test msr pmu realmode > + s3 sieve smap smptest smptest3 taskswitch taskswitch2 tsc_adjust" > + - ACCEL="kvm" > > - addons: > apt_packages: gcc gcc-multilib qemu-system-x86 > env: > - CONFIG="--arch=i386" > - BUILD_DIR="i386-builddir" > - - TESTS="vmexit_mov_from_cr8 vmexit_ipi vmexit_ipi_halt vmexit_mov_to_cr8 > - vmexit_cpuid vmexit_tscdeadline vmexit_tscdeadline_immed" > + - TESTS="tsx-ctrl umip vmexit_cpuid vmexit_ipi vmexit_ipi_halt > + vmexit_mov_from_cr8 vmexit_mov_to_cr8 vmexit_ple_round_robin > + vmexit_tscdeadline vmexit_tscdeadline_immed vmexit_vmcall" > + - ACCEL="kvm" > > - addons: > apt_packages: gcc-arm-linux-gnueabihf qemu-system-arm > Applied, thanks. But there are also some 32-bit specific tests (taskswitch, taskswitch2, cmpxchg8b) that we may want to add. Paolo
On 09/12/2019 18.07, Paolo Bonzini wrote: > On 05/12/19 18:04, Thomas Huth wrote: >> KVM works on Travis in 32-bit, too, so we can enable more tests there. >> >> Signed-off-by: Thomas Huth <thuth@redhat.com> >> --- >> .travis.yml | 10 +++++++--- >> 1 file changed, 7 insertions(+), 3 deletions(-) >> >> diff --git a/.travis.yml b/.travis.yml >> index 4162366..75bcf08 100644 >> --- a/.travis.yml >> +++ b/.travis.yml >> @@ -34,15 +34,19 @@ matrix: >> env: >> - CONFIG="--arch=i386" >> - BUILD_DIR="." >> - - TESTS="eventinj port80 sieve tsc taskswitch umip vmexit_ple_round_robin" >> + - TESTS="asyncpf hyperv_stimer hyperv_synic kvmclock_test msr pmu realmode >> + s3 sieve smap smptest smptest3 taskswitch taskswitch2 tsc_adjust" taskswitch and taskswitch2 are here ----------------^ >> + - ACCEL="kvm" >> >> - addons: >> apt_packages: gcc gcc-multilib qemu-system-x86 >> env: >> - CONFIG="--arch=i386" >> - BUILD_DIR="i386-builddir" >> - - TESTS="vmexit_mov_from_cr8 vmexit_ipi vmexit_ipi_halt vmexit_mov_to_cr8 >> - vmexit_cpuid vmexit_tscdeadline vmexit_tscdeadline_immed" >> + - TESTS="tsx-ctrl umip vmexit_cpuid vmexit_ipi vmexit_ipi_halt >> + vmexit_mov_from_cr8 vmexit_mov_to_cr8 vmexit_ple_round_robin >> + vmexit_tscdeadline vmexit_tscdeadline_immed vmexit_vmcall" >> + - ACCEL="kvm" >> >> - addons: >> apt_packages: gcc-arm-linux-gnueabihf qemu-system-arm >> > > Applied, thanks. But there are also some 32-bit specific tests > (taskswitch, taskswitch2, cmpxchg8b) that we may want to add. cmpxchg8b seems to be missing in x86/unittests.cfg ... so I think it should be added there first? Thomas
On 09/12/19 18:14, Thomas Huth wrote: > On 09/12/2019 18.07, Paolo Bonzini wrote: >> On 05/12/19 18:04, Thomas Huth wrote: >>> KVM works on Travis in 32-bit, too, so we can enable more tests there. >>> >>> Signed-off-by: Thomas Huth <thuth@redhat.com> >>> --- >>> .travis.yml | 10 +++++++--- >>> 1 file changed, 7 insertions(+), 3 deletions(-) >>> >>> diff --git a/.travis.yml b/.travis.yml >>> index 4162366..75bcf08 100644 >>> --- a/.travis.yml >>> +++ b/.travis.yml >>> @@ -34,15 +34,19 @@ matrix: >>> env: >>> - CONFIG="--arch=i386" >>> - BUILD_DIR="." >>> - - TESTS="eventinj port80 sieve tsc taskswitch umip vmexit_ple_round_robin" >>> + - TESTS="asyncpf hyperv_stimer hyperv_synic kvmclock_test msr pmu realmode >>> + s3 sieve smap smptest smptest3 taskswitch taskswitch2 tsc_adjust" > > taskswitch and taskswitch2 are here ----------------^ You're right, but I'm confused: what are the two separate configurations for? Worth a comment? >>> + - ACCEL="kvm" >>> >>> - addons: >>> apt_packages: gcc gcc-multilib qemu-system-x86 >>> env: >>> - CONFIG="--arch=i386" >>> - BUILD_DIR="i386-builddir" >>> - - TESTS="vmexit_mov_from_cr8 vmexit_ipi vmexit_ipi_halt vmexit_mov_to_cr8 >>> - vmexit_cpuid vmexit_tscdeadline vmexit_tscdeadline_immed" >>> + - TESTS="tsx-ctrl umip vmexit_cpuid vmexit_ipi vmexit_ipi_halt >>> + vmexit_mov_from_cr8 vmexit_mov_to_cr8 vmexit_ple_round_robin >>> + vmexit_tscdeadline vmexit_tscdeadline_immed vmexit_vmcall" >>> + - ACCEL="kvm" >>> >>> - addons: >>> apt_packages: gcc-arm-linux-gnueabihf qemu-system-arm >>> >> >> Applied, thanks. But there are also some 32-bit specific tests >> (taskswitch, taskswitch2, cmpxchg8b) that we may want to add. > > cmpxchg8b seems to be missing in x86/unittests.cfg ... so I think it > should be added there first? Good idea.
On 09/12/2019 18.42, Paolo Bonzini wrote: > On 09/12/19 18:14, Thomas Huth wrote: >> On 09/12/2019 18.07, Paolo Bonzini wrote: >>> On 05/12/19 18:04, Thomas Huth wrote: >>>> KVM works on Travis in 32-bit, too, so we can enable more tests there. >>>> >>>> Signed-off-by: Thomas Huth <thuth@redhat.com> >>>> --- >>>> .travis.yml | 10 +++++++--- >>>> 1 file changed, 7 insertions(+), 3 deletions(-) >>>> >>>> diff --git a/.travis.yml b/.travis.yml >>>> index 4162366..75bcf08 100644 >>>> --- a/.travis.yml >>>> +++ b/.travis.yml >>>> @@ -34,15 +34,19 @@ matrix: >>>> env: >>>> - CONFIG="--arch=i386" >>>> - BUILD_DIR="." >>>> - - TESTS="eventinj port80 sieve tsc taskswitch umip vmexit_ple_round_robin" >>>> + - TESTS="asyncpf hyperv_stimer hyperv_synic kvmclock_test msr pmu realmode >>>> + s3 sieve smap smptest smptest3 taskswitch taskswitch2 tsc_adjust" >> >> taskswitch and taskswitch2 are here ----------------^ > > You're right, but I'm confused: what are the two separate configurations > for? Worth a comment? For all architectures we've got one entry for in-tree builds and one in-tree entry for out-of-tree builds. Since we've got these two entries anyway, I simply split up the set of tests to speed up the CI process a little bit. Thomas
diff --git a/.travis.yml b/.travis.yml index 4162366..75bcf08 100644 --- a/.travis.yml +++ b/.travis.yml @@ -34,15 +34,19 @@ matrix: env: - CONFIG="--arch=i386" - BUILD_DIR="." - - TESTS="eventinj port80 sieve tsc taskswitch umip vmexit_ple_round_robin" + - TESTS="asyncpf hyperv_stimer hyperv_synic kvmclock_test msr pmu realmode + s3 sieve smap smptest smptest3 taskswitch taskswitch2 tsc_adjust" + - ACCEL="kvm" - addons: apt_packages: gcc gcc-multilib qemu-system-x86 env: - CONFIG="--arch=i386" - BUILD_DIR="i386-builddir" - - TESTS="vmexit_mov_from_cr8 vmexit_ipi vmexit_ipi_halt vmexit_mov_to_cr8 - vmexit_cpuid vmexit_tscdeadline vmexit_tscdeadline_immed" + - TESTS="tsx-ctrl umip vmexit_cpuid vmexit_ipi vmexit_ipi_halt + vmexit_mov_from_cr8 vmexit_mov_to_cr8 vmexit_ple_round_robin + vmexit_tscdeadline vmexit_tscdeadline_immed vmexit_vmcall" + - ACCEL="kvm" - addons: apt_packages: gcc-arm-linux-gnueabihf qemu-system-arm
KVM works on Travis in 32-bit, too, so we can enable more tests there. Signed-off-by: Thomas Huth <thuth@redhat.com> --- .travis.yml | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-)