Re: [PATCH v2] tests/functional: add hypervisor test for aarch64

["Alex Bennée" <alex.bennee@linaro.org>]

Philippe Mathieu-Daudé <philmd@linaro.org> writes:

> From: Alex Bennée <alex.bennee@linaro.org>
>
> This is a simple test case that runs an image with kvmtool and
> kvm-unit-tests which can validate virtualisation works. This is useful
> for exercising TCG but can also be applied to any nested virt setup
> which is why it doesn't specify an accelerator.

You might want to update this text give now...

<snip>
> +
> +    def do_test(self, kvm_mode='nvhe', accel='tcg'):
> +        self.require_accelerator(accel)

You are now forcing tcg. Or are you planning on adding additional tests
bellow to exercise split and KVM (when we have nested support for
HVF/KVM)?

> +        self._launch_guest(kvm_mode)
> +        self._run_kut(kvm_mode)
> +
> +    def test_aarch64_vhe_selftest_tcg(self):
> +        self.do_test('vhe')
> +
> +    def test_aarch64_nvhe_selftest_tcg(self):
> +        self.do_test('nvhe')
> +
> +if __name__ == '__main__':
> +    LinuxKernelTest.main()

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro