[PATCH] Revert "drm/amdgpu: Only update the CUR_SIZE register when necessary"

Alexander.Deucher@xxxxxxx (Deucher, Alexander) · Thu, 12 Jan 2017 13:27:14 +0000

> -----Original Message-----
> From: Michel DÃ¤nzer [mailto:michel at daenzer.net]
> Sent: Thursday, January 12, 2017 3:31 AM
> To: Deucher, Alexander
> Cc: amd-gfx at lists.freedesktop.org
> Subject: Re: [PATCH] Revert "drm/amdgpu: Only update the CUR_SIZE
> register when necessary"
> 
> On 12/01/17 02:35 AM, Deucher, Alexander wrote:
> >> -----Original Message-----
> >> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On
> Behalf
> >> Of Michel DÃ¤nzer
> >> Sent: Wednesday, January 11, 2017 4:36 AM
> >> To: amd-gfx at lists.freedesktop.org
> >> Cc: Harald Judt; Johannes Hirte
> >> Subject: [PATCH] Revert "drm/amdgpu: Only update the CUR_SIZE
> register
> >> when necessary"
> >>
> >> From: Michel DÃ¤nzer <michel.daenzer at amd.com>
> >>
> >> This reverts commits 7c83d7abc9997cf1efac2c0ce384b5e8453ee870 and
> >> a1f49cc179ce6b7b7758ae3ff5cdb138d0ee0f56.
> >>
> >> They caused the HW cursor to disappear under various circumstances in
> >> the wild. I wasn't able to reproduce any of them, and I'm not sure
> >> what's going on. But those changes aren't a big deal anyway, so let's
> >> just revert for now.
> >>
> >> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=191291
> >> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99143
> >> Signed-off-by: Michel DÃ¤nzer <michel.daenzer at amd.com>
> >
> > Acked-by: Alex Deucher <alexander.deucher at amd.com>
> 
> Thanks Alex. Which tree should I commit this to, in order for you to
> pick it up for 4.10?
> 

Go ahead and commit it to amd-staging.  I'll pull it into my -fixes branch.

Thanks!

Alex