Patchew URL: https://patchew.org/QEMU/20210123143128.1167797-1-pbonz...@redhat.com/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20210123143128.1167797-1-pbonz...@redhat.com Subject: [PULL 00/31] Misc patches for 2020-01-21 === 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/20210123143128.1167797-1-pbonz...@redhat.com -> patchew/20210123143128.1167797-1-pbonz...@redhat.com Switched to a new branch 'test' 4a3ceb3 qemu-option: warn for short-form boolean options 601145b qemu-option: move help handling to get_opt_name_value d44f2ed qemu-option: clean up id vs. list->merge_lists d70fb5f vnc: support "-vnc help" fe666fb qmp: remove deprecated "change" command 8581d24 hmp: remove "change vnc TARGET" command e784bbe acceptance: switch to QMP change-vnc-password command 82574e9 meson.build: Detect bzip2 program cc91009 meson.build: Declare global edk2_targets / install_edk2_blobs variables a8e31f6 meson: Add a section header for library dependencies 88a07a2 meson: Display crypto-related information altogether 0e8b853 meson: Display block layer information altogether 6336d44 meson: Display accelerators and selected targets altogether b332b01 meson: Summarize compilation-related information altogether b807fd5 meson: Summarize overall features altogether bfb3d06 meson: Display host binaries information altogether 86dcc24 meson: Summarize information related to directories first f678a10 meson: convert wixl detection to Meson f464b38 nsis: adjust for new MinGW paths e7b75e0 meson: Declare have_virtfs_proxy_helper in main meson.build a1e89ed slirp: update to git master 13a0c33 build-sys: add libvhost-user missing dependencies 51e6567 build-sys: set global arguments for cflags/ldflags 8922c06 build-sys: remove unused LIBS 08be6c9 ide: atapi: check logical block address and read size (CVE-2020-29443) 6f15f3c softmmu/physmem: Silence GCC 10 maybe-uninitialized error 201c7af util/cacheflush: Fix error generated by clang c6ae035 build-system: clean up TCG/TCI configury b981bdd x86/cpu: Use max host physical address if -cpu max option is applied 8c3b2ab configure: MinGW respect --bindir argument 35af975 runstate: cleanup reboot and panic actions === OUTPUT BEGIN === 1/31 Checking commit 35af975eb80f (runstate: cleanup reboot and panic actions) ERROR: line over 90 characters #118: FILE: softmmu/runstate.c:478: + || (panic_action == PANIC_ACTION_SHUTDOWN && shutdown_action == SHUTDOWN_ACTION_PAUSE)) { total: 1 errors, 0 warnings, 83 lines checked Patch 1/31 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 2/31 Checking commit 8c3b2ab803a2 (configure: MinGW respect --bindir argument) 3/31 Checking commit b981bddaf3b1 (x86/cpu: Use max host physical address if -cpu max option is applied) WARNING: line over 80 characters #27: FILE: target/i386/cpu.c:4322: + object_property_set_bool(OBJECT(cpu), "host-phys-bits", true, &error_abort); total: 0 errors, 1 warnings, 7 lines checked Patch 3/31 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 4/31 Checking commit c6ae035743a2 (build-system: clean up TCG/TCI configury) 5/31 Checking commit 201c7afcfdda (util/cacheflush: Fix error generated by clang) 6/31 Checking commit 6f15f3c12f74 (softmmu/physmem: Silence GCC 10 maybe-uninitialized error) 7/31 Checking commit 08be6c9b3812 (ide: atapi: check logical block address and read size (CVE-2020-29443)) 8/31 Checking commit 8922c06bbfeb (build-sys: remove unused LIBS) 9/31 Checking commit 51e6567ee41a (build-sys: set global arguments for cflags/ldflags) 10/31 Checking commit 13a0c33ae299 (build-sys: add libvhost-user missing dependencies) 11/31 Checking commit a1e89edfc8f7 (slirp: update to git master) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #162: deleted file mode 160000 total: 0 errors, 1 warnings, 98 lines checked Patch 11/31 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 12/31 Checking commit e7b75e08c98c (meson: Declare have_virtfs_proxy_helper in main meson.build) 13/31 Checking commit f464b383c7da (nsis: adjust for new MinGW paths) 14/31 Checking commit f678a100a38d (meson: convert wixl detection to Meson) 15/31 Checking commit 86dcc24dd9dc (meson: Summarize information related to directories first) 16/31 Checking commit bfb3d06eba81 (meson: Display host binaries information altogether) 17/31 Checking commit b807fd55855f (meson: Summarize overall features altogether) 18/31 Checking commit b332b017845e (meson: Summarize compilation-related information altogether) 19/31 Checking commit 6336d44fb99d (meson: Display accelerators and selected targets altogether) 20/31 Checking commit 0e8b8535cc27 (meson: Display block layer information altogether) 21/31 Checking commit 88a07a2eae5b (meson: Display crypto-related information altogether) 22/31 Checking commit a8e31f6502b3 (meson: Add a section header for library dependencies) 23/31 Checking commit cc9100905835 (meson.build: Declare global edk2_targets / install_edk2_blobs variables) 24/31 Checking commit 82574e933d76 (meson.build: Detect bzip2 program) 25/31 Checking commit e784bbe429cc (acceptance: switch to QMP change-vnc-password command) 26/31 Checking commit 8581d24544ed (hmp: remove "change vnc TARGET" command) 27/31 Checking commit fe666fbc2118 (qmp: remove deprecated "change" command) 28/31 Checking commit d70fb5faea9d (vnc: support "-vnc help") 29/31 Checking commit d44f2ed07723 (qemu-option: clean up id vs. list->merge_lists) 30/31 Checking commit 601145bf2b6a (qemu-option: move help handling to get_opt_name_value) 31/31 Checking commit 4a3ceb3536d1 (qemu-option: warn for short-form boolean options) WARNING: line over 80 characters #81: FILE: util/qemu-option.c:787: + warn_report("short-form boolean option '%s%s' deprecated", prefix, *name); WARNING: line over 80 characters #100: FILE: util/qemu-option.c:818: + p = get_opt_name_value(p, firstname, warn_on_flag, help_wanted, &option, &value); total: 0 errors, 2 warnings, 127 lines checked Patch 31/31 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/20210123143128.1167797-1-pbonz...@redhat.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com