Re: [PATCH v2 00/10] mirror: allow switching from background to active mode

2024-03-11 Thread Peter Krempa
On Mon, Mar 11, 2024 at 18:51:18 +0300, Vladimir Sementsov-Ogievskiy wrote: > On 11.03.24 00:07, Peter Krempa wrote: > > On Thu, Mar 07, 2024 at 22:42:56 +0300, Vladimir Sementsov-Ogievskiy wrote: [...] > > Libvirt can adapt to any option that will give us the above seman

Re: [PATCH v2 00/10] mirror: allow switching from background to active mode

2024-03-10 Thread Peter Krempa
On Thu, Mar 07, 2024 at 22:42:56 +0300, Vladimir Sementsov-Ogievskiy wrote: > On 04.03.24 14:09, Peter Krempa wrote: > > On Mon, Mar 04, 2024 at 11:48:54 +0100, Kevin Wolf wrote: > > > Am 28.02.2024 um 19:07 hat Vladimir Sementsov-Ogievskiy geschrieben: > > > > On 03.

Re: [PATCH v2 00/10] mirror: allow switching from background to active mode

2024-03-04 Thread Peter Krempa
On Mon, Mar 04, 2024 at 11:48:54 +0100, Kevin Wolf wrote: > Am 28.02.2024 um 19:07 hat Vladimir Sementsov-Ogievskiy geschrieben: > > On 03.11.23 18:56, Markus Armbruster wrote: > > > Kevin Wolf writes: [...] > > > Is the job abstraction a failure? > > > > > > We have > > > > > >

Re: [PATCH v3 0/2] block: commit/stream: Allow users to request only format driver names in backing file format

2024-01-03 Thread Peter Krempa
On Tue, Dec 05, 2023 at 18:14:40 +0100, Peter Krempa wrote: > Please see patches for rationale. > > Libvirt patches using this new flag will be posted soon-ish (after > cleanup). > > v3: > - changed name of flag to 'backing-mask-protocol' (Eric) > - decided to k

Re: [PATCH v3 0/2] block: commit/stream: Allow users to request only format driver names in backing file format

2023-12-05 Thread Peter Krempa
On Tue, Dec 05, 2023 at 18:14:40 +0100, Peter Krempa wrote: > Please see patches for rationale. > > Libvirt patches using this new flag will be posted soon-ish (after > cleanup). https://lists.libvirt.org/archives/list/de...@lists.libvirt.org/message/GCCZP5ANYTPVXCIEYGSM5NWYCGDL23V6/

[PATCH v3 1/2] block: commit: Allow users to request only format driver names in backing file format

2023-12-05 Thread Peter Krempa
versions which didn't expect that a protocol driver name can appear in the backing file format field. Signed-off-by: Peter Krempa Reviewed-by: Vladimir Sementsov-Ogievskiy --- block.c| 37 +- block/commit.c | 6

[PATCH v3 2/2] block: stream: Allow users to request only format driver names in backing file format

2023-12-05 Thread Peter Krempa
versions which didn't expect that a protocol driver name can appear in the backing file format field. Signed-off-by: Peter Krempa Reviewed-by: Vladimir Sementsov-Ogievskiy --- block/monitor/block-hmp-cmds.c | 2 +- block/stream.c | 10 +- blockdev.c

[PATCH v3 0/2] block: commit/stream: Allow users to request only format driver names in backing file format

2023-12-05 Thread Peter Krempa
in 'hmp_block_stream' - changed version in docs to 9.0 as getting this into RC 3 probably isn't realistic Peter Krempa (2): block: commit: Allow users to request only format driver names in backing file format block: stream: Allow users to request only format driver names in backing file format

Re: [PATCH v2 1/2] block: commit: Allow users to request only format driver names in backing file format

2023-11-30 Thread Peter Krempa
On Thu, Nov 30, 2023 at 13:24:18 -0600, Eric Blake wrote: > On Thu, Nov 30, 2023 at 05:06:03PM +0100, Peter Krempa wrote: > > Introduce a new flag 'backing_file_format_no_protocol' for the > > block-commit QMP command which instructs the internals to use 'raw' > > instead o

[PATCH v2 1/2] block: commit: Allow users to request only format driver names in backing file format

2023-11-30 Thread Peter Krempa
with older libvirt versions which didn't expect that a protocol driver name can appear in the backing file format field. Signed-off-by: Peter Krempa Reviewed-by: Vladimir Sementsov-Ogievskiy --- block.c| 37 +- block/commit.c

[PATCH v2 2/2] block: stream: Allow users to request only format driver names in backing file format

2023-11-30 Thread Peter Krempa
with older libvirt versions which didn't expect that a protocol driver name can appear in the backing file format field. Signed-off-by: Peter Krempa Reviewed-by: Vladimir Sementsov-Ogievskiy --- block/monitor/block-hmp-cmds.c | 2 +- block/stream.c | 10

[PATCH v2 0/2] block: commit/stream: Allow users to request only format driver names in backing file format

2023-11-30 Thread Peter Krempa
Please see patches for rationale. Libvirt patches using this new flag will be posted soon-ish (after cleanup). v2: - fixed mistaken argument order in 'hmp_block_stream' - changed version in docs to 9.0 as getting this into RC 3 probably isn't realistic Peter Krempa (2): block: commit

Re: [PATCH 1/2] block: commit: Allow users to request only format driver names in backing file format

2023-11-28 Thread Peter Krempa
On Tue, Nov 28, 2023 at 20:10:10 +0300, Vladimir Sementsov-Ogievskiy wrote: > On 24.11.23 17:52, Peter Krempa wrote: > > Introduce a new flag 'backing_file_format_no_protocol' for the > > block-commit QMP command which instructs the internals to use 'raw' > > instead of the pr

[PATCH 1/2] block: commit: Allow users to request only format driver names in backing file format

2023-11-24 Thread Peter Krempa
with older libvirt versions which didn't expect that a protocol driver name can appear in the backing file format field. Signed-off-by: Peter Krempa --- block.c| 37 +- block/commit.c | 6 - blockdev.c

[PATCH 2/2] block: stream: Allow users to request only format driver names in backing file format

2023-11-24 Thread Peter Krempa
with older libvirt versions which didn't expect that a protocol driver name can appear in the backing file format field. Signed-off-by: Peter Krempa --- block/monitor/block-hmp-cmds.c | 2 +- block/stream.c | 10 +- blockdev.c | 7

[PATCH 0/2] block: commit/stream: Allow users to request only format driver names in backing file format

2023-11-24 Thread Peter Krempa
Please see patches for rationale. Libvirt patches using this new flag will be posted soon-ish (after cleanup). Peter Krempa (2): block: commit: Allow users to request only format driver names in backing file format block: stream: Allow users to request only format driver names

Re: [PATCH] block/blkio: Fix inclusion of required headers

2023-01-23 Thread Peter Krempa
On Mon, Jan 23, 2023 at 15:01:37 +0100, Markus Armbruster wrote: > Peter Krempa writes: > > > After recent header file inclusion rework the build fails when the blkio > > module is enabled: > > > > ../block/blkio.c: In function ‘blkio_detach_aio_context’: >

[PATCH] block/blkio: Fix inclusion of required headers

2023-01-23 Thread Peter Krempa
)); | ^~~~ | | | int Fix it by including 'block/block-io.h' which contains the required declarations. Fixes: e2c1c34f139f49ef909bb4322607fb8b39002312 Signed-off-by: Peter Krempa --- block/blkio.c | 2 ++ 1 file

Re: [RFC v3 1/8] blkio: add io_uring block driver using libblkio

2022-08-03 Thread Peter Krempa
On Wed, Jul 27, 2022 at 21:33:40 +0200, Kevin Wolf wrote: > Am 08.07.2022 um 06:17 hat Stefan Hajnoczi geschrieben: > > libblkio (https://gitlab.com/libblkio/libblkio/) is a library for > > high-performance disk I/O. It currently supports io_uring and > > virtio-blk-vhost-vdpa with additional

Re: [PATCH v2 00/15] qdev: Add JSON -device

2021-10-15 Thread Peter Krempa
RRP_GUARD() > - Replaced error handling patch for qdev_set_id() with Damien's > - Drop the deprecation patch until libvirt uses the new JSON syntax Series: Tested-by: Peter Krempa

Re: [PATCH 09/11] qdev: Avoid QemuOpts in QMP device_add

2021-10-01 Thread Peter Krempa
On Fri, Sep 24, 2021 at 11:04:25 +0200, Kevin Wolf wrote: > Directly call qdev_device_add_from_qdict() for QMP device_add instead of > first going through QemuOpts and converting back to QDict. > > Note that this changes the behaviour of device_add, though in ways that > should be considered bug

Re: [PATCH for-6.1? v2 5/7] job: Add job_cancel_requested()

2021-08-04 Thread Peter Krempa
On Wed, Aug 04, 2021 at 12:34:31 +0200, Kevin Wolf wrote: > We could in theory keep allowing redundant completion requests when the > completion mode doesn't conflict, but I don't see the point of that. I don't see either. Especially since ... > Unless libvirt can actually issue multiple

Arbitrary truncating of SCSI disk serial number

2021-06-03 Thread Peter Krempa
Hi, recently I've got a report that an upgrade of libvirt (and qemu) caused a guest-visible change in the SCSI disk identification when a very long serial number is used. I've traced it back to the point where libvirt started to use the 'device_id=' property of the SCSI disk to pass in the alias

Re: [PATCH] block: Drop the sheepdog block driver

2021-05-03 Thread Peter Krempa
sertions(+), 3593 deletions(-) > delete mode 100644 block/sheepdog.c Libvirt will need to adjust one test case (lock it to qemu-6.0 test data), but other than that, this change is okay with us. ACKed-by: Peter Krempa

Re: [PATCH] virtio-blk: drop deprecated scsi=on|off property

2021-04-30 Thread Peter Krempa
On Fri, Apr 30, 2021 at 09:42:05 +0200, Markus Armbruster wrote: > Eduardo Habkost writes: > > > On Thu, Apr 29, 2021 at 04:52:21PM +0100, Stefan Hajnoczi wrote: > >> The scsi=on|off property was deprecated in QEMU 5.0 and can be removed > >> completely at this point. > >> > >> Drop the

Re: [PATCH] virtio-blk: drop deprecated scsi=on|off property

2021-04-29 Thread Peter Krempa
code again to see whether we do the correct thing if it's no longer available. > > Cc: Markus Armbruster > Cc: Christoph Hellwig > Cc: Peter Krempa > Cc: Dr. David Alan Gilbert > Signed-off-by: Stefan Hajnoczi > --- > docs/specs/tpm.rst | 2 +- > docs/s

Re: [PATCH] qapi: deprecate drive-backup

2021-04-26 Thread Peter Krempa
y enabling O_DIRECT and linux_aio > unconditionally for drive_backup target. But actually this just shows > that using drive-backup in blockdev era is a bad idea. So let's motivate > everyone (including Virtuozzo of course) to move to new interfaces and > avoid problems with all that outdated option inheritance. libvirt never used 'drive-backup' thus Reviewed-by: Peter Krempa

Re: issuing [block-]job-complete to jobs in STANDBY state

2021-04-06 Thread Peter Krempa
On Thu, Apr 01, 2021 at 15:02:35 -0400, John Snow wrote: > Hi; downstream we've run into an issue where VMs under heavy load with many > simultaneously concurrent block jobs running might occasionally flicker into > the STANDBY state, during which time they will be unable to receive JOB > COMPLETE

Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add

2021-03-12 Thread Peter Krempa
On Fri, Mar 12, 2021 at 09:46:54 +0100, Paolo Bonzini wrote: > On 12/03/21 09:14, Markus Armbruster wrote: > > Paolo Bonzini writes: > > > > > On 11/03/21 15:08, Markus Armbruster wrote: > > > > > I would rather keep the OptsVisitor here. Do the same check for JSON > > > > > syntax that you

Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add

2021-03-11 Thread Peter Krempa
On Thu, Mar 11, 2021 at 12:41:42 +0100, Kevin Wolf wrote: > Am 11.03.2021 um 12:24 hat Peter Krempa geschrieben: > > On Thu, Mar 11, 2021 at 09:37:11 +0100, Kevin Wolf wrote: > > > Am 11.03.2021 um 08:47 hat Peter Krempa geschrieben: > > > > On Wed, Mar 10, 2021 at 18

Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add

2021-03-11 Thread Peter Krempa
On Thu, Mar 11, 2021 at 09:37:11 +0100, Kevin Wolf wrote: > Am 11.03.2021 um 08:47 hat Peter Krempa geschrieben: > > On Wed, Mar 10, 2021 at 18:30:44 +0100, Kevin Wolf wrote: > > > Am 10.03.2021 um 15:31 hat Paolo Bonzini geschrieben: > > > > On 10/0

Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add

2021-03-10 Thread Peter Krempa
On Wed, Mar 10, 2021 at 18:30:44 +0100, Kevin Wolf wrote: > Am 10.03.2021 um 15:31 hat Paolo Bonzini geschrieben: > > On 10/03/21 15:22, Peter Krempa wrote: [...] > The keyval parser would create a list if multiple values are given for > the same key. Some care needs to be taken

Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add

2021-03-10 Thread Peter Krempa
On Wed, Mar 10, 2021 at 15:31:57 +0100, Paolo Bonzini wrote: > On 10/03/21 15:22, Peter Krempa wrote: > > I've stumbled upon a regression with this patchset applied: > > > > error: internal error: process exited while connecting to monitor: > > qemu-system-x86_64: -obj

Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add

2021-03-10 Thread Peter Krempa
On Mon, Mar 08, 2021 at 17:54:10 +0100, Kevin Wolf wrote: > This series adds a QAPI type for the properties of all user creatable > QOM types and finally makes the --object command line option (in all > binaries) and the object-add monitor commands (in QMP and HMP) use the > new ObjectOptions

Re: [PATCH v2 1/3] fdc: Drop deprecated floppy configuration

2021-03-04 Thread Peter Krempa
On Thu, Mar 04, 2021 at 17:23:05 +0100, Markus Armbruster wrote: > Daniel P. Berrangé writes: > > > On Thu, Mar 04, 2021 at 03:26:55PM +0100, Markus Armbruster wrote: > >> Daniel P. Berrangé writes: > >> > >> > On Thu, Mar 04, 2021 at 11:00:57AM +0100, Markus Armbruster wrote: > >> >> Drop the

Re: [RFC PATCH v2 3/4] block: Support multiple reopening with x-blockdev-reopen

2021-03-01 Thread Peter Krempa
On Mon, Mar 01, 2021 at 12:07:26 +0100, Kevin Wolf wrote: > Am 25.02.2021 um 18:02 hat Vladimir Sementsov-Ogievskiy geschrieben: > > 24.02.2021 15:33, Kevin Wolf wrote: > > > Am 09.02.2021 um 09:03 hat Vladimir Sementsov-Ogievskiy geschrieben: > > > > 08.02.2021 21:44, Alberto Garcia wrote: > > >

Re: [PATCH v2 00/31] qapi/qom: QAPIfy --object and object-add

2021-02-24 Thread Peter Krempa
2021-February/msg01212.html Libvirt's test pass after it without any change, so on behalf of libvirt ACKed-by: Peter Krempa

Re: [PATCH v3 1/3] migration: dirty-bitmap: Convert alias map inner members to BitmapMigrationBitmapAlias

2021-02-12 Thread Peter Krempa
On Fri, Feb 12, 2021 at 12:38:10 -0600, Eric Blake wrote: > On 2/12/21 11:34 AM, Peter Krempa wrote: > > Long subject line; if it's okay with you, I'd prefer to use: > > migration: dirty-bitmap: Use struct for alias map inner members > > > Currently the alias mapping

[PATCH v3 3/3] qemu-iotests: 300: Add test case for modifying persistence of bitmap

2021-02-12 Thread Peter Krempa
Verify that the modification of the bitmap persistence over migration which is controlled via BitmapMigrationBitmapAliasTransform works properly. Based on TestCrossAliasMigration Signed-off-by: Peter Krempa --- tests/qemu-iotests/300 | 91 ++ tests/qemu

[PATCH v3 0/3] migration: dirty-bitmap: Allow control of bitmap persistence

2021-02-12 Thread Peter Krempa
See 2/2 for explanation. Peter Krempa (3): migration: dirty-bitmap: Convert alias map inner members to BitmapMigrationBitmapAlias migration: dirty-bitmap: Allow control of bitmap persistence qemu-iotests: 300: Add test case for modifying persistence of bitmap migration/block-dirty

[PATCH v3 1/3] migration: dirty-bitmap: Convert alias map inner members to BitmapMigrationBitmapAlias

2021-02-12 Thread Peter Krempa
Currently the alias mapping hash stores just strings of the target objects internally. In further patches we'll be adding another member which will need to be stored in the map so pass a copy of the whole BitmapMigrationBitmapAlias QAPI struct into the map. Signed-off-by: Peter Krempa

[PATCH v3 2/3] migration: dirty-bitmap: Allow control of bitmap persistence

2021-02-12 Thread Peter Krempa
into a squashed image) but currently it would need to create another set of persistent bitmaps and merge them. This patch adds a 'transform' property to the alias map which allows to override the persistence of migrated bitmaps both on the source and destination sides. Signed-off-by: Peter Krempa

Re: [RFC PATCH v2 0/4] Allow changing bs->file on reopen

2021-02-12 Thread Peter Krempa
On Wed, Feb 10, 2021 at 18:26:57 +0100, Kevin Wolf wrote: > Am 08.02.2021 um 19:44 hat Alberto Garcia geschrieben: > > Hi, > > > > this series allows changing bs->file using x-blockdev-reopen. Read > > here for more details: > > > >

[PATCH v2 2/2] migration: dirty-bitmap: Allow control of bitmap persistance

2021-02-10 Thread Peter Krempa
into a squashed image) but currently it would need to create another set of persistent bitmaps and merge them. This patch adds a 'transform' property to the alias map which allows to override the persistance of migrated bitmaps both on the source and destination sides. Signed-off-by: Peter Krempa --- v2

[PATCH v2 0/2] migration: dirty-bitmap: Allow control of bitmap persistence

2021-02-10 Thread Peter Krempa
See 2/2 for explanation. Peter Krempa (2): migration: dirty-bitmap: Convert alias map inner members to a struct migration: dirty-bitmap: Allow control of bitmap persistance migration/block-dirty-bitmap.c | 73 +- qapi/migration.json| 20

[PATCH v2 1/2] migration: dirty-bitmap: Convert alias map inner members to a struct

2021-02-10 Thread Peter Krempa
Currently the alias mapping hash stores just strings of the target objects internally. In further patches we'll be adding another member which will need to be stored in the map so convert the members to a struct. Signed-off-by: Peter Krempa Reviewed-by: Eric Blake Reviewed-by: Vladimir

Re: [PATCH 2/2] migration: dirty-bitmap: Allow control of bitmap persistence on destination

2021-02-03 Thread Peter Krempa
On Wed, Feb 03, 2021 at 14:27:49 +0100, Peter Krempa wrote: > On Wed, Feb 03, 2021 at 16:23:21 +0300, Vladimir Sementsov-Ogievskiy wrote: > > 03.02.2021 16:00, Peter Krempa wrote: > > > Bitmap's source persistence is transported over the migration stream and > > &

Re: [PATCH 2/2] migration: dirty-bitmap: Allow control of bitmap persistence on destination

2021-02-03 Thread Peter Krempa
On Wed, Feb 03, 2021 at 16:23:21 +0300, Vladimir Sementsov-Ogievskiy wrote: > 03.02.2021 16:00, Peter Krempa wrote: > > Bitmap's source persistence is transported over the migration stream and > > the destination mirrors it. In some cases the destination might want to > >

[PATCH 2/2] migration: dirty-bitmap: Allow control of bitmap persistence on destination

2021-02-03 Thread Peter Krempa
into a squashed image) but currently it would need to create another set of persistent bitmaps and merge them. This adds 'dest-persistent' optional property to 'BitmapMigrationBitmapAlias' which when present overrides the bitmap presence state from the source. Signed-off-by: Peter Krempa --- migration

[PATCH 1/2] migration: dirty-bitmap: Convert alias map inner members to a struct

2021-02-03 Thread Peter Krempa
Currently the alias mapping hash stores just strings of the target objects internally. In further patches we'll be adding another member which will need to be stored in the map so convert the members to a struct. Signed-off-by: Peter Krempa --- migration/block-dirty-bitmap.c | 37

[PATCH 0/2] migration: dirty-bitmap: Allow control of bitmap persistence on destination

2021-02-03 Thread Peter Krempa
See 2/2 for explanation. Please let me know if I need to add any tests for this. Peter Krempa (2): migration: dirty-bitmap: Convert alias map inner members to a struct migration: dirty-bitmap: Allow control of bitmap persistence on destination migration/block-dirty-bitmap.c | 67

Re: To start multiple KVM guests from one qcow2 image with transient disk option

2021-01-15 Thread Peter Krempa
On Thu, Jan 07, 2021 at 16:32:59 -0500, Masayoshi Mizuma wrote: > On Thu, Jan 07, 2021 at 09:05:42AM +0100, Peter Krempa wrote: > > On Tue, Jan 05, 2021 at 15:12:55 +0100, Peter Krempa wrote: > > > On Mon, Jan 04, 2021 at 15:30:19 -0500, Masayoshi Mizuma wrote: > > > &

Re: To start multiple KVM guests from one qcow2 image with transient disk option

2021-01-07 Thread Peter Krempa
On Tue, Jan 05, 2021 at 15:12:55 +0100, Peter Krempa wrote: > On Mon, Jan 04, 2021 at 15:30:19 -0500, Masayoshi Mizuma wrote: > > On Sat, Dec 19, 2020 at 11:30:39PM -0500, Masayoshi Mizuma wrote: [...] > {"execute":"cont"} > > So that is a no-go. Some

Re: To start multiple KVM guests from one qcow2 image with transient disk option

2021-01-05 Thread Peter Krempa
On Mon, Jan 04, 2021 at 15:30:19 -0500, Masayoshi Mizuma wrote: > On Sat, Dec 19, 2020 at 11:30:39PM -0500, Masayoshi Mizuma wrote: [...] > I think following qemu command line options and QMP commands work for sharing > the qcow2 disks. The following uses disk hotplug instead of snapshot

Re: RFC: don't store backing filename in qcow2 image

2020-12-11 Thread Peter Krempa
On Thu, Dec 10, 2020 at 17:26:52 +0300, Vladimir Sementsov-Ogievskiy wrote: > Hi all! Hi, > > I have an idea, that not storing backing filename in qcow2 image at all may > be a good thing. I'll give some reasons and want to know what do you think > about it. > > 1. Libvirt has to manage and

Re: [PATCH 00/18] qapi/qom: QAPIfy object-add

2020-11-30 Thread Peter Krempa
On Mon, Nov 30, 2020 at 13:25:20 +0100, Kevin Wolf wrote: > This series adds a QAPI type for the properties of all user creatable > QOM types and finally makes QMP object-add use the new ObjectOptions > union so that QAPI introspection can be used for user creatable objects. FYI, here's a libvirt

Re: [PATCH v6 04/11] nbd: Update qapi to support exporting multiple bitmaps

2020-10-27 Thread Peter Krempa
| 6 +- > nbd/server.c | 19 -- > qemu-nbd.c | 18 - > 5 files changed, 58 insertions(+), 29 deletions(-) Reviewed-by: Peter Krempa

Re: [PATCH v5 06/12] nbd: Update qapi to support exporting multiple bitmaps

2020-10-26 Thread Peter Krempa
On Fri, Oct 23, 2020 at 13:36:46 -0500, Eric Blake wrote: > Since 'nbd-server-add' is deprecated, and 'block-export-add' is new to > 5.2, we can still tweak the interface. Allowing 'bitmaps':['str'] is > nicer than 'bitmap':'str'. This wires up the qapi and qemu-nbd > changes to permit passing

Re: Attached disk blockpull

2020-10-13 Thread Peter Krempa
This is a libvirt question, so asking it on the qemu-block might not get you an answer that quick, ... or ever if I didn't notice your other question also addressed incorrectly. [adding libvirt-us...@redhat.com to cc] On Tue, Sep 01, 2020 at 10:57:34 -0400, Yoonho Park wrote: > I am trying to

Re: Overlay limit bug

2020-10-13 Thread Peter Krempa
w the patch set that addressed this bug? qemu patch: commit facda5443f5a8676fb635b82ac1046ac6b6a67ce Author: Peter Krempa Date: Mon Jan 20 09:50:49 2020 +0100 qapi: Allow getting flat output from 'query-named-block-nodes' When a management application manages node names there's no reason to recurse

Re: [PATCH v2 0/2] block: deprecate the sheepdog driver

2020-09-23 Thread Peter Krempa
On Tue, Sep 22, 2020 at 18:42:52 +0100, Daniel Berrange wrote: > On Tue, Sep 22, 2020 at 01:09:06PM -0400, Neal Gompa wrote: > > On Tue, Sep 22, 2020 at 12:16 PM Daniel P. Berrangé > > wrote: > > > > > > 2 years back I proposed dropping the sheepdog mailing list from the > > > MAINTAINERS file,

Re: [PATCH v3 5/6] qemu: Introduce to handle transient disk

2020-09-18 Thread Peter Krempa
On Thu, Sep 17, 2020 at 09:30:44 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Here is the implementation of transient option for qcow2 and raw format > disk. This gets available directive in domain xml file > like as: > > > > > > > > > When

Re: [PATCH v3 4/6] qemu: update the validation for transient disk option

2020-09-18 Thread Peter Krempa
On Thu, Sep 17, 2020 at 09:30:43 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Update validation of transient disk option. The option for qemu is supported > with under condistions. > > - qemu has blockdev feature > - the type is file and the format is qcow2 and raw > - writable

Re: [PATCH v3 3/6] qemu: Block migration when transient disk option is enabled

2020-09-18 Thread Peter Krempa
virDomainDiskDefPtr disk = vm->def->disks[i]; > + > +if (disk->transient) { > +virReportError(VIR_ERR_OPERATION_INVALID, "%s", > + _("migration with transient disk is not > supported")); transient disk '%s' is not supported and use disk->dst as '%s' > +return false; > +} > +} Reviewed-by: Peter Krempa

Re: [PATCH v3 1/6] qemu: Block blockjobs when transient disk option is enabled

2020-09-18 Thread Peter Krempa
On Fri, Sep 18, 2020 at 14:53:53 +0200, Peter Krempa wrote: > On Thu, Sep 17, 2020 at 09:30:40 -0400, Masayoshi Mizuma wrote: > > From: Masayoshi Mizuma > > > > Signed-off-by: Masayoshi Mizuma > > --- > > src/qemu/qemu_domain.c | 7 +++ > > 1 file

Re: [PATCH v3 2/6] qemu: Block disk hotplug when transient disk option is enabled

2020-09-18 Thread Peter Krempa
age could actually describe the reasons. Reviewed-by: Peter Krempa

Re: [PATCH v3 1/6] qemu: Block blockjobs when transient disk option is enabled

2020-09-18 Thread Peter Krempa
On Thu, Sep 17, 2020 at 09:30:40 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Signed-off-by: Masayoshi Mizuma > --- > src/qemu/qemu_domain.c | 7 +++ > 1 file changed, 7 insertions(+) Reviewed-by: Peter Krempa

Re: [PATCH v2 6/7] qemu: Block disk hotplug when transient disk option is enabled

2020-09-08 Thread Peter Krempa
On Fri, Aug 28, 2020 at 10:08:36 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Block disk hotplug when transient disk option is enabled so far. > > Signed-off-by: Masayoshi Mizuma > --- > src/qemu/qemu_hotplug.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git

Re: [PATCH v2 5/7] qemu: Block migration when transient disk option is enabled

2020-09-08 Thread Peter Krempa
On Fri, Aug 28, 2020 at 10:08:35 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Block migration when transient disk option is enabled because migration > requires some blockjobs. > > Signed-off-by: Masayoshi Mizuma > --- > src/qemu/qemu_migration.c | 22 ++ >

Re: [PATCH v2 7/7] qemu: Block blockjobs when transient disk option is enabled

2020-09-08 Thread Peter Krempa
On Fri, Aug 28, 2020 at 10:08:34 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Block blockjobs when transient disk option is enabled so far. > > Signed-off-by: Masayoshi Mizuma > --- > src/qemu/qemu_domain.c | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git

Re: [PATCH v2 4/7] qemu: Transient option gets avaiable for qcow2 and raw format disk

2020-09-08 Thread Peter Krempa
On Fri, Aug 28, 2020 at 10:08:33 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Signed-off-by: Masayoshi Mizuma > --- > src/qemu/qemu_validate.c | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c >

Re: [PATCH v2 1/7] qemuSnapshotDiskPrepareOne: Get available even if snapdisk is NULL

2020-09-08 Thread Peter Krempa
On Fri, Aug 28, 2020 at 10:08:30 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Get available even if snapdisk argument is NULL at > qemuSnapshotDiskPrepareOne() > so that the caller can setup dd->src. > > Signed-off-by: Masayoshi Mizuma > --- > src/qemu/qemu_snapshot.c | 5

Re: [PATCH v2 2/7] qemu: Introduce functions to handle transient disk

2020-09-08 Thread Peter Krempa
On Fri, Aug 28, 2020 at 10:08:31 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Here is the implementation of transient option for qcow2 and raw format > disk. This gets available directive in domain xml file > like as: > > > > > > > > > When

Re: qcow2 overlay performance

2020-08-26 Thread Peter Krempa
On Wed, Aug 26, 2020 at 15:30:03 +0200, Peter Krempa wrote: > On Wed, Aug 26, 2020 at 15:18:32 +0200, Kevin Wolf wrote: > > Am 26.08.2020 um 02:46 hat Yoonho Park geschrieben: > > > Another issue I hit is that I cannot set or change the cluster size of > > > overlays. I

Re: qcow2 overlay performance

2020-08-26 Thread Peter Krempa
On Wed, Aug 26, 2020 at 15:18:32 +0200, Kevin Wolf wrote: > Am 26.08.2020 um 02:46 hat Yoonho Park geschrieben: > > Another issue I hit is that I cannot set or change the cluster size of > > overlays. Is this possible with "virsh snapshot-create-as"? > > That's a libvirt question. Peter, can you

Re: [RFC PATCH 07/22] block/export: Remove magic from block-export-add

2020-08-20 Thread Peter Krempa
On Thu, Aug 20, 2020 at 09:41:14 -0500, Eric Blake wrote: > On 8/20/20 6:05 AM, Kevin Wolf wrote: > > > As long as we can keep the compatibility code local to qmp_nbd_*(), I > > don't think it's too bad. In particular because it's already written. > > > > Instead of adjusting libvirt to changes

Re: [PATCH 1/3] qemu: implementation of transient option for qcow2 file

2020-08-06 Thread Peter Krempa
On Sun, Jul 19, 2020 at 22:56:50 -0400, Masayoshi Mizuma wrote: > On Sat, Jul 18, 2020 at 08:06:00AM +0200, Peter Krempa wrote: > > On Thu, Jul 16, 2020 at 20:55:29 -0400, Masayoshi Mizuma wrote: > > > Thank you for your review. > > > > > > On Tue, Jul 07,

Re: [PATCH 1/2] qcow2: Release read-only bitmaps when inactivated

2020-07-30 Thread Peter Krempa
inactive_all(), but bdrv_invalidate_cache_all() attempts to reload > them (which fails, because they are still present in memory). > I've tested it with same commands as I've used before and now the 'cont' succeeds and also the bitmaps after the cont call are loaded and active at least according to 'query-named-block-nodes' Tested-by: Peter Krempa

Re: [PATCH 1/3] qemu: implementation of transient option for qcow2 file

2020-07-18 Thread Peter Krempa
On Thu, Jul 16, 2020 at 20:55:29 -0400, Masayoshi Mizuma wrote: > Thank you for your review. > > On Tue, Jul 07, 2020 at 06:36:23AM -0500, Eric Blake wrote: > > On 7/7/20 2:12 AM, Peter Krempa wrote: > > > > > You can install a qcow2 overlay on top of a raw file t

Re: [PATCH 1/3] qemu: implementation of transient option for qcow2 file

2020-07-07 Thread Peter Krempa
On Tue, Jul 07, 2020 at 06:36:23 -0500, Eric Blake wrote: > On 7/7/20 2:12 AM, Peter Krempa wrote: > > > > 1) the virDomainBlockCopy operation flattens the backing chain into the > > top level only. This means that must be stripped or the > > operation rejected, as oth

Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands

2020-07-07 Thread Peter Krempa
On Tue, Jul 07, 2020 at 12:33:31 +0200, Kevin Wolf wrote: > Am 07.07.2020 um 08:38 hat Peter Krempa geschrieben: > > On Mon, Jul 06, 2020 at 18:15:55 +0200, Kevin Wolf wrote: > > > Am 03.07.2020 um 18:02 hat Daniel P. Berrangé geschrieben: [...] > > IMO we really wan

Re: [PATCH 3/3] qemublocktest: add test of transient option for qcow2 disk

2020-07-07 Thread Peter Krempa
On Mon, Jul 06, 2020 at 14:20:25 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Add a unit test for transient option for qcow2 file. > > Signed-off-by: Masayoshi Mizuma > --- > tests/qemublocktest.c | 10 ++ >

Re: [PATCH 1/3] qemu: implementation of transient option for qcow2 file

2020-07-07 Thread Peter Krempa
On Mon, Jul 06, 2020 at 14:20:23 -0400, Masayoshi Mizuma wrote: > From: Masayoshi Mizuma > > Here is the implementation of transient option for qcow2 file. > This gets available directive in domain xml file > like as: > > > > > > > > > The internal

Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands

2020-07-07 Thread Peter Krempa
On Mon, Jul 06, 2020 at 18:15:55 +0200, Kevin Wolf wrote: > Am 03.07.2020 um 18:02 hat Daniel P. Berrangé geschrieben: > > On Fri, Jul 03, 2020 at 04:49:33PM +0100, Dr. David Alan Gilbert wrote: > > > * Daniel P. Berrangé (berra...@redhat.com) wrote: > > > > On Thu, Jul 02, 2020 at 01:12:52PM

Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands

2020-07-03 Thread Peter Krempa
On Fri, Jul 03, 2020 at 17:10:12 +0100, Dr. David Alan Gilbert wrote: > * Daniel P. Berrangé (berra...@redhat.com) wrote: > > On Fri, Jul 03, 2020 at 04:49:33PM +0100, Dr. David Alan Gilbert wrote: > > > * Daniel P. Berrangé (berra...@redhat.com) wrote: > > > > On Thu, Jul 02, 2020 at 01:12:52PM

Re: [PATCH RFC v2 0/5] block: add block-dirty-bitmap-populate job

2020-06-09 Thread Peter Krempa
On Mon, May 18, 2020 at 16:52:45 +0200, Peter Krempa wrote: > On Wed, May 13, 2020 at 23:49:17 -0400, John Snow wrote: > > Hi, > > > > This is a new (very small) block job that writes a pattern into a > > bitmap. The only pattern implemented is the top allocation inf

Re: [PATCH RFC v2 1/5] block: add bitmap-populate job

2020-06-08 Thread Peter Krempa
On Mon, Jun 08, 2020 at 13:30:48 +0300, Vladimir Sementsov-Ogievskiy wrote: > 08.06.2020 12:38, Peter Krempa wrote: > > On Sat, Jun 06, 2020 at 09:55:13 +0300, Vladimir Sementsov-Ogievskiy wrote: > > > 05.06.2020 13:59, Peter Krempa wrote: [...] > > > > It's

Re: [PATCH RFC v2 1/5] block: add bitmap-populate job

2020-06-08 Thread Peter Krempa
On Sat, Jun 06, 2020 at 09:55:13 +0300, Vladimir Sementsov-Ogievskiy wrote: > 05.06.2020 13:59, Peter Krempa wrote: > > On Fri, Jun 05, 2020 at 12:07:47 +0200, Kevin Wolf wrote: > > > Am 05.06.2020 um 11:58 hat Peter Krempa geschrieben: > > > > On Fri, Jun 05, 2020

Re: [PATCH RFC v2 1/5] block: add bitmap-populate job

2020-06-05 Thread Peter Krempa
On Fri, Jun 05, 2020 at 12:07:47 +0200, Kevin Wolf wrote: > Am 05.06.2020 um 11:58 hat Peter Krempa geschrieben: > > On Fri, Jun 05, 2020 at 11:44:07 +0200, Kevin Wolf wrote: [...] > > The above was actually inspired by a very recent problem I have in my > > attempt to

Re: [PATCH RFC v2 1/5] block: add bitmap-populate job

2020-06-05 Thread Peter Krempa
On Fri, Jun 05, 2020 at 11:44:07 +0200, Kevin Wolf wrote: > Am 05.06.2020 um 11:24 hat Peter Krempa geschrieben: > > On Fri, Jun 05, 2020 at 11:01:23 +0200, Kevin Wolf wrote: > > > Am 04.06.2020 um 18:22 hat Peter Krempa geschrieben: > > > > On Thu, Jun 04, 2020 at 13

Re: [PATCH RFC v2 1/5] block: add bitmap-populate job

2020-06-05 Thread Peter Krempa
On Fri, Jun 05, 2020 at 11:01:23 +0200, Kevin Wolf wrote: > Am 04.06.2020 um 18:22 hat Peter Krempa geschrieben: > > On Thu, Jun 04, 2020 at 13:31:45 +0200, Kevin Wolf wrote: > > > Am 04.06.2020 um 11:16 hat Peter Krempa geschrieben: > > > > On Thu, Jun 04, 2020 at 11

Re: [PATCH RFC v2 1/5] block: add bitmap-populate job

2020-06-04 Thread Peter Krempa
On Thu, Jun 04, 2020 at 13:31:45 +0200, Kevin Wolf wrote: > Am 04.06.2020 um 11:16 hat Peter Krempa geschrieben: > > On Thu, Jun 04, 2020 at 11:12:31 +0200, Kevin Wolf wrote: > > > Am 18.05.2020 um 22:49 hat Eric Blake geschrieben: > > > > > + > > &g

Re: [PATCH RFC v2 1/5] block: add bitmap-populate job

2020-06-04 Thread Peter Krempa
On Thu, Jun 04, 2020 at 11:12:31 +0200, Kevin Wolf wrote: > Am 18.05.2020 um 22:49 hat Eric Blake geschrieben: > > > + > > > +/* NB: new bitmap is anonymous and enabled */ > > > +cluster_size = bdrv_dirty_bitmap_granularity(target_bitmap); > > > +new_bitmap =

Re: [RFC v2] migration: Add migrate-set-bitmap-node-mapping

2020-06-02 Thread Peter Krempa
On Tue, Jun 02, 2020 at 12:56:32 +0200, Max Reitz wrote: > On 18.05.20 18:26, Peter Krempa wrote: > > On Wed, May 13, 2020 at 16:56:10 +0200, Max Reitz wrote: > >> This command allows mapping block node names to aliases for the purpose > >> of block dirty bitmap mi

Re: [PATCH RFC v2 1/5] block: add bitmap-populate job

2020-05-19 Thread Peter Krempa
On Mon, May 18, 2020 at 15:49:02 -0500, Eric Blake wrote: > On 5/13/20 10:49 PM, John Snow wrote: [...] > > + > > +/* NB: new bitmap is anonymous and enabled */ > > +cluster_size = bdrv_dirty_bitmap_granularity(target_bitmap); > > +new_bitmap = bdrv_create_dirty_bitmap(bs,

Re: [RFC v2] migration: Add migrate-set-bitmap-node-mapping

2020-05-18 Thread Peter Krempa
On Mon, May 18, 2020 at 20:52:32 +0300, Vladimir Sementsov-Ogievskiy wrote: > [add Nikolay] > > 18.05.2020 19:26, Peter Krempa wrote: > > On Wed, May 13, 2020 at 16:56:10 +0200, Max Reitz wrote: [...] > > Is there any difference of handling of persistent and non-p

Re: [RFC v2] migration: Add migrate-set-bitmap-node-mapping

2020-05-18 Thread Peter Krempa
On Wed, May 13, 2020 at 16:56:10 +0200, Max Reitz wrote: > This command allows mapping block node names to aliases for the purpose > of block dirty bitmap migration. > > This way, management tools can use different node names on the source > and destination and pass the mapping of how bitmaps are

Re: [PATCH RFC v2 0/5] block: add block-dirty-bitmap-populate job

2020-05-18 Thread Peter Krempa
snapshot outside of libvirt. I've dusted-off my patches for using this blockjob for this very specific case and it works for me. Tested-by: Peter Krempa For now I'll continue the integration with other blockjobs where we merge bitmaps.

Re: [PATCH v5 7/7] qemu-img: Deprecate use of -b without -F

2020-05-05 Thread Peter Krempa
On Tue, May 05, 2020 at 10:11:03 +0200, Kevin Wolf wrote: > Am 03.04.2020 um 19:58 hat Eric Blake geschrieben: > > Creating an image that requires format probing of the backing image is > > inherently unsafe (we've had several CVEs over the years based on > > probes leaking information to the

Re: Backup of vm disk images

2020-05-04 Thread Peter Krempa
On Fri, May 01, 2020 at 16:05:47 +0100, Stefan Hajnoczi wrote: > On Wed, Apr 22, 2020 at 07:51:09AM +0200, Anders Östling wrote: > > I am fighting to understand the difference between backing up a VM by > > using a regular copy vs using the virsh blockcopy command. > > What I want to do is to

  1   2   3   >