Re: [PATCH] drm/amd/display: Don't cleanup new cursor fb in fast cursor update

2018-12-14 Thread Kazlauskas, Nicholas
On 12/14/18 10:19 AM, Kazlauskas, Nicholas wrote: > On 12/14/18 10:01 AM, Michel Dänzer wrote: >> On 2018-12-14 3:48 p.m., Nicholas Kazlauskas wrote: >>> [Why] >>> The behavior of dm_plane_helper_cleanup_fb differs depending on >>> whether the commit was asynchronous or not. When it's called from >

Re: [PATCH] drm/amd/display: Don't cleanup new cursor fb in fast cursor update

2018-12-14 Thread Kazlauskas, Nicholas
On 12/14/18 10:01 AM, Michel Dänzer wrote: > On 2018-12-14 3:48 p.m., Nicholas Kazlauskas wrote: >> [Why] >> The behavior of dm_plane_helper_cleanup_fb differs depending on >> whether the commit was asynchronous or not. When it's called from >> amdgpu_dm_atomic_commit_tail during a typical atomic c

Re: [PATCH] drm/amd/display: Don't cleanup new cursor fb in fast cursor update

2018-12-14 Thread Michel Dänzer
On 2018-12-14 3:48 p.m., Nicholas Kazlauskas wrote: > [Why] > The behavior of dm_plane_helper_cleanup_fb differs depending on > whether the commit was asynchronous or not. When it's called from > amdgpu_dm_atomic_commit_tail during a typical atomic commit the > plane state has been swapped so it ca

[PATCH] drm/amd/display: Don't cleanup new cursor fb in fast cursor update

2018-12-14 Thread Nicholas Kazlauskas
[Why] The behavior of dm_plane_helper_cleanup_fb differs depending on whether the commit was asynchronous or not. When it's called from amdgpu_dm_atomic_commit_tail during a typical atomic commit the plane state has been swapped so it calls cleanup_fb on the old plane state. However, in the asynch