[RFC PATCH v2 0/3] drm/i915/gvt: Enable vGPU local display direct flip

[Zhang, Tina]

> -----Original Message-----
> From: Gerd Hoffmann [mailto:kraxel at redhat.com]
> Sent: Tuesday, May 28, 2019 2:30 PM
> To: Zhang, Tina <tina.zhang at intel.com>
> Cc: Tian, Kevin <kevin.tian at intel.com>; zhenyuw at linux.intel.com; Yuan,
> Hang <hang.yuan at intel.com>; ville.syrjala at linux.intel.com; Lv, Zhiyuan
> <zhiyuan.lv at intel.com>; daniel at ffwll.ch; Kondapally, Kalyan
> <kalyan.kondapally at intel.com>; intel-gvt-dev at lists.freedesktop.org; Wang,
> Zhi A <zhi.a.wang at intel.com>
> Subject: Re: [RFC PATCH v2 0/3] drm/i915/gvt: Enable vGPU local display
> direct flip
> 
>   Hi,
> 
> > > Creating a lease doesn't work here.
> > >
> > > kernel is 5.1.5.  xorg server is is 1.20.4, reporting xrandr version
> > > 1.6.  Which should be new enough.
> > This is our referenced versions:
> > Kernel: 5.2.0 (however 5.1.5 should also work)
> > Xorg: 1.20.4 reporting xrandr version 1.6
> > Libxcb: 1.13
> > Xcb-proto: 1.13
> > Xorgproto: 2018.4-4
> > Libdrm: 2.4.95
> >
> > Also, you can reference the status of drm leases support:
> > https://haagch.frickel.club/#!drmlease.md
> 
> Ah, the non-desktop property looks important.
No, non-desktop property isn't needed.

> 
> Turned output off (xrandr --output HDMI-2 --off).
> Set non-desktop property (xrandr --output HDMI-2 --set non-desktop 1).
> Updated qemu code to look for non-desktop outputs.
> 
> Still not working :-(
> 
> > When you tried this work
> > https://github.com/intel/Igvtg-
> qemu/commit/2881d9a65beb319c42b947d23eb
> > 1d260224bcddc, did you add "enable-drmlease=on" mentioned in the
> > commit message?
> 
> Using https://git.kraxel.org/cgit/qemu/log/?h=sirius/display-drm, I get this:
> 
> # qemu-system-x86_64 -display drm,enable-drmlease=on
> qemu-system-x86_64: info: drm-lease: screen is 0
> qemu-system-x86_64: info: drm-lease: root window is 0x15d
> qemu-system-x86_64: info: drm-lease: xrandr: major-opcode 140, first-error
> 147, first-event 89
> qemu-system-x86_64: info: drm-lease: xrandr: client v1.6, server v1.6
> qemu-system-x86_64: info: drm-lease: xrandr:   output 0x42: name HDMI-1,
> non-desktop 0
> qemu-system-x86_64: info: drm-lease: xrandr:   output 0x43: name HDMI-2,
> non-desktop 1
> qemu-system-x86_64: info: drm-lease: xrandr:   output 0x44: name DP-1,
> non-desktop 0
> qemu-system-x86_64: info: drm-lease: xrandr: using output 0x43
> qemu-system-x86_64: info: drm-lease: xrandr: using crtc 0x40
> qemu-system-x86_64: drm-lease: xrandr: create lease failed: BadMatch (code
> 8, op 140:45)
We used your latest code and removed the part for "non-desktop", and it can work in our environment.
It seems the used xorg doesn't include drm-lease. We use the latest xorg in https://gitlab.freedesktop.org/xorg/xserver  branch: master

BR,
Tina
> 
> Hmm.
> 
> cheers,
>   Gerd