Re: [PATCH] libxl: add discard support to libxl_device_disk

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

 



On Fri, May 30, Jim Fehlig wrote:

> Olaf Hering wrote:
> > +    if (discard == VIR_DOMAIN_DISK_DISCARD_DEFAULT)
> > +        return;
> > +    virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
> > +                   _("This version of libxenlight does not support "
> > +                     "discard= option passing"));
> An error would be reported here, but the overall libxlMakeDisk operation
> would succeed right?  Shouldn't it fail if the user requests discard but
> it is not supported?

How would libvirt know if the underlying backing store supports discard?
It would need to maintain a list of supported targets.
If it remains just some sort of warning a given .xml file can be shared
among different libvirt versions. After all, discard is not operation
critical.

Olaf

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