[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/25] Vga 20210510 patches
From: |
no-reply |
Subject: |
Re: [PULL 00/25] Vga 20210510 patches |
Date: |
Mon, 10 May 2021 06:43:12 -0700 (PDT) |
Patchew URL:
20210510132051.2208563-1-kraxel@redhat.com/">https://patchew.org/QEMU/20210510132051.2208563-1-kraxel@redhat.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20210510132051.2208563-1-kraxel@redhat.com
Subject: [PULL 00/25] Vga 20210510 patches
=== 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
* [new tag] patchew/20210510132051.2208563-1-kraxel@redhat.com ->
patchew/20210510132051.2208563-1-kraxel@redhat.com
Switched to a new branch 'test'
13cb22d virtio-gpu: add virtio-vga-gl
ce63459 modules: add have_vga
ec77853 virtio-gpu: add virtio-gpu-gl-pci
9876e4d virtio-gpu: move fields to struct VirtIOGPUGL
db5bc9c virtio-gpu: drop use_virgl_renderer
ddd31ea virtio-gpu: move virtio-gpu-gl-device to separate module
ab7db8c virtio-gpu: drop VIRGL() macro
1adf390 virtio-gpu: move update_cursor_data
8905ff2 virtio-gpu: move virgl process_cmd
ac63831 virtio-gpu: move virgl gl_flushed
a3b8da4 virtio-gpu: move virgl handle_ctrl
89bff04 virtio-gpu: use class function for ctrl queue handlers
7221fc9 virtio-gpu: move virgl reset
ccd7f46 virtio-gpu: move virgl realize + properties
4f688a1 virtio-gpu: add virtio-gpu-gl-device
61598c7 virtio-gpu: rename virgl source file.
494bab3 virtio-gpu: handle partial maps properly
e4ad854 edid: add support for DisplayID extension (5k resolution)
f39498c edid: allow arbitrary-length checksums
dcb1807 edid: move timing generation into a separate function
2a89c33 edid: Make refresh rate configurable
59156e5 edid: use dta extension block descriptors
8dbf087 edid: move xtra3 descriptor
3dfd70f edid: edid_desc_next
3c226b3 qemu-edid: use qemu_edid_size()
=== OUTPUT BEGIN ===
1/25 Checking commit 3c226b321569 (qemu-edid: use qemu_edid_size())
2/25 Checking commit 3dfd70fb4782 (edid: edid_desc_next)
3/25 Checking commit 8dbf0877dbd6 (edid: move xtra3 descriptor)
4/25 Checking commit 59156e5e31ea (edid: use dta extension block descriptors)
5/25 Checking commit 2a89c3372a5c (edid: Make refresh rate configurable)
WARNING: line over 80 characters
#35: FILE: hw/display/edid-generate.c:239:
+ uint64_t clock = (uint64_t)refresh_rate * (xres + xblank) * (yres +
yblank);
ERROR: Macros with complex values should be enclosed in parenthesis
#80: FILE: include/hw/display/edid.h:25:
+#define DEFINE_EDID_PROPERTIES(_state, _edid_info) \
+ DEFINE_PROP_UINT32("xres", _state, _edid_info.prefx, 0), \
+ DEFINE_PROP_UINT32("yres", _state, _edid_info.prefy, 0), \
+ DEFINE_PROP_UINT32("xmax", _state, _edid_info.maxx, 0), \
+ DEFINE_PROP_UINT32("ymax", _state, _edid_info.maxy, 0), \
+ DEFINE_PROP_UINT32("refresh_rate", _state, _edid_info.refresh_rate, 0)
total: 1 errors, 1 warnings, 57 lines checked
Patch 5/25 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
6/25 Checking commit dcb1807991b5 (edid: move timing generation into a separate
function)
7/25 Checking commit f39498ca0c3e (edid: allow arbitrary-length checksums)
8/25 Checking commit e4ad8545a719 (edid: add support for DisplayID extension
(5k resolution))
9/25 Checking commit 494bab301857 (virtio-gpu: handle partial maps properly)
WARNING: line over 80 characters
#46: FILE: hw/display/virtio-gpu-3d.c:292:
+ ret = virtio_gpu_create_mapping_iov(g, &att_rb, cmd, NULL, &res_iovs,
&res_niov);
WARNING: line over 80 characters
#118: FILE: hw/display/virtio-gpu.c:652:
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: failed to map MMIO memory
for"
total: 0 errors, 2 warnings, 142 lines checked
Patch 9/25 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
10/25 Checking commit 61598c780eef (virtio-gpu: rename virgl source file.)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#16:
hw/display/{virtio-gpu-3d.c => virtio-gpu-virgl.c} | 0
total: 0 errors, 1 warnings, 8 lines checked
Patch 10/25 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
11/25 Checking commit 4f688a135e5c (virtio-gpu: add virtio-gpu-gl-device)
Use of uninitialized value $acpi_testexpected in string eq at
./scripts/checkpatch.pl line 1529.
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#34:
new file mode 100644
total: 0 errors, 1 warnings, 75 lines checked
Patch 11/25 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
12/25 Checking commit ccd7f46f7d09 (virtio-gpu: move virgl realize + properties)
13/25 Checking commit 7221fc95ccb5 (virtio-gpu: move virgl reset)
14/25 Checking commit 89bff048f3c8 (virtio-gpu: use class function for ctrl
queue handlers)
15/25 Checking commit a3b8da45651f (virtio-gpu: move virgl handle_ctrl)
16/25 Checking commit ac638311147a (virtio-gpu: move virgl gl_flushed)
17/25 Checking commit 8905ff2e695f (virtio-gpu: move virgl process_cmd)
WARNING: line over 80 characters
#103: FILE: include/hw/virtio/virtio-gpu.h:233:
+void virtio_gpu_simple_process_cmd(VirtIOGPU *g, struct
virtio_gpu_ctrl_command *cmd);
total: 0 errors, 1 warnings, 70 lines checked
Patch 17/25 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
18/25 Checking commit 1adf3905ab9f (virtio-gpu: move update_cursor_data)
19/25 Checking commit ab7db8cb32b1 (virtio-gpu: drop VIRGL() macro)
20/25 Checking commit ddd31ea968b8 (virtio-gpu: move virtio-gpu-gl-device to
separate module)
21/25 Checking commit db5bc9ca6371 (virtio-gpu: drop use_virgl_renderer)
ERROR: "foo * bar" should be "foo *bar"
#98: FILE: hw/display/virtio-gpu-gl.c:47:
+ memcpy(s->current_cursor->data, data, pixels * sizeof(uint32_t));
total: 1 errors, 0 warnings, 116 lines checked
Patch 21/25 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
22/25 Checking commit 9876e4d010ed (virtio-gpu: move fields to struct
VirtIOGPUGL)
23/25 Checking commit ec77853df000 (virtio-gpu: add virtio-gpu-gl-pci)
Use of uninitialized value $acpi_testexpected in string eq at
./scripts/checkpatch.pl line 1529.
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#36:
new file mode 100644
total: 0 errors, 1 warnings, 80 lines checked
Patch 23/25 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
24/25 Checking commit ce63459c6416 (modules: add have_vga)
25/25 Checking commit 13cb22dd106b (virtio-gpu: add virtio-vga-gl)
Use of uninitialized value $acpi_testexpected in string eq at
./scripts/checkpatch.pl line 1529.
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#36:
new file mode 100644
total: 0 errors, 1 warnings, 72 lines checked
Patch 25/25 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
20210510132051.2208563-1-kraxel@redhat.com/testing.checkpatch/?type=message">http://patchew.org/logs/20210510132051.2208563-1-kraxel@redhat.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
- [PULL 16/25] virtio-gpu: move virgl gl_flushed, (continued)
- [PULL 16/25] virtio-gpu: move virgl gl_flushed, Gerd Hoffmann, 2021/05/10
- [PULL 17/25] virtio-gpu: move virgl process_cmd, Gerd Hoffmann, 2021/05/10
- [PULL 18/25] virtio-gpu: move update_cursor_data, Gerd Hoffmann, 2021/05/10
- [PULL 20/25] virtio-gpu: move virtio-gpu-gl-device to separate module, Gerd Hoffmann, 2021/05/10
- [PULL 19/25] virtio-gpu: drop VIRGL() macro, Gerd Hoffmann, 2021/05/10
- [PULL 21/25] virtio-gpu: drop use_virgl_renderer, Gerd Hoffmann, 2021/05/10
- [PULL 22/25] virtio-gpu: move fields to struct VirtIOGPUGL, Gerd Hoffmann, 2021/05/10
- [PULL 23/25] virtio-gpu: add virtio-gpu-gl-pci, Gerd Hoffmann, 2021/05/10
- [PULL 24/25] modules: add have_vga, Gerd Hoffmann, 2021/05/10
- [PULL 25/25] virtio-gpu: add virtio-vga-gl, Gerd Hoffmann, 2021/05/10
- Re: [PULL 00/25] Vga 20210510 patches,
no-reply <=
- Re: [PULL 00/25] Vga 20210510 patches, Peter Maydell, 2021/05/12