Re: [PATCH 0/9] implicit fencing clarification

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi, Daniel!

It seems that this series misses xen-front's
"Use simple_display_pipe prepare_fb helper" change?

Thank you,
Oleksandr

On 04/05/2018 06:44 PM, Daniel Vetter wrote:
Hi all,

Somewhat motivated (but only really tangentially) by the dirtyfb
discussion with Rob and Thomas I started digging around in the various
driver implementations for implicit vs. explicit fencing.

There's definitely a huge pile of drivers which don't do any implicit
fencing at all - not sure that's good or not. And for some of the drivers
with more history I think they don't correctly overwrite implicit fencing
when explicit fencing is present. At least I've gotten lost in the mazes
before I found positive proof.

So this is just the lower hanging stuff, plus a doc patch to hopefully
clarify this all better.

Comments and review and especially in the case of the msm/vc4 patches,
also testing, very much welcome.

Thanks, Daniel

Daniel Vetter (9):
   drm/vmwgfx: Remove no-op prepare/cleanup_fb callbacks
   drm: Move simple_display_pipe prepare_fb helper into gem fb helpers
   drm/tve200: Use simple_display_pipe prepare_fb helper
   drm/pl111: Use simple_display_pipe prepare_fb helper
   drm/mxsfb: Use simple_display_pipe prepare_fb helper
   drm/atomic: better doc for implicit vs explicit fencing
   drm/gem-fb-helper: Always do implicit sync
   drm/vc4: Always obey implicit sync
   drm/msm: Always obey implicit fencing

  drivers/gpu/drm/drm_atomic.c                 |  8 +++++++
  drivers/gpu/drm/drm_gem_framebuffer_helper.c | 21 ++++++++++++++++-
  drivers/gpu/drm/msm/msm_atomic.c             |  2 +-
  drivers/gpu/drm/mxsfb/mxsfb_drv.c            |  8 +------
  drivers/gpu/drm/pl111/pl111_display.c        |  8 +------
  drivers/gpu/drm/tinydrm/core/tinydrm-pipe.c  | 17 --------------
  drivers/gpu/drm/tinydrm/ili9225.c            |  2 +-
  drivers/gpu/drm/tinydrm/mi0283qt.c           |  3 ++-
  drivers/gpu/drm/tinydrm/repaper.c            |  2 +-
  drivers/gpu/drm/tinydrm/st7586.c             |  2 +-
  drivers/gpu/drm/tinydrm/st7735r.c            |  2 +-
  drivers/gpu/drm/tve200/tve200_display.c      |  8 +------
  drivers/gpu/drm/vc4/vc4_plane.c              | 11 +++++----
  drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c          | 35 ----------------------------
  include/drm/drm_gem_framebuffer_helper.h     |  3 +++
  include/drm/drm_modeset_helper_vtables.h     |  5 +++-
  include/drm/drm_plane.h                      |  7 +++++-
  include/drm/drm_simple_kms_helper.h          |  3 +++
  include/drm/tinydrm/tinydrm.h                |  2 --
  19 files changed, 61 insertions(+), 88 deletions(-)


_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux