Re: [pve-devel] [PATCH kernel] Fix #927: add IPoIB performance regression fix

2016-10-13 Thread Michael Rasmussen
On Thu, 13 Oct 2016 16:49:30 +0200 Wolfgang Bumiller wrote: > Fixes kernel bug #111921 > --- > ...ck-the-IB-LL-address-into-the-hard-header.patch | 365 > + > Makefile | 2 + > 2 files changed, 367 insertions(+) > create mode 1006

Re: [pve-devel] pve-manager and disk IO monitoring

2016-10-13 Thread Michael Rasmussen
On Fri, 14 Oct 2016 07:42:38 +0200 (CEST) Alexandre DERUMIER wrote: > > Also, currently, we have virtio-scsi-single. I don't known if a lot of user > already use it, > but maybe it could be better to use it as an option > scsihw:virtio-scsi,type=generic|block,x=single > > ? I use virio-

Re: [pve-devel] pve-manager and disk IO monitoring

2016-10-13 Thread Alexandre DERUMIER
>> maybe could we add an option on scsihw ? >> scsihw:virtio-scsi,type=generic|block >> ? >>would be great if somebody provides a patch... Also, currently, we have virtio-scsi-single. I don't known if a lot of user already use it, but maybe it could be better to use it as an option scsihw:v

Re: [pve-devel] pve-manager and disk IO monitoring

2016-10-13 Thread Dietmar Maurer
> > @Alexandre: This was for performance reasons? > > > Any decisions made yet to revert this patch? I want to use scsi-block by default, but the suggestion was to provide a way to switch back to scsi-generic. Alrexandre suggested: > maybe could we add an option on scsihw ? scsihw:virtio-scsi,t

Re: [pve-devel] pve-manager and disk IO monitoring

2016-10-13 Thread Michael Rasmussen
On Fri, 30 Sep 2016 07:50:11 +0200 (CEST) Dietmar Maurer wrote: > > So my question is: Why use scsi-generic instead of scsi-block when > > scsi-generic prevents blockstats? > > commit d454d040338a6216c8d3e5cc9623d6223476cb5a > Author: Alexandre Derumier > Date: Tue Aug 28 12:46:07 2012 +020

[pve-devel] [PATCH kernel] Fix #927: add IPoIB performance regression fix

2016-10-13 Thread Wolfgang Bumiller
Fixes kernel bug #111921 --- ...ck-the-IB-LL-address-into-the-hard-header.patch | 365 + Makefile | 2 + 2 files changed, 367 insertions(+) create mode 100644 IB-ipoib-move-back-the-IB-LL-address-into-the-hard-header.patch diff --g

[pve-devel] [PATCH docs] Update the qm command line example to use the new volume format

2016-10-13 Thread Emmanuel Kasper
Also since the command line is now longer, move the NIC creation to a separate command. --- qm.adoc | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/qm.adoc b/qm.adoc index a0b6950..3624e2f 100644 --- a/qm.adoc +++ b/qm.adoc @@ -476,9 +476,16 @@ create and delete vir

[pve-devel] Applied: Re: [PATCH] qemu2.7 : qmp-fix-object-add-assert-without-props

2016-10-13 Thread Wolfgang Bumiller
Applied. Had to fixup the mail's content type though ;-) Looks like you accidentally answered with 'a' to the charset-query of git send-email ;-) (Also added the additional tags from the upstream commit message.) On Thu, Oct 13, 2016 at 11:25:36AM +0200, Alexandre Derumier wrote: > This fix objec

[pve-devel] [PATCH qemu-server] Fix typo

2016-10-13 Thread Emmanuel Kasper
--- PVE/API2/Qemu.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index 21fbebb..ad7a0c0 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@ -1732,7 +1732,7 @@ __PACKAGE__->register_method({ optional => 1,

[pve-devel] [PATCH docs] Add subchapter documenting the startup/shutdown options

2016-10-13 Thread Emmanuel Kasper
based on http://pve.proxmox.com/wiki/Virtual_Machine_Startup_and_Shutdown_Behavior --- NB: a related patch for Containers will follow. qm.adoc | 33 + 1 file changed, 33 insertions(+) diff --git a/qm.adoc b/qm.adoc index a30e7dc..a0b6950 100644 --- a/qm.adoc +++ b

[pve-devel] [PATCH] enable drive-mirror with iothread for qemu 2.7

2016-10-13 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 05edd7a..ec8df94 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -5933,8 +5933,11 @@ sub clone_disk { if

[pve-devel] [PATCH] enable virtio-scsi iothread hot-unplug for qemu 2.7 v2

2016-10-13 Thread Alexandre Derumier
changelog: add a check on current kvm binary version Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 0c6d604..05edd7a 100644 --- a/PVE/QemuServer.pm +++ b/PVE/Qem

Re: [pve-devel] qemu 2.7 : iothread hotplug (object-add) assert crash

2016-10-13 Thread Alexandre DERUMIER
I have tested the patch, it's working fine, no more crash with iothread hotplug. - Mail original - De: "aderumier" À: "pve-devel" Envoyé: Jeudi 13 Octobre 2016 11:07:23 Objet: [pve-devel] qemu 2.7 : iothread hotplug (object-add) assert crash When trying to hotplug an disk with iothread

[pve-devel] [PATCH] qemu2.7 : qmp-fix-object-add-assert-without-props

2016-10-13 Thread Alexandre Derumier
This fix object-add iothread crash Signed-off-by: Alexandre Derumier --- .../0004-qmp-fix-object-add-assert-without-props | 65 ++ debian/patches/series | 1 + 2 files changed, 66 insertions(+) create mode 100644 debian/patches/extra/0004-qmp

[pve-devel] qemu 2.7 : iothread hotplug (object-add) assert crash

2016-10-13 Thread Alexandre DERUMIER
When trying to hotplug an disk with iothread, the iothread object-add crash the vm with: kvm: qom/object_interfaces.c:115: user_creatable_add_type: Assertion `qdict' failed. Aborted Seem related to this fix: https://lists.gnu.org/archive/html/qemu-devel/2016-09/msg05286.html "qmp: fix objec

[pve-devel] Applied [PATCH kernel] update to Ubuntu 4.4.0-42.62, bump version to 4.4.21-68

2016-10-13 Thread Fabian Grünbichler
--- Note: already applied Makefile | 4 ++-- changelog.Debian | 6 ++ ubuntu-xenial.tgz | Bin 145633868 -> 145659146 bytes 3 files changed, 8 insertions(+), 2 deletions(-) diff --git a/Makefile b/Makefile index fedae5e..4055d41 100644 --- a/Makefile +++ b/Makefile @@ -2,7 +2,7

[pve-devel] [PATCH 3/4] cpu hotplug : add cpu hot-unplug support

2016-10-13 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 31 +-- 1 file changed, 29 insertions(+), 2 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 186fae1..40e9e52 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -3761,6 +3761,9 @@ su

[pve-devel] [PATCH 2/4] cpu hotplug : add coldplugged cpu to qemu command line

2016-10-13 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 3e069ea..186fae1 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -2975,8 +2975,18 @@ sub config_to_command

[pve-devel] [PATCH 1/4] cpu hotplug : add print_cpu_device

2016-10-13 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 22 ++ 1 file changed, 22 insertions(+) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index f4bb4dd..3e069ea 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -1674,6 +1674,28 @@ sub print_netdev_full {

[pve-devel] [PATCH 4/4] cpu hotplug : add new cpu hotplug method for qemu 2.7

2016-10-13 Thread Alexandre Derumier
Signed-off-by: Alexandre Derumier --- PVE/QemuServer.pm | 28 +--- 1 file changed, 25 insertions(+), 3 deletions(-) diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm index 40e9e52..0c6d604 100644 --- a/PVE/QemuServer.pm +++ b/PVE/QemuServer.pm @@ -3777,7 +3777,7 @@ sub q

[pve-devel] qemu-server : add support for qemu 2.7 cpu hot(un)plug v2

2016-10-13 Thread Alexandre Derumier
Changelog: - rebase on last git - remove some whitespace I have retested them, It's working fine ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel