On Thu, 8 Aug 2019 at 20:40, Andrew Jones drjones@redhat.com wrote:
On Thu, Aug 08, 2019 at 01:31:40PM +0100, Naresh Kamboju wrote:
selftests kvm all test cases need pre-required kernel configs for the tests to get pass.
The KVM tests are skipped without these configs:
dev_fd = open(KVM_DEV_PATH, O_RDONLY); if (dev_fd < 0) exit(KSFT_SKIP);Signed-off-by: Naresh Kamboju naresh.kamboju@linaro.org
tools/testing/selftests/kvm/config | 3 +++ 1 file changed, 3 insertions(+) create mode 100644 tools/testing/selftests/kvm/config
diff --git a/tools/testing/selftests/kvm/config b/tools/testing/selftests/kvm/config new file mode 100644 index 000000000000..63ed533f73d6 --- /dev/null +++ b/tools/testing/selftests/kvm/config @@ -0,0 +1,3 @@ +CONFIG_KVM=y +CONFIG_KVM_INTEL=y
+CONFIG_KVM_AMD=y
2.17.1
What does the kselftests config file do? I was about to complain that this would break compiling on non-x86 platforms, but 'make kselftest' and other forms of invoking the build work fine on aarch64 even with this config file. So is this just for documentation? If so, then its still obviously wrong for non-x86 platforms. The only config that makes sense here is KVM. If the other options need to be documented for x86, then should they get an additional config file? tools/testing/selftests/kvm/x86_64/config?
Addressed your comments and sent out v3 patch. Thank for the review.
- Naresh