Re: [Mesa-dev] [PATCH V2 2/3] glsl: add new expression types for interpolateAt*

2013-11-11 Thread Chris Forbes
Sorry, that commit message ended up a mess. On Tue, Nov 12, 2013 at 8:45 PM, Chris Forbes wrote: > Will be used to implement interpolateAtCentroid(), interpolateAtOffset() > and interpolateAtSample() from ARB_gpu_shader5 > Will be used to implement interpolateAtCentroid() from ARB_gpu_shader5. >

[Mesa-dev] [PATCH V2 0/3] ARB_gpu_shader5 interpolateAt* plumbing

2013-11-11 Thread Chris Forbes
This series adds the driver-independent bits for the following new builtin functions in ARB_gpu_shader5: - interpolateAtCentroid - interpolateAtOffset - interpolateAtSample Passes all the compiler piglits for these builtins. Big changes from the RFC series from the other day: - Don't abuse ir_v

[Mesa-dev] [PATCH V2 3/3] glsl: add new interpolateAt* builtin functions

2013-11-11 Thread Chris Forbes
V2: - Don't assume everyone wants interpolateAtSample() lowered to interpolateAtOffset. It turns out this isn't what we want most of the time for i965. Lowering can be added later in an ir pass which drivers opt into, rather than bolting it straight into the builtin definiti

[Mesa-dev] [PATCH V2 1/3] allow builtin functions to require parameters to be shader inputs

2013-11-11 Thread Chris Forbes
The new interpolateAt* builtins have strange restrictions on the parameter. - It must be a shader input, or an element of a shader input array. - It must not include a swizzle. V2: Don't abuse ir_var_mode_shader_in for this; make a new flag. Signed-off-by: Chris Forbes --- src/glsl/ast_functi

[Mesa-dev] [PATCH V2 2/3] glsl: add new expression types for interpolateAt*

2013-11-11 Thread Chris Forbes
Will be used to implement interpolateAtCentroid(), interpolateAtOffset() and interpolateAtSample() from ARB_gpu_shader5 Will be used to implement interpolateAtCentroid() from ARB_gpu_shader5. Signed-off-by: Chris Forbes --- src/glsl/ir.cpp| 6 ++ src/glsl/ir.h

Re: [Mesa-dev] [PATCH] nicer-no-wrap-patch

2013-11-11 Thread Rogovin, Kevin
Hello, A subsequent patch has been sent out, and to put my head firmly into a paper bag, what should have been a two dry runs were not dry runs and got sent out; worse, it was sent three times... Sighs. My sincere apologies. Command lines are dangerous. As a side note, I do not know how the 2

[Mesa-dev] [PATCH] nicer-no-wrap-patch

2013-11-11 Thread Kevin Rogovin
Track bytes written during no flush phases for debug builds --- src/mesa/drivers/dri/i965/brw_context.h | 85 +++ src/mesa/drivers/dri/i965/brw_draw.c | 4 +- src/mesa/drivers/dri/i965/brw_state_batch.c | 15 + src/mesa/drivers/dri/i965/intel_batchb

[Mesa-dev] [PATCH] nicer-no-wrap-patch

2013-11-11 Thread Kevin Rogovin
feedback integrated space-pace --- src/mesa/drivers/dri/i965/brw_context.h | 85 +++ src/mesa/drivers/dri/i965/brw_draw.c | 4 +- src/mesa/drivers/dri/i965/brw_state_batch.c | 15 + src/mesa/drivers/dri/i965/intel_batchbuffer.h | 6 ++ 4 files change

[Mesa-dev] [PATCH] nicer-no-wrap-patch

2013-11-11 Thread Kevin Rogovin
feedback integrated space-pace --- src/mesa/drivers/dri/i965/brw_context.h | 85 +++ src/mesa/drivers/dri/i965/brw_draw.c | 4 +- src/mesa/drivers/dri/i965/brw_state_batch.c | 15 + src/mesa/drivers/dri/i965/intel_batchbuffer.h | 6 ++ 4 files change

Re: [Mesa-dev] [PATCH] nicer-no-wrap-patch

2013-11-11 Thread Rogovin, Kevin
Hi all, I will later submit a patch taking into account comments, however one comment I will address *now*. Eric Anholt [e...@anholt.net] writes: >Kevin Rogovin writes: > >> This patch adds a function interface for enabling no wrap on batch commands, >>adds to it assert enforcement that the num

Re: [Mesa-dev] [PATCH] i965: Make the driver compile until a proper libdrm can be released.

2013-11-11 Thread Meng, Mengmeng
Hi Eric, Mesa complied error on 10.0 and it was fixed with this patch. Could you push this patch to Mesa 10.0? Errors(git-035cce8): brw_context.c: In function 'brwCreateContext': brw_context.c:721:9: error: implicit declaration of function 'drm_intel_get_reset_stats' [-Werror=implicit-function-

[Mesa-dev] [PATCH] EGL: fix build without libdrm

2013-11-11 Thread Samuel Thibault
This fixes building EGL without libdrm support. Signed-off-by: Samuel Thibault diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c index a64f4e8..e703f28 100644 --- a/src/egl/drivers/dri2/egl_dri2.c +++ b/src/egl/drivers/dri2/egl_dri2.c @@ -33,8 +33,10 @@ #include #i

Re: [Mesa-dev] [PATCH 15/37] r300: move the final sources list to Makefile.sources

2013-11-11 Thread Emil Velikov
On 11/11/13 23:00, Tom Stellard wrote: > On Sat, Nov 02, 2013 at 07:00:45PM +, Emil Velikov wrote: >> Signed-off-by: Emil Velikov > > Can you make sure make check still works with this patch, if it does, > then: > make check for in-tree builds works like a charm. But if I try an out-of tree

[Mesa-dev] [PATCH demos v2 1/3] Perf: Add command line capabilities to perf framework

2013-11-11 Thread Courtney Goeltzenleuchter
These were entirely interactive. Adding ability to pass in command line arguments allows future tests to include automated test capabilities. Signed-off-by: Courtney Goeltzenleuchter --- src/perf/copytex.c | 2 +- src/perf/drawoverhead.c | 2 +- src/perf/fbobind.c | 2 +- s

[Mesa-dev] [PATCH demos v2 2/3] perf: Update teximage to measure more formats

2013-11-11 Thread Courtney Goeltzenleuchter
Needed test to measure texture upload speed under a variety of modes (mipmap, source format, internal format, size, etc.) This new test has an interactive run mode like the other Mesa Perf tests but also includes command line options to make it automatable. Fix up code formatting. Integrate review

Re: [Mesa-dev] [PATCH 14/37] r300: add symlink to ralloc.c and register_allocate.c

2013-11-11 Thread Emil Velikov
On 11/11/13 23:03, Tom Stellard wrote: > On Sat, Nov 02, 2013 at 07:00:44PM +, Emil Velikov wrote: >> Make automake's subdir-objects work. >> Update includes. >> >> Signed-off-by: Emil Velikov >> --- >> src/gallium/drivers/r300/Makefile.am | 6 +++--- >> src/gallium/drivers/r300/rallo

[Mesa-dev] [PATCH demos v2 3/3] perf: Add script to run collection of texture formats

2013-11-11 Thread Courtney Goeltzenleuchter
bench_teximage.sh runs all the same formats that teximage ran. Also passes the command line arguments of the script to the benchmark to allow tester to set --csvstyle to full or data for output more easily parsed by scripts and such. Signed-off-by: Courtney Goeltzenleuchter --- src/perf/bench_te

[Mesa-dev] [PATCH demos v2 0/3] Customizable texture upload benchmark

2013-11-11 Thread Courtney Goeltzenleuchter
This patch series updates teximage to support additional test modes and commandline interface. I had started with adding a new test but it was similar to teximage and, as suggested, I'm submitting this patch that simply replaces teximage with this new test. Can run interactively, similar to previou

[Mesa-dev] [PATCH] dri: Remove redundant createNewContext function from __DRIimageDriverExtension

2013-11-11 Thread Kristian Høgsberg
createContextAttribs is a superset of what createNewContext provides. Also remove the function typedef, since createNewContext is deprecated and no longer used in multiple interfaces. Signed-off-by: Kristian Høgsberg --- include/GL/internal/dri_interface.h| 12 src/mesa/drivers

Re: [Mesa-dev] [PATCH] clover: Remove dead file from Makefile.sources.

2013-11-11 Thread Emil Velikov
On 11/11/13 23:54, Matt Turner wrote: Just stumbled on this a bit earlier. Seems line make dist has some benefits. :) Reviewed-by: Emil Velikov > --- > src/gallium/state_trackers/clover/Makefile.sources | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/src/gallium/state_trackers/clover/Ma

Re: [Mesa-dev] [PATCH] clover: Remove dead file from Makefile.sources.

2013-11-11 Thread Tom Stellard
On Mon, Nov 11, 2013 at 03:54:48PM -0800, Matt Turner wrote: > --- Reviewed-by: Tom Stellard > src/gallium/state_trackers/clover/Makefile.sources | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/src/gallium/state_trackers/clover/Makefile.sources > b/src/gallium/state_trackers/clover/Ma

Re: [Mesa-dev] [PATCH] meta: enable vertex attributes in the context of the newly created array object

2013-11-11 Thread Petr Sebor
Not having push access rights, I hope this fix gets merged soon. Hopefully a stable release candidate as well. Thanks, Petr On 12.11.2013 00:32, Brian Paul wrote: On 11/11/2013 04:19 PM, Petr Sebor wrote: Otherwise, the function would enable generic vertex attributes 0 and 1 of the array objec

[Mesa-dev] [PATCH] clover: Remove dead file from Makefile.sources.

2013-11-11 Thread Matt Turner
--- src/gallium/state_trackers/clover/Makefile.sources | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/Makefile.sources b/src/gallium/state_trackers/clover/Makefile.sources index 520f52f..42b3345 100644 --- a/src/gallium/state_trackers/clover/Makefile.sources

Re: [Mesa-dev] [PATCH 3/3] radeon/llvm: Free elf_buffer after use

2013-11-11 Thread Tom Stellard
On Thu, Nov 07, 2013 at 06:08:16PM -0600, Aaron Watry wrote: > Prevents a memory leak. > --- > src/gallium/drivers/radeon/radeon_llvm_emit.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/src/gallium/drivers/radeon/radeon_llvm_emit.c > b/src/gallium/drivers/radeon/radeon_llvm_emit.c

Re: [Mesa-dev] [PATCH 1/3] r600/llvm: initialize radeon_llvm_binary

2013-11-11 Thread Tom Stellard
On Thu, Nov 07, 2013 at 06:08:14PM -0600, Aaron Watry wrote: > use memset to initialize to 0's... otherwise code_size and config_size could > be uninitialized when read later in this method. It's also hard to do NULL > checks on uninitialized pointers. > --- > src/gallium/drivers/r600/r600_llvm.

Re: [Mesa-dev] [PATCH 2/3] r600/llvm: Free binary.code/binary.config in r600_llvm_compile

2013-11-11 Thread Tom Stellard
On Thu, Nov 07, 2013 at 06:08:15PM -0600, Aaron Watry wrote: > radeon_llvm_compile allocates memory for binary.code, binary.config, or > neither depending on > what's being done. > > We need to make sure to free that memory after it's no longer needed. > --- > src/gallium/drivers/r600/r600_llvm.

Re: [Mesa-dev] [PATCH] meta: enable vertex attributes in the context of the newly created array object

2013-11-11 Thread Ian Romanick
On 11/11/2013 03:19 PM, Petr Sebor wrote: > Otherwise, the function would enable generic vertex attributes 0 and 1 of the > array object it does not own. This was causing crashes in Euro Truck > Simulator 2, > since the incorrectly enabled generic attribute 0 in the foreign context got > precedenc

Re: [Mesa-dev] [PATCH 1/3] dri: Add helpers for implementing allocBuffer/releaseBuffer with __DRIimage

2013-11-11 Thread Chad Versace
On 11/11/2013 02:20 PM, Kristian Høgsberg wrote: On Mon, Nov 11, 2013 at 1:57 PM, Chad Versace wrote: On 11/11/2013 01:22 PM, Kristian Høgsberg wrote: Drivers that only call getBuffers to request color buffers can use these generic, __DRIimage based helpers to implement the allocBuffer and re

Re: [Mesa-dev] [PATCH] meta: enable vertex attributes in the context of the newly created array object

2013-11-11 Thread Brian Paul
On 11/11/2013 04:19 PM, Petr Sebor wrote: Otherwise, the function would enable generic vertex attributes 0 and 1 of the array object it does not own. This was causing crashes in Euro Truck Simulator 2, since the incorrectly enabled generic attribute 0 in the foreign context got precedence before

[Mesa-dev] [PATCH] meta: enable vertex attributes in the context of the newly created array object

2013-11-11 Thread Petr Sebor
Otherwise, the function would enable generic vertex attributes 0 and 1 of the array object it does not own. This was causing crashes in Euro Truck Simulator 2, since the incorrectly enabled generic attribute 0 in the foreign context got precedence before vertex position attribute at later time, le

[Mesa-dev] [PATCH] i965: Link test program with -ldl.

2013-11-11 Thread Matt Turner
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=71512 --- src/mesa/drivers/dri/i965/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/i965/Makefile.am b/src/mesa/drivers/dri/i965/Makefile.am index 8c0f9a3..6bce3c1 100644 --- a/src/mesa/drivers/dri/i965/Ma

Re: [Mesa-dev] [PATCH 14/37] r300: add symlink to ralloc.c and register_allocate.c

2013-11-11 Thread Tom Stellard
On Sat, Nov 02, 2013 at 07:00:44PM +, Emil Velikov wrote: > Make automake's subdir-objects work. > Update includes. > > Signed-off-by: Emil Velikov > --- > src/gallium/drivers/r300/Makefile.am | 6 +++--- > src/gallium/drivers/r300/ralloc.c| 1 + > src/gallium/drivers/r30

Re: [Mesa-dev] [PATCH 15/37] r300: move the final sources list to Makefile.sources

2013-11-11 Thread Tom Stellard
On Sat, Nov 02, 2013 at 07:00:45PM +, Emil Velikov wrote: > Signed-off-by: Emil Velikov Can you make sure make check still works with this patch, if it does, then: Reviewed-by: Tom Stellard > --- > src/gallium/drivers/r300/Makefile.am | 13 ++--- > src/gallium/drivers/r300/Ma

Re: [Mesa-dev] [PATCH 07/27] gtest: enable subdir-objects to prevent automake warnings

2013-11-11 Thread Matt Turner
On Mon, Nov 11, 2013 at 10:53 AM, Emil Velikov wrote: > Signed-off-by: Emil Velikov > --- > src/gtest/Makefile.am | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/gtest/Makefile.am b/src/gtest/Makefile.am > index 4188c6b..23ea749 100644 > --- a/src/gtest/Makefile.am > +++ b/src/gtest

[Mesa-dev] [Bug 61364] LLVM assertion when starting X11

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=61364 --- Comment #10 from Tom Stellard --- This should be fixed now in mesa master when linking with llvm static libraries. Can you re-test? -- You are receiving this mail because: You are the assignee for the bug. _

Re: [Mesa-dev] [PATCH 3/3] i965/fs: Try a different pre-scheduling heuristic if the first spills.

2013-11-11 Thread Matt Turner
On Wed, Nov 6, 2013 at 6:47 PM, Eric Anholt wrote: > Since LIFO fails on some shaders in one particular way, and non-LIFO > systematically fails in another way on different kinds of shaders, try > them both. and pick whichever one successfully register allocates first. > Slightly prefer non-LIFO i

[Mesa-dev] [Bug 71506] indirect_glx.c:350: multiple definition of `indirect_create_context'

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71506 --- Comment #8 from Armin K --- Even to be more precise, (manually) reverting this commit fixes the issue, at least here: http://cgit.freedesktop.org/mesa/mesa/commit/?id=0cce5538673148ffcd7aa9479f6b88cf9a641352 Reproducible with GCC 4.8.2 and

Re: [Mesa-dev] [PATCH] i965/fs: Fix message setup for SIMD8 spills.

2013-11-11 Thread Paul Berry
On 5 November 2013 16:21, Eric Anholt wrote: > In the SIMD16 spilling changes, I replaced a "1" in the spill path with > "mlen", but obviously it wasn't mlen before because spills have the g0 > header along with the payload. The interface I was trying to use was > asking for how many physical reg

Re: [Mesa-dev] [PATCH 1/3] i965: Add a warning if something ever hits a bug I noticed.

2013-11-11 Thread Matt Turner
On Tue, Nov 5, 2013 at 5:35 PM, Eric Anholt wrote: > We'd have to map the VBO and rewrite things to a lower stride to fix it. > --- These three are Reviewed-by: Matt Turner ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesk

[Mesa-dev] [Bug 71506] indirect_glx.c:350: multiple definition of `indirect_create_context'

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71506 --- Comment #7 from Armin K --- If you revert changes to Makefile.am made by these two commits, test builds and passess fine: http://cgit.freedesktop.org/mesa/mesa/commit/?id=8c5330226f391a7a29b6538851090b0ef730a239 http://cgit.freedesktop.org/

Re: [Mesa-dev] [PATCH 2/4] i965: Use drm_intel_get_aperture_sizes instead of hard-coded 2GiB

2013-11-11 Thread Daniel Vetter
On Mon, Nov 11, 2013 at 01:45:43PM -0800, Ian Romanick wrote: > On 11/11/2013 01:35 PM, Daniel Vetter wrote: > > On Mon, Nov 11, 2013 at 11:19:07AM -0800, Ian Romanick wrote: > >> From: Ian Romanick > >> > >> Systems with little physical memory installed will report less than > >> 2GiB, and some s

Re: [Mesa-dev] [PATCH 1/3] dri: Add helpers for implementing allocBuffer/releaseBuffer with __DRIimage

2013-11-11 Thread Kristian Høgsberg
On Mon, Nov 11, 2013 at 1:57 PM, Chad Versace wrote: > On 11/11/2013 01:22 PM, Kristian Høgsberg wrote: >> >> Drivers that only call getBuffers to request color buffers can use these >> generic, __DRIimage based helpers to implement the allocBuffer and >> releaseBuffer functions of __DRIdri2Extens

Re: [Mesa-dev] [PATCH] tests: Fix make check for out of tree builds.

2013-11-11 Thread Matt Turner
On Mon, Nov 11, 2013 at 12:18 PM, Rico Schüller wrote: > Signed-off-by: Rico Schüller > --- > src/mapi/shared-glapi/tests/Makefile.am | 1 + > src/mesa/main/tests/Makefile.am | 1 + > 2 Dateien geändert, 2 Zeilen hinzugefügt(+) > > diff --git a/src/mapi/shared-glapi/tests/Makefile.am >

Re: [Mesa-dev] [PATCH 1/2 v3] i965: add XRGB to tiled_memcpy

2013-11-11 Thread Matt Turner
On Mon, Nov 11, 2013 at 1:19 PM, Chad Versace wrote: > On 11/07/2013 01:59 PM, Courtney Goeltzenleuchter wrote: >> >> MESA_FORMAT_XRGB is equivalent to MESA_FORMAT_ARGB in terms >> of storage on the device, so okay to use this optimized copy routine. >> >> This series builds on work from F

[Mesa-dev] [Bug 71506] indirect_glx.c:350: multiple definition of `indirect_create_context'

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71506 --- Comment #6 from Ian Romanick --- I'm really confused now. :( You're sure that bisect is good? There's nothing in that commit that modifies any file under src/glx. I'm also not able to reproduce this even using your configure options. --

Re: [Mesa-dev] [PATCH 1/3] dri: Add helpers for implementing allocBuffer/releaseBuffer with __DRIimage

2013-11-11 Thread Chad Versace
On 11/11/2013 01:22 PM, Kristian Høgsberg wrote: Drivers that only call getBuffers to request color buffers can use these generic, __DRIimage based helpers to implement the allocBuffer and releaseBuffer functions of __DRIdri2Extension. For the intel dri driver, this consolidates window system co

Re: [Mesa-dev] [PATCH 2/4] i965: Use drm_intel_get_aperture_sizes instead of hard-coded 2GiB

2013-11-11 Thread Ian Romanick
On 11/11/2013 01:35 PM, Daniel Vetter wrote: > On Mon, Nov 11, 2013 at 11:19:07AM -0800, Ian Romanick wrote: >> From: Ian Romanick >> >> Systems with little physical memory installed will report less than >> 2GiB, and some systems may (hypothetically?) have a larger address space >> for the GPU.

[Mesa-dev] [Bug 71506] indirect_glx.c:350: multiple definition of `indirect_create_context'

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71506 --- Comment #5 from Vinson Lee --- I see the same output. $ nm -u src/glx/.libs/create_context.o U __assert_fail U free U GetGLXScreenConfigs U _GLOBAL_OFFSET_TABLE_

[Mesa-dev] [Bug 71506] indirect_glx.c:350: multiple definition of `indirect_create_context'

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71506 --- Comment #4 from Ian Romanick --- Hrm... diffing that with mine doesn't show anything obvious. How about 'nm -u src/glx/.libs/create_context.o'? I get: [idr@mumford-wire master-64]$ nm -u src/glx/.libs/create_context.o U _

[Mesa-dev] [Bug 71506] indirect_glx.c:350: multiple definition of `indirect_create_context'

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71506 --- Comment #3 from Vinson Lee --- Created attachment 89058 --> https://bugs.freedesktop.org/attachment.cgi?id=89058&action=edit nm -u src/glx/tests/*.o -- You are receiving this mail because: You are the assignee for the bug. ___

Re: [Mesa-dev] [PATCH v2] i915, i965: Fix memory leak in intel_miptree_create_for_bo.

2013-11-11 Thread Chad Versace
On 09/27/2013 10:20 PM, Vinson Lee wrote: Fixes "Resource leak" defects reported by Coverity. Signed-off-by: Vinson Lee Thanks. Committed. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-d

Re: [Mesa-dev] [PATCH 2/4] i965: Use drm_intel_get_aperture_sizes instead of hard-coded 2GiB

2013-11-11 Thread Daniel Vetter
On Mon, Nov 11, 2013 at 11:19:07AM -0800, Ian Romanick wrote: > From: Ian Romanick > > Systems with little physical memory installed will report less than > 2GiB, and some systems may (hypothetically?) have a larger address space > for the GPU. My IVB still reports 1534. > > Signed-off-by: Ian

[Mesa-dev] [Bug 71506] indirect_glx.c:350: multiple definition of `indirect_create_context'

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71506 --- Comment #2 from Ian Romanick --- That is bizarre. I always 'make check' as part of my build, and I haven't seen that failure. There must be something different about your configure flags that allows this to happen. Something is happening t

Re: [Mesa-dev] [PATCH] i965: Delete #define's and use local variables inside function

2013-11-11 Thread Chad Versace
On 11/07/2013 04:51 PM, Anuj Phogat wrote: X_f, Y_f, Xp_f, Yp_f variables are used just inside translate_dst_to_src().So, they can be defined just as local variables. Signed-off-by: Anuj Phogat Reviewed-by: Chad Versace ___ mesa-dev mailing list m

Re: [Mesa-dev] [PATCH 07/18] i915: Wire up initial support for DRI_RENDERER_QUERY extension

2013-11-11 Thread Daniel Vetter
On Mon, Nov 11, 2013 at 11:03:49AM -0800, Ian Romanick wrote: > On 11/09/2013 02:44 AM, Daniel Vetter wrote: > > On Fri, Oct 11, 2013 at 03:10:14PM -0700, Ian Romanick wrote: > >> From: Ian Romanick > >> > >> Signed-off-by: Ian Romanick > >> --- > >> src/mesa/drivers/dri/i915/intel_screen.c | 79

[Mesa-dev] [PATCH 3/3] wayland: Use __DRIimage based getBuffers implementation when available

2013-11-11 Thread Kristian Høgsberg
This lets us allocate color buffers as __DRIimages and pass them into the driver instead of having to create a __DRIbuffer with the flink that requires. Signed-off-by: Kristian Høgsberg Cc: "10.0" --- src/egl/drivers/dri2/egl_dri2.h | 3 +- src/egl/drivers/dri2/platform_wayland.c | 14

Re: [Mesa-dev] [PATCH 2/2] i965: Enhance tiled_memcpy to support all levels

2013-11-11 Thread Chad Versace
On 11/08/2013 08:13 AM, Courtney Goeltzenleuchter wrote: Support all levels of a supported texture format. Using 1024x1024, RGBA source, mipmap <> internal-format Before (MB/sec) XRGB (MB/sec) mipmap (MB/sec) GL_RGBA 628.

[Mesa-dev] [PATCH 1/3] dri: Add helpers for implementing allocBuffer/releaseBuffer with __DRIimage

2013-11-11 Thread Kristian Høgsberg
Drivers that only call getBuffers to request color buffers can use these generic, __DRIimage based helpers to implement the allocBuffer and releaseBuffer functions of __DRIdri2Extension. For the intel dri driver, this consolidates window system color buffer allocation in intel_create_image(). Sig

[Mesa-dev] [PATCH 2/3] gbm: Add support for __DRIimage based getBuffers when available

2013-11-11 Thread Kristian Høgsberg
This lets us allocate color buffers as __DRIimages and pass them into the driver instead of having to create a __DRIbuffer with the flink that requires. Signed-off-by: Kristian Høgsberg Cc: "10.0" --- src/egl/drivers/dri2/platform_drm.c | 46 ++--- src/gbm/backen

Re: [Mesa-dev] [PATCH 1/2 v3] i965: add XRGB to tiled_memcpy

2013-11-11 Thread Chad Versace
On 11/07/2013 01:59 PM, Courtney Goeltzenleuchter wrote: MESA_FORMAT_XRGB is equivalent to MESA_FORMAT_ARGB in terms of storage on the device, so okay to use this optimized copy routine. This series builds on work from Frank Henigman to optimize the process of uploading a texture to the

Re: [Mesa-dev] [PATCH] mesa: Add ARB_texture_view to Mesa core

2013-11-11 Thread Courtney Goeltzenleuchter
On Mon, Nov 11, 2013 at 11:15 AM, Ian Romanick wrote: > On 11/05/2013 11:36 AM, Courtney Goeltzenleuchter wrote: > > On Tue, Nov 5, 2013 at 12:22 PM, Ian Romanick > > wrote: > > > > On 11/05/2013 09:44 AM, Chris Forbes wrote: > > >> So, you can create a GL_LU

[Mesa-dev] [Bug 71506] indirect_glx.c:350: multiple definition of `indirect_create_context'

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71506 Vinson Lee changed: What|Removed |Added CC||i...@freedesktop.org, |

[Mesa-dev] [PATCH] tests: Fix make check for out of tree builds.

2013-11-11 Thread Rico Schüller
Signed-off-by: Rico Schüller --- src/mapi/shared-glapi/tests/Makefile.am | 1 + src/mesa/main/tests/Makefile.am | 1 + 2 Dateien geändert, 2 Zeilen hinzugefügt(+) diff --git a/src/mapi/shared-glapi/tests/Makefile.am b/src/mapi/shared-glapi/tests/Makefile.am index 98065fc..7e71b4f 100644

Re: [Mesa-dev] [PATCH V3 00/13] ARB_draw_indirect for i965

2013-11-11 Thread Chris Forbes
Thanks, Paul. Now just seeking r-b for 1, 6, and 12. Anyone? On Tue, Nov 12, 2013 at 8:37 AM, Paul Berry wrote: > On 9 November 2013 01:02, Chris Forbes wrote: >> >> This series adds support for ARB_draw_indirect and ARB_multi_draw_indirect >> on Gen7+. >> >> Big changes: >> - Get rid of overco

Re: [Mesa-dev] [PATCH] nicer-no-wrap-patch

2013-11-11 Thread Eric Anholt
Kevin Rogovin writes: > This patch adds a function interface for enabling no wrap on batch commands, > adds to it assert enforcement that the number bytes added to the > batch buffer does not exceed a passed value and finally this is used > in brw_try_draw_prims() to help make sure that estimated

Re: [Mesa-dev] [PATCH V3 00/13] ARB_draw_indirect for i965

2013-11-11 Thread Paul Berry
On 9 November 2013 01:02, Chris Forbes wrote: > This series adds support for ARB_draw_indirect and ARB_multi_draw_indirect > on Gen7+. > > Big changes: > - Get rid of overcomplicated state tracking for the indirect bo in i965. > - Make primcount==0 produce GL_INVALID_VALUE in MultiDraw*Indirect >

[Mesa-dev] [PATCH 4/4] i915: Actually enable __DRI2rendererQueryExtensionRec

2013-11-11 Thread Ian Romanick
From: Ian Romanick More rebase fail. This code was written long before i915 and i965 were split, so most of the code in i9[16]5/intel_screen.c only needed to exist in one place. It looks like I fixed n-1 of those places after rebasing on the split. I only found this from the defined-but-not-us

[Mesa-dev] [PATCH 3/4] dri: Change value param to unsigned

2013-11-11 Thread Ian Romanick
From: Ian Romanick This silences some compiler warnings in i915 and i965. See also 75982a5. Signed-off-by: Ian Romanick Cc: "10.0" --- src/mesa/drivers/dri/common/utils.c | 2 +- src/mesa/drivers/dri/common/utils.h | 2 +- src/mesa/drivers/dri/i915/intel_screen.c | 2 +- src/mesa/d

[Mesa-dev] [PATCH 1/4] i915: Use drm_intel_get_aperture_sizes instead of drmAgpSize

2013-11-11 Thread Ian Romanick
From: Ian Romanick Send the zombie back to the grave before it infects the townsfolk. Signed-off-by: Ian Romanick Cc: Daniel Vetter Cc: "10.0" --- src/mesa/drivers/dri/i915/intel_screen.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i915/in

[Mesa-dev] [PATCH 2/4] i965: Use drm_intel_get_aperture_sizes instead of hard-coded 2GiB

2013-11-11 Thread Ian Romanick
From: Ian Romanick Systems with little physical memory installed will report less than 2GiB, and some systems may (hypothetically?) have a larger address space for the GPU. My IVB still reports 1534. Signed-off-by: Ian Romanick Cc: Daniel Vetter Cc: "10.0" --- src/mesa/drivers/dri/i965/inte

Re: [Mesa-dev] [PATCH 07/18] i915: Wire up initial support for DRI_RENDERER_QUERY extension

2013-11-11 Thread Ian Romanick
On 11/09/2013 02:44 AM, Daniel Vetter wrote: > On Fri, Oct 11, 2013 at 03:10:14PM -0700, Ian Romanick wrote: >> From: Ian Romanick >> >> Signed-off-by: Ian Romanick >> --- >> src/mesa/drivers/dri/i915/intel_screen.c | 79 >> >> 1 file changed, 79 insertions(+) >

Re: [Mesa-dev] [RFC] Haiku viewport / framebuffer invalidation

2013-11-11 Thread Jose Fonseca
- Original Message - > I've been banging my head against a wall for a while now on this. > > So the Haiku applications that call glViewport(.. for window > resizes,etc never actually execute the Driver's Viewport call. > (aka ctx->Driver.Viewport: > http://cgit.freedesktop.org/mesa/mesa/

[Mesa-dev] [PATCH 22/27] targets/xvmc: use drop duplicated compiler flags

2013-11-11 Thread Emil Velikov
Automake.inc already has GALLIUM_VIDEO_CFLAGS, which provide the essential compiler flags needed. Note: this commit adds VISIBILITY_CFLAGS to nouveau. Signed-off-by: Emil Velikov --- src/gallium/targets/r600/xvmc/Makefile.am| 8 +--- src/gallium/targets/xvmc-nouveau/Makefile.am | 7 +---

[Mesa-dev] [PATCH 24/27] targets/xvmc: consolidate lib deps into Automake.inc

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/Automake.inc | 6 ++ src/gallium/targets/r600/xvmc/Makefile.am| 5 + src/gallium/targets/xvmc-nouveau/Makefile.am | 5 + 3 files changed, 8 insertions(+), 8 deletions(-) diff --git a/src/gallium/Automake.inc b/src/ga

[Mesa-dev] [PATCH 21/27] gallium/winsys: compact compiler flags into Automake.inc

2013-11-11 Thread Emil Velikov
Cleanup the duplicating flags and consolidate into a sigle variable. Note: this patch adds VISIBILITY_CFLAGS to the following targets * freedreno/drm * i915/{drm,sw} * nouveau/drm * sw/fbdev * sw/null * sw/wayland * sw/wrapper * sw/xlib Signed-off-by: Emil Velikov --- src/gallium/Automake.inc

[Mesa-dev] [PATCH 03/27] scons: drop obsolete enabled_apis variable

2013-11-11 Thread Emil Velikov
The variable was forgotten during the FEATURE_* removal. Signed-off-by: Emil Velikov --- src/mesa/SConscript | 7 --- 1 file changed, 7 deletions(-) diff --git a/src/mesa/SConscript b/src/mesa/SConscript index 4213498..a2bb9f1 100644 --- a/src/mesa/SConscript +++ b/src/mesa/SConscript @@ -1

[Mesa-dev] [PATCH 06/27] gbm: enable subdir-objects to prevent automake warnings

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gbm/Makefile.am | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gbm/Makefile.am b/src/gbm/Makefile.am index 1282b14..9b3fb0d 100644 --- a/src/gbm/Makefile.am +++ b/src/gbm/Makefile.am @@ -1,3 +1,5 @@ +AUTOMAKE_OPTIONS = subdir-objects + pkgconfigdir

[Mesa-dev] [PATCH 05/27] scons: move SConscript from gallium/targets/ to mesa/drivers/dri/common/

2013-11-11 Thread Emil Velikov
Store scons side by side with the other build systems. v2: cleanup after a failed rebase Signed-off-by: Emil Velikov --- src/gallium/SConscript | 1 - src/gallium/targets/SConscript.dri | 83 -- src/mesa/drivers/SConscript| 5 +-

[Mesa-dev] [PATCH 27/27] targets/dri: move linker flags out of configure into Automake.inc

2013-11-11 Thread Emil Velikov
Previous assumption was that the same set of flags can be reused for both classic and gallium drivers. With megadriver work done the classic drivers ended up using their own (single) instance of the flags. Move these into Automake.inc and rename to indicate that those are gallium specific. Additio

[Mesa-dev] [PATCH 11/27] r300: move the final sources list to Makefile.sources

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/drivers/r300/Makefile.am | 13 ++--- src/gallium/drivers/r300/Makefile.sources | 14 +- 2 files changed, 15 insertions(+), 12 deletions(-) diff --git a/src/gallium/drivers/r300/Makefile.am b/src/gallium/drivers/r300/Makefile.a

[Mesa-dev] [PATCH 01/27] st/egl: use *_FILE over *_SOURCES names for filelists

2013-11-11 Thread Emil Velikov
Silence automake warnings about missing program/library whenever the _SOURCES suffix is used for temporary variable names. warning: variable 'gdi_SOURCES' is defined but no program or library has 'gdi' as canonical name (possible typo) Reported-by: Ilia Mirkin Reported-by: Johannes Obermayr

[Mesa-dev] [PATCH 10/27] r300: add symlink to ralloc.c and register_allocate.c

2013-11-11 Thread Emil Velikov
Make automake's subdir-objects work. Update includes. Signed-off-by: Emil Velikov --- src/gallium/drivers/r300/Makefile.am | 6 +++--- src/gallium/drivers/r300/ralloc.c| 1 + src/gallium/drivers/r300/register_allocate.c | 1 + 3 files changed, 5 insertions(+), 3 deletions(-)

[Mesa-dev] [PATCH 17/27] targets/vdpau: compact compiler flags into Automake.inc

2013-11-11 Thread Emil Velikov
Store the compiler flags into a variable, in order to minimise flags duplication (amongst vdpau and xvmc). Note: this commit add VISIBILITY_CFLAGS to the nouveau target Signed-off-by: Emil Velikov --- src/gallium/Automake.inc | 11 +++ src/gallium/targets/r600/vdpa

[Mesa-dev] [PATCH 20/27] targets/vdpau: drop unused libraries from linker

2013-11-11 Thread Emil Velikov
In order for one to use trace, noop, rbug and/or galahad, they must set the corresponding GALLIUM_* CFLAG. Signed-off-by: Emil Velikov --- src/gallium/targets/r600/vdpau/Makefile.am | 1 - src/gallium/targets/radeonsi/vdpau/Makefile.am | 3 --- src/gallium/targets/vdpau-nouveau/Makefile.am

[Mesa-dev] [PATCH 04/27] freedreno: compact a2xx and a3xx makefiles into parent ones

2013-11-11 Thread Emil Velikov
From: Johannes Obermayr Nearly everything within the three Makefile.am's is identical. Let's simplify things a little. v2: Rebase and rewrite the commit message (Emil Velikov) Signed-off-by: Emil Velikov --- configure.ac | 2 -- src/gallium/drivers/freed

[Mesa-dev] [PATCH 19/27] targets/vdpau: consolidate lib deps into Automake.inc

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/Automake.inc | 7 +++ src/gallium/targets/r600/vdpau/Makefile.am | 5 + src/gallium/targets/radeonsi/vdpau/Makefile.am | 5 + src/gallium/targets/vdpau-nouveau/Makefile.am | 5 + 4 files changed, 10 insertions(

[Mesa-dev] [PATCH 15/27] targets/radeonsi: move drm_target.c to a common folder

2013-11-11 Thread Emil Velikov
... and symlink to each target. Make automake's subdir-objects work for radeonsi. Signed-off-by: Emil Velikov --- src/gallium/targets/radeonsi/common/drm_target.c | 69 src/gallium/targets/radeonsi/dri/Makefile.am | 2 +- src/gallium/targets/radeonsi/dri/drm_target.

[Mesa-dev] [PATCH 26/27] targets/dri: compact compiler flags into Automake.inc

2013-11-11 Thread Emil Velikov
Greatly reduce duplication and provide a sane minimum of CFLAGS for all DRI targets. Note: This commit adds VISIBILITY_CFLAGS to the following: * freedreno * i915 * ilo * nouveau * vmwgfx Signed-off-by: Emil Velikov --- src/gallium/Automake.inc | 13 + src/galli

[Mesa-dev] [PATCH 25/27] targets/xvmc: do not link against libtrace.la

2013-11-11 Thread Emil Velikov
In order to use the trace driver, one needs to define GALLIUM_TRACE. Neither one of the two targets was defining it, thus we're safe to remove libtrace.la. Signed-off-by: Emil Velikov --- src/gallium/targets/r600/xvmc/Makefile.am| 1 - src/gallium/targets/xvmc-nouveau/Makefile.am | 1 - 2 fi

[Mesa-dev] [PATCH 09/27] st/xvmc: enable automake subdir-objects

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/state_trackers/xvmc/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/xvmc/Makefile.am b/src/gallium/state_trackers/xvmc/Makefile.am index c110912..3968238 100644 --- a/src/gallium/state_trackers/xvmc/Makefile.a

[Mesa-dev] [PATCH 07/27] gtest: enable subdir-objects to prevent automake warnings

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gtest/Makefile.am | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gtest/Makefile.am b/src/gtest/Makefile.am index 4188c6b..23ea749 100644 --- a/src/gtest/Makefile.am +++ b/src/gtest/Makefile.am @@ -18,6 +18,7 @@ # LIABILITY, WHETHER IN AN ACTION OF CON

[Mesa-dev] [PATCH 23/27] targets/xvmc: move linker flags to Automake.inc

2013-11-11 Thread Emil Velikov
Minimise duplication and sources of error (eg nouveau was missing shared and no-undefined) Signed-off-by: Emil Velikov --- src/gallium/Automake.inc | 7 +++ src/gallium/targets/r600/xvmc/Makefile.am| 5 + src/gallium/targets/xvmc-nouveau/Makefile.am | 5 + 3 f

[Mesa-dev] [PATCH 08/27] dri/common: move source file lists to Makefile.sources

2013-11-11 Thread Emil Velikov
* Allow the lists to be shared among build systems. * Update automake and Android build systems. Signed-off-by: Emil Velikov Reviewed-by: Ian Romanick --- src/mesa/drivers/dri/common/Android.mk | 2 +- src/mesa/drivers/dri/common/Makefile.am | 10 -- src/mesa/drivers/dri/com

[Mesa-dev] [PATCH 18/27] targets/vdpau: move linker flags to Automake.inc

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/Automake.inc | 7 +++ src/gallium/targets/r600/vdpau/Makefile.am | 10 +++--- src/gallium/targets/radeonsi/vdpau/Makefile.am | 10 +++--- src/gallium/targets/vdpau-nouveau/Makefile.am | 7 +++ 4 files changed

[Mesa-dev] [PATCH 02/27] Android: remove unused MESA_ENABLED_APIS variable

2013-11-11 Thread Emil Velikov
The variable was forgotten during the FEATURE_* removal. Signed-off-by: Emil Velikov --- src/mesa/Android.libmesa_dricore.mk | 2 -- src/mesa/Android.libmesa_st_mesa.mk | 2 -- src/mesa/program/Android.mk | 2 -- 3 files changed, 6 deletions(-) diff --git a/src/mesa/Android.libmesa_dric

[Mesa-dev] [PATCH 14/27] targets/r600: move drm_target.c to common folder

2013-11-11 Thread Emil Velikov
... and symlink for each target. Make automake's subdir-objects work for r600. Signed-off-by: Emil Velikov --- src/gallium/targets/r600/common/drm_target.c | 69 src/gallium/targets/r600/dri/Makefile.am | 2 +- src/gallium/targets/r600/dri/drm_target.c| 1 +

[Mesa-dev] [PATCH 16/27] gallium/drivers: compact compiler flags into Automake.inc

2013-11-11 Thread Emil Velikov
* minimise flags duplication * distingush between VISIBILITY C and CXX flags * set only required flags - C and/or CXX Signed-off-by: Emil Velikov --- src/gallium/Automake.inc | 22 ++ src/gallium/drivers/freedreno/Makefile.am | 10 -- src/gallium/driv

[Mesa-dev] [PATCH 12/27] gallium/drivers: enable automake subdir-objects

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/gallium/drivers/freedreno/Makefile.am | 2 ++ src/gallium/drivers/ilo/Makefile.am | 2 ++ src/gallium/drivers/nouveau/Makefile.am | 2 ++ src/gallium/drivers/r300/Makefile.am | 2 ++ src/gallium/drivers/r600/Makefile.am | 2 ++ src/gallium/dri

[Mesa-dev] [PATCH 13/27] targets/r300: move drm_target.c to common folder

2013-11-11 Thread Emil Velikov
... and symlink for each target. Make automake's subdir-objects work for r300. Signed-off-by: Emil Velikov --- src/gallium/targets/r300/common/drm_target.c | 54 src/gallium/targets/r300/dri/Makefile.am | 2 +- src/gallium/targets/r300/dri/drm_target.c| 1 +

[Mesa-dev] Automake subdir-objects v2

2013-11-11 Thread Emil Velikov
Hello list, Here is an updated version of the subdir-objects series sent earlier. As a reminder the series aims to * make mesa build system "subdir-objects" compliant * minimise flags duplication across gallium Changes since previous version: * Resolved git-rebase mess in patch 5. * Dropped patc

  1   2   >