[Mesa-dev] [PATCH] winsys/amdgpu: Set RADEON_FLAG_GTT_WC for UVD buffer

[Christoph Haag]

Yes, vaapi decoding works with your patch.

On 26.01.2018 13:12, Marek Olšák wrote:
> Hi Christoph,
> 
> Can you test this patch instead?
> 
> Thanks,
> Marek
> 
> On Fri, Jan 19, 2018 at 3:24 PM, Christoph Haag <haagch at frickel.club> wrote:
>> without this flag, it fails an assertion
>> with assertions disabled it retries with this flag set
>> ---
>>  src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 3 ++-
>>  1 file changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
>> index 63cd63287f..515d30a959 100644
>> --- a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
>> +++ b/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
>> @@ -652,7 +652,8 @@ static bool amdgpu_ib_new_buffer(struct amdgpu_winsys *ws, struct amdgpu_ib *ib,
>>                                 RADEON_FLAG_READ_ONLY |
>>                                 (ring_type == RING_GFX ||
>>                                  ring_type == RING_COMPUTE ||
>> -                                ring_type == RING_DMA ?
>> +                                ring_type == RING_DMA ||
>> +                                ring_type == RING_UVD ?
>>                                     RADEON_FLAG_GTT_WC : 0));
>>     if (!pb)
>>        return false;
>> --
>> 2.16.0
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev