Patchew URL: https://patchew.org/QEMU/[email protected]/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: [email protected] Subject: [RFC PATCH 0/5] Experimenting with tb-lookup tweaks === 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 - [tag update] patchew/[email protected] -> patchew/[email protected] - [tag update] patchew/[email protected] -> patchew/[email protected] * [new tag] patchew/[email protected] -> patchew/[email protected] * [new tag] patchew/[email protected] -> patchew/[email protected] - [tag update] patchew/[email protected] -> patchew/[email protected] * [new tag] patchew/[email protected] -> patchew/[email protected] - [tag update] patchew/[email protected] -> patchew/[email protected] Switched to a new branch 'test' 0be54b4 include/exec/tb-lookup: try and reduce branch prediction issues c6233de include/exec: lightly re-arrange TranslationBlock 18534bf accel/tcg: drop the use of CF_HASH_MASK and rename params 3a30caf accel/tcg: move CF_CLUSTER calculation to curr_cflags a135bea accel/tcg: rename tb_lookup__cpu_state and hoist state extraction === OUTPUT BEGIN === 1/5 Checking commit a135bea36366 (accel/tcg: rename tb_lookup__cpu_state and hoist state extraction) ERROR: "foo * bar" should be "foo *bar" #84: FILE: include/exec/tb-lookup.h:20: +static inline TranslationBlock * tb_lookup(CPUState *cpu, WARNING: line over 80 characters #85: FILE: include/exec/tb-lookup.h:21: + target_ulong pc, target_ulong cs_base, total: 1 errors, 1 warnings, 80 lines checked Patch 1/5 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 2/5 Checking commit 3a30caf5f47d (accel/tcg: move CF_CLUSTER calculation to curr_cflags) 3/5 Checking commit 18534bff0f1f (accel/tcg: drop the use of CF_HASH_MASK and rename params) 4/5 Checking commit c6233de83263 (include/exec: lightly re-arrange TranslationBlock) WARNING: Block comments use a leading /* on a separate line #35: FILE: include/exec/exec-all.h:465: + uint16_t size; /* size of target code for this block (1 <= WARNING: Block comments use * on subsequent lines #36: FILE: include/exec/exec-all.h:466: + uint16_t size; /* size of target code for this block (1 <= + size <= TARGET_PAGE_SIZE) */ WARNING: Block comments use a trailing */ on a separate line #36: FILE: include/exec/exec-all.h:466: + size <= TARGET_PAGE_SIZE) */ total: 0 errors, 3 warnings, 20 lines checked Patch 4/5 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 5/5 Checking commit 0be54b4ee146 (include/exec/tb-lookup: try and reduce branch prediction issues) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/[email protected]/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to [email protected]
