Re: [PATCH v14 0/7] drm/i915/gvt: Dma-buf support for GVT-g

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Zhi and Gerd for testing v14 patch-set.

> -----Original Message-----
> From: Wang, Zhi A
> Sent: Friday, August 25, 2017 8:53 PM
> To: Gerd Hoffmann <kraxel@xxxxxxxxxx>; Zhang, Tina
> <tina.zhang@xxxxxxxxx>; zhenyuw@xxxxxxxxxxxxxxx; Lv, Zhiyuan
> <zhiyuan.lv@xxxxxxxxx>; Tian, Kevin <kevin.tian@xxxxxxxxx>;
> alex.williamson@xxxxxxxxxx; chris@xxxxxxxxxxxxxxxxxx; daniel@xxxxxxxx;
> joonas.lahtinen@xxxxxxxxxxxxxxx; kwankhede@xxxxxxxxxx
> Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; intel-gvt-dev@xxxxxxxxxxxxxxxxxxxxx
> Subject: RE: [PATCH v14 0/7] drm/i915/gvt: Dma-buf support for GVT-g
> 
> Hi Gerd:
>     Thanks for the testing. We will find out the problem and refresh the whole
> patch series.
> 
> Thanks,
> Zhi.
> 
> -----Original Message-----
> From: Gerd Hoffmann [mailto:kraxel@xxxxxxxxxx]
> Sent: Friday, August 25, 2017 2:47 PM
> To: Zhang, Tina <tina.zhang@xxxxxxxxx>; zhenyuw@xxxxxxxxxxxxxxx; Lv, Zhiyuan
> <zhiyuan.lv@xxxxxxxxx>; Wang, Zhi A <zhi.a.wang@xxxxxxxxx>; Tian, Kevin
> <kevin.tian@xxxxxxxxx>; alex.williamson@xxxxxxxxxx; chris@chris-
> wilson.co.uk; daniel@xxxxxxxx; joonas.lahtinen@xxxxxxxxxxxxxxx;
> kwankhede@xxxxxxxxxx
> Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx; intel-gvt-dev@xxxxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH v14 0/7] drm/i915/gvt: Dma-buf support for GVT-g
> 
> On Fri, 2017-08-18 at 18:21 +0800, Tina Zhang wrote:
> > v13->v14:
> > 1) add PROBE, DMABUF and REGION flags. (Alex)
> > 2) return -ENXIO when gem proxy object is banned by ioctl.
> >    (Chris) (Daniel)
> > 3) add some details about the float pixel format. (Daniel)
> > 4) add F suffix to the defined name. (Daniel)
> > 5) refine pixel format table. (Zhenyu)
> 
> Ok, patch series applies cleanly to 4.13-rc6.  The new flags are working fine.
> 
> However, I see VFIO_DEVICE_QUERY_GFX_PLANE failures which I think should
> not be there.  When the guest didn't define a plane yet I get "No such device"
> errors instead of a plane_info struct with fields (drm_format, width, height, size)
> set to zero.  I also see "Bad address" errors now and then with no obvious cause.
The idea is to return "No such device" error with fields set to zero.
There are two cases, in which the "No such device" error is returned: one is the guest IGD driver
has not finished the initialization and the other is the plane is disabled by guest IGD driver.
If we prefer to return success in these two situations with fields set to zero, I can add it in the
next version.

We didn't meet the "Bad address" errors before. I will try your qemu repo to see whether I can
meet the issue. Thanks.
 
> Cursor support isn't working too.
It seems there is some issue in i915 of version 4.13-rc6, which blocks the cursor on native platform.
I just tried the rc7 (the latest staging), there is on such issue.
Thanks.

Tina
> 
> I'm testing with "-display egl-headless -spice gl=off,port=...".  In that
> configuration qemu will import the dma-bufs as textures and reads them using
> ReadPixels for display.
> 
> qemu branch: https://www.kraxel.org/cgit/qemu/log/?h=work/intel-vgpu
> 
> The qemu branch has support for both dmabufs and regions.  The region- based
> display code is totaly untested though.
> 
> cheers,
>   Gerd

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux