Re: [Mesa-dev] [PATCH] radeonsi: Expose support for 10-bit VP9 decode

2019-07-05 Thread Koenig, Christian
Please ping Leo Liu since I'm currently on vacation. Christian. Am 05.07.2019 10:55 schrieb "Vishwakarma, Pratik" : On 6/28/2019 12:32 PM, Christian König wrote: > Am 28.06.19 um 07:09 schrieb Vishwakarma, Pratik: >> Fix si_vid_is_format_supported to expose support >> for 10-bit VP9 decode using

Re: [Mesa-dev] [PATCH] radv: Change memory type order for GPUs without dedicated VRAM

2019-06-03 Thread Koenig, Christian
Am 03.06.19 um 14:21 schrieb Alex Smith: On Mon, 3 Jun 2019 at 11:57, Koenig, Christian mailto:christian.koe...@amd.com>> wrote: Am 02.06.19 um 12:32 schrieb Alex Smith: > Put the uncached GTT type at a higher index than the visible VRAM type, > rather than having GTT first. >

Re: [Mesa-dev] [PATCH] radv: Change memory type order for GPUs without dedicated VRAM

2019-06-03 Thread Koenig, Christian
Am 02.06.19 um 12:32 schrieb Alex Smith: > Put the uncached GTT type at a higher index than the visible VRAM type, > rather than having GTT first. > > When we don't have dedicated VRAM, we don't have a non-visible VRAM > type, and the property flags for GTT and visible VRAM are identical. > Accordi

Re: [Mesa-dev] [PATCH] radeon/uvd: fix poc for hevc encode

2019-05-29 Thread Koenig, Christian
Ok, that sounds like it makes sense but I'm not so deeply into this handling anyway. One more comment: Please use the MIN2 or MAX2 macro here instead of open coding it. Apart from that looks good to me, Christian. Am 29.05.2019 12:53 schrieb "Zhang, Boyuan" : Yes, I agree that all interface de

Re: [Mesa-dev] [PATCH] winsys/amdgpu: add VCN JPEG to no user fence group

2019-05-08 Thread Koenig, Christian
Am 08.05.19 um 15:14 schrieb Liu, Leo: > On 5/8/19 9:02 AM, Christian König wrote: >> [CAUTION: External Email] >> >> Am 08.05.19 um 14:56 schrieb Liu, Leo: >>> There is no user fence for JPEG, the bug triggering >>> kernel WARN_ON(flags & AMDGPU_FENCE_FLAG_64BIT) >> Oh, we are probably going to ne

Re: [Mesa-dev] [PATCH 3/6] gallium\auxiliary\vl: Add compute shader to support video compositor render

2019-02-04 Thread Koenig, Christian
Am 04.02.19 um 20:20 schrieb Zhu, James: > On 2019-02-04 2:15 p.m., Christian König wrote: >> Am 04.02.19 um 20:12 schrieb James Zhu: >>> On 2019-02-04 1:47 p.m., Liu, Leo wrote: On 2/1/19 11:28 AM, Zhu, James wrote: > Add compute shader to support video compositor render. > > Sign

Re: [Mesa-dev] [PATCH 1/3] radeonsi: allow si_cp_dma_clear_buffer to clear GDS from any IB

2018-11-29 Thread Koenig, Christian
Hi Marek, you stumbled over a pretty fundamental bug in the memory management here. Essentially we where leaking BOs when we ran into an OOM situation. Patch to fix this is on the mailing list. A second problem is that eviction doesn't seem to work when GDS BOs aren't idle. In other words when

Re: [Mesa-dev] [PATCH] st/va:Aligned image width and height to 16.

2018-10-03 Thread Koenig, Christian
Am 03.10.2018 um 16:56 schrieb Ilia Mirkin: > On Wed, Oct 3, 2018 at 9:36 AM Christian König > wrote: >> What the heck are you talking about? As far as I can see this patch is >> about adding hw specific alignment to vlVaCreateImage which is a state >> tracker function. >> >> Completely agree that

Re: [Mesa-dev] [PATCH] st/va:Aligned image width and height to 16.

2018-10-02 Thread Koenig, Christian
Well that's the complete wrong place for that. The stride of the surface is determined by addrlib. That one should handle aligning the parameters. Christian. Am 02.10.2018 20:38 schrieb "Sharma, Deepak" : Christian, the issue which trying to address here is vlvaGetImage doesn’t use width/heigh

Re: [Mesa-dev] [PATCH] nouveau/video: avoid overwriting base codec init with template

2013-08-25 Thread Koenig, Christian
. 2013, 16:02 To: Martin Peres [martin.pe...@labri.fr] CC: Koenig, Christian [christian.koe...@amd.com]; Maarten Lankhorst [maarten.lankho...@ubuntu.com]; mesa-dev@lists.freedesktop.org [mesa-dev@lists.freedesktop.org] Subject: Re: [PATCH] nouveau/video: avoid overwriting base codec init with te