[PATCH Libvirt 0/4] Support for dirty-limit live migration

2023-10-11 Thread ~hyman
lowing reference: https://lore.kernel.org/qemu- devel/169024923116.19090.1082559906895003913...@git.sr.ht/ Please review, thanks, Yong Hyman Huang (4): Add VIR_MIGRATE_DIRTY_LIMIT flag qemu_migration: Implement VIR_MIGRATE_DIRTY_LIMIT flag virsh: Add support for VIR_MIGRATE_DIRTY_LIMI

[PATCH Libvirt 2/4] qemu_migration: Implement VIR_MIGRATE_DIRTY_LIMIT flag

2023-10-11 Thread ~hyman
From: Hyman Huang Signed-off-by: Hyman Huang --- src/qemu/qemu_migration.c| 8 src/qemu/qemu_migration.h| 1 + src/qemu/qemu_migration_params.c | 6 ++ src/qemu/qemu_migration_params.h | 1 + 4 files changed, 16 insertions(+) diff --git a/src/qemu/qemu_migration.c

[PATCH Libvirt 1/4] Add VIR_MIGRATE_DIRTY_LIMIT flag

2023-10-11 Thread ~hyman
From: Hyman Huang The flag can be used to enable dirty-limit capability for live migration. Signed-off-by: Hyman Huang --- include/libvirt/libvirt-domain.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h index

[PATCH Libvirt 3/4] virsh: Add support for VIR_MIGRATE_DIRTY_LIMIT flag

2023-10-11 Thread ~hyman
From: Hyman Huang Introudce option to enable dirty-limit convergence algorithim during live migration. Signed-off-by: Hyman Huang --- docs/manpages/virsh.rst | 10 +- src/libvirt-domain.c| 8 tools/virsh-domain.c| 10 ++ 3 files changed, 27 insertions(+), 1

[PATCH Libvirt 4/4] NEWS: document support for dirty-limit live migration

2023-10-11 Thread ~hyman
From: Hyman Huang Signed-off-by: Hyman Huang --- NEWS.rst | 8 1 file changed, 8 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index fbdd9674a3..dd0dda923e 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -25,6 +25,14 @@ v9.9.0 (unreleased) guest features to see if the current libvirt

[PATCH Libvirt v3 08/10] qemu_monitor: Implement qemuMonitorQueryVcpuDirtyLimit

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) Implement qemuMonitorQueryVcpuDirtyLimit which query vcpu dirty limit info by calling qmp 'query-vcpu-dirty-limit'. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_monitor.c | 12 +++ src/qemu/qemu_monitor.h | 17 ++ src/qemu/qemu_moni

[PATCH Libvirt v3 04/10] qemu_driver: Implement qemuDomainSetVcpuDirtyLimit

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) Implement qemuDomainSetVcpuDirtyLimit, which can be used to set or cancel the upper limit of the dirty page rate for virtual CPUs. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_driver.c | 131 +++ src/qemu/qemu_monitor.c

[PATCH Libvirt v3 09/10] qemu_driver: Extend dirtlimit statistics for domGetStats

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) Extend dirtylimit statistics for domGetStats to display the information of the upper limit of dirty page rate for virtual CPUs. Signed-off-by: Hyman Huang(黄勇) --- include/libvirt/libvirt-domain.h | 1 + src/libvirt-domain.c | 9 ++ src/qemu

[PATCH Libvirt v3 07/10] virsh: Introduce limit-dirty-page-rate api

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) Introduce limit-dirty-page-rate virsh api to set or cancel dirty page rate upper limit for virtual CPUs. Usage is below: $ virsh limit-dirty-page-rate --rate \ [--vcpu ] [--config] [--live] [--current] Set the dirty page rate upper limit for the given vcpu specified

[PATCH Libvirt v3 10/10] virsh: Introduce command 'virsh domstats --dirtylimit'

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) Introduce command 'virsh domstats --dirtylimit' for reporting dirty page rate upper limit (megabyte/s) infomation. The info is listed as follows: Domain: 'vm' dirtylimit.vcpu.0.limit=10 dirtylimit.vcpu.0.current=16 dirtylimit.vcpu.1.limit=10

[PATCH Libvirt v3 05/10] domain_validate: Export virDomainDefHasDirtyLimitStartupVcpus symbol

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) Export virDomainDefHasDirtyLimitStartupVcpus as a util function, which could be used in qemu_process.c file for the next commit. Signed-off-by: Hyman Huang(黄勇) --- src/conf/domain_validate.c | 2 +- src/conf/domain_validate.h | 2 ++ src/libvirt_private.syms | 1 + 3

[PATCH Libvirt v3 03/10] libvirt: Add virDomainSetVcpuDirtyLimit API

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) Introduce virDomainSetVcpuDirtyLimit API to set or cancel the dirty page rate upper limit. The API will throttle the virtual CPU as needed to keep their dirty page rate within the limit. Since it just throttles the virtual CPU, which dirties memory, read processes in the

[PATCH Libvirt v3 06/10] qemu_process: Setup dirty limit after launching VM

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) The dirty_limit attribute in XML requires setting up the upper limit of dirty page rate once after launching the VM, so add the implementation. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_process.c | 44 + 1 file changed, 44

[PATCH Libvirt v3 02/10] conf: Introduce XML for dirty limit configuration

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) The upper limit (megabyte/s) of the dirty page rate configured by the user can be tracked by the XML. To allow this, add the following XML: ... 3 ... The "dirty_limit" attribute in "vcpu" sub-element within "vcpus"

[PATCH Libvirt v3 01/10] qemu_capabilities: Introduce QEMU_CAPS_VCPU_DIRTY_LIMIT capability

2023-09-05 Thread ~hyman
From: Hyman Huang(黄勇) set-vcpu-dirty-limit/cancel-vcpu-dirty-limit/query-vcpu-dirty-limit were introduced since qemu >=7.1.0. Introduce corresponding capability. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilitie

[PATCH Libvirt v3 00/10] Support dirty page rate upper limit

2023-09-05 Thread ~hyman
iveness of large guests during live migration and can result in more stable read performance. The other application scenarios remain unexplored, before that, Libvirt could provide the basic API. Hyman Huang(黄勇) (10): qemu_capabilities: Introduce QEMU_CAPS_VCPU_DIRTY_LIMIT capability conf:

[PATCH Libvirt v2 09/10] qemu_driver: Extend dirtlimit statistics for domGetStats

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) Extend dirtylimit statistics for domGetStats to display the information of the upper limit of dirty page rate for virtual CPUs. Signed-off-by: Hyman Huang(黄勇) --- include/libvirt/libvirt-domain.h | 1 + src/libvirt-domain.c | 9 ++ src/qemu

[PATCH Libvirt v2 06/10] qemu_process: Setup dirty limit after launching VM

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) The dirty_limit attribute in XML requires setting up the upper limit of dirty page rate once after launching the VM, so add the implementation. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_process.c | 44 + 1 file changed, 44

[PATCH Libvirt v2 10/10] virsh: Introduce command 'virsh domstats --dirtylimit'

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) Introduce command 'virsh domstats --dirtylimit' for reporting dirty page rate upper limit infomation. The info is listed as follows: Domain: 'vm' dirtylimit.vcpu.0.limit=10 dirtylimit.vcpu.0.current=16 dirtylimit.vcpu.1.limit=10 dirtyli

[PATCH Libvirt v2 08/10] qemu_monitor: Implement qemuMonitorQueryVcpuDirtyLimit

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) Implement qemuMonitorQueryVcpuDirtyLimit which query vcpu dirty limit info by calling qmp 'query-vcpu-dirty-limit'. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_monitor.c | 12 +++ src/qemu/qemu_monitor.h | 17 ++ src/qemu/qemu_moni

[PATCH Libvirt v2 07/10] virsh: Introduce limit-dirty-page-rate api

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) Introduce limit-dirty-page-rate virsh api to set or cancel dirty page rate upper limit for virtual CPUs. Usage is below: $ virsh limit-dirty-page-rate --rate \ [--vcpu ] Set the dirty page rate upper limit for the given vcpu specified by the "vcpu"; s

[PATCH Libvirt v2 04/10] qemu_driver: Implement qemuDomainSetVcpuDirtyLimit

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) Implement qemuDomainSetVcpuDirtyLimit, which can be used to set or cancel the upper limit of the dirty page rate for virtual CPUs. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_driver.c | 131 +++ src/qemu/qemu_monitor.c

[PATCH Libvirt v2 05/10] domain_validate: Export virDomainDefHasDirtyLimitStartupVcpus symbol

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) Export virDomainDefHasDirtyLimitStartupVcpus as a util function, which could be used in qemu_process.c file for the next commit. Signed-off-by: Hyman Huang(黄勇) --- src/conf/domain_validate.c | 2 +- src/conf/domain_validate.h | 2 ++ src/libvirt_private.syms | 1 + 3

[PATCH Libvirt v2 01/10] qemu_capabilities: Introduce QEMU_CAPS_VCPU_DIRTY_LIMIT capability

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) set-vcpu-dirty-limit/cancel-vcpu-dirty-limit/query-vcpu-dirty-limit were introduced since qemu >=7.1.0. Introduce corresponding capability. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilitie

[PATCH Libvirt v2 02/10] conf: Introduce XML for dirty limit configuration

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) The upper limit (MB/s) of the dirty page rate configured by the user can be tracked by the XML. To allow this, add the following XML: ... 3 ... The "dirty_limit" attribute in "vcpu" sub-element within "vcpus" element al

[PATCH Libvirt v2 00/10] Support dirty page rate upper limit

2023-08-07 Thread ~hyman
e stable read performance. The other application scenarios remain unexplored, before that, Libvirt could provide the basic API. Please review, thanks Yong Hyman Huang(黄勇) (10): qemu_capabilities: Introduce QEMU_CAPS_VCPU_DIRTY_LIMIT capability conf: Introduce XML for dirty limi

[PATCH Libvirt v2 03/10] libvirt: Add virDomainSetVcpuDirtyLimit API

2023-08-07 Thread ~hyman
From: Hyman Huang(黄勇) Introduce virDomainSetVcpuDirtyLimit API to set or cancel the dirty page rate upper limit. The API will throttle the virtual CPU as needed to keep their dirty page rate within the limit set by @rate. Since it just throttles the virtual CPU, which dirties memory, read

[PATCH Libvirt 01/11] qemu_capabilities: Introduce QEMU_CAPS_VCPU_DIRTY_LIMIT capability

2023-07-30 Thread ~hyman
From: Hyman Huang(黄勇) set-vcpu-dirty-limit/cancel-vcpu-dirty-limit/query-vcpu-dirty-limit were introduced since qemu >=7.1.0. Introduce corresponding capability. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_capabilities.c | 2 ++ src/qemu/qemu_capabilitie

[PATCH Libvirt 08/11] libvirt: Add virDomainCancelVcpuDirtyLimit API

2023-07-30 Thread ~hyman
From: Hyman Huang(黄勇) Introduce the virDomainCancelVcpuDirtyLimit API to cancel the upper limit of dirty page rate. Signed-off-by: Hyman Huang(黄勇) --- include/libvirt/libvirt-domain.h | 4 +++ src/driver-hypervisor.h | 6 src/libvirt-domain.c | 52

[PATCH Libvirt 06/11] qemu_driver: Extend qemuDomainGetVcpus

2023-07-30 Thread ~hyman
From: Hyman Huang(黄勇) Extend qemuDomainGetVcpus for getting dirty page rate upper limit info so 'virsh vcpuinfo' api can display it. Signed-off-by: Hyman Huang(黄勇) --- include/libvirt/libvirt-domain.h | 2 ++ src/qemu/qemu_driver.c | 55 +++

[PATCH Libvirt 05/11] qemu_monitor: Implement qemuMonitorQueryVcpuDirtyLimit

2023-07-30 Thread ~hyman
From: Hyman Huang(黄勇) Implement qemuMonitorQueryVcpuDirtyLimit which query vcpu dirty limit info by calling qmp 'query-vcpu-dirty-limit'. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_monitor.c | 12 +++ src/qemu/qemu_monitor.h | 17 ++ src/qemu/qemu_moni

[PATCH Libvirt 02/11] libvirt: Add virDomainSetVcpuDirtyLimit API

2023-07-30 Thread ~hyman
From: Hyman Huang(黄勇) Introduce virDomainSetVcpuDirtyLimit API to set upper limit of dirty page rate. Signed-off-by: Hyman Huang(黄勇) --- include/libvirt/libvirt-domain.h | 16 ++ src/driver-hypervisor.h | 7 + src/libvirt-domain.c | 54

[PATCH Libvirt v2 2/3] qemu_command: Generate cmd line for discard and write-zeroes properties

2023-07-17 Thread ~hyman
From: Hyman Huang(黄勇) Generate cmd line for virtio-blk.discard and virtio-blk.write-zeroes properties. Also, validate that the requested feature is supported by QEMU. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_command.c | 2 + src/qemu/qemu_validate.c

[PATCH Libvirt v2 0/3] support discard and write-zeroes options for virtio-blk device

2023-07-17 Thread ~hyman
ce capabilities of discard and write-zeroes for virtio-blk 2. add virtio_discard and virtio_write_zeroes attributes of driver in disk xml element 3. generate cmd line when launching vm Hyman Huang(黄勇) (3): conf: Add 'virtio_discard' and 'virtio_write_zeroes' attributes qemu_comm

[PATCH Libvirt v2 1/3] conf: Add 'virtio_discard' and 'virtio_write_zeroes' attributes

2023-07-17 Thread ~hyman
From: Hyman Huang(黄勇) Add 'virtio_discard' and 'virtio_write_zeroes' attribute to control whether discard and write-zeroes requests are handled by the virtio-blk device. To distinguish the attributes in block drive layer, use the 'virtio' prefix to indicate that

[PATCH Libvirt v2 3/3] NEWS: Mention support for discard and write_zeroes of virtio-blk device

2023-07-17 Thread ~hyman
From: Hyman Huang(黄勇) Signed-off-by: Hyman Huang(黄勇) --- NEWS.rst | 8 1 file changed, 8 insertions(+) diff --git a/NEWS.rst b/NEWS.rst index 42c2c53091..b6fde7c353 100644 --- a/NEWS.rst +++ b/NEWS.rst @@ -17,6 +17,14 @@ v9.6.0 (unreleased) * **New features** + * qemu

[PATCH Libvirt 3/3] qemu_command: Generate cmd line for discard and write-zeroes properties

2023-07-17 Thread ~hyman
From: Hyman Huang(黄勇) Generate cmd line for virtio-blk.discard and virtio-blk.write-zeroes properties. Also, validate that the requested feature is supported by QEMU. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_command.c | 2 + src/qemu/qemu_validate.c

[PATCH Libvirt 0/3] support discard and write-zeroes options for virtio-blk device

2023-07-17 Thread ~hyman
roes attributes of driver in disk xml element 3. generate cmd line when launching vm Please review, any comments and suggestions are very appreciated, thanks! Yong Hyman Huang(黄勇) (3): qemu_capabilities: Introduce virtio-blk DISCARD and WRITE_ZEROES capabilities conf: Add &#

[PATCH Libvirt 1/3] qemu_capabilities: Introduce virtio-blk DISCARD and WRITE_ZEROES capabilities

2023-07-17 Thread ~hyman
From: Hyman Huang(黄勇) DISCARD and WRITE_ZEROES commands has been implemented in virtio-blk protocol since qemu >= 4.2.0. Introduce QEMU_CAPS_VIRTIO_BLK_DISCARD and QEMU_CAPS_VIRTIO_BLK_WRITE_ZEROES capability definitions. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_capabilitie

[PATCH Libvirt 2/3] conf: Add 'virtio_discard' and 'virtio_write_zeroes' attributes

2023-07-17 Thread ~hyman
From: Hyman Huang(黄勇) Add 'virtio_discard' and 'virtio_write_zeroes' attribute to control whether discard and write-zeroes requests are handled by the virtio-blk device. To distinguish the attributes in block drive layer, use the 'virtio' prefix to indicate that

Re: [PATCH] qemu: Don't report error from domblkinfo if vm is inactive

2022-09-06 Thread Hyman
在 2022/9/6 15:47, Peter Krempa 写道: On Tue, Sep 06, 2022 at 10:29:01 +0800, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Libvirt logs and reports error when executing domblkinfo if vm configured rbd storage and in inactive state. The steps to reproduce the problem: 1. define and

Re: [PATCH v6 0/8] support mode option for dirtyrate calculation

2022-02-21 Thread Hyman Huang
在 2022/2/21 20:35, Michal Prívozník 写道: On 2/20/22 14:28, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) v6: - rebase on mastr - drop the commit [PATCH v5 1/9] in this patchset, post an extra commit if needed in the future. Please review, thanks ! Regards Yong v5: - [PATCH v5

Re: [PATCH v6 7/8] virsh: Add mode option to domdirtyrate-calc virsh api

2022-02-21 Thread Hyman Huang
在 2022/2/21 20:34, Michal Prívozník 写道: On 2/20/22 14:28, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Extend domdirtyrate-calc virsh api with mode option, either of these three options "page-sampling,dirty-bitmap,dirty-ring" can be specified when calculating dirty

Re: [PATCH v6 2/8] qemu_driver: Probe capability before calculating dirty page rate

2022-02-21 Thread Hyman Huang
在 2022/2/21 20:34, Michal Prívozník 写道: On 2/20/22 14:28, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Probing QEMU_CAPS_CALC_DIRTY_RATE capability in advance in case of failure when calculating dirty page rate. Signed-off-by: Hyman Huang(黄勇) --- src/qemu/qemu_driver.c | 8

Re: [PATCH v5 1/9] qemu: Refactor dirty page rate calculation status implementation

2022-02-17 Thread Hyman Huang
在 2022/2/18 0:19, Michal Prívozník 写道: On 2/16/22 01:28, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) For any virTypedParameter API normal practice is to use a string to expose the data, not the rather enum integer value. So let's drop the virDomainDirtyRateStatus in p

Re: [PATCH v3 4/7] include: Introduce enum for qemuDomainStartDirtyRateCalc

2022-02-14 Thread Hyman Huang
在 2022/2/14 19:31, Daniel P. Berrangé 写道: On Mon, Feb 14, 2022 at 07:21:58PM +0800, Hyman Huang wrote: 在 2022/2/14 18:13, Michal Prívozník 写道: On 1/28/22 08:35, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Introduce virDomainDirtyRateCalcFlags and virDomainDirtyRateCalcMode to

Re: [PATCH v3 4/7] include: Introduce enum for qemuDomainStartDirtyRateCalc

2022-02-14 Thread Hyman Huang
在 2022/2/14 18:13, Michal Prívozník 写道: On 1/28/22 08:35, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Introduce virDomainDirtyRateCalcFlags and virDomainDirtyRateCalcMode to get ready for adding mode parameter to qemuDomainStartDirtyRateCalc. Signed-off-by: Hyman Huang(黄勇

Re: [PATCH v3 6/7] virsh: Add mode option to domdirtyrate-calc virsh api

2022-02-14 Thread Hyman Huang
在 2022/2/14 18:13, Michal Prívozník 写道: On 1/28/22 08:35, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Extend domdirtyrate-calc virsh api with mode option, either of these three options "page-sampling,dirty-bitmap,dirty-ring" can be specified when calculating dirty

Re: [PATCH v3 0/7] support mode option for dirtyrate calculation

2022-02-10 Thread Hyman
Ping for this patchset. Thanks, Yong 在 2022/1/28 15:35, huang...@chinatelecom.cn 写道: From: Hyman Huang(黄勇) v3: - Rebase the master - [PATCH v2 2/6]: Fix the usage of virQEMUCapsGet - [PATCH v2 4/6]: Fix the cleanup missed in qemuDomainStartDirtyRateCalc - [PATCH v2 4/6]: Move all blocks

Re: [PATCH v2 4/6] qemu_driver: Add mode option for qemuDomainStartDirtyRateCalc

2022-01-27 Thread Hyman Huang
在 2022/1/27 16:34, Peter Krempa 写道: On Thu, Jan 27, 2022 at 15:25:20 +0800, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Add mode option to extend qemuDomainStartDirtyRateCalc API, which is introduced since qemu >= 6.2. Signed-off-by: Hyman Huang(黄勇) --- include/libv

Re: [PATCH v2 2/6] qemu_driver: Probe capability before calculating dirty page rate

2022-01-27 Thread Hyman Huang
在 2022/1/27 16:23, Peter Krempa 写道: On Thu, Jan 27, 2022 at 15:25:18 +0800, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Probing QEMU_CAPS_CALC_DIRTY_RATE capability in advance in case of failure when calculating dirty page rate. Signed-off-by: Hyman Huang(黄勇) --- src/qemu

Re: [PATCH] Fix missing-field-initializers error

2022-01-12 Thread Hyman
在 2022/1/13 1:31, Daniel P. Berrangé 写道: On Thu, Jan 13, 2022 at 12:37:45AM +0800, Hyman wrote: 在 2022/1/13 0:22, Daniel P. Berrangé 写道: On Thu, Jan 13, 2022 at 12:11:47AM +0800, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) When compile libvirt via ninja tool with default

Re: [PATCH] Fix missing-field-initializers error

2022-01-12 Thread Hyman
在 2022/1/13 0:22, Daniel P. Berrangé 写道: On Thu, Jan 13, 2022 at 12:11:47AM +0800, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) When compile libvirt via ninja tool with default configuration, it report the error message as the following: "missing initializer for field 

Re: [PATCH v7 1/2] qemu: support dirty ring feature

2021-12-14 Thread Hyman Huang
On 12/14/21 20:21, Michal Prívozník wrote: On 12/14/21 10:22, Michal Prívozník wrote: On 11/23/21 15:36, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Dirty ring feature was introduced in qemu-6.1.0, this patch add the corresponding feature named 'dirty-ring', which en

Re: [PATCH v7 1/2] qemu: support dirty ring feature

2021-12-14 Thread Hyman Huang
On 12/14/21 17:22, Michal Prívozník wrote: On 11/23/21 15:36, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) Dirty ring feature was introduced in qemu-6.1.0, this patch add the corresponding feature named 'dirty-ring', which enable dirty ring feature when starting vm. To

Re: [PATCH v7 0/2] Dirty Ring support (Libvirt)

2021-11-29 Thread Hyman
Ping Any corrections and suggetions are welcome. Please review, thanks! 在 2021/11/23 22:36, huang...@chinatelecom.cn 写道: From: "Hyman Huang(黄勇)" v7 - rebase on master - modify the following points according to the advice given by Peter 1. skip the -accel switch and reuse th

Re: [PATCH v6 0/4] Dirty Ring support (Libvirt)

2021-11-22 Thread Hyman
在 2021/11/22 22:48, Peter Krempa 写道: On Sat, Nov 20, 2021 at 03:20:44 -0500, huang...@chinatelecom.cn wrote: From: "Hyman Huang(黄勇)" Ping for this series. I still keep thinking the dirty ring feature is something good to have for libvirt. qemu-6.1 has supported dirty ring f

Re: [PATCH v6 4/4] qemu: support dirty ring feature

2021-11-22 Thread Hyman
在 2021/11/22 16:55, Peter Krempa 写道: On Sat, Nov 20, 2021 at 03:20:48 -0500, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) dirty ring feature was introduced in qemu-6.1, this patch add corresponding feature named 'dirty-ring', which enable dirty ring feature when starti

Re: [PATCH v6 3/4] conf: introduce dirty_ring_size field

2021-11-22 Thread Hyman
在 2021/11/22 16:54, Peter Krempa 写道: On Sat, Nov 20, 2021 at 03:20:47 -0500, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) introduce dirty_ring_size in struct "_virDomainDef" to hold the ring size configured by user, and pass dirty_ring_size when building qemu commandlin

Re: [PATCH v6 3/4] conf: introduce dirty_ring_size field

2021-11-22 Thread Hyman
在 2021/11/22 16:50, Peter Krempa 写道: On Sat, Nov 20, 2021 at 03:20:47 -0500, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) introduce dirty_ring_size in struct "_virDomainDef" to hold the ring size configured by user, and pass dirty_ring_size when building qemu commandlin

Re: [PATCH v6 2/4] qemu_command: switch accelerator option to new style

2021-11-22 Thread Hyman Huang
在 2021/11/22 16:43, Peter Krempa 写道: On Sat, Nov 20, 2021 at 03:20:46 -0500, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) QEMU greater than 2.9.0 support '-accel' option, change the way of assembling commandline from "accel=kvm" to "-accel kvm" when s

Re: [PATCH v3 0/4] Dirty Ring support (Libvirt)

2021-08-29 Thread Hyman
ping - hi, what would you think about this patchset ? :) 在 2021/6/25 0:51, huang...@chinatelecom.cn 写道: From: Hyman Huang(黄勇) v3 - rebase master and fix the confilict when apply "conf: introduce dirty_ring_size in struct "_virDomainDef" to current master. v2 - split

Re: [PATCH 2/2] qemu: support dirty ring feature

2021-06-21 Thread Hyman
在 2021/6/21 15:29, Peter Krempa 写道: On Sat, Jun 19, 2021 at 17:53:04 +0800, huang...@chinatelecom.cn wrote: From: Hyman Huang(黄勇) QEMU has introduced a dirty ring feature, this patch add corresponding feature named 'dirty-ring', which enable dirty ring feature when starting vm.

Re: [PATCH 0/2] specify the accelerator type using "-accel" parameter

2021-01-25 Thread Hyman
r can be specified via Libvirt,such as kernel-irqchip,kvm-shadow-mem,tb-size,etc。 I think this series can make sense, and i would like to know whether there is something wrong about this series, or other concern about this series。I would be very thankful if letting me know. Thanks! Hyman 在 2021

Re: [PATCH 0/2] specify the accelerator type using "-accel" parameter

2021-01-20 Thread Hyman
ing this work, rather than hard-writing code or using QEMU command line. Thanks! 鍦� 2021/1/13 23:06, huang...@chinatelecom.cn 鍐欓亾: From: Hyman This patchset aims to support dirty ring feature which has been introduced in kernel: https://lore.kernel.org/lkml/8b3f68dd-c61c-16a0-2077-0

Re: [RFC v4 1/4] tests: Add 'end-of-file' character to args file in qemuxml2argvdata

2021-01-14 Thread Hyman
ely, me, Daniel Berrange, and Jan Tomko. Thank you for remainding me. Now, I've seen the Authors.rst.in file and do not try it anymore : ) From: Hyman The test file disk-backing-chains-noindex.x86_64-latest.args has 'No newline at end of file', this patch rewrite it in case unexpe

Re: [PATCH v3 3/3] qemu: suppor dirty ring feature

2021-01-13 Thread Hyman
在 2021/1/13 21:32, Han Han 写道: On Wed, Jan 13, 2021 at 1:09 AM <mailto:huang...@chinatelecom.cn>> wrote: From: Hyman mailto:huang...@chinatelecom.cn>> QEMU introduced a dirty ring feature, this patch add a new KVM feature 'dirty-ring' to set this featur

Re: [PATCH RFC 1/2] qemu: use "-accel" option to specify accelerator instead of "-machine"

2021-01-12 Thread Hyman
ow -accel combined with -machine $ /usr/bin/qemu-kvm -accel tc qemu-system-x86_64: The -accel and "-machine accel=" options are incompatible That script is useless, a symlink will do ever since QEMU 4.0. Fedora can and should get rid of it, I'll take a look. Is there any other distro that does the same?  Libvirt cannot use newer KVM features with "-machine accel". Paolo Best Regards ! Hyman