Re: [PATCH v2 00/86] refactor main RAM allocation to use hostmem backend

[no-reply]

Patchew URL: https://patchew.org/QEMU/address@hidden/



Hi,

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

Subject: [PATCH v2 00/86] refactor main RAM allocation to use hostmem backend
Type: series
Message-id: address@hidden

=== 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'
2d14010 numa: remove deprecated implicit RAM distribution between nodes
bac70b8 numa: make exit() usage consistent
53861b0 tests:numa-test: use explicit memdev to specify node RAM
56297b3 tests:numa-test: make top level args dynamic and g_autofree(cli) 
cleanups
3e6445f numa: forbid '-numa node, mem' for 5.0 and newer machine types
d70a9f2 hostmem: fix strict bind policy
92d7974 hostmem: introduce "prealloc-threads" property
4fd9cfe make mem_path local variable
74d0083 exec: drop bogus mem_path from qemu_ram_alloc_from_fd()
afb329b exec: cleanup qemu_minrampagesize()/qemu_maxrampagesize()
7dfb5ed post conversion default_ram_id cleanup
f7cb074 remove no longer used memory_region_allocate_system_memory()
738f1a3 sparc:niagara: use memdev for RAM
e120e67 sparc:sun4m: use memdev for RAM
24b259e sparc:leon3: use memdev for RAM
4904e75 ppc:virtex_ml507: use memdev for RAM
65dbc2c ppc:virtex_ml507: remove unused arguments
ab63a98 ppc:spapr: use memdev for RAM
b4d5ea5 ppc:prep: use memdev for RAM
036f996 ppc:ppc440_bamboo/sam460ex: use memdev for RAM
73d3d69 ppc/{ppc440_bamboo,sam460x}: drop RAM size fixup
4525782 ppc:ppc405_boards: use memdev for RAM
53365b4 ppc:ppc405_boards: add RAM size checks
f025b4f ppc:pnv: use memdev for RAM
0329900 ppc:mac_oldworld: use memdev for RAM
92fb2bc ppc:mac_newworld: use memdev for RAM
eaab307 ppc:e500: use memdev for RAM
5bc78d1 ppc:e500: drop RAM size fixup
b959a8d mips:mips_r4k: use memdev for RAM
4fa91f4 mips:mips_mipssim: use memdev for RAM
f726ba3 mips:mips_malta: use memdev for RAM
e50c092 mips:mips_jazz: use memdev for RAM
d9a07ca mips:mips_fulong2e: use memdev for RAM
05d6a5e mips:mips_fulong2e: drop RAM size fixup
73027e0 mips:boston-cube: use memdev for RAM
eb6e93f m68k:next-cube: use memdev for RAM
3ad54a8 m68k:mcf5208: use memdev for RAM
31b4fbd m68k:an5206: use memdev for RAM
17a90ad lm32:milkymist: use memdev for RAM
934be97 lm32:lm32_boards: use memdev for RAM
debb3a4 x86:pc: use memdev for RAM
64a0031 x86:microvm: use memdev for RAM
c14b753 hppa: use memdev for RAM
9838169 hw/hppa/machine: Map the PDC memory region with higher priority
329d37d hw/hppa/machine: Restrict the total memory size to 3GB
82196f2 hw/hppa/machine: Correctly check the firmware is in PDC range
18f08804 cris:axis_dev88: use memdev for RAM
420420e null-machine: use memdev for RAM
fab1efc s390x:s390-virtio-ccw: use memdev for RAM
cb1a184 arm:xlnx-zcu102: use memdev for RAM
dafc5fe arm:xlnx-versal-virt: use memdev for RAM
8939184 arm:xilinx_zynq: use memdev for RAM
64008ac arm:xilinx_zynq: drop RAM size fixup
edc3824 arm:virt: use memdev for RAM
9c8ff93 arm:vexpress: use memdev for RAM
9f032bc arm:versatilepb: use memdev for RAM
03de308 arm:sbsa-ref: use memdev for RAM
bc94738 arm:sabrelite: use memdev for RAM
af2175c arm:raspi: use memdev for RAM
1943756 arm:palm: use memdev for RAM
b329f68 arm:omap_sx1: use memdev for RAM
04a9490 arm:nseries: use memdev for RAM
7769298 arm:musicpal: use memdev for RAM
6d13cec arm:mps2: use memdev for RAM
e0e52ec arm:mps2-tz: use memdev for RAM
d9daba7 arm:mcimx7d-sabre: use memdev for RAM
a701677 arm:mcimx6ul-evk: use memdev for RAM
b3879f1 arm:kzm: use memdev for RAM
91d718e arm:kzm: drop RAM size fixup
18d4ce5 arm:integratorcp: use memdev for RAM
a396753 arm:imx25_pdk: use memdev for RAM
77e06e0 arm:imx25_pdk: drop RAM size fixup
f404c60 arm:highbank: use memdev for RAM
c60b4f9 arm:digic_boards: use memdev for RAM
deb657f arm:cubieboard: use memdev for RAM
be1ddd2 arm:collie: use memdev for RAM
fca3baa arm:aspeed: use memdev for RAM
99a74b3 hw:aspeed: drop warning and bogus ram_size fixup
f2ccbf8 arm:aspeed: actually check RAM size
0fd04a9 arm:aspeed: convert valid RAM sizes to data
06f609f alpha:dp264: use memdev for RAM
40a1eec initialize MachineState::ram in NUMA case
9ef003b machine: introduce convenience MachineState::ram
16ac6d5 machine: alias -mem-path and -mem-prealloc into memory-foo backend
3026057 machine: introduce ram-memdev property
58ea704 numa: remove deprecated -mem-path fallback to anonymous RAM

=== OUTPUT BEGIN ===
1/86 Checking commit 58ea704ecde0 (numa: remove deprecated -mem-path fallback 
to anonymous RAM)
2/86 Checking commit 3026057508a7 (machine: introduce ram-memdev property)
3/86 Checking commit 16ac6d5dfca0 (machine: alias -mem-path and -mem-prealloc 
into memory-foo backend)
4/86 Checking commit 9ef003bf86b0 (machine: introduce convenience 
MachineState::ram)
5/86 Checking commit 40a1eec31617 (initialize MachineState::ram in NUMA case)
6/86 Checking commit 06f609f99e7f (alpha:dp264: use memdev for RAM)
ERROR: spaces required around that '*' (ctx:WxV)
#30: FILE: hw/alpha/alpha_sys.h:14:
+PCIBus *typhoon_init(MemoryRegion *, ISABus **, qemu_irq *, AlphaCPU *[4],
                                                                      ^

total: 1 errors, 0 warnings, 49 lines checked

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

7/86 Checking commit 0fd04a92fedc (arm:aspeed: convert valid RAM sizes to data)
8/86 Checking commit f2ccbf8db44d (arm:aspeed: actually check RAM size)
9/86 Checking commit 99a74b3ee938 (hw:aspeed: drop warning and bogus ram_size 
fixup)
10/86 Checking commit fca3baaab72b (arm:aspeed: use memdev for RAM)
11/86 Checking commit be1ddd2cd5f5 (arm:collie: use memdev for RAM)
12/86 Checking commit deb657fd5ee9 (arm:cubieboard: use memdev for RAM)
13/86 Checking commit c60b4f9d6cd2 (arm:digic_boards: use memdev for RAM)
14/86 Checking commit f404c60b1167 (arm:highbank: use memdev for RAM)
15/86 Checking commit 77e06e009693 (arm:imx25_pdk: drop RAM size fixup)
16/86 Checking commit a396753185ed (arm:imx25_pdk: use memdev for RAM)
17/86 Checking commit 18d4ce56967b (arm:integratorcp: use memdev for RAM)
18/86 Checking commit 91d718e5a482 (arm:kzm: drop RAM size fixup)
19/86 Checking commit b3879f1c1c31 (arm:kzm: use memdev for RAM)
20/86 Checking commit a701677008f3 (arm:mcimx6ul-evk: use memdev for RAM)
21/86 Checking commit d9daba789ed4 (arm:mcimx7d-sabre: use memdev for RAM)
22/86 Checking commit e0e52ecc43a7 (arm:mps2-tz: use memdev for RAM)
23/86 Checking commit 6d13cec0d94c (arm:mps2: use memdev for RAM)
24/86 Checking commit 77692988c319 (arm:musicpal: use memdev for RAM)
25/86 Checking commit 04a94904d1c1 (arm:nseries: use memdev for RAM)
26/86 Checking commit b329f683db50 (arm:omap_sx1: use memdev for RAM)
27/86 Checking commit 19437567583f (arm:palm: use memdev for RAM)
28/86 Checking commit af2175c080a5 (arm:raspi: use memdev for RAM)
29/86 Checking commit bc947389baf6 (arm:sabrelite: use memdev for RAM)
30/86 Checking commit 03de30800363 (arm:sbsa-ref: use memdev for RAM)
31/86 Checking commit 9f032bcc0136 (arm:versatilepb: use memdev for RAM)
32/86 Checking commit 9c8ff937f348 (arm:vexpress: use memdev for RAM)
33/86 Checking commit edc382449643 (arm:virt: use memdev for RAM)
34/86 Checking commit 64008acd6753 (arm:xilinx_zynq: drop RAM size fixup)
35/86 Checking commit 8939184e5a43 (arm:xilinx_zynq: use memdev for RAM)
36/86 Checking commit dafc5feec590 (arm:xlnx-versal-virt: use memdev for RAM)
37/86 Checking commit cb1a184d0d59 (arm:xlnx-zcu102: use memdev for RAM)
38/86 Checking commit fab1efc6394f (s390x:s390-virtio-ccw: use memdev for RAM)
39/86 Checking commit 420420e49f1c (null-machine: use memdev for RAM)
40/86 Checking commit 18f08804dee9 (cris:axis_dev88: use memdev for RAM)
41/86 Checking commit 82196f232d2f (hw/hppa/machine: Correctly check the 
firmware is in PDC range)
42/86 Checking commit 329d37dd2228 (hw/hppa/machine: Restrict the total memory 
size to 3GB)
43/86 Checking commit 98381695ee75 (hw/hppa/machine: Map the PDC memory region 
with higher priority)
44/86 Checking commit c14b75322640 (hppa: use memdev for RAM)
45/86 Checking commit 64a0031a7bad (x86:microvm: use memdev for RAM)
46/86 Checking commit debb3a43be01 (x86:pc: use memdev for RAM)
47/86 Checking commit 934be970aa9a (lm32:lm32_boards: use memdev for RAM)
48/86 Checking commit 17a90ad29627 (lm32:milkymist: use memdev for RAM)
49/86 Checking commit 31b4fbd90a84 (m68k:an5206: use memdev for RAM)
50/86 Checking commit 3ad54a87de75 (m68k:mcf5208: use memdev for RAM)
51/86 Checking commit eb6e93f3e88b (m68k:next-cube: use memdev for RAM)
52/86 Checking commit 73027e0d5845 (mips:boston-cube: use memdev for RAM)
53/86 Checking commit 05d6a5e8846c (mips:mips_fulong2e: drop RAM size fixup)
54/86 Checking commit d9a07ca1579d (mips:mips_fulong2e: use memdev for RAM)
55/86 Checking commit e50c092c974f (mips:mips_jazz: use memdev for RAM)
56/86 Checking commit f726ba38fe1f (mips:mips_malta: use memdev for RAM)
57/86 Checking commit 4fa91f449f97 (mips:mips_mipssim: use memdev for RAM)
58/86 Checking commit b959a8d62bde (mips:mips_r4k: use memdev for RAM)
59/86 Checking commit 5bc78d153004 (ppc:e500: drop RAM size fixup)
60/86 Checking commit eaab307ef25b (ppc:e500: use memdev for RAM)
61/86 Checking commit 92fb2bc7ec58 (ppc:mac_newworld: use memdev for RAM)
62/86 Checking commit 03299009858f (ppc:mac_oldworld: use memdev for RAM)
63/86 Checking commit f025b4fa1ed0 (ppc:pnv: use memdev for RAM)
64/86 Checking commit 53365b477d95 (ppc:ppc405_boards: add RAM size checks)
65/86 Checking commit 452578232871 (ppc:ppc405_boards: use memdev for RAM)
66/86 Checking commit 73d3d6916582 (ppc/{ppc440_bamboo,sam460x}: drop RAM size 
fixup)
67/86 Checking commit 036f996aa26f (ppc:ppc440_bamboo/sam460ex: use memdev for 
RAM)
68/86 Checking commit b4d5ea58deba (ppc:prep: use memdev for RAM)
69/86 Checking commit ab63a98c1459 (ppc:spapr: use memdev for RAM)
70/86 Checking commit 65dbc2c76e20 (ppc:virtex_ml507: remove unused arguments)
71/86 Checking commit 4904e75da051 (ppc:virtex_ml507: use memdev for RAM)
72/86 Checking commit 24b259e879cf (sparc:leon3: use memdev for RAM)
73/86 Checking commit e120e6728b65 (sparc:sun4m: use memdev for RAM)
74/86 Checking commit 738f1a334ba2 (sparc:niagara: use memdev for RAM)
75/86 Checking commit f7cb0749de91 (remove no longer used 
memory_region_allocate_system_memory())
76/86 Checking commit 7dfb5ede2cbe (post conversion default_ram_id cleanup)
77/86 Checking commit afb329be6ce1 (exec: cleanup 
qemu_minrampagesize()/qemu_maxrampagesize())
78/86 Checking commit 74d0083098a9 (exec: drop bogus mem_path from 
qemu_ram_alloc_from_fd())
79/86 Checking commit 4fd9cfe4fef6 (make mem_path local variable)
80/86 Checking commit 92d79746afa0 (hostmem: introduce "prealloc-threads" 
property)
81/86 Checking commit d70a9f290f8b (hostmem: fix strict bind policy)
82/86 Checking commit 3e6445f81e47 (numa: forbid '-numa node, mem' for 5.0 and 
newer machine types)
83/86 Checking commit 56297b3d07ef (tests:numa-test: make top level args 
dynamic and g_autofree(cli) cleanups)
84/86 Checking commit 53861b0c262a (tests:numa-test: use explicit memdev to 
specify node RAM)
85/86 Checking commit bac70b89e82c (numa: make exit() usage consistent)
86/86 Checking commit 2d14010cf86c (numa: remove deprecated implicit RAM 
distribution between nodes)
=== 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