Re: [Qemu-block] [Qemu-devel] [PULL 00/17] Block layer patches

2018-05-31 Thread Peter Maydell
On 30 May 2018 at 16:56, Kevin Wolf  wrote:
>   Merge remote-tracking branch 
> 'remotes/edgar/tags/edgar/xilinx-next-2018-05-29-v1.for-upstream' into 
> staging (2018-05-29 13:01:11 +0100)
>
> are available in the git repository at:
>
>   git://repo.or.cz/qemu/kevin.git tags/for-upstream
>
> for you to fetch changes up to 3fb588a0f2c006122c34e1960a15c87ae2b927eb:
>
>   block/create: Mark blockdev-create stable (2018-05-30 13:31:18 +0200)
>
> 
> Block layer patches:
>
> - Add blockdev-create job
> - qcow2: Silence Coverity false positive
>

Applied, thanks.

-- PMM



Re: [Qemu-block] [Qemu-devel] [PULL 00/17] Block layer patches

2018-05-30 Thread no-reply
Hi,

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

Type: series
Message-id: 20180530155647.14090-1-kw...@redhat.com
Subject: [Qemu-devel] [PULL 00/17] Block layer patches

=== 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/20180530061035.22592-1-...@kaod.org -> 
patchew/20180530061035.22592-1-...@kaod.org
 t [tag update]
patchew/20180530112424.204835-1-vsement...@virtuozzo.com -> 
patchew/20180530112424.204835-1-vsement...@virtuozzo.com
 * [new tag]   patchew/20180530155647.14090-1-kw...@redhat.com -> 
patchew/20180530155647.14090-1-kw...@redhat.com
Switched to a new branch 'test'
88e058942a block/create: Mark blockdev-create stable
139ec5ed96 qemu-iotests: Rewrite 213 for blockdev-create job
d62e0ba08d qemu-iotests: Rewrite 212 for blockdev-create job
70de2cafaf qemu-iotests: Rewrite 211 for blockdev-create job
4d54cadc55 qemu-iotests: Rewrite 210 for blockdev-create job
53d9bc5bcc qemu-iotests: Rewrite 207 for blockdev-create job
b63d2e6b5d qemu-iotests: Rewrite 206 for blockdev-create job
c23f6e5ca3 qemu-iotests: iotests.py helper for non-file protocols
28c4344887 qemu-iotests: Add VM.run_job()
d753acd0c3 qemu-iotests: Add iotests.img_info_log()
5eec72445e qemu-iotests: Add VM.qmp_log()
9384dd2be8 qemu-iotests: Add VM.get_qmp_events_filtered()
b42f5f7dd4 block/create: Make x-blockdev-create a job
4cbf01e842 job: Add error message for failing jobs
5399bceb2c vhdx: Fix vhdx_co_create() return value
e7e26c08e7 vdi: Fix vdi_co_do_create() return value
b412f757bc qcow2: Fix Coverity warning when calculating the refcount cache size

=== OUTPUT BEGIN ===
Checking PATCH 1/17: qcow2: Fix Coverity warning when calculating the refcount 
cache size...
Checking PATCH 2/17: vdi: Fix vdi_co_do_create() return value...
Checking PATCH 3/17: vhdx: Fix vhdx_co_create() return value...
Checking PATCH 4/17: job: Add error message for failing jobs...
Checking PATCH 5/17: block/create: Make x-blockdev-create a job...
Checking PATCH 6/17: qemu-iotests: Add VM.get_qmp_events_filtered()...
Checking PATCH 7/17: qemu-iotests: Add VM.qmp_log()...
Checking PATCH 8/17: qemu-iotests: Add iotests.img_info_log()...
ERROR: line over 90 characters
#44: FILE: tests/qemu-iotests/iotests.py:227:
+line = re.sub('uuid: [-a-f0-9]+', 'uuid: 
----', line)

total: 1 errors, 0 warnings, 30 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 9/17: qemu-iotests: Add VM.run_job()...
Checking PATCH 10/17: qemu-iotests: iotests.py helper for non-file protocols...
Checking PATCH 11/17: qemu-iotests: Rewrite 206 for blockdev-create job...
Checking PATCH 12/17: qemu-iotests: Rewrite 207 for blockdev-create job...
Checking PATCH 13/17: qemu-iotests: Rewrite 210 for blockdev-create job...
Checking PATCH 14/17: qemu-iotests: Rewrite 211 for blockdev-create job...
Checking PATCH 15/17: qemu-iotests: Rewrite 212 for blockdev-create job...
Checking PATCH 16/17: qemu-iotests: Rewrite 213 for blockdev-create job...
Checking PATCH 17/17: block/create: Mark blockdev-create stable...
=== OUTPUT END ===

Test command exited with code: 1


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