add +sep cpuflags to kvm32 && kvm64 cpu models

Need for win8 boot.

This flag was missing from rhel < 6.4 host kernel. It's ok now.
But it's also missing from kvm64 model. (It's exist in other cpu models, amd or intel).
So it's pretty safe to enable it.
If the host kernel is older, qemu filter the flag.

This also improve performance of winxp && win7 32 bits guests.

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

enable x2apic by default for kvm machines

This reduce context switch with multicore guests.

Even if the host cpu don't have x2apic, it's working because qemu have an virtual x2apic implementation for guest.
We need  in-kernel irqchip support for this, which is enable for kvm guest since qemu 1.3.
(I don't enable it if nokvm param is set)

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

spiceproxy: set delete-this-file

spiceconfig: port can't be 0

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

spice : fix proxy address

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

vm status API: return info about spice

bump version to 3.0-21

spiceproxy: remove socat, and return data to access the new spiceproxy server

kill socat if we do not get any connection within $timeout seconds

depend on socat (used for spice)

spice proxy: try to cirrectly count open socat connections

limit spice ticket length to 59 charachters

add spice proxy API

This is experimental code, spice connections are not encryped and thus insecure.
We use ticket passwords for spice auth, and do direct spice connections to
the nodes instead of using a tunnel.

add support for spice display driver (vga=qxl)

This is based on the patch from Alexandre, but we use vga=qlx instead
of introducing a new 'spice: 1' boolean flag.

correctly handle undef values when calling qemu_block_set_io_throttle

qemu-server : bridge_add : fix uninitialized value

fix : Use of uninitialized value $bridgeid in numeric lt (<) at /usr/share/perl5/PVE/QemuServer.pm line 2774.

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

code cleanups

removed trailing white space

simplify previous patch

create_disks : avoid storage scan v3

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

bump version to 3.0-20

update_vm_async: new asynchronous API

snapshot rollback: use pc-i440fx-1.4 as default

require VM.Config.HWType priviledges to modify 'machine' configuration

bump version to 3.0-19

migrate: pass --machine parameter to remote 'qm start' command

snapshot: save/restore 'machine' configuration

implement get_current_qemu_machine

API extension: add 'machine' parameter to vm_start

This allows us to overwrite the machine type at start - useful for
migration to make sure we use the same machine at the target node.

add new 'machine' configuration

fix typo

qemu_drive_mirror: correctly call block-job-cancel if block-job-complete fails

bump version to 3.0-18

move_disk: implement delete flag

rename move to move_disk

Because that API call can only move a single disk (not the whole VM).

bump version to 3.0-17

clone_disk: test if requested format is supported - else use default

move_vm: add original disk as unusedX

move_vm: make it work with PVE::QemuServer:::clone_disk

- use POST instead of PUT (this is not idempotent).
- return task ID
- use declarative permission check

add qm move (storage migration)

qm move <vmid> <disk> <storage> --format [raw|qcow2|vmdk]

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

allow to clone VM when name is not set

factor out clone_disk

As suggested by Alexandre. Also cleanup  qemu_drive_mirror (fix white space).

bump version to 3.0-16

qm rescan: do not add aliases as unused disks.

fix unused disk handling

Show unused disks even if disk is used inside snapshots. But do not allow
to remove those disks.

rescan: add check if config has really changed

fix bug 395: correctly handle unused disk with storage alias

bump version to 3.0-15

qemu_img_format : use raw for as default for other storage (nexenta, sheepdog, ...)

Seem that something have change in qemu.

iscsi qemu block driver need to use raw, or we get a segfault. (like rbd).

sheepdog works also with .raw, so we don't need anymore specific storage format, we can use raw.

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

bump version to 3.0-14

fix bug #389: avoid error if balloon is undefined

bump version to 3.0-13

fix bug #391 - restore: test if requested format is supported

bump version to 3.0-12

use add_vm_to_pool/remove_vm_from_pool from PVE::AccessControl

bump version to 3.0-11

clone disk : keep source volume params

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

clone: check is we can clone to target storage

bump version to 3.0-10

fix bug #381: use PVE::Tools::next_migrate_port()

bump version to 3.0-9

restore: do not restore template flag

A restored VM is never a template. Else we would need to create
base images after the restore, and not all storages supports base
images. Anyways, the user can easily convert the restored VM into a
template later.

bump version to 3.0-8

qemu_img_format: use 'raw' for lvm

With host_device we get the following error:

qemu-img: error while writing sector 0: Bad file descriptor

drive-mirror : die if stats are empty.

If drive have bad sectors, the block job die.
we need to die if stats are empty to avoid this:

transferred: 21440086016 bytes remaining: 34668544 bytes total: 21474754560 bytes progression: 99.84 %
Use of uninitialized value $transferred in subtraction (-) at /usr/share/perl5/PVE/QemuServer.pm line 4611.
Use of uninitialized value $total in subtraction (-) at /usr/share/perl5/PVE/QemuServer.pm line 4611.
Use of uninitialized value $transferred in multiplication (*) at /usr/share/perl5/PVE/QemuServer.pm line 4612.
Use of uninitialized value $total in division (/) at /usr/share/perl5/PVE/QemuServer.pm line 4612.
clone failed: mirroring error: Illegal division by zero at /usr/share/perl5/PVE/QemuServer.pm line 4612.

Maybe it should be improved by catching qmp events, but doesn't seem to work for now

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

set long timeout for query-block-jobs

Like we do for other block job query commands. Seems those commands
can hangs for some time.

bump version to 3.0-7

has_features: also return a list of allowed nodes

This is useful if a feature is restricted to a limited set of nodes.

has_feature: add 'copy' to enum

create template: fix permission check

This function only works if the VM already exists. But pool permissions
are automatically inherited on existing VMs, so there is no need
to explicitly check pool permissions.

clone: correctly add VM to pool

fix typo, whitespace cleanups

vm_clone : add running vm drive mirror

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

add qemu_drive_mirror

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

bump version to 3.0-6

restore: allow to overwrite existing VMs if user has VM.Backup permissions

The requirement to have VM.Allocate it non-obvious and confusing
if the VM already exists. If the user can backup, he should also be able
to restore that backup.

remane copy to clone

check has_feature copy for full clone

copy_vm: re-structure code to allow more checks before fork

fix : signal interrupt don't delete volume on full copy

Currently we  push newvolid to newvollist after qemu-img convert,
so if signal interrupt occur during qemu-img convert, the newvollist is empty and we can't free the volume

Instead, We need to push newvolid to newvollist just after volume creation

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

fix "Use of uninitialized value $target"

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

white-space cleanups

copy_vm: new option to move final VM to other node (option target)

This only works if the VM is on shared storage.

vm_copy : add optional format parameter

can be qcow2,raw,vmdk

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

copy_vm: add name and description option

copy_vm: allow to copy from snapshot

copy_vm: use exclusive lock for running VM

Because we can only run one block job to copy data.

copy_vm : add optional storage parameter

Optionnal target storage for full copy

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

don't send qmp balloon commands if vm is started with a state file

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

parse_drive : return correct format

Currently format is always empty, we need to parse file extension

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

vm_copy : fix typo in description

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

implement copy_vm

This is based on patches from Alexandre Derumier <aderumier@odiso.com>.

It just implements copy/clone for stopped VMs and Templates. Copying
running VMs or copy from snapshots is not implemented.

add qemu_img_convert

also work with snapshot as source for qcow2,rbd,sheepdog.

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

implement shared file locks

and add a first prototype for copy_vm

bump version to 3.0-4

use correct package prefix

fix assertion

move assertion so that we have all checks at the same place.

set template flag earlier

do not allow template creation if there are snapshots.

fix wrong comment