Re: [Qemu-block] [PATCH] block: Simplify bdrv_filter_default_perms()

2019-08-06 Thread Max Reitz
On 02.08.19 16:03, Kevin Wolf wrote: > The same change as commit 2b23f28639 ('block/copy-on-read: Fix > permissions for inactive node') made for the copy-on-read driver can be > made for bdrv_filter_default_perms(): Retaining the old permissions from > the BdrvChild if it is given complicates

Re: [Qemu-block] [PATCH v5 0/6] Allow Valgrind checking all QEMU processes

2019-08-06 Thread Andrey Shinkevich
PINGING... On 19/07/2019 19:30, Andrey Shinkevich wrote: > In the current implementation of the QEMU bash iotests, only qemu-io > processes may be run under the Valgrind, which is a useful tool for > finding memory usage issues. Let's allow the common.rc bash script > runing all the QEMU

Re: [Qemu-block] [PATCH v2] util/hbitmap: strict hbitmap_reset

2019-08-06 Thread Vladimir Sementsov-Ogievskiy
06.08.2019 19:09, Max Reitz wrote: > On 06.08.19 17:26, Vladimir Sementsov-Ogievskiy wrote: >> hbitmap_reset has an unobvious property: it rounds requested region up. >> It may provoke bugs, like in recently fixed write-blocking mode of >> mirror: user calls reset on unaligned region, not keeping

Re: [Qemu-block] [PATCH v2] util/hbitmap: strict hbitmap_reset

2019-08-06 Thread Max Reitz
On 06.08.19 17:26, Vladimir Sementsov-Ogievskiy wrote: > hbitmap_reset has an unobvious property: it rounds requested region up. > It may provoke bugs, like in recently fixed write-blocking mode of > mirror: user calls reset on unaligned region, not keeping in mind that > there are possible

[Qemu-block] [PATCH v2] util/hbitmap: strict hbitmap_reset

2019-08-06 Thread Vladimir Sementsov-Ogievskiy
hbitmap_reset has an unobvious property: it rounds requested region up. It may provoke bugs, like in recently fixed write-blocking mode of mirror: user calls reset on unaligned region, not keeping in mind that there are possible unrelated dirty bytes, covered by rounded-up region and information

Re: [Qemu-block] [PULL v2 0/7] Block patches for 4.1.0-rc4

2019-08-06 Thread Peter Maydell
On Tue, 6 Aug 2019 at 12:59, Max Reitz wrote: > > The following changes since commit 9bb68d34dda9be60335e73e65c8fb61bca035362: > > Merge remote-tracking branch > 'remotes/philmd-gitlab/tags/edk2-next-20190803' into staging (2019-08-05 > 11:05:36 +0100) > > are available in the Git repository

Re: [Qemu-block] [PATCH] util/hbitmap: fix unaligned reset

2019-08-06 Thread Vladimir Sementsov-Ogievskiy
06.08.2019 16:30, John Snow wrote: > > > On 8/6/19 8:39 AM, Vladimir Sementsov-Ogievskiy wrote: >> 03.08.2019 0:19, Max Reitz wrote: >>> On 02.08.19 20:58, Vladimir Sementsov-Ogievskiy wrote: hbitmap_reset is broken: it rounds up the requested region. It leads to the following bug,

Re: [Qemu-block] [PATCH] util/hbitmap: fix unaligned reset

2019-08-06 Thread John Snow
On 8/6/19 8:39 AM, Vladimir Sementsov-Ogievskiy wrote: > 03.08.2019 0:19, Max Reitz wrote: >> On 02.08.19 20:58, Vladimir Sementsov-Ogievskiy wrote: >>> hbitmap_reset is broken: it rounds up the requested region. It leads to >>> the following bug, which is shown by fixed test: >>> >>> assume

Re: [Qemu-block] [Qemu-devel] [PATCH 2/3] iotests: Enable -d for Python non-unittest tests

2019-08-06 Thread John Snow
On 8/6/19 5:51 AM, Kevin Wolf wrote: > Am 06.08.2019 um 00:19 hat John Snow geschrieben: >> >> >> On 8/2/19 10:07 AM, Kevin Wolf wrote: >>> Am 01.08.2019 um 19:57 hat Max Reitz geschrieben: On 01.08.19 17:17, Kevin Wolf wrote: > The part of iotests.main() that is related to the

Re: [Qemu-block] [PATCH] util/hbitmap: fix unaligned reset

2019-08-06 Thread Vladimir Sementsov-Ogievskiy
03.08.2019 0:19, Max Reitz wrote: > On 02.08.19 20:58, Vladimir Sementsov-Ogievskiy wrote: >> hbitmap_reset is broken: it rounds up the requested region. It leads to >> the following bug, which is shown by fixed test: >> >> assume granularity = 2 >> set(0, 3) # count becomes 4 >> reset(0, 1) #

[Qemu-block] [PULL v2 7/7] block/backup: disable copy_range for compressed backup

2019-08-06 Thread Max Reitz
From: Vladimir Sementsov-Ogievskiy Enabled by default copy_range ignores compress option. It's definitely unexpected for user. It's broken since introduction of copy_range usage in backup in 9ded4a011496. Signed-off-by: Vladimir Sementsov-Ogievskiy Message-id:

[Qemu-block] [PULL v2 5/7] mirror: Only mirror granularity-aligned chunks

2019-08-06 Thread Max Reitz
In write-blocking mode, all writes to the top node directly go to the target. We must only mirror chunks of data that are aligned to the job's granularity, because that is how the dirty bitmap works. Therefore, the request alignment for writes must be the job's granularity (in write-blocking

[Qemu-block] [PULL v2 4/7] iotests: Test incremental backup after truncation

2019-08-06 Thread Max Reitz
Signed-off-by: Max Reitz Message-id: 20190805152840.32190-1-mre...@redhat.com Signed-off-by: Max Reitz --- tests/qemu-iotests/124 | 38 ++ tests/qemu-iotests/124.out | 4 ++-- 2 files changed, 36 insertions(+), 6 deletions(-) diff --git

[Qemu-block] [PULL v2 6/7] iotests: Test unaligned blocking mirror write

2019-08-06 Thread Max Reitz
Signed-off-by: Max Reitz Message-id: 20190805113526.20319-1-mre...@redhat.com Reviewed-by: Vladimir Sementsov-Ogievskiy Signed-off-by: Max Reitz --- tests/qemu-iotests/151 | 25 + tests/qemu-iotests/151.out | 4 ++-- 2 files changed, 27 insertions(+), 2

[Qemu-block] [PULL v2 2/7] iotests: Test backup job with two guest writes

2019-08-06 Thread Max Reitz
Perform two guest writes to not yet backed up areas of an image, where the former touches an inner area of the latter. Before HEAD^, copy offloading broke this in two ways: (1) The target image differs from the reference image (what the source was when the backup started). (2) But you will

[Qemu-block] [PULL v2 3/7] util/hbitmap: update orig_size on truncate

2019-08-06 Thread Max Reitz
From: Vladimir Sementsov-Ogievskiy Without this, hbitmap_next_zero and hbitmap_next_dirty_area are broken after truncate. So, orig_size is broken since it's introduction in 76d570dc495c56bb. Fixes: 76d570dc495c56bb Signed-off-by: Vladimir Sementsov-Ogievskiy Message-id:

[Qemu-block] [PULL v2 0/7] Block patches for 4.1.0-rc4

2019-08-06 Thread Max Reitz
The following changes since commit 9bb68d34dda9be60335e73e65c8fb61bca035362: Merge remote-tracking branch 'remotes/philmd-gitlab/tags/edk2-next-20190803' into staging (2019-08-05 11:05:36 +0100) are available in the Git repository at: https://github.com/XanClic/qemu.git

[Qemu-block] [PULL v2 1/7] backup: Copy only dirty areas

2019-08-06 Thread Max Reitz
The backup job must only copy areas that the copy_bitmap reports as dirty. This is always the case when using traditional non-offloading backup, because it copies each cluster separately. When offloading the copy operation, we sometimes copy more than one cluster at a time, but we only check

Re: [Qemu-block] [Qemu-devel] [PULL 0/7] Block patches for 4.1.0-rc4

2019-08-06 Thread Peter Maydell
On Tue, 6 Aug 2019 at 12:12, Max Reitz wrote: > > On 06.08.19 12:12, Peter Maydell wrote: > > On Mon, 5 Aug 2019 at 18:01, Max Reitz wrote: > >> > >> On 05.08.19 18:59, Vladimir Sementsov-Ogievskiy wrote: > >>> 05.08.2019 19:37, Max Reitz wrote: > The following changes since commit >

Re: [Qemu-block] [Qemu-devel] [PULL 0/7] Block patches for 4.1.0-rc4

2019-08-06 Thread Max Reitz
On 06.08.19 12:12, Peter Maydell wrote: > On Mon, 5 Aug 2019 at 18:01, Max Reitz wrote: >> >> On 05.08.19 18:59, Vladimir Sementsov-Ogievskiy wrote: >>> 05.08.2019 19:37, Max Reitz wrote: The following changes since commit 9bb68d34dda9be60335e73e65c8fb61bca035362: Merge

[Qemu-block] [PATCH] block: file-posix: Fix alignment probing on glsuter

2019-08-06 Thread Nir Soffer
On Gluster storage with sector size of 4096 bytes, buf_align may be wrong; reading 4096 bytes into unaligned buffer succeeds. This probably happens because the actual read happens on the Gluster node with aligned buffer, and Gluster client does not enforce any alignment on the host. However

Re: [Qemu-block] [Qemu-devel] [PULL 0/7] Block patches for 4.1.0-rc4

2019-08-06 Thread Peter Maydell
On Mon, 5 Aug 2019 at 18:01, Max Reitz wrote: > > On 05.08.19 18:59, Vladimir Sementsov-Ogievskiy wrote: > > 05.08.2019 19:37, Max Reitz wrote: > >> The following changes since commit > >> 9bb68d34dda9be60335e73e65c8fb61bca035362: > >> > >>Merge remote-tracking branch > >>

Re: [Qemu-block] [Qemu-devel] [PATCH 2/3] iotests: Enable -d for Python non-unittest tests

2019-08-06 Thread Kevin Wolf
Am 06.08.2019 um 00:19 hat John Snow geschrieben: > > > On 8/2/19 10:07 AM, Kevin Wolf wrote: > > Am 01.08.2019 um 19:57 hat Max Reitz geschrieben: > >> On 01.08.19 17:17, Kevin Wolf wrote: > >>> The part of iotests.main() that is related to the implementation of the > >>> debug option -d and