[libvirt] [PATCH] Add qxl ram size to ABI stability check

Peter Krempa pkrempa at redhat.com
Fri Dec 6 14:10:17 UTC 2013


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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20131206/7b30d1b5/attachment-0001.sig>


More information about the libvir-list mailing list