Re: [PULL v3 00/15] virtio,pc,acpi: fixes, tests

[no-reply]

Patchew URL: 20200921112913.555392-1-mst@redhat.com/">https://patchew.org/QEMU/20200921112913.555392-1-mst@redhat.com/



Hi,

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

Type: series
Message-id: 20200921112913.555392-1-mst@redhat.com
Subject: [PULL v3 00/15] virtio,pc,acpi: fixes, tests

=== 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
Switched to a new branch 'test'
5ef45e3 virtio-iommu-pci: force virtio version 1
ee9176b virtio-iommu: Check gtrees are non null before destroying them
70803757 cphp: remove deprecated cpu-add command(s)
747e251 tests/qtest/vhost-user-test: enable the reconnect tests
a283884 tests/qtest/vhost-user-test: add migrate_reconnect test
21199da tests/qtest/vhost-user-test: add support for the vhost-user-blk device
d07a0dd tests/qtest/libqos/virtio-blk: add support for vhost-user-blk
270b93f tests/qtest/vhost-user-test: prepare the tests for adding new dev class
5c2d89c vhost: check queue state in the vhost_dev_set_log routine
929fa1b vhost: recheck dev state in the vhost_migration_log routine
426794b pc: fix auto_enable_numa_with_memhp/auto_enable_numa_with_memdev for 
the 5.0 machine
c06b877 virtio-mem: detach the element from the virtqueue when error occurs
38dfe22 vhost-vdpa: batch updating IOTLB mappings
aa21167 vhost: switch to use IOTLB v2 format
745e38b linux headers: sync to 5.9-rc4

=== OUTPUT BEGIN ===
1/15 Checking commit 745e38b52a25 (linux headers: sync to 5.9-rc4)
2/15 Checking commit aa2116789222 (vhost: switch to use IOTLB v2 format)
3/15 Checking commit 38dfe229f3e0 (vhost-vdpa: batch updating IOTLB mappings)
4/15 Checking commit c06b8779ce68 (virtio-mem: detach the element from the 
virtqueue when error occurs)
5/15 Checking commit 426794b44fc9 (pc: fix 
auto_enable_numa_with_memhp/auto_enable_numa_with_memdev for the 5.0 machine)
6/15 Checking commit 929fa1bfff8e (vhost: recheck dev state in the 
vhost_migration_log routine)
7/15 Checking commit 5c2d89cfca7e (vhost: check queue state in the 
vhost_dev_set_log routine)
8/15 Checking commit 270b93f4e8d9 (tests/qtest/vhost-user-test: prepare the 
tests for adding new dev class)
9/15 Checking commit d07a0ddb618a (tests/qtest/libqos/virtio-blk: add support 
for vhost-user-blk)
10/15 Checking commit 21199da52737 (tests/qtest/vhost-user-test: add support 
for the vhost-user-blk device)
ERROR: code indent should never use tabs
#252: FILE: tests/qtest/vhost-user-test.c:1070:
+^I    sizeof(msg->payload.config.region) + msg->payload.config.size;$

total: 1 errors, 0 warnings, 256 lines checked

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

11/15 Checking commit a2838844dadc (tests/qtest/vhost-user-test: add 
migrate_reconnect test)
12/15 Checking commit 747e2515ffaa (tests/qtest/vhost-user-test: enable the 
reconnect tests)
13/15 Checking commit 70803757e6f4 (cphp: remove deprecated cpu-add command(s))
14/15 Checking commit ee9176bebd38 (virtio-iommu: Check gtrees are non null 
before destroying them)
15/15 Checking commit 5ef45e322a30 (virtio-iommu-pci: force virtio version 1)
=== OUTPUT END ===

Test command exited with code: 1


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