Re: [Mesa-dev] [RFC PATCH 0/5] ARB_gpu_shader5 interpolateAt* GLSL plumbing

2013-11-11 Thread Marek Olšák
On Sun, Nov 10, 2013 at 9:51 AM, Chris Forbes chr...@ijw.co.nz wrote: Here is the driver-independent part of ARB_gpu_shader5's interpolateAtCentroid, interpolateAtOffset builtins. Before I go further with this approach, I'd like feedback on the following: 1) I've (ab)used ir_var_shader_in

Re: [Mesa-dev] [PATCH] i965: implement blit path for PBO glDrawPixels

2013-11-11 Thread Alexander Monakov
Ping. A week ago a few people who looked at the older version of this patch posted on IRC didn't point out obvious flaws. Can this get a review please? On Mon, Nov 4, 2013 at 1:34 AM, Alexander Monakov amona...@gmail.com wrote: Hello, This patch implements accelerated path for glDrawPixels

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

2013-11-11 Thread Maarten Lankhorst
op 10-11-13 19:32, Samuel Thibault schreef: This fixes building EGL without libdrm support. Signed-off-by: Samuel Thibault samuel.thiba...@ens-lyon.org Applied, thanks. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

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

2013-11-11 Thread Kevin Rogovin
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_max_prim_size is a good value.

Re: [Mesa-dev] Batchbuffer question

2013-11-11 Thread Rogovin, Kevin
Hi, I've submitted a patch, though not in this thread, not too sure if it should have or not. The patch label is [PATCH] nicer-no-wrap-patch Bheers, -Kevin From: Kenneth Graunke [kenn...@whitecape.org] Sent: Friday, November 08, 2013 11:38 PM To:

[Mesa-dev] [Bug 71492] New: Regression with clearview

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71492 Priority: medium Bug ID: 71492 Assignee: mesa-dev@lists.freedesktop.org Summary: Regression with clearview Severity: normal Classification: Unclassified OS: All

[Mesa-dev] [Bug 71492] Regression with clearview

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71492 --- Comment #1 from José Fonseca jfons...@vmware.com --- Created attachment 89027 -- https://bugs.freedesktop.org/attachment.cgi?id=89027action=edit Screenshot at call 20022 Screenshot at call 20022 -- You are receiving this mail because:

[Mesa-dev] [Bug 71492] Regression with clearview

2013-11-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=71492 José Fonseca jfons...@vmware.com changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |fred...@kde.org

[Mesa-dev] [PATCH] gallivm, llvmpipe: fix float-srgb conversion to handle NaNs

2013-11-11 Thread sroland
From: Roland Scheidegger srol...@vmware.com d3d10 requires us to convert NaNs to zero for any float-int conversion. We don't really do that but mostly seems to work. In particular I suspect the very common float-unorm8 path only really passes because it relies on sse2 pack intrinsics which just

[Mesa-dev] [PATCH] r600/llvm: Store inputs in function arguments

2013-11-11 Thread Vincent Lejeune
--- src/gallium/drivers/r600/r600_llvm.c | 119 +++ src/gallium/drivers/r600/r600_shader.c | 1 + src/gallium/drivers/radeon/radeon_llvm.h | 1 + 3 files changed, 121 insertions(+) diff --git a/src/gallium/drivers/r600/r600_llvm.c

Re: [Mesa-dev] [PATCH] r600/llvm: Store inputs in function arguments

2013-11-11 Thread Tom Stellard
On Mon, Nov 11, 2013 at 03:45:53PM +0100, Vincent Lejeune wrote: Reviewed-by: Tom Stellard thomas.stell...@amd.com --- src/gallium/drivers/r600/r600_llvm.c | 119 +++ src/gallium/drivers/r600/r600_shader.c | 1 + src/gallium/drivers/radeon/radeon_llvm.h

Re: [Mesa-dev] Batchbuffer question

2013-11-11 Thread Paul Berry
On 8 November 2013 13:38, Kenneth Graunke kenn...@whitecape.org wrote: On 11/08/2013 04:49 AM, Rogovin, Kevin wrote: Hi all, As I was poking into the magicks for the batchbuffer, I saw the following logical bits of code, that make sense by themselves but get me paranoid together.

Re: [Mesa-dev] dri3proto requirement (Was: Add DRI3+Present loader)

2013-11-11 Thread Brian Paul
On 11/08/2013 11:49 AM, Eric Anholt wrote: Jose Fonseca jfons...@vmware.com writes: This change seems makes dri3proto a hard requirement to build with automake. Is that strictly necessary? I tried to find ubuntu packages for it to install on my build slave, but it doesn't seem they exist

Re: [Mesa-dev] [PATCH] glx: conditionaly build dri3 and present loader (v3)

2013-11-11 Thread Brian Paul
On 11/08/2013 05:00 PM, Matt Turner wrote: On Fri, Nov 8, 2013 at 3:06 PM, Armin K kre...@email.com wrote: This patch makes it possible to disable DRI3 if desired. Reviewed-by: Matt Turner matts...@gmail.com Unless non-Linux users still need to be able to disable DRI3, by the time Mesa 10.0

Re: [Mesa-dev] dri3proto requirement (Was: Add DRI3+Present loader)

2013-11-11 Thread Jose Fonseca
I - Original Message - On 11/08/2013 11:49 AM, Eric Anholt wrote: Jose Fonseca jfons...@vmware.com writes: This change seems makes dri3proto a hard requirement to build with automake. Is that strictly necessary? I tried to find ubuntu packages for it to install on my build

Re: [Mesa-dev] dri3proto requirement (Was: Add DRI3+Present loader)

2013-11-11 Thread Matt Turner
On Mon, Nov 11, 2013 at 7:58 AM, Brian Paul bri...@vmware.com wrote: On 11/08/2013 11:49 AM, Eric Anholt wrote: Jose Fonseca jfons...@vmware.com writes: This change seems makes dri3proto a hard requirement to build with automake. Is that strictly necessary? I tried to find ubuntu packages

Re: [Mesa-dev] dri3proto requirement (Was: Add DRI3+Present loader)

2013-11-11 Thread Brian Paul
On 11/11/2013 10:14 AM, Matt Turner wrote: On Mon, Nov 11, 2013 at 7:58 AM, Brian Paul bri...@vmware.com wrote: On 11/08/2013 11:49 AM, Eric Anholt wrote: Jose Fonseca jfons...@vmware.com writes: This change seems makes dri3proto a hard requirement to build with automake. Is that strictly

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

2013-11-11 Thread Matt Turner
Need a better commit message. I like the kernel's guidelines which remind us that the summary becomes the globally-unique identifier for the patch. Style comments: On Mon, Nov 11, 2013 at 1:35 AM, Kevin Rogovin kevin.rogo...@intel.com wrote: This patch adds a function interface for enabling no

Re: [Mesa-dev] dri3proto requirement (Was: Add DRI3+Present loader)

2013-11-11 Thread Matt Turner
On Mon, Nov 11, 2013 at 9:19 AM, Brian Paul bri...@vmware.com wrote: The tarball doesn't contain the autogen.sh script. And I don't see a pointer to the git tree in the tarball. There's no README or anything. Oh. :( The repo is git://anongit.freedesktop.org/xorg/lib/libxshmfence You can get

Re: [Mesa-dev] Batchbuffer question

2013-11-11 Thread Kenneth Graunke
On 11/11/2013 07:31 AM, Paul Berry wrote: On 8 November 2013 13:38, Kenneth Graunke kenn...@whitecape.org mailto:kenn...@whitecape.org wrote: On 11/08/2013 04:49 AM, Rogovin, Kevin wrote: Hi all, As I was poking into the magicks for the batchbuffer, I saw the

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

2013-11-11 Thread Alexander von Gluck IV
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:

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

2013-11-11 Thread Ian Romanick
On 11/05/2013 11:36 AM, Courtney Goeltzenleuchter wrote: On Tue, Nov 5, 2013 at 12:22 PM, Ian Romanick i...@freedesktop.org mailto:i...@freedesktop.org wrote: On 11/05/2013 09:44 AM, Chris Forbes wrote: So, you can create a GL_LUMINANCE view of a GL_LUMINANCE texture. Hmm...

[Mesa-dev] [Bug 71506] New: 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 Priority: medium Bug ID: 71506 Keywords: regression Assignee: mesa-dev@lists.freedesktop.org Summary: indirect_glx.c:350: multiple definition of `indirect_create_context'

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

2013-11-11 Thread Ian Romanick
On 11/11/2013 01:35 AM, Kevin Rogovin wrote: 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

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

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- 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

[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 emil.l.veli...@gmail.com --- src/gallium/targets/r300/common/drm_target.c | 54 src/gallium/targets/r300/dri/Makefile.am | 2 +-

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

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- 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

[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 emil.l.veli...@gmail.com --- 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

[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 emil.l.veli...@gmail.com --- src/gallium/targets/r600/common/drm_target.c | 69 src/gallium/targets/r600/dri/Makefile.am | 2 +-

[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 emil.l.veli...@gmail.com --- src/gallium/Automake.inc | 22 ++ src/gallium/drivers/freedreno/Makefile.am | 10

[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 emil.l.veli...@gmail.com Reviewed-by: Ian Romanick ian.d.roman...@intel.com --- src/mesa/drivers/dri/common/Android.mk | 2 +-

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

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- 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,

[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 emil.l.veli...@gmail.com --- src/gallium/Automake.inc | 7 +++ src/gallium/targets/r600/xvmc/Makefile.am| 5 +

[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 emil.l.veli...@gmail.com --- src/gallium/targets/radeonsi/common/drm_target.c | 69 src/gallium/targets/radeonsi/dri/Makefile.am | 2 +-

[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 emil.l.veli...@gmail.com --- src/gallium/Automake.inc | 13

[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 emil.l.veli...@gmail.com --- src/gallium/targets/r600/xvmc/Makefile.am| 1 -

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

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- 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 ---

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

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- 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

[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 emil.l.veli...@gmail.com --- 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

[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 imir...@alum.mit.edu

[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 emil.l.veli...@gmail.com --- 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

[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 emil.l.veli...@gmail.com --- src/gallium/Automake.inc | 11 +++

[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 emil.l.veli...@gmail.com --- src/gallium/targets/r600/vdpau/Makefile.am | 1 - src/gallium/targets/radeonsi/vdpau/Makefile.am | 3 ---

[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.

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

2013-11-11 Thread Emil Velikov
From: Johannes Obermayr johannesoberm...@gmx.de 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 emil.l.veli...@gmail.com --- configure.ac

[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 emil.l.veli...@gmail.com --- src/gallium/SConscript | 1 - src/gallium/targets/SConscript.dri | 83 --

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

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- 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 =

[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 emil.l.veli...@gmail.com --- 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 +++

[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 emil.l.veli...@gmail.com ---

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

2013-11-11 Thread Emil Velikov
Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- 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

[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 emil.l.veli...@gmail.com --- src/gallium/targets/r600/xvmc/Makefile.am| 8 +---

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:

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 ian.d.roman...@intel.com Signed-off-by: Ian Romanick ian.d.roman...@intel.com --- src/mesa/drivers/dri/i915/intel_screen.c | 79

[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 ian.d.roman...@intel.com 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 ian.d.roman...@intel.com Cc: Daniel Vetter

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

2013-11-11 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com Send the zombie back to the grave before it infects the townsfolk. Signed-off-by: Ian Romanick ian.d.roman...@intel.com Cc: Daniel Vetter dan...@ffwll.ch Cc: 10.0 mesa-sta...@lists.freedesktop.org --- src/mesa/drivers/dri/i915/intel_screen.c | 8

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

2013-11-11 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com This silences some compiler warnings in i915 and i965. See also 75982a5. Signed-off-by: Ian Romanick ian.d.roman...@intel.com Cc: 10.0 mesa-sta...@lists.freedesktop.org --- src/mesa/drivers/dri/common/utils.c | 2 +-

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

2013-11-11 Thread Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com 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

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 chr...@ijw.co.nz 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

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

2013-11-11 Thread Eric Anholt
Kevin Rogovin kevin.rogo...@intel.com 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

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 stereotype...@gmail.com wrote: On 9 November 2013 01:02, Chris Forbes chr...@ijw.co.nz wrote: This series adds support for ARB_draw_indirect and ARB_multi_draw_indirect on Gen7+. Big

[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 kgbric...@web.de --- 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

[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 v...@freedesktop.org changed: What|Removed |Added CC||i...@freedesktop.org,

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 i...@freedesktop.org wrote: On 11/05/2013 11:36 AM, Courtney Goeltzenleuchter wrote: On Tue, Nov 5, 2013 at 12:22 PM, Ian Romanick i...@freedesktop.org mailto:i...@freedesktop.org wrote: On 11/05/2013 09:44 AM, Chris Forbes wrote:

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

[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 k...@bitplanet.net Cc: 10.0 mesa-sta...@lists.freedesktop.org --- src/egl/drivers/dri2/platform_drm.c | 46

[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().

[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 k...@bitplanet.net Cc: 10.0 mesa-sta...@lists.freedesktop.org --- src/egl/drivers/dri2/egl_dri2.h |

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 THIS PATCH internal-format Before (MB/sec) XRGB (MB/sec) mipmap (MB/sec) GL_RGBA

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 ian.d.roman...@intel.com Signed-off-by: Ian Romanick ian.d.roman...@intel.com ---

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 anuj.pho...@gmail.com Reviewed-by: Chad Versace chad.vers...@linux.intel.com

[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 i...@freedesktop.org --- 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.

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 ian.d.roman...@intel.com 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.

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 v...@freedesktop.org Thanks. Committed. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

[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 v...@freedesktop.org --- Created attachment 89058 -- https://bugs.freedesktop.org/attachment.cgi?id=89058action=edit nm -u src/glx/tests/*.o -- You are receiving this mail because: You are the assignee for the

[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 i...@freedesktop.org --- 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

[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 v...@freedesktop.org --- I see the same output. $ nm -u src/glx/.libs/create_context.o U __assert_fail U free U GetGLXScreenConfigs U

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 ian.d.roman...@intel.com Systems with little physical memory installed will report less than 2GiB, and some systems may (hypothetically?) have a larger address space

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

[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...@freedesktop.org --- 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

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 chad.vers...@linux.intel.com 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

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 kgbric...@web.de wrote: Signed-off-by: Rico Schüller kgbric...@web.de --- 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

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 chad.vers...@linux.intel.com 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

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 ian.d.roman...@intel.com Systems with little physical memory installed will report less than 2GiB,

[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 kre...@email.com --- 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

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 e...@anholt.net 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 matts...@gmail.com ___ mesa-dev mailing list

[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 kre...@email.com --- 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

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 e...@anholt.net 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

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 e...@anholt.net 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

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 emil.l.veli...@gmail.com wrote: Signed-off-by: Emil Velikov emil.l.veli...@gmail.com --- 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 ---

[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 tstel...@gmail.com --- 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 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 emil.l.veli...@gmail.com Can you make sure make check still works with this patch, if it does, then: Reviewed-by: Tom Stellard thomas.stell...@amd.com --- src/gallium/drivers/r300/Makefile.am | 13

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 emil.l.veli...@gmail.com --- src/gallium/drivers/r300/Makefile.am | 6 +++--- src/gallium/drivers/r300/ralloc.c| 1 +

[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 ---

[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,

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

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 chad.vers...@linux.intel.com 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

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 precedence

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.c | 7

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.c |

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

  1   2   >