RE: [PATCH 16/19] drm/amdkfd: Update PM4 packet headers

2017-08-16 Thread Deucher, Alexander
> -Original Message- > From: Kuehling, Felix > Sent: Wednesday, August 16, 2017 6:34 PM > To: Deucher, Alexander; Oded Gabbay; Bridgman, John > Cc: amd-gfx list > Subject: Re: [PATCH 16/19] drm/amdkfd: Update PM4 packet headers > > On 2017-08-16 12:10 PM,

Re: [PATCH 16/19] drm/amdkfd: Update PM4 packet headers

2017-08-16 Thread Felix Kuehling
On 2017-08-16 12:10 PM, Deucher, Alexander wrote: >> -Original Message- >> From: Kuehling, Felix >> Sent: Tuesday, August 15, 2017 9:20 PM >> To: Oded Gabbay; Bridgman, John; Deucher, Alexander >> Cc: amd-gfx list >> Subject: Re: [PATCH 16/19] drm/amdkfd

Re: [PATCH 16/19] drm/amdkfd: Update PM4 packet headers

2017-08-16 Thread Felix Kuehling
ewest microcode for both amdgpu and amdkfd WITHOUT >>>> breaking Kaveri. >>>> >>>> IOW this is the "scratch fix for Kaveri KFD" you have wanted for a >>>> couple of years :) >>> ah, ok. >>> >>> In that case, this patch

Re: [PATCH 16/19] drm/amdkfd: Update PM4 packet headers

2017-08-16 Thread Oded Gabbay
nd amdkfd WITHOUT breaking Kaveri. >>>> >>>> IOW this is the "scratch fix for Kaveri KFD" you have wanted for a >>>> couple of years :) >>> >>> ah, ok. >>> >>> In that case, this patch is: >>> Reviewed-by: Oded Gabbay

Re: [PATCH 16/19] drm/amdkfd: Update PM4 packet headers

2017-08-15 Thread Felix Kuehling
; > >>> -Original Message- >>> From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf >>> Of Kuehling, Felix >>> Sent: Saturday, August 12, 2017 2:16 PM >>> To: Oded Gabbay >>> Cc: amd-gfx list >>> Subject: Re

RE: [PATCH 16/19] drm/amdkfd: Update PM4 packet headers

2017-08-12 Thread Bridgman, John
gt;but they should be binary compatible. So we simplified the code to use only >one set of headers. I'll grab a Kaveri system to confirm that it works. > >Regards, > Felix > >From: Oded Gabbay >Sent: Saturday, August 12, 2017 11:10 AM >To: Kuehling, Felix >Cc: amd-gfx

Re: [PATCH 16/19] drm/amdkfd: Update PM4 packet headers

2017-08-12 Thread Oded Gabbay
On Sat, Aug 12, 2017 at 12:56 AM, Felix Kuehling wrote: > To match current firmware. The map process packet has been extended > to support scratch. This is a non-backwards compatible change and > it's about two years old. So no point keeping the old version around > conditionally. Do you mean tha