. [PATCH] drm/amd/amdgpu: Fix ring initialization for GFX9

Andres Rodriguez andresx7 at gmail.com
Mon Jun 5 20:07:45 UTC 2017



On 2017-06-05 03:48 PM, Xie, AlexBin wrote:
> Hi Andres,
> 
> I think the original patch was written by you. Would you comment? Is it a bug or intentional?
> 
> Thank you.
> 
> Alex Bin Xie
> 
> 
> Message: 1
> Date: Mon, 5 Jun 2017 10:36:59 -0400
> From: Tom St Denis <tom.stdenis at amd.com>
> To: amd-gfx at lists.freedesktop.org
> Subject: Re: amd-gfx Digest, Vol 13, Issue 29
> Message-ID: <58773ac0-93a7-e494-d447-88e93ded999c at amd.com>
> Content-Type: text/plain; charset=utf-8; format=flowed
> 
> On 05/06/17 10:24 AM, Xie, AlexBin wrote:
>> Hi, Tom,
>>
>> You have found a bug.
>>
>> Your patch looks fine for me.
>>
>> Have you confirmed the deleted part is older version? Perhaps search email list or git history to confirm?
> 
> It looks like the edits to the older GFX files (7/8) simply changed that
> block of code whereas the gfx9 version they pasted in the fixed block
> moving the old block down.

Yeah there was a problem with the rebase here. Tom/Alex's fix on the 
mailing list is the intended solution.

Regards,
Andres

> 
> Tom
> 
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> 


More information about the amd-gfx mailing list