[PATCH] pseries: disable migration-test if /dev/kvm cannot be used

Laurent Vivier posted 1 patch 4 years, 4 months ago
Test asan passed
Test checkpatch passed
Test FreeBSD passed
Test docker-mingw@fedora passed
Test docker-clang@ubuntu passed
Test docker-quick@centos7 passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20191120170955.242900-1-lvivier@redhat.com
Maintainers: Laurent Vivier <lvivier@redhat.com>, "Dr. David Alan Gilbert" <dgilbert@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Thomas Huth <thuth@redhat.com>, Juan Quintela <quintela@redhat.com>
tests/migration-test.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
[PATCH] pseries: disable migration-test if /dev/kvm cannot be used
Posted by Laurent Vivier 4 years, 4 months ago
On ppc64, migration-test only works with kvm_hv, and we already
have a check to verify the module is loaded.

kvm_hv module can be loaded in memory and /sys/module/kvm_hv exists,
but on some systems (like build systems) /dev/kvm can be missing
(by administrators choice).

And as kvm_hv exists test-migration is started but QEMU falls back to
TCG because it cannot be used:

    Could not access KVM kernel module: No such file or directory
    failed to initialize KVM: No such file or directory
    Back to tcg accelerator

And as the test is done with TCG, it fails.

As for s390x, we must check for the existence and the access rights
of /dev/kvm.

Reported-by: Cole Robinson <crobinso@redhat.com>
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
---
 tests/migration-test.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tests/migration-test.c b/tests/migration-test.c
index ac780dffdaad..2b25ba6d77f6 100644
--- a/tests/migration-test.c
+++ b/tests/migration-test.c
@@ -1349,7 +1349,8 @@ int main(int argc, char **argv)
      * some reason)
      */
     if (g_str_equal(qtest_get_arch(), "ppc64") &&
-        access("/sys/module/kvm_hv", F_OK)) {
+        (access("/sys/module/kvm_hv", F_OK) ||
+         access("/dev/kvm", R_OK | W_OK))) {
         g_test_message("Skipping test: kvm_hv not available");
         return g_test_run();
     }
-- 
2.23.0


Re: [PATCH] pseries: disable migration-test if /dev/kvm cannot be used
Posted by Thomas Huth 4 years, 4 months ago
On 20/11/2019 18.09, Laurent Vivier wrote:
> On ppc64, migration-test only works with kvm_hv, and we already
> have a check to verify the module is loaded.
> 
> kvm_hv module can be loaded in memory and /sys/module/kvm_hv exists,
> but on some systems (like build systems) /dev/kvm can be missing
> (by administrators choice).
> 
> And as kvm_hv exists test-migration is started but QEMU falls back to
> TCG because it cannot be used:
> 
>     Could not access KVM kernel module: No such file or directory
>     failed to initialize KVM: No such file or directory
>     Back to tcg accelerator
> 
> And as the test is done with TCG, it fails.
> 
> As for s390x, we must check for the existence and the access rights
> of /dev/kvm.
> 
> Reported-by: Cole Robinson <crobinso@redhat.com>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
>  tests/migration-test.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/migration-test.c b/tests/migration-test.c
> index ac780dffdaad..2b25ba6d77f6 100644
> --- a/tests/migration-test.c
> +++ b/tests/migration-test.c
> @@ -1349,7 +1349,8 @@ int main(int argc, char **argv)
>       * some reason)
>       */
>      if (g_str_equal(qtest_get_arch(), "ppc64") &&
> -        access("/sys/module/kvm_hv", F_OK)) {
> +        (access("/sys/module/kvm_hv", F_OK) ||
> +         access("/dev/kvm", R_OK | W_OK))) {
>          g_test_message("Skipping test: kvm_hv not available");
>          return g_test_run();
>      }
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>


Re: [PATCH] pseries: disable migration-test if /dev/kvm cannot be used
Posted by Greg Kurz 4 years, 4 months ago
On Wed, 20 Nov 2019 18:09:55 +0100
Laurent Vivier <lvivier@redhat.com> wrote:

> On ppc64, migration-test only works with kvm_hv, and we already
> have a check to verify the module is loaded.
> 
> kvm_hv module can be loaded in memory and /sys/module/kvm_hv exists,
> but on some systems (like build systems) /dev/kvm can be missing
> (by administrators choice).
> 
> And as kvm_hv exists test-migration is started but QEMU falls back to
> TCG because it cannot be used:
> 
>     Could not access KVM kernel module: No such file or directory
>     failed to initialize KVM: No such file or directory
>     Back to tcg accelerator
> 
> And as the test is done with TCG, it fails.
> 
> As for s390x, we must check for the existence and the access rights
> of /dev/kvm.
> 
> Reported-by: Cole Robinson <crobinso@redhat.com>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---

Reviewed-by: Greg Kurz <groug@kaod.org>

>  tests/migration-test.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/migration-test.c b/tests/migration-test.c
> index ac780dffdaad..2b25ba6d77f6 100644
> --- a/tests/migration-test.c
> +++ b/tests/migration-test.c
> @@ -1349,7 +1349,8 @@ int main(int argc, char **argv)
>       * some reason)
>       */
>      if (g_str_equal(qtest_get_arch(), "ppc64") &&
> -        access("/sys/module/kvm_hv", F_OK)) {
> +        (access("/sys/module/kvm_hv", F_OK) ||
> +         access("/dev/kvm", R_OK | W_OK))) {
>          g_test_message("Skipping test: kvm_hv not available");
>          return g_test_run();
>      }


Re: [PATCH] pseries: disable migration-test if /dev/kvm cannot be used
Posted by Juan Quintela 4 years, 4 months ago
Laurent Vivier <lvivier@redhat.com> wrote:
> On ppc64, migration-test only works with kvm_hv, and we already
> have a check to verify the module is loaded.
>
> kvm_hv module can be loaded in memory and /sys/module/kvm_hv exists,
> but on some systems (like build systems) /dev/kvm can be missing
> (by administrators choice).
>
> And as kvm_hv exists test-migration is started but QEMU falls back to
> TCG because it cannot be used:
>
>     Could not access KVM kernel module: No such file or directory
>     failed to initialize KVM: No such file or directory
>     Back to tcg accelerator
>
> And as the test is done with TCG, it fails.
>
> As for s390x, we must check for the existence and the access rights
> of /dev/kvm.
>
> Reported-by: Cole Robinson <crobinso@redhat.com>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>

Reviewed-by: Juan Quintela <quintela@redhat.com>

Oh, why it is so difficult!!!

Thanks, Juan.


Re: [PATCH] pseries: disable migration-test if /dev/kvm cannot be used
Posted by Laurent Vivier 4 years, 4 months ago
On 21/11/2019 08:18, Juan Quintela wrote:
> Laurent Vivier <lvivier@redhat.com> wrote:
>> On ppc64, migration-test only works with kvm_hv, and we already
>> have a check to verify the module is loaded.
>>
>> kvm_hv module can be loaded in memory and /sys/module/kvm_hv exists,
>> but on some systems (like build systems) /dev/kvm can be missing
>> (by administrators choice).
>>
>> And as kvm_hv exists test-migration is started but QEMU falls back to
>> TCG because it cannot be used:
>>
>>     Could not access KVM kernel module: No such file or directory
>>     failed to initialize KVM: No such file or directory
>>     Back to tcg accelerator
>>
>> And as the test is done with TCG, it fails.
>>
>> As for s390x, we must check for the existence and the access rights
>> of /dev/kvm.
>>
>> Reported-by: Cole Robinson <crobinso@redhat.com>
>> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> 
> Reviewed-by: Juan Quintela <quintela@redhat.com>
> 
> Oh, why it is so difficult!!!
> 
> Thanks, Juan.
> 

David,

could you add this one too to your ppc-for-4.2 queue?

Thanks,
Laurent


Re: [PATCH] pseries: disable migration-test if /dev/kvm cannot be used
Posted by Juan Quintela 4 years, 4 months ago
Laurent Vivier <lvivier@redhat.com> wrote:
> On 21/11/2019 08:18, Juan Quintela wrote:
>> Laurent Vivier <lvivier@redhat.com> wrote:
>>> On ppc64, migration-test only works with kvm_hv, and we already
>>> have a check to verify the module is loaded.
>>>
>>> kvm_hv module can be loaded in memory and /sys/module/kvm_hv exists,
>>> but on some systems (like build systems) /dev/kvm can be missing
>>> (by administrators choice).
>>>
>>> And as kvm_hv exists test-migration is started but QEMU falls back to
>>> TCG because it cannot be used:
>>>
>>>     Could not access KVM kernel module: No such file or directory
>>>     failed to initialize KVM: No such file or directory
>>>     Back to tcg accelerator
>>>
>>> And as the test is done with TCG, it fails.
>>>
>>> As for s390x, we must check for the existence and the access rights
>>> of /dev/kvm.
>>>
>>> Reported-by: Cole Robinson <crobinso@redhat.com>
>>> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
>> 
>> Reviewed-by: Juan Quintela <quintela@redhat.com>
>> 
>> Oh, why it is so difficult!!!
>> 
>> Thanks, Juan.
>> 
>
> David,
>
> could you add this one too to your ppc-for-4.2 queue?

I was about to ask if you preffered it to go through the migration tree.

David?

Later, Juan.


Re: [PATCH] pseries: disable migration-test if /dev/kvm cannot be used
Posted by David Gibson 4 years, 4 months ago
On Thu, Nov 21, 2019 at 09:30:27AM +0100, Juan Quintela wrote:
> Laurent Vivier <lvivier@redhat.com> wrote:
> > On 21/11/2019 08:18, Juan Quintela wrote:
> >> Laurent Vivier <lvivier@redhat.com> wrote:
> >>> On ppc64, migration-test only works with kvm_hv, and we already
> >>> have a check to verify the module is loaded.
> >>>
> >>> kvm_hv module can be loaded in memory and /sys/module/kvm_hv exists,
> >>> but on some systems (like build systems) /dev/kvm can be missing
> >>> (by administrators choice).
> >>>
> >>> And as kvm_hv exists test-migration is started but QEMU falls back to
> >>> TCG because it cannot be used:
> >>>
> >>>     Could not access KVM kernel module: No such file or directory
> >>>     failed to initialize KVM: No such file or directory
> >>>     Back to tcg accelerator
> >>>
> >>> And as the test is done with TCG, it fails.
> >>>
> >>> As for s390x, we must check for the existence and the access rights
> >>> of /dev/kvm.
> >>>
> >>> Reported-by: Cole Robinson <crobinso@redhat.com>
> >>> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> >> 
> >> Reviewed-by: Juan Quintela <quintela@redhat.com>
> >> 
> >> Oh, why it is so difficult!!!
> >> 
> >> Thanks, Juan.
> >> 
> >
> > David,
> >
> > could you add this one too to your ppc-for-4.2 queue?
> 
> I was about to ask if you preffered it to go through the migration tree.
> 
> David?

Migration tree is good for me.

> 
> Later, Juan.
> 

-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson
Re: [PATCH] pseries: disable migration-test if /dev/kvm cannot be used
Posted by Laurent Vivier 4 years, 4 months ago
Le 21/11/2019 à 09:30, Juan Quintela a écrit :
> Laurent Vivier <lvivier@redhat.com> wrote:
>> On 21/11/2019 08:18, Juan Quintela wrote:
>>> Laurent Vivier <lvivier@redhat.com> wrote:
>>>> On ppc64, migration-test only works with kvm_hv, and we already
>>>> have a check to verify the module is loaded.
>>>>
>>>> kvm_hv module can be loaded in memory and /sys/module/kvm_hv exists,
>>>> but on some systems (like build systems) /dev/kvm can be missing
>>>> (by administrators choice).
>>>>
>>>> And as kvm_hv exists test-migration is started but QEMU falls back to
>>>> TCG because it cannot be used:
>>>>
>>>>     Could not access KVM kernel module: No such file or directory
>>>>     failed to initialize KVM: No such file or directory
>>>>     Back to tcg accelerator
>>>>
>>>> And as the test is done with TCG, it fails.
>>>>
>>>> As for s390x, we must check for the existence and the access rights
>>>> of /dev/kvm.
>>>>
>>>> Reported-by: Cole Robinson <crobinso@redhat.com>
>>>> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
>>>
>>> Reviewed-by: Juan Quintela <quintela@redhat.com>
>>>
>>> Oh, why it is so difficult!!!
>>>
>>> Thanks, Juan.
>>>
>>
>> David,
>>
>> could you add this one too to your ppc-for-4.2 queue?
> 
> I was about to ask if you preffered it to go through the migration tree.
> 
> David?
> 
> Later, Juan.

Juan, as it's a fix it should go in rc3.

Thanks,
Laurent