Re: [PATCH 00/14]: hw/i386/vmport: Bug fixes and improvements

[no-reply@patchew.org]

Patchew URL: https://patchew.org/QEMU/20200309235411.76587-1-liran.alon@oracle.com/



Hi,

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

Subject: [PATCH 00/14]: hw/i386/vmport: Bug fixes and improvements
Message-id: 20200309235411.76587-1-liran.alon@oracle.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/20200309235411.76587-1-liran.alon@oracle.com -> patchew/20200309235411.76587-1-liran.alon@oracle.com
Switched to a new branch 'test'
3a9a905 hw/i386/vmport: Assert vmport initialized before registering commands
a1e45e6 hw/i386/vmport: Add support for CMD_GETHZ
10e0ccf i386/cpu: Store LAPIC bus frequency in CPU structure
b4b58a3 hw/i386/vmport: Allow x2apic without IR
04d32bb hw/i386/vmport: Add support for CMD_GET_VCPU_INFO
7925d7e hw/i386/vmport: Add support for CMD_GETTIMEFULL
b919c87 hw/i386/vmport: Add support for CMD_GETTIME
87fe8e2 hw/i386/vmport: Add support for CMD_GETBIOSUUID
8cad96e hw/i386/vmport: Define enum for all commands
e167453 hw/i386/vmport: Report VMX type in CMD_GETVERSION
e70b2d1 hw/i386/vmport: Introduce vmx-version property
4c37f8b hw/i386/vmport: Add device properties
6aeb201 hw/i386/vmport: Set EAX to -1 on failed and unsupported commands
78864e8 hw/i386/vmport: Propagate IOPort read to vCPU EAX register

=== OUTPUT BEGIN ===
1/14 Checking commit 78864e828d9a (hw/i386/vmport: Propagate IOPort read to vCPU EAX register)
2/14 Checking commit 6aeb2014d982 (hw/i386/vmport: Set EAX to -1 on failed and unsupported commands)
3/14 Checking commit 4c37f8b7fcc5 (hw/i386/vmport: Add device properties)
4/14 Checking commit e70b2d1864b7 (hw/i386/vmport: Introduce vmx-version property)
5/14 Checking commit e1674539df49 (hw/i386/vmport: Report VMX type in CMD_GETVERSION)
6/14 Checking commit 8cad96e6c606 (hw/i386/vmport: Define enum for all commands)
7/14 Checking commit 87fe8e21b595 (hw/i386/vmport: Add support for CMD_GETBIOSUUID)
ERROR: "(foo*)" should be "(foo *)"
#33: FILE: hw/i386/vmport.c:128:
+    uint32_t *uuid_parts = (uint32_t*)(qemu_uuid.data);

total: 1 errors, 0 warnings, 39 lines checked

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

8/14 Checking commit b919c87b2d71 (hw/i386/vmport: Add support for CMD_GETTIME)
9/14 Checking commit 7925d7e58bc7 (hw/i386/vmport: Add support for CMD_GETTIMEFULL)
10/14 Checking commit 04d32bb772b9 (hw/i386/vmport: Add support for CMD_GET_VCPU_INFO)
ERROR: return is not a function, parentheses are not required
#41: FILE: hw/i386/vmport.c:185:
+    return (1 << VCPU_INFO_RESERVED_BIT);

total: 1 errors, 0 warnings, 38 lines checked

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

11/14 Checking commit b4b58a3754a3 (hw/i386/vmport: Allow x2apic without IR)
12/14 Checking commit 10e0ccf9b53a (i386/cpu: Store LAPIC bus frequency in CPU structure)
13/14 Checking commit a1e45e66e18f (hw/i386/vmport: Add support for CMD_GETHZ)
14/14 Checking commit 3a9a9056d1b3 (hw/i386/vmport: Assert vmport initialized before registering commands)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/20200309235411.76587-1-liran.alon@oracle.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com