Re: [Mesa-dev] [PATCH] st/va: change frame_idx from array to hash table

blem. The following bug contains a test that reproduces the problem by running a couple of vaapih264enc in the same process. The above also explains why there was no pb when running them in separated processes. https://bugzilla.gnome.org/show_bug.cgi?id=785085 Apart from the type above, patch is

Re: [Mesa-dev] [PATCH] radeon/vcn: move message buffer to vram for now

Am 25.07.2017 um 14:13 schrieb Leo Liu: On 07/25/2017 05:19 AM, Christian König wrote: Am 25.07.2017 um 08:27 schrieb Michel Dänzer: On 25/07/17 02:54 AM, Leo Liu wrote: To workaround an unknown bug. Signed-off-by: Leo Liu <leo@amd.com> --- src/gallium/drivers/radeon/radeon_vcn

Re: [Mesa-dev] [PATCH] radeon/vcn: move message buffer to vram for now

Am 25.07.2017 um 08:27 schrieb Michel Dänzer: On 25/07/17 02:54 AM, Leo Liu wrote: To workaround an unknown bug. Signed-off-by: Leo Liu --- src/gallium/drivers/radeon/radeon_vcn_dec.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH] radeon/vce: support all firmwares with major ver 53

return true; default: - return false; + switch (rscreen->info.vce_fw_version & (0xff << 24)) { + case FW_53: + return true; + + default: +

Re: [Mesa-dev] [RFC PATCH] radeonsi: set a per-buffer flag that disables inter-process sharing (v2)

Am 20.07.2017 um 16:59 schrieb Marek Olšák: On Jul 19, 2017 10:21 PM, "zhoucm1" > wrote: On 2017年07月19日 23:34, Marek Olšák wrote: On Jul 19, 2017 3:36 AM, "zhoucm1" > wrote:

Re: [Mesa-dev] [PATCH v2] st/va: Fix scaling list ordering for H.265

Thompson <s...@jkqxz.net> Reviewed-by: Christian König <christian.koe...@amd.com> You still don't have write access to Mesa, don't you? Thanks for the help, Christian. --- On 14/07/17 11:01, Christian König wrote: That just looks like the inversed Z-scan order we already have in

Re: [Mesa-dev] [PATCH] st/va: Fix scaling list ordering for H.265

Am 13.07.2017 um 22:38 schrieb Mark Thompson: Mesa here requires the scaling lists in diagonal scan order, but VAAPI passes them in raster scan order. Therefore, rearrange the elements when copying. Nice catch, just one note below. (This ordering was likely inherited from VDPAU, which does

Re: [Mesa-dev] How to write using EGLImage?

Hi Julien, sorry, totally missed that question. I think the cleanest approach would be that the OpenMAX state tracker dlopen()s the EGL and tries to export the eglImage into a dma_buf handle. No idea how to do this, but I'm pretty sure somebody on the mailing list should know the details

Re: [Mesa-dev] How to write using EGLImage?

Hi Julien, sorry, totally missed that question. I think the cleanest approach would be that the OpenMAX state tracker dlopen()s the EGL and tries to export the eglImage into a dma_buf handle. No idea how to do this, but I'm pretty sure somebody on the mailing list should know the details

Re: [Mesa-dev] [PATCH] [rfc] radv: offset images by a differing amount.

Am 08.07.2017 um 00:27 schrieb Marek Olšák: On Fri, Jul 7, 2017 at 9:37 PM, Dave Airlie <airl...@gmail.com> wrote: On 8 July 2017 at 04:07, Christian König <deathsim...@vodafone.de> wrote: Am 07.07.2017 um 18:51 schrieb Marek Olšák: On Fri, Jul 7, 2017 at 11:18 AM, Christian Kön

Re: [Mesa-dev] [PATCH] [rfc] radv: offset images by a differing amount.

Am 07.07.2017 um 18:51 schrieb Marek Olšák: On Fri, Jul 7, 2017 at 11:18 AM, Christian König <deathsim...@vodafone.de> wrote: What tilling format have the destination textures? Sounds like the offset is just added so that we distribute memory accesses more equally over memory channels

Re: [Mesa-dev] [PATCH] [rfc] radv: offset images by a differing amount.

What tilling format have the destination textures? Sounds like the offset is just added so that we distribute memory accesses more equally over memory channels. Regards, Christian. Am 07.07.2017 um 09:18 schrieb Dave Airlie: From: Dave Airlie (this patch doesn't seem

Re: [Mesa-dev] [PATCH 09/19] radeonsi: add basic memory object support

Am 04.07.2017 um 15:13 schrieb Nicolai Hähnle: On 01.07.2017 01:03, Andres Rodriguez wrote: From: Dave Airlie Signed-off-by: Andres Rodriguez --- src/gallium/drivers/radeon/r600_pipe_common.h | 6 ++ src/gallium/drivers/radeon/r600_texture.c |

Re: [Mesa-dev] [PATCH] radeon/vcn: enable h264 decode entension support

Am 26.06.2017 um 15:29 schrieb Leo Liu: It's enabled through message buffer for UVD Signed-off-by: Leo Liu <leo@amd.com> Acked-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/radeon/radeon_vcn_dec.c | 1 + src/gallium/drivers/radeon/radeon_vcn_dec.

Re: [Mesa-dev] [PATCH 6/8] mesa: fix using texture id 0 with VDPAURegisterSurfaceNV()

Am 21.06.2017 um 11:05 schrieb Samuel Pitoiset: Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> Reviewed-by: Christian König <christian.koe...@amd.com> --- src/mesa/main/vdpau.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mesa/ma

Re: [Mesa-dev] [st/va] change va max_entrypoints

Am 14.06.2017 um 17:34 schrieb Emil Velikov: On 14 June 2017 at 15:21, Christian König <deathsim...@vodafone.de> wrote: Am 14.06.2017 um 15:40 schrieb Namburu, Chandu-babu: From: Chandu Babu N <cha...@amd.com> As encode support is added along with decode, now max_entrypo

Re: [Mesa-dev] [st/va] change va max_entrypoints

Am 14.06.2017 um 15:40 schrieb Namburu, Chandu-babu: From: Chandu Babu N <cha...@amd.com> As encode support is added along with decode, now max_entrypoints are two. vaMaxNumEntrypoints will return correct value. Signed-off-by: Chandu Babu N <cha...@amd.com> Reviewed-by: Chr

[Mesa-dev] [PATCH] winsys/amdgpu: align VA allocations to fragment size

From: Christian König <christian.koe...@amd.com> BOs larger than the minimum fragment size should have their VA alignet to at least the fragment size for optimal performance. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/amd/common/ac_gpu_info.c

Re: [Mesa-dev] [PATCH v2 12/27] auxiliary/vl: use vl_*_screen_create stubs when building w/o platform

Am 05.05.2017 um 14:11 schrieb Emil Velikov: On 5 May 2017 at 12:37, Christian König <christian.koe...@amd.com> wrote: Am 05.05.2017 um 13:23 schrieb Emil Velikov: On 5 May 2017 at 10:30, Christian König <christian.koe...@amd.com> wrote: Am 04.05.2017 um 18:33 schrieb Emil Ve

Re: [Mesa-dev] [PATCH v2 20/27] st/omx: add DRI3 support

Am 04.05.2017 um 18:33 schrieb Emil Velikov: From: Emil Velikov <emil.veli...@collabora.com> Cc: Christian König <christian.koe...@amd.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- UNTESTED. Looks sane to me, but what bothers me a bit is that it is un

Re: [Mesa-dev] [PATCH v2 12/27] auxiliary/vl: use vl_*_screen_create stubs when building w/o platform

Am 05.05.2017 um 13:23 schrieb Emil Velikov: On 5 May 2017 at 10:30, Christian König <christian.koe...@amd.com> wrote: Am 04.05.2017 um 18:33 schrieb Emil Velikov: From: Emil Velikov <emil.veli...@collabora.com> Provide a dummy stub when the user has opted w/o said platform, thus

Re: [Mesa-dev] [PATCH v2 13/27] st/va: move variable declaration to where its used

Patches #13 - #17 are Reviewed-by: Christian König <christian.koe...@amd.com>, but I need to take a closer look at the rest. BTW: Can you send and commit those minor cleanups separately the next time? Makes live much easier to have the trivial stuff in master there is autohell or other non

Re: [Mesa-dev] [PATCH v2 12/27] auxiliary/vl: use vl_*_screen_create stubs when building w/o platform

he two. Cc: Leo Liu <leo@amd.com> Cc: "Guttula, Suresh" <suresh.gutt...@amd.com> Cc: mesa-sta...@lists.freedesktop.org Cc: Christian König <christian.koe...@amd.com> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- Christian, others How do you feel abou

Re: [Mesa-dev] [PATCH 2/2] st/vdpau: fold vdpau_interop.h and vdpau_dmabuf.h

Am 25.04.2017 um 15:17 schrieb Ilia Mirkin: [SNIP] Is there a patch I should test? Patch is attached, but only compile tested. Basically if OpenGL/VDPAU interop worked before with Kodi/MPV it should still keep working. Christian. -ilia >From

Re: [Mesa-dev] [PATCH 2/2] st/vdpau: fold vdpau_interop.h and vdpau_dmabuf.h

Am 21.04.2017 um 15:38 schrieb Emil Velikov: On 21 April 2017 at 13:31, Christian König <deathsim...@vodafone.de> wrote: Am 21.04.2017 um 14:11 schrieb Emil Velikov: Both headers are used everywhere, plus they both provide interop mechanism. Since they define [consecutive] VDPAU driver

Re: [Mesa-dev] [PATCH 2/2] st/vdpau: fold vdpau_interop.h and vdpau_dmabuf.h

: Christian König <christian.koe...@amd.com> Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> NAK, the seperation is intentional. vdpau_interop.h is the old one we sooner or later want to remove. And vdpau_dmabuf.h is the new one we want to keep. Regards, Christian. --- src/gal

Re: [Mesa-dev] [PATCH 1/2] st/vdpau: split the VdpFormatRGBAToPipe helper in two

f libvdpau. With this commit we effectively revert "st/vdpau: move FormatRGBAToPipe into the interop" Cc: Christian König <christian.koe...@amd.com> Signed-off-by: Emil Velikov <emil.l.veli...@gmail.com> NAK, that would break DMA-buf based interop since we need R8, R8G8 a

Re: [Mesa-dev] [PATCH] radeonsi: add new polaris10 pci id

Am 05.04.2017 um 15:46 schrieb Alex Deucher: Cc: 13.0 17.0 <mesa-sta...@lists.freedesktop.org> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com> Reviewed-by: Christian König <christian.koe...@amd.com> --- include/pci_ids/radeonsi_pci_ids.h | 1 + 1 file changed, 1

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

. Regards, Christian. As it stands with cabac on I can make (what I think is) a perfectly legal stream with ffmpeg or gstreamer (ignoring current mesa issue). ... -c:v h264_vaapi -profile:v 77 -bf 0 ... ... max-bframes=0 ! video/x-h264, profile=main ... Christian König wrote: Am 04.04.2017

Re: [Mesa-dev] [PATCH] radeon/video: only support h264 baseline encode

Am 04.04.2017 um 17:38 schrieb boyuan.zh...@amd.com: From: Boyuan Zhang <boyuan.zh...@amd.com> Signed-off-by: Boyuan Zhang <boyuan.zh...@amd.com> Reviewed-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/radeon/radeon_video.c | 2 +- 1 file ch

Re: [Mesa-dev] [PATCH] targets/va: export radeon winsys_create functions

: 17.0 <mesa-sta...@lists.freedesktop.org> Reviewed-by: Christian König <christian.koe...@amd.com> --- src/gallium/targets/omx/omx.sym | 5 + src/gallium/targets/pipe-loader/pipe.sym | 5 + src/gallium/targets/va/va.sym| 5 + 3 files changed,

Re: [Mesa-dev] [PATCH] st/omx/dec: Properly undefine DEBUG macro

Am 03.04.2017 um 06:35 schrieb Shaleen Jain: --- src/gallium/state_trackers/omx/vid_dec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/omx/vid_dec.c b/src/gallium/state_trackers/omx/vid_dec.c index 9a6efb8e28..94664eba04 100644 ---

Re: [Mesa-dev] [PATCH] st/omx/enc: use PIPE_USAGE_STAGING for output buffer

Am 23.03.2017 um 15:35 schrieb Leo Liu: Workaround an unknown bug with inside the transfer_map for certain ASIC, also tested with un-affected ASICs, the performance actually improved slightly. Signed-off-by: Leo Liu <leo@amd.com> Reviewed-by: Christian König <christian.koe..

Re: [Mesa-dev] [PATCH 000/140] RadeonSI: Initial Vega10 support

Am 20.03.2017 um 23:42 schrieb Marek Olšák: Hi, This is initial Vega10 support for radeonsi. It supports everything except geometry shaders and tessellation, so it's limited to GL 3.1. The missing features are under way. There is also UVD and VCE support. The first 57 patches only update

Re: [Mesa-dev] [PATCH 139/140] radeon/vce: update vce support to firmware 53.19.4

Am 20.03.2017 um 23:49 schrieb Marek Olšák: From: Leo Liu <leo@amd.com> Signed-off-by: Leo Liu <leo@amd.com> Ah, here it is. This one and the change to version 53.17 should be squashed with the change to 53.14. With that done the patch is Reviewed-by: Chr

Re: [Mesa-dev] [PATCH 136/140] radeon/vce: add support for firmware 53.14.4

Am 20.03.2017 um 23:49 schrieb Marek Olšák: From: Leo Liu Signed-off-by: Leo Liu Acked-by: Alex Deucher Did we ever released 53.14.4? My last status was the firmware on release should be 53.17 or something like that.

Re: [Mesa-dev] [PATCH 135/140] radeon/vce: adapt gfx9 surface to vce

Am 20.03.2017 um 23:49 schrieb Marek Olšák: From: Leo Liu <leo@amd.com> Signed-off-by: Leo Liu <leo@amd.com> Acked-by: Alex Deucher <alexander.deuc...@amd.com> Reviewed-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/rad

Re: [Mesa-dev] [PATCH 133/140] radeon/uvd: clear message buffer when reuse

Am 20.03.2017 um 23:49 schrieb Marek Olšák: From: Leo Liu <leo@amd.com> As required by firmware Signed-off-by: Leo Liu <leo@amd.com> Acked-by: Alex Deucher <alexander.deuc...@amd.com> Reviewed-by: Christian König <christian.koe...@amd.com> --- sr

Re: [Mesa-dev] [PATCH 132/140] radeon/uvd: adapt gfx9 surface to uvd

Am 20.03.2017 um 23:49 schrieb Marek Olšák: From: Leo Liu <leo@amd.com> Signed-off-by: Leo Liu <leo@amd.com> Acked-by: Alex Deucher <alexander.deuc...@amd.com> Reviewed-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/r600/r600_uv

Re: [Mesa-dev] [PATCH 131/140] radeon/uvd: add uvd soc15 register

Am 20.03.2017 um 23:49 schrieb Marek Olšák: From: Leo Liu <leo@amd.com> Signed-off-by: Leo Liu <leo@amd.com> Acked-by: Alex Deucher <alexander.deuc...@amd.com> Reviewed-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI v2

vpp and before maping va buffers to GL. I suggest to keep it simple from a driver perspective and require applications to use vaSyncSurface *Gesendet:* Sonntag, 19. März 2017 um 15:28 Uhr *Von:* "Christian König" <deathsim...@vodafone.de> *An:* "Peter Frühberger" <

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI v2

specific. What do you think? Best regards Peter 2017-03-19 14:49 GMT+01:00 Christian König <deathsim...@vodafone.de <mailto:deathsim...@vodafone.de>>: Hi Peter, Adding Michel and Marek for the Mesa interop side and Harry for the display side. How do you want u

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI v2

he underlying type of hardware/driver. Regards, Rainer *Gesendet:* Mittwoch, 08. März 2017 um 13:29 Uhr *Von:* "Christian König" <deathsim...@vodafone.de <mailto:deathsim...@vodafone.de>> *An:* mesa-dev@lists.freedesktop.org <mailto:mesa-dev@lis

Re: [Mesa-dev] [PATCH 01/11] gallium: add P016 format

Am 13.03.2017 um 12:48 schrieb Emil Velikov: On 13 March 2017 at 11:47, Christian König <deathsim...@vodafone.de> wrote: Am 13.03.2017 um 11:33 schrieb Emil Velikov: Hi Christian, On 8 March 2017 at 12:29, Christian König <deathsim...@vodafone.de> wrote: From: Christian König &l

Re: [Mesa-dev] [PATCH 01/11] gallium: add P016 format

Am 13.03.2017 um 11:33 schrieb Emil Velikov: Hi Christian, On 8 March 2017 at 12:29, Christian König <deathsim...@vodafone.de> wrote: From: Christian König <christian.koe...@amd.com> Same layout as NV12, but 16bit per channel instead of 8. --- a/src/gallium/include/pipe/p_format

[Mesa-dev] [PATCH] svga: handle P016 format as well

From: Christian König <christian.koe...@amd.com> Fix for "gallium: add P016 format". --- src/gallium/drivers/svga/svga_format.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/svga/svga_format.c b/src/gallium/drivers/svga/svga_format.c index 948b276

Re: [Mesa-dev] [PATCH 11/11] st/va: add config support for 10bit decoding

Am 09.03.2017 um 13:46 schrieb Mark Thompson: On 09/03/17 07:54, Christian König wrote: Replaying here from the comment in your other mail as well: Um, libav* is querying the capabilities and finding that only 8-bit output is supported for Main10: [SNIP] Unable to create config to test

[Mesa-dev] [PATCH 6/9] st/va: clear the video surface on allocation

From: Christian König <christian.koe...@amd.com> This makes debugging of decoding problems quite a bit easier. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/va/surface.c | 39 + 1 file changed, 35 inse

[Mesa-dev] [PATCH 7/9] st/va: add support for P010 and P016 formats v2

From: Christian König <christian.koe...@amd.com> No hardware I know off can actually support P010 natively. But we can easily support P016 and as long as nobody decodes anything into the lower 6bits it doesn't make any difference to P010. v2: allow P0160 for post processing as well Sign

[Mesa-dev] [PATCH 4/9] radeon/uvd: enable 10bit HEVC decode v2

From: Christian König <christian.koe...@amd.com> Just use whatever the state tracker allocated. v2: fix msb mode Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/radeon/radeon_uvd.c | 17 ++--- src/gallium/drivers/radeon/radeon_

[Mesa-dev] [PATCH 5/9] st/va: cleanup error handling in vlVaCreateSurfaces2

From: Christian König <christian.koe...@amd.com> No need to have that twice. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/va/surface.c | 52 + 1 file changed, 27 insertions(+), 25 deletions(-) diff --git a

[Mesa-dev] [PATCH 2/9] vl/video_buffer: add support for P016

From: Christian König <christian.koe...@amd.com> Just simply the description of the planes. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/auxiliary/vl/vl_video_buffer.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/gallium/

[Mesa-dev] [PATCH 8/9] st/va: add support for allocating 10bpp surfaces

From: Christian König <christian.koe...@amd.com> We support P010 and P016 as targets for 10bpp video decoding. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/va/surface.c | 24 +++- 1 file changed, 15 insertions(+),

[Mesa-dev] [PATCH 3/9] radeon/UVD: fix the decoding target pitch calculation

From: Christian König <christian.koe...@amd.com> The firmware expects the value in pixel not bytes. Didn't made a difference so far because we only used 8bpp surfaces. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/radeon/radeon_uvd.c | 2 +- 1 fil

[Mesa-dev] [PATCH 1/9] gallium: add P016 format

From: Christian König <christian.koe...@amd.com> Same layout as NV12, but 16bit per channel instead of 8. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/auxiliary/util/u_format.csv | 2 ++ src/gallium/auxiliary/util/u_format_yuv.c | 19

[Mesa-dev] [PATCH 9/9] st/va: add config support for 10bit decoding v2

From: Christian König <christian.koe...@amd.com> Advertise 10bpp support if the driver supports decoding to a P016 surface. v2: Advertise 10bpp for the decoder as well. Signed-off-by: Christian König <christian.koe...@amd.com> Signed-off-by: Mark Thompson <s...@jkqxz.net>

Re: [Mesa-dev] [PATCH 11/11] st/va: add config support for 10bit decoding

objections merging that into my original patch and adding your signed-of-by line? Thanks for the help, Christian. Am 08.03.2017 um 22:36 schrieb Mark Thompson: On 08/03/17 21:32, Mark Thompson wrote: On 08/03/17 12:29, Christian König wrote: From: Christian König <christian.koe...@amd.

[Mesa-dev] [PATCH 08/11] st/va: clear the video surface on allocation

From: Christian König <christian.koe...@amd.com> This makes debugging of decoding problems quite a bit easier. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/va/surface.c | 39 + 1 file changed, 35 inse

[Mesa-dev] [PATCH 11/11] st/va: add config support for 10bit decoding

From: Christian König <christian.koe...@amd.com> Advertise 10bpp support if the driver supports decoding to a P016 surface. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/va/config.c | 15 +-- src/gallium/state_trackers/va/

[Mesa-dev] [PATCH 09/11] st/va: add support for P010 and P016 formats

From: Christian König <christian.koe...@amd.com> No hardware I know off can actually support P010 natively. But we can easily support P016 and as long as nobody decodes anything into the lower 6bits it doesn't make any difference to P010. Signed-off-by: Christian König <christian.koe..

[Mesa-dev] [PATCH 03/11] radeon/UVD: fix the decoding target pitch calculation

From: Christian König <christian.koe...@amd.com> The firmware expects the value in pixel not bytes. Didn't made a difference so far because we only used 8bpp surfaces. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/radeon/radeon_uvd.c | 2 +- 1 fil

[Mesa-dev] [PATCH 06/11] st/vdpau: always use the prefered surface format for decode

From: Christian König <christian.koe...@amd.com> That allows us to switch between different surface formats for different codecs. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/vdpau/decode.c | 11 ++- 1 file changed, 6 inse

[Mesa-dev] [PATCH 10/11] st/va: add support for allocating 10bpp surfaces

From: Christian König <christian.koe...@amd.com> We support P010 and P016 as targets for 10bpp video decoding. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/va/surface.c | 24 +++- 1 file changed, 15 insertions(+),

[Mesa-dev] 10bit HEVC decoding for RadeonSI v2

Hi guys, I finally found time testing this and hammering out (hopefully) all the remaining bugs. Playing a 10bit HEVC file through VAAPI with mpv/ffmpeg git master from about two days ago now works flawlessly and has only about 15% CPU load on one core on a Kaveri system. The VDPAU path should

[Mesa-dev] [PATCH 07/11] st/va: cleanup error handling in vlVaCreateSurfaces2

From: Christian König <christian.koe...@amd.com> No need to have that twice. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/va/surface.c | 52 + 1 file changed, 27 insertions(+), 25 deletions(-) diff --git a

[Mesa-dev] [PATCH 05/11] st/vdpau: add support for 16bit interop

From: Christian König <christian.koe...@amd.com> That should make it possible to use the 16bit surfaces in OpenGL as well. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/include/state_tracker/vdpau_dmabuf.h | 2 ++ src/gallium/include/state_tracker/v

[Mesa-dev] [PATCH 04/11] radeon/uvd: enable 10bit HEVC decode v2

From: Christian König <christian.koe...@amd.com> Just use whatever the state tracker allocated. v2: fix msb mode Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/drivers/radeon/radeon_uvd.c | 17 ++--- src/gallium/drivers/radeon/radeon_

[Mesa-dev] [PATCH 01/11] gallium: add P016 format

From: Christian König <christian.koe...@amd.com> Same layout as NV12, but 16bit per channel instead of 8. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/auxiliary/util/u_format.csv | 2 ++ src/gallium/auxiliary/util/u_format_yuv.c | 19

[Mesa-dev] [PATCH 02/11] vl/video_buffer: add support for P016

From: Christian König <christian.koe...@amd.com> Just simply the description of the planes. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/auxiliary/vl/vl_video_buffer.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/gallium/

[Mesa-dev] [PATCH] st/va: cleanup error handling in vlVaCreateSurfaces2

From: Christian König <christian.koe...@amd.com> No need to have that twice. Signed-off-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/va/surface.c | 52 + 1 file changed, 27 insertions(+), 25 deletions(-) diff --git a

Re: [Mesa-dev] [PATCH] st/omx: Set end-of-frame flag on bitstream output buffers

output buffers are fragments of the first frame when it is not set. Reviewed-by: Christian König <christian.koe...@amd.com> --- src/gallium/state_trackers/omx/vid_enc.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/state_trackers/omx/vid_enc.c b/src/g

Re: [Mesa-dev] [RFC PATCH 0/7] gallium: Video postprocessor rework

Hi Thomas, Am 02.03.2017 um 21:00 schrieb Thomas Hellstrom: This patch series introduces a postprocessor abstraction. It could be promoted to a gallium interface but for now it's implemented as a utility. Well, first of all use a gallium interface for this. Putting this into utility doesn't

Re: [Mesa-dev] [PATCH] st/va: Fix forward/backward referencing for deinterlacing

deinterlacing. Signed-off-by: Mark Thompson <s...@jkqxz.net> That explains a couple of things, thanks for looking into it. Patch is Reviewed-by: Christian König <christian.koe...@amd.com>. Regards, Christian. --- src/gallium/state_trackers/va/postproc.c | 10 +- src/gallium/state

Re: [Mesa-dev] Putbits patches

Am 24.02.2017 um 15:40 schrieb Thomas Hellstrom: Hi, Christian! Are you OK with the updated Putbits patches? https://lists.freedesktop.org/archives/mesa-dev/2017-February/145269.html https://lists.freedesktop.org/archives/mesa-dev/2017-February/145258.html Sorry the second patch never made

Re: [Mesa-dev] [PATCH 1/2] st/vdpau: Provide YV12 to NV12 putBits conversion v2

König <cthristian.koe...@amd.com> Signed-off-by: Thomas Hellstrom <thellst...@vmware.com> Acked-by: Christian König <christian.koe...@amd.com> for both patches. --- src/gallium/auxiliary/util/u_video.h | 42 + src/gallium/state_trackers/vdpau/query.c | 13 +

Re: [Mesa-dev] [PATCH 0/8] gallium: A number of video code fixes

Am 22.02.2017 um 18:05 schrieb Thomas Hellstrom: On 02/22/2017 05:54 PM, Christian König wrote: Am 22.02.2017 um 17:23 schrieb Thomas Hellstrom: On 02/22/2017 04:46 PM, Christian König wrote: Am 22.02.2017 um 16:31 schrieb Thomas Hellstrom: On 02/22/2017 04:00 PM, Emil Velikov wrote: On 22

Re: [Mesa-dev] [PATCH 0/8] gallium: A number of video code fixes

Am 22.02.2017 um 17:23 schrieb Thomas Hellstrom: On 02/22/2017 04:46 PM, Christian König wrote: Am 22.02.2017 um 16:31 schrieb Thomas Hellstrom: On 02/22/2017 04:00 PM, Emil Velikov wrote: On 22 February 2017 at 09:30, Thomas Hellstrom <thellst...@vmware.com> wrote: On 02/22/2017 09

Re: [Mesa-dev] [PATCH 0/8] gallium: A number of video code fixes

Am 22.02.2017 um 16:31 schrieb Thomas Hellstrom: On 02/22/2017 04:00 PM, Emil Velikov wrote: On 22 February 2017 at 09:30, Thomas Hellstrom <thellst...@vmware.com> wrote: On 02/22/2017 09:56 AM, Christian König wrote: Am 21.02.2017 um 21:52 schrieb Thomas Hellstrom: A couple of

Re: [Mesa-dev] [PATCH 1/2] st/vdpau: Don't reallocate video surfaces on putBitsYCbCr format mismatch

Am 22.02.2017 um 14:42 schrieb Thomas Hellstrom: mplayer likes putting YV12 data, and if there is a buffer format mismatch, the vdpau state tracker would try to reallocate the video surface as an YV12 surface. A virtual driver doesn't like reallocating and doesn't like YV12 surfaces, so before

Re: [Mesa-dev] [PATCH 0/8] gallium: A number of video code fixes

Am 21.02.2017 um 21:52 schrieb Thomas Hellstrom: A couple of fixes / improvements for things I've encountered while looking through and testing the video code in preparation for a virtual hardware video driver. Reviewed-by: Christian König <christian.koe...@amd.com> for the who

Re: [Mesa-dev] [PATCH 2/2] vdpau: skip vlVdpOutputSurfacePutBitsNative with a zero-area rectangle

le, this is most likely a bug"? Actually I'm not sure if that is an application bug. Trying to put a box with zero width and hight might not make much sense, but the interface doesn't forbids that explicitly. Anyway we need to catch this case in the state tracker and so the patch is Reviewed-by: Chri

Re: [Mesa-dev] [PATCH v2] st/va: remove assert for single slice

Am 10.02.2017 um 16:36 schrieb Nayan Deshmukh: we anyway allow for multiple slices v2: do not remove assert to check for buf->size Signed-off-by: Nayan Deshmukh <nayan26deshm...@gmail.com> Reviewed-by: Christian König <christian.koe...@amd.com>. --- src/gallium/st

Re: [Mesa-dev] [PATCH] st/va: remove assert for single slice

Sorry for the delay, as noted in the other mail I was on sick leave for a while. Am 03.02.2017 um 05:52 schrieb Nayan Deshmukh: On Thu, Feb 2, 2017 at 3:34 PM, Christian König <deathsim...@vodafone.de> wrote: Am 01.02.2017 um 13:59 schrieb Nayan Deshmukh: we anyway allow for multiple

Re: [Mesa-dev] [PATCH] st/va: remove assert for single slice

Am 01.02.2017 um 13:59 schrieb Nayan Deshmukh: we anyway allow for multiple slices Signed-off-by: Nayan Deshmukh --- src/gallium/state_trackers/va/picture_mpeg12.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/state_trackers/va/picture_mpeg12.c

Re: [Mesa-dev] vdpau_interop extension oddities

Am 01.02.2017 um 16:50 schrieb Thomas Hellstrom: Hi, Again, (No need answering while on sick leave!). Actually I'm bored, but can't concentrate on coding when my body temperature isn't in the normal range :( Hui? That's not how I remembered it. For the dma-buf part we export one layer of

Re: [Mesa-dev] vdpau_interop extension oddities

Hi Thomas, I'm on sick leave, but will try to quickly answer your questions. But don't be disappointed if you don't hear back from me before Monday. Am 01.02.2017 um 15:56 schrieb Thomas Hellstrom: Hi, Christian, I'm looking through the mesa vdpau interop code and found something that looks

Re: [Mesa-dev] [PATCH] drm/radeon: Fix vram_size/visible values in DRM_RADEON_GEM_INFO ioctl

e. Reviewed-by: Christian König <christian.koe...@amd.com> for this one as well the mesa patch. Regards, Christian. --- drivers/gpu/drm/radeon/radeon_drv.c | 3 ++- drivers/gpu/drm/radeon/radeon_gem.c | 4 ++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/r

Re: [Mesa-dev] [PATCH] st/va encode handle ntsc framerate rate control

Am 30.01.2017 um 09:17 schrieb Michel Dänzer: On 30/01/17 12:08 AM, Andy Furniss wrote: Christian König wrote: That looks correct to me, but I'm not deeply enough into the H264 encoding any more. Adding Leo and Boyuan to comment as well. I just sent v2 to list and tried to cc Boyuan and Leo

Re: [Mesa-dev] [PATCH 3/3] winsys/radeon: clamp vram_vis_size to 256MB

NAK, that isn't correct. Especially on APUs we can now use more than 256MB visible VRAM. Christian. Am 30.01.2017 um 01:33 schrieb Marek Olšák: From: Marek Olšák the value from the kernel is wrong --- src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 2 +- 1 file

Re: [Mesa-dev] [PATCH v2] st/va encode handle ntsc framerate rate control

.li...@gmail.com> --- src/gallium/state_trackers/va/picture.c | 19 +-- 1 file changed, 13 insertions(+), 6 deletions(-) Tested, LGTM. (0.5fps works nicely now :) Reviewed-by: Mark Thompson <s...@jkqxz.net> Acked-by: Christian König <christian.koe...@amd.com>

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI

Am 27.01.2017 um 20:44 schrieb Mark Thompson: On 27/01/17 14:27, Christian König wrote: Am 27.01.2017 um 13:51 schrieb Mark Thompson: On 26/01/17 16:59, Christian König wrote: [SNIP] (For that matter, is there a list somewhere of the set of formats/layouts and what they are used for?) Well

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI

Am 27.01.2017 um 13:51 schrieb Mark Thompson: On 26/01/17 16:59, Christian König wrote: Am 26.01.2017 um 13:14 schrieb Mark Thompson: [SNIP] The problem here is I need to know what will be done with the surface from the very beginning. E.g. if you want to scan it out directly to hardware you

Re: [Mesa-dev] [PATCH 08/10] gallium/vl: use the common uploader

Reviewed-by: Christian König <christian.koe...@amd.com>. Am 27.01.2017 um 12:02 schrieb Marek Olšák: From: Marek Olšák <marek.ol...@amd.com> --- src/gallium/auxiliary/vl/vl_compositor.c | 14 ++ src/gallium/auxiliary/vl/vl_compositor.h | 1 - src/gallium/state

Re: [Mesa-dev] [PATCH] st/va encode handle ntsc framerate rate control

That looks correct to me, but I'm not deeply enough into the H264 encoding any more. Adding Leo and Boyuan to comment as well. Regards, Christian. Am 26.01.2017 um 19:26 schrieb Andy Furniss: Tested with ffmpeg and gst-vaapi. Without this bits per frame is set way too low. Signed-off-by:

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

Am 27.01.2017 um 10:33 schrieb Samuel Pitoiset: On 01/26/2017 12:07 PM, Christian König wrote: Am 26.01.2017 um 12:01 schrieb Samuel Pitoiset: On 01/26/2017 03:45 AM, Michel Dänzer wrote: On 25/01/17 11:19 PM, Samuel Pitoiset wrote: On 01/25/2017 03:56 AM, Michel Dänzer wrote: On 25/01

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI

Am 26.01.2017 um 16:59 schrieb Peter Frühberger: 2017-01-26 16:36 GMT+01:00 Christian König <deathsim...@vodafone.de <mailto:deathsim...@vodafone.de>>: Am 26.01.2017 um 12:16 schrieb Peter Frühberger: Hi Christian, 2017-01-26 12:00 GMT+01:00 Christian König

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI

Am 26.01.2017 um 13:14 schrieb Mark Thompson: On 26/01/17 11:00, Christian König wrote: Hi Peter, Am 25.01.2017 um 19:45 schrieb Peter Frühberger: Peter, Rainer any idea what I'm missing here? Do you guys use some modified ffmpeg for Kodi or how does that work for you? do you set

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI

Am 26.01.2017 um 12:16 schrieb Peter Frühberger: Hi Christian, 2017-01-26 12:00 GMT+01:00 Christian König <deathsim...@vodafone.de <mailto:deathsim...@vodafone.de>>: Hi Peter, Am 25.01.2017 um 19:45 schrieb Peter Frühberger: Peter, Rainer any idea what

Re: [Mesa-dev] [PATCH] gallium/radeon: add a new HUD query for the number of mapped buffers

Am 26.01.2017 um 12:01 schrieb Samuel Pitoiset: On 01/26/2017 03:45 AM, Michel Dänzer wrote: On 25/01/17 11:19 PM, Samuel Pitoiset wrote: On 01/25/2017 03:56 AM, Michel Dänzer wrote: On 25/01/17 12:05 AM, Marek Olšák wrote: On Tue, Jan 24, 2017 at 2:17 PM, Christian König <death

Re: [Mesa-dev] 10bit HEVC decoding for RadeonSI

Hi Peter, Am 25.01.2017 um 19:45 schrieb Peter Frühberger: Peter, Rainer any idea what I'm missing here? Do you guys use some modified ffmpeg for Kodi or how does that work for you? do you set the format correctly, e.g.:

<    1   2   3   4   5   6   7   8   9   10   >