On 04/05/2014 01:13 AM, Ian Romanick wrote:
I've been thinking about this a bit, and I think this is more complex
than it needs to be. I don't have a complete design, but here are some
thoughts on the matter.
When linking is all done, we just want the UniformRemapTable. It's
entires should
The next few patches will introduce an optimization that only works when
integers are not represented as floating point values.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/glsl_parser_extras.cpp| 6 --
src/glsl/ir_optimization.h | 3 ++-
Performance warnings are logged via KHR_debug in addition to when the
INTEL_DEBUG=perf environment variable is set. Without this, messages in
debug contexts would have (null) for the reason.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 2 +-
1
ir_binop_ubo_load takes unsigned integer operands. However, the array
index used to compute these offsets may be a signed integer. (For
example, see Piglit's spec/glsl-1.40/uniform_buffer/fs-bvec-array).
For some reason, we were missing an ir_binop_i2u cast, and ir_validator
was failing to
The vector backend already implemented this optimization, but
surprisingly, we never bothered to implement it in the scalar backend.
In addition to saving two instructions, this eliminates a use of the
accumulator as an explicit source, which is unsupported in SIMD16 mode
on Gen7+, which could
The IR is not supposed to support implicit type conversions; we just
failed to validate it.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/ir_validate.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/glsl/ir_validate.cpp b/src/glsl/ir_validate.cpp
index
The i965 MUL instruction doesn't natively support 32-bit by 32-bit
integer multiplication; additional instructions (MACH/MOV) are required.
However, we can avoid those if we know one of the operands can be
represented in 16 bits or less. The vector backend's is_16bit_constant
static helper
The next patch will introduce an optimization that only works when
integers are not represented as floating point values.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/glsl/glsl_parser_extras.cpp | 2 +-
src/glsl/ir_optimization.h | 2 +-
src/glsl/opt_algebraic.cpp | 8
This makes the function available from core Mesa code, including the
GLSL compiler.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/i965/brw_context.h | 6 --
src/mesa/main/macros.h | 11 +++
2 files changed, 11 insertions(+), 6
Integer shifts are basically always well supported and efficient; that
isn't always true of integer division, and sometimes even integer
multiplication isn't without issues.
On some Intel hardware, INTDIV can't be used in SIMD16 mode. It also
doesn't support immediate operands (on any
Cc: 10.1 mesa-sta...@lists.freedesktop.org
Signed-off-by: Thomas Hellstrom thellst...@vmware.com
---
src/gallium/winsys/svga/drm/vmwgfx_drm.h | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
diff --git a/src/gallium/winsys/svga/drm/vmwgfx_drm.h
Implement guest-backed surface sharing using prime fds. Previously only
legacy surfaces could use this functionality. Also use the vmwgfx 2.6
single-ioctl prime fd reference if available.
Cc: 10.1 mesa-sta...@lists.freedesktop.org
Signed-off-by: Thomas Hellstrom thellst...@vmware.com
---
On Mon, Apr 7, 2014 at 11:50 AM, Thomas Hellstrom thellst...@vmware.com wrote:
Cc: 10.1 mesa-sta...@lists.freedesktop.org
Signed-off-by: Thomas Hellstrom thellst...@vmware.com
---
src/gallium/winsys/svga/drm/vmwgfx_drm.h | 13 -
1 file changed, 12 insertions(+), 1 deletion(-)
Binding a new destination may cause the svga driver to emit draw calls
while propagating the surface. Make sure this doesn't happen in the middle
of sampler state setup where state may be incosistent.
In practice, surface propagation should never happen here and even if it did,
it wouldn't be a
Ping.
Richard Sandiford rsand...@linux.vnet.ibm.com writes:
Ping (with fixed subject)
Richard Sandiford rsand...@linux.vnet.ibm.com writes:
This is a refresh of:
http://lists.freedesktop.org/archives/mesa-dev/2013-June/040594.html
At the moment the python code uses sys.byteorder to
op 03-04-14 18:19, Marek Olšák schreef:
Acked-by: Marek Olšák marek.ol...@amd.com
Nacked by self btw, the alternative fix below doesn't break other testcases.
The original fix did..
No idea why it is legal to move a constant struct/array here, but it appears to
pass all tests...
---
diff
On Mon, Apr 7, 2014 at 12:32 PM, Thomas Hellstrom thellst...@vmware.com wrote:
Binding a new destination may cause the svga driver to emit draw calls
while propagating the surface. Make sure this doesn't happen in the middle
of sampler state setup where state may be incosistent.
In practice,
On 06/04/2014 17:45, Emil Velikov wrote:
Sorry about this breakage. I assumed that the library extensions are handled
consistently across mesa. Seems like I was wrong.
Guessing that you meant to send this to the mesa-dev ?
Yes, sorry about that.
On 06/04/14 15:59, Jon TURNEY wrote:
A
On Mon, Apr 7, 2014 at 2:49 AM, Kenneth Graunke kenn...@whitecape.org wrote:
The i965 MUL instruction doesn't natively support 32-bit by 32-bit
integer multiplication; additional instructions (MACH/MOV) are required.
However, we can avoid those if we know one of the operands can be
represented
Previously the version was specified in configure.ac as well as
one of the xa headers. Thus incleasing the risk of discrepancies
when a developer bumps one but forgets about the other.
Currently only automake builds the xa state-tracker + targets,
and we should move the version definitions to a
Parsing source files through build systems is never a good idea.
Especially when the issue can be resolved by adding a couple of
definitions to CPPFLAGS via the build system.
This reverts commit 61bedc3d6b08943f015f9d590c07a6af36c2a92c.
Cc: Thomas Hellstrom thellst...@vmware.com
---
Seems like
The kernel driver expects the class to be based on chipset generation
rather than VP generation. Make sure to pass 90b1 for NVDX chipsets
instead of 95b1.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=77102
Fixes: 40dd777b33073
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu
Cc: 10.1
On 07/04/14 10:50, Thomas Hellstrom wrote:
Cc: 10.1 mesa-sta...@lists.freedesktop.org
Signed-off-by: Thomas Hellstrom thellst...@vmware.com
Rather silly question:
Why do you guys pull this header inside mesa and over using the one provided
by libdrm ? AFAICS every other driver seems to do the
On 04/07/2014 02:44 PM, Emil Velikov wrote:
Parsing source files through build systems is never a good idea.
Especially when the issue can be resolved by adding a couple of
definitions to CPPFLAGS via the build system.
I've nothing against reverting the commit, but the log message is
On 04/07/2014 02:44 PM, Emil Velikov wrote:
Previously the version was specified in configure.ac as well as
one of the xa headers. Thus incleasing the risk of discrepancies
when a developer bumps one but forgets about the other.
Currently only automake builds the xa state-tracker + targets,
On 04/04/2014 05:52 PM, Matt Turner wrote:
On Fri, Apr 4, 2014 at 5:18 PM, Ian Romanick i...@freedesktop.org wrote:
Fast forwarding 3 months from the 10.1 release (March 4th, planned for
February 28th) is May 30th. I'd like to propose the following set of dates:
May 2nd: Feature freeze /
On 04/06/2014 09:31 PM, Chia-I Wu wrote:
From: Chia-I Wu o...@lunarg.com
Given
mov vgrf7, vgrf9.xyxz
add vgrf9.xyz, vgrf4.xyzw, vgrf5.xyzw
add vgrf10.x, vgrf6.xyzw, vgrf7.
the last instruction would be wrongly changed to
add vgrf10.x, vgrf6.xyzw, vgrf9.
during
On 04/07/2014 02:49 PM, Emil Velikov wrote:
On 07/04/14 10:50, Thomas Hellstrom wrote:
Cc: 10.1 mesa-sta...@lists.freedesktop.org
Signed-off-by: Thomas Hellstrom thellst...@vmware.com
Rather silly question:
Why do you guys pull this header inside mesa and over using the one provided
by
On 04/06/2014 11:49 PM, Kenneth Graunke wrote:
The i965 MUL instruction doesn't natively support 32-bit by 32-bit
integer multiplication; additional instructions (MACH/MOV) are required.
However, we can avoid those if we know one of the operands can be
represented in 16 bits or less. The
On 04/06/2014 11:49 PM, Kenneth Graunke wrote:
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 3bf2789..d8edc95 100644
--- a/src/glsl/linker.cpp
+++ b/src/glsl/linker.cpp
@@ -2298,7 +2298,7 @@ link_shaders(struct gl_context *ctx, struct
gl_shader_program *prog)
This reverts commit 61bedc3d6b08943f015f9d590c07a6af36c2a92c.
As the header is the one defining the API/ABI and is distributed
during installation, we should be using it rather than re-defining
the XA version in configure.ac.
Bump the version in the header to 2.2.0, to reflect what was the
On 04/06/2014 11:49 PM, Kenneth Graunke wrote:
Integer shifts are basically always well supported and efficient; that
isn't always true of integer division, and sometimes even integer
multiplication isn't without issues.
On some Intel hardware, INTDIV can't be used in SIMD16 mode. It also
I made a couple random comments on several patches. I also like Ilia's
suggestion on patch 3.
Patches 1 through 8 (with Ilia's suggestion on 3) are
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
I think patch 9 still needs some work for signed numerator handling in
division.
On 04/06/2014
On 04/06/2014 10:27 PM, Tapani Pälli wrote:
On 04/05/2014 12:58 AM, Ian Romanick wrote:
On 04/04/2014 02:42 PM, Ian Romanick wrote:
On 03/27/2014 11:45 PM, Tapani Pälli wrote:
Patch adds a preprocessor define for the extension and stores
explicit location data for uniforms during AST-HIR
On 07/04/14 13:05, Jon TURNEY wrote:
On 06/04/2014 17:45, Emil Velikov wrote:
IMHO if we're starting from scratch we should use the platform specific
extensions consistently but neither suggestion matter in this case :'(
Can you explain your reasoning? This seems to me to be a foolish
While linux uses .so as a default extension of a shared library that is
not the case for other platforms. The loader in libGL (and others) assume
that the dri module will always have a .so extension, thus it will fail
to load it on some platforms.
Spotted-by: Jon TURNEY
On 04/05/2014 03:50 PM, Brian Paul wrote:
Rename functions to match format names.
sed commands:
s/signed_rgba_rev/R8G8B8A8_SNORM/g
s/signed_rgba/A8B8G8R8_SNORM/g
s/f_rgba_rev/R8G8B8A_UNORM/g
s/f_rgba/A8B8G8R8_UNORM/g
s/f_rgbx_rev/R8G8B8X8_UNORM/g
The series is
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
On 04/05/2014 03:50 PM, Brian Paul wrote:
So the function names match the format names.
---
src/mesa/main/format_unpack.c | 80
-
1 file changed, 40 insertions(+), 40 deletions(-)
On 04/07/2014 09:17 AM, Ian Romanick wrote:
On 04/05/2014 03:50 PM, Brian Paul wrote:
Rename functions to match format names.
sed commands:
s/signed_rgba_rev/R8G8B8A8_SNORM/g
s/signed_rgba/A8B8G8R8_SNORM/g
s/f_rgba_rev/R8G8B8A_UNORM/g
s/f_rgba/A8B8G8R8_UNORM/g
https://bugs.freedesktop.org/show_bug.cgi?id=61750
--- Comment #4 from Will Wagner wi...@carallon.com ---
This appears to be a duplicate of bug 75098?
--
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
On 03/13/2014 08:15 AM, Mika Kuoppala wrote:
Ian Romanick i...@freedesktop.org writes:
On 03/12/2014 01:43 AM, Kenneth Graunke wrote:
arekm reported that using Chrome with GPU acceleration enabled on GM45
triggered the hw_ctx != NULL assertion in brw_get_graphics_reset_status.
We
Kenneth Graunke kenn...@whitecape.org writes:
On 04/04/2014 03:28 PM, Matt Turner wrote:
The generator uses its destination as a source implicitly, which breaks
some assumptions in dead code elimination. Giving the instruction a
source allows us to reason about it better.
---
Ian Romanick i...@freedesktop.org writes:
On 04/04/2014 05:52 PM, Matt Turner wrote:
On Fri, Apr 4, 2014 at 5:18 PM, Ian Romanick i...@freedesktop.org wrote:
Fast forwarding 3 months from the 10.1 release (March 4th, planned for
February 28th) is May 30th. I'd like to propose the following
Iago Toral Quiroga ito...@igalia.com writes:
Commit 11baad35088dfd4bdabc1710df650dbfb413e7a3 produces a regression when
switching a single context between multiple drawables.
The problem is that we check whether we have a viewport set to decide if we
need to generate buffers for the drawble,
Matt Turner matts...@gmail.com writes:
It's more likely that we won't find writes to all channels than one will
interfere, and calculating interference is more expensive. This change
will also help prepare for coalescing load_payload instructions'
operands.
Also update the live intervals
Notice our multiple values for M_PI_2, which rounded ...32 up to
...4 and ...5.
---
The float casts are ugly. I tried to define M_PI_2f using the
preprocessor -- something like
#define M_PI_2f M_PI_2##f
but no luck.
src/glsl/builtin_functions.cpp | 14 +++---
1 file changed, 7
On Mon, Apr 7, 2014 at 12:19 PM, Matt Turner matts...@gmail.com wrote:
Notice our multiple values for M_PI_2, which rounded ...32 up to
...4 and ...5.
---
The float casts are ugly. I tried to define M_PI_2f using the
preprocessor -- something like
#define M_PI_2f M_PI_2##f
but no luck.
On Mon, Apr 7, 2014 at 10:28 AM, Aaron Watry awa...@gmail.com wrote:
On Mon, Apr 7, 2014 at 12:19 PM, Matt Turner matts...@gmail.com wrote:
Notice our multiple values for M_PI_2, which rounded ...32 up to
...4 and ...5.
---
The float casts are ugly. I tried to define M_PI_2f using the
Am 07.04.2014 15:52, schrieb Ian Romanick:
On 04/06/2014 11:49 PM, Kenneth Graunke wrote:
Integer shifts are basically always well supported and efficient; that
isn't always true of integer division, and sometimes even integer
multiplication isn't without issues.
On some Intel hardware,
Iago Toral Quiroga ito...@igalia.com writes:
glClearBuffer() is currently clearing all active draw color buffers (all
buffers that have not been set to GL_NONE when calling glDrawBuffers) instead
of only clearing the one it receives as parameter. Altough brw_clear()
receives a bit mask
Kenneth Graunke kenn...@whitecape.org writes:
These are clearly needed---the comments in the function are even present
for each one of them. I originally had two separate state atoms for
3DSTATE_SBE and 3DSTATE_SBE_SWIZ. When I combined the functions, I must
have forgotten to add the atoms
Iago Toral Quiroga ito...@igalia.com writes:
When doing software rendering (i.e. rendering to the selection buffer) we need
to make sure that we have valid index bounds before calling _tnl_draw_prims(),
otherwise we can crash.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59455
---
Fixes failures in Khronos OpenGL CTS test conditional_render_test9
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
src/mesa/drivers/dri/i965/intel_fbo.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/intel_fbo.c
b/src/mesa/drivers/dri/i965/intel_fbo.c
On Sun, Apr 6, 2014 at 11:49 PM, Kenneth Graunke kenn...@whitecape.org wrote:
Integer shifts are basically always well supported and efficient; that
isn't always true of integer division, and sometimes even integer
multiplication isn't without issues.
On some Intel hardware, INTDIV can't be
On Sun, Apr 6, 2014 at 11:49 PM, Kenneth Graunke kenn...@whitecape.org wrote:
Performance warnings are logged via KHR_debug in addition to when the
INTEL_DEBUG=perf environment variable is set. Without this, messages in
debug contexts would have (null) for the reason.
Signed-off-by: Kenneth
https://bugs.freedesktop.org/show_bug.cgi?id=77152
Priority: medium
Bug ID: 77152
Keywords: regression
CC: matts...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: libglapi.so: undefined reference to
On 04/07/2014 10:28 AM, Aaron Watry wrote:
On Mon, Apr 7, 2014 at 12:19 PM, Matt Turner matts...@gmail.com wrote:
Notice our multiple values for M_PI_2, which rounded ...32 up to
...4 and ...5.
---
The float casts are ugly. I tried to define M_PI_2f using the
preprocessor -- something like
On 04/07/2014 09:14 AM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
On 04/04/2014 05:52 PM, Matt Turner wrote:
On Fri, Apr 4, 2014 at 5:18 PM, Ian Romanick i...@freedesktop.org wrote:
Fast forwarding 3 months from the 10.1 release (March 4th, planned for
February 28th) is
I thought I was seeing a bug in the code while reviewing, but it's not
there.
---
src/mesa/drivers/dri/i965/.gitignore | 1 +
src/mesa/drivers/dri/i965/Makefile.am | 7 +
.../dri/i965/test_vec4_copy_propagation.cpp| 156 +
3 files
Chia-I Wu olva...@gmail.com writes:
From: Chia-I Wu o...@lunarg.com
Given
mov vgrf7, vgrf9.xyxz
add vgrf9.xyz, vgrf4.xyzw, vgrf5.xyzw
add vgrf10.x, vgrf6.xyzw, vgrf7.
the last instruction would be wrongly changed to
add vgrf10.x, vgrf6.xyzw, vgrf9.
during copy
From: Marek Olšák marek.ol...@amd.com
This replaces u_gen_mipmap with an extremely simple implementation based
on pipe-blit. st/mesa is also cleaned up.
Pros:
- less code
- correct mipmap generation for NPOT 3D textures (u_blitter uses a better
formula)
- queries are not affected by mipmap
Signed-off-by: Anuj Phogat anuj.pho...@gmail.com
---
src/mesa/swrast/s_blit.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/swrast/s_blit.c b/src/mesa/swrast/s_blit.c
index 1ba188c..e3b45f1 100644
--- a/src/mesa/swrast/s_blit.c
+++ b/src/mesa/swrast/s_blit.c
@@ -29,6 +29,7
Updated patch to include reference to bug that it resolves.
Since this fixes a buffer overrun in the driver (found running GLBenchmark's
Manhattan test) I think it would be a good candidate for the stable branch.
___
mesa-dev mailing list
Just a trivial comment, otherwise looks good to me.
Thanks!
Roland
Am 07.04.2014 21:05, schrieb Marek Olšák:
From: Marek Olšák marek.ol...@amd.com
This replaces u_gen_mipmap with an extremely simple implementation based
on pipe-blit. st/mesa is also cleaned up.
Pros:
- less code
-
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
Reviewed-by: Matt Turner matts...@gmail.com
---
src/mesa/tnl/t_vertex.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/src/mesa/tnl/t_vertex.c b/src/mesa/tnl/t_vertex.c
index b3deac0..5a4bb0b
Set of patches which mostly were here earlier, hope I did not this time miss
any comments. #12 is new to the set. These pass Piglit quick tests with no
regression on my Ivybridge.
/Juha-Pekka
Ian Romanick (1):
mesa: Add _mesa_error_no_memory for logging out-of-memory messages
Juha-Pekka
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
Reviewed-by: Matt Turner matts...@gmail.com
---
src/mesa/vbo/vbo_rebase.c | 19 +++
1 file changed, 19 insertions(+)
diff --git a/src/mesa/vbo/vbo_rebase.c b/src/mesa/vbo/vbo_rebase.c
index f3fe5f7..374d129 100644
From: Ian Romanick ian.d.roman...@intel.com
This can be called from locations that don't have a context pointer
handy. This patch also adds enough infrastructure so that the unit
tests for the GLSL compiler and the stand-alone compiler will build and
function.
This patch was originally signed
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glsl/link_varyings.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index c925c00..c3cc509 100644
--- a/src/glsl/link_varyings.cpp
+++
Check calloc return values in hash_table_insert() and
hash_table_replace()
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/program/prog_hash_table.c | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/program/prog_hash_table.c
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/main/hash.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/mesa/main/hash.c b/src/mesa/main/hash.c
index 4c92005..976f7b8 100644
--- a/src/mesa/main/hash.c
+++ b/src/mesa/main/hash.c
@@ -115,10
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/drivers/dri/i965/intel_resolve_map.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/intel_resolve_map.c
b/src/mesa/drivers/dri/i965/intel_resolve_map.c
index 04b5c94..c5a4cd7 100644
On 04/07/2014 11:47 AM, Ian Romanick wrote:
On 04/07/2014 09:14 AM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
On 04/04/2014 05:52 PM, Matt Turner wrote:
On Fri, Apr 4, 2014 at 5:18 PM, Ian Romanick i...@freedesktop.org wrote:
Fast forwarding 3 months from the 10.1 release
On 04/07/2014 11:51 AM, Eric Anholt wrote:
I thought I was seeing a bug in the code while reviewing, but it's not
there.
One observation below. Regardless,
Reviewed-by: Ian Romanick ian.d.roman...@intel.com
---
src/mesa/drivers/dri/i965/.gitignore | 1 +
On 04/07/2014 12:19 PM, Courtney Goeltzenleuchter wrote:
Updated patch to include reference to bug that it resolves.
Where?
The proper way to do this is to send the path with git-send-mail, and
include extra verbage in the commit message like:
v3: Updated patch to include reference to bug that
Hmm... Wonder where my patches 9..12 vanished. They show in git
send-email as Result: 250 ok: Message ### accepted. If they don't
magically show up here I will resend entire set tomorrow from better
connection.
/Juha-Pekka
On Mon, Apr 7, 2014 at 10:48 PM, Juha-Pekka Heikkila
On 07/04/14 21:06, Juha-Pekka Heikkilä wrote:
Hmm... Wonder where my patches 9..12 vanished. They show in git
send-email as Result: 250 ok: Message ### accepted. If they don't
magically show up here I will resend entire set tomorrow from better
connection.
An innocent bystander type of
On Mon, Apr 7, 2014 at 11:39 AM, Ian Romanick i...@freedesktop.org wrote:
On 04/07/2014 10:28 AM, Aaron Watry wrote:
On Mon, Apr 7, 2014 at 12:19 PM, Matt Turner matts...@gmail.com wrote:
Notice our multiple values for M_PI_2, which rounded ...32 up to
...4 and ...5.
---
The float casts are
renderer_copy_prepare was setting the first sampler but never telling
the cso code how many samplers were actually used. Fix this.
Cc: 10.1 mesa-sta...@freedesktop.org
Signed-off-by: Thomas Hellstrom thellst...@vmware.com
---
src/gallium/state_trackers/xa/xa_renderer.c | 5 +++--
1 file changed,
On 04/07/2014 02:48 PM, Thomas Hellstrom wrote:
renderer_copy_prepare was setting the first sampler but never telling
the cso code how many samplers were actually used. Fix this.
Cc: 10.1 mesa-sta...@freedesktop.org
Signed-off-by: Thomas Hellstrom thellst...@vmware.com
---
On Mon, Apr 7, 2014 at 11:21 PM, Emil Velikov emil.l.veli...@gmail.com wrote:
On 07/04/14 21:06, Juha-Pekka Heikkilä wrote:
Hmm... Wonder where my patches 9..12 vanished. They show in git
send-email as Result: 250 ok: Message ### accepted. If they don't
magically show up here I will resend
On 12.03.2014 10:43, Kenneth Graunke wrote:
arekm reported that using Chrome with GPU acceleration enabled on GM45
triggered the hw_ctx != NULL assertion in brw_get_graphics_reset_status.
We definitely do not want to advertise reset notification support on
Gen4-5 systems, since it needs
On 07/04/14 22:10, Juha-Pekka Heikkilä wrote:
On Mon, Apr 7, 2014 at 11:21 PM, Emil Velikov emil.l.veli...@gmail.com
wrote:
On 07/04/14 21:06, Juha-Pekka Heikkilä wrote:
Hmm... Wonder where my patches 9..12 vanished. They show in git
send-email as Result: 250 ok: Message ### accepted. If
Type mismatch caused random memory to be copied when casted
memory area was smaller than expected type.
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/main/ff_fragment_shader.cpp | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/program/symbol_table.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/src/mesa/program/symbol_table.c b/src/mesa/program/symbol_table.c
index 9462978..5b22745 100644
---
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glsl/link_uniform_blocks.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/glsl/link_uniform_blocks.cpp b/src/glsl/link_uniform_blocks.cpp
index 72d6c53..add3ef4 100644
--- a/src/glsl/link_uniform_blocks.cpp
check variable_storage() found the requested fs_reg.
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
Check return value from hash_table_find before using it as a pointer
Signed-off-by: Juha-Pekka Heikkila juhapekka.heikk...@gmail.com
---
src/glsl/loop_analysis.cpp | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/glsl/loop_analysis.cpp b/src/glsl/loop_analysis.cpp
Ian Romanick i...@freedesktop.org writes:
On 04/07/2014 09:14 AM, Eric Anholt wrote:
Ian Romanick i...@freedesktop.org writes:
On 04/04/2014 05:52 PM, Matt Turner wrote:
On Fri, Apr 4, 2014 at 5:18 PM, Ian Romanick i...@freedesktop.org wrote:
Fast forwarding 3 months from the 10.1 release
https://bugs.freedesktop.org/show_bug.cgi?id=77152
Matt Turner matts...@gmail.com changed:
What|Removed |Added
Status|NEW |RESOLVED
Hi Ian,
I didn't think this even got out. I was getting a weird error from
git-send-mail when it tried to send to the gmail server and I got
distracted with other things.
Here's log:
Send this email? ([y]es|[n]o|[q]uit|[a]ll): y
Password for 'smtp://court...@lunarg.com@smtp.gmail.com:587':
OK.
Decompressing ETC2 textures was causing intermitent segfault
by copying resulting 4x4 texel block to the destination texture
regardless of the size of the destination texture. Issue found
via application crash in GLBenchmark 3.0's Manhattan test.
v2: add more explanatory comments
v3: add bugzilla
On 08/04/14 01:18, Courtney Goeltzenleuchter wrote:
Hi Ian,
I didn't think this even got out. I was getting a weird error from
git-send-mail when it tried to send to the gmail server and I got
distracted with other things.
Here's log:
Send this email? ([y]es|[n]o|[q]uit|[a]ll): y
https://bugs.freedesktop.org/show_bug.cgi?id=77161
Priority: medium
Bug ID: 77161
Keywords: regression
CC: mar...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] piglit fbo-generatemipmap-cubemap S3TC_DXT1
Thanks Emil, I've got it sorted out I think. The format of the cc: was
incorrect, it shouldn't have the mailto:... piece. Should look like: Cc:
9.2 10.0 10.1 mesa-sta...@lists.freedesktop.org
I'll send out updated patch.
Courtney
On Mon, Apr 7, 2014 at 6:28 PM, Emil Velikov
Decompressing ETC2 textures was causing intermitent segfault
by copying resulting 4x4 texel block to the destination texture
regardless of the size of the destination texture. Issue found
via application crash in GLBenchmark 3.0's Manhattan test.
v2: add more explanatory comments
v3: add bugzilla
Hi!
Thanks, This looks good to me.
Reviewed-by: Thomas Hellstrom thellst...@vmware.com
/Thomas
On 04/07/2014 03:45 PM, Emil Velikov wrote:
This reverts commit 61bedc3d6b08943f015f9d590c07a6af36c2a92c.
As the header is the one defining the API/ABI and is distributed
during installation, we
97 matches
Mail list logo