05.05.2021 11:06, no-re...@patchew.org wrote:
Patchew URL:
https://patchew.org/QEMU/20210505075001.45041-1-vsement...@virtuozzo.com/
Hi,
This series seems to have some coding style problems. See output below for
more information:
Type: series
Message-id: 20210505075001.45041-1-vsement...@virtuozzo.com
Subject: [PATCH v5 00/11] 64bit block-layer: part II
=== 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/20210505045824.33880-1-liq...@163.com ->
patchew/20210505045824.33880-1-liq...@163.com
* [new tag] patchew/20210505075001.45041-1-vsement...@virtuozzo.com
-> patchew/20210505075001.45041-1-vsement...@virtuozzo.com
Switched to a new branch 'test'
3700aaf block/io: allow 64bit discard requests
54ba548 block: use int64_t instead of int in driver discard handlers
7a3aaa5 block: make BlockLimits::max_pdiscard 64bit
f80faf7 block/io: allow 64bit write-zeroes requests
4cc3310 block: use int64_t instead of int in driver write_zeroes handlers
136dd63 block: make BlockLimits::max_pwrite_zeroes 64bit
03a7c45 block: use int64_t instead of uint64_t in copy_range driver handlers
496fe8b block: use int64_t instead of uint64_t in driver write handlers
eefb31d block: use int64_t instead of uint64_t in driver read handlers
fb7a119 qcow2: check request on vmstate save/load path
23ba0f4 block/io: bring request check to bdrv_co_(read, write)v_vmstate
=== OUTPUT BEGIN ===
1/11 Checking commit 23ba0f4a09c1 (block/io: bring request check to
bdrv_co_(read, write)v_vmstate)
ERROR: Author email address is mangled by the mailing list
#2:
Author: Vladimir Sementsov-Ogievskiy via <qemu-de...@nongnu.org>
total: 1 errors, 0 warnings, 42 lines checked
Patch 1/11 has style problems, please review. If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
So, s/{}/()/ doesn't help.
2/11 Checking commit fb7a119cd9aa (qcow2: check request on vmstate save/load
path)
3/11 Checking commit eefb31d480b4 (block: use int64_t instead of uint64_t in
driver read handlers)
4/11 Checking commit 496fe8b2e7ce (block: use int64_t instead of uint64_t in
driver write handlers)
5/11 Checking commit 03a7c4534a58 (block: use int64_t instead of uint64_t in
copy_range driver handlers)
6/11 Checking commit 136dd63cc723 (block: make BlockLimits::max_pwrite_zeroes
64bit)
7/11 Checking commit 4cc3310d243e (block: use int64_t instead of int in driver
write_zeroes handlers)
8/11 Checking commit f80faf73a84a (block/io: allow 64bit write-zeroes requests)
9/11 Checking commit 7a3aaa587314 (block: make BlockLimits::max_pdiscard 64bit)
10/11 Checking commit 54ba54860b4a (block: use int64_t instead of int in driver
discard handlers)
11/11 Checking commit 3700aafec1ff (block/io: allow 64bit discard requests)
=== OUTPUT END ===
Test command exited with code: 1
The full log is available at
http://patchew.org/logs/20210505075001.45041-1-vsement...@virtuozzo.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-de...@redhat.com
--
Best regards,
Vladimir