On Fri, 2023-09-01 at 17:58 +0200, Nina Schoetterl-Glausch wrote:
> From: Pierre Morel <pmo...@linux.ibm.com>
>
> Test changes in the entitlement from both a guest and a host point of
> view, depending on the polarization.
>
> Signed-off-by: Pierre Morel <pmo...@linux.ibm.com>
> Co-developed-by: Nina Schoetterl-Glausch <n...@linux.ibm.com>
> Signed-off-by: Nina Schoetterl-Glausch <n...@linux.ibm.com>
Reviewed-by: Nina Schoetterl-Glausch <n...@linux.ibm.com>
> ---
> tests/avocado/s390_topology.py | 55 ++++++++++++++++++++++++++++++++++
> 1 file changed, 55 insertions(+)
>
> diff --git a/tests/avocado/s390_topology.py b/tests/avocado/s390_topology.py
> index d7ee36cb89..733093723a 100644
> --- a/tests/avocado/s390_topology.py
> +++ b/tests/avocado/s390_topology.py
> @@ -243,3 +243,58 @@ def test_polarization(self):
> res = self.vm.qmp('query-s390x-cpu-polarization')
> self.assertEqual(res['return']['polarization'], 'horizontal')
> self.check_topology(0, 0, 0, 0, 'medium', False)
> +
> +
> + def check_polarization(self, polarization):
> + """
> + We need the polarization change to get stabilized in kernel
> + before we check it through the sysfs.
> + """
IMO this should be a comment, not a function doc.
#We need to wait for the change to have been propagated to the kernel
> + exec_command_and_wait_for_pattern(self,
> + """ sleep 1 ;
> + /bin/cat /sys/devices/system/cpu/cpu0/polarization """,
> + polarization)
[...]
