Re: [Mesa-dev] [PATCH] st/vdpau: use bicubic filter for scaling

Secondly, I also need to set destination_rect clip. I am not yet sure how that could be done. See how clipping is done in the compositor, function vl_compositor_render() and vl_compositor_set_dst_clip(). Basically you just use a scissor state to limit where on the surface we are going to writ

Re: [Mesa-dev] [PATCH] st/vdpau: use bicubic filter for scaling

Am 21.06.2016 um 23:51 schrieb Andy Furniss: Andy Furniss wrote: Nayan Deshmukh wrote: I forgot to CC the list, so I am attaching our conversation here. Also probably my R7 M265 uses a progressive shader. OK, maybe that could be it. I may be able to try progressive, but need to remember what

Re: [Mesa-dev] [PATCH 3/3] radeon/vce: use vce structures for vce_52 firmware

he structure by just using the values from the picture descriptor directly. Regards, Christian. Regards, Boyuan -Original Message- From: Christian König [mailto:deathsim...@vodafone.de] Sent: June-22-16 3:34 AM To: Zhang, Boyuan; mesa-dev@lists.freedesktop.org Subject: Re: [PATCH 3/3]

Re: [Mesa-dev] [PATCH 1/4] radeon/vce: add vce structures

Am 22.06.2016 um 23:44 schrieb Boyuan Zhang: Signed-off-by: Boyuan Zhang I'm still not completely happy with this but I think we should move forward and can fix the fallout later on. So patches #1 and #2 are Reviewed-by: Christian König . If Leo agrees I think we can commit this ups

Re: [Mesa-dev] [PATCH 4/4] radeon/vce: handle newly added parameters

+ if (pic->enable_low_level_control == true) { Well using this enable_low_level_control switch to choose between two sets of hardcoded values is clearly a no go. For the motion estimation and most of the other flags I would say just try to expose the parameters we have in the hardware

Re: [Mesa-dev] [PATCH 4/4] radeon/vce: handle newly added parameters

Am 24.06.2016 um 14:46 schrieb Ilia Mirkin: On Fri, Jun 24, 2016 at 8:39 AM, Christian König wrote: Ilia and/or maybe other nouveau developers: You guys don't have any plans to expose the NVidia encoding functionality through the OpenMAX/VA-API state trackers in the near future don't

Re: [Mesa-dev] [PATCH 0/3] st/omx: add initial support for nouveau

Reviewed-by: Christian König for the whole series. Regards, Christian. Am 27.06.2016 um 10:16 schrieb Julien Isorce: 3 patches I made in last january to allow running st/omx with the nouveau video driver. But currently the video is full of blockiness because nouveau driver requires manual

Re: [Mesa-dev] Fwd: [PATCH] st/vdpau: use bicubic filter for scaling

Hi guys, Nayan have you taken into account that the pixel center is at 0.5 and not 0.0? Regards, Christian. Am 26.06.2016 um 22:30 schrieb Andy Furniss: Nayan Deshmukh wrote: Hi Andy, On Sun, Jun 26, 2016 at 12:25 AM, Andy Furniss wrote: Nayan Deshmukh wrote: Hi Andy, Thanks for te

Re: [Mesa-dev] Fwd: [PATCH] st/vdpau: use bicubic filter for scaling

ing inspiration from the way matrix_filter uses offsets. Regards, Nayan. On Mon, Jun 27, 2016 at 6:55 PM, Christian König mailto:deathsim...@vodafone.de>> wrote: Hi guys, Nayan have you taken into account that the pixel center is at 0.5 and not 0.0? Regards, Christian

Re: [Mesa-dev] [PATCH 2/3] st/omx: add support for nouveau / interlaced

Hi Leo, nice catch patch is Reviewed-by: Christian König . But we still need to fix transcoding issue with interlaced as true. Our transcode support tunneling, basic the decode buffer will be used directly for encode. Ah, yes of course. Sorry I was a bit fast with giving my rb on that

Re: [Mesa-dev] [PATCH 1/2] vl: add a bicubic interpolation filter(v4)

Am 28.06.2016 um 11:25 schrieb Nayan Deshmukh: This is a shader based bicubic interpolater which uses cubic Hermite spline algorithm. v2: set dst_area and dst_clip during scaling (Christian) v3: clear the render target before rendering v4: intialize offsets while initializing shaders use a

Re: [Mesa-dev] Fwd: [PATCH] st/vdpau: use bicubic filter for scaling

Am 28.06.2016 um 20:11 schrieb Nayan Deshmukh: Hi Andy, Thanks for testing the patches. On Tue, Jun 28, 2016 at 11:26 PM, Andy Furniss > wrote: Nayan Deshmukh wrote: Hi Christian and Andy, I have sent new series of patches which takes care of

Re: [Mesa-dev] [PATCH 3/4] st/va: properly set max number of reference frames

On 19.11.2015 10:37, Julien Isorce wrote: It fixes asserts like assert(templ->max_references <= 2) in nvc0_video.c::nvc0_create_decodier This patch also post-update the correct value for the number of reference frames in the h264 case, see below. In VA-API the max num ref is retrieved later in h

Re: [Mesa-dev] [PATCH 1/2] radeon/vce: add new firmware interface support

On 20.11.2015 16:12, Leo Liu wrote: Add new interface to create and encode Signed-off-by: Leo Liu Reviewed-by: Alex Deucher Didn't had a chance to take a look before releasing them, so I thought I better do now. Both patches are Reviewed-by: Christian König --- src/ga

[Mesa-dev] [PATCH 3/8] st/va: move MPEG12 functions into separate file

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/Makefile.sources | 1 + src/gallium/state_trackers/va/picture.c| 57 +++--- src/gallium/state_trackers/va/picture_mpeg12.c | 80 ++ src/gallium/state_trackers/va

[Mesa-dev] [PATCH 5/8] st/va: move VC-1 functions into separate file

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/Makefile.sources | 1 + src/gallium/state_trackers/va/picture.c| 36 +- src/gallium/state_trackers/va/picture_vc1.c| 67 ++ src/gallium/state_trackers/va

[Mesa-dev] [PATCH 4/8] st/va: move H264 functions into separate file

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/Makefile.sources | 1 + src/gallium/state_trackers/va/picture.c| 76 + src/gallium/state_trackers/va/picture_h264.c | 113 + src/gallium/state_trackers/va

[Mesa-dev] [PATCH] st/va: disable MPEG4 by default

From: Christian König The workarounds are to hacky to enable them by default and otherwise MPEG4 doesn't work reliable. Signed-off-by: Christian König --- src/gallium/state_trackers/va/config.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/ga

[Mesa-dev] [PATCH 1/8] st/va: fix post process dirty area handling

From: Christian König The dirty area in this call isn't related to the screen at all. Signed-off-by: Christian König --- src/gallium/state_trackers/va/picture.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/va/picture.c b/src/ga

[Mesa-dev] [PATCH 7/8] st/va: move HEVC functions into separate file

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/Makefile.sources | 1 + src/gallium/state_trackers/va/picture.c| 174 + src/gallium/state_trackers/va/picture_hevc.c | 159 ++ src/gallium

[Mesa-dev] [PATCH 8/8] st/va: fix indentation

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/picture.c | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/gallium/state_trackers/va/picture.c b/src/gallium/state_trackers/va/picture.c index ce75e2d..25d2940 100644

[Mesa-dev] [PATCH 2/8] st/va: move post processing function into own file

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/Makefile.sources | 1 + src/gallium/state_trackers/va/picture.c| 58 +--- src/gallium/state_trackers/va/postproc.c | 94 ++ src/gallium/state_trackers/va

[Mesa-dev] [PATCH 6/8] st/va: move MPEG4 functions into separate file

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/Makefile.sources | 1 + src/gallium/state_trackers/va/picture.c| 186 +- src/gallium/state_trackers/va/picture_mpeg4.c | 209 + src/gallium

Re: [Mesa-dev] [PATCH v2 1/2] st/va: if h264 then delay decoder creation until max_references is known

On 25.11.2015 10:12, Julien Isorce wrote: For commit message please read: "HEVC case is left unchanged since delaying decoder creation is not needed on AMD hardware." In this case please update the commit message, but honestly I'm not sure if we don't use the max_references somewhere in the

Re: [Mesa-dev] [PATCH v2 2/2] st/va: also retrieve reference frames info for h264

On 25.11.2015 10:13, Julien Isorce wrote: From: Julien Isorce Other hardwares than AMD require to parse: VAPictureParameterBufferH264.ReferenceFrames[16] Signed-off-by: Julien Isorce Reviewed-by: Christian König --- src/gallium/state_trackers/va/picture_h264.c | 53

Re: [Mesa-dev] [PATCH] st/va: add missing break statement

Coverity (CID 1341052) Fixes: 64761a841db "st/va: move MPEG4 functions into separate file" Cc: Christian König Cc: Julien Isorce Signed-off-by: Emil Velikov Crap, stupid typo. Patch is Reviewed-by: Christian König --- src/gallium/state_trackers/va/picture.c | 1 + 1 file

Re: [Mesa-dev] [PATCH 1/2] st/va: ensure linear memory for dmabuf

On 29.11.2015 18:26, Emil Velikov wrote: On 27 November 2015 at 08:57, Julien Isorce wrote: In order to do zero-copy between two different devices the memory should not be tiled. This is currently no way to set pipe_resource template's flag from pipe_video_buffer template. So disabled_tiling i

Re: [Mesa-dev] [PATCH v3] st/va: delay decoder creation until max_references is known

HEVC case defaults to num_render_targets as before. But it could also benefits this change by setting a more accurate max_references number in handlePictureParameterBuffer. Signed-off-by: Julien Isorce Reviewed-by: Christian König --- src/gallium/state_trackers/va/context.c | 48

[Mesa-dev] [PATCH 2/2] st/va: disable MPEG4 by default v2

From: Christian König The workarounds are too hacky to enable them by default and otherwise MPEG4 doesn't work reliably. v2: add docs/envvars.html, CC stable and fix typos Signed-off-by: Christian König Reviewed-by: Emil Velikov (v1) Reviewed-by: Ilia Mirkin (v1) Cc: "11.1.0&q

[Mesa-dev] [PATCH 1/2] st/va: move HEVC functions into separate file v2

From: Christian König v2: actually copy all of it Signed-off-by: Christian König --- src/gallium/state_trackers/va/Makefile.sources | 1 + src/gallium/state_trackers/va/picture.c| 172 +--- src/gallium/state_trackers/va/picture_hevc.c | 207

[Mesa-dev] [PATCH] st/va: WIP remove nonesense HEVC picture id handling

From: Christian König The picture id in this case is a VA-API surface handle, checking for a certain value is can't be correct. Does anybody have any idea what the original intention was and how to fix it correctly? Signed-off-by: Christian König --- src/gallium/state_tracke

[Mesa-dev] [PATCH 6/8] st/va: remove fence handling

From: Christian König It's nonsense to drain the pipeline like this. Signed-off-by: Christian König --- src/gallium/state_trackers/va/buffer.c | 5 - src/gallium/state_trackers/va/image.c | 1 - src/gallium/state_trackers/va/postproc.c | 6 -- src/gallium/state_tra

[Mesa-dev] [PATCH 8/8] st/va: add BOB/WEAVE deinterlacing

From: Christian König Tested with MPV. Signed-off-by: Christian König --- src/gallium/state_trackers/va/postproc.c | 72 src/gallium/state_trackers/va/surface.c | 16 ++- 2 files changed, 78 insertions(+), 10 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH 1/8] st/va: remove nonesense HEVC picture id handling

From: Christian König The picture id in this case is a VA-API surface handle, checking for a certain value can't be correct. Signed-off-by: Christian König --- src/gallium/state_trackers/va/picture_hevc.c | 5 - 1 file changed, 5 deletions(-) diff --git a/src/gallium/state_tracke

[Mesa-dev] [PATCH 3/8] st/va: clean up post process includes

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/postproc.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/src/gallium/state_trackers/va/postproc.c b/src/gallium/state_trackers/va/postproc.c index 1fdb4e7..2d17694 100644 --- a/src/gallium

[Mesa-dev] [PATCH 4/8] st/va: fix unused variable warning

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/picture.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/state_trackers/va/picture.c b/src/gallium/state_trackers/va/picture.c index 8623139..7b30bf8 100644 --- a/src/gallium/state_trackers

[Mesa-dev] [PATCH 2/8] st/va: cleanup filter color standard handling

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/surface.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/va/surface.c b/src/gallium/state_trackers/va/surface.c index c052c8f..4a18a6f 100644 --- a

[Mesa-dev] [PATCH 7/8] st/va: add NV12 -> NV12 post processing

From: Christian König Usefull for mpv and GStreamer. Signed-off-by: Indrajit-kumar Das Signed-off-by: Christian König --- src/gallium/state_trackers/va/picture.c | 10 +- src/gallium/state_trackers/va/postproc.c | 152 --- 2 files changed, 125 insertions(+), 37

[Mesa-dev] [PATCH 5/8] st/va: handle default post process regions

From: Christian König Avoid referencing NULL pointers. Signed-off-by: Christian König --- src/gallium/state_trackers/va/postproc.c | 36 +--- 1 file changed, 28 insertions(+), 8 deletions(-) diff --git a/src/gallium/state_trackers/va/postproc.c b/src/gallium

Re: [Mesa-dev] [PATCH 1/8] st/va: remove nonesense HEVC picture id handling

gs & VA_PICTURE_HEVC_INVALID || hevc->ReferenceFrames[i].picture_id == VA_INVALID_SURFACE ? On 11 December 2015 at 12:33, Christian König <mailto:deathsim...@vodafone.de>> wrote: From: Christian König mailto:christian.koe...@amd.com>> The picture id in this case is a

Re: [Mesa-dev] [PATCH 6/8] st/va: remove fence handling

his pipeline, HW decoding is done with nouveau driver and rendering is done with intel. dmabuf in between. Maybe the idea of the patch is good but something is still wrong. I can test any update if it helps. Cheers Julien On 11 December 2015 at 12:33, Christian König <mailto:deathsi

Re: [Mesa-dev] [PATCH 6/8] st/va: remove fence handling

(the one that vpp output), not the NV12 one that come from the decoder directly. Cheers Julien On 14 December 2015 at 10:11, Christian König mailto:deathsim...@vodafone.de>> wrote: Also note that in this pipeline, HW decoding is done with nouveau dri

Re: [Mesa-dev] [PATCH 2/8] st/va: cleanup filter color standard handling

On 15.12.2015 11:06, Emil Velikov wrote: On 11 December 2015 at 12:33, Christian König wrote: From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/surface.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium

Re: [Mesa-dev] [PATCH 4/8] st/va: fix unused variable warning

On 15.12.2015 11:08, Emil Velikov wrote: On 11 December 2015 at 12:33, Christian König wrote: From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/picture.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/state_trackers/va/picture.c b/src

Re: [Mesa-dev] [PATCH 5/8] st/va: handle default post process regions

On 15.12.2015 11:11, Emil Velikov wrote: On 11 December 2015 at 12:33, Christian König wrote: From: Christian König Avoid referencing NULL pointers. Lacking any prior knowledge of the sequential patches, I'm afraid this commit message doesn't make any sense. How about "Will

Re: [Mesa-dev] [Libva] [PATCH] st/va: retrieve size from the temporary img variable

Yeah, agree reviewing them on the mesa list is probably more appropriate. Anyway patch lgtm as well and is Reviewed-by: Christian König Regards, Christian. On 15.12.2015 18:53, Julien Isorce wrote: + mesa-dev of course :) Thx Sean On 15 December 2015 at 17:46, Sean V Kelley <mailto:

[Mesa-dev] [PATCH 1/9] st/va: make the implementation thread save

From: Christian König Otherwise we might crash with MPV. Signed-off-by: Christian König --- src/gallium/state_trackers/va/buffer.c | 69 +- src/gallium/state_trackers/va/context.c| 6 +++ src/gallium/state_trackers/va/image.c | 68

[Mesa-dev] [PATCH 8/9] vl: use preferred format for deinterlacing

From: Christian König Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_deint_filter.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/vl/vl_deint_filter.c b/src/gallium/auxiliary/vl/vl_deint_filter.c index f919867..9e782e5 100644

[Mesa-dev] [PATCH 6/9] st/va: add BOB deinterlacing v2

From: Christian König Tested with MPV. v2: correctly handle compositor deinterlacing as well. Signed-off-by: Christian König --- src/gallium/state_trackers/va/postproc.c | 74 src/gallium/state_trackers/va/surface.c | 16 ++- 2 files changed, 79

[Mesa-dev] [PATCH 9/9] st/va: add motion adaptive deinterlacing

From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/va/context.c| 5 +++ src/gallium/state_trackers/va/postproc.c | 63 -- src/gallium/state_trackers/va/surface.c| 22 +-- src/gallium/state_trackers/va/va_private.h

[Mesa-dev] [PATCH 5/9] st/va: add NV12 -> NV12 post processing v2

From: Christian König Usefull for mpv and GStreamer. v2: use common functionality for size adjustment. Signed-off-by: Indrajit-kumar Das Signed-off-by: Christian König --- src/gallium/state_trackers/va/picture.c | 10 +- src/gallium/state_trackers/va/postproc.c | 151

[Mesa-dev] [PATCH 7/9] vl: improve motion adaptive deinterlacer

From: Christian König Handle other formats than YV12 as well. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_deint_filter.c | 69 +- src/gallium/auxiliary/vl/vl_deint_filter.h | 2 +- 2 files changed, 49 insertions(+), 22 deletions(-) diff --git a

[Mesa-dev] [PATCH 4/9] st/va: use vl_video_buffer_adjust_size

From: Christian König Use the new helper function instead of open coding it. Signed-off-by: Christian König --- src/gallium/state_trackers/va/image.c | 13 - 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/src/gallium/state_trackers/va/image.c b/src/gallium

[Mesa-dev] [PATCH 3/9] st/vdpau: use vl_video_buffer_adjust_size

From: Christian König Use the new helper function instead of open coding it. Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/surface.c | 13 +++-- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/src/gallium/state_trackers/vdpau/surface.c b/src

[Mesa-dev] [PATCH 2/9] vl/buffers: extract vl_video_buffer_adjust_size helper

From: Christian König Useful for the state trackers as well. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_video_buffer.c | 10 ++ src/gallium/auxiliary/vl/vl_video_buffer.h | 18 ++ 2 files changed, 20 insertions(+), 8 deletions(-) diff --git a/src

Re: [Mesa-dev] [PATCH 7/8] st/va: add NV12 -> NV12 post processing

On 15.12.2015 11:16, Emil Velikov wrote: On 11 December 2015 at 12:33, Christian König wrote: +static void vlVaBoxAdjust(struct pipe_video_buffer *buf, unsigned idx, + struct pipe_box *box) +{ + if (buf->interlaced) { + box->y /= 2; + box->he

Re: [Mesa-dev] [PATCH 1/9] st/va: make the implementation thread save

On 21.12.2015 11:05, Andy Furniss wrote: Julien Isorce wrote: Hi Christian, I have 2 remarks: 1: I am sure you have the clear answer to the following questions, so maybe add a comment in the commit message: Isn't the thread safety delegated to the application that use vaapi ? like it is design

Re: [Mesa-dev] [PATCH] st/va: count number of slices

On 23.12.2015 10:28, Julien Isorce wrote: The counter was not set but used by the driver. It is required otherwise visual output is garbage. Signed-off-by: Julien Isorce One minor nit pick below, apart from that the patch look good to me. --- src/gallium/state_trackers/va/picture.c

Re: [Mesa-dev] [PATCH v4] winsys/radeon: fix nop packet padding for hawaii

Am 04.08.2014 um 12:48 schrieb Andreas Boll: The initial firmware for hawaii does not support type3 nop packet. Detect the new hawaii firmware with query RADEON_INFO_ACCEL_WORKING2. If the returned value is 3, then the new firmware is used. This patch uses type2 for the old firmware and type3 fo

Re: [Mesa-dev] [PATCH v4] winsys/radeon: fix nop packet padding for hawaii

Am 04.08.2014 um 12:48 schrieb Andreas Boll: The initial firmware for hawaii does not support type3 nop packet. Detect the new hawaii firmware with query RADEON_INFO_ACCEL_WORKING2. If the returned value is 3, then the new firmware is used. This patch uses type2 for the old firmware and type3 fo

[Mesa-dev] [PATCH 4/4] radeon: cache the last used userptr

From: Christian König Signed-off-by: Christian König --- src/gallium/drivers/radeon/r600_pipe_common.c | 9 ++ src/gallium/drivers/radeon/r600_pipe_common.h | 11 +++ src/gallium/drivers/radeon/r600_texture.c | 41 +-- 3 files changed, 59 insertions(+), 2

[Mesa-dev] [PATCH 1/4] radeonsi: implement partial DMA copies v2

From: Christian König v2: fix a couple of typos and bugs Signed-off-by: Christian König --- src/gallium/drivers/radeonsi/si_dma.c | 85 +++ src/gallium/drivers/radeonsi/sid.h| 1 + 2 files changed, 68 insertions(+), 18 deletions(-) diff --git a/src

[Mesa-dev] [PATCH 2/4] winsys/radeon: add user pointer support

From: Christian König Signed-off-by: Christian König --- src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 102 ++ src/gallium/winsys/radeon/drm/radeon_winsys.h | 11 +++ 2 files changed, 113 insertions(+) diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_bo.c b

[Mesa-dev] [PATCH 3/4] radeon: accelerate transfer_inline_write

From: Christian König Not completely implemented, cause we need DMA copy support for every hw generation. Signed-off-by: Christian König --- src/gallium/drivers/radeon/r600_buffer_common.c | 2 +- src/gallium/drivers/radeon/r600_pipe_common.c | 2 +- src/gallium/drivers/radeon

Re: [Mesa-dev] [PATCH 4/4] radeon: cache the last used userptr

:24 schrieb Marek Olšák: What is this patch good for? Marek On Tue, Aug 5, 2014 at 7:31 PM, Christian König wrote: From: Christian König Signed-off-by: Christian König --- src/gallium/drivers/radeon/r600_pipe_common.c | 9 ++ src/gallium/drivers/radeon/r600_pipe_common.h | 11

Re: [Mesa-dev] [PATCH 1/4] radeonsi: implement partial DMA copies v2

mpositing that with subtitles. Seems to provide a quite good way of providing all kind of different texture and rectangle sizes (because of the different letters in the subtitle). It took me a while to get everything working. Christian. Marek On Tue, Aug 5, 2014 at 7:31 PM, Christian Kön

Re: [Mesa-dev] [PATCH 3/4] radeon: accelerate transfer_inline_write

Am 06.08.2014 um 13:45 schrieb Marek Olšák: On Tue, Aug 5, 2014 at 7:31 PM, Christian König wrote: From: Christian König Not completely implemented, cause we need DMA copy support for every hw generation. Signed-off-by: Christian König --- src/gallium/drivers/radeon/r600_buffer_common.c

Re: [Mesa-dev] [PATCH 1/6] r600g: remove useless r600_resource_va calls

Am 06.08.2014 um 23:49 schrieb Marek Olšák: From: Marek Olšák Wanted to do this for a while as well. The whole series is: Reviewed-by: Christian König R600-R700 don't support virtual memory. --- src/gallium/drivers/r600/r600_state.c | 27 +-- 1 file chang

[Mesa-dev] [PATCH] winsys/radeon: use zero sized buffers for fences if the kernel supports it

From: Christian König Signed-off-by: Christian König --- src/gallium/auxiliary/pipebuffer/pb_buffer.h | 2 +- src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 2 +- src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 3 ++- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/gallium

[Mesa-dev] [PATCH] st/vdpau: add device reference counting

From: Christian König This fixes an issue with flash where it tries to destroy a decoder after already destroying the device associated with the decoder. Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=82517 Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/bitmap.c

Re: [Mesa-dev] [PATCH] st/vdpau: add device reference counting

Am 13.08.2014 um 17:13 schrieb Ilia Mirkin: On Wed, Aug 13, 2014 at 11:07 AM, Christian König wrote: From: Christian König This fixes an issue with flash where it tries to destroy a decoder after already destroying the device associated with the decoder. Fixes: https://bugs.freedesktop.org

[Mesa-dev] [PATCH 2/2] vl/compositor: set the scissor before clearing the render target

From: Christian König Otherwise we clear areas that shouldn't be cleared. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_compositor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/vl/vl_compositor.c b/src/gallium/auxilia

[Mesa-dev] [PATCH 1/2] st/vdpau: fix vlVdpOutputSurfaceRender(Output|Bitmap)Surface

From: Christian König Correctly handle that the source_surface is only optional. Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/device.c| 43 +++- src/gallium/state_trackers/vdpau/output.c| 42 +++ src/gallium

Re: [Mesa-dev] [PATCH 16/22] st/vdpau: remove obsolete define VL_HANDLES

Am 19.08.2014 um 01:20 schrieb Emil Velikov: This define is always set and it had no real purpose according to git log. Seems like it is a leftover from the vl/vdpau prototype stage. Cc: Christian König Signed-off-by: Emil Velikov Reviewed-by: Christian König --- src/gallium

Re: [Mesa-dev] [PATCH 17/22] st/vdpau: pickup/ship the private header

Am 19.08.2014 um 01:20 schrieb Emil Velikov: Signed-off-by: Emil Velikov Is it necessary to sort the files by name? They where more or less sorted by importance and time of creation in the VDPAU interface. Christian. --- src/gallium/state_trackers/vdpau/Makefile.sources | 17 +--

Re: [Mesa-dev] [PATCH 18/22] st/omx: use makefile.sources to handle sources lists

Am 19.08.2014 um 01:20 schrieb Emil Velikov: ... and add the headers so that 'make check' is happy. Cc: Christian König Signed-off-by: Emil Velikov Reviewed-by: Christian König --- src/gallium/state_trackers/omx/Makefile.am | 8 ++-- src/gallium/state_tr

Re: [Mesa-dev] [PATCH 17/22] st/vdpau: pickup/ship the private header

Am 19.08.2014 um 11:09 schrieb Emil Velikov: On 19/08/14 09:59, Christian König wrote: Am 19.08.2014 um 01:20 schrieb Emil Velikov: Signed-off-by: Emil Velikov Is it necessary to sort the files by name? They where more or less sorted by importance and time of creation in the VDPAU interface

Re: [Mesa-dev] [RFC PATCH 00/16] A new IR for Mesa

I think we can fix this by introducing new structured variants of the branch instruction in a way that doesn't alter the fundamental structure of the IR. E.g. an if branch could look like: ifbr i1 , label , label , label Where both branches are guaranteed to converge at . Sure, this will requ

Re: [Mesa-dev] [RFC PATCH 00/16] A new IR for Mesa

Am 20.08.2014 um 14:33 schrieb Connor Abbott: On Tue, Aug 19, 2014 at 11:57 PM, Christian König wrote: I think we can fix this by introducing new structured variants of the branch instruction in a way that doesn't alter the fundamental structure of the IR. E.g. an if branch could look

Re: [Mesa-dev] [RFC] gallium: add support for rectangle primitives

ds it might actually worth it. Anyway the patch itself looks good and is Reviewed-by: Christian König --- src/gallium/auxiliary/tgsi/tgsi_strings.c | 3 ++- src/gallium/auxiliary/util/u_prim.h | 1 + src/gallium/docs/source/screen.rst| 6 ++ src/gallium/drivers

Re: [Mesa-dev] [RFC PATCH 00/16] A new IR for Mesa

Am 22.08.2014 um 17:13 schrieb Connor Abbott: On Thu, Aug 21, 2014 at 11:08 PM, Dave Airlie wrote: On 22 August 2014 12:46, Jason Ekstrand wrote: On Thu, Aug 21, 2014 at 7:36 PM, Dave Airlie wrote: On 21 August 2014 19:10, Henri Verbeet wrote: On 21 August 2014 04:56, Michel Dänzer wrote

Re: [Mesa-dev] [RFC PATCH 00/16] A new IR for Mesa

Am 22.08.2014 um 18:01 schrieb Connor Abbott: On Fri, Aug 22, 2014 at 11:27 AM, Christian König wrote: Am 22.08.2014 um 17:13 schrieb Connor Abbott: On Thu, Aug 21, 2014 at 11:08 PM, Dave Airlie wrote: On 22 August 2014 12:46, Jason Ekstrand wrote: On Thu, Aug 21, 2014 at 7:36 PM, Dave

[Mesa-dev] [PATCH] radeon/uvd: fix field handling on R6XX style UVD

From: Christian König The first UVD generation can only do frame based output. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_video.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/radeon_video.c b/src/gallium

Re: [Mesa-dev] [RFC PATCH 00/16] A new IR for Mesa

Am 26.08.2014 um 03:54 schrieb Rob Clark: On Wed, Aug 20, 2014 at 2:13 PM, Kenneth Graunke wrote: we've already had problems where distros refused to ship newer Mesa releases because radeon depended on a version of LLVM newer than the one they were shipping, [...] That's news to me, can you be

Re: [Mesa-dev] [RFC PATCH 00/16] A new IR for Mesa

Am 26.08.2014 um 18:00 schrieb Jose Fonseca: On 26/08/14 10:04, Christian König wrote: Am 26.08.2014 um 03:54 schrieb Rob Clark: On Wed, Aug 20, 2014 at 2:13 PM, Kenneth Graunke wrote: we've already had problems where distros refused to ship newer Mesa releases because radeon depended

Re: [Mesa-dev] [PATCH 1/2] st/vdpau: fix vlVdpOutputSurfaceRender(Output|Bitmap)Surface

Am 26.08.2014 um 18:50 schrieb Emil Velikov: On 15/08/14 19:33, Emil Velikov wrote: On 14/08/14 10:59, Christian König wrote: From: Christian König Correctly handle that the source_surface is only optional. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=80561 Cc: mesa-sta

Re: [Mesa-dev] [PATCH] radeon/uvd: remove comment about RV770

Am 27.08.2014 um 05:09 schrieb Alex Deucher: It doesn't seem to support field based decode after testing. Signed-off-by: Alex Deucher Reviewed-by: Christian König --- src/gallium/drivers/radeon/radeon_video.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/drivers/r

Re: [Mesa-dev] [PATCH] drm/radeon: Add RADEON_GEM_CPU_ACCESS BO creation flag

Reviewed-by: Christian König I think we need a similar negative flags as well, e.g. RADEON_GEM_NO_CPU_ACCESS. This way we can stop forcing buffers into the visible VRAM while pinning them for scanout. Regards, Christian. --- drivers/gpu/drm/radeon/radeon_object.c | 11 +-- include

Re: [Mesa-dev] [RFC PATCH 00/16] A new IR for Mesa

At least with the other components on which Mesa relies (e.g., libdrm, 2D drivers, etc.) it's largely the same group of people with the same set of goals. This was only true until Tom Stellard started to manage LLVM point releases. Christian. Am 28.08.2014 um 00:07 schrieb Ian Romanick: On 08

Re: [Mesa-dev] [PATCH 2/3] radeonsi: add sampling of 4:2:2 subsampled textures

Am 29.08.2014 um 01:54 schrieb Grigori Goronzy: On 04.07.2014 01:24, Andy Furniss wrote: Maybe not 1/frame but anyway the first couple of a run have numbers rather than s [27977.386795] radeon :01:00.0: GPU fault detected: 146 0x0c035014 [27977.386800] radeon :01:00.0: VM_CONTEXT1

[Mesa-dev] [PATCH] mesa/st: don't advertise NV_vdpau_interop if it doesn't work.

From: Christian König As long as we don't have a workaround for field based decoding in VDPAu we should not advertise NV_vdpau_interop. Signed-off-by: Christian König Cc: mesa-sta...@lists.freedesktop.org --- src/mesa/state_tracker/st_extensions.c | 7 ++- 1 file changed, 6 inser

Re: [Mesa-dev] [RFC PATCH 00/16] A new IR for Mesa

Maybe a heretical thought, but how close are you guys to shortcut the front end compiler with let's say the LLVM back end in radeonsi? Regards, Christian. Am 29.08.2014 um 17:51 schrieb Greg Fischer: Some additional information on GlassyMesa from the engineer who integrated LunarGlass and wrot

Re: [Mesa-dev] [PATCH] vdpau: unlock the mutex on error paths in attribute setting.

Am 02.09.2014 um 01:00 schrieb Dave Airlie: From: Dave Airlie Coverity pointed out we never dropped the lock here, so fix it by using a common exit path. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- src/gallium/state_trackers/vdpau/mixer.c | 38

Re: [Mesa-dev] [PATCH] omx/h264: remove stray semicolon after if

Am 02.09.2014 um 01:40 schrieb Dave Airlie: From: Dave Airlie Coverity reported this, looks wrong to me. And is probably one of the reasons why streams with multiple slices in one frame didn't worked. Signed-off-by: Dave Airlie Reviewed-by: Christian König --- src/ga

[Mesa-dev] [PATCH] mesa/st: don't advertise NV_vdpau_interop if it doesn't work.

From: Christian König As long as we don't have a workaround for frame based decoding in VDPAU we should not advertise NV_vdpau_interop. v2: fix commit message, check if get_video_param is present Signed-off-by: Christian König Cc: mesa-sta...@lists.freedesktop.org --- src/mesa/state_tr

[Mesa-dev] [PATCH] radeon/video: use more common buffer infrastructure

From: Christian König This allows us to clear the video buffers using the gfx engine(s). Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c| 46 +++--- src/gallium/drivers/radeon/radeon_vce.c| 17 + src/gallium/drivers/radeon

[Mesa-dev] [PATCH 3/3] radeon/uvd: use PIPE_USAGE_STAGING for msg&fb buffers

From: Christian König That better matches the actual userspace use case, the kernel will force it to VRAM if the hardware requires it. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium

[Mesa-dev] [PATCH 1/3] radeon/video: use more of the common buffer code v2

From: Christian König In preparation to using buffers clears with the hw engine(s). v2: split out flipping to using hw buffer clears. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c| 40 ++- src/gallium/drivers/radeon/radeon_vce.c

[Mesa-dev] [PATCH 2/3] radeon/video: use the hw to initial clear the buffers

From: Christian König Less CPU overhead and avoids contention over CPU accessible memory on startup. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c | 6 +++--- src/gallium/drivers/radeon/radeon_video.c | 10 -- src/gallium/drivers/radeon/radeon_video.h

[Mesa-dev] [PATCH 2/3] radeon/winsys: keep track of the last CS a BO was used in

From: Christian König Signed-off-by: Christian König --- src/gallium/winsys/radeon/drm/radeon_drm_bo.h | 3 +++ src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 11 +-- src/gallium/winsys/radeon/drm/radeon_drm_cs.h | 2 +- 3 files changed, 13 insertions(+), 3 deletions(-) diff --git

<    5   6   7   8   9   10   11   12   13   14   >