Re: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for bdrv_drain_begin/end

2017-03-22 Thread Jeff Cody
On Thu, Mar 16, 2017 at 02:28:47PM -0700, no-re...@patchew.org wrote: > Hi, > > This series seems to have some coding style problems. See output below for > more information: > > Type: series > Subject: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for > bdrv_drai

Re: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for bdrv_drain_begin/end

2017-03-22 Thread Jeff Cody
e'll chalk that up as "not very feasible", at least for the moment :) > >> > >> On 03/16/2017 05:28 PM, no-re...@patchew.org wrote: > >>> Hi, > >>> > >>> This series seems to have some coding style problems. See output below for > >>>

Re: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for bdrv_drain_begin/end

2017-03-22 Thread Jeff Cody
this case, or is it not very feasible to create one? > > On 03/16/2017 05:28 PM, no-re...@patchew.org wrote: > > Hi, > > > > This series seems to have some coding style problems. See output below for > > more information: > > > > Type: series > > Sub

Re: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for bdrv_drain_begin/end

2017-03-22 Thread John Snow
a more fool-proof automated testing method, but at the second I'm drawing a blank. >> >> On 03/16/2017 05:28 PM, no-re...@patchew.org wrote: >>> Hi, >>> >>> This series seems to have some coding style problems. See output below for >>> more information: &

Re: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for bdrv_drain_begin/end

2017-03-22 Thread John Snow
: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for > bdrv_drain_begin/end > Message-id: 20170316212351.13797-1-js...@redhat.com > > === TEST SCRIPT BEGIN === > #!/bin/bash > > BASE=base > n=1 > total=$(git log --oneline $BASE.. | wc -l) > failed=0 > > #

Re: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for bdrv_drain_begin/end

2017-03-16 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Subject: [Qemu-devel] [PATCH v2 0/3] block: pause block jobs for bdrv_drain_begin/end Message-id: 20170316212351.13797-1-js...@redhat.com === TEST SCRIPT BEGIN === #!/bin/bash BASE

[Qemu-devel] [PATCH v2 0/3] block: pause block jobs for bdrv_drain_begin/end

2017-03-16 Thread John Snow
Reference: https://bugzilla.redhat.com/show_bug.cgi?id=1367369#c8 It's possible to wedge QEMU if the guest tries to reset a virtio-pci device as QEMU is also using the drive for a blockjob. This patchset aims to allow us to safely pause/resume jobs attached to individual nodes in a manner similar