Re: [Mesa-dev] [PATCH 3/3] configure: check for core xcb (libxcb.so) and link VL targets against it.

2014-07-12 Thread Christian König
module. Small typo "gellium" instead of "gallium". Signed-off-by: Emil Velikov The dependency to x11-xcb and xcb-dri2 doesn't pull in core xcb as well? Anyway patch looks good to me and is Reviewed-by: Christian König Regards, Christian. --- configure

Re: [Mesa-dev] [PATCH 0/5] radeon: Write-combined CPU mappings of BOs in GTT

2014-07-17 Thread Christian König
adeon: Pass GART page flags to [PATCH 3/5] drm/radeon: Allow write-combined CPU mappings of BOs in [PATCH 4/5] drm/radeon: Use write-combined CPU mappings of rings and Those four are Reviewed-by: Christian König [PATCH 5/5] drm/radeon: Use VRAM for indirect buffers on >= SI I'm still

Re: [Mesa-dev] [PATCH 0/5] radeon: Write-combined CPU mappings of BOs in GTT

2014-07-18 Thread Christian König
Am 18.07.2014 05:07, schrieb Michel Dänzer: [PATCH 5/5] drm/radeon: Use VRAM for indirect buffers on >= SI I'm still not very keen with this change since I still don't understand the reason why it's faster than with GTT. Definitely needs more testing on a wider range of systems. Sure. If anyone

Re: [Mesa-dev] [PATCH 0/5] radeon: Write-combined CPU mappings of BOs in GTT

2014-07-21 Thread Christian König
Am 19.07.2014 03:15, schrieb Michel Dänzer: On 19.07.2014 00:47, Christian König wrote: Am 18.07.2014 05:07, schrieb Michel Dänzer: [PATCH 5/5] drm/radeon: Use VRAM for indirect buffers on >= SI I'm still not very keen with this change since I still don't understand the reason wh

Re: [Mesa-dev] [PATCH 0/5] radeon: Write-combined CPU mappings of BOs in GTT

2014-07-22 Thread Christian König
Am 23.07.2014 05:54, schrieb Michel Dänzer: On 21.07.2014 17:07, Christian König wrote: Am 19.07.2014 03:15, schrieb Michel Dänzer: On 19.07.2014 00:47, Christian König wrote: Am 18.07.2014 05:07, schrieb Michel Dänzer: [PATCH 5/5] drm/radeon: Use VRAM for indirect buffers on >= SI

Re: [Mesa-dev] [PATCH 0/5] radeon: Write-combined CPU mappings of BOs in GTT

2014-07-23 Thread Christian König
Am 23.07.2014 09:21, schrieb Michel Dänzer: On 23.07.2014 15:42, Christian König wrote: Am 23.07.2014 05:54, schrieb Michel Dänzer: On 21.07.2014 17:07, Christian König wrote: Am 19.07.2014 03:15, schrieb Michel Dänzer: On 19.07.2014 00:47, Christian König wrote: Am 18.07.2014 05:07

Re: [Mesa-dev] [PATCH 2/2] r600g/radeonsi: Prefer VRAM for CPU -> GPU streaming buffers

2014-07-31 Thread Christian König
Am 31.07.2014 um 11:43 schrieb Michel Dänzer: From: Michel Dänzer Signed-off-by: Michel Dänzer At least for PIPE_USAGE_STREAM buffers that's a bad idea, cause they are used by VDPAU to read back to data to a CPU buffer and that's really slow from VRAM. Christian. --- src/gallium/driv

Re: [Mesa-dev] [PATCH 2/2] r600g/radeonsi: Prefer VRAM for CPU -> GPU streaming buffers

2014-07-31 Thread Christian König
Am 31.07.2014 um 11:57 schrieb Michel Dänzer: On 31.07.2014 18:52, Christian König wrote: Am 31.07.2014 um 11:43 schrieb Michel Dänzer: From: Michel Dänzer Signed-off-by: Michel Dänzer At least for PIPE_USAGE_STREAM buffers that's a bad idea, cause they are used by VDPAU to read ba

Re: [Mesa-dev] [PATCH 1/2] radeonsi: Add helper function for emitting constant loads

2013-07-20 Thread Christian König
Am 19.07.2013 22:18, schrieb Tom Stellard: From: Tom Stellard Both patches are: Reviewed-by: Christian König --- src/gallium/drivers/radeonsi/radeonsi_shader.c | 29 ++ 1 file changed, 20 insertions(+), 9 deletions(-) diff --git a/src/gallium/drivers/radeonsi

Re: [Mesa-dev] [PATCH] gallium/vl: add prime support

2013-07-25 Thread Christian König
a further issue with final displaying, it only works sometimes, but this patch is at least necessary to help debug further. Signed-off-by: Dave Airlie Looks good to me. Reviewed-by: Christian König --- src/gallium/auxiliary/vl/vl_winsys_dri.c | 20 +++- 1 file changed, 19

Re: [Mesa-dev] [PATCH] radeonsi: Number of SGPRs retrieved from LLVM already includes VCC

2013-08-06 Thread Christian König
Am 06.08.2013 10:53, schrieb Michel Dänzer: From: Michel Dänzer Fixes spurious 'Assertion `num_sgprs <= 104' failed.' with shaders using all 104 SGPRs. Cc: mesa-sta...@lists.freedesktop.org Signed-off-by: Michel Dänzer Reviewed-by: Christian König --- src/galliu

Re: [Mesa-dev] [PATCH 16/20] radeonsi: add FMASK texture binding slots and resource setup

2013-08-08 Thread Christian König
Am 08.08.2013 02:20, schrieb Marek Olšák: FMASK is bound as a separate texture. For every texture, there can be an FMASK. Therefore a separate array of resource slots has to be added. This adds a new mechanism for emitting resource descriptors, its features are: - resource descriptors are stored

Re: [Mesa-dev] [PATCH 18/20] radeonsi: port texture improvements from r600g

2013-08-08 Thread Christian König
Am 08.08.2013 02:20, schrieb Marek Olšák: This started as an attempt to add support for MSAA texture transfers and MSAA depth-stencil decompression for the DB->CB copy path. It has gotten a bit out of control, but it's for the greater good. Some changes do not make much sense, they are there jus

Re: [Mesa-dev] [PATCH 15/20] radeonsi: add basic infrastructure for atom-based states

2013-08-08 Thread Christian König
Am 08.08.2013 02:20, schrieb Marek Olšák: It's the same as in r600g. Look how simple it is. That concept has the problem that we don't necessary know in which order the state is emitted. Why not just add an "emit" callback to si_pm4_state for the short term instead? For the long term we s

Re: [Mesa-dev] [PATCH 14/20] radeonsi: reserve a parameter slot for FMASK resources in shaders

2013-08-08 Thread Christian König
Am 08.08.2013 10:30, schrieb Michel Dänzer: On Don, 2013-08-08 at 02:20 +0200, Marek Olšák wrote: diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c b/src/gallium/drivers/radeonsi/si_state_draw.c index 746ace6..4208fa7 100644 --- a/src/gallium/drivers/radeonsi/si_state_draw.c +++ b/src/g

Re: [Mesa-dev] [PATCH 15/20] radeonsi: add basic infrastructure for atom-based states

2013-08-08 Thread Christian König
ow you the documents on that). Anyway, the order is determined by the order of si_add_atom calls, which should be done once in create_context. Then why don't you just use a constant structure for this? Christian. Marek On Thu, Aug 8, 2013 at 10:32 AM, Christian König wrote: Am 0

Re: [Mesa-dev] [PATCH 16/20] radeonsi: add FMASK texture binding slots and resource setup

2013-08-08 Thread Christian König
Am 08.08.2013 14:38, schrieb Marek Olšák: .On Thu, Aug 8, 2013 at 9:47 AM, Christian König wrote: Am 08.08.2013 02:20, schrieb Marek Olšák: FMASK is bound as a separate texture. For every texture, there can be an FMASK. Therefore a separate array of resource slots has to be added. This adds

Re: [Mesa-dev] [PATCH 16/20] radeonsi: add FMASK texture binding slots and resource setup

2013-08-08 Thread Christian König
Am 08.08.2013 16:33, schrieb Marek Olšák: On Thu, Aug 8, 2013 at 3:09 PM, Christian König wrote: Am 08.08.2013 14:38, schrieb Marek Olšák: .On Thu, Aug 8, 2013 at 9:47 AM, Christian König wrote: Am 08.08.2013 02:20, schrieb Marek Olšák: FMASK is bound as a separate texture. For every

Re: [Mesa-dev] [PATCH] r600g: disable GPUVM by default

2013-08-09 Thread Christian König
=65958 Signed-off-by: Alex Deucher CC: "9.2" CC: "9.1" Sounds like a good idea to me. Reviewed-by: Christian König --- src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/winsys/radeon/drm/ra

Re: [Mesa-dev] [PATCH 16/20] radeonsi: add FMASK texture binding slots and resource setup

2013-08-09 Thread Christian König
Am 08.08.2013 21:38, schrieb Alex Deucher: On Thu, Aug 8, 2013 at 1:34 PM, Marek Olšák wrote: On Thu, Aug 8, 2013 at 6:57 PM, Christian König wrote: Am 08.08.2013 16:33, schrieb Marek Olšák: On Thu, Aug 8, 2013 at 3:09 PM, Christian König wrote: Am 08.08.2013 14:38, schrieb Marek Olšák

Re: [Mesa-dev] [PATCH 16/20] radeonsi: add FMASK texture binding slots and resource setup

2013-08-09 Thread Christian König
Am 09.08.2013 15:29, schrieb Marek Olšák: On Fri, Aug 9, 2013 at 10:34 AM, Christian König wrote: Am 08.08.2013 21:38, schrieb Alex Deucher: On Thu, Aug 8, 2013 at 1:34 PM, Marek Olšák wrote: On Thu, Aug 8, 2013 at 6:57 PM, Christian König wrote: Am 08.08.2013 16:33, schrieb Marek Olšák

Re: [Mesa-dev] [PATCH 16/20] radeonsi: add FMASK texture binding slots and resource setup

2013-08-10 Thread Christian König
Am 09.08.2013 20:06, schrieb Marek Olšák: [SNIP] What if I kept the current emission code, and only allocated a new buffer at the end of the emit function, copied all descriptors to it using CP_DMA or COPY_DATA, and pointed SPI_SHADER_USER_DATA to it. The buffer where the descriptors are updated

Re: [Mesa-dev] [PATCH 16/20] radeonsi: add FMASK texture binding slots and resource setup

2013-08-10 Thread Christian König
ext), so I strongly think we should be on the save side with 16 slots here. I'm just not sure if the SQ could add some more depth to our pipeline, maybe Alex knows more on this. Christian. Marek On Sat, Aug 10, 2013 at 10:45 AM, Christian König wrote: Am 09.08.2013 20:06, schrieb Marek Olšák

Re: [Mesa-dev] [PATCH] vl/decoder: Add support for max level query.

2013-08-14 Thread Christian König
Am 14.08.2013 08:13, schrieb Rico Schüller: Hi, this patch adds the level query support to the video decoders. This uses some more reasonable defaults. This improves the situation in bug 67530 . The patch is: Reviewed-by: Christian König I assume you don't have commit access,

Re: [Mesa-dev] [PATCH] vl/decoder: Add support for max level query.

2013-08-14 Thread Christian König
Am 14.08.2013 13:04, schrieb Rico Schüller: On 14.08.2013 12:47, Christian König wrote: Am 14.08.2013 08:13, schrieb Rico Schüller: Hi, this patch adds the level query support to the video decoders. This uses some more reasonable defaults. This improves the situation in bug 67530 . The

Re: [Mesa-dev] [PATCH] radeonsi: add flexible shader descriptor management and use it for sampler views

2013-08-15 Thread Christian König
Am 15.08.2013 09:33, schrieb Michel Dänzer: On Don, 2013-08-15 at 05:25 +0200, Marek Olšák wrote: (This should be applied before MSAA, which will need to be rebased.) It moves all sampler view descriptors to a buffer. It supports partial resource updates and it can also unbind resources (requir

Re: [Mesa-dev] [PATCH] radeonsi: add flexible shader descriptor management and use it for sampler views

2013-08-15 Thread Christian König
Am 15.08.2013 05:25, schrieb Marek Olšák: (This should be applied before MSAA, which will need to be rebased.) It moves all sampler view descriptors to a buffer. It supports partial resource updates and it can also unbind resources (required for FMASK texturing). The buffer contains all sampler

Re: [Mesa-dev] [PATCH] radeonsi: add flexible shader descriptor management and use it for sampler views

2013-08-15 Thread Christian König
Am 15.08.2013 12:54, schrieb Marek Olšák: On Thu, Aug 15, 2013 at 10:27 AM, Christian König wrote: Am 15.08.2013 05:25, schrieb Marek Olšák: (This should be applied before MSAA, which will need to be rebased.) It moves all sampler view descriptors to a buffer. It supports partial resource

Re: [Mesa-dev] [PATCH] radeonsi: add flexible shader descriptor management and use it for sampler views

2013-08-15 Thread Christian König
Am 15.08.2013 19:01, schrieb Marek Olšák: (This should be applied before MSAA, which will need to be rebased.) It moves all sampler view descriptors to a buffer. It supports partial resource updates and it can also unbind resources (required for FMASK texturing). The buffer contains all sampler

Re: [Mesa-dev] [PATCH] nouveau/video: use correct parameter name

2013-08-15 Thread Christian König
Am 15.08.2013 14:54, schrieb Emil Velikov: On 15/08/13 13:41, Rico Schüller wrote: On 15.08.2013 02:10, Emil Velikov wrote: Fix a typo introduced with commit d1ba1055d9 - vl: Add support for max level query v2 Cc: Rico Schüller Cc: Christian König Bugzilla: https://bugs.freedesktop.org

[Mesa-dev] [PATCH 2/5] vl: rename enum pipe_video_codec to pipe_video_format

2013-08-16 Thread Christian König
From: Christian König Signed-off-by: Christian König --- src/gallium/auxiliary/util/u_video.h | 12 +++ src/gallium/auxiliary/vl/vl_decoder.c |4 +-- src/gallium/auxiliary/vl/vl_mpeg12_bitstream.c |2 +- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c

[Mesa-dev] [PATCH 4/5] vl: add entrypoint to get_video_param

2013-08-16 Thread Christian König
From: Christian König Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_decoder.c |4 +++- src/gallium/auxiliary/vl/vl_decoder.h |3 ++- src/gallium/auxiliary/vl/vl_video_buffer.c |1 + src/gallium/drivers/ilo/ilo_screen.c|3

[Mesa-dev] [PATCH 5/5] vl: add entrypoint to is_video_format_supported

2013-08-16 Thread Christian König
From: Christian König Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_video_buffer.c |3 ++- src/gallium/auxiliary/vl/vl_video_buffer.h |3 ++- src/gallium/drivers/ilo/ilo_format.c|5 +++-- src/gallium/drivers/nouveau/nouveau_vp3_video.c |5

Re: [Mesa-dev] [PATCH] st/vdpau: drop unnecessary variable prof

2013-08-17 Thread Christian König
Am 16.08.2013 22:41, schrieb Emil Velikov: Signed-off-by: Emil Velikov Well, any halve way sane compiler should be able to optimize that anyway, but on the other hand the patch looks good to me. Reviewed-by: Christian König --- src/gallium/state_trackers/vdpau/mixer.c | 5 ++--- 1

Re: [Mesa-dev] [PATCH 1/7] vdpau/vl 422 chroma width/height mix up

2013-08-18 Thread Christian König
e patch is: Reviewed-by: Christian König --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 4 ++-- src/gallium/auxiliary/vl/vl_video_buffer.c | 2 +- src/gallium/state_trackers/vdpau/surface.c | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/auxiliary/vl

Re: [Mesa-dev] [PATCH 2/7] st/vdpau: exit gracefully if we fail to create video buffer

2013-08-18 Thread Christian König
Am 17.08.2013 23:51, schrieb Emil Velikov: Otherwise we risk causing memory corruption. v2: forgot the mutex_unlock() Signed-off-by: Emil Velikov NAK. Failing is actually not correct here, cause we can still make it work by allocating the video buffer later on decoding or uploading of imag

Re: [Mesa-dev] [PATCH 3/7] st/xvmc: exit gracefully if we fail to create video buffer

2013-08-18 Thread Christian König
Am 17.08.2013 23:51, schrieb Emil Velikov: Free any allocated memory and return BadAlloc if create_video_buffer() has failed to create a buffer. Signed-off-by: Emil Velikov Reviewed-by: Christian König --- src/gallium/state_trackers/xvmc/surface.c | 4 1 file changed, 4 insertions

Re: [Mesa-dev] [PATCH 4/7] vl/buffer: add sanity check after CALLOC_STRUCT

2013-08-18 Thread Christian König
Am 17.08.2013 23:51, schrieb Emil Velikov: Check if we have successfully allocated memory. Signed-off-by: Emil Velikov Reviewed-by: Christian König --- src/gallium/auxiliary/vl/vl_video_buffer.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/auxiliary/vl

Re: [Mesa-dev] [PATCH 5/7] vl/idct: cleanup all idct buffers

2013-08-18 Thread Christian König
. Reviewed-by: Christian König --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c b/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c index de890fe..cff326f 100644 --- a/src/gallium/auxiliary/vl

Re: [Mesa-dev] [PATCH 7/7] vl/buffers: consistent use on VL_MAX_SURFACES

2013-08-18 Thread Christian König
Am 17.08.2013 23:51, schrieb Emil Velikov: Signed-off-by: Emil Velikov Reviewed-by: Christian König --- src/gallium/auxiliary/vl/vl_video_buffer.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_video_buffer.c b/src/gallium

Re: [Mesa-dev] [PATCH 2/7] st/vdpau: exit gracefully if we fail to create video buffer

2013-08-20 Thread Christian König
Am 18.08.2013 14:20, schrieb Emil Velikov: On 18/08/13 12:31, Christian König wrote: Am 17.08.2013 23:51, schrieb Emil Velikov: Otherwise we risk causing memory corruption. v2: forgot the mutex_unlock() Signed-off-by: Emil Velikov NAK. Failing is actually not correct here, cause we can

Re: [Mesa-dev] [PATCH 2/7] st/vdpau: don't try to create video buffer when the format is FORMAT_NONE

2013-08-20 Thread Christian König
Am 19.08.2013 18:00, schrieb Emil Velikov: Not seen in the wild yet, but seems like a reasonable thing to do. [suggested by Christian] Signed-off-by: Emil Velikov Reviewed-by: Christian König Do you have commit access? And by the way I have enough on my to do list for the VDPAU state

Re: [Mesa-dev] [PATCH 2/7] st/vdpau: don't try to create video buffer when the format is FORMAT_NONE

2013-08-20 Thread Christian König
Am 19.08.2013 18:17, schrieb Emil Velikov: On 19/08/13 17:09, Christian König wrote: Am 19.08.2013 18:00, schrieb Emil Velikov: Not seen in the wild yet, but seems like a reasonable thing to do. [suggested by Christian] Signed-off-by: Emil Velikov Reviewed-by: Christian König Do you have

Re: [Mesa-dev] [PATCH 1/2] vl/query: Only support VDP_CHROMA_TYPE_420 for 12 bit formats.

2013-08-21 Thread Christian König
Reviewed and committed both patches. Thanks for the help, Christian. Am 21.08.2013 10:06, schrieb Rico Schüller: --- src/gallium/state_trackers/vdpau/query.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) ___ mesa-dev mailing list me

Re: [Mesa-dev] [PATCH] [RFC] r600g: enable SB backend by default

2013-08-23 Thread Christian König
Am 22.08.2013 18:20, schrieb Vadim Girlin: Signed-off-by: Vadim Girlin Sounds good if it's stable enough. Reviewed-by: Christian König --- src/gallium/drivers/r600/r600_asm.c| 3 ++- src/gallium/drivers/r600/r600_pipe.c | 4 ++-- src/gallium/drivers/r600/r600_pipe.h

Re: [Mesa-dev] [PATCH] r600g/llvm: don't export more colors than the number of CBs

2013-08-24 Thread Christian König
Am 24.08.2013 03:30, schrieb Vadim Girlin: Currently llvm backend always exports at least one color in pixel shader even if no color buffers are enabled. With depth/stencil exports this can result in the following code: EXPORT PIXEL 0 R0.xyzw VPM EXPORT PIXEL 61R

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

2013-08-25 Thread Christian König
initialized earlier wasn't obviously to me. Patch is: Reviewed-by: Christian König --- src/gallium/drivers/nv50/nv98_video.c | 2 +- src/gallium/drivers/nvc0/nvc0_video.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/nv50/nv98_video.c b/src/ga

Re: [Mesa-dev] [PATCH] r600g: fix color exports when we have no CBs

2013-08-28 Thread Christian König
Well, for this discussion let's just assume that we fixed the delay in the upper layers of the stack and the driver sees the shader code as soon as the application (if I understood it correctly Vadim has just volunteered for the job). Also let's assume that shaders are small and having allot o

[Mesa-dev] [PATCH] radeon/uvd: fix MPEG2/4 ref frame index limit

2013-08-28 Thread Christian König
From: Christian König Otherwise the first few frames have an incorrect reference index. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/radeon_uvd.c b/src/gallium

Re: [Mesa-dev] [PATCH 0/6] radeonsi: Minor cleanups and improvements

2013-08-29 Thread Christian König
Am 28.08.2013 19:17, schrieb Marek Olšák: This series contains the changes my transform feedback work depends on, but there are some useful fixes too, making it worth comitting earlier. The last patch is the most important one, because it fixes the issues we had with the emission of resource d

Re: [Mesa-dev] [PATCH 0/6] radeonsi: Minor cleanups and improvements

2013-08-29 Thread Christian König
Am 29.08.2013 12:38, schrieb Marek Olšák: On Thu, Aug 29, 2013 at 12:20 PM, Christian König wrote: Am 28.08.2013 19:17, schrieb Marek Olšák: This series contains the changes my transform feedback work depends on, but there are some useful fixes too, making it worth comitting earlier. The

Re: [Mesa-dev] [PATCH 1/2] gallium: comment that INSTANCEID doesn't include start_instance

2013-09-05 Thread Christian König
I was aware that this was only correct in one use case, but didn't know which one. Probably should have added a comment about that. Thanks for fixing it, both patches are: Reviewed-by: Christian König Am 05.09.2013 16:41, schrieb Marek Olšák: --- src/gallium/include

Re: [Mesa-dev] [PATCH] r600g: pad IBs to a multiple of 4 DWs on r6xx

2013-09-06 Thread Christian König
Actually Type 2 packets are handled much faster on the R6xx compared to most type 3 packets, cause they are handled by the PFP/fetch hw and doesn't need to be forwarded to the ME. Christian. Am 06.09.2013 02:31, schrieb Dominik Behr: 0x8000 is Type 2 NOP. You could make it a little better

Re: [Mesa-dev] Update: UVD status on loongson 3a platform

2013-09-06 Thread Christian König
Am 06.09.2013 04:52, schrieb cee1: 2013/9/6 Jerome Glisse : On Thu, Sep 05, 2013 at 03:29:52PM -0400, Jerome Glisse wrote: On Thu, Sep 05, 2013 at 10:14:32PM +0800, Chen Jie wrote: Hi all, This thread is about http://lists.freedesktop.org/archives/dri-devel/2013-April/037598.html. We recentl

Re: [Mesa-dev] [PATCH 1/2] radeon/winsys: pad IBs to a multiple of 8 DWs

2013-09-07 Thread Christian König
Am 06.09.2013 23:00, schrieb Alex Deucher: This aligns the gfx, compute, and dma IBs to 8 DW boundries. This aligns the the IB to the fetch size of the CP for optimal performance. Additionally, r6xx hardware requires at least 4 DW alignment to avoid a hw bug. This also aligns the DMA IBs to 8 DW

[Mesa-dev] [PATCH 1/2] radeon/uvd: use more sane defaults for bitstream buffer size

2013-09-11 Thread Christian König
From: Christian König Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c |4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/radeon/radeon_uvd.c b/src/gallium/drivers/radeon/radeon_uvd.c index 5e7eedb..981d5c5 100644 --- a/src

[Mesa-dev] [PATCH 2/2] radeon/uvd: move more logic into the common files

2013-09-11 Thread Christian König
From: Christian König Move the code back into the common UVD files since we now have base structures for R600 and radeonsi. Signed-off-by: Christian König --- src/gallium/drivers/r600/r600_pipe.c|2 +- src/gallium/drivers/r600/r600_pipe.h|5 - src/gallium/drivers

Re: [Mesa-dev] The long way to a faster build with shared libs and some fixes ...

2013-09-11 Thread Christian König
I completely agree. Building everything shared might speed up the build process a little bit and save some space, but for the cost of having to handle allot of rather small shared libraries where which each clashing the symbol space of any application using these drivers with a bunch of unnece

Re: [Mesa-dev] The long way to a faster build with shared libs and some fixes ...

2013-09-12 Thread Christian König
Am Donnerstag, 12. September 2013, 00:44:58 schrieb Kenneth Graunke: On 09/11/2013 11:41 PM, Christian König wrote: I completely agree. Building everything shared might speed up the build process a little bit and save some space, but for the cost of having to handle allot of rather small shared

Re: [Mesa-dev] [PATCH] vdpau/decode: Check max width and max height.

2013-09-15 Thread Christian König
Reviewed and committed. Thanks, Christian. - Ursprüngliche Mail - Von: "Rico Schüller" An: mesa-dev@lists.freedesktop.org CC: "Rico Schüller" Gesendet: Samstag, 14. September 2013 20:27:07 Betreff: [Mesa-dev] [PATCH] vdpau/decode: Check max width and max height. --- src/galli

[Mesa-dev] [PATCH] implement NV_vdpau_interop v2

2013-09-20 Thread Christian König
From: Christian König v2: Actually implement interop between the gallium state tracker and the VDPAU backend. Signed-off-by: Christian König --- src/gallium/include/state_tracker/vdpau_interop.h | 49 src/gallium/state_trackers/vdpau/ftab.c | 31 ++- src/gallium

Re: [Mesa-dev] [PATCH] implement NV_vdpau_interop v2

2013-09-20 Thread Christian König
Am 20.09.2013 17:39, schrieb Marek Olšák: On Fri, Sep 20, 2013 at 4:34 PM, Christian König wrote: [SNIP] diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index 34615e3..7070812 100644 --- a/src/mesa/main/extensions.c +++ b/src/mesa/main/extensions.c @@ -332,6 +332,7

[Mesa-dev] [PATCH 1/2] winsys/radeon: share winsys between different fd's

2013-09-21 Thread Christian König
From: Christian König Share the winsys between different fd's if they point to the same device. Signed-off-by: Christian König --- src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/src/gallium/w

[Mesa-dev] [PATCH 2/2] implement NV_vdpau_interop v3

2013-09-21 Thread Christian König
From: Christian König v2: Actually implement interop between the gallium state tracker and the VDPAU backend. v3: Make it also available in non legacy contexts, fix video buffer sharing. Signed-off-by: Christian König --- src/gallium/include/state_tracker/vdpau_interop.h | 49

[Mesa-dev] [PATCH 1/5] winsys/radeon: fix killing the CS thread

2013-09-21 Thread Christian König
From: Christian König Kill the thread only after we checked that it's not used any more, not before. Signed-off-by: Christian König --- src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gallium/winsys/radeo

[Mesa-dev] [PATCH 4/5] implement NV_vdpau_interop v3

2013-09-21 Thread Christian König
From: Christian König v2: Actually implement interop between the gallium state tracker and the VDPAU backend. v3: Make it also available in non legacy contexts, fix video buffer sharing. Signed-off-by: Christian König --- src/gallium/include/state_tracker/vdpau_interop.h | 49

[Mesa-dev] [PATCH 5/5] radeon/uvd: async flush the UVD cs

2013-09-21 Thread Christian König
From: Christian König No need to block for the CS thread here. 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/drivers/radeon/radeon_uvd.c b/src/gallium/drivers/radeon/radeon_uvd.c

[Mesa-dev] [PATCH 3/5] winsys/radeon: share winsys between different fd's

2013-09-21 Thread Christian König
From: Christian König Share the winsys between different fd's if they point to the same device. Signed-off-by: Christian König --- src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/src/gallium/w

[Mesa-dev] [PATCH 2/5] winsys/radeon: remove cs_queue_empty

2013-09-21 Thread Christian König
From: Christian König Waiting for an empty queue is nonsense and can lead to deadlocks if we have multiple waiters or another thread that continuously sends down new commands. Just post the cs to the queue and immediately wait for it to finish. This is a candidate for the stable branch

Re: [Mesa-dev] [PATCH 2/2] implement NV_vdpau_interop v3

2013-09-21 Thread Christian König
l VDPAU textures should be y-inverted. Is that actually the case here? 8) Do we actually have enough tests, so that we can say this feature works? Marek On Sat, Sep 21, 2013 at 12:32 PM, Christian König wrote: From: Christian König v2: Actually implement interop between the gallium state t

Re: [Mesa-dev] [PATCH 2/2] implement NV_vdpau_interop v3

2013-09-21 Thread Christian König
Am 21.09.2013 17:35, schrieb Stephan Raue: Am 21.09.2013 16:56, schrieb Marek Olšák: 8) Do we actually have enough tests, so that we can say this feature works? I have integrated v2 of this patches yesterday in OpenELEC, an embedded XBMC distro and it looks already nice. I will start to cr

Re: [Mesa-dev] [PATCH 20/29] st/vdpau: consolidate C sources list into Makefile.sources

2013-09-23 Thread Christian König
Am 22.09.2013 22:29, schrieb Emil Velikov: Signed-off-by: Emil Velikov While it's not absolutely necessary for the VDPAU state tracker it makes sense consequently use Makefile.source all around the place. Reviewed-by: Christian König --- src/gallium/state_trackers/vdpau/Makefi

Re: [Mesa-dev] [PATCH 25/29] st/xvmc: consolidate C sources list into Makefile.sources

2013-09-23 Thread Christian König
Am 22.09.2013 22:29, schrieb Emil Velikov: Signed-off-by: Emil Velikov Reviewed-by: Christian König --- src/gallium/state_trackers/xvmc/Makefile.am | 8 ++-- src/gallium/state_trackers/xvmc/Makefile.sources | 6 ++ 2 files changed, 8 insertions(+), 6 deletions(-) create

[Mesa-dev] [PATCH 3/7] st/vdpau: use new vlc function to serach for VC-1 start codes

2013-09-23 Thread Christian König
From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/decode.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/vdpau/decode.c b/src/gallium/state_trackers/vdpau/decode.c index b144b83..e884fb2 100644 --- a

[Mesa-dev] [PATCH 2/7] vl/mpeg12: use new vlc function to search for start codes

2013-09-23 Thread Christian König
From: Christian König Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_mpeg12_bitstream.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/vl/vl_mpeg12_bitstream.c b/src/gallium/auxiliary/vl/vl_mpeg12_bitstream.c index cfa5eab..b03ad07

[Mesa-dev] [PATCH 1/7] vl/vlc: add fast forward search for byte value

2013-09-23 Thread Christian König
From: Christian König Commonly used to find start codes and has far less overhead to searching manually. Signed-off-by: Christian König --- src/gallium/auxiliary/vl/vl_vlc.h | 84 ++- 1 file changed, 74 insertions(+), 10 deletions(-) diff --git a/src

[Mesa-dev] [PATCH 4/7] st/vdpau: use a separate lock per decoder

2013-09-23 Thread Christian König
From: Christian König Signed-off-by: Christian König --- src/gallium/state_trackers/vdpau/decode.c| 20 +++- src/gallium/state_trackers/vdpau/vdpau_private.h | 1 + 2 files changed, 12 insertions(+), 9 deletions(-) diff --git a/src/gallium/state_trackers/vdpau

[Mesa-dev] [PATCH 6/7] radeon/uvd: try to place msg/fb buffer into GART

2013-09-23 Thread Christian König
From: Christian König This is only supported on NI+, but the kernel takes care of those limitations. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/radeon/radeon_uvd.c b

[Mesa-dev] [PATCH 5/7] radeon/uvd: move alignment to winsys

2013-09-23 Thread Christian König
From: Christian König Similar to GFX and DMA. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c | 6 -- src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 6 ++ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/gallium/drivers/radeon

[Mesa-dev] [PATCH 7/7] radeon/winsys: keep screen pointer in winsys

2013-09-23 Thread Christian König
From: Christian König Only create one screen for each winsys instance. This helps with buffer sharing and interop handling. Signed-off-by: Christian König --- src/gallium/drivers/r300/r300_screen.c| 3 +++ src/gallium/drivers/r600/r600_pipe.c | 3 +++ src/gallium

Re: [Mesa-dev] [PATCH] R600/SI: fix MIMG writemask adjustement

2013-09-24 Thread Christian König
Why should that be necessary? Christian. Am 23.09.2013 21:09, schrieb mar...@gmail.com: From: Marek Olšák This fixes piglit: - shaders/glsl-fs-texture2d-masked - shaders/glsl-fs-texture2d-masked-4 Signed-off-by: Marek Olšák --- lib/Target/R600/SIISelLowering.cpp | 27 +

Re: [Mesa-dev] [PATCH 7/7] radeon/winsys: keep screen pointer in winsys

2013-09-24 Thread Christian König
|xvmc ... radeonsi |-dri |-vdpau |-xorg ... Christian. PS: Sorry that I send this mail from the AMD address, but my mail provider once more has server problems. Marek On Mon, Sep 23, 2013 at 3:58 PM, Christian König wrote: From: Christian König Only create one screen for each winsys insta

Re: [Mesa-dev] [PATCH] R600/SI: fix MIMG writemask adjustement

2013-09-24 Thread Christian König
, 2013 at 9:00 AM, Christian König wrote: Why should that be necessary? Christian. Am 23.09.2013 21:09, schrieb mar...@gmail.com: From: Marek Olšák This fixes piglit: - shaders/glsl-fs-texture2d-masked - shaders/glsl-fs-texture2d-masked-4 Signed-off-by: Marek Olšák --- lib/Target/R600

Re: [Mesa-dev] [PATCH] R600/SI: fix MIMG writemask adjustement

2013-09-24 Thread Christian König
. Maybe some pass is executed twice? Marek On Tue, Sep 24, 2013 at 11:48 AM, Christian König wrote: Sorry, my fault let me refine the question: Why the heck is the function applied twice? Christian. Am 24.09.2013 11:44, schrieb Marek Olšák: If the TGSI writemask is .xzw and the initial dma

[Mesa-dev] [PATCH 1/4] build/r300: group build target in common subdir

2013-09-25 Thread Christian König
From: Christian König Allows us to share more code between different targets. Signed-off-by: Christian König --- configure.ac | 8 ++-- src/gallium/targets/Makefile.am| 6 +-- src/gallium/targets/dri-r300/Makefile.am | 72

[Mesa-dev] [PATCH 4/4] radeon/winsys: keep screen pointer in winsys v2

2013-09-25 Thread Christian König
From: Christian König Only create one screen for each winsys instance. This helps with buffer sharing and interop handling. v2: rebased and some minor cleanup Signed-off-by: Christian König --- src/gallium/drivers/r300/r300_screen.c| 3 +++ src/gallium/drivers/r600/r600_pipe.c

[Mesa-dev] [PATCH 3/4] build/radeonsi: group all targets in common subdir

2013-09-25 Thread Christian König
From: Christian König Allows us to share more code between different targets. Signed-off-by: Christian König --- configure.ac | 8 +- src/gallium/targets/Makefile.am| 6 +- src/gallium/targets/dri-radeonsi/Makefile.am | 72

Re: [Mesa-dev] [PATCH 00/18] Consolidating r600_texture.c

2013-09-25 Thread Christian König
Am 26.09.2013 03:35, schrieb Marek Olšák: Nothing too exciting, I'm just consolidating some code between r600g and radeonsi. There are some small improvements though: 1) The CMASK buffer for MSAA colorbuffers is cleared with CP DMA instead of using the CPU. 2) This series enables 2D tiling for

[Mesa-dev] [PATCH 1/2] radeon/uvd: optimize message handling a bit

2013-09-26 Thread Christian König
From: Christian König No need to keep a copy of the message in system memory anymore, since it should now be in GART memory on newer chips. Signed-off-by: Christian König --- src/gallium/drivers/radeon/radeon_uvd.c | 97 ++--- 1 file changed, 53 insertions(+), 44

[Mesa-dev] [PATCH 2/2] radeon/vdpau: only export necessary symbols

2013-09-26 Thread Christian König
From: Christian König Export only the absolutely necessary symbols in radeon vdpau targets. Signed-off-by: Christian König --- src/gallium/targets/r300/vdpau/Makefile.am | 3 +++ src/gallium/targets/r600/vdpau/Makefile.am | 3 +++ src/gallium/targets/radeonsi/vdpau/Makefile.am | 3

Re: [Mesa-dev] [PATCH 09/12] R600/SI: replace AllReg_* with [SV]Src_*

2013-02-13 Thread Christian König
Am 12.02.2013 21:49, schrieb Michel Dänzer: On Die, 2013-02-12 at 18:13 +0100, Christian König wrote: From: Christian König Mark all the operands that can also have an immediate. Signed-off-by: Christian König --- lib/Target/R600/SIInstrFormats.td | 32 +- lib/Target/R600

Re: [Mesa-dev] [PATCH 07/12] R600/SI: move *_Helper definitions to SIInstrFormat.td

2013-02-13 Thread Christian König
Am 13.02.2013 01:20, schrieb Tom Stellard: On Tue, Feb 12, 2013 at 06:13:19PM +0100, Christian König wrote: From: Christian König SIInstrFormats.td should contain the instruction encoding definitions and everything else should go in SIInstrInfo.td. I got this backwards, when I first created

Re: [Mesa-dev] [PATCH 10/12] R600/SI: cleanup literal handling

2013-02-13 Thread Christian König
Am 13.02.2013 01:39, schrieb Tom Stellard: [SNIP] Way back when I first started working on the backend I was using immediate operands in instructions defined to only uses registers, and it worked most of the time, but I ran into a few cases where some of the passes weren't able to handle it. So

Re: [Mesa-dev] [PATCH 10/12] R600/SI: cleanup literal handling

2013-02-13 Thread Christian König
Am 13.02.2013 08:00, schrieb Michel Dänzer: On Die, 2013-02-12 at 19:39 -0500, Tom Stellard wrote: On Tue, Feb 12, 2013 at 06:13:22PM +0100, Christian König wrote: From: Christian König Seems to be allot simpler, and also paves the way for further improvements. [...] diff --git a/lib

Re: [Mesa-dev] [PATCH v2] R600/SI: Fix INTERP_CONST.

2013-02-13 Thread Christian König
Am 13.02.2013 17:07, schrieb Michel Dänzer: From: Michel Dänzer The important fix is that the constant interpolation value is stored in the parameter slot P0, which is encoded as 2. In addition, pass the parameter slot as an operand to V_INTERP_MOV_F32 instead of hardcoding it there, and add a

Re: [Mesa-dev] [PATCH 01/12] R600/structurizer: add class to find the Nearest Common Dominator

2013-02-13 Thread Christian König
Am 13.02.2013 18:11, schrieb Michel Dänzer: On Mit, 2013-02-13 at 11:34 -0500, Tom Stellard wrote: There's just the one cleanup on patch 10 that you mentioned, but otherwise the series looks good to me. Should we mark all these patches as candidates for the stable branch? I think so, at least

Re: [Mesa-dev] [PATCH] R600: Add lit tests for texture sampling instruction selection.

2013-02-14 Thread Christian König
Am 14.02.2013 09:20, schrieb Michel Dänzer: On Die, 2013-02-12 at 15:22 +0100, Tom Stellard wrote: Reviewed-by: Tom Stellard Thanks, committed as revision 175138. Unfortunately, I only noticed afterwards that the new SI test failed an assertion on trunk, see below. I committed revision 175139

Re: [Mesa-dev] [PATCH 01/12] R600/structurizer: add class to find the Nearest Common Dominator

2013-02-14 Thread Christian König
Am 13.02.2013 18:22, schrieb Michel Dänzer: On Mit, 2013-02-13 at 18:17 +0100, Christian König wrote: Am 13.02.2013 18:11, schrieb Michel Dänzer: On Mit, 2013-02-13 at 11:34 -0500, Tom Stellard wrote: There's just the one cleanup on patch 10 that you mentioned, but otherwise the series

Re: [Mesa-dev] r600g: status of my work on the shader optimization

2013-02-14 Thread Christian König
Hi Vadim, nice work, I think you've made quite a progress here, but on the other hand it should be clear that the LLVM backend is the future and we should concentrate on that. To sum it up I'm not sure what we should do with this branch :) As Dragomir already wrote even if the code won't be

[Mesa-dev] [PATCH 02/12] R600/structurizer: improve PHI value finding

2013-02-14 Thread Christian König
From: Christian König Using the new NearestCommonDominator class. Signed-off-by: Christian König --- lib/Target/R600/AMDGPUStructurizeCFG.cpp |6 ++ 1 file changed, 6 insertions(+) diff --git a/lib/Target/R600/AMDGPUStructurizeCFG.cpp b/lib/Target/R600/AMDGPUStructurizeCFG.cpp index

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