* [PATCH] tests/acceptance: test hot(un)plug of ccw devices
@ 2020-12-03 15:39 Cornelia Huck
2020-12-03 17:22 ` Thomas Huth
0 siblings, 1 reply; 6+ messages in thread
From: Cornelia Huck @ 2020-12-03 15:39 UTC (permalink / raw)
To: qemu-devel
Cc: Thomas Huth, Cornelia Huck, Wainer dos Santos Moschetta,
Halil Pasic, Christian Borntraeger, qemu-s390x, Cleber Rosa,
Philippe Mathieu-Daudé
Hotplug a virtio-net-ccw device, and then hotunplug it again.
Signed-off-by: Cornelia Huck <cohuck@redhat.com>
---
This is on top of "tests/acceptance: enhance s390x devices test"
---
tests/acceptance/machine_s390_ccw_virtio.py | 14 ++++++++++++++
1 file changed, 14 insertions(+)
diff --git a/tests/acceptance/machine_s390_ccw_virtio.py b/tests/acceptance/machine_s390_ccw_virtio.py
index 53b8484f8f9c..487c25c31d3c 100644
--- a/tests/acceptance/machine_s390_ccw_virtio.py
+++ b/tests/acceptance/machine_s390_ccw_virtio.py
@@ -97,3 +97,17 @@ class S390CCWVirtioMachine(Test):
exec_command_and_wait_for_pattern(self,
'cat /sys/bus/pci/devices/000a\:00\:00.0/function_id',
'0x0000000c')
+ # add another device
+ self.vm.command('device_add', driver='virtio-net-ccw',
+ devno='fe.0.4711', id='xxx')
+ exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
+ exec_command_and_wait_for_pattern(self, 'ls /sys/bus/ccw/devices/',
+ '0.0.4711')
+ # and detach it again
+ self.vm.command('device_del', id='xxx')
+ self.vm.event_wait(name='DEVICE_DELETED',
+ match={'data': {'device': 'xxx'}})
+ exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
+ exec_command_and_wait_for_pattern(self,
+ 'ls /sys/bus/ccw/devices/0.0.4711',
+ 'No such file or directory')
--
2.26.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: test hot(un)plug of ccw devices
2020-12-03 15:39 [PATCH] tests/acceptance: test hot(un)plug of ccw devices Cornelia Huck
@ 2020-12-03 17:22 ` Thomas Huth
2020-12-03 17:38 ` Cornelia Huck
2020-12-03 18:11 ` Philippe Mathieu-Daudé
0 siblings, 2 replies; 6+ messages in thread
From: Thomas Huth @ 2020-12-03 17:22 UTC (permalink / raw)
To: Cornelia Huck, qemu-devel
Cc: Wainer dos Santos Moschetta, Halil Pasic, Christian Borntraeger,
qemu-s390x, Cleber Rosa, Philippe Mathieu-Daudé
On 03/12/2020 16.39, Cornelia Huck wrote:
> Hotplug a virtio-net-ccw device, and then hotunplug it again.
Good idea! ... is it also possible with a pci device?
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>
> This is on top of "tests/acceptance: enhance s390x devices test"
>
> ---
> tests/acceptance/machine_s390_ccw_virtio.py | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/tests/acceptance/machine_s390_ccw_virtio.py b/tests/acceptance/machine_s390_ccw_virtio.py
> index 53b8484f8f9c..487c25c31d3c 100644
> --- a/tests/acceptance/machine_s390_ccw_virtio.py
> +++ b/tests/acceptance/machine_s390_ccw_virtio.py
> @@ -97,3 +97,17 @@ class S390CCWVirtioMachine(Test):
> exec_command_and_wait_for_pattern(self,
> 'cat /sys/bus/pci/devices/000a\:00\:00.0/function_id',
> '0x0000000c')
> + # add another device
> + self.vm.command('device_add', driver='virtio-net-ccw',
> + devno='fe.0.4711', id='xxx')
Could we use a different id, please? xxx sounds so ... well, use your
imagination.
> + exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
That looks like it could be a little bit racy ... what if the kernel outputs
another log message by chance, so that tail -n 1 reports that instead.
I think it would be better to clear the dmesg log ("dmesg -c") before
plugging, and then look at all the new output of "dmesg" without using
"tail" afterwards.
> + exec_command_and_wait_for_pattern(self, 'ls /sys/bus/ccw/devices/',
> + '0.0.4711')
> + # and detach it again
> + self.vm.command('device_del', id='xxx')
> + self.vm.event_wait(name='DEVICE_DELETED',
> + match={'data': {'device': 'xxx'}})
> + exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
dito
> + exec_command_and_wait_for_pattern(self,
> + 'ls /sys/bus/ccw/devices/0.0.4711',
> + 'No such file or directory')
>
Thomas
PS: Another idea for a test: Looks like that initrd also has a
virtio-balloon driver ... we could maybe start with "-device
virito-balloon", then change the size of the balloon and check whether the
MemTotal in /proc/meminfo changed...
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: test hot(un)plug of ccw devices
2020-12-03 17:22 ` Thomas Huth
@ 2020-12-03 17:38 ` Cornelia Huck
2020-12-03 18:11 ` Philippe Mathieu-Daudé
1 sibling, 0 replies; 6+ messages in thread
From: Cornelia Huck @ 2020-12-03 17:38 UTC (permalink / raw)
To: Thomas Huth
Cc: qemu-devel, Wainer dos Santos Moschetta, Halil Pasic,
Christian Borntraeger, qemu-s390x, Cleber Rosa,
Philippe Mathieu-Daudé
On Thu, 3 Dec 2020 18:22:35 +0100
Thomas Huth <thuth@redhat.com> wrote:
> On 03/12/2020 16.39, Cornelia Huck wrote:
> > Hotplug a virtio-net-ccw device, and then hotunplug it again.
>
> Good idea! ... is it also possible with a pci device?
It should be, I guess; it's not part of my normal test workflow,
however.
>
> > Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> > ---
> >
> > This is on top of "tests/acceptance: enhance s390x devices test"
> >
> > ---
> > tests/acceptance/machine_s390_ccw_virtio.py | 14 ++++++++++++++
> > 1 file changed, 14 insertions(+)
> >
> > diff --git a/tests/acceptance/machine_s390_ccw_virtio.py b/tests/acceptance/machine_s390_ccw_virtio.py
> > index 53b8484f8f9c..487c25c31d3c 100644
> > --- a/tests/acceptance/machine_s390_ccw_virtio.py
> > +++ b/tests/acceptance/machine_s390_ccw_virtio.py
> > @@ -97,3 +97,17 @@ class S390CCWVirtioMachine(Test):
> > exec_command_and_wait_for_pattern(self,
> > 'cat /sys/bus/pci/devices/000a\:00\:00.0/function_id',
> > '0x0000000c')
> > + # add another device
> > + self.vm.command('device_add', driver='virtio-net-ccw',
> > + devno='fe.0.4711', id='xxx')
>
> Could we use a different id, please? xxx sounds so ... well, use your
> imagination.
It is taken straight from my usual testing workflow :) But yeah, I can
call this net_4711 or so.
>
> > + exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
>
> That looks like it could be a little bit racy ... what if the kernel outputs
> another log message by chance, so that tail -n 1 reports that instead.
>
> I think it would be better to clear the dmesg log ("dmesg -c") before
> plugging, and then look at all the new output of "dmesg" without using
> "tail" afterwards.
Yes, good idea. Would need to do the same dance below as well (we get a
new crw).
>
> > + exec_command_and_wait_for_pattern(self, 'ls /sys/bus/ccw/devices/',
> > + '0.0.4711')
> > + # and detach it again
> > + self.vm.command('device_del', id='xxx')
> > + self.vm.event_wait(name='DEVICE_DELETED',
> > + match={'data': {'device': 'xxx'}})
> > + exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
>
> dito
>
> > + exec_command_and_wait_for_pattern(self,
> > + 'ls /sys/bus/ccw/devices/0.0.4711',
> > + 'No such file or directory')
> >
>
> Thomas
>
>
> PS: Another idea for a test: Looks like that initrd also has a
> virtio-balloon driver ... we could maybe start with "-device
> virito-balloon", then change the size of the balloon and check whether the
> MemTotal in /proc/meminfo changed...
Added to my to-test list.
Thanks!
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: test hot(un)plug of ccw devices
2020-12-03 17:22 ` Thomas Huth
2020-12-03 17:38 ` Cornelia Huck
@ 2020-12-03 18:11 ` Philippe Mathieu-Daudé
2020-12-03 18:14 ` Thomas Huth
1 sibling, 1 reply; 6+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-12-03 18:11 UTC (permalink / raw)
To: Thomas Huth, Cornelia Huck, qemu-devel
Cc: Halil Pasic, Christian Borntraeger, qemu-s390x,
Wainer dos Santos Moschetta, Cleber Rosa
On 12/3/20 6:22 PM, Thomas Huth wrote:
> On 03/12/2020 16.39, Cornelia Huck wrote:
>> Hotplug a virtio-net-ccw device, and then hotunplug it again.
>
> Good idea! ... is it also possible with a pci device?
>
>> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
>> ---
...
>> + exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
>
> That looks like it could be a little bit racy ... what if the kernel outputs
> another log message by chance, so that tail -n 1 reports that instead.
>
> I think it would be better to clear the dmesg log ("dmesg -c") before
> plugging, and then look at all the new output of "dmesg" without using
> "tail" afterwards.
"dmesg --follow"?
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: test hot(un)plug of ccw devices
2020-12-03 18:11 ` Philippe Mathieu-Daudé
@ 2020-12-03 18:14 ` Thomas Huth
2020-12-03 20:12 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 6+ messages in thread
From: Thomas Huth @ 2020-12-03 18:14 UTC (permalink / raw)
To: Philippe Mathieu-Daudé, Cornelia Huck, qemu-devel
Cc: Halil Pasic, Christian Borntraeger, qemu-s390x,
Wainer dos Santos Moschetta, Cleber Rosa
On 03/12/2020 19.11, Philippe Mathieu-Daudé wrote:
> On 12/3/20 6:22 PM, Thomas Huth wrote:
>> On 03/12/2020 16.39, Cornelia Huck wrote:
>>> Hotplug a virtio-net-ccw device, and then hotunplug it again.
>>
>> Good idea! ... is it also possible with a pci device?
>>
>>> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
>>> ---
> ...
>>> + exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
>>
>> That looks like it could be a little bit racy ... what if the kernel outputs
>> another log message by chance, so that tail -n 1 reports that instead.
>>
>> I think it would be better to clear the dmesg log ("dmesg -c") before
>> plugging, and then look at all the new output of "dmesg" without using
>> "tail" afterwards.
>
> "dmesg --follow"?
Then you'd need to send control-c afterwards to stop it? ... not sure
whether that's such a better solution...
Thomas
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] tests/acceptance: test hot(un)plug of ccw devices
2020-12-03 18:14 ` Thomas Huth
@ 2020-12-03 20:12 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 6+ messages in thread
From: Philippe Mathieu-Daudé @ 2020-12-03 20:12 UTC (permalink / raw)
To: Thomas Huth, Cornelia Huck, qemu-devel
Cc: Halil Pasic, Christian Borntraeger, qemu-s390x,
Wainer dos Santos Moschetta, Cleber Rosa
On 12/3/20 7:14 PM, Thomas Huth wrote:
> On 03/12/2020 19.11, Philippe Mathieu-Daudé wrote:
>> On 12/3/20 6:22 PM, Thomas Huth wrote:
>>> On 03/12/2020 16.39, Cornelia Huck wrote:
>>>> Hotplug a virtio-net-ccw device, and then hotunplug it again.
>>>
>>> Good idea! ... is it also possible with a pci device?
>>>
>>>> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
>>>> ---
>> ...
>>>> + exec_command_and_wait_for_pattern(self, 'dmesg | tail -n 1', 'CRW')
>>>
>>> That looks like it could be a little bit racy ... what if the kernel outputs
>>> another log message by chance, so that tail -n 1 reports that instead.
>>>
>>> I think it would be better to clear the dmesg log ("dmesg -c") before
>>> plugging, and then look at all the new output of "dmesg" without using
>>> "tail" afterwards.
>>
>> "dmesg --follow"?
>
> Then you'd need to send control-c afterwards to stop it? ... not sure
> whether that's such a better solution...
Oh indeed, I missed there is further interaction.
Thanks,
Phil.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2020-12-03 20:13 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-03 15:39 [PATCH] tests/acceptance: test hot(un)plug of ccw devices Cornelia Huck
2020-12-03 17:22 ` Thomas Huth
2020-12-03 17:38 ` Cornelia Huck
2020-12-03 18:11 ` Philippe Mathieu-Daudé
2020-12-03 18:14 ` Thomas Huth
2020-12-03 20:12 ` Philippe Mathieu-Daudé
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.