Re: [PATCH v3 2/3] storage: optimize calls to virStorageFileInit and friends

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Dec 05, 2016 at 18:55:18 +0530, Prasanna Kumar Kalever wrote:
> Currently, each among virStorageFileGetMetadataRecurse,
> qemuSecurityChownCallback, qemuDomainSnapshotPrepareDiskExternal and
> qemuDomainSnapshotCreateSingleDiskActive makes calls to virStorageFileInit
> and friends for simple operations like stat, read headers, chown and etc.
> 
> This patch optimize/unify calls to virStorageFileInit and virStorageFileDeinit.
> 
> Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@xxxxxxxxxx>
> ---
>  src/qemu/qemu_domain.c       |  2 +-
>  src/qemu/qemu_domain.h       |  5 +++++
>  src/qemu/qemu_driver.c       | 40 +++++++++++++++++++++++++++++----------
>  src/qemu/qemu_process.c      | 45 ++++++++++++++++++++++++++++++++++++++++++++
>  src/qemu/qemu_process.h      |  4 ++++
>  src/storage/storage_driver.c | 11 ++++++++---
>  6 files changed, 93 insertions(+), 14 deletions(-)

After this patch qemuxml2argvtest gets stuck. This is because you call
the code accessing the volumes in the test code which attempts to call
gluster. This is not acceptable in tests. There should be some existing
mechanisms which skips certain parts of the process in tests.

See how VIR_QEMU_PROCESS_START_PRETEND is used.

Peter

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux