switch to 'ha-manager set'

ha-manager enabled/disabled will get removed

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

vmstate snapshot : activate|deactivate volume

This fix vmstate snasphot on krbd volume

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

qemu_volume_snapshot_delete : fix krbd snapshot delete

like for snapshot, we need to check if krbd is enabled, to known
if we need to use qmp delete-drive-snapshot or storage command directly

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

VZDump/QemuServer: set bless clas correctly

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

qxlnum: use $winversion

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

hv_vendor_id : for winversion >= 6 , not >=7.

as It was previously.

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

cleanup windows version handling and hyperv enlightments

This cleanup windows guest os version handling,
with normalizing ostype with numbers in a new windows_version sub.

if($ostype eq 'wxp' || $ostype eq 'w2k3' || $ostype eq 'w2k') {
      $winversion = 5;
} elsif($ostype eq 'w2k8' || $ostype eq 'wvista') {
      $winversion = 6;
} elsif ($ostype =~ m/^win(\d+)$/) {
      $winversion = $1;
}

so we can simply do test on windows version with lower or upper version

Hyperv enlightments configuration is centralized
in a new add_hyperv_enlighments sub.

Also disable hyperv with win < 8 + ovmf.

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

remove format_description from $met_fmt->{model}

remove unnecessary format_description from cputype

register new standard option 'pve-qm-image-format'

bump version to 4.0-94

fix add_throttle_desc - pass correct units

bump version to 4.0-93

Close #1195: support iops/bps_max_length throttling options

fix a perl warning when failing to parse a new drive string

fix a warning: discard is not a number

allow dedicated migration network, bug #1177

Without this patch we use the network were the cluster traffic runs
for sending migration traffic. This is not ideal as it may hinder
cluster traffic. Further some users have a powerful network which
would be perfect for migrations, with this patch they can run the
migration traffic over such a network without having the corosync
traffic on the same network.

The network is configurable through /etc/pve/datacenter.cfg which
got a new property, namely migration. migration has two
subproperties: type (replaces the old migration_unsecure property)
and network.

For the case of a network failure or that a VM has to be moved over
another network for arbitrary other reasons I added the
migration_type and migration_network parameters to qm migrate (and
respectively vm_start as this gets used on migration).
They allow overwriting the datacenter.cfg settings.

Fixes bug #1177

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

change default value for cpuunits to 1024

Use the same default as containers.

bump version  to 4.0-92

Close #351: add more info to backup log

Before patch:
INFO: exclude disk 'scsi1' (backup=no)
INFO: skip unused drive 'local:401/vm-401-disk-3.raw' (not included into backup)
INFO: skip unused drive 'local:401/vm-401-disk-1.raw' (not included into backup)

After patch applied:
INFO: include disk 'scsi0' local:401/vm-401-disk-4.qcow2
INFO: exclude disk 'scsi1' local:401/vm-401-disk-2.raw (backup=no)
INFO: include disk 'scsi2' pve4tank:vm-401-disk-1
INFO: skip unused drive 'local:401/vm-401-disk-3.raw' (not included into backup)
INFO: skip unused drive 'local:401/vm-401-disk-1.raw' (not included into backup)

snapshot_list: add bash completion for vmid

add qm listsnapshot call

the api call is already there, just not used in qm.pm
copied mostly from pct.pm

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

Fix #1174: remove pve-qm-drive

Let 'cdrom' use the pve-qm-ide format, as it's supposed to
be an alias to ide2.
We're not using the 'alias' schema property since the qemu
configs still use a custom parser (due to the
pending-changes system and the filename-to-volume-id
conversion for legacy support) which does not deal with
schema aliases.

Fix typo

improve error messages

enable drive-mirror with iothread for qemu 2.7 v2

changelog : check running qemu binary version

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

add get_running_qemu_version

return current running qemu process version

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

cpu hotplug : add new cpu hotplug method for qemu 2.7

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

cpu hotplug : add cpu hot-unplug support

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

cpu hotplug : add coldplugged cpu to qemu command line

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

cpu hotplug : add print_cpu_device

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

numaX : use cpus option multiple time if cpulist

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

bump version to 4.0-91

fix #1131: activate volume before copying efidisk

before copying the efidisk image to a storage,
we first have to activate the volume

also, add the -n flag to qemu-img convert (prevents
creation of the target volume)

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

bump version to 4.0-90

fix #1111: qm showcmd wrong escape sequence

Avoid to parse empty property string.

Check if string exists before you try to parse it. Remove unnecessary
eval.

fix manual page, bump version to 4.0-89

bump version to 4.0-88

restore: better error handling for vdisk deletion

when restoring into an existing VM, we don't want to die
half-way through because we can't delete one of the existing
volumes. instead, warn about the deletion failure, but
continue anyway. the not deleted disk is then added as
unused automatically.

forbid migration of template with local base image

forbid restore into existing template

bump version to 4.0-87

add seabios bootsplash and use it

this adds a bootsplash image in /usr/share/qemu-server
and if this file exists, use it for seabios

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

use efidisk0 for efivars

if efidisk0 is defined, use it as a efivars disk,
to permanently store efivars (such as boot options)

we check if the files exist, and act accordingly

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

copy efivars disk on create

when we create the efidisk0 over the api,
we discard the size, and create a 128kbyte vdisk
and copy it there with qemu-img convert

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

add efidisk0 to config

just a simple disk (only size, format and volid) for
efivars disk

also do not add it to command line in foreach_drive

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

bump version to 4.0-86

hostpci: bring back multifunction pass-through shortcut

disable drive-mirror when iothread is enabled

drive-mirror is not working with qemu 2.6 when iothread is enabled.

with virtio-blk : mirror is working, but block-job-completed crash the vm
with virtio-scsi : mirror hang at start.

This should be fixed in qemu 2.7

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

memory: hugepages: map numa node IDs to host and guest correctly

foreach_dimm() provides a guest numa node index, when used
in conjunction with the guest-to-host numa node topology
mapping one has to make sure that the correct host-side
indices are used.

This covers situations where the user defines a numaX with
hostnodes=Y with Y != X.

For instance:
  cores: 2
  hotplug: disk,network,cpu,memory
  hugepages: 2
  memory: 2048
  numa: 1
  numa1: memory=512,hostnodes=0,cpus=0-1,policy=bind
  numa2: memory=512,hostnodes=0,cpus=2-3,policy=bind

Both numa IDs 1 and 2 passed by foreach_dimm() have to be
mapped to host node 0.

Note that this also reverses the foreach_reverse_dimm() numa
node numbering as the current code, while walking sizes
backwards, walked the numa IDs inside each size forward,
which makes more sense. (Memory hot-unplug is still working
with this.)

hugepages: use hostnodes value as numanode for topology

also disallow more than 1 hostnode value

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

qemu-img convert : use default cache=unsafe instead writeback

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

hugepages: check if numa node exist only if hugepages are enabled

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

fix verbose qm status output

we did not check if some values were hash refs in
the verbose output.

this patch adds a recursive hash print sub and uses it

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

Fix #1057: make protection a fast-plug option

Otherwise you need to shutdown a VM to disable protection,
which is inconvenient for a few tasks such as for instance
deleting an unused disk.

add lock check for move_disk API call

this API call changes the config quite drastically, and as
such should not be possible while an operation that holds a
lock is ongoing (e.g., migration, backup, snapshot).

only activate the newly created volume

instead of the whole list.

deactivate new volumes after clone to other node

this might otherwise lead to volumes activated on the
source and target node, which is problematic for at least
LVM and Ceph.

pass datacenter.cfg's mac_prefix to random_ether_addr

bump version to 4.0-85

Fix #1051: typo: vpcus -> vcpus

Otherwise only root can change this setting.

bump version to 4.0-84

fix 1046: add non-snapshotted disks as unused

buildsys: only deb should be phony; start by removing debs

disable usb hotplug for now

we have a few problems with hotplug at the moment:
qemu may add usb hubs when adding usb devices but fails to remove them
when removing the usb device (this is a qemu bug)

also when starting a guest with a usb device we add ehci and uchi
controllers, which we cannot hot unplug

with those devices, it is impossible to live migrate the guest
to another host, meaning even if you remove all usb devices,
the migrate fails

so we deactivate usb hotplugging for now

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

remove old move disk snapshot check

this is no longer needed, if the moved disk was referenced
by a snapshot we have already died early, and the config is
locked.

fix #1040: warn early about moving a snapshotted disk

die if "delete source disk" is enabled, because a disk that
is referenced by snapshots must not be deleted.

collect errors from all local volumes

and then die with more meaningful output, instead of on the
first encountered error.

add comments and rename volhash

switch order of disk checks

to make log message more meaningful.
'storage' < 'snapshot' < 'config'

collect and log origin of found local volumes

just knowing that local disks prevent a migration is not
very helpful, so be a bit more verbose here.

new make upload target

fix dependency for upload target

update changelog

fix #146 add name to backup log.

bump version to 4.0-83

fix syntax in pve-q35.cfg

there was a wrong } instead of a ]

either it worked or nobody noticed,
but this patch fixes it

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

implement usb hotplug

this patch introduces working usb hotplugging

you can now add a usb device while a vm is running

this does not work with spice at the moment, only
with usb passthrough

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

add qemu_usb_hotplug

this adds a function for hotplugging usb devices

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

get usbdevices in vm_devices_list

since usb devices do not have their own
"query" command in qmp, we have to use
qom-list /machines/peripheral

which essentially gets a list of peripheral devices of
the vm

there we only get the usb devices

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

use the refactored usb functions

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

refactor usb functions out of QemuServer.pm

this moves most of the usb functionality into its own
module, making the QemuServer.pm smaller

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

refactor PCI into own file

to reduce QemuServer.pm size
also move the $device hash out of any function

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

add hugepages option

vm configuration
----------------
hugepages: (any|2|1024)

any: we'll try to allocate 1GB hugepage if possible, if not we use 2MB hugepage
2: we want to use 2MB hugepage
1024: we want to use 1GB hugepage. (memory need to be multiple of 1GB in this case)

optionnal host configuration for 1GB hugepages
----------------------------------------------
1GB hugepages can be allocated at boot if user want it.
hugepages need to be contiguous, so sometime it's not possible to reserve them on the fly

/etc/default/grub : GRUB_CMDLINE_LINUX_DEFAULT="quiet hugepagesz=1G hugepages=x"

Signed-off-by: Alexandre Derumier <aderumier@odiso.com>
Acked-by: Wolfgang Bumiller <w.bumiller@proxmox.com>

bump version to 4.0-82

add dependency on DBus

required for our systemd-run replacement (systemd itself
only recommends DBus and falls back to an internal IPC
mechanism if unavailable)

drop unncessary cdromhash

use foreach_drive instead of foreach_volid

foreach_volid recurses over snapshots as well, resulting in
lots of repeated checks (especially for VMs with lots of
snapshots and disks).

a potential vmstate volume must be checked explicitly,
because foreach_drive does not care about those.

fix whitespace/indent

don't repeat storage check for each volid

add @param to foreach_drive

bump version to 4.0-81

fix perl scope issues

Add parameter array to foreach_volid to use is in the functions.
correct typos.

add check for snapshots at migration

We cannot migrate snapshots on local disks, for example lvmthin snapshots.

Add LVM and LVMThin to QemuMigration

Offline migration on LVM and LVMThin are possible offline.

update changelog

fix unnecessary root check for unused disks

we needed root@pam rights to remove an unused disk
from a vm (instead of the correct Storage rights)

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

bump version to 4.0-80

split old style pipe open call