bump version to 3.4-9

split old style pipe open call

bump version to 3.4-8

fix #909: pass rate to tap_plug()

When using OVS tap_plug() resets rate limiting so we need
to pass it along to reapply it.

The rate on its own can still be hot-plugged with the
regular tap_rate_limit() call.

bump version to 3.4-7

deactivate volumes if vm start command fails

prevents volumes to be active when they are not actually in use
this is a cleanup of Dhaussy Alexandre's patch from 02/15/2016

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>

bump version to 3.4-6

qmp drive-mirror : set big timeout

drive-mirror is doing lseek on source image before starting, and this can take a lot of time for big nfs volume
during this time, qmp socket is hanging
http://lists.nongnu.org/archive/html/qemu-devel/2015-05/msg01838.html
so we need to setup a big timeout

qemu devs are currently working to fix this

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

qemu-mirror : block job complete : use ready flag

Since qemu 2.2, a new "ready" flag has been added to blockjob
http://git.qemu.org/?p=qemu.git;a=commit;h=ef6dbf1e46ebd1d41ab669df5bba0bbdec6bd374
to known if we can complete it.

we can't use len==offset to known if all block are mirrored, because behaviour will change soon in qemu 2.3

http://git.qemu.org/?p=qemu.git;a=commit;h=b21c76529d55bf7bb02ac736b312f5f8bf033ea2
"block/mirror: Improve progress report

Instead of taking the total length of the block device as the block
job's length, use the number of dirty sectors. The progress is now the
number of sectors mirrored to the target block device. Note that this
may result in the job's length increasing during operation, which is
however in fact desirable.
"

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 3.4-5

implement offline migration on zfs

Signed-off-by: Wolfgang Link <w.link@proxmox.com>

fix bug #618: correct typo

Signed-off-by: Wolfgang Link <w.link@proxmox.com>

bump version to 3.4-4

fix bug #617: change command arrangement

the ovs-vsctl del-port flush the tc settings.
So we have to change the arrangement of setting up the network.

Signed-off-by: Wolfgang Link <w.link@proxmox.com>

use aio=native only with O_DIRECT (cache=none|directsync)

Currently qemu auto fallback to aio=threads if cache=none|directsync
It's better to handle that correctly

see:
https://bugzilla.redhat.com/show_bug.cgi?id=1086704

http://wiki.qemu.org/ChangeLog/2.3
Future incompatible changes:
Block device parameter aio=native has no effect without cache.direct=on. It will be made an error.

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

update changelog

always setup balloon polling interval.

we always need to enable pooling interval, because it doesn't seem to be setup with -machine option

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 3.4-3

vmstatus : use vcpus if defined

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

Revert "balloon: use qom-get for guest balloon statistics V5"

This reverts commit 09f7c4dfbc5de3e4c63f7e54e198bf2eb3374c02.

bump version to 3.4-2

balloon: use qom-get for guest balloon statistics V5

changelog:

we use MB, not bytes

$d->{balloon} = int($info->{stats}->{"stat-total-memory"}/1024/1024);
$d->{freemem} = int($info->{stats}->{"stat-free-memory"}/1024/1024);

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

drive_add : escape \ character

storage.cfg
-----------
rbd mystorage
    monhost 10.5.0.11:6789; 10.5.0.12:6789; 10.5.0.13:6789

drive_add auto "file=rbd:poolceph1/vm-101-disk-8:mon_host=10.5.0.11\\:6789; 10.5.0.12\\:6789; 10.5.0.13\\:6789:id=admin:auth_supported=cephx:keyring=/etc/pve/priv/ceph/cephzimbra.keyring,if=none,id=drive-virtio2,aio=native,cache=none,detect-zeroes=on"

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bugfix : allow manual balloning if shares = 0

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 3.4-1

fix max allowed vcpus error message

correct use hash with keys

bump version to 3.3-20

correctly set and remove lock

bump version to 3.3-19

commit pending values when changing CDROM

bump version to 3.3-18

bugfix : add missing queues nic option in print_net

Currently the nic queues option is removed when we try to update config

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 3.3-17

QemuServer: check snapshot befor rollback

this will check, if it is possibel to rollback a snapshot befor VM will shutdown and get locked.

Signed-off-by: Wolfgang Link <w.link@proxmox.com>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vm_devices_list: make sure $device->{'qdev_id'} is set

vm_devices_list : list devices behind pci bridge

This fix hotplug for devices behind bridges, like nic6->24 for example

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

fix CDROM hotplug

cleanup error messages

bump version to 3.3-16

QemuServer: fix wrong binding of pci root ports, bridges or switches to vfio

Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

update changelog

bug 597: hotplug fix

wrap params in dopple quotes, so spaces will accept.

Signed-off-by: Wolfgang Link <w.link@proxmox.com>

memory hotplug option is not hotpluggable

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 3.3-15

add foreach_dimm sub

and use it in memory hotplug and config_to_cmd

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 3.3-14

memory hotplug patch v10

This patch allow to hotplug memory dimm modules
though a new option : dimm_memory

The dimm modules are generated from a map

     dimmid       size  dimm_memory
     dimm0        512        512     100.00 0
     dimm1        512       1024      50.00 1
     dimm2        512       1536      33.33 2
     dimm3        512       2048      25.00 3
     dimm4        512       2560      20.00 0
     dimm5        512       3072      16.67 1
     dimm6        512       3584      14.29 2
     dimm7        512       4096      12.50 3
     dimm8        512       4608      11.11 0
     dimm9        512       5120      10.00 1
    dimm10        512       5632       9.09 2
    dimm11        512       6144       8.33 3
    dimm12        512       6656       7.69 0
    dimm13        512       7168       7.14 1
    dimm14        512       7680       6.67 2
    dimm15        512       8192       6.25 3
    dimm16        512       8704       5.88 0
    dimm17        512       9216       5.56 1
    dimm18        512       9728       5.26 2
    dimm19        512      10240       5.00 3
    dimm20        512      10752       4.76 0
    ...

   dimm241      65536    3260416       2.01 1
   dimm242      65536    3325952       1.97 2
   dimm243      65536    3391488       1.93 3
   dimm244      65536    3457024       1.90 0
   dimm245      65536    3522560       1.86 1
   dimm246      65536    3588096       1.83 2
   dimm247      65536    3653632       1.79 3
   dimm248      65536    3719168       1.76 0
   dimm249      65536    3784704       1.73 1
   dimm250      65536    3850240       1.70 2
   dimm251      65536    3915776       1.67 3
   dimm252      65536    3981312       1.65 0
   dimm253      65536    4046848       1.62 1
   dimm254      65536    4112384       1.59 2
   dimm255      65536    4177920       1.57 3

max dimm_memory size is 4TB, which is the current qemu limit

If the dimm_memory value is not aligned on memory module, we align the dimm_memory on the next module.

vmid.conf
---------
memory: 1024
numa:1
hotplug: memmory

when hotplug memory option is enabled, the minimum memory value must be 1GB, and also numa need to be enabled.

we assign the first 1GB as static memory, splitted on each numa nodes.
The remaining memory is assigned on hotpluggable dimm devices.

The static memory need to be also 128MB aligned, to have other dimm devices aligned too.

This 128MB alignment is a linux limitation, windows can align on 2MB size.

Numa need to be aligned, as linux guest don't boot on some setup with multi sockets,
and windows need numa to be able to hotplug memory

hotplug
----
qm set <vmid> -memory X    (where X is bigger than current value)

unplug (not yet implemented in qemu)
------
qm set <vmid> -memory X  (where X is lower than current value)

linux guest
-----------
-acpi hotplug module should be loaded in guest
-need a recent kernel. (tested with 3.10)

can be enable automaticaly, adding:

/lib/udev/rules.d/80-hotplug-cpu-mem.rules
SUBSYSTEM=="cpu", ACTION=="add", TEST=="online", ATTR{online}=="0", \
 ATTR{online}="1"

SUBSYSTEM=="memory", ACTION=="add", TEST=="state", ATTR{state}=="offline", \
 ATTR{state}="online"

windows guest
-------------

tested with:

- windows 2012 standard
- windows 2008 enterprise/datacenter

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 3.3-13

use PVE::Tools::split_list

hotplug config: allow to enable specific features

bump version to 3.3-12

enable vcpus hotplug

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

add vcpus option

vcpus = current allocate vpus to virtual machine

maxcpus is now compute from $sockets*cores
vcpus = maxcpus if not defined

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

apply pending changes at vm_stop

bump version to 3.3-11

enable hotplug by default

Add link_down flag to network config

Original patch by Wolfgang, adopted for new hotplug implementation.

I do not verify link status, because that patch was rejected upstream.

Signed-off-by: Wolfgang Link <wolfgang@linksystems.org>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

Change check for better understanding!

Signed-off-by: Wolfgang Link <wolfgang@linksystems.org>

bump version to 3.3-10

Support additional e1000 variants for VM machines

It is required for OS X on qemu

Signed-off-by: Kamil Trzcinski <ayufan@ayufan.eu>

bump version to 3.3-9

pending api : fix parsing 0 value

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

fix test for ballon hotplug

set boot strict=on to prevent booting from not listed boot devices

Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

do not set romfile= if network is not in boot order

Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

bump version to 3.3-8

fix previous commit: forgot API definitions

update_vm api : add optionnal revert option

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

code cleanup

fix balloon pending update

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

don't try to hotplug|unplug ide && sata drive

We can't hotplug theses devices,
so we can avoid to throw always an error when adding them

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

add optionnal current param to config api

config api return by default the pending config, like this it don't break current behaviour

Add an optionnal current param to get to current running config

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

API: list new 'pending' method

pending : add disk hot-unplug

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

remove unused code

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vmconfig_update_net: do not call vm_deviceplug() if hotplug == 0

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vmconfig_hotplug_pending : add update_disk

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vm_deviceplug: always raise exception on error

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vm_deviceunplug: raise expection if something fail

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vmconfig_hotplug_pending : add update_net

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vmconfig_hotplug_pending: correctly skip values

Do not use $skip variable (simply raise an exception)

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

rename qemu_bridgeadd to qemu_add_pci_bridge

To make it obvious that we add a PCI device, and not a network bridge.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

implement API/CLI to get pending changes

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vmconfig_hotplug_pending: improve hotplug error handling

Simplify code, and allow to partially apply pending changes using
a new $selection parameter.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

code cleanup, delete trailing white space

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vmconfig_cleanup_pending: new method to clenup setting in [PENDING]

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vmconfig_hotplug_pending: implement tablet hotplug

Remove special case: now, we only hotplug if 'hotplug' is enabled.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

implement trivial hotplug

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

fix balloon consistency check (consider pending changes)

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

vm_start: apply pending changes

I move related helper methods into PVE::QemuServer.

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

implement vmconfig_apply_pending for stopped VM

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

update_vm_api: always write into pending section

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

write_vm_config : write pending change

example:

$conf->{pending}->{virtio1}
$conf->{pending}->{delete} = "net0,net1"

[PENDING]
virtio1: ...
delete: net0,net1

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

parse_vm_config: only allow 'delete' inside [PENDING]

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

parse_vm_config: correctly handle $descr

Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

parse_vm_config : parse pending changes

example:

[PENDING]
virtio1:...
delete:net0,net1

$conf->{pending}->{virtio1}
$conf->{pending}->{del}->{net0}
$conf->{pending}->{del}->{net1}

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
Signed-off-by: Dietmar Maurer <dietmar@proxmox.com>

bump version to 3.3-7

this commit:

commit 1c0c1c17b020710fd841c399c1e147ccc10007cd
Author: Wolfgang Link <wolfgang@linksystems.org>
Date:   Wed Nov 26 11:11:40 2014 +0100

    shutdown by Qemu Guest Agent if the agent flag in the config is set

    Important: "guest-shutdown" returns only by error a message.

Signed-off-by: Wolfgang Link <wolfgang@linksystems.org>
breaks live migration as it always tries to load the vm config - even in case of $nocheck. Also it double loads the config ($conf && $config)
Signed-off-by: Stefan Priebe <s.priebe@profihost.ag>

white space cleanups