mbox series

[v6,0/3] Fix cursor FB unpinning.

Message ID 20240522053341.137592-1-maarten.lankhorst@linux.intel.com (mailing list archive)
Headers show
Series Fix cursor FB unpinning. | expand

Message

Maarten Lankhorst May 22, 2024, 5:33 a.m. UTC
Hopefully last attempt.
Small bug in drm_vblank_work_flush_all left, fixed now hopefully.

Maarten Lankhorst (2):
  drm: Add drm_vblank_work_flush_all().
  drm/i915: Use the same vblank worker for atomic unpin

Ville Syrjälä (1):
  drm/i915: Use vblank worker to unpin old legacy cursor fb safely

 drivers/gpu/drm/drm_vblank_work.c             | 22 +++++++++++++
 .../gpu/drm/i915/display/intel_atomic_plane.c | 13 +++++++-
 .../gpu/drm/i915/display/intel_atomic_plane.h |  2 ++
 drivers/gpu/drm/i915/display/intel_crtc.c     | 31 +++++++++++++++++++
 drivers/gpu/drm/i915/display/intel_cursor.c   | 26 ++++++++++++++--
 drivers/gpu/drm/i915/display/intel_cursor.h   |  3 ++
 drivers/gpu/drm/i915/display/intel_display.c  |  3 ++
 .../drm/i915/display/intel_display_types.h    |  3 ++
 include/drm/drm_vblank_work.h                 |  2 ++
 9 files changed, 102 insertions(+), 3 deletions(-)

Comments

Shankar, Uma May 28, 2024, 8:01 a.m. UTC | #1
> -----Original Message-----
> From: Intel-xe <intel-xe-bounces@lists.freedesktop.org> On Behalf Of Maarten
> Lankhorst
> Sent: Wednesday, May 22, 2024 11:04 AM
> To: intel-gfx@lists.freedesktop.org
> Cc: intel-xe@lists.freedesktop.org; Maarten Lankhorst
> <maarten.lankhorst@linux.intel.com>
> Subject: [v6 0/3] Fix cursor FB unpinning.
> 
> Hopefully last attempt.
> Small bug in drm_vblank_work_flush_all left, fixed now hopefully.
> 
> Maarten Lankhorst (2):
>   drm: Add drm_vblank_work_flush_all().
>   drm/i915: Use the same vblank worker for atomic unpin
> 
> Ville Syrjälä (1):
>   drm/i915: Use vblank worker to unpin old legacy cursor fb safely

The series looks good now, please get the CI clearance and plan for merge.

Regards,
Uma Shankar

>  drivers/gpu/drm/drm_vblank_work.c             | 22 +++++++++++++
>  .../gpu/drm/i915/display/intel_atomic_plane.c | 13 +++++++-
> .../gpu/drm/i915/display/intel_atomic_plane.h |  2 ++
>  drivers/gpu/drm/i915/display/intel_crtc.c     | 31 +++++++++++++++++++
>  drivers/gpu/drm/i915/display/intel_cursor.c   | 26 ++++++++++++++--
>  drivers/gpu/drm/i915/display/intel_cursor.h   |  3 ++
>  drivers/gpu/drm/i915/display/intel_display.c  |  3 ++
>  .../drm/i915/display/intel_display_types.h    |  3 ++
>  include/drm/drm_vblank_work.h                 |  2 ++
>  9 files changed, 102 insertions(+), 3 deletions(-)
> 
> --
> 2.43.0
Jani Nikula May 30, 2024, 2:44 p.m. UTC | #2
On Wed, 22 May 2024, Maarten Lankhorst <maarten.lankhorst@linux.intel.com> wrote:
> Hopefully last attempt.
> Small bug in drm_vblank_work_flush_all left, fixed now hopefully.

Acked-by: Jani Nikula <jani.nikula@intel.com>

for merging via drm-misc, but drm-intel is also fine by me. No big deal
this time of the cycle when we can get the backmerge fairly quickly.

>
> Maarten Lankhorst (2):
>   drm: Add drm_vblank_work_flush_all().
>   drm/i915: Use the same vblank worker for atomic unpin
>
> Ville Syrjälä (1):
>   drm/i915: Use vblank worker to unpin old legacy cursor fb safely
>
>  drivers/gpu/drm/drm_vblank_work.c             | 22 +++++++++++++
>  .../gpu/drm/i915/display/intel_atomic_plane.c | 13 +++++++-
>  .../gpu/drm/i915/display/intel_atomic_plane.h |  2 ++
>  drivers/gpu/drm/i915/display/intel_crtc.c     | 31 +++++++++++++++++++
>  drivers/gpu/drm/i915/display/intel_cursor.c   | 26 ++++++++++++++--
>  drivers/gpu/drm/i915/display/intel_cursor.h   |  3 ++
>  drivers/gpu/drm/i915/display/intel_display.c  |  3 ++
>  .../drm/i915/display/intel_display_types.h    |  3 ++
>  include/drm/drm_vblank_work.h                 |  2 ++
>  9 files changed, 102 insertions(+), 3 deletions(-)