Re: [PATCH v3 0/2] vfio: add vgpu edid support

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

 



On 2018.09.25 12:23:56 -0600, Alex Williamson wrote:
> On Fri, 21 Sep 2018 10:30:11 +0200
> Gerd Hoffmann <kraxel@xxxxxxxxxx> wrote:
> 
> > Update vfio api, adapt mbochs sample driver.
> > 
> > v3: refine vfio region layout, improve documentation.
> > v2: try vfio region approach.
> > 
> > Gerd Hoffmann (2):
> >   vfio: add edid api for display (vgpu) devices.
> >   vfio: add edid support to mbochs sample driver
> > 
> >  include/uapi/linux/vfio.h  |  50 +++++++++++++++++
> >  samples/vfio-mdev/mbochs.c | 136 ++++++++++++++++++++++++++++++++++++++-------
> >  2 files changed, 167 insertions(+), 19 deletions(-)
> > 
> 
> This looks good to me, but I'm not clear if the discussion about pixel
> clocks fully ran its course.  Zhenyu, are you satisfied GVT-g can work
> within this interface?  Any comments from NVIDIA?  Thanks,
> 

For current vfio gfx, I think this is fine. For really limited HW
that might have clock gap, we could skip edid region support on it.

I'm not sure if we'd like to extend it for multiple display support?
As it seems edid blob definition is self-contained, maybe we'll only
have some region index reference added?

thanks

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux