Re: [kvm-unit-tests PATCH] travis.yml: Run 32-bit tests with KVM, too

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

 



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@xxxxxxxxxx>
>>> ---
>>>  .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.




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux