Re: [PATCH 2/2] qemu: snapshot: Forbid snapshots of iSCSI passthrough devices

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

 



On 07/18/2014 03:22 AM, Peter Krempa wrote:
> As with the local SCSI passthrough devicesm qemu can't support snapshots
> on those as the block ops are handled by the device. This is also true
> for iSCSI backing of the disk. Remove the check for the local block
> device and just forbid snapshot when the disk is of type 'lun'.
> 
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1073368
> ---
>  src/qemu/qemu_driver.c | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)

ACK

> 
> diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
> index 33541d3..3096688 100644
> --- a/src/qemu/qemu_driver.c
> +++ b/src/qemu/qemu_driver.c
> @@ -12434,10 +12434,7 @@ qemuDomainSnapshotPrepareDiskExternalBackingInactive(virDomainDiskDefPtr disk)
>  static int
>  qemuDomainSnapshotPrepareDiskExternalBackingActive(virDomainDiskDefPtr disk)
>  {
> -    int actualType = virStorageSourceGetActualType(disk->src);
> -
> -    if (actualType == VIR_STORAGE_TYPE_BLOCK &&
> -        disk->device == VIR_DOMAIN_DISK_DEVICE_LUN) {
> +    if (disk->device == VIR_DOMAIN_DISK_DEVICE_LUN) {
>          virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
>                         _("external active snapshots are not supported on scsi "
>                           "passthrough devices"));
> 

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital 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]