[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [Qemu-devel] [PATCH v5 00/13] vhost-user for input & GPU
From: |
no-reply |
Subject: |
Re: [Qemu-ppc] [Qemu-devel] [PATCH v5 00/13] vhost-user for input & GPU |
Date: |
Fri, 12 Apr 2019 09:17:37 -0700 (PDT) |
Patchew URL: https://patchew.org/QEMU/address@hidden/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Message-id: address@hidden
Subject: [Qemu-devel] [PATCH v5 00/13] vhost-user for input & GPU
Type: series
=== 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
e1be98540e..13c24edaa7 master -> master
* [new tag] patchew/address@hidden -> patchew/address@hidden
* [new tag] patchew/address@hidden -> patchew/address@hidden
* [new tag] patchew/address@hidden -> patchew/address@hidden
* [new tag] patchew/address@hidden -> patchew/address@hidden
* [new tag] patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
d5cc322df5 hw/display: add vhost-user-vga & gpu-pci
3e48843814 virtio-gpu: split virtio-gpu-pci & virtio-vga
27de3a9c6e virtio-gpu: split virtio-gpu, introduce virtio-gpu-base
3d2340a207 contrib: add vhost-user-gpu
9a14706dd9 virtio: add virtio-gpu bswap helpers header
03da2a93d1 vhost-user: add vhost_user_gpu_set_socket()
e47966fdc8 contrib: add vhost-user-input
beb467adc0 libvhost-user: add PROTOCOL_F_CONFIG if {set, get}_config
0cf7f81cf7 Add vhost-user-input-pci
2ce13ff56b Add vhost-user-backend
68dc09b28e qemu-img: fix .hx and .texi disparity
=== OUTPUT BEGIN ===
1/11 Checking commit 68dc09b28e31 (qemu-img: fix .hx and .texi disparity)
2/11 Checking commit 2ce13ff56b22 (Add vhost-user-backend)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#44:
new file mode 100644
WARNING: Block comments use a leading /* on a separate line
#143: FILE: backends/vhost-user.c:95:
+ /* guest_notifier_mask/pending not used yet, so just unmask
total: 0 errors, 2 warnings, 280 lines checked
Patch 2/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
3/11 Checking commit 0cf7f81cf724 (Add vhost-user-input-pci)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#58:
new file mode 100644
total: 0 errors, 1 warnings, 250 lines checked
Patch 3/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/11 Checking commit beb467adc013 (libvhost-user: add PROTOCOL_F_CONFIG if
{set, get}_config)
5/11 Checking commit e47966fdc89a (contrib: add vhost-user-input)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#79:
new file mode 100644
total: 0 errors, 1 warnings, 466 lines checked
Patch 5/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
6/11 Checking commit 03da2a93d186 (vhost-user: add vhost_user_gpu_set_socket())
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#74:
new file mode 100644
total: 0 errors, 1 warnings, 313 lines checked
Patch 6/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
7/11 Checking commit 9a14706dd99f (virtio: add virtio-gpu bswap helpers header)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#74:
new file mode 100644
total: 0 errors, 1 warnings, 116 lines checked
Patch 7/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
8/11 Checking commit 3d2340a20702 (contrib: add vhost-user-gpu)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#131:
new file mode 100644
total: 0 errors, 1 warnings, 2523 lines checked
Patch 8/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
9/11 Checking commit 27de3a9c6ece (virtio-gpu: split virtio-gpu, introduce
virtio-gpu-base)
WARNING: Block comments use a leading /* on a separate line
#100: FILE: hw/display/virtio-gpu-3d.c:181:
+ info.flags & 1 /* FIXME: Y_0_TOP */,
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#188:
new file mode 100644
ERROR: Macros with complex values should be enclosed in parenthesis
#1253: FILE: include/hw/virtio/virtio-gpu.h:122:
+#define VIRTIO_GPU_BASE_PROPERTIES(_state, _conf) \
+ DEFINE_PROP_UINT32("max_outputs", _state, _conf.max_outputs, 1), \
+ DEFINE_PROP_BIT("edid", _state, _conf.flags, \
+ VIRTIO_GPU_FLAG_EDID_ENABLED, false), \
+ DEFINE_PROP_UINT32("xres", _state, _conf.xres, 1024), \
+ DEFINE_PROP_UINT32("yres", _state, _conf.yres, 768)
total: 1 errors, 2 warnings, 1184 lines checked
Patch 9/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
10/11 Checking commit 3e4884381437 (virtio-gpu: split virtio-gpu-pci &
virtio-vga)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#392:
new file mode 100644
total: 0 errors, 1 warnings, 390 lines checked
Patch 10/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
11/11 Checking commit d5cc322df527 (hw/display: add vhost-user-vga & gpu-pci)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#56:
new file mode 100644
ERROR: Macros with complex values should be enclosed in parenthesis
#829: FILE: include/hw/virtio/virtio-gpu-pci.h:35:
+#define DEFINE_VIRTIO_GPU_PCI_PROPERTIES(_state) \
+ DEFINE_PROP_BIT("ioeventfd", _state, flags, \
+ VIRTIO_PCI_FLAG_USE_IOEVENTFD_BIT, false), \
+ DEFINE_PROP_UINT32("vectors", _state, nvectors, 3)
total: 1 errors, 1 warnings, 818 lines checked
Patch 11/11 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/address@hidden/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to address@hidden
- [Qemu-ppc] [PATCH v5 06/13] libvhost-user: add PROTOCOL_F_CONFIG if {set, get}_config, (continued)
- [Qemu-ppc] [PATCH v5 06/13] libvhost-user: add PROTOCOL_F_CONFIG if {set, get}_config, Marc-André Lureau, 2019/04/12
- [Qemu-ppc] [PATCH v5 07/13] contrib: add vhost-user-input, Marc-André Lureau, 2019/04/12
- [Qemu-ppc] [PATCH v5 08/13] vhost-user: add vhost_user_gpu_set_socket(), Marc-André Lureau, 2019/04/12
- [Qemu-ppc] [PATCH v5 09/13] virtio: add virtio-gpu bswap helpers header, Marc-André Lureau, 2019/04/12
- [Qemu-ppc] [PATCH v5 10/13] contrib: add vhost-user-gpu, Marc-André Lureau, 2019/04/12
- [Qemu-ppc] [PATCH v5 11/13] virtio-gpu: split virtio-gpu, introduce virtio-gpu-base, Marc-André Lureau, 2019/04/12
- [Qemu-ppc] [PATCH v5 12/13] virtio-gpu: split virtio-gpu-pci & virtio-vga, Marc-André Lureau, 2019/04/12
- [Qemu-ppc] [PATCH v5 13/13] hw/display: add vhost-user-vga & gpu-pci, Marc-André Lureau, 2019/04/12
- Re: [Qemu-ppc] [Qemu-devel] [PATCH v5 00/13] vhost-user for input & GPU, Marc-André Lureau, 2019/04/12
- Re: [Qemu-ppc] [Qemu-devel] [PATCH v5 00/13] vhost-user for input & GPU, no-reply, 2019/04/12
- Re: [Qemu-ppc] [Qemu-devel] [PATCH v5 00/13] vhost-user for input & GPU,
no-reply <=