use old netdevice bios files for older machine types

bump version to 4.0-31

migrate : add nocheck for resume

Users have reported resume bug when HA is used.

They seem to have a little race (bench show >0s < 1s) between the vm conf file move on source node and replication to,
and resume on target node.

I don't known why this is only with HA, maybe this occur will standard migration too.

Anyway, we don't need to read the vm config file to resume the vm on target host,
as we are sure that the vm is migrated, and config file move action is correct in the cluster.

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

bump version to 4.0-30

convert qmrestore into a PVE::CLI class

and install bash completion helpers.

create: add better check for unused IDs

bump version to 4.0-29

support serial numbers and models for disks

Note that the model is only supported for IDE harddisks via
the -device option, whereas the serial is part of the -drive
option.

Closes #153

bump version to 4.0-28

disable kvm_steal_time

It's currently buggy with live migration

https://bugs.launchpad.net/qemu/+bug/1494350
Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

bump version to 4.0-27

boot order permissions change

to change to boot order it is needed to assign a user the
VM.Config.Disk privilege set with the add/modify/delete disk
permissions -> now the patch allows the boot order to be (re)defined
with the VM.Config.Options set - the modify any other VM configuration

bump version to 4.0-26

migration: disable compress

it's already disable by default,
but we want to be sure if it's change in later release

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

enable xbzrle

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

fix kvm version parser for CVE stable releases, bump version to 4.0-25

bump version to 4.0-24

VM protection mode

[PATCH v2] changes:

- changed call also for remove VM

pci passthough : make vfio default

and remove old legacy code.
(deprecated and disabled by default since kernel 4.2)

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

code cleanups

VM protection mode

changes:

- added common check_protection subroutine
- disk removal prevention
- restore over protected VM prevention
- changed man page message

add qm.bash-completion to .gitignore

fix error message: allocate to much v-CPUs.

bump version to 4.0-23

avoid warning about undefined value

bump version to 4.0-22

add possibility to restore backup on rbd in krbd mode

add krbd support to online snapshot

bump version to 4.0-21

fix install path

remove generated qm.bash-completion on make clean

improve bash completion

implement a few bash completion helpers

covert qm into a PVE::CLI class, enable bash completions

VM protection mode added

used to prevent an unintended virtual machine remove operation

v3 changes:

- changed man page message
- removed protection parameter (where not needed)

fix move_disk on RBD

activating the disk when use RBD is necessary to have the given path.

cleanup previous patch

HA resources check before destroying vm

bump version to 4.0-20

fix: start kvm with os type other

this check is necessary, because we do not set ostype in qemu config when type is other

Seem that x2apic is now enabled by default when kvm is used

since this commit
http://git.qemu.org/?p=qemu.git;a=commit;h=ef02ef5f4536dba090b12360a6c862ef0e57e3bc

So we just need to disable it for solaris

fix 'uninitialized value in concat' due to unnamed VM

This is an minor fix which let's you start the spice console on an
unnamed VM without getting the 'Use of uninitialized value in
concatenation' error.
Also changes the trailing comma from the $conf definition lineto an
semicolon.

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

bump version to 4.0-19

clone vm firewall config file

Signed-off-by: Alen Grizonic <a.grizonic@proxmox.com>

bump version to 4.0-18

fix bug #688: if vm is not owner of this disk remove from config

fix bug #688: prevent copy unused disks in the config.

bump version to 4.0-17

fix bug #603: vmid.fw file not deleted

Signed-off-by: Alen Grizonic <a.grizonic@proxmox.com>

fix bug #517: improve error message

Get a understandble message,
if someone try to increase a qcow2 image where one or more snapshots reside

adapt /config and /pending API calls to force-delete

bump version to 4.0-16

remove vm access permissions after destroy

removed duplicated code which was used in
PVE::AccessControl -> now in PVE::API2::Qemu

Signed-off-by: Alen Grizonic <a.grizonic@proxmox.com>

merge delete_drive into try_deallocate_drive

It used to be private and is only used once.

pending-delete: remember force-deletes

The -force flag didn't have any effect since the pending
changes didn't carry over the the flag.
Now forced deletes have an exclamation mark prepended to the
option name.

correctly handle empty description in pending section

So that we can delete descriptions with

 qm set <vmid> --descr ''

bump version to 4.0-15

Fixed wrong UUID in Qemu VZDump backup

As the format of the uuid which the qemu monitor returned changed,
it is here adapted to fix errors with not matching uuids.

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

add memory_unplug support V2

qemu 2.4 feature

changelog: rebase on last git

Note that currently linux guest don't support unplug of dimm when it'sused by kernel memory.
They are some tunning to do with memory zone movable.
http://events.linuxfoundation.org/sites/events/files/lcjp13_chen.pdf

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

bump version to 4.0-14

fixed bug 662, wrong subroutine for parsing startup order

Changed from old, now missing, subroutine parse_startup() to new
pve_parse_startup_order() in qemu-server and pve-manager

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

cpuflags : don't enforce with tcg mode

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

cpuflags : remove enforce for cpumodel=host

Currently enforce with cpumodel=host on amd cpus don't work,
because amd cpus have unsupported flags in qemu.

This is a protection, and this is good.

but cpumodel host should be never use by users for production (only for testing).
For production and stability, users need to choose a true cpu model which filter
the supported cpuflags by qemu.

So I think we can remove the enforce for host model as for testing it's ok.

warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 0]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 1]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 2]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 3]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 4]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 5]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 6]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 7]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 8]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 9]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 12]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 13]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 14]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 15]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 16]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 17]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 23]
warning: host doesn't support requested feature: CPUID.80000001H:EDX [bit 24]
Signed-off-by: Alexandre Derumier <aderumier@odiso.com>

cpuflags : remove -rdtscp for Opteron cpu models

rdtscp is not supported by qemu and with enforce it's not starting
warning: host doesn't support requested feature: CPUID.80000001H:EDX.rdtscp [bit 27]

from to qemu wiki
http://wiki.qemu.org/Features/CPUModels#Disabling_features_that_were_always_disabled_on_KVM
"Fact: currently libvirt runs CPU models having rdtscp without the "enforce" flag, and rdtscp is silently disabled
Consequence: libvirt SHOULD use something like "-cpu Opteron_G5,-rdtscp",
especially when it starts using (or emulating) enforce mode
This will require a solution on libvirt side. QEMU will just provide the mechanisms to report CPU model information
and check what the host and QEMU supports, but the decision to disable rdtscp to be able
to run Opteron_G[2345] needs to be taken by libvirt."

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

vzdump : abord backup if disk have iothread enabled.

Currently backup don't work with iothread feature, and crash qemu

For now, abord the backup if one of the vm drives have iothread enabled until backup code is fixed.

Upstream qemu backup cde already support iothread.
http://git.qemu.org/?p=qemu.git;a=commit;h=761731b1805f6ef64eb615e5b82a0801db3cde78

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

bump version to 4.0-13

qemu-server : drive-mirror : allow to interrupts at the scanning bitmap phase

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

qemuserver : vm_status : add extended stats (disks, nics, memory) V3

Add extended stats results for each nics,disks and memory on full stats mode only.

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

bump version to 4.0-12

remove outdated host_device format

Newer qemu does not need it.

https://lists.nongnu.org/archive/html/qemu-devel/2011-06/msg02790.html

parse_drive: do not overwrite configured format

Instead, we use qemu_img_format() to query the format when we need it.
We also pass the format to qemu in print_drive_full to avoid qemu warnings.

fix aio O_DIRECT check for cdrom drives

Only non-cdrom drives default to cache=none, so the check
for whether to default to aio=native needs to take the same
condition into account.
I combined them close together to make their relation more
visible.

cpulimit: use number instead of integer

implement cpuunit with cgroups

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

bump version to 4.0-11

use noawait triggers for pve-api-updates

bump version to 4.0-10

code cleanup

remove old openvz fairscheduler code

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

implement hotplug for cpuunits

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

implement cgroups through systemd-run

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

localhost instead of 127.0.0.1 makes ipv6 life easier

migration: put the source address in brackets

Always adding brackets around the address works. They're required for
ipv6 and qemu also accepts them for ipv4 and hostnames.

pass port family to next_*_port() calls

bump version to 4.0-9

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>

Switch from netcat-traditional to netcat6

netcat6 supports both ipv4 and ipv6, contrary to netcat-traditional.
Additionally rather than using the alternatives-managed /bin/nc path,
/bin/nc6 is now used which always points to the netcat6 implementation,
allowing us to drop the netcat-openbsd conflict entry.

remove running from Storage and check it in QemuServer

It is better to check if a VM is running in QemuServer then in Storage.
for the Storage there is no difference if it is running or not.

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

bump version to 4.0-8

trigger pve-api-updates event

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 4.0-7

use new standard option 'pve-startup-order'

bump version to 4.0-6

fix ha resource names

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>

implement virtio-scsi multiqueues

Signed-off-by: Alexandre Derumier <aderumier@odiso.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>

forbid hot-unplug of virtioscsi + iothread drive

qemu (<=2.3) currently segfault on drive_del

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