Bug 1972122

Summary: [Hyper-V][RHEL-9] Enable DRM driver for hyperv synthetic video device
Product: Red Hat Enterprise Linux 9 Reporter: Vitaly Kuznetsov <vkuznets>
Component: kernelAssignee: Vitaly Kuznetsov <vkuznets>
kernel sub component: Hyper-V QA Contact: xxiong
Status: CLOSED CURRENTRELEASE Docs Contact: Jiri Herrmann <jherrman>
Severity: unspecified    
Priority: unspecified CC: cavery, decui, gfialova, hhei, hkrzesin, huzhao, jherrman, litian, mmorsy, ribarry, vkuznets, xuli, xxiong, yacao, yuxisun
Version: 9.0Keywords: FutureFeature, TestOnly, Triaged
Target Milestone: beta   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: kernel-5.14.0-0.rc3.29.el9 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2021-12-07 21:55:02 UTC Type: Feature Request
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Vitaly Kuznetsov 2021-06-15 09:45:50 UTC
Hyper-V DRM driver made it to drm/next:

24994b91ecc5 MAINTAINERS: Add maintainer for hyperv video device
25bfa956561f drm/hyperv: Handle feature change message from device
76c56a5affeb drm/hyperv: Add DRM driver for hyperv synthetic video device

Consider enabling this driver for RHEL-9.0.

Note: DRM_HYPERV is mutually exclusive with legacy CONFIG_FB_HYPERV, we should either add 'hyperv_fb' to blocklist or disable it completely.

Comment 1 John Ferlan 2021-06-17 21:20:56 UTC
Rick - assigning directly to you to (rather than Vitaly)... 

This is a case of a bug w/ Triaged keyword being automatically set due to ITR being set and I wouldn't want to see it "lost" on virt-maint's "backlog" queue.

Comment 3 Vitaly Kuznetsov 2021-07-14 11:53:07 UTC
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/1259 enables DRM_HYPERV and disables FB_HYPERV.

Comment 4 xxiong 2021-07-16 13:22:07 UTC
hi Vitaly, 
 

we checked and found it has been included in the upstream build, but still not been enabled, could you help give an earlier build for check?  thanks



 kernel-5.14.0-0.rc1.20210714git40226a3d96ef.18.eln112.x86_64.rpm

kernel-core-5.14.0-0.rc1.20210714git40226a3d96ef.18.eln112.x86_64.rpm

kernel-modules-5.14.0-0.rc1.20210714git40226a3d96ef.18.eln112.x86_64.rpm



[rhel@vm-197-177 boot]$ cat config-5.14.0-0.rc1.20210714git40226a3d96ef.18.eln112.x86_64 | grep -i DRM_HYPER

# CONFIG_DRM_HYPERV is not set

[rhel@vm-197-177 boot]$ cat config-5.14.0-0.rc1.20210714git40226a3d96ef.18.eln112.x86_64 | grep -i config_fb_HY

CONFIG_FB_HYPERV=m

[rhel@vm-197-177 boot]$ lsmod | grep -i hyper

hid_hyperv 20480 0

hyperv_keyboard 20480 0

hyperv_fb 53248 2

hv_vmbus 344064 7 hv_balloon,hv_utils,hv_netvsc,hid_hyperv,hv_storvsc,hyperv_keyboard,hyperv_fb

[rhel@vm-197-177 boot]$ lsmod | grep -i hyperv_fb

hyperv_fb 53248 2

hv_vmbus 344064 7 hv_balloon,hv_utils,hv_netvsc,hid_hyperv,hv_storvsc,hyperv_keyboard,hyperv_fb

[rhel@vm-197-177 boot]$

Comment 5 Vitaly Kuznetsov 2021-07-16 13:39:11 UTC
Apparently, the change is not yet in kernel-5.14.0-0.rc1.20210714git40226a3d96ef.18.eln112, the merge request just got merged
yesterday (20210715). I'd expect next build to come with CONFIG_DRM_HYPERV.

Comment 6 xxiong 2021-08-10 14:07:14 UTC
Hi Vitaly,

We checked on below build, and found the drm already merged, could you help change this bug to On_QA? thanks

RHEL-9.0.0-20210729.2-x86_64-dvd1.iso

kernel-5.14.0-0.rc3.29.el9.x86_64.rpm   

[rhel@vm-198-147 ~]$ lsmod | grep drm
hyperv_drm             24576  1
drm_kms_helper        303104  3 hyperv_drm
syscopyarea            16384  1 drm_kms_helper
sysfillrect            16384  1 drm_kms_helper
sysimgblt              16384  1 drm_kms_helper
fb_sys_fops            16384  1 drm_kms_helper
cec                    53248  1 drm_kms_helper
drm                   638976  4 drm_kms_helper,hyperv_drm
hv_vmbus              143360  7 hv_balloon,hv_utils,hv_netvsc,hid_hyperv,hv_storvsc,hyperv_keyboard,hyperv_drm
[rhel@vm-198-147 ~]$ lsmod | grep  hyperv_fb
[rhel@vm-198-147 ~]$ 
[rhel@vm-198-147 ~]$ uname -r
5.14.0-0.rc3.29.el9.x86_64
[rhel@vm-198-147 ~]$

Comment 11 xxiong 2021-09-01 09:23:58 UTC
Checked on RHEL-9.0.0-20210828.2-x86_64 (5.14.0-0.rc7.54.el9.x86_64)
Now the hyperv_drm module was loaded and enabled

[root@bootp-73-199-12 rhel]# lsmod | grep hyperv_drm
hyperv_drm             24576  1
drm_kms_helper        303104  3 hyperv_drm
drm                   638976  4 drm_kms_helper,hyperv_drm
hv_vmbus              143360  7 hv_balloon,hv_utils,hv_netvsc,hid_hyperv,hv_storvsc,hyperv_keyboard,hyperv_drm

[rhel@bootp-73-199-12 ~]$ lsmod | grep hyperv_fb
[rhel@bootp-73-199-12 ~]$ 



For gen1, it blocked by bug: https://bugzilla.redhat.com/show_bug.cgi?id=1988922

and check the basic function works on gen2 (filed a bug may related to it: https://bugzilla.redhat.com/show_bug.cgi?id=1999697)

For detail test will check as: https://docs.google.com/document/d/1y6UuoC8-QPZ89s78hCkiYvkAlwF1sYEayGQ3pC17eX8/edit#

Mark this bug as verified, and will file new bug if encounter issue during testing.

Thanks.

Comment 14 xxiong 2022-04-02 07:41:00 UTC
Hi Vitaly/Jiri,

I don't find this in RHEL9.0-Beta release note, so I set requires_doc_text ?
https://access.redhat.com/documentation/en-us/red_hat_enterprise_linux/9-beta/html/9.0_release_notes/overview#overview-major-changes

Could you help check we need add this new feature/change into RHEL9.0 release note? thanks

Comment 15 Vitaly Kuznetsov 2022-04-04 07:57:11 UTC
(In reply to xxiong from comment #14)
> 
> Could you help check we need add this new feature/change into RHEL9.0
> release note?

I'm not exactly sure this deserves a release note but we can certainly state something like

"DRM driver replaces legacy framebuffer driver for Hyper-V synthetic video device"