Bug 1723580 - [downstream clone - 4.3.5] [RFE] Support console VNC for mediated devices
Summary: [downstream clone - 4.3.5] [RFE] Support console VNC for mediated devices
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-engine
Version: 4.4.0
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ovirt-4.3.5
: 4.3.5
Assignee: Tomasz Barański
QA Contact: Nisim Simsolo
URL:
Whiteboard:
Depends On: 1654889
Blocks: 1633623
TreeView+ depends on / blocked
 
Reported: 2019-06-24 20:39 UTC by RHV bug bot
Modified: 2019-08-12 11:53 UTC (History)
7 users (show)

Fixed In Version: ovirt-engine-4.3.5.4
Doc Type: Enhancement
Doc Text:
Clone Of: 1654889
Environment:
Last Closed: 2019-08-12 11:53:28 UTC
oVirt Team: Virt
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2019:2431 0 None None None 2019-08-12 11:53:40 UTC
oVirt gerrit 98553 0 'None' MERGED core: Add display="on" for mdevs, use nodisplay to override 2021-01-07 15:49:27 UTC
oVirt gerrit 98791 0 'None' MERGED core: Add display="on" for mdevs, use nodisplay to override 2021-01-07 15:49:29 UTC
oVirt gerrit 99811 0 'None' MERGED core: Set type=none for video element of mdevices 2021-01-07 15:49:29 UTC
oVirt gerrit 99962 0 'None' MERGED core: Set type=none for video element of mdevices 2021-01-07 15:50:06 UTC
oVirt gerrit 100525 0 'None' MERGED core: Video element is required in libvirt XML 2021-01-07 15:50:05 UTC
oVirt gerrit 101610 0 'None' MERGED core: Video element is required in libvirt XML 2021-01-07 15:50:06 UTC

Description RHV bug bot 2019-06-24 20:39:30 UTC
+++ This bug is a downstream clone. The original bug is: +++
+++   bug 1654889 +++
======================================================================

Description of problem:
Platform supports direct VNC passthrough for vGPU -- see https://bugzilla.redhat.com/show_bug.cgi?id=1475770

In order to support this in RHV, a new display type must be added which users can configure after drivers are installed in the guest, which removes QXL and allows a direct mapping to the vGPU

qemu-cmdline argument should be enough.

(Originally by Ryan Barry)

Comment 4 RHV bug bot 2019-06-24 20:39:37 UTC
WARN: Bug status wasn't changed from MODIFIED to ON_QA due to the following reason:

[Found non-acked flags: '{}', ]

For more info please contact: rhv-devops: Bug status wasn't changed from MODIFIED to ON_QA due to the following reason:

[Found non-acked flags: '{}', ]

For more info please contact: rhv-devops

(Originally by rhv-bugzilla-bot)

Comment 9 RHV bug bot 2019-06-24 20:39:46 UTC
Reassigned. 
- VNC is showing display only on the first run of the VM. after powering off and running the VM again, the VNC display freezes before the VM completely booted.
- Running diff of VM qemu process shows that when there is no VNC display, there is  cirrus-vga device that was added (please see diff.png file attached).
and also inside the VM there is Cirrus VGA controller:
00:05.0 VGA compatible controller [0300]: Cirrus Logic GD 5446 [1013:00b8]
	Subsystem: Red Hat, Inc. QEMU Virtual Machine [1af4:1100]
	Kernel driver in use: cirrus
	Kernel modules: cirrus
- Possible workaround for this issue, it to edit VM -> Console and change video type from QXL to VGA or vice versa.

qemu proccess files for diff, vdsm.log and engine.log attached.

(Originally by Nisim Simsolo)

Comment 10 RHV bug bot 2019-06-24 20:39:47 UTC
Created attachment 1572507 [details]
screenshot of qemu proccesses diff  (the failed one is with cirrus)

(Originally by Nisim Simsolo)

Comment 11 RHV bug bot 2019-06-24 20:39:49 UTC
Created attachment 1572521 [details]
vdsm.log.xz

(Originally by Nisim Simsolo)

Comment 12 RHV bug bot 2019-06-24 20:39:51 UTC
Created attachment 1572522 [details]
engine.log.xz

(Originally by Nisim Simsolo)

Comment 13 RHV bug bot 2019-06-24 20:39:52 UTC
Created attachment 1572523 [details]
qemu process when VNC is working

(Originally by Nisim Simsolo)

Comment 14 RHV bug bot 2019-06-24 20:39:54 UTC
Created attachment 1572524 [details]
qemu process when VNC failed to display VM

(Originally by Nisim Simsolo)

Comment 15 RHV bug bot 2019-06-24 20:39:55 UTC
Another thing, when using virt-viewer or virt-manager the mouse cursor is invisible although mouse is present (windows are highlighted and mouse clicks are working).
using TigerVNC mouse cursor is visible.
Please see RHEL7 bug: https://bugzilla.redhat.com/show_bug.cgi?id=1620204

(Originally by Nisim Simsolo)

Comment 16 RHV bug bot 2019-06-24 20:39:57 UTC
(In reply to Nisim Simsolo from comment #9)
> Reassigned. 
> - Running diff of VM qemu process shows that when there is no VNC display,
> there is  cirrus-vga device that was added (please see diff.png file
> attached).
> and also inside the VM there is Cirrus VGA controller:
> 00:05.0 VGA compatible controller [0300]: Cirrus Logic GD 5446 [1013:00b8]
> 	Subsystem: Red Hat, Inc. QEMU Virtual Machine [1af4:1100]
> 	Kernel driver in use: cirrus
> 	Kernel modules: cirrus
> - Possible workaround for this issue, it to edit VM -> Console and change
> video type from QXL to VGA or vice versa.

Found the reason behind it. On the first run, the XML contains no <video> tag but contains <graphics> element for VNC. Libvirt's doc says, that:

    For backwards compatibility, if no video is set but there is a graphics in domain xml, then libvirt will add a default video according to the guest type.
    For a guest of type "kvm", the default video is: type with value "cirrus", vram with value "16384" and heads with value "1".

I can add a quick fix (detect that we have graphics but no video, then add <video> with type none if appropriate) if we're pressed for time, but I'd rather find the root cause (why there isn't a video element at all).

(Originally by Tomasz Baranski)

Comment 20 Sandro Bonazzola 2019-07-10 10:04:44 UTC
$ git tag --contains f5b40cc1d3ecbd85a17f527d843466cdccadf85b
ovirt-engine-4.3.5.4

$ git tag --contains d3de5f1d155a095e46433a3c7e04c91c28b2225e
ovirt-engine-4.3.3

$ git tag --contains d02963e3725158b3b0e29260e57a8d20dc538df7
ovirt-engine-4.3.4.1

marking this as fixd in 4.3.5.4

Comment 21 Nisim Simsolo 2019-07-17 15:32:49 UTC
Verified: 
rhvm-4.3.5.4-0.1.el7
vdsm-4.40.0-472.git7714057.el7.x86_64
libvirt-4.5.0-10.el7_6.12.x86_64
qemu-kvm-ev-2.12.0-18.el7_6.5.1.x86_64
2 X Nvidia Tesla M60 under the same host, drivers:  GRID 9.0 GA

Verification scenario:
Polarion test case added to external trackers.

Comment 23 errata-xmlrpc 2019-08-12 11:53:28 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2019:2431


Note You need to log in before you can comment on or make changes to this bug.