| Message ID | b5d78f1452a85057c257e1f9efca527d05ff0388.1633929457.git.jag.raman@oracle.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | vfio-user server in QEMU | expand |
On 10/11/21 07:31, Jagannathan Raman wrote: > Acceptance test for libvfio-user in QEMU > > Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com> > Signed-off-by: John G Johnson <john.g.johnson@oracle.com> > Signed-off-by: Jagannathan Raman <jag.raman@oracle.com> > --- > MAINTAINERS | 1 + > tests/acceptance/vfio-user.py | 96 +++++++++++++++++++++++++++++++++++ > 2 files changed, 97 insertions(+) > create mode 100644 tests/acceptance/vfio-user.py > +class VfioUser(Test): > + """ > + :avocado: tags=vfiouser > + """ > + KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 ' > + > + def do_test(self, kernel_url, initrd_url, kernel_command_line, > + machine_type): > + """Main test method""" > + self.require_accelerator('kvm') > + > + kernel_path = self.fetch_asset(kernel_url) > + initrd_path = self.fetch_asset(initrd_url) > + > + socket = os.path.join('/tmp', str(uuid.uuid4())) > + if os.path.exists(socket): > + os.remove(socket) > + > + # Create remote process > + remote_vm = self.get_vm() > + remote_vm.add_args('-machine', 'x-remote') > + remote_vm.add_args('-nodefaults') > + remote_vm.add_args('-device', 'lsi53c895a,id=lsi1') > + remote_vm.add_args('-object', 'vfio-user,id=vfioobj1,' > + 'devid=lsi1,socket='+socket) Nitpicking for style: spaces around '+' here, > + remote_vm.launch() > + > + # Create proxy process > + self.vm.set_console() > + self.vm.add_args('-machine', machine_type) > + self.vm.add_args('-accel', 'kvm') > + self.vm.add_args('-cpu', 'host') > + self.vm.add_args('-object', > + 'memory-backend-memfd,id=sysmem-file,size=2G') > + self.vm.add_args('--numa', 'node,memdev=sysmem-file') > + self.vm.add_args('-m', '2048') > + self.vm.add_args('-kernel', kernel_path, > + '-initrd', initrd_path, > + '-append', kernel_command_line) > + self.vm.add_args('-device', > + 'vfio-user-pci,' > + 'socket='+socket) and here. Otherwise: Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com> Thanks for adding this test :)
On Mon, Oct 11, 2021 at 01:31:17AM -0400, Jagannathan Raman wrote: > Acceptance test for libvfio-user in QEMU > > Signed-off-by: Elena Ufimtseva <elena.ufimtseva@oracle.com> > Signed-off-by: John G Johnson <john.g.johnson@oracle.com> > Signed-off-by: Jagannathan Raman <jag.raman@oracle.com> > --- > MAINTAINERS | 1 + > tests/acceptance/vfio-user.py | 96 +++++++++++++++++++++++++++++++++++ > 2 files changed, 97 insertions(+) > create mode 100644 tests/acceptance/vfio-user.py > > diff --git a/MAINTAINERS b/MAINTAINERS > index 79ff8331dc..a98d37423b 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -3422,6 +3422,7 @@ F: hw/remote/iohub.c > F: include/hw/remote/iohub.h > F: subprojects/libvfio-user > F: hw/remote/vfio-user-obj.c > +F: tests/acceptance/vfio-user.py > > EBPF: > M: Jason Wang <jasowang@redhat.com> > diff --git a/tests/acceptance/vfio-user.py b/tests/acceptance/vfio-user.py > new file mode 100644 > index 0000000000..5eb5cabc68 > --- /dev/null > +++ b/tests/acceptance/vfio-user.py > @@ -0,0 +1,96 @@ > +# vfio-user protocol sanity test > +# > +# This work is licensed under the terms of the GNU GPL, version 2 or > +# later. See the COPYING file in the top-level directory. > + > + > +import os > +import socket > +import uuid > + > +from avocado_qemu import Test > +from avocado_qemu import wait_for_console_pattern > +from avocado_qemu import exec_command > +from avocado_qemu import exec_command_and_wait_for_pattern > + > +class VfioUser(Test): > + """ > + :avocado: tags=vfiouser > + """ > + KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 ' > + > + def do_test(self, kernel_url, initrd_url, kernel_command_line, > + machine_type): > + """Main test method""" > + self.require_accelerator('kvm') > + > + kernel_path = self.fetch_asset(kernel_url) > + initrd_path = self.fetch_asset(initrd_url) > + > + socket = os.path.join('/tmp', str(uuid.uuid4())) > + if os.path.exists(socket): > + os.remove(socket) > + > + # Create remote process > + remote_vm = self.get_vm() > + remote_vm.add_args('-machine', 'x-remote') > + remote_vm.add_args('-nodefaults') > + remote_vm.add_args('-device', 'lsi53c895a,id=lsi1') > + remote_vm.add_args('-object', 'vfio-user,id=vfioobj1,' > + 'devid=lsi1,socket='+socket) The object is called "vfio-user-server" and the parameter syntax seems to be outdated. Does this test pass?
diff --git a/MAINTAINERS b/MAINTAINERS index 79ff8331dc..a98d37423b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -3422,6 +3422,7 @@ F: hw/remote/iohub.c F: include/hw/remote/iohub.h F: subprojects/libvfio-user F: hw/remote/vfio-user-obj.c +F: tests/acceptance/vfio-user.py EBPF: M: Jason Wang <jasowang@redhat.com> diff --git a/tests/acceptance/vfio-user.py b/tests/acceptance/vfio-user.py new file mode 100644 index 0000000000..5eb5cabc68 --- /dev/null +++ b/tests/acceptance/vfio-user.py @@ -0,0 +1,96 @@ +# vfio-user protocol sanity test +# +# This work is licensed under the terms of the GNU GPL, version 2 or +# later. See the COPYING file in the top-level directory. + + +import os +import socket +import uuid + +from avocado_qemu import Test +from avocado_qemu import wait_for_console_pattern +from avocado_qemu import exec_command +from avocado_qemu import exec_command_and_wait_for_pattern + +class VfioUser(Test): + """ + :avocado: tags=vfiouser + """ + KERNEL_COMMON_COMMAND_LINE = 'printk.time=0 ' + + def do_test(self, kernel_url, initrd_url, kernel_command_line, + machine_type): + """Main test method""" + self.require_accelerator('kvm') + + kernel_path = self.fetch_asset(kernel_url) + initrd_path = self.fetch_asset(initrd_url) + + socket = os.path.join('/tmp', str(uuid.uuid4())) + if os.path.exists(socket): + os.remove(socket) + + # Create remote process + remote_vm = self.get_vm() + remote_vm.add_args('-machine', 'x-remote') + remote_vm.add_args('-nodefaults') + remote_vm.add_args('-device', 'lsi53c895a,id=lsi1') + remote_vm.add_args('-object', 'vfio-user,id=vfioobj1,' + 'devid=lsi1,socket='+socket) + remote_vm.launch() + + # Create proxy process + self.vm.set_console() + self.vm.add_args('-machine', machine_type) + self.vm.add_args('-accel', 'kvm') + self.vm.add_args('-cpu', 'host') + self.vm.add_args('-object', + 'memory-backend-memfd,id=sysmem-file,size=2G') + self.vm.add_args('--numa', 'node,memdev=sysmem-file') + self.vm.add_args('-m', '2048') + self.vm.add_args('-kernel', kernel_path, + '-initrd', initrd_path, + '-append', kernel_command_line) + self.vm.add_args('-device', + 'vfio-user-pci,' + 'socket='+socket) + self.vm.launch() + wait_for_console_pattern(self, 'as init process', + 'Kernel panic - not syncing') + exec_command(self, 'mount -t sysfs sysfs /sys') + exec_command_and_wait_for_pattern(self, + 'cat /sys/bus/pci/devices/*/uevent', + 'PCI_ID=1000:0012') + + def test_multiprocess_x86_64(self): + """ + :avocado: tags=arch:x86_64 + :avocado: tags=distro:centos + """ + kernel_url = ('https://archives.fedoraproject.org/pub/archive/fedora' + '/linux/releases/31/Everything/x86_64/os/images' + '/pxeboot/vmlinuz') + initrd_url = ('https://archives.fedoraproject.org/pub/archive/fedora' + '/linux/releases/31/Everything/x86_64/os/images' + '/pxeboot/initrd.img') + kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE + + 'console=ttyS0 rdinit=/bin/bash') + machine_type = 'pc' + self.do_test(kernel_url, initrd_url, kernel_command_line, machine_type) + + def test_multiprocess_aarch64(self): + """ + :avocado: tags=arch:aarch64 + :avocado: tags=distro:ubuntu + """ + kernel_url = ('https://archives.fedoraproject.org/pub/archive/fedora' + '/linux/releases/31/Everything/aarch64/os/images' + '/pxeboot/vmlinuz') + initrd_url = ('https://archives.fedoraproject.org/pub/archive/fedora' + '/linux/releases/31/Everything/aarch64/os/images' + '/pxeboot/initrd.img') + kernel_command_line = (self.KERNEL_COMMON_COMMAND_LINE + + 'rdinit=/bin/bash console=ttyAMA0') + machine_type = 'virt,gic-version=3' + self.do_test(kernel_url, initrd_url, kernel_command_line, machine_type)