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

[Gerd Hoffmann]

On Wed, Oct 10, 2018 at 05:05:26PM +0800, Zhenyu Wang wrote:
> 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?

We do not have multiple display support in the vfio display api anyway.

cheers,
  Gerd