Patchew URL: https://patchew.org/QEMU/20200219160953.13771-1-imamm...@redhat.com/
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PATCH v6 00/79] refactor main RAM allocation to use hostmem backend Message-id: 20200219160953.13771-1-imamm...@redhat.com 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 === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu - [tag update] patchew/20200211160318.453650-1-stefa...@redhat.com -> patchew/20200211160318.453650-1-stefa...@redhat.com - [tag update] patchew/20200217150246.29180-1-vsement...@virtuozzo.com -> patchew/20200217150246.29180-1-vsement...@virtuozzo.com * [new tag] patchew/20200219160953.13771-1-imamm...@redhat.com -> patchew/20200219160953.13771-1-imamm...@redhat.com Switched to a new branch 'test' 9584b56 tests:numa-test: use explicit memdev to specify node RAM 786ed5c tests/numa-test: make top level args dynamic and g_autofree(cli) cleanups 4ebc74d hostmem: fix strict bind policy ffac16f hostmem: introduce "prealloc-threads" property 8b38de9 make mem_path local variable c001c3b exec: drop bogus mem_path from qemu_ram_alloc_from_fd() ad1172d exec: cleanup qemu_minrampagesize()/qemu_maxrampagesize() f0530f1 remove no longer used memory_region_allocate_system_memory() 769e8d9 sparc/niagara: use memdev for RAM b255475 sparc/sun4m: use memdev for RAM fe3e7b7 sparc/leon3: use memdev for RAM 9fe680e ppc/virtex_ml507: use memdev for RAM ab74e54 ppc/spapr: use memdev for RAM b28f018 ppc/{ppc440_bamboo, sam460ex}: use memdev for RAM a0258e4 ppc/{ppc440_bamboo, sam460ex}: drop RAM size fixup 2dc9ce1 ppc/ppc405_boards: use memdev for RAM 4428dcf ppc/ppc405_boards: add RAM size checks 173a36d ppc/pnv: use memdev for RAM 8ee06e4 ppc/mac_oldworld: use memdev for RAM a5b5de0 ppc/mac_newworld: use memdev for RAM 9731664 ppc/e500: use memdev for RAM 3538e84 ppc/e500: drop RAM size fixup ec88838 mips/mips_r4k: use memdev for RAM ceefaa3 mips/mips_mipssim: use memdev for RAM 3a6e6ac mips/mips_malta: use memdev for RAM 7c3dd4c mips/mips_jazz: add max ram size check 2a9bded mips/mips_jazz: use memdev for RAM 0de3d9f mips/mips_fulong2e: use memdev for RAM dc7b6ba mips/mips_fulong2e: drop RAM size fixup 9389d6c mips/boston: use memdev for RAM 49b64ba m68k/next-cube: use memdev for RAM 32c245c m68k/mcf5208: use memdev for RAM 8591a17 m68k/q800: use memdev for RAM c55f97a m68k/an5206: use memdev for RAM dc8953c lm32/milkymist: use memdev for RAM 6047c08 lm32/lm32_boards: use memdev for RAM bd45778 x86/pc: use memdev for RAM 9ad5468 x86/microvm: use memdev for RAM 7c59c1e hppa: use memdev for RAM 17c38c7 cris/axis_dev88: use memdev for RAM c74e719 null-machine: use memdev for RAM 3a12fc6 s390x/s390-virtio-ccw: use memdev for RAM 87c8047 arm/xlnx-zcu102: use memdev for RAM e920159 arm/xlnx-versal-virt: use memdev for RAM 8182d3d arm/xilinx_zynq: use memdev for RAM c980096 arm/xilinx_zynq: drop RAM size fixup a72f680 arm/virt: use memdev for RAM 08b8ba0 arm/vexpress: use memdev for RAM 6cf41f5 arm/versatilepb: use memdev for RAM 3818ed9 arm/sbsa-ref: use memdev for RAM a4317ae arm/raspi: use memdev for RAM 778f432 arm/sabrelite: use memdev for RAM 7f1679d arm/palm: use memdev for RAM 238ea0e arm/omap_sx1: use memdev for RAM 7998beb arm/nseries: use memdev for RAM 3ed6131 arm/musicpal: use memdev for RAM 68637c3 arm/mps2: use memdev for RAM 70a2cb8 arm/mps2-tz: use memdev for RAM 4076cc9 arm/mcimx7d-sabre: use memdev for RAM 14dbfa5 arm/mcimx6ul-evk: use memdev for RAM 3865cfa arm/kzm: use memdev for RAM 462f1f4 arm/kzm: drop RAM size fixup 3f25b3f arm/integratorcp: use memdev for RAM eebd06a arm/imx25_pdk: use memdev for RAM bf350da arm/imx25_pdk: drop RAM size fixup 89c43bd arm/highbank: use memdev for RAM 4daf95d arm/digic_boards: use memdev for RAM 0f07fe3 arm/cubieboard: use memdev for RAM 00b9829 arm/collie: use memdev for RAM afcbaed arm/aspeed: use memdev for RAM 533eb41 arm/aspeed: actually check RAM size b844d82 alpha/dp264: use memdev for RAM fe64d06 vl.c: ensure that ram_size matches size of machine.memory-backend a1b18df vl.c: move -m parsing after memory backends has been processed 6b61c2c initialize MachineState::ram in NUMA case 82b911a machine: introduce convenience MachineState::ram 900c0ba machine: alias -mem-path and -mem-prealloc into memory-foo backend aa8b183 machine: introduce memory-backend property 68a86dc numa: remove deprecated -mem-path fallback to anonymous RAM === OUTPUT BEGIN === 1/79 Checking commit 68a86dc15ccd (numa: remove deprecated -mem-path fallback to anonymous RAM) 2/79 Checking commit aa8b183974b2 (machine: introduce memory-backend property) 3/79 Checking commit 900c0ba373aa (machine: alias -mem-path and -mem-prealloc into memory-foo backend) 4/79 Checking commit 82b911aaff3b (machine: introduce convenience MachineState::ram) 5/79 Checking commit 6b61c2c596e7 (initialize MachineState::ram in NUMA case) 6/79 Checking commit a1b18df9a484 (vl.c: move -m parsing after memory backends has been processed) 7/79 Checking commit fe64d06afc1c (vl.c: ensure that ram_size matches size of machine.memory-backend) 8/79 Checking commit b844d822cf62 (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 8/79 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 9/79 Checking commit 533eb415df2e (arm/aspeed: actually check RAM size) 10/79 Checking commit afcbaed668fc (arm/aspeed: use memdev for RAM) 11/79 Checking commit 00b9829f83c0 (arm/collie: use memdev for RAM) 12/79 Checking commit 0f07fe38e4a1 (arm/cubieboard: use memdev for RAM) 13/79 Checking commit 4daf95d60791 (arm/digic_boards: use memdev for RAM) 14/79 Checking commit 89c43bdf20ad (arm/highbank: use memdev for RAM) 15/79 Checking commit bf350daae024 (arm/imx25_pdk: drop RAM size fixup) 16/79 Checking commit eebd06abc6ac (arm/imx25_pdk: use memdev for RAM) 17/79 Checking commit 3f25b3f4e88f (arm/integratorcp: use memdev for RAM) 18/79 Checking commit 462f1f4bde2a (arm/kzm: drop RAM size fixup) 19/79 Checking commit 3865cfacfe47 (arm/kzm: use memdev for RAM) 20/79 Checking commit 14dbfa556ba2 (arm/mcimx6ul-evk: use memdev for RAM) 21/79 Checking commit 4076cc942932 (arm/mcimx7d-sabre: use memdev for RAM) 22/79 Checking commit 70a2cb8e8d0f (arm/mps2-tz: use memdev for RAM) 23/79 Checking commit 68637c3a3687 (arm/mps2: use memdev for RAM) 24/79 Checking commit 3ed61312bdb8 (arm/musicpal: use memdev for RAM) 25/79 Checking commit 7998beb9c2e2 (arm/nseries: use memdev for RAM) 26/79 Checking commit 238ea0e31156 (arm/omap_sx1: use memdev for RAM) 27/79 Checking commit 7f1679dc2cee (arm/palm: use memdev for RAM) 28/79 Checking commit 778f43267a3f (arm/sabrelite: use memdev for RAM) 29/79 Checking commit a4317ae8ba5e (arm/raspi: use memdev for RAM) 30/79 Checking commit 3818ed92dc5d (arm/sbsa-ref: use memdev for RAM) 31/79 Checking commit 6cf41f5586c4 (arm/versatilepb: use memdev for RAM) 32/79 Checking commit 08b8ba04c9d0 (arm/vexpress: use memdev for RAM) 33/79 Checking commit a72f6805f30c (arm/virt: use memdev for RAM) 34/79 Checking commit c9800965c1be (arm/xilinx_zynq: drop RAM size fixup) 35/79 Checking commit 8182d3d1f186 (arm/xilinx_zynq: use memdev for RAM) 36/79 Checking commit e9201598f428 (arm/xlnx-versal-virt: use memdev for RAM) 37/79 Checking commit 87c8047f65b4 (arm/xlnx-zcu102: use memdev for RAM) 38/79 Checking commit 3a12fc61af5c (s390x/s390-virtio-ccw: use memdev for RAM) 39/79 Checking commit c74e71908d28 (null-machine: use memdev for RAM) 40/79 Checking commit 17c38c759c05 (cris/axis_dev88: use memdev for RAM) 41/79 Checking commit 7c59c1e0cced (hppa: use memdev for RAM) 42/79 Checking commit 9ad54686924b (x86/microvm: use memdev for RAM) 43/79 Checking commit bd457782b3b0 (x86/pc: use memdev for RAM) 44/79 Checking commit 6047c08fd714 (lm32/lm32_boards: use memdev for RAM) 45/79 Checking commit dc8953c6be14 (lm32/milkymist: use memdev for RAM) 46/79 Checking commit c55f97a0e157 (m68k/an5206: use memdev for RAM) 47/79 Checking commit 8591a179af02 (m68k/q800: use memdev for RAM) 48/79 Checking commit 32c245cfaf27 (m68k/mcf5208: use memdev for RAM) 49/79 Checking commit 49b64ba90649 (m68k/next-cube: use memdev for RAM) 50/79 Checking commit 9389d6ce0b89 (mips/boston: use memdev for RAM) 51/79 Checking commit dc7b6ba5b20c (mips/mips_fulong2e: drop RAM size fixup) 52/79 Checking commit 0de3d9fba60f (mips/mips_fulong2e: use memdev for RAM) 53/79 Checking commit 2a9bded9a337 (mips/mips_jazz: use memdev for RAM) 54/79 Checking commit 7c3dd4c6a50e (mips/mips_jazz: add max ram size check) 55/79 Checking commit 3a6e6ac76293 (mips/mips_malta: use memdev for RAM) 56/79 Checking commit ceefaa3b245f (mips/mips_mipssim: use memdev for RAM) 57/79 Checking commit ec88838cdc77 (mips/mips_r4k: use memdev for RAM) 58/79 Checking commit 3538e846cb8d (ppc/e500: drop RAM size fixup) 59/79 Checking commit 9731664559a3 (ppc/e500: use memdev for RAM) 60/79 Checking commit a5b5de02ac5b (ppc/mac_newworld: use memdev for RAM) 61/79 Checking commit 8ee06e4ccb0f (ppc/mac_oldworld: use memdev for RAM) 62/79 Checking commit 173a36d8d165 (ppc/pnv: use memdev for RAM) 63/79 Checking commit 4428dcf7b990 (ppc/ppc405_boards: add RAM size checks) 64/79 Checking commit 2dc9ce13d210 (ppc/ppc405_boards: use memdev for RAM) 65/79 Checking commit a0258e4afa10 (ppc/{ppc440_bamboo, sam460ex}: drop RAM size fixup) 66/79 Checking commit b28f01880eda (ppc/{ppc440_bamboo, sam460ex}: use memdev for RAM) 67/79 Checking commit ab74e5431129 (ppc/spapr: use memdev for RAM) 68/79 Checking commit 9fe680ee7578 (ppc/virtex_ml507: use memdev for RAM) 69/79 Checking commit fe3e7b71e68a (sparc/leon3: use memdev for RAM) 70/79 Checking commit b2554752b1da (sparc/sun4m: use memdev for RAM) 71/79 Checking commit 769e8d93bebf (sparc/niagara: use memdev for RAM) 72/79 Checking commit f0530f14c7c3 (remove no longer used memory_region_allocate_system_memory()) 73/79 Checking commit ad1172d8e56e (exec: cleanup qemu_minrampagesize()/qemu_maxrampagesize()) 74/79 Checking commit c001c3b3d93a (exec: drop bogus mem_path from qemu_ram_alloc_from_fd()) 75/79 Checking commit 8b38de9f62cf (make mem_path local variable) 76/79 Checking commit ffac16fab33b (hostmem: introduce "prealloc-threads" property) 77/79 Checking commit 4ebc74dbbf7a (hostmem: fix strict bind policy) 78/79 Checking commit 786ed5c497fd (tests/numa-test: make top level args dynamic and g_autofree(cli) cleanups) 79/79 Checking commit 9584b5641981 (tests:numa-test: use explicit memdev to specify node RAM) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20200219160953.13771-1-imamm...@redhat.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com