Patchew URL: https://patchew.org/QEMU/20190128223118.5255-1-richard.hender...@linaro.org/
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Qemu-devel] [PATCH v2 00/12] target/arm: Implement ARMv8.5-BTI Type: series Message-id: 20190128223118.5255-1-richard.hender...@linaro.org === TEST SCRIPT BEGIN === #!/bin/bash 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 t [tag update] patchew/20190128223118.5255-1-richard.hender...@linaro.org -> patchew/20190128223118.5255-1-richard.hender...@linaro.org Switched to a new branch 'test' edf251e560 tests/tcg/aarch64: Add bti smoke test 7a7646284f linux-user/aarch64: Reset btype for syscalls and signals 59d6dcfdca target/arm: Enable BTI for -cpu max f7377fcdc1 target/arm: Add x-guarded-pages cpu property for user-only 23690ae82f target/arm: Set btype for indirect branches 10de2a2461 target/arm: Reset btype for direct branches a12bad9666 target/arm: Default handling of BTYPE during translation d0f08030ef target/arm: Cache the GP bit for a page in MemTxAttrs b226128ce3 exec: Add target-specific tlb bits to MemTxAttrs 7eab305810 target/arm: Add BT and BTYPE to tb->flags 49f36a5c20 target/arm: Add PSTATE.BTYPE ef5152464e target/arm: Introduce isar_feature_aa64_bti === OUTPUT BEGIN === 1/12 Checking commit ef5152464e73 (target/arm: Introduce isar_feature_aa64_bti) 2/12 Checking commit 49f36a5c2086 (target/arm: Add PSTATE.BTYPE) 3/12 Checking commit 7eab30581026 (target/arm: Add BT and BTYPE to tb->flags) 4/12 Checking commit b226128ce320 (exec: Add target-specific tlb bits to MemTxAttrs) ERROR: spaces prohibited around that ':' (ctx:WxW) #28: FILE: include/exec/memattrs.h:47: + unsigned int target_tlb_bit0 : 1; ^ ERROR: spaces prohibited around that ':' (ctx:WxW) #29: FILE: include/exec/memattrs.h:48: + unsigned int target_tlb_bit1 : 1; ^ ERROR: spaces prohibited around that ':' (ctx:WxW) #30: FILE: include/exec/memattrs.h:49: + unsigned int target_tlb_bit2 : 1; ^ total: 3 errors, 0 warnings, 16 lines checked Patch 4/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 5/12 Checking commit d0f08030ef1d (target/arm: Cache the GP bit for a page in MemTxAttrs) 6/12 Checking commit a12bad9666e5 (target/arm: Default handling of BTYPE during translation) ERROR: return is not a function, parentheses are not required #97: FILE: target/arm/translate-a64.c:13757: + return (tlb_hit(entry->addr_code, addr) && total: 1 errors, 0 warnings, 196 lines checked Patch 6/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 7/12 Checking commit 10de2a246194 (target/arm: Reset btype for direct branches) 8/12 Checking commit 23690ae82f20 (target/arm: Set btype for indirect branches) 9/12 Checking commit f7377fcdc121 (target/arm: Add x-guarded-pages cpu property for user-only) 10/12 Checking commit 59d6dcfdca71 (target/arm: Enable BTI for -cpu max) 11/12 Checking commit 7a7646284f29 (linux-user/aarch64: Reset btype for syscalls and signals) 12/12 Checking commit edf251e5609e (tests/tcg/aarch64: Add bti smoke test) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #32: new file mode 100644 ERROR: externs should be avoided in .c files #116: FILE: tests/tcg/aarch64/bti-crt.inc.c:13: +int main(void); total: 1 errors, 1 warnings, 128 lines checked Patch 12/12 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/20190128223118.5255-1-richard.hender...@linaro.org/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to patchew-de...@redhat.com
