Re: [Qemu-block] [Qemu-devel] [PATCH] iotests: Tweak 030 in order to trigger a race condition with parallel jobs

2017-12-09 Thread Alberto Garcia
On Fri 08 Dec 2017 08:13:48 PM CET, John Snow wrote: qemu_io('-f', iotests.imgfmt, -'-c', 'write -P %d %d %d' % (i, i*1024*1024, num_kb * 1024), +'-c', 'write -P 0xFF %dk %dk' % (i * 512, num_kb), >>> >>> I guess changing f

Re: [Qemu-block] [Qemu-devel] [PATCH] iotests: Tweak 030 in order to trigger a race condition with parallel jobs

2017-12-08 Thread John Snow
On 12/07/2017 02:34 PM, Alberto Garcia wrote: > On Thu 07 Dec 2017 08:16:41 PM CET, Eric Blake wrote: >>> qemu_io('-f', iotests.imgfmt, >>> -'-c', 'write -P %d %d %d' % (i, i*1024*1024, num_kb * >>> 1024), >>> +'-c', 'write -P 0xFF %dk %dk' %

Re: [Qemu-block] [Qemu-devel] [PATCH] iotests: Tweak 030 in order to trigger a race condition with parallel jobs

2017-12-07 Thread Alberto Garcia
On Thu 07 Dec 2017 08:16:41 PM CET, Eric Blake wrote: >> qemu_io('-f', iotests.imgfmt, >> -'-c', 'write -P %d %d %d' % (i, i*1024*1024, num_kb * >> 1024), >> +'-c', 'write -P 0xFF %dk %dk' % (i * 512, num_kb), > > I guess changing from a variabl

Re: [Qemu-block] [Qemu-devel] [PATCH] iotests: Tweak 030 in order to trigger a race condition with parallel jobs

2017-12-07 Thread Eric Blake
On 12/07/2017 11:08 AM, Alberto Garcia wrote: > This patch tweaks TestParallelOps in iotest 030 so it allocates data > in smaller regions (256KB/512KB instead of 512KB/1MB) and the > block-stream job in test_stream_commit() only needs to copy data that > is at the very end of the image. > > This w