Re: [Qemu-devel] [PATCH] nbd/server: Silence gcc false positive

[no-reply]

Hi,

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

Type: series
Message-id: address@hidden
Subject: [Qemu-devel] [PATCH] nbd/server: Silence gcc false positive

=== TEST SCRIPT BEGIN ===
#!/bin/bash

BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0

git config --local diff.renamelimit 0
git config --local diff.renames True
git config --local diff.algorithm histogram

commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
    echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..."
    if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then
        failed=1
        echo
    fi
    n=$((n+1))
done

exit $failed
=== TEST SCRIPT END ===

Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384
From https://github.com/patchew-project/qemu
 t [tag update]            patchew/address@hidden -> patchew/address@hidden
 * [new tag]               patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
6489e4b10c Merge remote-tracking branch 
'remotes/pmaydell/tags/pull-target-arm-20180622' into staging
10e84d2504 xen: Don't use memory_region_init_ram_nomigrate() in 
pci_assign_dev_load_option_rom()
aea21afaef vl.c: Don't zero-initialize statics for serial_hds
fea64baeab target/arm: Strict alignment for ARMv6-M and ARMv8-M Baseline
12c583c45b target/arm: Introduce ARM_FEATURE_M_MAIN
42fc936a30 hw/arm/mps2-tz.c: Instantiate MPCs
b03e0d28a2 hw/arm/iotkit: Wire up MPC interrupt lines
985db83c4a hw/arm/iotkit: Instantiate MPC
769d3448af hw/misc/iotkit-secctl.c: Implement SECMPCINTSTATUS
cdf0c49a6a hw/misc/tz_mpc.c: Honour the BLK_LUT settings in translate
7790999ca7 hw/misc/tz-mpc.c: Implement correct blocked-access behaviour
fa6dd4bba5 hw/misc/tz-mpc.c: Implement registers
d3cb43aea7 hw/misc/tz-mpc.c: Implement the Arm TrustZone Memory Protection 
Controller
80c3703588 xlnx-zynqmp: Swap Cortex-R5 for Cortex-R5F
60e383fc83 target-arm: Add the Cortex-R5F
6603ff9f6b hw/arm/virt: Increase max_cpus to 512
ad37e0623f hw/arm/virt: Use 256MB ECAM region by default
08e38e7a7e hw/arm/virt: Add virt-3.0 machine type
d844e5a9a5 hw/arm/virt: Add a new 256MB ECAM region
82ca568747 hw/arm/virt: Register two redistributor regions when necessary
f3a20d2d88 hw/arm/virt-acpi-build: Advertise one or two GICR structures
0bd011094d hw/arm/virt: GICv3 DT node with one or two redistributor regions
b095e84e4f hw/intc/arm_gicv3_kvm: Get prepared to handle multiple redist regions
77737a0dc0 hw/intc/arm_gicv3: Introduce redist-region-count array property
356069f49b target/arm: Allow KVM device address overwriting
0d9ee1e956 linux-headers: Update to kernel mainline commit b357bf602
e9eb7de541 target-arm: fix a segmentation fault due to illegal memory access
a2bd1a2dce target/arm: Minor cleanup for ARMv6-M 32-bit instructions
787260735f hw/intc/arm_gicv3: fix an extra left-shift when reading IPRIORITYR

=== OUTPUT BEGIN ===
Checking PATCH 1/29: hw/intc/arm_gicv3: fix an extra left-shift when reading 
IPRIORITYR...
Checking PATCH 2/29: target/arm: Minor cleanup for ARMv6-M 32-bit 
instructions...
Checking PATCH 3/29: target-arm: fix a segmentation fault due to illegal memory 
access...
Checking PATCH 4/29: linux-headers: Update to kernel mainline commit 
b357bf602...
Checking PATCH 5/29: target/arm: Allow KVM device address overwriting...
Checking PATCH 6/29: hw/intc/arm_gicv3: Introduce redist-region-count array 
property...
Checking PATCH 7/29: hw/intc/arm_gicv3_kvm: Get prepared to handle multiple 
redist regions...
Checking PATCH 8/29: hw/arm/virt: GICv3 DT node with one or two redistributor 
regions...
Checking PATCH 9/29: hw/arm/virt-acpi-build: Advertise one or two GICR 
structures...
Checking PATCH 10/29: hw/arm/virt: Register two redistributor regions when 
necessary...
Checking PATCH 11/29: hw/arm/virt: Add a new 256MB ECAM region...
Checking PATCH 12/29: hw/arm/virt: Add virt-3.0 machine type...
Checking PATCH 13/29: hw/arm/virt: Use 256MB ECAM region by default...
Checking PATCH 14/29: hw/arm/virt: Increase max_cpus to 512...
Checking PATCH 15/29: target-arm: Add the Cortex-R5F...
Checking PATCH 16/29: xlnx-zynqmp: Swap Cortex-R5 for Cortex-R5F...
Checking PATCH 17/29: hw/misc/tz-mpc.c: Implement the Arm TrustZone Memory 
Protection Controller...
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#86: 
new file mode 100644

total: 0 errors, 1 warnings, 504 lines checked

Your patch has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
Checking PATCH 18/29: hw/misc/tz-mpc.c: Implement registers...
Checking PATCH 19/29: hw/misc/tz-mpc.c: Implement correct blocked-access 
behaviour...
Checking PATCH 20/29: hw/misc/tz_mpc.c: Honour the BLK_LUT settings in 
translate...
Checking PATCH 21/29: hw/misc/iotkit-secctl.c: Implement SECMPCINTSTATUS...
ERROR: spaces required around that '*' (ctx:VxV)
#92: FILE: hw/misc/iotkit-secctl.c:711:
+    .subsections = (const VMStateDescription*[]) {
                                             ^

total: 1 errors, 0 warnings, 100 lines checked

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

Checking PATCH 22/29: hw/arm/iotkit: Instantiate MPC...
Checking PATCH 23/29: hw/arm/iotkit: Wire up MPC interrupt lines...
Checking PATCH 24/29: hw/arm/mps2-tz.c: Instantiate MPCs...
Checking PATCH 25/29: target/arm: Introduce ARM_FEATURE_M_MAIN...
Checking PATCH 26/29: target/arm: Strict alignment for ARMv6-M and ARMv8-M 
Baseline...
Checking PATCH 27/29: vl.c: Don't zero-initialize statics for serial_hds...
Checking PATCH 28/29: xen: Don't use memory_region_init_ram_nomigrate() in 
pci_assign_dev_load_option_rom()...
Checking PATCH 29/29: Merge remote-tracking branch 
'remotes/pmaydell/tags/pull-target-arm-20180622' into staging...
=== OUTPUT END ===

Test command exited with code: 1


---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to address@hidden