Re: [libvirt] xsave-related CPU features implementation

2016-05-20 Thread Alexander Burluka
On Fri, May 20, 2016 at 09:29:38AM +0200, Jiri Denemark wrote: > On Thu, May 19, 2016 at 19:56:18 +0300, Alexander Burluka wrote: > > Hello, > > > > I would like to add support for the next CPU features which presents in > > QEMU - xsaveopt, xsavec, xgetbv1, xsaves. The

[libvirt] xsave-related CPU features implementation

2016-05-19 Thread Alexander Burluka
actual? This feature set needs ecx 0x1. I can see that it was a temporary solution in commit by Peter Krempa 012f9b19ef3812884e207dc431571502de4cebce. Does these features addtion require CPU detection improvement? Or can I simply put them into src/cpu/cpu_map.xml? Thank you -- Regards, Alexander

[libvirt] [PATCH] cpu: Add support for clflushopt and tsc_adjust Intel features

2016-05-14 Thread Alexander Burluka
Corresponding QEMU commits: clflushopt f7fda280948a5e74aeb076ef346b991ecb173c56 tsc_adjust 7b458bfd12a71b3da6b531daedc417492c9334e0 Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/cpu/cpu_map.xml | 6 ++ 1 file changed, 6 insertions(+) diff --git a/s

Re: [libvirt] [PATCH v7 0/6] Global domain cpu.cfs_period_us and cpu.cfs_quota_us setup

2016-03-01 Thread Alexander Burluka
Good news, thanks a lot! On 03/01/2016 05:41 PM, Daniel P. Berrange wrote: On Sat, Feb 27, 2016 at 01:47:38PM +0300, Alexander Burluka wrote: On 02/26/2016 08:34 PM, Daniel P. Berrange wrote: On Tue, Feb 16, 2016 at 04:43:32PM +0300, Alexander Burluka wrote: This patchset implements

Re: [libvirt] [PATCH v7 0/6] Global domain cpu.cfs_period_us and cpu.cfs_quota_us setup

2016-02-27 Thread Alexander Burluka
On 02/26/2016 08:34 PM, Daniel P. Berrange wrote: On Tue, Feb 16, 2016 at 04:43:32PM +0300, Alexander Burluka wrote: This patchset implements an ability to specify values for domain top level cpu.cfs_period_us and cpu.cfs_quota_us cgroups. These parameters are opt-in and named "global_p

Re: [libvirt] devAlias correct parsing

2016-02-17 Thread Alexander Burluka
Oh, I see, thanks for fast help. And what is recommended way of identifying device? For example, if application need to change ISO on that event in domain cdrom? On 02/17/2016 04:26 PM, Daniel P. Berrange wrote: On Wed, Feb 17, 2016 at 04:20:58PM +0300, Alexander Burluka wrote: Hello, I

[libvirt] devAlias correct parsing

2016-02-17 Thread Alexander Burluka
delimiter '-'. However, I've found private virDomainDefFindDevice function within libvirt and seems that its make correct parsing. How can I use it in my code? Thank you! -- Regards, Alexander Burluka -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH v7 0/6] Global domain cpu.cfs_period_us and cpu.cfs_quota_us setup

2016-02-16 Thread Alexander Burluka
ontrolling CPU usage. Changes in v2: add XML validation test Changes in v3: remove unneccessary cgroup copying Changes in v4: fix little rebase error Changes in v5: rebase to version 1.3.1 Changes in v6: remove unnecessary check Changes in v7: rebase to current master Alexander Burluka (6):

[libvirt] [PATCH v7 4/6] Add global_period and global_quota XML validation test

2016-02-16 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- tests/qemuxml2argvdata/qemuxml2argv-cputune.xml | 2 ++ tests/qemuxml2xmloutdata/qemuxml2xmlout-cputune.xml | 2 ++ 2 files changed, 4 insertions(+) diff --git a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml b

[libvirt] [PATCH v7 6/6] Implement handling of per-domain bandwidth settings

2016-02-16 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_driver.c | 97 -- 1 file changed, 95 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 2bbc724..d211a32 100644 --- a/sr

[libvirt] [PATCH v7 1/6] Add global period definitions

2016-02-16 Thread Alexander Burluka
This parameter represents top level period cgroup that limits whole domain enforcement period for a quota Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_

[libvirt] [PATCH v7 5/6] Implement qemuSetupGlobalCpuCgroup

2016-02-16 Thread Alexander Burluka
This functions setups per-domain cpu bandwidth parameters Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 49 + src/qemu/qemu_cgroup.h | 1 + src/qemu/qemu_process.c | 4 3 files chang

[libvirt] [PATCH v7 3/6] Add error checking on global quota and period

2016-02-16 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_command.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 7a8ae73..d2217b0 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_com

[libvirt] [PATCH v7 2/6] Add global quota parameter necessary definitions

2016-02-16 Thread Alexander Burluka
This parameter controls the maximum bandwidth to be used within a period for whole domain. Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_conf.c

Re: [libvirt] [PATCH v6 0/7] Global domain cpu.cfs_period_us and cpu.cfs_quota_us setup

2016-01-28 Thread Alexander Burluka
Ping On 01/27/2016 07:17 PM, Alexander Burluka wrote: This patchset implements an ability to specify values for domain top level cpu.cfs_period_us and cpu.cfs_quota_us cgroups. These parameters are opt-in and named "global_period" and "global_quota". Introduction of

[libvirt] [PATCH v6 6/7] Implement qemuSetupGlobalCpuCgroup

2016-01-27 Thread Alexander Burluka
This functions setups per-domain cpu bandwidth parameters Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 54 + src/qemu/qemu_cgroup.h | 1 + src/qemu/qemu_process.c | 4 3 files chang

[libvirt] [PATCH v6 4/7] Add global_period and global_quota XML validation test

2016-01-27 Thread Alexander Burluka
--- tests/qemuxml2argvdata/qemuxml2argv-cputune.xml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml b/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml index 813d201..a056089 100644 --- a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml +++

[libvirt] [PATCH v6 3/7] Add error checking on global quota and period

2016-01-27 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_command.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 5d3ab3a..087e9ad 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_com

[libvirt] [PATCH v6 2/7] Add global quota parameter necessary definitions

2016-01-27 Thread Alexander Burluka
This parameter controls the maximum bandwidth to be used within a period for whole domain. Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_conf.c

[libvirt] [PATCH v6 0/7] Global domain cpu.cfs_period_us and cpu.cfs_quota_us setup

2016-01-27 Thread Alexander Burluka
ontrolling CPU usage. Changes in v2: add XML validation test Changes in v3: remove unneccessary cgroup copying Changes in v4: fix little rebase error Changes in v5: rebase to version 1.3.1 Changes in v6: remove unnecessary check Alexander Burluka (7): Add global period definitions Add gl

[libvirt] [PATCH v6 5/7] Rename qemuSetupCgroupVcpuBW to qemuSetupBandwidthCgroup

2016-01-27 Thread Alexander Burluka
This rename is required to reuse this function in per-domain bandwidth setup routine Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 14 +++--- src/qemu/qemu_cgroup.h | 6 +++--- src/qemu/qemu_driver.c | 5 ++--- 3 files changed, 12 inse

[libvirt] [PATCH v6 7/7] Implement handling of per-domain bandwidth settings

2016-01-27 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_driver.c | 97 -- 1 file changed, 95 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 48aeab6..1932d7f 100644 --- a/sr

[libvirt] [PATCH v6 1/7] Add global period definitions

2016-01-27 Thread Alexander Burluka
This parameter represents top level period cgroup that limits whole domain enforcement period for a quota Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_

Re: [libvirt] Ability to add arbitrary data to snapshots

2016-01-26 Thread Alexander Burluka
On 01/25/2016 04:16 PM, Daniel P. Berrange wrote: On Mon, Jan 25, 2016 at 02:55:30PM +0300, Alexander Burluka wrote: For example, we want to store suspended state of VM. I'm aware that some other careless application dealing with libvirt may erase metadata section and info about additional

Re: [libvirt] Ability to add arbitrary data to snapshots

2016-01-25 Thread Alexander Burluka
Just save this data near main snapshot data (e.g. in /var/lib/libvirt/qemu/snapshot//) and restore it on previous or specified path. On 01/25/2016 11:46 AM, Daniel P. Berrange wrote: On Sat, Jan 23, 2016 at 03:17:43PM +0300, Alexander Burluka wrote: Hello! Currently our Virtuozzo company

Re: [libvirt] Ability to add arbitrary data to snapshots

2016-01-25 Thread Alexander Burluka
, Alexander Burluka wrote: Just save this data near main snapshot data (e.g. in /var/lib/libvirt/qemu/snapshot//) and restore it on previous or specified path. What sort of data is this - we already have ability to record arbitrary metadata in the XML, so it is not particularly attractive to add

[libvirt] Ability to add arbitrary data to snapshots

2016-01-23 Thread Alexander Burluka
, Alexander Burluka -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

Re: [libvirt] [PATCH v5 0/7] Per domain bandwidth settings

2016-01-23 Thread Alexander Burluka
Hi! Unfortunately I've no reply on this patchset from community and would like to remind about it. Thank you! On 01/18/2016 01:08 PM, Alexander Burluka wrote: We decide to make a global per domain bandwidth setting as were discussed in mailing list earlier. This patchset implements

[libvirt] [PATCH v5 2/7] Add global quota parameter necessary definitions

2016-01-18 Thread Alexander Burluka
This parameter controls the maximum bandwidth to be used within a period for whole domain. Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_conf.c

[libvirt] [PATCH v5 0/7] Per domain bandwidth settings

2016-01-18 Thread Alexander Burluka
Changes in v3: remove unneccessary cgroup copying Changes in v4: fix little rebase error Changes in v5: rebase to version 1.3.1 Alexander Burluka (7): Add global period definitions Add global quota parameter necessary definitions Add error checking on global quota and period Add

[libvirt] [PATCH v5 4/7] Add global_period and global_quota XML validation test

2016-01-18 Thread Alexander Burluka
--- tests/qemuxml2argvdata/qemuxml2argv-cputune.xml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml b/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml index 813d201..a056089 100644 --- a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml +++

[libvirt] [PATCH v5 7/7] Implement handling of per-domain bandwidth settings

2016-01-18 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_driver.c | 97 -- 1 file changed, 95 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 48aeab6..1932d7f 100644 --- a/sr

[libvirt] [PATCH v5 5/7] Rename qemuSetupCgroupVcpuBW to qemuSetupBandwidthCgroup

2016-01-18 Thread Alexander Burluka
This rename is required to reuse this function in per-domain bandwidth setup routine Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 14 +++--- src/qemu/qemu_cgroup.h | 6 +++--- src/qemu/qemu_driver.c | 5 ++--- 3 files changed, 12 inse

[libvirt] [PATCH v5 1/7] Add global period definitions

2016-01-18 Thread Alexander Burluka
This parameter represents top level period cgroup that limits whole domain enforcement period for a quota Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_

[libvirt] [PATCH v5 6/7] Implement qemuSetupGlobalCpuCgroup

2016-01-18 Thread Alexander Burluka
This functions setups per-domain cpu bandwidth parameters Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 54 + src/qemu/qemu_cgroup.h | 1 + src/qemu/qemu_process.c | 4 3 files chang

[libvirt] [PATCH v5 3/7] Add error checking on global quota and period

2016-01-18 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_command.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 5d3ab3a..087e9ad 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_com

[libvirt] [PATCH v4 5/7] Rename qemuSetupCgroupVcpuBW to qemuSetupBandwidthCgroup

2016-01-14 Thread Alexander Burluka
This rename is required to reuse this function in per-domain bandwidth setup routine Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 14 +++--- src/qemu/qemu_cgroup.h | 6 +++--- src/qemu/qemu_driver.c | 5 ++--- 3 files changed, 12 inse

[libvirt] [PATCH v4 3/7] Add error checking on global quota and period

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_command.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 5d3ab3a..087e9ad 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_com

[libvirt] [PATCH v4 1/7] Add global period definitions

2016-01-14 Thread Alexander Burluka
This parameter represents top level period cgroup that limits whole domain enforcement period for a quota Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_

[libvirt] [PATCH v4 4/7] Add global_period and global_quota XML validation test

2016-01-14 Thread Alexander Burluka
--- tests/qemuxml2argvdata/qemuxml2argv-cputune.xml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml b/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml index 813d201..a056089 100644 --- a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml +++

[libvirt] [PATCH v4 0/7] Per domain bandwidth settings

2016-01-14 Thread Alexander Burluka
Changes in v3: remove unneccessary cgroup copying Changes in v4: fix little rebase error Alexander Burluka (7): Add global period definitions Add global quota parameter necessary definitions Add error checking on global quota and period Add global_period and global_quota XML validation

[libvirt] [PATCH v4 6/7] Implement qemuSetupGlobalCpuCgroup

2016-01-14 Thread Alexander Burluka
This functions setups per-domain cpu bandwidth parameters Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 54 + src/qemu/qemu_cgroup.h | 1 + src/qemu/qemu_process.c | 4 3 files chang

[libvirt] [PATCH v4 2/7] Add global quota parameter necessary definitions

2016-01-14 Thread Alexander Burluka
This parameter controls the maximum bandwidth to be used within a period for whole domain. Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_conf.c

[libvirt] [PATCH v2 0/8] Per domain bandwidth settings

2016-01-14 Thread Alexander Burluka
Alexander Burluka (8): Add global period definitions Add global quota parameter necessary definitions Add error checking on global quota and period Add global_period and global_quota XML validation test Add new cgroup thread type Rename qemuSetupCgroupVcpuBW to qemuSetupBandwidthCgroup

[libvirt] [PATCH v2 7/8] Implement qemuSetupGlobalCpuCgroup

2016-01-14 Thread Alexander Burluka
This functions setups per-domain cpu bandwidth parameters Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 64 + src/qemu/qemu_cgroup.h | 1 + src/qemu/qemu_process.c | 4 3 files chang

[libvirt] [PATCH 6/7] Implement qemuSetupGlobalCpuCgroup

2016-01-14 Thread Alexander Burluka
This functions setups per-domain cpu bandwidth parameters Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 64 + src/qemu/qemu_cgroup.h | 1 + src/qemu/qemu_process.c | 4 3 files chang

[libvirt] [PATCH v2 1/8] Add global period definitions

2016-01-14 Thread Alexander Burluka
This parameter represents top level period cgroup that limits whole domain enforcement period for a quota Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_

[libvirt] [PATCH v2 8/8] Implement handling of per-domain bandwidth settings

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_driver.c | 120 - 1 file changed, 118 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 48aeab6..6a0fa9b 100644 ---

[libvirt] [PATCH v2 2/8] Add global quota parameter necessary definitions

2016-01-14 Thread Alexander Burluka
This parameter controls the maximum bandwidth to be used within a period for whole domain. Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_conf.c

[libvirt] [PATCH 7/7] Implement handling of per-domain bandwidth settings

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_driver.c | 120 - 1 file changed, 118 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 48aeab6..6a0fa9b 100644 ---

[libvirt] [PATCH 4/7] Add new cgroup thread type

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/util/vircgroup.c | 4 src/util/vircgroup.h | 1 + 2 files changed, 5 insertions(+) diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c index 78f519c..b829794 100644 --- a/src/util/vircgroup.c +++ b/src/util/vircg

[libvirt] [PATCH v2 4/8] Add global_period and global_quota XML validation test

2016-01-14 Thread Alexander Burluka
--- tests/qemuxml2argvdata/qemuxml2argv-cputune.xml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml b/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml index 813d201..a056089 100644 --- a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml +++

[libvirt] [PATCH v2 5/8] Add new cgroup thread type

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/util/vircgroup.c | 4 src/util/vircgroup.h | 1 + 2 files changed, 5 insertions(+) diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c index 78f519c..b829794 100644 --- a/src/util/vircgroup.c +++ b/src/util/vircg

[libvirt] [PATCH v2 3/8] Add error checking on global quota and period

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_command.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 5d3ab3a..087e9ad 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_com

Re: [libvirt] [PATCH v2 0/8] Per domain bandwidth settings

2016-01-14 Thread Alexander Burluka
I am sorry, the old version was resend with v2, please ignore it. On 01/14/2016 01:27 PM, Alexander Burluka wrote: We decide to make a global per domain bandwidth setting as were discussed in mailing list earlier. This patchset implements hierarchy top level cpu.cfs_period_us

[libvirt] [PATCH 5/7] Rename qemuSetupCgroupVcpuBW to qemuSetupBandwidthCgroup

2016-01-14 Thread Alexander Burluka
This rename is required to reuse this function in per-domain bandwidth setup routine Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 14 +++--- src/qemu/qemu_cgroup.h | 6 +++--- src/qemu/qemu_driver.c | 5 ++--- 3 files changed, 12 inse

[libvirt] [PATCH v2 6/8] Rename qemuSetupCgroupVcpuBW to qemuSetupBandwidthCgroup

2016-01-14 Thread Alexander Burluka
This rename is required to reuse this function in per-domain bandwidth setup routine Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 14 +++--- src/qemu/qemu_cgroup.h | 6 +++--- src/qemu/qemu_driver.c | 5 ++--- 3 files changed, 12 inse

[libvirt] [PATCH v4 7/7] Implement handling of per-domain bandwidth settings

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_driver.c | 97 -- 1 file changed, 95 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 48aeab6..1932d7f 100644 --- a/sr

[libvirt] [PATCH v3 2/7] Add global quota parameter necessary definitions

2016-01-14 Thread Alexander Burluka
This parameter controls the maximum bandwidth to be used within a period for whole domain. Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_conf.c

[libvirt] [PATCH v3 4/7] Add global_period and global_quota XML validation test

2016-01-14 Thread Alexander Burluka
--- tests/qemuxml2argvdata/qemuxml2argv-cputune.xml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml b/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml index 813d201..a056089 100644 --- a/tests/qemuxml2argvdata/qemuxml2argv-cputune.xml +++

[libvirt] [PATCH v3 5/7] Rename qemuSetupCgroupVcpuBW to qemuSetupBandwidthCgroup

2016-01-14 Thread Alexander Burluka
This rename is required to reuse this function in per-domain bandwidth setup routine Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 14 +++--- src/qemu/qemu_cgroup.h | 6 +++--- src/qemu/qemu_driver.c | 5 ++--- 3 files changed, 12 inse

[libvirt] [PATCH v3 6/7] Implement qemuSetupGlobalCpuCgroup

2016-01-14 Thread Alexander Burluka
This functions setups per-domain cpu bandwidth parameters Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 64 + src/qemu/qemu_cgroup.h | 1 + src/qemu/qemu_process.c | 4 3 files chang

[libvirt] [PATCH v3 3/7] Add error checking on global quota and period

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_command.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 5d3ab3a..087e9ad 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_com

[libvirt] [PATCH v3 1/7] Add global period definitions

2016-01-14 Thread Alexander Burluka
This parameter represents top level period cgroup that limits whole domain enforcement period for a quota Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_

[libvirt] [PATCH v3 7/7] Implement handling of per-domain bandwidth settings

2016-01-14 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 12 +-- src/qemu/qemu_driver.c | 97 -- 2 files changed, 96 insertions(+), 13 deletions(-) diff --git a/src/qemu/qemu_cgroup.c b/src/qemu/qemu_cgroup.c

[libvirt] [PATCH v3 0/7] Per domain bandwidth settings

2016-01-14 Thread Alexander Burluka
Changes in v3: remove unneccessary cgroup copying Alexander Burluka (7): Add global period definitions Add global quota parameter necessary definitions Add error checking on global quota and period Add global_period and global_quota XML validation test Rename qemuSetupCgroupVcpuBW

Re: [libvirt] [PATCH 0/7] Per domain bandwidth settings

2016-01-13 Thread Alexander Burluka
. Thank you! On 01/12/2016 08:30 PM, Michal Privoznik wrote: On 12.01.2016 17:42, Alexander Burluka wrote: We decide to make a global per domain bandwidth setting as were discussed in mailing list earlier. This patchset implements hierarchy top level cpu.cfs_period_us and cpu.cfs_quota_us control

Re: [libvirt] [PATCH 0/7] Per domain bandwidth settings

2016-01-13 Thread Alexander Burluka
our Parallels Cloud Server 6. Our customers really want it. By the way, period and quota can be set on running domain. On 01/13/2016 01:42 PM, Michal Privoznik wrote: On 13.01.2016 11:17, Alexander Burluka wrote: You are absolutely right, this patchset allows to set domain top-level cpu.cfs_period

Re: [libvirt] [PATCH 0/7] Per domain bandwidth settings

2016-01-13 Thread Alexander Burluka
On 01/12/2016 08:19 PM, Laine Stump wrote: On 01/12/2016 11:42 AM, Alexander Burluka wrote: We decide to make a global per domain bandwidth setting as were discussed in mailing list earlier. This patchset implements hierarchy top level cpu.cfs_period_us and cpu.cfs_quota_us control knob. I've

[libvirt] [PATCH 7/7] Implement handling of per-domain bandwidth settings

2016-01-12 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_driver.c | 120 - 1 file changed, 118 insertions(+), 2 deletions(-) diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c index 48aeab6..6a0fa9b 100644 ---

[libvirt] [PATCH 2/7] Add global quota parameter necessary definitions

2016-01-12 Thread Alexander Burluka
This parameter controls the maximum bandwidth to be used within a period for whole domain. Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_conf.c

[libvirt] [PATCH 6/7] Implement qemuSetupGlobalCpuCgroup

2016-01-12 Thread Alexander Burluka
This functions setups per-domain cpu bandwidth parameters Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 64 + src/qemu/qemu_cgroup.h | 1 + src/qemu/qemu_process.c | 4 3 files chang

[libvirt] [PATCH 0/7] Per domain bandwidth settings

2016-01-12 Thread Alexander Burluka
We decide to make a global per domain bandwidth setting as were discussed in mailing list earlier. This patchset implements hierarchy top level cpu.cfs_period_us and cpu.cfs_quota_us control knob. I've named this parameters as global_period and global_quota. Alexander Burluka (7): Add global

[libvirt] [PATCH 4/7] Add new cgroup thread type

2016-01-12 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/util/vircgroup.c | 4 src/util/vircgroup.h | 1 + 2 files changed, 5 insertions(+) diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c index 78f519c..b829794 100644 --- a/src/util/vircgroup.c +++ b/src/util/vircg

[libvirt] [PATCH 3/7] Add error checking on global quota and period

2016-01-12 Thread Alexander Burluka
Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_command.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c index 5d3ab3a..087e9ad 100644 --- a/src/qemu/qemu_command.c +++ b/src/qemu/qemu_com

[libvirt] [PATCH 5/7] Rename qemuSetupCgroupVcpuBW to qemuSetupBandwidthCgroup

2016-01-12 Thread Alexander Burluka
This rename is required to reuse this function in per-domain bandwidth setup routine Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- src/qemu/qemu_cgroup.c | 14 +++--- src/qemu/qemu_cgroup.h | 6 +++--- src/qemu/qemu_driver.c | 5 ++--- 3 files changed, 12 inse

[libvirt] [PATCH 1/7] Add global period definitions

2016-01-12 Thread Alexander Burluka
This parameter represents top level period cgroup that limits whole domain enforcement period for a quota Signed-off-by: Alexander Burluka <aburl...@virtuozzo.com> --- docs/schemas/domaincommon.rng| 5 + include/libvirt/libvirt-domain.h | 16 src/conf/domain_

Re: [libvirt] Global domain cpu.cfs_quota_us setup

2015-12-21 Thread Alexander Burluka
Thank you for help! On 12/21/2015 02:57 PM, Daniel P. Berrange wrote: On Mon, Dec 21, 2015 at 02:47:14PM +0300, Alexander Burluka wrote: Hi! Thank you for answer, Daniel, I should excuse for a long delay before reply. It was caused by long offline discussion in our company . Our business logic

Re: [libvirt] Global domain cpu.cfs_quota_us setup

2015-12-21 Thread Alexander Burluka
limit of 1200MHz and equal load among vCPUs would result in a max of 300MHz per vCPU." Would anyone mind if we implement this feature as an optional parameter in libvirt? On 12/08/2015 12:27 PM, Daniel P. Berrange wrote: On Mon, Dec 07, 2015 at 03:17:53PM +0300, Alexander Burluka wrote: Hello all

[libvirt] Global domain cpu.cfs_quota_us setup

2015-12-07 Thread Alexander Burluka
: cpu.cfs_quota_us (5) | |-emulator/cpu.cfs_quota_us (-1) |-vcpu0/cpu.cfs_quota_us (-1) |-vcpu1/cpu.cfs_quota_us (-1) |-vcpu2/cpu.cfs_quota_us (-1) \-vcpuN/cpu.cfs_quota_us (-1) Thank you! -- Regards, Alexander Burluka -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman

[libvirt] [PATCH v5 1/3] Parallels: remove disk serial number check

2015-04-09 Thread Alexander Burluka
OpenStack needs disk serial number setup because nova boot --block-device-mapping command generates that param in libvirt xml. I took QEMU libvirt driver behavior as a base. QEMU driver skips inability to set serial and continues work. So Parallels driver will ignore this param too and let domain

[libvirt] [PATCH v5 3/3] Parallels: implemented domainAttachDevice

2015-04-09 Thread Alexander Burluka
That function uses domainAttachDeviceFlags Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c index 8058903

[libvirt] [PATCH v5 0/3] Parallels disk device attach

2015-04-09 Thread Alexander Burluka
This patchset implements disk device attachment and allows OpenStack to attach volumes to Parallels-driven instances. Parallels Cloud Server SDK supports live attachment of disk devices and virtual interfaces cards. Alexander Burluka (3): Parallels: remove disk serial number check Parallels

[libvirt] [PATCH v5 2/3] Parallels: implement domainAttachDeviceFlags

2015-04-09 Thread Alexander Burluka
Parallels Cloud Server supports block devices and virtual NIC live attachment. I implemented that function for block devices so OpenStack volume attachment is now works. Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c | 65

[libvirt] [PATCH v2 0/3] Parallels disk device attach

2015-03-27 Thread Alexander Burluka
This patchset implements disk device attachment and allows OpenStack to attach volumes to Parallels-driven instances. Parallels Cloud Server SDK supports live attachment of disk devices and virtual interfaces cards. Alexander Burluka (3): Parallels: remove disk serial number check Parallels

[libvirt] [PATCH v2 2/3] Parallels: implement domainAttachDeviceFlags

2015-03-27 Thread Alexander Burluka
Parallels Cloud Server supports block devices and virtual NIC live attachment. I implemented that function for block devices so OpenStack volume attachment is now works. Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c | 51

[libvirt] [PATCH v2 1/3] Parallels: remove disk serial number check

2015-03-27 Thread Alexander Burluka
OpenStack needs disk serial number setup because nova boot --block-device-mapping command generates that param in libvirt xml. I took QEMU libvirt driver behavior as a base. QEMU driver skips inability to set serial and continues work. So Parallels driver will ignore this param too and let domain

[libvirt] [PATCH v2 3/3] Parallels: implemented domainAttachDevice

2015-03-27 Thread Alexander Burluka
That function uses domainAttachDeviceFlags Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c index dd8ed7d

[libvirt] [PATCH v3 3/3] Parallels: implemented domainAttachDevice

2015-03-27 Thread Alexander Burluka
That function uses domainAttachDeviceFlags Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c index 8058903

[libvirt] [PATCH v3 2/3] Parallels: implement domainAttachDeviceFlags

2015-03-27 Thread Alexander Burluka
Parallels Cloud Server supports block devices and virtual NIC live attachment. I implemented that function for block devices so OpenStack volume attachment is now works. Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c | 65

[libvirt] [PATCH v3 0/3] Parallels disk device attach

2015-03-27 Thread Alexander Burluka
This patchset implements disk device attachment and allows OpenStack to attach volumes to Parallels-driven instances. Parallels Cloud Server SDK supports live attachment of disk devices and virtual interfaces cards. Alexander Burluka (3): Parallels: remove disk serial number check Parallels

[libvirt] [PATCH v3 1/3] Parallels: remove disk serial number check

2015-03-27 Thread Alexander Burluka
OpenStack needs disk serial number setup because nova boot --block-device-mapping command generates that param in libvirt xml. I took QEMU libvirt driver behavior as a base. QEMU driver skips inability to set serial and continues work. So Parallels driver will ignore this param too and let domain

[libvirt] [PATCH v4 3/3] Parallels: implemented domainAttachDevice

2015-03-27 Thread Alexander Burluka
That function uses domainAttachDeviceFlags Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c index 8058903

[libvirt] [PATCH v4 2/3] Parallels: implement domainAttachDeviceFlags

2015-03-27 Thread Alexander Burluka
Parallels Cloud Server supports block devices and virtual NIC live attachment. I implemented that function for block devices so OpenStack volume attachment is now works. Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c | 65

[libvirt] [PATCH v4 1/3] Parallels: remove disk serial number check

2015-03-27 Thread Alexander Burluka
OpenStack needs disk serial number setup because nova boot --block-device-mapping command generates that param in libvirt xml. I took QEMU libvirt driver behavior as a base. QEMU driver skips inability to set serial and continues work. So Parallels driver will ignore this param too and let domain

[libvirt] [PATCH v4 0/3] Parallels disk device attach

2015-03-27 Thread Alexander Burluka
This patchset implements disk device attachment and allows OpenStack to attach volumes to Parallels-driven instances. Parallels Cloud Server SDK supports live attachment of disk devices and virtual interfaces cards. Alexander Burluka (3): Parallels: remove disk serial number check Parallels

[libvirt] [PATCH 0/3] Parallels disk device attach

2015-03-26 Thread Alexander Burluka
This patchset implements disk device attachment and allows OpenStack to attach volumes to Parallels-driven instances. Parallels Cloud Server SDK supports live attachment of disk devices and virtual interfaces cards. Alexander Burluka (3): Parallels: remove disk serial number check Parallels

[libvirt] [PATCH 3/3] Parallels: implemented domainAttachDevice

2015-03-26 Thread Alexander Burluka
That function uses domainAttachDeviceFlags Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/src/parallels/parallels_driver.c b/src/parallels/parallels_driver.c index 87297cf

[libvirt] [PATCH 2/3] Parallels: implement domainAttachDeviceFlags

2015-03-26 Thread Alexander Burluka
Parallels Cloud Server supports block devices and virtual NIC live attachment. I implemented that function for block devices so OpenStack volume attachment is now works. Signed-off-by: Alexander Burluka aburl...@parallels.com --- src/parallels/parallels_driver.c | 45

[libvirt] [PATCH 1/3] Parallels: remove disk serial number check

2015-03-26 Thread Alexander Burluka
OpenStack needs disk serial number setup because nova boot --block-device-mapping command generates that param in libvirt xml. I took QEMU libvirt driver behavior as a base. QEMU driver skips inability to set serial and continues work. So Parallels driver will ignore this param too and let domain

  1   2   >