[libvirt] [PATCH v4 0/4] Add startupPolicy attribute support for hard disks

Guannan Ren gren at redhat.com
Tue Jul 2 09:35:55 UTC 2013


v3: https://www.redhat.com/archives/libvir-list/2013-June/thread.html

v3 to v4: Rebase

v2 to v3: Not only check disk source, startupPolicy should work if any
          backing file is missing. The commit 039a3283 break the limition
          of contiguous device boot orders, so I remove my previous patch
          about it.

v1 to v2: Added relax schema for disk of block and dir type
          Removed original patch 3/5.

The set of patches is trying to add 'startupPolicy' attribute support
to the source element of hard disks. Policy levels are using the
mandatory, requisite, optional levels as originally documented.

For the 'optional' policy, there is a little difference from CDROM and
Floppy which only drop its source path, for disks, if missing, the
checking function will drop their definitions, because qemu doesn't
allow missing source path for hard disk.

Guannan Ren
 [PATCH v3 1/4] conf: add startupPolicy attribute for harddisk
 [PATCH v3 2/4] storage: report error rather than warning if backing files doesn't exist
 [PATCH v3 3/4] qemu: drop disk with 'optional' startupPolicy if it is not present
 [PATCH v3 4/4] security: restore DAC for every disk file in disk chain

 docs/schemas/domaincommon.rng |   6 ++++++
 include/libvirt/libvirt.h.in  |   1 +
 src/conf/domain_conf.c        |  20 +++++++++++++-------
 src/qemu/qemu_domain.c        | 119 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++----------------------------------------------
 src/qemu/qemu_process.c       |   7 -------
 src/security/security_dac.c   |  15 ++++++++++++++-
 src/util/virstoragefile.c     |  23 +++++++++++++++--------
 tests/virstoragetest.c        |  16 ++++++++--------
 8 files changed, 130 insertions(+), 77 deletions(-)




More information about the libvir-list mailing list