[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 00/19] target/arm: sve load/store improvements
From: |
no-reply |
Subject: |
Re: [PATCH v5 00/19] target/arm: sve load/store improvements |
Date: |
Fri, 8 May 2020 18:45:58 -0700 (PDT) |
Patchew URL: https://patchew.org/QEMU/address@hidden/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Message-id: address@hidden
Subject: [PATCH v5 00/19] target/arm: sve load/store improvements
Type: series
=== 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 ===
Switched to a new branch 'test'
5531df6 target/arm: Remove sve_memopidx
32fcdbe target/arm: Reuse sve_probe_page for gather loads
18e6634 target/arm: Reuse sve_probe_page for scatter stores
1ed3f4e target/arm: Reuse sve_probe_page for gather first-fault loads
c8c5809 target/arm: Use SVEContLdSt for contiguous stores
ff13c74 target/arm: Update contiguous first-fault and no-fault loads
7a4fc9f target/arm: Use SVEContLdSt for multi-register contiguous loads
e162663 target/arm: Handle watchpoints in sve_ld1_r
0a4a70e target/arm: Use SVEContLdSt in sve_ld1_r
6bc90b5 target/arm: Adjust interface of sve_ld1_host_fn
17db59e target/arm: Add sve infrastructure for page lookup
b9da291 target/arm: Drop manual handling of set/clear_helper_retaddr
8fd1eb2 target/arm: Use cpu_*_data_ra for sve_ldst_tlb_fn
474601a accel/tcg: Add endian-specific cpu_{ld, st}* operations
bdf03c8 accel/tcg: Add probe_access_flags
0c484ed accel/tcg: Adjust probe_access call to page_check_range
8a9847f accel/tcg: Add block comment for probe_access
7ae56c8 exec: Fix cpu_watchpoint_address_matches address length
3f79d9a exec: Add block comments for watchpoint routines
=== OUTPUT BEGIN ===
1/19 Checking commit 3f79d9ad401d (exec: Add block comments for watchpoint
routines)
2/19 Checking commit 7ae56c835cf0 (exec: Fix cpu_watchpoint_address_matches
address length)
3/19 Checking commit 8a9847f53fbc (accel/tcg: Add block comment for
probe_access)
4/19 Checking commit 0c484ed7b14e (accel/tcg: Adjust probe_access call to
page_check_range)
5/19 Checking commit bdf03c877b12 (accel/tcg: Add probe_access_flags)
6/19 Checking commit 474601a95614 (accel/tcg: Add endian-specific cpu_{ld, st}*
operations)
7/19 Checking commit 8fd1eb2928fa (target/arm: Use cpu_*_data_ra for
sve_ldst_tlb_fn)
ERROR: spaces required around that '*' (ctx:VxV)
#63: FILE: target/arm/sve_helper.c:4029:
+ TLB(env, addr, (TYPEM)*(TYPEE *)(vd + H(reg_off)), ra); \
^
ERROR: spaces required around that '*' (ctx:WxV)
#153: FILE: target/arm/sve_helper.c:4162:
+ sve_ldst1_tlb_fn *tlb_fn)
^
total: 2 errors, 0 warnings, 455 lines checked
Patch 7/19 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
8/19 Checking commit b9da291f2b6c (target/arm: Drop manual handling of
set/clear_helper_retaddr)
9/19 Checking commit 17db59e78016 (target/arm: Add sve infrastructure for page
lookup)
WARNING: Block comments use a leading /* on a separate line
#32: FILE: target/arm/sve_helper.c:1633:
+/* Big-endian hosts need to frob the byte indices. If the copy
total: 0 errors, 1 warnings, 281 lines checked
Patch 9/19 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
10/19 Checking commit 6bc90b548569 (target/arm: Adjust interface of
sve_ld1_host_fn)
11/19 Checking commit 0a4a70e7e0ea (target/arm: Use SVEContLdSt in sve_ld1_r)
12/19 Checking commit e162663bddd0 (target/arm: Handle watchpoints in sve_ld1_r)
13/19 Checking commit 7a4fc9f72f73 (target/arm: Use SVEContLdSt for
multi-register contiguous loads)
14/19 Checking commit ff13c741c638 (target/arm: Update contiguous first-fault
and no-fault loads)
15/19 Checking commit c8c58097770c (target/arm: Use SVEContLdSt for contiguous
stores)
16/19 Checking commit 1ed3f4ec9df2 (target/arm: Reuse sve_probe_page for gather
first-fault loads)
17/19 Checking commit 18e66342e1c1 (target/arm: Reuse sve_probe_page for
scatter stores)
18/19 Checking commit 32fcdbe6e7eb (target/arm: Reuse sve_probe_page for gather
loads)
19/19 Checking commit 5531df6852bc (target/arm: Remove sve_memopidx)
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/address@hidden/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to address@hidden
- [PATCH v5 09/19] target/arm: Add sve infrastructure for page lookup, (continued)
- [PATCH v5 09/19] target/arm: Add sve infrastructure for page lookup, Richard Henderson, 2020/05/08
- [PATCH v5 12/19] target/arm: Handle watchpoints in sve_ld1_r, Richard Henderson, 2020/05/08
- [PATCH v5 10/19] target/arm: Adjust interface of sve_ld1_host_fn, Richard Henderson, 2020/05/08
- [PATCH v5 14/19] target/arm: Update contiguous first-fault and no-fault loads, Richard Henderson, 2020/05/08
- [PATCH v5 15/19] target/arm: Use SVEContLdSt for contiguous stores, Richard Henderson, 2020/05/08
- [PATCH v5 17/19] target/arm: Reuse sve_probe_page for scatter stores, Richard Henderson, 2020/05/08
- [PATCH v5 16/19] target/arm: Reuse sve_probe_page for gather first-fault loads, Richard Henderson, 2020/05/08
- [PATCH v5 18/19] target/arm: Reuse sve_probe_page for gather loads, Richard Henderson, 2020/05/08
- [PATCH v5 13/19] target/arm: Use SVEContLdSt for multi-register contiguous loads, Richard Henderson, 2020/05/08
- [PATCH v5 19/19] target/arm: Remove sve_memopidx, Richard Henderson, 2020/05/08
- Re: [PATCH v5 00/19] target/arm: sve load/store improvements,
no-reply <=
- Re: [PATCH v5 00/19] target/arm: sve load/store improvements, no-reply, 2020/05/09
- Re: [PATCH v5 00/19] target/arm: sve load/store improvements, Peter Maydell, 2020/05/11