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

[Zhenyu Wang]

On 2018.09.25 12:23:56 -0600, Alex Williamson wrote:
> On Fri, 21 Sep 2018 10:30:11 +0200
> Gerd Hoffmann <kraxel at redhat.com> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/intel-gvt-dev/attachments/20181010/61dda6de/attachment.sig>