This series adds support for ARB_[multi_]draw_indirect on IVB and later.
Patches 1-2 are updated versions of Christoph's initial ARB_draw_indirect
patches, adjusted only to resolve rebase issues, etc.
Patches 3-7 are the meat of the implementation; the indirect buffer is
referenced from the
From: Christoph Bumiller e0425...@student.tuwien.ac.at
Split from patch implementing ARB_draw_indirect.
v2: Const-qualify the struct gl_buffer_object *indirect argument.
v3: Fix up some more draw calls for new argument.
v4: Fix up rebase conflicts in i965.
---
From: Christoph Bumiller e0425...@student.tuwien.ac.at
v2:
Removed some stray extern qualifiers.
Documented use of Draw*IndirectCommand sizes.
Removed separate extension enable flag for ARB_multi_draw_indirect
since this can always be supported by looping.
Kept generation of
Just prior to emitting the 3DPRIMITIVE command, we load each of the
indirect registers. The values loaded are either from offsets into the
current indirect BO, or constant zero if the parameter is not used for
this draw.
Enabling use of the indirect registers is done by turning on a bit in
the
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_context.h | 10 +
src/mesa/drivers/dri/i965/brw_draw_upload.c | 32
src/mesa/drivers/dri/i965/brw_state.h| 1 +
src/mesa/drivers/dri/i965/brw_state_upload.c | 2 ++
---
src/mesa/drivers/dri/i965/brw_draw.c | 2 +-
src/mesa/drivers/dri/i965/brw_draw.h | 3 ++-
src/mesa/drivers/dri/i965/brw_primitive_restart.c | 5 +++--
3 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw.c
---
src/mesa/drivers/dri/i965/brw_primitive_restart.c | 2 +-
src/mesa/vbo/vbo.h| 3 ++-
src/mesa/vbo/vbo_exec_array.c | 2 +-
src/mesa/vbo/vbo_primitive_restart.c | 3 ++-
4 files changed, 6 insertions(+), 4 deletions(-)
diff
- MMIO registers for draw parameters
- New bit in 3DPRIMITIVE command to enable indirection
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_defines.h | 1 +
src/mesa/drivers/dri/i965/intel_reg.h | 8
2 files changed, 9 insertions(+)
diff --git
.. and mark them off on the extensions list as done.
V2: Enable only if pipelined register writes work.
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
docs/GL3.txt | 4 ++--
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
2 files changed, 3 insertions(+), 2
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/vbo/vbo_primitive_restart.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/src/mesa/vbo/vbo_primitive_restart.c
b/src/mesa/vbo/vbo_primitive_restart.c
index 48d04e1..ffe0c41 100644
---
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/drivers/dri/i965/brw_draw.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_draw.c
b/src/mesa/drivers/dri/i965/brw_draw.c
index e6c3508..9f8ad45 100644
---
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/vbo/vbo.h| 3 ++-
src/mesa/vbo/vbo_exec_array.c | 5 +
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/mesa/vbo/vbo.h b/src/mesa/vbo/vbo.h
index 113374a..942b34a 100644
--- a/src/mesa/vbo/vbo.h
+++
---
src/mesa/main/compiler.h | 35 ---
src/mesa/math/m_debug_util.h |2 --
src/mesa/x86/assyntax.h |8
3 files changed, 45 deletions(-)
diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
index 2490ee3..61ce5db 100644
---
---
include/GL/gl.h |6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/include/GL/gl.h b/include/GL/gl.h
index 6b94e3f..2235f22 100644
--- a/include/GL/gl.h
+++ b/include/GL/gl.h
@@ -33,11 +33,8 @@
On Sun, Nov 03, 2013 at 08:35:47PM -0800, Vinson Lee wrote:
LLVM 3.4 r193971 removed llvm::DisablePrettyStackTrace and made the
pretty stack trace opt-in rather than opt-out.
Could you add a comment explaining that the default value of
DisablePrettyStackTrace has changed to true in LLVM 3.4
---
src/mesa/main/compiler.h | 10 --
src/mesa/main/glheader.h |2 +-
2 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
index d806d5b..2490ee3 100644
--- a/src/mesa/main/compiler.h
+++ b/src/mesa/main/compiler.h
@@
---
include/GL/gl.h |4
1 file changed, 4 deletions(-)
diff --git a/include/GL/gl.h b/include/GL/gl.h
index 2235f22..0efa6ef 100644
--- a/include/GL/gl.h
+++ b/include/GL/gl.h
@@ -113,10 +113,6 @@
#define GLAPIENTRYP GLAPIENTRY *
#endif
-#ifdef CENTERLINE_CLPP
-#define signed
On Mon, Nov 4, 2013 at 9:41 AM, Tom Stellard t...@stellard.net wrote:
On Sun, Nov 03, 2013 at 08:35:47PM -0800, Vinson Lee wrote:
LLVM 3.4 r193971 removed llvm::DisablePrettyStackTrace and made the
pretty stack trace opt-in rather than opt-out.
Could you add a comment explaining that the
On 11/04/2013 02:09 AM, Chris Forbes wrote:
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
src/mesa/vbo/vbo_primitive_restart.c | 27 +++
1 file changed, 27 insertions(+)
diff --git a/src/mesa/vbo/vbo_primitive_restart.c
b/src/mesa/vbo/vbo_primitive_restart.c
index
On 11/04/2013 02:09 AM, Chris Forbes wrote:
From: Christoph Bumiller e0425...@student.tuwien.ac.at
v2:
Removed some stray extern qualifiers.
Documented use of Draw*IndirectCommand sizes.
Removed separate extension enable flag for ARB_multi_draw_indirect
since this can always be supported by
On 11/03/2013 09:35 PM, Vinson Lee wrote:
LLVM 3.4 r193971 removed llvm::DisablePrettyStackTrace and made the
pretty stack trace opt-in rather than opt-out.
Signed-off-by: Vinson Lee v...@freedesktop.org
---
src/gallium/auxiliary/gallivm/lp_bld_misc.cpp | 2 ++
1 file changed, 2
On Fri, Nov 01, 2013 at 10:25:43AM -0500, Aaron Watry wrote:
dso_list was added as an argument for createInternalizePass in 3.4, and then
it was removed again in the same llvm version.
I've pushed this patch, thanks!
-Tom
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 5 -
On Sun, Nov 03, 2013 at 06:58:18PM +0100, Vincent Lejeune wrote:
---
src/gallium/drivers/r600/r600_llvm.c | 125
-
src/gallium/drivers/r600/r600_shader.c | 2 +
src/gallium/drivers/radeon/radeon_llvm.h | 1 +
From: José Fonseca jfons...@vmware.com
Just happened to notice it was missing while looking at it.
---
src/gallium/auxiliary/util/u_format.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_format.h
b/src/gallium/auxiliary/util/u_format.h
index dc777c3..31283e5
On Fri, Nov 1, 2013 at 3:44 PM, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
The x, y, width, and height parameters aren't used by radeon_viewport,
so don't pass them. This should make future changes to the
dd_function_table::Viewport interface a
Matt Turner matts...@gmail.com writes:
Removes IF/ENDIF and IF/ELSE/ENDIF with no intervening instructions.
total instructions in shared programs: 1360393 - 1360387 (-0.00%)
instructions in affected programs: 157 - 151 (-3.82%)
(no change in vertex shaders)
Reviewed-by: Paul Berry
Matt Turner matts...@gmail.com writes:
diff --git a/src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp
b/src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp
new file mode 100644
index 000..9626751
--- /dev/null
+++ b/src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp
+bool
Matt Turner matts...@gmail.com writes:
One program affected.
instructions in affected programs: 436 - 428 (-1.83%)
Reviewed-by: Paul Berry stereotype...@gmail.com
Couldn't we just detect this in opt_algebraic() in general?
pgpv6tl4BZCNH.pgp
Description: PGP signature
Paul Berry stereotype...@gmail.com writes:
On 31 October 2013 18:57, Matt Turner matts...@gmail.com wrote:
Before this patch, the following code would not be optimized even though
the first two instructions were common to the then and else blocks:
(+f0) IF
MOV dst0 ...
MOV dst1
Paul Berry stereotype...@gmail.com writes:
Previously, when packing geometry shader input varyings like this:
in float foo[3];
in float bar[3];
lower_packed_varyings would declare a packed varying like this:
(declare (shader_in flat) (array ivec4 3) packed:foo[0],bar[0])
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 11/01/2013 04:12 PM, Francisco Jerez wrote:
Ian Romanick i...@freedesktop.org writes:
On 11/01/2013 02:04 PM, Courtney Goeltzenleuchter wrote:
[...]
More often, the dd_function_table functions allow core Mesa to
signal the driver driver that
Fredrik Höglund fred...@kde.org writes:
On Thursday 31 October 2013, Eric Anholt wrote:
There was some spec text, and what there isn't text for we have obvious
intended behavior from other buffer object bindings.
The first revision of the specification didn't say anything about it,
but the
On 4 November 2013 10:31, Eric Anholt e...@anholt.net wrote:
Matt Turner matts...@gmail.com writes:
diff --git a/src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp
b/src/mesa/drivers/dri/i965/brw_fs_sel_peephole.cpp
new file mode 100644
index 000..9626751
--- /dev/null
+++
On 11/04/2013 10:56 AM, Ian Romanick wrote:
On 11/04/2013 01:09 AM, Chris Forbes wrote:
diff --git a/src/mesa/main/bufferobj.c b/src/mesa/main/bufferobj.c
index 1f55061..69ea1c0 100644
--- a/src/mesa/main/bufferobj.c
+++ b/src/mesa/main/bufferobj.c
@@ -86,6 +86,10 @@ get_buffer_target(struct
Paul Berry stereotype...@gmail.com writes:
This series adds precompilation for geometry shaders.
I'm not 100% certain this is worth doing, because:
(a) I'm aware of very few programs out in the wild that actually use
geometry shaders right now.
(b) Since i965 hardware has no ability to
On 11/04/2013 01:09 AM, Chris Forbes wrote:
.. and mark them off on the extensions list as done.
V2: Enable only if pipelined register writes work.
If you update docs/relnotes/10.0.html,
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Chris Forbes chr...@ijw.co.nz
---
Am 04.11.2013 18:21, schrieb jfons...@vmware.com:
From: José Fonseca jfons...@vmware.com
Just happened to notice it was missing while looking at it.
---
src/gallium/auxiliary/util/u_format.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_format.h
On 11/04/2013 01:09 AM, Chris Forbes wrote:
This series adds support for ARB_[multi_]draw_indirect on IVB and later.
Patches 1-2 are updated versions of Christoph's initial ARB_draw_indirect
patches, adjusted only to resolve rebase issues, etc.
Patches 3-7 are the meat of the
* Call mesa viewport call on winndow resize
* Add initial postprocessing code
* Pass hgl_context to private statetracker
as it is more useful than GalliumContext
* Use Lock and Unlock functions to standardize
GalliumContext locking
* Create texture resources in texture validation
---
- Original Message -
Am 04.11.2013 18:21, schrieb jfons...@vmware.com:
From: José Fonseca jfons...@vmware.com
Just happened to notice it was missing while looking at it.
---
src/gallium/auxiliary/util/u_format.h | 2 ++
1 file changed, 2 insertions(+)
diff --git
These are:
Reviewed-by: Eric Anholt e...@anholt.net
pgp77HPIuZ_o5.pgp
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 4 November 2013 11:23, Matt Turner matts...@gmail.com wrote:
On Mon, Nov 4, 2013 at 10:57 AM, Paul Berry stereotype...@gmail.com
wrote:
On 4 November 2013 10:31, Eric Anholt e...@anholt.net wrote:
Matt Turner matts...@gmail.com writes:
diff --git
Hi Ian,
Gallium uses the following interface:
void (*set_scissor_states)( struct pipe_context *,
unsigned start_slot,
unsigned num_scissors,
const struct pipe_scissor_state * );
void
On 11/04/2013 11:43 AM, Ian Romanick wrote:
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 11/01/2013 04:12 PM, Francisco Jerez wrote:
Ian Romanick i...@freedesktop.org writes:
On 11/01/2013 02:04 PM, Courtney Goeltzenleuchter wrote:
[...]
More often, the dd_function_table functions
On Mon, Nov 4, 2013 at 11:30 AM, Paul Berry stereotype...@gmail.com wrote:
On 4 November 2013 11:23, Matt Turner matts...@gmail.com wrote:
On Mon, Nov 4, 2013 at 10:57 AM, Paul Berry stereotype...@gmail.com
wrote:
On 4 November 2013 10:31, Eric Anholt e...@anholt.net wrote:
Matt Turner
Kenneth Graunke kenn...@whitecape.org writes:
The changes between Gen6-7 are minimal, and can easily be solved with
an extra generation check. This cuts a lot of duplicated code.
It also helps prevent even more duplication for Broadwell.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
https://bugs.freedesktop.org/show_bug.cgi?id=71110
Emil Velikov emil.l.veli...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
On 11/02/2013 12:00 PM, Emil Velikov wrote:
Hello list,
Here is a follow up patchset to the Makefile.sources, targeting the following
* move source files lists to Makefile.sources (symmetry and reuse accross
build systems)
* creates symlinks and enables subdir-objects (preparation as the
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 11/03/2013 02:35 PM, Keith Packard wrote:
Kristian Høgsberg hoegsb...@gmail.com writes:
I sent a reply to the sourceforge addresses in the original
emails, but I didn't see it show up in the archives. Trying
again with the freedesktop
On 10/28/2013 03:33 PM, Fredrik Höglund wrote:
Split out the code for updating the array format into a new function
called update_array_format(). This function will be called by both
update_array() and the new glVertexAttrib*Format() entry points in
ARB_vertex_attrib_binding.
Other than the
On 10/28/2013 03:33 PM, Fredrik Höglund wrote:
This needs some extension checks somewhere.
---
src/mesa/main/get.c | 24
src/mesa/main/get_hash_params.py |3 +++
src/mesa/main/varray.c |9 +
3 files changed, 36 insertions(+)
On 10/28/2013 03:33 PM, Fredrik Höglund wrote:
update_array() and update_array_format() are changed to update the new
attrib and binding states, and the client arrays become derived state.
I don't think we should enable the extension until after the other
patches. Otherwise people can
Patches 1, 3, 4, 6, 7, 8, and 10 are
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
I sent trivial comments for patch 2.
I sent comments for patches 5 and 9.
On 10/28/2013 03:33 PM, Fredrik Höglund wrote:
This extension decouples the vertex attributes from the vertex buffer
bindings,
On 11/01/2013 12:01 PM, Fredrik Höglund wrote:
On Thursday 31 October 2013, Eric Anholt wrote:
There was some spec text, and what there isn't text for we have obvious
intended behavior from other buffer object bindings.
The first revision of the specification didn't say anything about it,
i965/gen7 hardware doesn't perform alpha blending correctly when
compressed (CMS) multisampled buffers are in use. Therefore, we need
to detect when alpha blending is used on a compressed multisampled
buffer, and convert the buffer to uncompressed (UMS) layout.
Once a given buffer has been
Is this broken the same way on all of IVB|HSW|BYT?
On Tue, Nov 5, 2013 at 11:24 AM, Paul Berry stereotype...@gmail.com wrote:
i965/gen7 hardware doesn't perform alpha blending correctly when
compressed (CMS) multisampled buffers are in use. Therefore, we need
to detect when alpha blending is
On Monday 04 November 2013, Ian Romanick wrote:
On 10/28/2013 03:33 PM, Fredrik Höglund wrote:
This needs some extension checks somewhere.
I've added a check for ARB_instanced_arrays in the VERTEX_BINDING_DIVISOR
case. I didn't add checks for ARB_vertex_attrib_binding since the intent is to
On Monday 04 November 2013, Ian Romanick wrote:
On 10/28/2013 03:33 PM, Fredrik Höglund wrote:
update_array() and update_array_format() are changed to update the new
attrib and binding states, and the client arrays become derived state.
I don't think we should enable the extension until
This patch make changes to correctly set up the Dispatch GRF Start
Register in case of 'SIMD16 only' FS dispatch.
This fixes an issue of incorrect rendering on dolphin emulator with
GL_SAMPLE_SHADING enabled.
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
---
Makefile.am | 2 +-
configure.ac | 1 -
src/mesa/Makefile.am | 24 +++
src/mesa/program/Makefile.am | 45
4 files changed, 17 insertions(+), 55 deletions(-)
delete mode 100644
---
src/mesa/Makefile.am | 15 ---
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/src/mesa/Makefile.am b/src/mesa/Makefile.am
index f86caee..6ce31d2 100644
--- a/src/mesa/Makefile.am
+++ b/src/mesa/Makefile.am
@@ -85,8 +85,7 @@ if HAVE_GALLIUM
noinst_LTLIBRARIES +=
---
src/mesa/main/compiler.h | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
index 61ce5db..6a25bf5 100644
--- a/src/mesa/main/compiler.h
+++ b/src/mesa/main/compiler.h
@@ -252,6 +252,21 @@ static INLINE GLuint
---
src/mesa/drivers/dri/i965/brw_reg.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/brw_reg.h
b/src/mesa/drivers/dri/i965/brw_reg.h
index e9cf892..bfc99e1 100644
--- a/src/mesa/drivers/dri/i965/brw_reg.h
+++ b/src/mesa/drivers/dri/i965/brw_reg.h
@@ -299,6 +299,7
On 11/04/2013 03:50 PM, Eric Anholt wrote:
You can't return stack-initialized values and expect anything good to
happen.
---
src/mesa/drivers/dri/i965/brw_context.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
On 11/04/2013 05:43 PM, Matt Turner wrote:
---
src/mesa/main/compiler.h | 15 +++
1 file changed, 15 insertions(+)
diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
index 61ce5db..6a25bf5 100644
--- a/src/mesa/main/compiler.h
+++ b/src/mesa/main/compiler.h
@@
IIRC, this is MacOS 9.x stuff.
---
include/GL/gl.h|7 ---
src/mesa/math/m_clip_tmp.h | 12 ++--
2 files changed, 6 insertions(+), 13 deletions(-)
diff --git a/include/GL/gl.h b/include/GL/gl.h
index 31a2813..1d21df7 100644
--- a/include/GL/gl.h
+++ b/include/GL/gl.h
WGLAPI was defined in glheader.h but wasn't used anywhere.
---
src/mesa/main/glheader.h | 16
1 file changed, 16 deletions(-)
diff --git a/src/mesa/main/glheader.h b/src/mesa/main/glheader.h
index e027218..7f7f9a3 100644
--- a/src/mesa/main/glheader.h
+++
---
src/mesa/main/compiler.h |7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
index 61ce5db..2752ca8 100644
--- a/src/mesa/main/compiler.h
+++ b/src/mesa/main/compiler.h
@@ -36,11 +36,7 @@
#include assert.h
---
include/GL/gl.h |2 +-
include/GL/osmesa.h |5 -
2 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/include/GL/gl.h b/include/GL/gl.h
index 0efa6ef..31a2813 100644
--- a/include/GL/gl.h
+++ b/include/GL/gl.h
@@ -69,7 +69,7 @@
# define GLAPIENTRY
#endif /* WIN32
---
include/GL/gl.h |6 +-
src/mesa/main/compiler.h |2 +-
2 files changed, 2 insertions(+), 6 deletions(-)
diff --git a/include/GL/gl.h b/include/GL/gl.h
index 1d21df7..d0710ba 100644
--- a/include/GL/gl.h
+++ b/include/GL/gl.h
@@ -40,15 +40,11 @@
#include stdlib.h /*
On 11/04/2013 12:05 PM, Alexander von Gluck IV wrote:
* Call mesa viewport call on winndow resize
* Add initial postprocessing code
* Pass hgl_context to private statetracker
as it is more useful than GalliumContext
* Use Lock and Unlock functions to standardize
GalliumContext locking
*
On Mon, Nov 4, 2013 at 4:48 PM, Brian Paul bri...@vmware.com wrote:
---
src/mesa/main/compiler.h |7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
index 61ce5db..2752ca8 100644
--- a/src/mesa/main/compiler.h
+++
On Mon, Nov 4, 2013 at 6:48 PM, Brian Paul bri...@vmware.com wrote:
---
include/GL/gl.h |2 +-
include/GL/osmesa.h |5 -
2 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/include/GL/gl.h b/include/GL/gl.h
index 0efa6ef..31a2813 100644
--- a/include/GL/gl.h
+++
On Mon, Nov 4, 2013 at 5:04 PM, Matt Turner matts...@gmail.com wrote:
On Mon, Nov 4, 2013 at 4:48 PM, Brian Paul bri...@vmware.com wrote:
---
src/mesa/main/compiler.h |7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/src/mesa/main/compiler.h
Commit 2f89662 added the driconf option 'clamp_max_samples'. In that
commit, the option did not alter the context version.
Specifically, Mesa constructed a GL 3.0 context for Baytrail even if
clamp_max_samples=0, which clamps GL_MAX_SAMPLES to 0. This violates the
GL 3.0 spec, which requires
* Goodbye BeOS, we hardly knew thee
* As BeOS was gcc2 only, there was little chance
of this being useful.
* Doesn't effect Haiku in any meaningful way
---
include/GL/gl.h |8
scons/gallium.py |2 --
2 files changed, 10 deletions(-)
diff --git a/include/GL/gl.h
Chad Versace chad.vers...@linux.intel.com writes:
Commit 2f89662 added the driconf option 'clamp_max_samples'. In that
commit, the option did not alter the context version.
Specifically, Mesa constructed a GL 3.0 context for Baytrail even if
clamp_max_samples=0, which clamps GL_MAX_SAMPLES
On Mon, Nov 4, 2013 at 3:50 PM, Eric Anholt e...@anholt.net wrote:
You can't return stack-initialized values and expect anything good to
happen.
---
src/mesa/drivers/dri/i965/brw_context.c | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git
Matt Turner matts...@gmail.com writes:
---
src/mesa/drivers/dri/i965/brw_reg.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/brw_reg.h
b/src/mesa/drivers/dri/i965/brw_reg.h
index e9cf892..bfc99e1 100644
--- a/src/mesa/drivers/dri/i965/brw_reg.h
+++
Series Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On Fri, Nov 1, 2013 at 12:44 PM, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
The x, y, width, and height parameters aren't used by radeon_viewport,
so don't pass them. This should make future
On Mon, Nov 4, 2013 at 6:23 PM, Keith Packard kei...@keithp.com wrote:
The __DRI_IMAGE_FORMAT codes are used by the image extension, drivers need to
be able to translate between them. Instead of duplicating this translation in
each driver, create a shared version.
Signed-off-by: Keith Packard
Jordan Justen jljus...@gmail.com writes:
After patch 6, this will add SARGB8, right? So, maybe add this to the
commit message, or separate out adding SARGB8 into a separate commit?
I added the SARGB8 define in patch 4; is there some other separation you
think would be warranted?
Oh, just so
82 matches
Mail list logo