[Mesa-dev] [PATCH 1/4] state_trackers/vdpau: Implement VdpGenerateCSCMatrix

2011-10-31 Thread Maarten Lankhorst
With the smpte240 profile, which was missing. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/auxiliary/vl/vl_csc.c| 19 +++ src/gallium/auxiliary/vl/vl_csc.h|3 ++- src/gallium/state_trackers/vdpau/mixer.c | 20

[Mesa-dev] [PATCH 02/04] state_trackers/vdpau: Add num_slices to mpeg12 picture structure

2011-10-31 Thread Maarten Lankhorst
Bitstream parsers might need that field. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/include/pipe/p_video_state.h |1 + src/gallium/state_trackers/vdpau/decode.c |1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/gallium/include/pipe

[Mesa-dev] [PATCH 03/04] state_trackers/vdpau: Test if profile is supported first before trying to create decoder

2011-10-31 Thread Maarten Lankhorst
So a nicer error message is returned. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/vdpau/decode.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/src/gallium/state_trackers/vdpau/decode.c b/src/gallium

[Mesa-dev] [PATCH 04/04] state_trackers/vdpau: Add support for MPEG4 Part 2

2011-10-31 Thread Maarten Lankhorst
Just the support patch, no decoder implements it currently. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- C König could you check if it doesn't break mpeg1? I can't confirm the visual because shaders seem bugged for me. src/gallium/include/pipe/p_video_state.h | 27

[Mesa-dev] [PATCH 05/04] state_trackers/vdpau: Add mpeg4 part2 to PipeToProfile and ProfileToPipe

2011-10-31 Thread Maarten Lankhorst
So it can actually be used when someone implements it. :) Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/vdpau/vdpau_private.h |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/src/gallium/state_trackers/vdpau

[Mesa-dev] [PATCH 06/04] state_trackers/vdpau: Add support for VC-1 decoding

2011-10-31 Thread Maarten Lankhorst
Add a struct with all the fields. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- Just because vdpau state tracker supports it, doesn't mean anything uses it yet. Patience.. src/gallium/include/pipe/p_video_state.h | 34 +++ src/gallium/state_trackers/vdpau

[Mesa-dev] reworking pipe_video_decoder / pipe_video_buffer

2011-11-15 Thread Maarten Lankhorst
Hey all, I'm convinced that right now the pipe_video_decoder and pipe_video_buffer are unnecessarily complicated, and require simplification. struct pipe_video_decoder { struct pipe_context *context; enum pipe_video_profile profile; enum pipe_video_entrypoint entrypoint; enum

[Mesa-dev] [PATCH 1/6] pipe_video: Add more mpeg1 fields to pipe_mpeg12_picture_desc

2011-11-16 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/include/pipe/p_video_state.h |3 +++ src/gallium/state_trackers/vdpau/decode.c |3 +++ 2 files changed, 6 insertions(+), 0 deletions(-) diff --git a/src/gallium/include/pipe/p_video_state.h b/src/gallium/include

[Mesa-dev] [PATCH 2/6] g3dvl: Remove get_sampler_view_planes, and replace with get/put bits

2011-11-16 Thread Maarten Lankhorst
Doesn't break nouveau_video, apparently a check already causes it to use vl_video_buffer for vdpau. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 15 +++-- src/gallium/auxiliary/vl/vl_video_buffer.c | 92

[Mesa-dev] [PATCH 3/6] pipe_video: Get rid of a few pipe_video_decoder members

2011-11-16 Thread Maarten Lankhorst
You won't be missed, functionality folded into picture_desc and decode.. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/auxiliary/vl/vl_mpeg12_bitstream.c |4 +- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 68 --- src/gallium/drivers/nouveau

[Mesa-dev] [PATCH 4/6] xorg/xvmc: Remove recursion from RecursiveEndFrame

2011-11-16 Thread Maarten Lankhorst
There should be no way for endframe to ever need recursion.. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/xorg/xvmc/surface.c | 23 +-- 1 files changed, 13 insertions(+), 10 deletions(-) diff --git a/src/gallium/state_trackers

[Mesa-dev] [PATCH 5/6] pipe_video: Get rid of unnecessary state tracking

2011-11-16 Thread Maarten Lankhorst
destroying a surface in XvMC, otherwise there exists the possibility that the decoder still held a reference to it. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- begin_frame was moved, end_frame was renamed to flush for vl_mpeg12_decoder (with some checks added) src/gallium/auxiliary/vl

[Mesa-dev] [PATCH 6/6] pipe_video: Remove set_picture_parameters

2011-11-16 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/auxiliary/vl/vl_mpeg12_bitstream.c |4 +- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 12 +- src/gallium/drivers/nouveau/nouveau_video.c| 25 +-- src/gallium/include/pipe/p_video_decoder.h |9

[Mesa-dev] [PATCH 7/6] pipe_video: remove PIPE_VIDEO_CAP_NUM_BUFFERS_DESIRED

2011-11-16 Thread Maarten Lankhorst
This is no longer used with the removal of decoder buffers. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/auxiliary/vl/vl_decoder.c| 13 - src/gallium/auxiliary/vl/vl_decoder.h|6 -- src/gallium/drivers/nouveau

Re: [Mesa-dev] reworking pipe_video_decoder / pipe_video_buffer

2011-11-16 Thread Maarten Lankhorst
Hey, On 11/16/2011 02:47 PM, Christian König wrote: On 15.11.2011 17:52, Maarten Lankhorst wrote: Deleted: - begin_frame/end_frame: Was only useful for XvMC, should be folded into flush.. I'm not completely happy with the current interface also, but if you remove the state tracker

Re: [Mesa-dev] reworking pipe_video_decoder / pipe_video_buffer

2011-11-22 Thread Maarten Lankhorst
On 11/22/2011 10:00 PM, Younes Manton wrote: 2011/11/21 Christian König deathsim...@vodafone.de: On 16.11.2011 15:38, Maarten Lankhorst wrote: If the decode_bitstream interface is changed to get all bitstream buffers at the same time, there wouldn't be overhead to doing it like

Re: [Mesa-dev] reworking pipe_video_decoder / pipe_video_buffer

2011-11-25 Thread Maarten Lankhorst
Hey Christian, On 11/21/2011 05:49 PM, Christian König wrote: On 16.11.2011 15:38, Maarten Lankhorst wrote: Hey, On 11/16/2011 02:47 PM, Christian König wrote: On 15.11.2011 17:52, Maarten Lankhorst wrote: Deleted: - begin_frame/end_frame: Was only useful for XvMC, should be folded

Re: [Mesa-dev] reworking pipe_video_decoder / pipe_video_buffer

2011-11-26 Thread Maarten Lankhorst
Hey Andy, On 11/26/2011 01:44 PM, Andy Furniss wrote: Maarten Lankhorst wrote: Testing interlaced videos that decode correctly with nvidia vdpau would help a lot to figure out what the proper way to handle interlacing would be, so if someone has a bunch that play with nvidia accelerated

[Mesa-dev] [PATCH 1/6] vl: Add missing mpeg fields to pipe_mpeg12_picture_desc

2011-12-01 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/include/pipe/p_video_state.h |3 +++ src/gallium/state_trackers/vdpau/decode.c |3 +++ 2 files changed, 6 insertions(+), 0 deletions(-) diff --git a/src/gallium/include/pipe/p_video_state.h b/src/gallium/include

[Mesa-dev] [PATCH 2/6] vl: Get rid of video_buffer.sampler_view_components

2011-12-01 Thread Maarten Lankhorst
No point in having it when just having sampler_view_planes is good enough. :-) Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/auxiliary/vl/vl_compositor.c| 36 ++- src/gallium/auxiliary/vl/vl_compositor.h|2 +- src/gallium/auxiliary/vl

[Mesa-dev] [PATCH 3/6] nvc0: Add support for autobinding

2011-12-01 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/drivers/nvc0/nvc0_winsys.h | 19 --- 1 files changed, 12 insertions(+), 7 deletions(-) diff --git a/src/gallium/drivers/nvc0/nvc0_winsys.h b/src/gallium/drivers/nvc0/nvc0_winsys.h index 0fcf994..dd4c887

[Mesa-dev] [PATCH 4/6] nvc0: Add NVC0_RESOURCE_FLAG_VIDEO

2011-12-01 Thread Maarten Lankhorst
Video buffers require a specific tiling mode. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/drivers/nvc0/nvc0_miptree.c |2 ++ src/gallium/drivers/nvc0/nvc0_resource.h |1 + 2 files changed, 3 insertions(+), 0 deletions(-) diff --git a/src/gallium/drivers

[Mesa-dev] [PATCH 5/6] vl: Remove most members of pipe_video_decoder

2011-12-01 Thread Maarten Lankhorst
should instead be passed as argument to decode_(bitstream,macroblocks). flush is used to signal in XvMC that current frame has ended. begin_frame and end_frame are moved into vl_mpeg12_decoder internally. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/auxiliary/vl

[Mesa-dev] [PATCH 6/6] vl: Add support for h264

2011-12-01 Thread Maarten Lankhorst
And the reason why I needed quant/reference_frames calls gone.. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/include/pipe/p_video_state.h | 47 + src/gallium/state_trackers/vdpau/decode.c | 63 + 2 files changed

[Mesa-dev] [PATCH 2/2] vdpau: Handle destination rectangles correctly

2011-12-02 Thread Maarten Lankhorst
The brokenness in vlVdpVideoMixerRender was compensating for brokenness in vlVdpPresentationQueueDisplay, so fix both at the same time. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/vdpau/mixer.c| 22 +- src/gallium

[Mesa-dev] [PATCH 2/6 v2] g3dvl: Get rid of video_buffer.sampler_view_components

2011-12-07 Thread Maarten Lankhorst
No point in having it when just having sampler_view_planes is good enough. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- Changes since v1: - increase nouveau_video_buffer sampler_view_planes array size to 3, which is the only valid size, since all planes are tested against null

Re: [Mesa-dev] [PATCH 5/6] vl: Remove most members of pipe_video_decoder

2011-12-07 Thread Maarten Lankhorst
Hey Andy, On 12/06/2011 10:54 PM, Andy Furniss wrote: Maarten Lankhorst wrote: create_buffer, destroy_buffer and set_buffer are a curiosity of vl_mpeg12_decoder and shouldn't be part of the api. set_quant_matrix and set_reference_frames shouldn't be separate calls, but part of picparm

Re: [Mesa-dev] [PATCH 2/6] vl: Get rid of video_buffer.sampler_view_components

2011-12-07 Thread Maarten Lankhorst
Hey Andy, On 12/07/2011 05:37 PM, Andy Furniss wrote: Maarten Lankhorst wrote: Hey Andy, On 12/06/2011 10:45 PM, Andy Furniss wrote: Maarten Lankhorst wrote: No point in having it when just having sampler_view_planes is good enough. :-) This patch causes R600 xvmc to render incorrect

[Mesa-dev] [PATCH 2/6 v3] g3dvl: Get rid of video_buffer.sampler_view_components

2011-12-08 Thread Maarten Lankhorst
No point in having it when just having sampler_view_planes is good enough. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- Changes since v2: - fixed borked ureg_MOV, I have no idea how it worked when I tested this on r600 to begin with, it renders correctly now with XvMC

Re: [Mesa-dev] [PATCH 2/6 v3] g3dvl: Get rid of video_buffer.sampler_view_components

2011-12-08 Thread Maarten Lankhorst
Hey Andy, On 12/08/2011 01:24 PM, Andy Furniss wrote: Maarten Lankhorst wrote: No point in having it when just having sampler_view_planes is good enough. Signed-off-by: Maarten Lankhorstm.b.lankho...@gmail.com --- Changes since v2: - fixed borked ureg_MOV, I have no idea how it worked

Re: [Mesa-dev] [PATCH 5/6] vl: Remove most members of pipe_video_decoder

2011-12-18 Thread Maarten Lankhorst
Hey Andy, 2011/12/19 Andy Furniss andy...@ukfsn.org: Maarten Lankhorst wrote: Hey Andy, On 12/07/2011 05:48 PM, Andy Furniss wrote: Maarten Lankhorst wrote: Hm, could you test with some added sanity checks? mplayer: vl/vl_vlc.h:139: vl_vlc_eatbits: Assertion `vlc-valid_bits  num_bits

[Mesa-dev] [PATCH] nouveau: Fix bugs in nouveau_video_buffer

2011-12-19 Thread Maarten Lankhorst
Double free and array overflow, even if only 2 members are used the last one needs to be set to NULL explicitly. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/drivers/nouveau/nouveau_video.c |1 - src/gallium/drivers/nouveau/nouveau_video.h |6 +++--- 2 files

[Mesa-dev] [PATCH 1/3] vdpau: Add support for parameters

2011-12-19 Thread Maarten Lankhorst
Currently only validating, since nothing else can be done with it yet Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/vdpau/mixer.c | 95 +- src/gallium/state_trackers/vdpau/query.c | 41 +- src/gallium

[Mesa-dev] [PATCH 2/3] vdpau: Add support for mixer attributes

2011-12-19 Thread Maarten Lankhorst
It's harmless to add support for attributes we don't support, since they require a feature enabled for them to affect something. As long as they aren't enabled, nothing happens. This enables support for custom colorspaces and background colors. Signed-off-by: Maarten Lankhorst m.b.lankho

[Mesa-dev] [PATCH 3/3] vdpau: Add background surface support

2011-12-19 Thread Maarten Lankhorst
Sets rgba layer as zeroth layer if a custom background_surface is specified. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- Can be applied without dependency on previous 2 patches in this series src/gallium/state_trackers/vdpau/mixer.c | 11 ++- 1 files changed, 10

[Mesa-dev] [PATCH] vl: Use pipe clear_render_target instead of util_clear_render_target

2011-12-19 Thread Maarten Lankhorst
Mapping to software and uploading again clearing is killing performance. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/auxiliary/vl/vl_compositor.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_compositor.c b

Re: [Mesa-dev] [PATCH 5/6] vl: Remove most members of pipe_video_decoder

2011-12-19 Thread Maarten Lankhorst
Hey Andy, On 12/19/2011 10:17 PM, Andy Furniss wrote: Andy Furniss wrote: I compiled with -O0 but the backtraces are different - maybe there is some randomness. Remembered to attach them this time :-) Thanks, but nothing seems to be obviously wrong there. I don't suppose you could find a

Re: [Mesa-dev] [PATCH 5/6] vl: Remove most members of pipe_video_decoder

2011-12-19 Thread Maarten Lankhorst
On 12/19/2011 11:46 PM, Andy Furniss wrote: Maarten Lankhorst wrote: Hey Andy, On 12/19/2011 10:17 PM, Andy Furniss wrote: Andy Furniss wrote: I compiled with -O0 but the backtraces are different - maybe there is some randomness. Remembered to attach them this time :-) Thanks

[Mesa-dev] [PATCH] xvmc: Remove unused variable

2011-12-20 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/xorg/xvmc/attributes.c |3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/xorg/xvmc/attributes.c b/src/gallium/state_trackers/xorg/xvmc/attributes.c index

[Mesa-dev] [PATCH 1/3] vl: Only initialize vlc once

2011-12-20 Thread Maarten Lankhorst
And add more sanity checks to stream. This shouldn't break things beyond those that aren't broken already. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- And yes Andy, I mean that I haven't found a good video yet to fix the playback parts that are still broken.. src/gallium

[Mesa-dev] [PATCH 2/3] vl: Remove decode buffers

2011-12-20 Thread Maarten Lankhorst
It's something for vl_mpeg12_bitstream internally, so treat it as such. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- v1: Split up from previous patch and with performance restored.. src/gallium/auxiliary/vl/vl_decoder.c | 13 src/gallium/auxiliary/vl

[Mesa-dev] [PATCH 3/3] vl: And remove remaining unnecessary members

2011-12-20 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- begin_frame and end_frame are implied for bitstream acceleration, and a frame can be explicitly ended with the flush call, so there's no need for the extra calls. Only reason patch is big is because I unified set_picture_parameters

Re: [Mesa-dev] [PATCH 1/3] vl: Only initialize vlc once

2011-12-20 Thread Maarten Lankhorst
Hey Christian, On 12/20/2011 02:16 PM, Christian König wrote: On 20.12.2011 12:43, Maarten Lankhorst wrote: And add more sanity checks to stream. This shouldn't break things beyond those that aren't broken already. Signed-off-by: Maarten Lankhorstm.b.lankho...@gmail.com --- And yes Andy

[Mesa-dev] [PATCH] egl_dri2/x11: Fix build failure for XCB_DRI2_MINOR_VERSION 3

2011-12-21 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/egl/drivers/dri2/platform_x11.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/egl/drivers/dri2/platform_x11.c b/src/egl/drivers/dri2/platform_x11.c index a00e3fe..6ca7838 100644 --- a/src/egl

Re: [Mesa-dev] [PATCH 1/3] vl: Only initialize vlc once

2011-12-21 Thread Maarten Lankhorst
Hey Christian, On 12/20/2011 09:08 PM, Lucas Stach wrote: Hi all! Just jumping in with regard to the assert. Am Dienstag, den 20.12.2011, 19:20 +0100 schrieb Maarten Lankhorst: [snip] return vlc-buffer (64 - num_bits); } @@ -130,7 +132,7 @@ vl_vlc_peekbits(struct vl_vlc *vlc

Re: [Mesa-dev] [PATCH 1/3] vl: Only initialize vlc once

2011-12-21 Thread Maarten Lankhorst
Hey Christian, On 12/21/2011 04:41 PM, Christian König wrote: On 20.12.2011 19:20, Maarten Lankhorst wrote: Hey Christian, On 12/20/2011 02:16 PM, Christian König wrote: Why do you want to change that anyway? The search for start codes was especially split out of the VLC stuff, because

Re: [Mesa-dev] [PATCH 1/3] vl: Only initialize vlc once

2011-12-21 Thread Maarten Lankhorst
Also a remark about your patch, wish you had inlined it.. On 12/21/2011 04:41 PM, Christian König wrote: On 20.12.2011 19:20, Maarten Lankhorst wrote: Hey Christian, On 12/20/2011 02:16 PM, Christian König wrote: Why do you want to change that anyway? The search for start codes

Re: [Mesa-dev] [PATCH 1/3] vl: Only initialize vlc once

2011-12-21 Thread Maarten Lankhorst
Hey Christian, On 12/21/2011 07:25 PM, Christian König wrote: Hi guys, On 21.12.2011 17:27, Maarten Lankhorst wrote: Also a remark about your patch, wish you had inlined it.. Sorry, not at home right now and I can't get this Thunderbird here to inline the patches correctly. diff --git

Re: [Mesa-dev] [PATCH] Add initial Haiku build support

2011-12-22 Thread Maarten Lankhorst
Hey Alexander, On 12/21/2011 07:16 PM, Alexander von Gluck wrote: * Doesn't reintroduce legacy drivers * Adds Haiku mklib code * Removes some broken PIPE_OS_HAIKU defines * Removes an NDEBUG ifdef in link_uniforms.cpp, there is an item that uses the union without checking NDEBUG below.

Re: [Mesa-dev] [PATCH 1/3] vl: Only initialize vlc once

2011-12-23 Thread Maarten Lankhorst
On 12/22/2011 02:25 PM, Christian König wrote: Hi Maarten, On 21.12.2011 21:52, Maarten Lankhorst wrote: It would be nice if you inlined patches for easier reviewing. :) Well I can try, but I can't promise that Thunderbird isn't badly fucking up all whitespaces, newest version of the patch

[Mesa-dev] [PATCH] vl: Fix inverted logic in vlc checks

2011-12-24 Thread Maarten Lankhorst
Reported-by: Andy Furniss andy...@ukfsn.org Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- diff --git a/src/gallium/auxiliary/vl/vl_vlc.h b/src/gallium/auxiliary/vl/vl_vlc.h index ff0c4b5..baaa48a 100644 --- a/src/gallium/auxiliary/vl/vl_vlc.h +++ b/src/gallium/auxiliary/vl

[Mesa-dev] [PATCH 1/2] vl: Remove decode buffers

2011-12-24 Thread Maarten Lankhorst
It's still a curiosity used by vl_mpeg12_bitstream only. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- Version n+1: split up, and fixed to apply with new vl_vlc, although for testing you might want to grab the 'vl: Fix inverted logic in vlc checks' first. src/gallium/auxiliary

[Mesa-dev] [PATCH 2/2] vl: Merge remaining members

2011-12-24 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- Flush explicitly ends the current frame so having a separate begin/endframe is unneeded, and decode_bitstream always has to operate on a full picture. src/gallium/auxiliary/vl/vl_mpeg12_bitstream.c |4 src/gallium/auxiliary/vl

Re: [Mesa-dev] [PATCH 2/3] vl: seperate shader buffers from components

2011-12-29 Thread Maarten Lankhorst
Hey, Op 26-12-11 14:00, Christian König schreef: Buffers for shader based decoding can now be released without its component still being around. Signed-off-by: Christian König deathsim...@vodafone.de Looks good to me. Acked-by: Maarten Lankhorst m.b.lankho...@gmail.com ~Maarten

Re: [Mesa-dev] [PATCH 3/3] vl: replace decode_buffers with auxiliary data field

2011-12-29 Thread Maarten Lankhorst
Hey Christian, Op 26-12-11 14:00, Christian König schreef: Based on patches from Maarten Lankhorst m.b.lankho...@gmail.com Signed-off-by: Christian König deathsim...@vodafone.de diff --git a/src/gallium/include/pipe/p_context.h b/src/gallium/include/pipe/p_context.h index de79a9b..f7ee522

Re: [Mesa-dev] [PATCH 3/3] vl: replace decode_buffers with auxiliary data field

2012-01-03 Thread Maarten Lankhorst
Hi Christian, 2012/1/2 Christian König deathsim...@vodafone.de: Hi Maarten, first of all: Happy new Year and sorry for the late reply, have been on vacation for the last week. On 29.12.2011 16:41, Maarten Lankhorst wrote: Hey Christian, Op 26-12-11 14:00, Christian König schreef

Re: [Mesa-dev] [PATCH 3/3] vl: replace decode_buffers with auxiliary data field

2012-01-08 Thread Maarten Lankhorst
Hey Christian, Op 04-01-12 11:42, Christian König schreef: On 03.01.2012 17:03, Maarten Lankhorst wrote: Hi Christian, 2012/1/2 Christian Königdeathsim...@vodafone.de: Hi Maarten, first of all: Happy new Year and sorry for the late reply, have been on vacation for the last week

[Mesa-dev] hardware xvmc video decoding with nouveau

2011-07-29 Thread Maarten Lankhorst
for the original code which I based this on, even though this code is significantly different from the original. :) Cheers, Maarten From 254b659ad0f36039687e01c5ab37d9e2e4804ad7 Mon Sep 17 00:00:00 2001 From: Maarten Lankhorst m.b.lankho...@gmail.com Date: Fri, 29 Jul 2011 13:26:45 +0200 Subject

[Mesa-dev] [PATCH] st/xorg: Advertise support for XvMC

2011-07-29 Thread Maarten Lankhorst
Formats were based on a patch sent to xf86-video-nouveau by Bryan Cain Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/xorg/xorg_tracker.h |7 ++ src/gallium/state_trackers/xorg/xorg_xv.c |2 + src/gallium/state_trackers/xorg/xorg_xvmc.c

Re: [Mesa-dev] hardware xvmc video decoding with nouveau

2011-07-29 Thread Maarten Lankhorst
On 07/30/2011 12:23 AM, Younes Manton wrote: On Fri, Jul 29, 2011 at 9:37 AM, Maarten Lankhorst m.b.lankho...@gmail.com wrote: Hi guys, With some help from the nouveau team I managed to get video acceleration working for my nv96 card. The video buffer api works well enough for nouveau, I

Re: [Mesa-dev] hardware xvmc video decoding with nouveau

2011-07-29 Thread Maarten Lankhorst
On 07/30/2011 12:48 AM, Jimmy Rentz wrote: On Fri, 29 Jul 2011 15:37:19 +0200 Maarten Lankhorst m.b.lankho...@gmail.com wrote: Hi guys, With some help from the nouveau team I managed to get video acceleration working for my nv96 card. The video buffer api works well enough for nouveau, I

Re: [Mesa-dev] hardware xvmc video decoding with nouveau

2011-07-29 Thread Maarten Lankhorst
On 07/30/2011 01:05 AM, Younes Manton wrote: On Fri, Jul 29, 2011 at 6:46 PM, Maarten Lankhorst m.b.lankho...@gmail.com wrote: 2nd patch isn't needed. You shouldn't call vl_video_buffer_create_ex, you should override the create_buffer hook yourself and do what you want. I'll push the 1st one

Re: [Mesa-dev] hardware xvmc video decoding with nouveau

2011-07-29 Thread Maarten Lankhorst
On 07/30/2011 01:57 AM, Younes Manton wrote: On Fri, Jul 29, 2011 at 7:45 PM, Maarten Lankhorst m.b.lankho...@gmail.com wrote: On 07/30/2011 01:05 AM, Younes Manton wrote: On Fri, Jul 29, 2011 at 6:46 PM, Maarten Lankhorst m.b.lankho...@gmail.com wrote: 2nd patch isn't needed. You shouldn't

Re: [Mesa-dev] hardware xvmc video decoding with nouveau

2011-08-07 Thread Maarten Lankhorst
Hey Younes, On 08/06/2011 08:37 PM, Younes Manton wrote: 2011/7/31 Christian König deathsim...@vodafone.de: Am Freitag, den 29.07.2011, 18:23 -0400 schrieb Younes Manton: On Fri, Jul 29, 2011 at 9:37 AM, Maarten Lankhorst m.b.lankho...@gmail.com wrote: With some help from the nouveau team I

Re: [Mesa-dev] hardware xvmc video decoding with nouveau

2011-08-08 Thread Maarten Lankhorst
On 08/08/2011 12:10 PM, Christian König wrote: Am Samstag, den 06.08.2011, 14:37 -0400 schrieb Younes Manton: The attached patch I believe should satisfy everyone's needs here. It removes the use of pipe_video_decode_buffer from the state tracker and moves it to the shader decoder. This lets

Re: [Mesa-dev] GPL'd vl_mpeg12_bitstream.c

2011-08-12 Thread Maarten Lankhorst
Hey, On 08/12/2011 07:42 PM, Christian König wrote: Am Freitag, den 12.08.2011, 10:49 -0400 schrieb Younes Manton: Sorry, by incompatible I didn't mean that you couldn't use them together, but that one is more restrictive than the other. Like the discussion you quoted states, if you combine

Re: [Mesa-dev] hardware xvmc video decoding with nouveau

2011-08-16 Thread Maarten Lankhorst
On 08/16/2011 02:10 PM, Christian König wrote: Am Dienstag, den 16.08.2011, 01:15 -0400 schrieb Younes Manton: Anyway, here are some specific comments: + for (; num_macroblocks 0; --num_macroblocks) { + mb-base.codec = PIPE_VIDEO_CODEC_MPEG12; + mb-macroblock_address = xvmc_mb-x

Re: [Mesa-dev] [PATCH 3/7] g3dvl: Rework the decoder interface part 3/5

2011-08-27 Thread Maarten Lankhorst
Hey, On 08/24/2011 11:51 PM, deathsim...@vodafone.de wrote: From: Christian König deathsim...@vodafone.de Revert back to a macroblock based interface. The structure used tries to keep as close to the spec as possible. Minor nitpick, but is XvMC PMV equal to macroblock PMV? iirc they swapped

[Mesa-dev] [PATCH 1/3] xvmc tests: Clean up test_rendering slightly

2011-08-27 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- .../xorg/xvmc/tests/test_rendering.c | 38 --- 1 files changed, 24 insertions(+), 14 deletions(-) diff --git a/src/gallium/state_trackers/xorg/xvmc/tests/test_rendering.c b/src/gallium/state_trackers

[Mesa-dev] [PATCH 2/3] xorg/xvmc: Only set decode buffer when available

2011-08-27 Thread Maarten Lankhorst
The nouveau xvmc decoder doesn't need it. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/xorg/xvmc/surface.c |9 ++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/gallium/state_trackers/xorg/xvmc/surface.c b/src/gallium

[Mesa-dev] [PATCH 3/3] xorg/xvmc: Add missing call to set_picture_parameters

2011-08-27 Thread Maarten Lankhorst
picture_structure is needed to determine what type of picture this is Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- src/gallium/state_trackers/xorg/xvmc/surface.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/gallium/state_trackers/xorg/xvmc

[Mesa-dev] xvmc decoding redux, testers wanted

2011-08-27 Thread Maarten Lankhorst
Hi all, With the interface changes to xvmc I'm finally happy with the xvmc decoder, so if you have a nv40-nv96 or a nva0 spare, could you test http://repo.or.cz/w/mesa/nouveau-pmpeg.git and report the results? To get it working correctly you need to preload libXvMCnouveau.so, usually programs

Re: [Mesa-dev] llvm-config on a biarch machine

2011-08-27 Thread Maarten Lankhorst
Hey Kevin, On 08/24/2011 05:34 PM, Kevin DeKorte wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On a biarch machine like Fedora llvm-config defaults to outputting the values for a 64bit build. With a recent patch to llvm-devel there are now arch specific versions of llvm-config and

Re: [Mesa-dev] [PATCH 2/3] xorg/xvmc: Only set decode buffer when available

2011-08-28 Thread Maarten Lankhorst
On 08/28/2011 06:23 PM, Younes Manton wrote: On Sun, Aug 28, 2011 at 12:13 PM, Younes Manton youne...@gmail.com wrote: On Sat, Aug 27, 2011 at 7:58 PM, Maarten Lankhorst m.b.lankho...@gmail.com wrote: The nouveau xvmc decoder doesn't need it. Signed-off-by: Maarten Lankhorst m.b.lankho

Re: [Mesa-dev] [PATCH 3/3] xorg/xvmc: Add missing call to set_picture_parameters

2011-08-28 Thread Maarten Lankhorst
Hey, On 08/28/2011 05:04 PM, Christian König wrote: Hi Maarten, could you put this into SetDecoderStatus instead? This makes the picture structure also available in end_frame. Erm it doesn't make sense there. It might make sense to move it to after the begin_frame though, but with interlaced

Re: [Mesa-dev] [PATCH 3/3] xorg/xvmc: Add missing call to set_picture_parameters

2011-08-29 Thread Maarten Lankhorst
Hey Christian, On 08/29/2011 10:14 AM, Christian König wrote: Am Sonntag, den 28.08.2011, 19:13 +0200 schrieb Maarten Lankhorst: Hey, On 08/28/2011 05:04 PM, Christian König wrote: Hi Maarten, could you put this into SetDecoderStatus instead? This makes the picture structure also

Re: [Mesa-dev] [PATCH 2/3] xorg/xvmc: Only set decode buffer when available

2011-08-29 Thread Maarten Lankhorst
Hey, On 08/29/2011 10:08 AM, Christian König wrote: Am Montag, den 29.08.2011, 00:36 -0400 schrieb Younes Manton: [snip] Well, that was what the last discussion was all about, whether or not decode buffers should be handled internally by each driver or externally. It was decided to handle it

[Mesa-dev] [PATCH] xvmc: Replace frame_started by picture_structure

2011-09-02 Thread Maarten Lankhorst
The preferred solution to keeping track of the picture structure has been putting it in the state tracker, so use picture_structure instead of frame_started to check if a frame needs to begin. If picture_structure has been changed, end the frame and start again. Signed-off-by: Maarten Lankhorst

Re: [Mesa-dev] [PATCH] xvmc: Replace frame_started by picture_structure

2011-09-03 Thread Maarten Lankhorst
On 09/03/2011 10:44 AM, Christian König wrote: Looks good, I'm going to push it, but one small thing: Am Freitag, den 02.09.2011, 16:20 +0200 schrieb Maarten Lankhorst: The preferred solution to keeping track of the picture structure has been putting it in the state tracker, so use

Re: [Mesa-dev] [PATCH v2] nouveau: Add support for XvMC acceleration

2011-09-10 Thread Maarten Lankhorst
Hey ★, On 09/10/2011 06:28 PM, ★ Emeric wrote: Hi Maarten, I tried the v2 patch with my good old geforce 8600M GS, and it worked well. The CPU usage seems a little higher than usual, I'd said 60% instead of 40% for a 720p video, but I'll try with a higher bitrate video and without building

[Mesa-dev] [PATCH] gallium/targets: Add vdpau target for nouveau

2011-10-18 Thread Maarten Lankhorst
Should fall back to shader based decoding (g3dvl) for now. This is probably broken on systems that support xvmc, because nouveau_video_buffer_create has no way to know for what api the buffer is created, so I think this call might need a separate argument as workaround. Signed-off-by: Maarten

[Mesa-dev] [PATCH] state_trackers/vdpau: Implement VdpGenerateCSCMatrix partially

2011-10-25 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- diff --git a/src/gallium/state_trackers/vdpau/mixer.c b/src/gallium/state_trackers/vdpau/mixer.c index 8728157..83daddf 100644 --- a/src/gallium/state_trackers/vdpau/mixer.c +++ b/src/gallium/state_trackers/vdpau/mixer.c @@ -257,9

Re: [Mesa-dev] [PATCH] state_trackers/vdpau: Implement VdpGenerateCSCMatrix partially

2011-10-25 Thread Maarten Lankhorst
On 10/25/2011 08:00 PM, Younes Manton wrote: On Tue, Oct 25, 2011 at 1:35 PM, Maarten Lankhorst m.b.lankho...@gmail.com wrote: Is there a reason not to use/add to the code in vl_csc.c in this case? It supports 601 and 709, wider gamut, and and it takes procamp into account

[Mesa-dev] [PATCH 1/2] build libgallium shared by default.

2013-03-18 Thread Maarten Lankhorst
for discussion purposes, although if there is no objection I'll change the default to disabling shared gallium, and just let every distro enable shared llvm and libgallium for themselves. 8--- Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com --- a/configure.ac +++ b/configure.ac

Re: [Mesa-dev] [PATCH 1/2] build libgallium shared by default.

2013-03-19 Thread Maarten Lankhorst
Op 18-03-13 16:28, Andreas Boll schreef: 2013/3/18 Maarten Lankhorst maarten.lankho...@canonical.com: This is one of the 2 patches used in ubuntu for decreasing size of mesa build. The other one is more hacky, and links libmesagallium into libgallium, and then links libgallium against

Re: [Mesa-dev] [PATCH] radeon/llvm: Link against libgallium.la to fix an undefined symbol

2013-03-19 Thread Maarten Lankhorst
Hey, Op 19-03-13 11:55, Andreas Boll schreef: Ported from downstream: http://anonscm.debian.org/gitweb/?p=pkg-xorg/lib/mesa.git;a=blob;f=debian/patches/119-libllvmradeon-link.patch;h=ee47f8a07dbf33c32f8b57faed923680ed6648fb;hb=refs/heads/ubuntu%2B1 Fixes a regression introduced with

[Mesa-dev] [PATCH 1/2 v2] build libgallium shared by default.

2013-03-19 Thread Maarten Lankhorst
Op 19-03-13 12:37, Andreas Boll schreef: 2013/3/19 Maarten Lankhorst maarten.lankho...@canonical.com: Op 18-03-13 16:28, Andreas Boll schreef: 2013/3/18 Maarten Lankhorst maarten.lankho...@canonical.com: This is one of the 2 patches used in ubuntu for decreasing size of mesa build

[Mesa-dev] [PATCH] osmesa: link against static libglapi library too to get the gl exports

2013-07-07 Thread Maarten Lankhorst
. This is a candidate for the stable series. Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com --- --- a/src/mesa/drivers/osmesa/Makefile.am +++ b/src/mesa/drivers/osmesa/Makefile.am @@ -40,11 +40,11 @@ lib_LTLIBRARIES = lib@OSMESA_LIB@.la lib@OSMESA_LIB@_la_LDFLAGS = -module

[Mesa-dev] [PATCH] configure: link against -lLLVM to determine build type

2013-08-13 Thread Maarten Lankhorst
Fixes a build failure of 9.2 on ubuntu, because libLLVM-3.3.so is not present in /usr/lib/llvm-3.2/lib. Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com --- diff --git a/configure.ac b/configure.ac index 35f6797..579d8d4 100644 --- a/configure.ac +++ b/configure.ac @@ -1870,7

[Mesa-dev] [PATCH] radeon/llvm: fix compile error with -Werror=format-security

2013-08-13 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com --- diff --git a/src/gallium/drivers/radeon/radeon_llvm_emit.c b/src/gallium/drivers/radeon/radeon_llvm_emit.c index 1a4d4fd..2dd7bf7 100644 --- a/src/gallium/drivers/radeon/radeon_llvm_emit.c +++ b/src/gallium/drivers/radeon

Re: [Mesa-dev] [PATCH] configure: link against -lLLVM to determine build type

2013-08-14 Thread Maarten Lankhorst
Op 14-08-13 04:37, Tom Stellard schreef: On Tue, Aug 13, 2013 at 05:53:52PM +0200, Maarten Lankhorst wrote: Fixes a build failure of 9.2 on ubuntu, because libLLVM-3.3.so is not present in /usr/lib/llvm-3.2/lib. I'm trying to understand the problem here, could you give a little more

Re: [Mesa-dev] Mesa 9.2 release candidate 1

2013-08-20 Thread Maarten Lankhorst
Op 20-08-13 02:05, Ian Romanick schreef: Mesa 9.2 release candidate 1 is now available for testing. The tag in the GIT repository for Mesa 9.2-rc1 is 'mesa-9.2-rc1'. Mesa 9.2 release candidate 1 is available for download at ftp://freedesktop.org/pub/mesa/9.2/ md5sums:

Re: [Mesa-dev] Mesa 9.2 release candidate 1

2013-08-20 Thread Maarten Lankhorst
Op 20-08-13 20:14, Matt Turner schreef: On Tue, Aug 20, 2013 at 8:54 AM, Maarten Lankhorst maarten.lankho...@canonical.com wrote: 1. There is a new gallium based osmesa, which doesn't work correctly with wine. My testcase is running opengl32_test from wine/dlls/opengl32/tests. I've pushed

[Mesa-dev] [PATCH] radeon: Remove unnecessary pipe_thread_destroy in radeon_drm_cs_destroy

2012-05-22 Thread Maarten Lankhorst
Fixes crash bug introduced with 210ddf0819b5 fd.o #49198 pthread_detach after a pthread_join is unneeded. Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c b/src/gallium/winsys/radeon/drm/radeon_drm_cs.c index 2239059

[Mesa-dev] [PATCH] configure.ac: Enable building nouveau by default

2012-05-31 Thread Maarten Lankhorst
Signed-off-by: Maarten Lankhorst m.b.lankho...@gmail.com --- diff --git a/configure.ac b/configure.ac index 9fb8149..30f895c 100644 --- a/configure.ac +++ b/configure.ac @@ -660,7 +660,7 @@ AC_ARG_ENABLE([gallium_tests], [enable_gallium_tests=no]) # Option for Gallium drivers

[Mesa-dev] [PATCH] makefiles: use configured name for -ldrm* where possible

2012-11-13 Thread Maarten Lankhorst
for the stable branches. Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com --- diff --git a/configs/current.in b/configs/current.in index 1802271..62edfa5 100644 --- a/configs/current.in +++ b/configs/current.in @@ -144,7 +144,7 @@ GLAPI_LIB_DEPS = @GLAPI_LIB_DEPS@ DRI_LIB_DEPS

Re: [Mesa-dev] [PATCH] makefiles: use configured name for -ldrm* where possible

2012-11-16 Thread Maarten Lankhorst
Op 13-11-12 19:31, Matt Turner schreef: On Tue, Nov 13, 2012 at 4:00 AM, Maarten Lankhorst maarten.lankho...@canonical.com wrote: For precise lts support I had to do some magic with the library names, which works fine as long as the libraries from pkg-config are used. The parts with src

[Mesa-dev] [PATCH] vl/compositor: fix weave shader bugs

2012-12-18 Thread Maarten Lankhorst
YCbCr[62] = 0e70b0 -YCbCr[63] = e080c0 +YCbCr[63] = e070b0 Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com --- diff --git a/src/gallium/auxiliary/vl/vl_compositor.c b/src/gallium/auxiliary/vl/vl_compositor.c index 7ea13bf..bc69669 100644 --- a/src/gallium/auxiliary/vl

Re: [Mesa-dev] [PATCH] vl/compositor: fix weave shader bugs

2012-12-18 Thread Maarten Lankhorst
Op 18-12-12 11:42, Christian König schreef: On 18.12.2012 09:18, Maarten Lankhorst wrote: Writemask was XY instead of YZ (thanks to calim for spotting it). The pixel calculation resulted in the pixel always being off by one. If y was .5: y' = round(y) + 0.5 = 1.5 Fixing this also means

[Mesa-dev] [PATCH] vl/video_buffer: fix up surface ordering for the interlaced case

2012-12-18 Thread Maarten Lankhorst
It seems the other code expects surface[0..1] to be the luma field in interlaced case. See for example vdpau/surface.c vlVdpVideoSurfaceClear and vlVdpVideoSurfacePutBitsYCbCr. Signed-off-by: Maarten Lankhorst maarten.lankho...@canonical.com --- diff --git a/src/gallium/auxiliary/vl

  1   2   >