[PATCH] drm: Don't prepare or cleanup unchanging frame buffers [v2]

2016-08-08 Thread Daniel Vetter
On Sat, Aug 06, 2016 at 10:26:09PM -0700, Keith Packard wrote: > Daniel Vetter writes: > > > Hm, I can't see v1 anywhere, but I think it'd be better. You can't store > > any transient state related to the current update in struct drm_plane. In > > this case the cleanup_buffers from a previous upd

[PATCH] drm: Don't prepare or cleanup unchanging frame buffers [v2]

2016-08-08 Thread Keith Packard
Daniel Vetter writes: > Rebased onto 4.8 while applying, which did simplify the patch a lot (4.8 > is already using for_each_plane_in_state, but slightly differently). Your rebase looks great, thanks! -- -keith -- next part -- A non-text attachment was scrubbed... Name:

[PATCH] drm: Don't prepare or cleanup unchanging frame buffers [v2]

2016-08-06 Thread Keith Packard
Daniel Vetter writes: > Hm, I can't see v1 anywhere, but I think it'd be better. You can't store > any transient state related to the current update in struct drm_plane. In > this case the cleanup_buffers from a previous update might overlap (for > nonblocking atomic commits) with the prepare_pla

[PATCH] drm: Don't prepare or cleanup unchanging frame buffers [v2]

2016-08-02 Thread Daniel Vetter
On Sun, Jul 31, 2016 at 01:08:54AM -0700, Keith Packard wrote: > When reconfiguring a plane position (as in moving the cursor), the > frame buffer for the cursor isn't changing, so don't call the prepare > or cleanup driver functions. > > This avoids making cursor position updates block on all pen

[PATCH] drm: Don't prepare or cleanup unchanging frame buffers [v2]

2016-07-31 Thread kbuild test robot
Hi, [auto build test WARNING on v4.7-rc7] [cannot apply to drm/drm-next next-20160729] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Keith-Packard/drm-Don-t-prepare-or-cleanup-unchanging-frame-

[PATCH] drm: Don't prepare or cleanup unchanging frame buffers [v2]

2016-07-31 Thread Keith Packard
When reconfiguring a plane position (as in moving the cursor), the frame buffer for the cursor isn't changing, so don't call the prepare or cleanup driver functions. This avoids making cursor position updates block on all pending rendering. v2: Track which planes have been prepared to know which