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

2018-09-17 Thread Gerd Hoffmann
On Mon, Sep 17, 2018 at 05:15:50PM +0800, Zhenyu Wang wrote: > On 2018.09.17 10:50:33 +0200, Gerd Hoffmann wrote: > > > > +#define VFIO_DEVICE_INFO_CAP_EDID 1 > > > > + > > > > +struct vfio_device_info_edid_cap { > > > > + struct vfio_info_cap_header header; > > > > + __u32

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

2018-09-17 Thread Gerd Hoffmann
On Mon, Sep 17, 2018 at 05:15:50PM +0800, Zhenyu Wang wrote: > On 2018.09.17 10:50:33 +0200, Gerd Hoffmann wrote: > > > > +#define VFIO_DEVICE_INFO_CAP_EDID 1 > > > > + > > > > +struct vfio_device_info_edid_cap { > > > > + struct vfio_info_cap_header header; > > > > + __u32

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

2018-09-17 Thread Zhenyu Wang
On 2018.09.17 10:50:33 +0200, Gerd Hoffmann wrote: > > > +#define VFIO_DEVICE_INFO_CAP_EDID1 > > > + > > > +struct vfio_device_info_edid_cap { > > > + struct vfio_info_cap_header header; > > > + __u32 max_x; /* Max display height (zero == no limit) */ > > > + __u32 max_y; /*

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

2018-09-17 Thread Zhenyu Wang
On 2018.09.17 10:50:33 +0200, Gerd Hoffmann wrote: > > > +#define VFIO_DEVICE_INFO_CAP_EDID1 > > > + > > > +struct vfio_device_info_edid_cap { > > > + struct vfio_info_cap_header header; > > > + __u32 max_x; /* Max display height (zero == no limit) */ > > > + __u32 max_y; /*

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

2018-09-17 Thread Gerd Hoffmann
> > +#define VFIO_DEVICE_INFO_CAP_EDID 1 > > + > > +struct vfio_device_info_edid_cap { > > + struct vfio_info_cap_header header; > > + __u32 max_x; /* Max display height (zero == no limit) */ > > + __u32 max_y; /* Max display height (zero == no limit) */ > > +}; > > As current

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

2018-09-17 Thread Gerd Hoffmann
> > +#define VFIO_DEVICE_INFO_CAP_EDID 1 > > + > > +struct vfio_device_info_edid_cap { > > + struct vfio_info_cap_header header; > > + __u32 max_x; /* Max display height (zero == no limit) */ > > + __u32 max_y; /* Max display height (zero == no limit) */ > > +}; > > As current

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

2018-09-17 Thread Zhenyu Wang
On 2018.09.14 14:25:52 +0200, Gerd Hoffmann wrote: > diff --git a/include/uapi/linux/vfio.h b/include/uapi/linux/vfio.h > index 1aa7b82e81..901f279033 100644 > --- a/include/uapi/linux/vfio.h > +++ b/include/uapi/linux/vfio.h > @@ -200,12 +200,25 @@ struct vfio_device_info { > #define

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

2018-09-17 Thread Zhenyu Wang
On 2018.09.14 14:25:52 +0200, Gerd Hoffmann wrote: > diff --git a/include/uapi/linux/vfio.h b/include/uapi/linux/vfio.h > index 1aa7b82e81..901f279033 100644 > --- a/include/uapi/linux/vfio.h > +++ b/include/uapi/linux/vfio.h > @@ -200,12 +200,25 @@ struct vfio_device_info { > #define

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

2018-09-14 Thread Alex Williamson
On Fri, 14 Sep 2018 14:25:52 +0200 Gerd Hoffmann wrote: > 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 > >

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

2018-09-14 Thread Alex Williamson
On Fri, 14 Sep 2018 14:25:52 +0200 Gerd Hoffmann wrote: > 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 > >

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

2018-09-14 Thread Gerd Hoffmann
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 >

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

2018-09-14 Thread Gerd Hoffmann
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 >

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

2018-09-13 Thread Alex Williamson
On Thu, 13 Sep 2018 07:47:44 +0200 Gerd Hoffmann wrote: Some sort of commit log indicating the motivation for the change is always appreciated. > Signed-off-by: Gerd Hoffmann > --- > include/uapi/linux/vfio.h | 38 ++ > 1 file changed, 38 insertions(+) > >

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

2018-09-13 Thread Alex Williamson
On Thu, 13 Sep 2018 07:47:44 +0200 Gerd Hoffmann wrote: Some sort of commit log indicating the motivation for the change is always appreciated. > Signed-off-by: Gerd Hoffmann > --- > include/uapi/linux/vfio.h | 38 ++ > 1 file changed, 38 insertions(+) > >

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

2018-09-12 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- include/uapi/linux/vfio.h | 38 ++ 1 file changed, 38 insertions(+) diff --git a/include/uapi/linux/vfio.h b/include/uapi/linux/vfio.h index 1aa7b82e81..38b591e909 100644 --- a/include/uapi/linux/vfio.h +++

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

2018-09-12 Thread Gerd Hoffmann
Signed-off-by: Gerd Hoffmann --- include/uapi/linux/vfio.h | 38 ++ 1 file changed, 38 insertions(+) diff --git a/include/uapi/linux/vfio.h b/include/uapi/linux/vfio.h index 1aa7b82e81..38b591e909 100644 --- a/include/uapi/linux/vfio.h +++