Matt Turner matts...@gmail.com writes:
We'd like to CSE some instructions, like CMP, that often have null
destinations. Instead of replacing them with MOVs to null, just don't
emit the MOV.
---
src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 42
+++-
1 file
Matt Turner matts...@gmail.com writes:
Since compare instructions write the flag register, they should not be
considered dead even if their destination is never read. Instead of
removing them if found to be dead, set their destination to null to free
a register.
---
Matt Turner matts...@gmail.com writes:
Optimizes
cmp.ge.f0(8) null g458,8,1F 0F
(+f0) sel(8)g501F g408,8,1F g108,8,1F
cmp.ge.f0(8) null g458,8,1F 0F
(+f0) sel(8)g511F g418,8,1F g118,8,1F
cmp.ge.f0(8) null g458,8,1F 0F
(+f0) sel(8)
Paul Berry stereotype...@gmail.com writes:
On 21 October 2013 11:20, Eric Anholt e...@anholt.net wrote:
Orbital Explorer was generating a 4000 instruction geometry shader, which
was taking 275 trips through dead code elimination and register
coalescing, each of which updated live variables
On 10/28/2013 10:19 PM, Eric Anholt wrote:
The idea of the original order was that you'd dead code eliminate accesses
to push constants. But I've never seen a case of that (nor has
shader-db), while we frequently see sparse accesses of large constant
arrays that would overflow into pull
Anuj Phogat anuj.pho...@gmail.com writes:
V2:
- Update comments.
- Make changes to support simd16 mode.
- Add compute_pos_offset variable in brw_wm_prog_key.
- Add variable uses_omask in brw_wm_prog_data.
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
+fs_reg *
Reviewed-by: Marek Olšák marek.ol...@amd.com
Marek
On Tue, Oct 29, 2013 at 1:37 AM, Brian Paul bri...@vmware.com wrote:
Before we were only checking the st-vertex_array_out_of_memory flag
after updating array state. But if there's two consecutive glDrawArrays
calls and the first one is
On Mon, Oct 28, 2013 at 11:31:32AM -0700, Matt Turner wrote:
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 30 ++
src/mesa/drivers/dri/i965/brw_fs.h | 1 +
2 files changed, 31 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
On Mon, Oct 28, 2013 at 11:31:33AM -0700, Matt Turner wrote:
fs_visitor::try_replace_with_sel optimizes only if statements whose
then and else bodies contain a single MOV instruction. It also did
could not handle constant arguments, since they cause an extra MOV
immediate to be generated
On 29 October 2013 00:28, Eric Anholt e...@anholt.net wrote:
Paul Berry stereotype...@gmail.com writes:
On 21 October 2013 11:20, Eric Anholt e...@anholt.net wrote:
Orbital Explorer was generating a 4000 instruction geometry shader,
which
was taking 275 trips through dead code
On 29 October 2013 00:53, Tapani tapani.pa...@intel.com wrote:
On 10/28/2013 10:09 PM, Paul Berry wrote:
+static uint32_t _unique_id(ir_variable *var)
+{
+ char buffer[256];
+ _mesa_snprintf(buffer, 256, %s_%p, var-name, var);
+ return _mesa_str_checksum(buffer);
Two problems:
On 10/28/2013 06:01 PM, Alexander von Gluck IV wrote:
* Instead of assuming the displaytarget is the same
stride / colorspace as the destination, lets
actually check the source bitmap.
* Fixes random stride issues in rendering
---
src/gallium/winsys/sw/hgl/bitmap_wrapper.cpp | 20
On 26 October 2013 00:13, Vinson Lee v...@freedesktop.org wrote:
Fixes Uninitialized pointer field defect reported by Coverity.
Signed-off-by: Vinson Lee v...@freedesktop.org
Reviewed-by: Paul Berry stereotype...@gmail.com
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 7 ++-
On 27 October 2013 12:58, Marek Olšák mar...@gmail.com wrote:
On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry stereotype...@gmail.com
wrote:
On 20 October 2013 09:20, Marek Olšák mar...@gmail.com wrote:
From: Marek Olšák marek.ol...@amd.com
This avoids a defect in lower_output_reads.
On Fri, Oct 25, 2013 at 05:46:38PM -0500, Aaron Watry wrote:
Reviewed-by: Aaron Watry awa...@gmail.com
I have tested this on a Radeon 5400 (Cedar), and I just sent a few
generated tests to the piglit list.
--Aaron
Thanks for testing, I have committed this patch.
-Tom
On Wed, Oct 23,
On Sun, Oct 27, 2013 at 07:36:07PM +0100, Kai Wasserbäch wrote:
Fixes a boat load of Piglit tests for me, which crashed like fdo#70913
before.
Thanks to Michel Dänzer for the tip.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70913
Signed-off-by: Kai Wasserbäch
From: Tom Stellard thomas.stell...@amd.com
The ICD loader should be responsible for installing headers.
---
src/gallium/state_trackers/clover/Makefile.am | 21 +++--
1 file changed, 11 insertions(+), 10 deletions(-)
diff --git a/src/gallium/state_trackers/clover/Makefile.am
On Sat, Oct 26, 2013 at 07:54:27PM +0200, David Heidelberger wrote:
From f188c10862f30205a7838a2e5f09d3e3a9b18f59 Mon Sep 17 00:00:00 2001
From: David Heidelberger david.heidelber...@ixit.cz
Date: Fri, 25 Oct 2013 18:07:05 +0200
Subject: [PATCH v2] clover: mark gcc 4.7.0 as broken
It
On 27 October 2013 14:59, Ian Romanick i...@freedesktop.org wrote:
From: Gregory Hainaut gregory.hain...@gmail.com
The return value has been unused since commit d348b0c. This was
originally included in another patch, but it was split out by Ian
Romanick.
Reviewed-by: Ian Romanick
On 27 October 2013 14:59, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
I made this a function (instead of a method of ir_variable) because it
made the change set smaller, and I expect that there will be an overload
that takes an ir_var_mode enum.
On 27 October 2013 14:59, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Future patches will add some extra code to this path, and some of that
code will want to exit from the explicit location code early.
Signed-off-by: Ian Romanick
On 27 October 2013 14:59, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
This will simplify the addition of layout(location) qualifiers for
separate shader objects. This was validated with new piglit tests
On Mon, Oct 28, 2013 at 7:00 PM, Paul Berry stereotype...@gmail.com wrote:
On 28 October 2013 18:14, Paul Berry stereotype...@gmail.com wrote:
On 25 October 2013 16:45, Anuj Phogat anuj.pho...@gmail.com wrote:
- Enable GEN6_WM_MSDISPMODE_PERSAMPLE, GEN6_WM_POSOFFSET_SAMPLE,
Paul Berry stereotype...@gmail.com writes:
On 27 October 2013 12:58, Marek Olšák mar...@gmail.com wrote:
On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry stereotype...@gmail.com
wrote:
On 20 October 2013 09:20, Marek Olšák mar...@gmail.com wrote:
From: Marek Olšák marek.ol...@amd.com
From: Tom Stellard thomas.stell...@amd.com
This results in huge performance improvements for applications like
pyrit, which depend on the implementation to determine the optimal
work group size.
---
src/gallium/state_trackers/clover/api/kernel.cpp | 9 -
From: Tom Stellard thomas.stell...@amd.com
This results in huge performance improvements for applications like
pyrit, which depend on the implementation to determine the optimal
work group size.
---
src/gallium/state_trackers/clover/api/kernel.cpp | 9 -
On 29 October 2013 11:20, Eric Anholt e...@anholt.net wrote:
Paul Berry stereotype...@gmail.com writes:
On 27 October 2013 12:58, Marek Olšák mar...@gmail.com wrote:
On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry stereotype...@gmail.com
wrote:
On 20 October 2013 09:20, Marek Olšák
On 10/28/2013 04:29 PM, Emil Velikov wrote:
On 28/10/13 08:07, Adrian Negreanu wrote:
From: Adrian Negreanu adrian.m.negre...@intel.com
Fixes linker error:
ld: .../libmesa_dri_common_intermediates/libmesa_dri_common.a(dri_util.o):
in function globalDriverAPI:dri_util.c(.data.rel+0x0):
Paul Berry stereotype...@gmail.com writes:
On 21 October 2013 17:48, Eric Anholt e...@anholt.net wrote:
Things blew up when I enabled the debug register spill code without
disabling 16-wide, so I decided to just fix 16-wide spilling.
We still don't generate 16-wide when register spilling
On Tue, Oct 29, 2013 at 12:36 AM, Eric Anholt e...@anholt.net wrote:
Anuj Phogat anuj.pho...@gmail.com writes:
V2:
- Update comments.
- Make changes to support simd16 mode.
- Add compute_pos_offset variable in brw_wm_prog_key.
- Add variable uses_omask in brw_wm_prog_data.
Paul Berry stereotype...@gmail.com writes:
On 29 October 2013 11:20, Eric Anholt e...@anholt.net wrote:
Paul Berry stereotype...@gmail.com writes:
On 27 October 2013 12:58, Marek Olšák mar...@gmail.com wrote:
On Sun, Oct 27, 2013 at 4:19 PM, Paul Berry stereotype...@gmail.com
wrote:
On 29 October 2013 12:04, Eric Anholt e...@anholt.net wrote:
Paul Berry stereotype...@gmail.com writes:
On 29 October 2013 11:20, Eric Anholt e...@anholt.net wrote:
Paul Berry stereotype...@gmail.com writes:
On 27 October 2013 12:58, Marek Olšák mar...@gmail.com wrote:
On Sun,
Tom Stellard schrieb am 29.10.2013 17:48:
On Sun, Oct 27, 2013 at 07:36:07PM +0100, Kai Wasserbäch wrote:
Fixes a boat load of Piglit tests for me, which crashed like fdo#70913
before.
Thanks to Michel Dänzer for the tip.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=70913
Matt Turner matts...@gmail.com writes:
Helps a lot of Steam games.
total instructions in shared programs: 1287432 - 1287130 (-0.02%)
instructions in affected programs: 21447 - 21145 (-1.41%)
13-15 are also:
Reviewed-by: Eric Anholt e...@anholt.net
I'm spending a while thinking about
Hi,
As I understand it, Mesa is in charge of providing OpenGL drivers for Intel
graphics hardware running on Linux. I'm in the process of porting our game
engine over to Linux, and have been pleasantly surprised so far that the OpenGL
4 drivers for Nvidia and ATI on Linux work perfectly:
From: Rob Clark robcl...@freedesktop.org
Use u_primconvert to convert unsupported primitives into supported
primitive plus index buffer.
Signed-off-by: Rob Clark robcl...@freedesktop.org
---
src/gallium/drivers/freedreno/a2xx/fd2_context.c | 24 ++-
From: Rob Clark robcl...@freedesktop.org
A convenient front end to indices generate/translate code, for emulating
primitives which are not supported natively by the driver.
This handles saving/restoring index buffer state, etc.
Signed-off-by: Rob Clark robcl...@freedesktop.org
Reviewed-by:
From: Rob Clark robcl...@freedesktop.org
This patchset (compared to RFC I sent previously) changes u_primconvert
to just be a front-end to the u_indices stuff. It handles binding/
restoring new index buffer state, etc. So driver using it just has
to put this at the top of their pipe-draw_vbo():
From: Rob Clark robcl...@freedesktop.org
Add 'start' parameter to generator/translator.
Signed-off-by: Rob Clark robcl...@freedesktop.org
Reviewed-by: Brian Paul bri...@vmware.com
---
v1: original
v2: add comments
src/gallium/auxiliary/indices/u_indices.c | 6 --
---
src/mesa/drivers/common/meta.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
index 798efa6..aa50dde 100644
--- a/src/mesa/drivers/common/meta.c
+++
---
src/mesa/swrast/s_texfetch_tmp.h |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/swrast/s_texfetch_tmp.h b/src/mesa/swrast/s_texfetch_tmp.h
index 7a68753..e2521b5 100644
--- a/src/mesa/swrast/s_texfetch_tmp.h
+++ b/src/mesa/swrast/s_texfetch_tmp.h
@@
---
src/glsl/glsl_types.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/glsl/glsl_types.h b/src/glsl/glsl_types.h
index fdb1f3a..96eee5e 100644
--- a/src/glsl/glsl_types.h
+++ b/src/glsl/glsl_types.h
@@ -459,7 +459,7 @@ struct glsl_type {
*/
bool
Fixes warning: cast from pointer to integer of different size for
64-bit builds.
---
src/mesa/state_tracker/st_draw.h |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_draw.h b/src/mesa/state_tracker/st_draw.h
index 3313fc8..394473b 100644
---
---
src/mesa/main/performance_monitor.c |6 +++---
src/mesa/main/uniforms.c|2 +-
src/mesa/main/vdpau.c |6 --
3 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/performance_monitor.c
b/src/mesa/main/performance_monitor.c
index
---
src/mesa/vbo/vbo_attrib_tmp.h |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/vbo/vbo_attrib_tmp.h b/src/mesa/vbo/vbo_attrib_tmp.h
index 02c283d..bbc0205 100644
--- a/src/mesa/vbo/vbo_attrib_tmp.h
+++ b/src/mesa/vbo/vbo_attrib_tmp.h
@@ -140,7 +140,7 @@
Josh Klint joshkl...@leadwerks.com writes:
Hi,
As I understand it, Mesa is in charge of providing OpenGL drivers for
Intel graphics hardware running on Linux. I'm in the process of
porting our game engine over to Linux, and have been pleasantly
surprised so far that the OpenGL 4 drivers
On 10/29/2013 02:10 PM, Rob Clark wrote:
From: Rob Clark robcl...@freedesktop.org
A convenient front end to indices generate/translate code, for emulating
primitives which are not supported natively by the driver.
This handles saving/restoring index buffer state, etc.
Signed-off-by: Rob Clark
I'm going to be introducing gen7 variants, and the previous naming was
going to get confusing.
Reviewed-by: Paul Berry stereotype...@gmail.com
---
src/mesa/drivers/dri/i965/brw_defines.h | 7 +++
src/mesa/drivers/dri/i965/brw_fs.cpp| 4 ++--
This hasn't been true since SIMD16 mode was added.
---
src/mesa/drivers/dri/i965/brw_fs.h | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.h
b/src/mesa/drivers/dri/i965/brw_fs.h
index 5b78313..5f331e1 100644
---
This avoids a lot of message setup we had to do otherwise. Improves
GLB2.7 performance with register spilling force enabled by 1.6442% +/-
0.553218% (n=4).
v2: Use BRW_PREDICATE_NONE, improve a comment (by Paul).
Reviewed-by: Paul Berry stereotype...@gmail.com
---
Things blew up when I enabled the debug register spill code without
disabling 16-wide, so I decided to just fix 16-wide spilling.
We still don't generate 16-wide when register spilling happens as part of
allocation (since we expect it to be slower), but now we can experiment
with allowing it in
Now that reg spilling generates new vgrfs, we were looping forever if you
ever turned it on.
Instead, move the debug code into the register allocator right near where
we'd be doing spilling anyway, which should more accurately reflect how
register spilling occurs in the wild.
Reviewed-by: Paul
I'm going to need to reuse this for fixing register spilling on SIMD16.
Note that BRW_MAX_MRF is 16, which is the same as BRW_MAX_GRF -
GEN7_MRF_HACK_START.
---
src/mesa/drivers/dri/i965/brw_fs.h| 1 +
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 33 ---
I believe this will never happen in SIMD8 mode, but it could for SIMD16
when we fix it.
---
src/mesa/drivers/dri/i965/brw_fs.h| 1 +
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 23 +++
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 2 ++
3 files
We were clearing the reg_offset before trying to use it. Oops. Fixes
glsl-fs-texture2drect with the reg spilling debug enabled.
Reviewed-by: Paul Berry stereotype...@gmail.com
---
src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
This is a resend of the series after updates for Paul's review feedback.
This time I've done a full piglit run with spilling forced on and no
regressions were found. It can also be found at gen7-scratch-read of my
tree.
___
mesa-dev mailing list
From: Tom Stellard thomas.stell...@amd.com
You can use the --enable-pipe-loader-render-nodes configure flag to
make the pipe-loader use render nodes for talking with the device.
---
configure.ac | 6 ++
src/gallium/auxiliary/pipe-loader/Makefile.am |
From: Tom Stellard thomas.stell...@amd.com
DRM_IOCTL_VERSION is not currently a legal ioctl for render nodes, so we
need to hard code the drm version when drmGetVersion() returns NULL.
---
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 36 ---
1 file changed, 25
On Tue, Oct 29, 2013 at 12:36 PM, Eric Anholt e...@anholt.net wrote:
Matt Turner matts...@gmail.com writes:
Helps a lot of Steam games.
total instructions in shared programs: 1287432 - 1287130 (-0.02%)
instructions in affected programs: 21447 - 21145 (-1.41%)
13-15 are also:
On 28/09/13 15:46, Emil Velikov wrote:
Slight extension over the previous two patches, removing unused variables
and headers. Last two patches do not strictly below in this series, as
they provide more consistent automake/configure usage of pkg-config
(PKG_CHECK_MODULES).
Please comment and
https://bugs.freedesktop.org/show_bug.cgi?id=71022
Priority: medium
Bug ID: 71022
Keywords: regression
CC: emil.l.veli...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: configure: error: Expat required for DRI.
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #1 from Bryce Harrington b.harring...@samsung.com ---
Same error on ubuntu 12.04
$ apt-cache policy expat
expat:
Installed: 2.0.1-7.2ubuntu1.1
Candidate: 2.0.1-7.2ubuntu1.1
Looks like expat.pc was added in expat 2.1. 2.0
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #2 from Bryce Harrington b.harring...@samsung.com ---
Reverting patch aec20d66 enables it to pass the configure check.
commit aec20d66d9d13e0acd6a7199b63e1383e1e9900a
Author: Emil Velikov emil.l.veli...@gmail.com
Date: Tue Oct 29
Crap. Two patches were mistakenly squashed together resulting in this.
I'll resend.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 29 October 2013 15:59, Anuj Phogat anuj.pho...@gmail.com wrote:
On Mon, Oct 28, 2013 at 6:48 PM, Paul Berry stereotype...@gmail.comwrote:
On 25 October 2013 16:45, Anuj Phogat anuj.pho...@gmail.com wrote:
- Enable GEN7_WM_MSDISPMODE_PERSAMPLE, GEN7_WM_POSOFFSET_SAMPLE,
I merged most of my atomic-counters branch a few hours ago, however
some of the patches haven't received much attention and I don't feel
confident enough with pushing them yet, even though they have been
sitting on the mailing list for more than six weeks.
I'm resending only those patches that
v2: Mark atomic counters as read-only variables. Move offset overlap
code to the linker. Use the contains_atomic() convenience method.
v3: Use pointer to integer instead of non-const reference. Add
comment so we remember to add a spec quotation from the next GLSL
release once the
This patch fixes the three dead code elimination passes and the
VEC4/FS instruction scheduling passes so they leave instructions with
side effects alone.
At some point it might be interesting to have the instruction
scheduler calculate the exact memory dependencies between atomic ops,
but they're
v2: Add comments on the purpose of the auxiliary data structures.
Check for atomic counter overlaps. Use the contains_atomic()
convenience method. Add static assert with the number of expected
shader stages.
v3: Don't resize atomic arrays.
---
src/glsl/Makefile.sources | 1 +
This can deal with all the 15 32-bit untyped atomic operations the
hardware supports, but only INC and PREDEC are going to be exposed
through the API for now.
v2: Represent atomics as GLSL intrinsics. Add support for variably
indexed atomic counter arrays. Fix interaction with fragment
The latency information has been obtained empirically from
measurements taken on Haswell and Ivy Bridge.
---
.../drivers/dri/i965/brw_schedule_instructions.cpp | 41 ++
1 file changed, 41 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
This can deal with all the 15 32-bit untyped atomic operations the
hardware supports, but only INC and PREDEC are going to be exposed
through the API for now.
v2: Represent atomics as GLSL intrinsics. Add support for variably
indexed atomic counter arrays.
---
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #3 from Matt Turner matts...@gmail.com ---
expat-2.1.0 was released March 24 2012. That's kind of a long time for it to
not be in your distribution.
I'm personally okay with requiring 2.1.0, but I suppose people want new Mesa on
old
From: Marek Olšák marek.ol...@amd.com
This doesn't make any difference on Bonaire, but it might help on Hawaii.
---
src/gallium/drivers/radeonsi/si_state.c | 1 +
src/gallium/drivers/radeonsi/si_state.h | 1 +
src/gallium/drivers/radeonsi/si_state_draw.c | 8 ++--
3 files changed,
From: Marek Olšák marek.ol...@amd.com
The type-2 NOPs are said to be unstable. It doesn't make a difference here.
---
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_cs.c
On Tue, Oct 29, 2013 at 2:26 PM, Matt Turner matts...@gmail.com wrote:
But hopefully your patch just avoids this problem entirely. I'll
provide an update later.
7 shader helped, 4 shaders hurt by going from my series and your
patch, to my series, your patch, and disabling the original
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #4 from Emil Velikov emil.l.veli...@gmail.com ---
(In reply to comment #3)
expat-2.1.0 was released March 24 2012. That's kind of a long time for it to
not be in your distribution.
I'm personally okay with requiring 2.1.0, but I
On Mon, Oct 28, 2013 at 11:08 PM, Eric Anholt e...@anholt.net wrote:
Matt Turner matts...@gmail.com writes:
Since compare instructions write the flag register, they should not be
considered dead even if their destination is never read. Instead of
removing them if found to be dead, set their
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #5 from Bryce Harrington b.harring...@samsung.com ---
Created attachment 88328
-- https://bugs.freedesktop.org/attachment.cgi?id=88328action=edit
Hardcode EXPAT in configure.ac
expat 2.0 does not provide the pkgconfig file.
Hi Tom,
this has been on my todo list for quite a while.
Your patch looks good to me, but in my experience a block with approximately
the same size for each dimension gives slightly better performance in many
cases when compared to one where one dimension is significantly larger.
Maybe you
We'd like to CSE some instructions, like CMP, that often have null
destinations. Instead of replacing them with MOVs to null, just don't
emit the MOV.
---
Updated version: Replaced two comments with Eric's suggested comment.
Left the comment about the basic block boundary in place. Let me know
if
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #6 from Bryce Harrington b.harring...@samsung.com ---
CVE-2012-1147 was determined not to be relevant on Ubuntu:
http://people.canonical.com/~ubuntu-security/cve/2012/CVE-2012-1147.html
--
You are receiving this mail because:
You
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #7 from Emil Velikov emil.l.veli...@gmail.com ---
Created attachment 88329
-- https://bugs.freedesktop.org/attachment.cgi?id=88329action=edit
handle expat versions pre 2.1
(In reply to comment #6)
CVE-2012-1147 was determined not
On Tue, Oct 29, 2013 at 4:31 PM, Paul Berry stereotype...@gmail.com wrote:
On 29 October 2013 15:59, Anuj Phogat anuj.pho...@gmail.com wrote:
On Mon, Oct 28, 2013 at 6:48 PM, Paul Berry stereotype...@gmail.comwrote:
On 25 October 2013 16:45, Anuj Phogat anuj.pho...@gmail.com wrote:
-
Commit aec20d66d9d13e0acd6a7199b63e1383e1e9900a
(automake: properly handle non-default expat installation),
assumed that up-to date distributions use a recent version
of expat that handles security vunerabilities CVE-2012-1147
and CVE-2012-1148. Seems like this is not always the case
and they
On Tue, Oct 29, 2013 at 7:06 PM, Niels Ole Salscheider
niels_...@salscheider-online.de wrote:
Hi Tom,
this has been on my todo list for quite a while.
Your patch looks good to me, but in my experience a block with approximately
the same size for each dimension gives slightly better
From: Ian Romanick ian.d.roman...@intel.com
These will be used to determine whether to signal a GPU reset after
another context in the share group has observed a reset.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/context.c | 1 +
src/mesa/main/mtypes.h | 18
From: Ian Romanick ian.d.roman...@intel.com
Drivers still have to implement dd_function_table::GetGraphicsResetStatus.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/getstring.c | 47 +++
1 file changed, 43 insertions(+), 4
From: Ian Romanick ian.d.roman...@intel.com
No drivers advertise the DRI2 extension yet, so no driver should ever
see a value other than false for notify_reset.
The changes in nouveau use tabs because nouveau seems to have it's own
indentation rules.
Signed-off-by: Ian Romanick
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/drivers/dri/i965/intel_screen.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/intel_screen.c
b/src/mesa/drivers/dri/i965/intel_screen.c
index
From: Ian Romanick ian.d.roman...@intel.com
Soon some drivers will support a different set of flags than other
drivers. If some flags have to be filtered in the driver, we might as
well filter all of them in the driver.
The changes in nouveau use tabs because nouveau seems to have it's own
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/drivers/dri/i965/brw_context.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
From: Ian Romanick ian.d.roman...@intel.com
This isn't going to be used in the actual implemenation of
glGetGraphicsResetStatus.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/main/context.c | 1 -
src/mesa/main/getstring.c | 2 +-
src/mesa/main/mtypes.h| 3 ---
3 files
From: Ian Romanick ian.d.roman...@intel.com
v2: Update based on kernel interface / libdrm changes.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/drivers/dri/i965/Makefile.sources | 1 +
src/mesa/drivers/dri/i965/brw_context.h| 3 ++
From: Ian Romanick ian.d.roman...@intel.com
If the application requests reset notifiction, connect up the reset
status query method and set gl_context::ResetStrategy.
v2: Update based on kernel interface / libdrm changes.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
https://bugs.freedesktop.org/show_bug.cgi?id=71022
--- Comment #8 from Bryce Harrington b.harring...@samsung.com ---
Yep, that does the trick.
Tested-by: Bryce Harrington b.harring...@samsung.com
--
You are receiving this mail because:
You are the assignee for the bug.
On Mon, Oct 28, 2013 at 5:10 PM, Paul Berry stereotype...@gmail.com wrote:
On 25 October 2013 16:45, Anuj Phogat anuj.pho...@gmail.com wrote:
V2:
- Update comments
- Use fs_reg(0x) in AND instruction to get the 16 bit
sample_mask.
- Add a special backend instructions to
On 29 October 2013 17:16, Anuj Phogat anuj.pho...@gmail.com wrote:
On Tue, Oct 29, 2013 at 4:31 PM, Paul Berry stereotype...@gmail.comwrote:
I think the right thing to do is to add:
if (dispatch_width == 16)
fail(...);
to whatever parts of the visitor you aren't confident will
On 29 October 2013 21:28, Paul Berry stereotype...@gmail.com wrote:
On 29 October 2013 17:16, Anuj Phogat anuj.pho...@gmail.com wrote:
On Tue, Oct 29, 2013 at 4:31 PM, Paul Berry stereotype...@gmail.comwrote:
I think the right thing to do is to add:
if (dispatch_width == 16)
On 29 October 2013 19:22, Anuj Phogat anuj.pho...@gmail.com wrote:
On Mon, Oct 28, 2013 at 5:10 PM, Paul Berry stereotype...@gmail.comwrote:
On 25 October 2013 16:45, Anuj Phogat anuj.pho...@gmail.com wrote:
V2:
- Update comments
- Use fs_reg(0x) in AND instruction to get the
100 matches
Mail list logo