Re: [PATCH] Add qxl ram size to ABI stability check

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

 



On 12/06/13 15:01, Ján Tomko wrote:
> 55bfd02 added a 'ram' attribute for qxl video devices
> but didn't update the ABI check.
> 
> https://bugzilla.redhat.com/show_bug.cgi?id=1035123
> ---
>  src/conf/domain_conf.c | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index 0268847..b76cf26 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -13191,6 +13191,13 @@ virDomainVideoDefCheckABIStability(virDomainVideoDefPtr src,
>          return false;
>      }
>  
> +    if (src->ram != dst->ram) {
> +        virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
> +                       _("Target video card ram %u does not match source %u"),
> +                       dst->ram, src->ram);
> +        return false;
> +    }
> +
>      if (src->vram != dst->vram) {
>          virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
>                         _("Target video card vram %u does not match source %u"),
> 

ACK.

Peter

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]