Patchew URL: https://patchew.org/QEMU/20210601150106.12761-1-richard.hender...@linaro.org/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20210601150106.12761-1-richard.hender...@linaro.org Subject: [PATCH v7 00/27] TCI fixes and cleanups === 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/20210601143749.1669-1-anaidu.go...@samsung.com -> patchew/20210601143749.1669-1-anaidu.go...@samsung.com * [new tag] patchew/20210601150106.12761-1-richard.hender...@linaro.org -> patchew/20210601150106.12761-1-richard.hender...@linaro.org Switched to a new branch 'test' ff73f2a tests/tcg: Increase timeout for TCI f786057 tcg/tci: Use {set,clear}_helper_retaddr 190ad1f tcg/tci: Remove the qemu_ld/st_type macros 1e2156a Revert "tcg/tci: Use exec/cpu_ldst.h interfaces" 5500fe7 tcg/tci: Split out tci_qemu_ld, tci_qemu_st cc70f98 tcg/tci: Implement add2, sub2 fe9d5c6 tcg/tci: Implement mulu2, muls2 2da224a tcg/tci: Implement clz, ctz, ctpop e4cd4f1 tcg/tci: Implement extract, sextract e637074 tcg/tci: Implement andc, orc, eqv, nand, nor fe78aa9 tcg/tci: Implement movcond 4244f97 tcg/tci: Implement goto_ptr d5129d0 tcg/tci: Change encoding to uint32_t units b9936ff tcg/tci: Remove tci_write_reg 55ddc15 tcg/tci: Emit setcond before brcond 7d88fcd tcg/tci: Reserve r13 for a temporary e4d9785 tcg/tci: Use ffi for calls 7c53811 tcg/tci: Move call-return regs to end of tcg_target_reg_alloc_order df9f663 tcg/tci: Improve tcg_target_call_clobber_regs 7b199c1 tcg: Build ffi data structures for helpers 4cdaf7c tcg: Add tcg_call_func b5d0c45 tcg: Store the TCGHelperInfo in the TCGOp for call 998791a accel/tcg: Add tcg call flags to plugins helpers 31812d5 plugins: Drop tcg_flags from struct qemu_plugin_dyn_cb b12d667 accel/tcg/plugin-gen: Drop inline markers 857f3d7 tcg: Add tcg_call_flags 7507c52 tcg: Combine dh_is_64bit and dh_is_signed to dh_typecode === OUTPUT BEGIN === 1/27 Checking commit 7507c521494e (tcg: Combine dh_is_64bit and dh_is_signed to dh_typecode) 2/27 Checking commit 857f3d7ba371 (tcg: Add tcg_call_flags) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #21: new file mode 100644 total: 0 errors, 1 warnings, 106 lines checked Patch 2/27 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 3/27 Checking commit b12d66702acb (accel/tcg/plugin-gen: Drop inline markers) 4/27 Checking commit 31812d57a48c (plugins: Drop tcg_flags from struct qemu_plugin_dyn_cb) 5/27 Checking commit 998791ad868e (accel/tcg: Add tcg call flags to plugins helpers) WARNING: line over 80 characters #27: FILE: accel/tcg/plugin-helpers.h:3: +DEF_HELPER_FLAGS_4(plugin_vcpu_mem_cb, TCG_CALL_NO_RWG, void, i32, i32, i64, ptr) total: 0 errors, 1 warnings, 6 lines checked Patch 5/27 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 6/27 Checking commit b5d0c45e0f8b (tcg: Store the TCGHelperInfo in the TCGOp for call) 7/27 Checking commit 4cdaf7c2074a (tcg: Add tcg_call_func) 8/27 Checking commit 7b199c1dd9c5 (tcg: Build ffi data structures for helpers) 9/27 Checking commit df9f6637a168 (tcg/tci: Improve tcg_target_call_clobber_regs) 10/27 Checking commit 7c538119a809 (tcg/tci: Move call-return regs to end of tcg_target_reg_alloc_order) 11/27 Checking commit e4d97852d0bf (tcg/tci: Use ffi for calls) ERROR: suspect code indent for conditional statements (8, 11) #74: FILE: tcg/tcg.c:2148: if (TCG_TARGET_REG_BITS < 64 && is_64bit) { + /* total: 1 errors, 0 warnings, 392 lines checked Patch 11/27 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 12/27 Checking commit 7d88fcde0f51 (tcg/tci: Reserve r13 for a temporary) 13/27 Checking commit 55ddc15b0444 (tcg/tci: Emit setcond before brcond) 14/27 Checking commit b9936ff729dd (tcg/tci: Remove tci_write_reg) 15/27 Checking commit d5129d03bd95 (tcg/tci: Change encoding to uint32_t units) 16/27 Checking commit 4244f975b69a (tcg/tci: Implement goto_ptr) 17/27 Checking commit fe78aa960301 (tcg/tci: Implement movcond) 18/27 Checking commit e6370741791a (tcg/tci: Implement andc, orc, eqv, nand, nor) 19/27 Checking commit e4cd4f1ec298 (tcg/tci: Implement extract, sextract) 20/27 Checking commit 2da224ab55ce (tcg/tci: Implement clz, ctz, ctpop) 21/27 Checking commit fe9d5c60a7de (tcg/tci: Implement mulu2, muls2) 22/27 Checking commit cc70f986db4d (tcg/tci: Implement add2, sub2) 23/27 Checking commit 5500fe728bcf (tcg/tci: Split out tci_qemu_ld, tci_qemu_st) 24/27 Checking commit 1e2156aa0738 (Revert "tcg/tci: Use exec/cpu_ldst.h interfaces") 25/27 Checking commit 190ad1f3880e (tcg/tci: Remove the qemu_ld/st_type macros) 26/27 Checking commit f7860570bfc8 (tcg/tci: Use {set,clear}_helper_retaddr) 27/27 Checking commit ff73f2a5fabe (tests/tcg: Increase timeout for TCI) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20210601150106.12761-1-richard.hender...@linaro.org/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com
