| Message ID | 20200813062113.11030-1-andr2000@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Fixes and improvements for Xen pvdrm | expand |
Juergen, Boris, can we please merge these via Xen Linux tree as I have collected enough Ack/R-b? The series has DRM patches, but those anyway are Xen related, so I think this should be fine from DRI point of view. Thank you, Oleksandr On 8/13/20 9:21 AM, Oleksandr Andrushchenko wrote: > From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com> > > Hello, > > This series contains an assorted set of fixes and improvements for > the Xen para-virtualized display driver and grant device driver which > I have collected over the last couple of months: > > 1. Minor fixes to grant device driver and drm/xen-front. > > 2. New format (YUYV) added to the list of the PV DRM supported formats > which allows the driver to be used in zero-copying use-cases when > a camera device is the source of the dma-bufs. > > 3. Synchronization with the latest para-virtualized protocol definition > in Xen [1]. > > 4. SGT offset is now propagated to the backend: while importing a dmabuf > it is possible that the data of the buffer is put with offset which is > indicated by the SGT offset. This is needed for some GPUs which have > non-zero offset. > > Thank you, > Oleksandr Andrushchenko > > [1] https://urldefense.com/v3/__https://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=c27a184225eab54d20435c8cab5ad0ef384dc2c0__;!!GF_29dbcQIUBPA!iAHOdk4M167VNM1AypMGVmyKJu-iqC9e5cXyu6N595Np3iyIZDnZl0MIBX3IROJSD1GSMX_GfQ$ [xenbits[.]xen[.]org] > > Changes since v1: > ================= > > 1. Removed patch which adds EDID to PV DRM as it needs more time for review: > "5. Version 2 of the Xen displif protocol adds XENDISPL_OP_GET_EDID > request which allows frontends to request EDID structure per > connector. This request is optional and if not supported by the > backend then visible area is still defined by the relevant > XenStore's "resolution" property. > If backend provides EDID with XENDISPL_OP_GET_EDID request then > its values must take precedence over the resolutions defined in > XenStore." > I will send this as a dedicated patch. > > 2. Added missing CC stable for the patches with fixes > > Oleksandr Andrushchenko (5): > xen/gntdev: Fix dmabuf import with non-zero sgt offset > drm/xen-front: Fix misused IS_ERR_OR_NULL checks > drm/xen-front: Add YUYV to supported formats > xen: Sync up with the canonical protocol definition in Xen > drm/xen-front: Pass dumb buffer data offset to the backend > > drivers/gpu/drm/xen/xen_drm_front.c | 10 +-- > drivers/gpu/drm/xen/xen_drm_front.h | 2 +- > drivers/gpu/drm/xen/xen_drm_front_conn.c | 1 + > drivers/gpu/drm/xen/xen_drm_front_gem.c | 11 +-- > drivers/gpu/drm/xen/xen_drm_front_kms.c | 2 +- > drivers/xen/gntdev-dmabuf.c | 8 +++ > include/xen/interface/io/displif.h | 91 +++++++++++++++++++++++- > 7 files changed, 111 insertions(+), 14 deletions(-) >
On 13.08.20 08:32, Oleksandr Andrushchenko wrote: > Juergen, Boris, > > can we please merge these via Xen Linux tree as I have collected enough Ack/R-b? > > The series has DRM patches, but those anyway are Xen related, so I think > > this should be fine from DRI point of view. Yes, fine with me. Juergen > > Thank you, > > Oleksandr > > On 8/13/20 9:21 AM, Oleksandr Andrushchenko wrote: >> From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com> >> >> Hello, >> >> This series contains an assorted set of fixes and improvements for >> the Xen para-virtualized display driver and grant device driver which >> I have collected over the last couple of months: >> >> 1. Minor fixes to grant device driver and drm/xen-front. >> >> 2. New format (YUYV) added to the list of the PV DRM supported formats >> which allows the driver to be used in zero-copying use-cases when >> a camera device is the source of the dma-bufs. >> >> 3. Synchronization with the latest para-virtualized protocol definition >> in Xen [1]. >> >> 4. SGT offset is now propagated to the backend: while importing a dmabuf >> it is possible that the data of the buffer is put with offset which is >> indicated by the SGT offset. This is needed for some GPUs which have >> non-zero offset. >> >> Thank you, >> Oleksandr Andrushchenko >> >> [1] https://urldefense.com/v3/__https://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=c27a184225eab54d20435c8cab5ad0ef384dc2c0__;!!GF_29dbcQIUBPA!iAHOdk4M167VNM1AypMGVmyKJu-iqC9e5cXyu6N595Np3iyIZDnZl0MIBX3IROJSD1GSMX_GfQ$ [xenbits[.]xen[.]org] >> >> Changes since v1: >> ================= >> >> 1. Removed patch which adds EDID to PV DRM as it needs more time for review: >> "5. Version 2 of the Xen displif protocol adds XENDISPL_OP_GET_EDID >> request which allows frontends to request EDID structure per >> connector. This request is optional and if not supported by the >> backend then visible area is still defined by the relevant >> XenStore's "resolution" property. >> If backend provides EDID with XENDISPL_OP_GET_EDID request then >> its values must take precedence over the resolutions defined in >> XenStore." >> I will send this as a dedicated patch. >> >> 2. Added missing CC stable for the patches with fixes >> >> Oleksandr Andrushchenko (5): >> xen/gntdev: Fix dmabuf import with non-zero sgt offset >> drm/xen-front: Fix misused IS_ERR_OR_NULL checks >> drm/xen-front: Add YUYV to supported formats >> xen: Sync up with the canonical protocol definition in Xen >> drm/xen-front: Pass dumb buffer data offset to the backend >> >> drivers/gpu/drm/xen/xen_drm_front.c | 10 +-- >> drivers/gpu/drm/xen/xen_drm_front.h | 2 +- >> drivers/gpu/drm/xen/xen_drm_front_conn.c | 1 + >> drivers/gpu/drm/xen/xen_drm_front_gem.c | 11 +-- >> drivers/gpu/drm/xen/xen_drm_front_kms.c | 2 +- >> drivers/xen/gntdev-dmabuf.c | 8 +++ >> include/xen/interface/io/displif.h | 91 +++++++++++++++++++++++- >> 7 files changed, 111 insertions(+), 14 deletions(-) >> >
On 8/13/20 10:05 AM, Jürgen Groß wrote: > On 13.08.20 08:32, Oleksandr Andrushchenko wrote: >> Juergen, Boris, >> >> can we please merge these via Xen Linux tree as I have collected enough Ack/R-b? >> >> The series has DRM patches, but those anyway are Xen related, so I think >> >> this should be fine from DRI point of view. > > Yes, fine with me. Great, thank you > > > Juergen > >> >> Thank you, >> >> Oleksandr >> >> On 8/13/20 9:21 AM, Oleksandr Andrushchenko wrote: >>> From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com> >>> >>> Hello, >>> >>> This series contains an assorted set of fixes and improvements for >>> the Xen para-virtualized display driver and grant device driver which >>> I have collected over the last couple of months: >>> >>> 1. Minor fixes to grant device driver and drm/xen-front. >>> >>> 2. New format (YUYV) added to the list of the PV DRM supported formats >>> which allows the driver to be used in zero-copying use-cases when >>> a camera device is the source of the dma-bufs. >>> >>> 3. Synchronization with the latest para-virtualized protocol definition >>> in Xen [1]. >>> >>> 4. SGT offset is now propagated to the backend: while importing a dmabuf >>> it is possible that the data of the buffer is put with offset which is >>> indicated by the SGT offset. This is needed for some GPUs which have >>> non-zero offset. >>> >>> Thank you, >>> Oleksandr Andrushchenko >>> >>> [1] https://urldefense.com/v3/__https://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=c27a184225eab54d20435c8cab5ad0ef384dc2c0__;!!GF_29dbcQIUBPA!iAHOdk4M167VNM1AypMGVmyKJu-iqC9e5cXyu6N595Np3iyIZDnZl0MIBX3IROJSD1GSMX_GfQ$ [xenbits[.]xen[.]org] >>> >>> Changes since v1: >>> ================= >>> >>> 1. Removed patch which adds EDID to PV DRM as it needs more time for review: >>> "5. Version 2 of the Xen displif protocol adds XENDISPL_OP_GET_EDID >>> request which allows frontends to request EDID structure per >>> connector. This request is optional and if not supported by the >>> backend then visible area is still defined by the relevant >>> XenStore's "resolution" property. >>> If backend provides EDID with XENDISPL_OP_GET_EDID request then >>> its values must take precedence over the resolutions defined in >>> XenStore." >>> I will send this as a dedicated patch. >>> >>> 2. Added missing CC stable for the patches with fixes >>> >>> Oleksandr Andrushchenko (5): >>> xen/gntdev: Fix dmabuf import with non-zero sgt offset >>> drm/xen-front: Fix misused IS_ERR_OR_NULL checks >>> drm/xen-front: Add YUYV to supported formats >>> xen: Sync up with the canonical protocol definition in Xen >>> drm/xen-front: Pass dumb buffer data offset to the backend >>> >>> drivers/gpu/drm/xen/xen_drm_front.c | 10 +-- >>> drivers/gpu/drm/xen/xen_drm_front.h | 2 +- >>> drivers/gpu/drm/xen/xen_drm_front_conn.c | 1 + >>> drivers/gpu/drm/xen/xen_drm_front_gem.c | 11 +-- >>> drivers/gpu/drm/xen/xen_drm_front_kms.c | 2 +- >>> drivers/xen/gntdev-dmabuf.c | 8 +++ >>> include/xen/interface/io/displif.h | 91 +++++++++++++++++++++++- >>> 7 files changed, 111 insertions(+), 14 deletions(-) >>> >> >
On 13.08.20 08:21, Oleksandr Andrushchenko wrote:
> From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com>
Series pushed to:
xen/tip.git for-linus-5.9
Juergen
On 8/13/20 6:02 PM, Jürgen Groß wrote: > On 13.08.20 08:21, Oleksandr Andrushchenko wrote: >> From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com> > > Series pushed to: > > xen/tip.git for-linus-5.9 > The top patch has strange title though: "Subject: [PATCH v2 5/5] drm/xen-front: Pass dumb buffer data offset to the backend" > > Juergen Thank you, Oleksandr
On 13.08.20 17:10, Oleksandr Andrushchenko wrote: > > On 8/13/20 6:02 PM, Jürgen Groß wrote: >> On 13.08.20 08:21, Oleksandr Andrushchenko wrote: >>> From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com> >> >> Series pushed to: >> >> xen/tip.git for-linus-5.9 >> > The top patch has strange title though: > > "Subject: [PATCH v2 5/5] drm/xen-front: Pass dumb buffer data offset to the backend" Oh, indeed. I had to repair it manually as it seems some mail system (probably on my end) mangled it a little bit. Will repair it. Juergen
On 8/13/20 6:13 PM, Jürgen Groß wrote: > On 13.08.20 17:10, Oleksandr Andrushchenko wrote: >> >> On 8/13/20 6:02 PM, Jürgen Groß wrote: >>> On 13.08.20 08:21, Oleksandr Andrushchenko wrote: >>>> From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com> >>> >>> Series pushed to: >>> >>> xen/tip.git for-linus-5.9 >>> >> The top patch has strange title though: >> >> "Subject: [PATCH v2 5/5] drm/xen-front: Pass dumb buffer data offset to the backend" > > Oh, indeed. I had to repair it manually as it seems some mail system > (probably on my end) mangled it a little bit. > > Will repair it. > Now everything is great, Thank you! > > Juergen
From: Oleksandr Andrushchenko <oleksandr_andrushchenko@epam.com> Hello, This series contains an assorted set of fixes and improvements for the Xen para-virtualized display driver and grant device driver which I have collected over the last couple of months: 1. Minor fixes to grant device driver and drm/xen-front. 2. New format (YUYV) added to the list of the PV DRM supported formats which allows the driver to be used in zero-copying use-cases when a camera device is the source of the dma-bufs. 3. Synchronization with the latest para-virtualized protocol definition in Xen [1]. 4. SGT offset is now propagated to the backend: while importing a dmabuf it is possible that the data of the buffer is put with offset which is indicated by the SGT offset. This is needed for some GPUs which have non-zero offset. Thank you, Oleksandr Andrushchenko [1] https://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=c27a184225eab54d20435c8cab5ad0ef384dc2c0 Changes since v1: ================= 1. Removed patch which adds EDID to PV DRM as it needs more time for review: "5. Version 2 of the Xen displif protocol adds XENDISPL_OP_GET_EDID request which allows frontends to request EDID structure per connector. This request is optional and if not supported by the backend then visible area is still defined by the relevant XenStore's "resolution" property. If backend provides EDID with XENDISPL_OP_GET_EDID request then its values must take precedence over the resolutions defined in XenStore." I will send this as a dedicated patch. 2. Added missing CC stable for the patches with fixes Oleksandr Andrushchenko (5): xen/gntdev: Fix dmabuf import with non-zero sgt offset drm/xen-front: Fix misused IS_ERR_OR_NULL checks drm/xen-front: Add YUYV to supported formats xen: Sync up with the canonical protocol definition in Xen drm/xen-front: Pass dumb buffer data offset to the backend drivers/gpu/drm/xen/xen_drm_front.c | 10 +-- drivers/gpu/drm/xen/xen_drm_front.h | 2 +- drivers/gpu/drm/xen/xen_drm_front_conn.c | 1 + drivers/gpu/drm/xen/xen_drm_front_gem.c | 11 +-- drivers/gpu/drm/xen/xen_drm_front_kms.c | 2 +- drivers/xen/gntdev-dmabuf.c | 8 +++ include/xen/interface/io/displif.h | 91 +++++++++++++++++++++++- 7 files changed, 111 insertions(+), 14 deletions(-)