Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1712947

Summary: qemu-kvm build is broken due to spice_qxl_set_max_monitors being deprecated
Product: Red Hat Enterprise Linux Advanced Virtualization Reporter: Danilo de Paula <ddepaula>
Component: qemu-kvmAssignee: Ademar Reis <areis>
Status: CLOSED NOTABUG QA Contact: Virtualization Bugs <virt-bugs>
Severity: unspecified Docs Contact:
Priority: high    
Version: 8.1CC: rbalakri, ribarry, virt-bugs, virt-maint
Target Milestone: rc   
Target Release: 8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: qemu-4.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1712946 Environment:
Last Closed: 2019-05-22 19:57:04 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1712946    
Bug Blocks:    

Description Danilo de Paula 2019-05-22 14:49:20 UTC 
+++ This bug was initially created as a clone of Bug #1712946 +++

Trying to build qemu-kvm and it failed with the following message:
hw/display/qxl.c: In function 'qxl_spice_monitors_config_async':
hw/display/qxl.c:280:13: error: 'spice_qxl_set_max_monitors' is deprecated [-Werror=deprecated-declarations]
             spice_qxl_set_max_monitors(&qxl->ssd.qxl, qxl->max_outputs);
             ^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /usr/include/spice-server/spice.h:26,
                 from /builddir/build/BUILD/qemu-2.12.0/include/ui/qemu-spice.h:25,
                 from hw/display/qxl.h:11,
                 from hw/display/qxl.c:33:
/usr/include/spice-server/spice-qxl.h:105:6: note: declared here
 void spice_qxl_set_max_monitors(QXLInstance *instance,
      ^~~~~~~~~~~~~~~~~~~~~~~~~~

I didn't look into this very deeply, but looks like spice deprecated one of the functions we use in RHEL-8.1.0, making qemu-kvm for RHEL-8.1.0 and RHEL-AV-8.1.0 unbuildable.

Good news is that this has been fixed upstream, but we need to apply this ASAP.


commit be812c0ab7d5ab741d0d87387a75a0e8bb6461e7
Author: Lukáš Hrázký <lhrazky>
Date:   Fri Feb 15 16:09:19 2019 +0100
spice: set device address and device display ID in QXL interface
Comment 1 Ademar Reis 2019-05-22 19:30:18 UTC 
(In reply to Danilo Cesar de Paula from comment #0)
> +++ This bug was initially created as a clone of Bug #1712946 +++
> 
> Trying to build qemu-kvm and it failed with the following message:
> hw/display/qxl.c: In function 'qxl_spice_monitors_config_async':
> hw/display/qxl.c:280:13: error: 'spice_qxl_set_max_monitors' is deprecated
> [-Werror=deprecated-declarations]
>              spice_qxl_set_max_monitors(&qxl->ssd.qxl, qxl->max_outputs);
>              ^~~~~~~~~~~~~~~~~~~~~~~~~~
> In file included from /usr/include/spice-server/spice.h:26,
>                  from
> /builddir/build/BUILD/qemu-2.12.0/include/ui/qemu-spice.h:25,
>                  from hw/display/qxl.h:11,
>                  from hw/display/qxl.c:33:
> /usr/include/spice-server/spice-qxl.h:105:6: note: declared here
>  void spice_qxl_set_max_monitors(QXLInstance *instance,
>       ^~~~~~~~~~~~~~~~~~~~~~~~~~
> 
> I didn't look into this very deeply, but looks like spice deprecated one of
> the functions we use in RHEL-8.1.0, making qemu-kvm for RHEL-8.1.0 and
> RHEL-AV-8.1.0 unbuildable.
> 
> Good news is that this has been fixed upstream, but we need to apply this
> ASAP.
> 
> 
> commit be812c0ab7d5ab741d0d87387a75a0e8bb6461e7
> Author: Lukáš Hrázký <lhrazky>
> Date:   Fri Feb 15 16:09:19 2019 +0100
> spice: set device address and device display ID in QXL interface

This patch is in QEMU-4.0, so marking it POST.
Comment 2 Danilo de Paula 2019-05-22 19:57:04 UTC 
This report is misleading.

I cloned imagining this would happen in RHEL-AV. It doesn't.
I will close this.