Patchew URL: https://patchew.org/QEMU/[email protected]/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: [email protected] Subject: [PATCH 00/20] Various vhost-user-gpu & UI fixes === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu - [tag update] patchew/[email protected] -> patchew/[email protected] - [tag update] patchew/[email protected] -> patchew/[email protected] * [new tag] patchew/[email protected] -> patchew/[email protected] Switched to a new branch 'test' 22d6630 RFC: tests: add some virtio-gpu & vhost-user-gpu acceptance test e3c9583 chardev: check if the chardev is registered for yanking b1fb728 display/ui: add a callback to indicate GL state is flushed 8e9a9d9 virtio-gpu: avoid re-entering cmdq processing 6495eb7 ui: add egl dmabuf import to gtkglarea 9fe1210 ui: check gtk-egl dmabuf support 165cebf ui: add qemu_egl_has_dmabuf helper 6339b5b ui: check hw requirements during DCL registration 483c6af ui: add a DCLOps callback to check dmabuf support 43e7283 ui: add an optional get_flags callback to GraphicHwOps 24a96b4 vhost-user-gpu: add a configuration flag for dmabuf usage 851661e ui: remove console_has_gl_dmabuf() 6f62883 ui: annotate DCLOps callback requirements 0843e91 ui: add gd_gl_area_scanout_disable ea06ae9 ui: remove gl_ctx_get_current 82f8fd2 ui: remove extra #ifdef CONFIG_OPENGL 3eeb59f vhost-user-gpu: handle display-info in a callback ca98f97 vhost-user-gpu: use an extandable state enum for commands f3f5b71 vhost-user-gpu: handle vhost-user-gpu features in a callback 679108c vhost-user-gpu: check backend for EDID support === OUTPUT BEGIN === 1/20 Checking commit 679108c6db9d (vhost-user-gpu: check backend for EDID support) 2/20 Checking commit f3f5b712d643 (vhost-user-gpu: handle vhost-user-gpu features in a callback) 3/20 Checking commit ca98f9798940 (vhost-user-gpu: use an extandable state enum for commands) 4/20 Checking commit 3eeb59f5eda8 (vhost-user-gpu: handle display-info in a callback) WARNING: line over 80 characters #34: FILE: contrib/vhost-user-gpu/vhost-user-gpu.c:274: + if (!vg_recv_msg(vg, VHOST_USER_GPU_GET_DISPLAY_INFO, sizeof(dpy_info), &dpy_info)) { total: 0 errors, 1 warnings, 47 lines checked Patch 4/20 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 5/20 Checking commit 82f8fd22e61a (ui: remove extra #ifdef CONFIG_OPENGL) 6/20 Checking commit ea06ae968a65 (ui: remove gl_ctx_get_current) 7/20 Checking commit 0843e91e2077 (ui: add gd_gl_area_scanout_disable) 8/20 Checking commit 6f6288374e98 (ui: annotate DCLOps callback requirements) 9/20 Checking commit 851661e80efb (ui: remove console_has_gl_dmabuf()) 10/20 Checking commit 24a96b4c5616 (vhost-user-gpu: add a configuration flag for dmabuf usage) 11/20 Checking commit 43e7283e2d2e (ui: add an optional get_flags callback to GraphicHwOps) ERROR: braces {} are necessary for all arms of this statement #33: FILE: hw/display/virtio-gpu-base.c:123: + if (virtio_gpu_virgl_enabled(g->conf)) [...] ERROR: braces {} are necessary for all arms of this statement #36: FILE: hw/display/virtio-gpu-base.c:126: + if (virtio_gpu_dmabuf_enabled(g->conf)) [...] total: 2 errors, 0 warnings, 68 lines checked Patch 11/20 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 12/20 Checking commit 483c6affb8c5 (ui: add a DCLOps callback to check dmabuf support) 13/20 Checking commit 6339b5bcfe3b (ui: check hw requirements during DCL registration) ERROR: line over 90 characters #21: FILE: ui/console.c:1479: +static bool dpy_compatible_with(QemuConsole *con, DisplayChangeListener *dcl, Error **errp) WARNING: line over 80 characters #34: FILE: ui/console.c:1492: + if (flags & GRAPHIC_FLAGS_DMABUF && !displaychangelistener_has_dmabuf(dcl)) { total: 1 errors, 1 warnings, 45 lines checked Patch 13/20 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 14/20 Checking commit 165cebfa7beb (ui: add qemu_egl_has_dmabuf helper) WARNING: line over 80 characters #36: FILE: ui/egl-helpers.c:447: + !epoxy_has_egl_extension(qemu_egl_display, "EGL_EXT_image_dma_buf_import")) { total: 0 errors, 1 warnings, 21 lines checked Patch 14/20 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 15/20 Checking commit 9fe1210ac000 (ui: check gtk-egl dmabuf support) 16/20 Checking commit 6495eb768a5f (ui: add egl dmabuf import to gtkglarea) 17/20 Checking commit 8e9a9d99a7cc (virtio-gpu: avoid re-entering cmdq processing) 18/20 Checking commit b1fb7283b8da (display/ui: add a callback to indicate GL state is flushed) 19/20 Checking commit e3c958379468 (chardev: check if the chardev is registered for yanking) 20/20 Checking commit 22d663059dd3 (RFC: tests: add some virtio-gpu & vhost-user-gpu acceptance test) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #21: new file mode 100644 WARNING: line over 80 characters #78: FILE: tests/acceptance/virtio-gpu.py:53: + self, success_message, failure_message="Kernel panic - not syncing", vm=vm WARNING: line over 80 characters #112: FILE: tests/acceptance/virtio-gpu.py:87: + exec_command_and_wait_for_pattern(self, "/usr/sbin/modprobe virtio_gpu", "") WARNING: line over 80 characters #135: FILE: tests/acceptance/virtio-gpu.py:110: + qemu_sock, vug_sock = socket.socketpair(socket.AF_UNIX, socket.SOCK_STREAM) WARNING: line over 80 characters #139: FILE: tests/acceptance/virtio-gpu.py:114: + self._vug_log_path = os.path.join(self.vm._test_dir, "vhost-user-gpu.log") WARNING: line over 80 characters #170: FILE: tests/acceptance/virtio-gpu.py:145: + exec_command_and_wait_for_pattern(self, "/usr/sbin/modprobe virtio_gpu", "") total: 0 errors, 6 warnings, 150 lines checked Patch 20/20 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/[email protected]/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to [email protected]
