mbox series

[PULL,00/76] target/microblaze improvements

Message ID 20200831160601.833692-1-richard.henderson@linaro.org
Headers show
Series target/microblaze improvements | expand

Message

Richard Henderson Aug. 31, 2020, 4:04 p.m. UTC
At Edgar's request, generating the pull-request for this.

r~



The following changes since commit 39335fab59e11cfda9b7cf63929825db2dd3a3e0:

  Merge remote-tracking branch 'remotes/vivier2/tags/linux-user-for-5.2-pull-request' into staging (2020-08-28 22:30:11 +0100)

are available in the Git repository at:

  https://github.com/rth7680/qemu.git tags/pull-mb-20200831

for you to fetch changes up to 7233c0d0b0f5b65fcf6d6423de069ddf9f916392:

  target/microblaze: Reduce linux-user address space to 32-bit (2020-08-31 08:45:45 -0700)

----------------------------------------------------------------
Convert microblaze to generic translator loop
Convert microblaze to decodetree
Fix mb_cpu_transaction_failed
Other misc cleanups

----------------------------------------------------------------
Richard Henderson (76):
      tests/tcg: Add microblaze to arches filter
      tests/tcg: Do not require FE_TOWARDZERO
      tests/tcg: Do not require FE_* exception bits
      target/microblaze: Tidy gdbstub
      target/microblaze: Split out PC from env->sregs
      target/microblaze: Split out MSR from env->sregs
      target/microblaze: Split out EAR from env->sregs
      target/microblaze: Split out ESR from env->sregs
      target/microblaze: Split out FSR from env->sregs
      target/microblaze: Split out BTR from env->sregs
      target/microblaze: Split out EDR from env->sregs
      target/microblaze: Split the cpu_SR array
      target/microblaze: Fix width of PC and BTARGET
      target/microblaze: Fix width of MSR
      target/microblaze: Fix width of ESR
      target/microblaze: Fix width of FSR
      target/microblaze: Fix width of BTR
      target/microblaze: Fix width of EDR
      target/microblaze: Remove cpu_ear
      target/microblaze: Tidy raising of exceptions
      target/microblaze: Mark raise_exception as noreturn
      target/microblaze: Remove helper_debug and env->debug
      target/microblaze: Rename env_* tcg variables to cpu_*
      target/microblaze: Tidy mb_tcg_init
      target/microblaze: Split out MSR[C] to its own variable
      target/microblaze: Use DISAS_NORETURN
      target/microblaze: Check singlestep_enabled in gen_goto_tb
      target/microblaze: Convert to DisasContextBase
      target/microblaze: Convert to translator_loop
      target/microblaze: Remove SIM_COMPAT
      target/microblaze: Remove DISAS_GNU
      target/microblaze: Remove empty D macros
      target/microblaze: Remove LOG_DIS
      target/microblaze: Ensure imm constant is always available
      target/microblaze: Add decodetree infrastructure
      target/microblaze: Convert dec_add to decodetree
      target/microblaze: Convert dec_sub to decodetree
      target/microblaze: Implement cmp and cmpu inline
      target/microblaze: Convert dec_pattern to decodetree
      target/microblaze: Convert dec_and, dec_or, dec_xor to decodetree
      target/microblaze: Convert dec_mul to decodetree
      target/microblaze: Convert dec_div to decodetree
      target/microblaze: Unwind properly when raising divide-by-zero
      target/microblaze: Convert dec_bit to decodetree
      target/microblaze: Convert dec_barrel to decodetree
      target/microblaze: Convert dec_imm to decodetree
      target/microblaze: Convert dec_fpu to decodetree
      target/microblaze: Fix cpu unwind for fpu exceptions
      target/microblaze: Mark fpu helpers TCG_CALL_NO_WG
      target/microblaze: Replace MSR_EE_FLAG with MSR_EE
      target/microblaze: Cache mem_index in DisasContext
      target/microblaze: Fix cpu unwind for stackprot
      target/microblaze: Convert dec_load and dec_store to decodetree
      target/microblaze: Assert no overlap in flags making up tb_flags
      target/microblaze: Move bimm to BIMM_FLAG
      target/microblaze: Fix no-op mb_cpu_transaction_failed
      target/microblaze: Store "current" iflags in insn_start
      tcg: Add tcg_get_insn_start_param
      target/microblaze: Use cc->do_unaligned_access
      target/microblaze: Replace clear_imm with tb_flags_to_set
      target/microblaze: Replace delayed_branch with tb_flags_to_set
      target/microblaze: Tidy mb_cpu_dump_state
      target/microblaze: Convert brk and brki to decodetree
      target/microblaze: Convert mbar to decodetree
      target/microblaze: Reorganize branching
      target/microblaze: Convert dec_br to decodetree
      target/microblaze: Convert dec_bcc to decodetree
      target/microblaze: Convert dec_rts to decodetree
      target/microblaze: Tidy do_rti, do_rtb, do_rte
      target/microblaze: Convert msrclr, msrset to decodetree
      target/microblaze: Convert dec_msr to decodetree
      target/microblaze: Convert dec_stream to decodetree
      target/microblaze: Remove last of old decoder
      target/microblaze: Remove cpu_R[0]
      target/microblaze: Add flags markup to some helpers
      target/microblaze: Reduce linux-user address space to 32-bit

 include/tcg/tcg.h                     |   15 +
 target/microblaze/cpu-param.h         |   15 +
 target/microblaze/cpu.h               |   67 +-
 target/microblaze/helper.h            |   49 +-
 target/microblaze/microblaze-decode.h |   59 -
 tests/tcg/multiarch/float_helpers.h   |   17 +
 target/microblaze/insns.decode        |  256 +++
 linux-user/elfload.c                  |    9 +-
 linux-user/microblaze/cpu_loop.c      |   26 +-
 linux-user/microblaze/signal.c        |    8 +-
 target/microblaze/cpu.c               |    9 +-
 target/microblaze/gdbstub.c           |  193 +-
 target/microblaze/helper.c            |  164 +-
 target/microblaze/mmu.c               |    4 +-
 target/microblaze/op_helper.c         |  194 +-
 target/microblaze/translate.c         | 3223 +++++++++++++++++----------------
 tests/tcg/multiarch/float_convs.c     |    2 +
 tests/tcg/multiarch/float_madds.c     |    2 +
 target/microblaze/meson.build         |    3 +
 tests/tcg/configure.sh                |    2 +-
 20 files changed, 2292 insertions(+), 2025 deletions(-)
 delete mode 100644 target/microblaze/microblaze-decode.h
 create mode 100644 target/microblaze/insns.decode

Comments

no-reply@patchew.org Sept. 1, 2020, 12:21 a.m. UTC | #1
Patchew URL: https://patchew.org/QEMU/20200831160601.833692-1-richard.henderson@linaro.org/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Type: series
Message-id: 20200831160601.833692-1-richard.henderson@linaro.org
Subject: [PULL 00/76] target/microblaze improvements

=== 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
   4bc08c6..2f4c51c  master     -> master
 - [tag update]      patchew/20200826171005.4055015-1-ehabkost@redhat.com -> patchew/20200826171005.4055015-1-ehabkost@redhat.com
 - [tag update]      patchew/20200827113259.25064-1-peter.maydell@linaro.org -> patchew/20200827113259.25064-1-peter.maydell@linaro.org
 - [tag update]      patchew/20200827124335.30586-1-leif@nuviainc.com -> patchew/20200827124335.30586-1-leif@nuviainc.com
 - [tag update]      patchew/20200827173051.31050-1-sw@weilnetz.de -> patchew/20200827173051.31050-1-sw@weilnetz.de
 - [tag update]      patchew/20200827175520.32355-1-sw@weilnetz.de -> patchew/20200827175520.32355-1-sw@weilnetz.de
 - [tag update]      patchew/20200827210920.111611-1-sw@weilnetz.de -> patchew/20200827210920.111611-1-sw@weilnetz.de
 - [tag update]      patchew/20200828141929.77854-1-richard.henderson@linaro.org -> patchew/20200828141929.77854-1-richard.henderson@linaro.org
 - [tag update]      patchew/20200828180243.443016-1-richard.henderson@linaro.org -> patchew/20200828180243.443016-1-richard.henderson@linaro.org
 * [new tag]         patchew/20200831235146.36045-1-david@gibson.dropbear.id.au -> patchew/20200831235146.36045-1-david@gibson.dropbear.id.au
Switched to a new branch 'test'
0d7dedb target/microblaze: Reduce linux-user address space to 32-bit
5f8c3e2 target/microblaze: Add flags markup to some helpers
466a318 target/microblaze: Remove cpu_R[0]
b1fb8b5 target/microblaze: Remove last of old decoder
0dfefb5 target/microblaze: Convert dec_stream to decodetree
c8b363c target/microblaze: Convert dec_msr to decodetree
7694f94 target/microblaze: Convert msrclr, msrset to decodetree
7dd5ab7 target/microblaze: Tidy do_rti, do_rtb, do_rte
539026b target/microblaze: Convert dec_rts to decodetree
4a40a45 target/microblaze: Convert dec_bcc to decodetree
eac8dd6 target/microblaze: Convert dec_br to decodetree
224c085 target/microblaze: Reorganize branching
f419c16 target/microblaze: Convert mbar to decodetree
c8837be target/microblaze: Convert brk and brki to decodetree
3a87b51 target/microblaze: Tidy mb_cpu_dump_state
1424196 target/microblaze: Replace delayed_branch with tb_flags_to_set
0ae7328 target/microblaze: Replace clear_imm with tb_flags_to_set
0954fbf target/microblaze: Use cc->do_unaligned_access
3c786be tcg: Add tcg_get_insn_start_param
15f0392 target/microblaze: Store "current" iflags in insn_start
dd4c8f4 target/microblaze: Fix no-op mb_cpu_transaction_failed
dd0128a target/microblaze: Move bimm to BIMM_FLAG
640419e target/microblaze: Assert no overlap in flags making up tb_flags
c53dbea target/microblaze: Convert dec_load and dec_store to decodetree
c725fe0 target/microblaze: Fix cpu unwind for stackprot
dbd989d target/microblaze: Cache mem_index in DisasContext
d927735 target/microblaze: Replace MSR_EE_FLAG with MSR_EE
d284d29 target/microblaze: Mark fpu helpers TCG_CALL_NO_WG
7e91cb3 target/microblaze: Fix cpu unwind for fpu exceptions
0a82365 target/microblaze: Convert dec_fpu to decodetree
fdc326c target/microblaze: Convert dec_imm to decodetree
46ffb06 target/microblaze: Convert dec_barrel to decodetree
fc30359 target/microblaze: Convert dec_bit to decodetree
9e7444e target/microblaze: Unwind properly when raising divide-by-zero
38ac916 target/microblaze: Convert dec_div to decodetree
a05d432 target/microblaze: Convert dec_mul to decodetree
15fe607 target/microblaze: Convert dec_and, dec_or, dec_xor to decodetree
6320c68 target/microblaze: Convert dec_pattern to decodetree
9551f17 target/microblaze: Implement cmp and cmpu inline
873ee28 target/microblaze: Convert dec_sub to decodetree
165699d target/microblaze: Convert dec_add to decodetree
ff24f67 target/microblaze: Add decodetree infrastructure
212839a target/microblaze: Ensure imm constant is always available
c6fc5d4 target/microblaze: Remove LOG_DIS
6b9d549 target/microblaze: Remove empty D macros
634c628 target/microblaze: Remove DISAS_GNU
f6e12e6 target/microblaze: Remove SIM_COMPAT
fd60a2a target/microblaze: Convert to translator_loop
137c956 target/microblaze: Convert to DisasContextBase
e9d715a target/microblaze: Check singlestep_enabled in gen_goto_tb
c2aa981 target/microblaze: Use DISAS_NORETURN
24e5371 target/microblaze: Split out MSR[C] to its own variable
089d389 target/microblaze: Tidy mb_tcg_init
35e3d1a target/microblaze: Rename env_* tcg variables to cpu_*
504bece target/microblaze: Remove helper_debug and env->debug
d8b5940 target/microblaze: Mark raise_exception as noreturn
7608d65 target/microblaze: Tidy raising of exceptions
8945896 target/microblaze: Remove cpu_ear
b74816d target/microblaze: Fix width of EDR
9be24ee target/microblaze: Fix width of BTR
3f9dfbc target/microblaze: Fix width of FSR
cb55e5f target/microblaze: Fix width of ESR
8859a93 target/microblaze: Fix width of MSR
e7de39d target/microblaze: Fix width of PC and BTARGET
4b26167 target/microblaze: Split the cpu_SR array
1455001 target/microblaze: Split out EDR from env->sregs
67e6dce target/microblaze: Split out BTR from env->sregs
256029a target/microblaze: Split out FSR from env->sregs
7989da1 target/microblaze: Split out ESR from env->sregs
bb70861 target/microblaze: Split out EAR from env->sregs
dd5b138 target/microblaze: Split out MSR from env->sregs
97999ea target/microblaze: Split out PC from env->sregs
7de26dc target/microblaze: Tidy gdbstub
32104b84 tests/tcg: Do not require FE_* exception bits
2381e76 tests/tcg: Do not require FE_TOWARDZERO
32ade00 tests/tcg: Add microblaze to arches filter

=== OUTPUT BEGIN ===
1/76 Checking commit 32ade0099e18 (tests/tcg: Add microblaze to arches filter)
ERROR: line over 90 characters
#24: FILE: tests/tcg/configure.sh:97:
+    alpha|cris|hppa|i386|lm32|microblaze|microblazeel|m68k|openrisc|riscv64|s390x|sh4|sparc64)

total: 1 errors, 0 warnings, 8 lines checked

Patch 1/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

2/76 Checking commit 2381e76ba247 (tests/tcg: Do not require FE_TOWARDZERO)
3/76 Checking commit 32104b84fc2f (tests/tcg: Do not require FE_* exception bits)
4/76 Checking commit 7de26dc06890 (target/microblaze: Tidy gdbstub)
5/76 Checking commit 97999eaea67f (target/microblaze: Split out PC from env->sregs)
6/76 Checking commit dd5b138c2b12 (target/microblaze: Split out MSR from env->sregs)
7/76 Checking commit bb70861493b2 (target/microblaze: Split out EAR from env->sregs)
8/76 Checking commit 7989da1fafe5 (target/microblaze: Split out ESR from env->sregs)
9/76 Checking commit 256029a96446 (target/microblaze: Split out FSR from env->sregs)
10/76 Checking commit 67e6dce9f408 (target/microblaze: Split out BTR from env->sregs)
11/76 Checking commit 14550010ea94 (target/microblaze: Split out EDR from env->sregs)
12/76 Checking commit 4b26167a7837 (target/microblaze: Split the cpu_SR array)
ERROR: switch and case should be at the same indent
#125: FILE: target/microblaze/translate.c:543:
         switch (sr) {
+            case SR_PC:
[...]
+            case SR_MSR:
[...]
+            case SR_BTR:
[...]
+            case SR_EDR:

ERROR: switch and case should be at the same indent
#154: FILE: target/microblaze/translate.c:579:
         switch (sr) {
+            case SR_PC:
[...]
+            case SR_MSR:

total: 2 errors, 0 warnings, 287 lines checked

Patch 12/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

13/76 Checking commit e7de39d8569d (target/microblaze: Fix width of PC and BTARGET)
14/76 Checking commit 8859a934c315 (target/microblaze: Fix width of MSR)
15/76 Checking commit cb55e5fd1522 (target/microblaze: Fix width of ESR)
16/76 Checking commit 3f9dfbc2d82b (target/microblaze: Fix width of FSR)
17/76 Checking commit 9be24eea7275 (target/microblaze: Fix width of BTR)
18/76 Checking commit b74816d8c2ba (target/microblaze: Fix width of EDR)
19/76 Checking commit 89458966a79e (target/microblaze: Remove cpu_ear)
20/76 Checking commit 7608d65e99e3 (target/microblaze: Tidy raising of exceptions)
21/76 Checking commit d8b5940670c2 (target/microblaze: Mark raise_exception as noreturn)
22/76 Checking commit 504becee93e2 (target/microblaze: Remove helper_debug and env->debug)
23/76 Checking commit 35e3d1abbe73 (target/microblaze: Rename env_* tcg variables to cpu_*)
24/76 Checking commit 089d389698fa (target/microblaze: Tidy mb_tcg_init)
25/76 Checking commit 24e537176135 (target/microblaze: Split out MSR[C] to its own variable)
26/76 Checking commit c2aa981dc20c (target/microblaze: Use DISAS_NORETURN)
27/76 Checking commit e9d715a25322 (target/microblaze: Check singlestep_enabled in gen_goto_tb)
28/76 Checking commit 137c956c34e4 (target/microblaze: Convert to DisasContextBase)
29/76 Checking commit fd60a2a6cfe5 (target/microblaze: Convert to translator_loop)
30/76 Checking commit f6e12e6a6cd8 (target/microblaze: Remove SIM_COMPAT)
31/76 Checking commit 634c6287cd12 (target/microblaze: Remove DISAS_GNU)
32/76 Checking commit 6b9d54925c00 (target/microblaze: Remove empty D macros)
33/76 Checking commit c6fc5d4224a0 (target/microblaze: Remove LOG_DIS)
34/76 Checking commit 212839a3cd07 (target/microblaze: Ensure imm constant is always available)
35/76 Checking commit ff24f676c604 (target/microblaze: Add decodetree infrastructure)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#16: 
new file mode 100644

total: 0 errors, 1 warnings, 60 lines checked

Patch 35/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
36/76 Checking commit 165699da76c5 (target/microblaze: Convert dec_add to decodetree)
ERROR: spaces required around that '*' (ctx:WxV)
#256: FILE: target/microblaze/translate.c:330:
+static bool trans_zero(DisasContext *dc, arg_zero *arg)
                                                   ^

total: 1 errors, 0 warnings, 279 lines checked

Patch 36/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

37/76 Checking commit 873ee2858743 (target/microblaze: Convert dec_sub to decodetree)
38/76 Checking commit 9551f1759c3a (target/microblaze: Implement cmp and cmpu inline)
39/76 Checking commit 6320c68f6ccb (target/microblaze: Convert dec_pattern to decodetree)
40/76 Checking commit 15fe607e07f7 (target/microblaze: Convert dec_and, dec_or, dec_xor to decodetree)
41/76 Checking commit a05d43282444 (target/microblaze: Convert dec_mul to decodetree)
42/76 Checking commit 38ac91626de4 (target/microblaze: Convert dec_div to decodetree)
43/76 Checking commit 9e7444e60cb7 (target/microblaze: Unwind properly when raising divide-by-zero)
44/76 Checking commit fc30359d501b (target/microblaze: Convert dec_bit to decodetree)
ERROR: spaces required around that '*' (ctx:WxV)
#160: FILE: target/microblaze/translate.c:545:
+static bool trans_wdic(DisasContext *dc, arg_wdic *a)
                                                   ^

total: 1 errors, 0 warnings, 227 lines checked

Patch 44/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

45/76 Checking commit 46ffb06bbf4c (target/microblaze: Convert dec_barrel to decodetree)
46/76 Checking commit fdc326c0c8ae (target/microblaze: Convert dec_imm to decodetree)
ERROR: spaces required around that '*' (ctx:WxV)
#33: FILE: target/microblaze/translate.c:480:
+static bool trans_imm(DisasContext *dc, arg_imm *arg)
                                                 ^

total: 1 errors, 0 warnings, 44 lines checked

Patch 46/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

47/76 Checking commit 0a823659b840 (target/microblaze: Convert dec_fpu to decodetree)
48/76 Checking commit 7e91cb362270 (target/microblaze: Fix cpu unwind for fpu exceptions)
49/76 Checking commit d284d293a1f6 (target/microblaze: Mark fpu helpers TCG_CALL_NO_WG)
50/76 Checking commit d9277357cbd6 (target/microblaze: Replace MSR_EE_FLAG with MSR_EE)
51/76 Checking commit dbd989dfe4e1 (target/microblaze: Cache mem_index in DisasContext)
52/76 Checking commit c725fe02b326 (target/microblaze: Fix cpu unwind for stackprot)
53/76 Checking commit c53dbeaa6e85 (target/microblaze: Convert dec_load and dec_store to decodetree)
WARNING: Block comments use a leading /* on a separate line
#350: FILE: target/microblaze/translate.c:944:
+        /* FIXME: if the alignment is wrong, we should restore the value

total: 0 errors, 1 warnings, 791 lines checked

Patch 53/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
54/76 Checking commit 640419e5c31b (target/microblaze: Assert no overlap in flags making up tb_flags)
55/76 Checking commit dd0128a2fa65 (target/microblaze: Move bimm to BIMM_FLAG)
56/76 Checking commit dd4c8f4b01f8 (target/microblaze: Fix no-op mb_cpu_transaction_failed)
57/76 Checking commit 15f0392b1027 (target/microblaze: Store "current" iflags in insn_start)
58/76 Checking commit 3c786be3d027 (tcg: Add tcg_get_insn_start_param)
59/76 Checking commit 0954fbfc7898 (target/microblaze: Use cc->do_unaligned_access)
ERROR: spaces required around that '<<' (ctx:VxV)
#37: FILE: target/microblaze/cpu.h:82:
+#define          ESR_W         (1<<11) /* Unaligned word access */
                                  ^

total: 1 errors, 0 warnings, 204 lines checked

Patch 59/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

60/76 Checking commit 0ae7328b9266 (target/microblaze: Replace clear_imm with tb_flags_to_set)
61/76 Checking commit 1424196cc193 (target/microblaze: Replace delayed_branch with tb_flags_to_set)
62/76 Checking commit 3a87b5189820 (target/microblaze: Tidy mb_cpu_dump_state)
63/76 Checking commit c8837be1e305 (target/microblaze: Convert brk and brki to decodetree)
64/76 Checking commit f419c16adfcc (target/microblaze: Convert mbar to decodetree)
ERROR: spaces required around that '+' (ctx:ExV)
#59: FILE: target/microblaze/translate.c:1153:
+                       +offsetof(CPUState, halted));
                        ^

total: 1 errors, 0 warnings, 111 lines checked

Patch 64/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

65/76 Checking commit 224c085312a4 (target/microblaze: Reorganize branching)
66/76 Checking commit eac8dd68662b (target/microblaze: Convert dec_br to decodetree)
67/76 Checking commit 4a40a45501d7 (target/microblaze: Convert dec_bcc to decodetree)
ERROR: spaces required around that '*' (ctx:WxV)
#125: FILE: target/microblaze/translate.c:1134:
+    static bool trans_##NAME(DisasContext *dc, arg_typea_bc *arg)       \
                                                             ^

ERROR: spaces required around that '*' (ctx:WxV)
#127: FILE: target/microblaze/translate.c:1136:
+    static bool trans_##NAME##d(DisasContext *dc, arg_typea_bc *arg)    \
                                                                ^

ERROR: spaces required around that '*' (ctx:WxV)
#129: FILE: target/microblaze/translate.c:1138:
+    static bool trans_##NAME##i(DisasContext *dc, arg_typeb_bc *arg)    \
                                                                ^

ERROR: spaces required around that '*' (ctx:WxV)
#131: FILE: target/microblaze/translate.c:1140:
+    static bool trans_##NAME##id(DisasContext *dc, arg_typeb_bc *arg)   \
                                                                 ^

total: 4 errors, 0 warnings, 179 lines checked

Patch 67/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

68/76 Checking commit 539026b12c05 (target/microblaze: Convert dec_rts to decodetree)
69/76 Checking commit 7dd5ab73d21a (target/microblaze: Tidy do_rti, do_rtb, do_rte)
70/76 Checking commit 7694f94e3e85 (target/microblaze: Convert msrclr, msrset to decodetree)
71/76 Checking commit c8b363c50946 (target/microblaze: Convert dec_msr to decodetree)
72/76 Checking commit 0dfefb59890f (target/microblaze: Convert dec_stream to decodetree)
73/76 Checking commit b1fb8b5e7ace (target/microblaze: Remove last of old decoder)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#17: 
deleted file mode 100644

total: 0 errors, 1 warnings, 128 lines checked

Patch 73/76 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
74/76 Checking commit 466a31810f81 (target/microblaze: Remove cpu_R[0])
75/76 Checking commit 5f8c3e2d6c17 (target/microblaze: Add flags markup to some helpers)
76/76 Checking commit 0d7dedb8e895 (target/microblaze: Reduce linux-user address space to 32-bit)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20200831160601.833692-1-richard.henderson@linaro.org/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
no-reply@patchew.org Sept. 1, 2020, 12:32 a.m. UTC | #2
Patchew URL: https://patchew.org/QEMU/20200831160601.833692-1-richard.henderson@linaro.org/



Hi,

This series failed the docker-mingw@fedora build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.

=== TEST SCRIPT BEGIN ===
#! /bin/bash
export ARCH=x86_64
make docker-image-fedora V=1 NETWORK=1
time make docker-test-mingw@fedora J=14 NETWORK=1
=== TEST SCRIPT END ===

Host machine cpu: x86_64
Target machine cpu family: x86
Target machine cpu: x86_64
../src/meson.build:10: WARNING: Module unstable-keyval has no backwards or forwards compatibility and might not exist in future releases.
Program sh found: YES
Program python3 found: YES (/usr/bin/python3)
C++ compiler for the host machine: x86_64-w64-mingw32-g++ (gcc 9.2.1 "x86_64-w64-mingw32-g++ (GCC) 9.2.1 20190827 (Fedora MinGW 9.2.1-6.fc32)")
---
             Linux keyring: NO

Found ninjatool-1.8 at /tmp/qemu-test/build/ninjatool
WARNING: custom_target 'shared QAPI source files' has more than one output! Using the first one.
WARNING: custom_target 'QGA QAPI files' has more than one output! Using the first one.
WARNING: custom_target 'QAPI files for qemu-storage-daemon' has more than one output! Using the first one.
WARNING: custom_target 'QAPI doc' has more than one output! Using the first one.
WARNING: custom_target 'tools man pages' has more than one output! Using the first one.
WARNING: custom_target 'system man pages' has more than one output! Using the first one.
Command line for building ['libcommon.fa'] is long, using a response file
/usr/bin/python3 -B /tmp/qemu-test/src/meson/meson.py introspect --tests | /usr/bin/python3 -B scripts/mtest2make.py > Makefile.mtest
./ninjatool -t ninja2make --omit clean dist uninstall < build.ninja > Makefile.ninja
---
Host machine cpu: i386
Target machine cpu family: x86
Target machine cpu: i386
../src/meson.build:10: WARNING: Module unstable-keyval has no backwards or forwards compatibility and might not exist in future releases.
Program sh found: YES
Program python3 found: YES (/usr/bin/python3)
C++ compiler for the host machine: i686-w64-mingw32-g++ (gcc 9.2.1 "i686-w64-mingw32-g++ (GCC) 9.2.1 20190827 (Fedora MinGW 9.2.1-6.fc32)")
---
             Linux keyring: NO

Found ninjatool-1.8 at /tmp/qemu-test/build/ninjatool
WARNING: custom_target 'shared QAPI source files' has more than one output! Using the first one.
WARNING: custom_target 'QGA QAPI files' has more than one output! Using the first one.
WARNING: custom_target 'QAPI files for qemu-storage-daemon' has more than one output! Using the first one.
WARNING: custom_target 'QAPI doc' has more than one output! Using the first one.
WARNING: custom_target 'tools man pages' has more than one output! Using the first one.
WARNING: custom_target 'system man pages' has more than one output! Using the first one.
Command line for building ['libcommon.fa'] is long, using a response file
/usr/bin/python3 -B /tmp/qemu-test/src/meson/meson.py introspect --tests | /usr/bin/python3 -B scripts/mtest2make.py > Makefile.mtest
./ninjatool -t ninja2make --omit clean dist uninstall < build.ninja > Makefile.ninja
---
                 from /tmp/qemu-test/src/include/exec/exec-all.h:26,
                 from ../src/target/i386/machine.c:3:
/tmp/qemu-test/src/include/tcg/tcg.h: In function 'tcg_get_insn_start_param':
/tmp/qemu-test/src/include/tcg/tcg.h:796:49: error: left shift count >= width of type [-Werror=shift-count-overflow]
  796 |            (tcg_get_insn_param(op, arg * 2 + 1) << 32);
      |                                                 ^~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:1533: libqemu-x86_64-softmmu.fa.p/target_i386_machine.c.obj] Error 1
make: *** Waiting for unfinished jobs....
In file included from ../src/hw/i386/kvmvapic.c:24:
/tmp/qemu-test/src/include/tcg/tcg.h: In function 'tcg_get_insn_start_param':
/tmp/qemu-test/src/include/tcg/tcg.h:796:49: error: left shift count >= width of type [-Werror=shift-count-overflow]
  796 |            (tcg_get_insn_param(op, arg * 2 + 1) << 32);
      |                                                 ^~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:1546: libqemu-x86_64-softmmu.fa.p/hw_i386_kvmvapic.c.obj] Error 1
In file included from /tmp/qemu-test/src/include/exec/cpu_ldst.h:295,
                 from ../src/target/i386/mpx_helper.c:23:
/tmp/qemu-test/src/include/tcg/tcg.h: In function 'tcg_get_insn_start_param':
/tmp/qemu-test/src/include/tcg/tcg.h:796:49: error: left shift count >= width of type [-Werror=shift-count-overflow]
  796 |            (tcg_get_insn_param(op, arg * 2 + 1) << 32);
      |                                                 ^~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:1554: libqemu-x86_64-softmmu.fa.p/target_i386_mpx_helper.c.obj] Error 1
In file included from /tmp/qemu-test/src/include/exec/cpu_ldst.h:295,
                 from /tmp/qemu-test/src/include/exec/exec-all.h:26,
                 from ../src/target/i386/bpt_helper.c:22:
/tmp/qemu-test/src/include/tcg/tcg.h: In function 'tcg_get_insn_start_param':
/tmp/qemu-test/src/include/tcg/tcg.h:796:49: error: left shift count >= width of type [-Werror=shift-count-overflow]
  796 |            (tcg_get_insn_param(op, arg * 2 + 1) << 32);
      |                                                 ^~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:1559: libqemu-x86_64-softmmu.fa.p/target_i386_bpt_helper.c.obj] Error 1
In file included from /tmp/qemu-test/src/include/exec/cpu_ldst.h:295,
                 from /tmp/qemu-test/src/include/exec/exec-all.h:26,
                 from ../src/target/i386/misc_helper.c:24:
/tmp/qemu-test/src/include/tcg/tcg.h: In function 'tcg_get_insn_start_param':
/tmp/qemu-test/src/include/tcg/tcg.h:796:49: error: left shift count >= width of type [-Werror=shift-count-overflow]
  796 |            (tcg_get_insn_param(op, arg * 2 + 1) << 32);
      |                                                 ^~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:1553: libqemu-x86_64-softmmu.fa.p/target_i386_misc_helper.c.obj] Error 1
In file included from /tmp/qemu-test/src/include/exec/cpu_ldst.h:295,
                 from /tmp/qemu-test/src/include/exec/exec-all.h:26,
                 from ../src/target/i386/cpu.c:27:
/tmp/qemu-test/src/include/tcg/tcg.h: In function 'tcg_get_insn_start_param':
/tmp/qemu-test/src/include/tcg/tcg.h:796:49: error: left shift count >= width of type [-Werror=shift-count-overflow]
  796 |            (tcg_get_insn_param(op, arg * 2 + 1) << 32);
      |                                                 ^~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:1556: libqemu-x86_64-softmmu.fa.p/target_i386_cpu.c.obj] Error 1
In file included from /tmp/qemu-test/src/include/exec/cpu_ldst.h:295,
                 from /tmp/qemu-test/src/include/exec/exec-all.h:26,
                 from ../src/target/i386/seg_helper.c:25:
/tmp/qemu-test/src/include/tcg/tcg.h: In function 'tcg_get_insn_start_param':
/tmp/qemu-test/src/include/tcg/tcg.h:796:49: error: left shift count >= width of type [-Werror=shift-count-overflow]
  796 |            (tcg_get_insn_param(op, arg * 2 + 1) << 32);
      |                                                 ^~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:1557: libqemu-x86_64-softmmu.fa.p/target_i386_seg_helper.c.obj] Error 1
In file included from /tmp/qemu-test/src/include/exec/cpu_ldst.h:295,
                 from /tmp/qemu-test/src/include/exec/exec-all.h:26,
                 from ../src/target/i386/translate.c:24:
/tmp/qemu-test/src/include/tcg/tcg.h: In function 'tcg_get_insn_start_param':
/tmp/qemu-test/src/include/tcg/tcg.h:796:49: error: left shift count >= width of type [-Werror=shift-count-overflow]
  796 |            (tcg_get_insn_param(op, arg * 2 + 1) << 32);
      |                                                 ^~
cc1: all warnings being treated as errors
make: *** [Makefile.ninja:1560: libqemu-x86_64-softmmu.fa.p/target_i386_translate.c.obj] Error 1
Traceback (most recent call last):
  File "./tests/docker/docker.py", line 709, in <module>
    sys.exit(main())
---
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['sudo', '-n', 'docker', 'run', '--label', 'com.qemu.instance.uuid=08be3efbe88a4e8ba88243d02c759bbc', '-u', '1001', '--security-opt', 'seccomp=unconfined', '--rm', '-e', 'TARGET_LIST=', '-e', 'EXTRA_CONFIGURE_OPTS=', '-e', 'V=', '-e', 'J=14', '-e', 'DEBUG=', '-e', 'SHOW_ENV=', '-e', 'CCACHE_DIR=/var/tmp/ccache', '-v', '/home/patchew/.cache/qemu-docker-ccache:/var/tmp/ccache:z', '-v', '/var/tmp/patchew-tester-tmp-h13z94vm/src/docker-src.2020-08-31-20.22.26.29191:/var/tmp/qemu:z,ro', 'qemu/fedora', '/var/tmp/qemu/run', 'test-mingw']' returned non-zero exit status 2.
filter=--filter=label=com.qemu.instance.uuid=08be3efbe88a4e8ba88243d02c759bbc
make[1]: *** [docker-run] Error 1
make[1]: Leaving directory `/var/tmp/patchew-tester-tmp-h13z94vm/src'
make: *** [docker-run-test-mingw@fedora] Error 2

real    9m57.739s
user    0m22.204s


The full log is available at
http://patchew.org/logs/20200831160601.833692-1-richard.henderson@linaro.org/testing.docker-mingw@fedora/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
Peter Maydell Sept. 1, 2020, 12:17 p.m. UTC | #3
On Mon, 31 Aug 2020 at 17:06, Richard Henderson
<richard.henderson@linaro.org> wrote:
>

> At Edgar's request, generating the pull-request for this.

>

> r~

>

>

>

> The following changes since commit 39335fab59e11cfda9b7cf63929825db2dd3a3e0:

>

>   Merge remote-tracking branch 'remotes/vivier2/tags/linux-user-for-5.2-pull-request' into staging (2020-08-28 22:30:11 +0100)

>

> are available in the Git repository at:

>

>   https://github.com/rth7680/qemu.git tags/pull-mb-20200831

>

> for you to fetch changes up to 7233c0d0b0f5b65fcf6d6423de069ddf9f916392:

>

>   target/microblaze: Reduce linux-user address space to 32-bit (2020-08-31 08:45:45 -0700)

>

> ----------------------------------------------------------------

> Convert microblaze to generic translator loop

> Convert microblaze to decodetree

> Fix mb_cpu_transaction_failed

> Other misc cleanups


The patchew complaints about shifts greater than the width of the
type look legit -- could you fix them and resend, please?

thanks
-- PMM