Re: [PATCH v21 20/20] tests/avocado: s390x cpu topology bad move

[Pierre Morel]

On 7/5/23 12:32, Thomas Huth wrote:
> On 30/06/2023 11.17, Pierre Morel wrote:
> > This test verifies that QEMU refuses to move a CPU to an
> > unexistant location.
>
> s/unexistant/nonexistent/ ?
>
> > Signed-off-by: Pierre Morel <pmorel@linux.ibm.com>
> > ---
> >   tests/avocado/s390_topology.py | 25 +++++++++++++++++++++++++
> >   1 file changed, 25 insertions(+)
> >
> > diff --git a/tests/avocado/s390_topology.py 
> > b/tests/avocado/s390_topology.py
> > index 99d9508cef..ea39168b53 100644
> > --- a/tests/avocado/s390_topology.py
> > +++ b/tests/avocado/s390_topology.py
> > @@ -388,3 +388,28 @@ def test_dedicated_error(self):
> >           res = self.vm.qmp('set-cpu-topology',
> >                             {'core-id': 0, 'entitlement': 'medium', 
> > 'dedicated': False})
> >           self.assertEqual(res['return'], {})
> > +
> > +    def test_move_error(self):
> > +        """
> > +        This test verifies that QEMU refuses to move a CPU to an
> > +        unexistant location
>
> s/unexistant/nonexistent/ ?
>
> With the words fixed:
> Reviewed-by: Thomas Huth <thuth@redhat.com>
I fix it, thanks,

Regards,

Pierre