Re: [PATCH 1/2] vfio: add edid api for display (vgpu) devices.

From: Gerd Hoffmann
Date: Fri Sep 14 2018 - 08:26:01 EST


Hi,

> Another possible implementation would be via a vfio region, we already
> support device specific regions via capabilities with vfio_region_info,
> so we could have an edid region which could handle both input and
> output using a defined structure and protocol within the region. With
> an edid blob of up to 512 bytes now, that means the vendor driver would
> need to buffer writes to that section of the region until some sort of
> activation, possibly using another "register" within the field to
> trigger the link state and only processing the edid blob on link down
> to link up transition.

Hmm, using a virtual register space makes things more complicated for no
good reason. This is a configuration interface for qemu, not something
we expose to the guest. So, I'm not a fan ...

New revision of the vfio.h patch attached below, how does that look
like? If it is ok I'll go continue with that next week (more verbose
docs, update qemu code and test, ...)

cheers,
Gerd