Re: [Qemu-devel] [PATCH v2 00/18] Block layer thread safety, part 1

2017-05-25 Thread Paolo Bonzini
On 25/05/2017 09:40, Fam Zheng wrote: > On Wed, 05/24 10:32, Paolo Bonzini wrote: >> >> Ping? >> > > Looks good to me except for the mingw 32bit build failure reported by patchew: > > http://patchew.org/QEMU/20170511144208.24075-1-pbonz...@redhat.com/ > > Do you want to fix it? Yes, I misread

Re: [Qemu-devel] [PATCH v2 00/18] Block layer thread safety, part 1

2017-05-25 Thread Fam Zheng
On Wed, 05/24 10:32, Paolo Bonzini wrote: > > Ping? > Looks good to me except for the mingw 32bit build failure reported by patchew: http://patchew.org/QEMU/20170511144208.24075-1-pbonz...@redhat.com/ Do you want to fix it? Fam

Re: [Qemu-devel] [PATCH v2 00/18] Block layer thread safety, part 1

2017-05-24 Thread Paolo Bonzini
On 11/05/2017 16:41, Paolo Bonzini wrote: > This series uses mutexes or atomic operations around core block layer > operations. The remaining parts include: > > I've removed the failing assertion in bdrv_aligned_pwritev in order to > test block migration. > > Paolo > > v1->v2: add missing com

Re: [Qemu-devel] [PATCH v2 00/18] Block layer thread safety, part 1

2017-05-16 Thread Stefan Hajnoczi
On Thu, May 11, 2017 at 04:41:50PM +0200, Paolo Bonzini wrote: > This series uses mutexes or atomic operations around core block layer > operations. The remaining parts include: > > I've removed the failing assertion in bdrv_aligned_pwritev in order to > test block migration. > > Paolo > > v1->

Re: [Qemu-devel] [PATCH v2 00/18] Block layer thread safety, part 1

2017-05-11 Thread no-reply
Hi, This series seems to have some coding style problems. See output below for more information: Message-id: 20170511144208.24075-1-pbonz...@redhat.com Subject: [Qemu-devel] [PATCH v2 00/18] Block layer thread safety, part 1 Type: series === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1

[Qemu-devel] [PATCH v2 00/18] Block layer thread safety, part 1

2017-05-11 Thread Paolo Bonzini
This series uses mutexes or atomic operations around core block layer operations. The remaining parts include: I've removed the failing assertion in bdrv_aligned_pwritev in order to test block migration. Paolo v1->v2: add missing comment for 'wakeup' member [Fam] rewrite throttle-groups