[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 0/5] Experimenting with tb-lookup tweaks
From: |
no-reply |
Subject: |
Re: [RFC PATCH 0/5] Experimenting with tb-lookup tweaks |
Date: |
Thu, 25 Feb 2021 07:45:46 -0800 (PST) |
Patchew URL:
20210224165811.11567-1-alex.bennee@linaro.org/">https://patchew.org/QEMU/20210224165811.11567-1-alex.bennee@linaro.org/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20210224165811.11567-1-alex.bennee@linaro.org
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/20210218201528.127099-1-eblake@redhat.com ->
patchew/20210218201528.127099-1-eblake@redhat.com
- [tag update] patchew/20210224055401.492407-1-jasowang@redhat.com ->
patchew/20210224055401.492407-1-jasowang@redhat.com
* [new tag] patchew/20210224165811.11567-1-alex.bennee@linaro.org ->
patchew/20210224165811.11567-1-alex.bennee@linaro.org
* [new tag] patchew/20210224165837.21983-1-vgoyal@redhat.com ->
patchew/20210224165837.21983-1-vgoyal@redhat.com
- [tag update] patchew/20210225032335.64245-1-aik@ozlabs.ru ->
patchew/20210225032335.64245-1-aik@ozlabs.ru
* [new tag] patchew/20210225054756.35962-1-linuxmaker@163.com ->
patchew/20210225054756.35962-1-linuxmaker@163.com
- [tag update] patchew/20210225131316.631940-1-pbonzini@redhat.com ->
patchew/20210225131316.631940-1-pbonzini@redhat.com
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
20210224165811.11567-1-alex.bennee@linaro.org/testing.checkpatch/?type=message">http://patchew.org/logs/20210224165811.11567-1-alex.bennee@linaro.org/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
- [RFC PATCH 0/5] Experimenting with tb-lookup tweaks, Alex Bennée, 2021/02/24
- [RFC PATCH 4/5] include/exec: lightly re-arrange TranslationBlock, Alex Bennée, 2021/02/24
- [RFC PATCH 5/5] include/exec/tb-lookup: try and reduce branch prediction issues, Alex Bennée, 2021/02/24
- [RFC PATCH 1/5] accel/tcg: rename tb_lookup__cpu_state and hoist state extraction, Alex Bennée, 2021/02/24
- [RFC PATCH 2/5] accel/tcg: move CF_CLUSTER calculation to curr_cflags, Alex Bennée, 2021/02/24
- [RFC PATCH 3/5] accel/tcg: drop the use of CF_HASH_MASK and rename params, Alex Bennée, 2021/02/24
- Re: [RFC PATCH 0/5] Experimenting with tb-lookup tweaks, Richard Henderson, 2021/02/24
- Re: [RFC PATCH 0/5] Experimenting with tb-lookup tweaks,
no-reply <=