Series is
Reviewed-by: Jose Fonseca jfons...@vmware.com
- Original Message -
We weren't properly checking the return value of these calls (and
calls to u_upload_data()) to detect OOM errors.
---
src/mesa/state_tracker/st_cb_bitmap.c |5 ++---
On Tue, Jan 22, 2013 at 12:52 AM, Ian Romanick i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
Calculate all of the block member offsets, the IndexNames, and
everything else to do with every UBO.
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
23 26-31
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On Tue, Jan 22, 2013 at 12:51 AM, Ian Romanick i...@freedesktop.org wrote:
So here it is.
This is the last of the UBO instance and array instance rework for the
linker. It's a giant pile of patches, so let me explain what's going
https://bugs.freedesktop.org/show_bug.cgi?id=59831
--- Comment #1 from Michel Dänzer mic...@daenzer.net ---
Is that with or without --with-llvm-shared-libs for the Mesa build?
--
You are receiving this mail because:
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=59831
--- Comment #2 from Alex Deucher ag...@yahoo.com ---
without. Is that required now?
My configure options are:
./autogen.sh --prefix=/usr --libdir=/usr/lib64 --with-dri-drivers=radeon,r200
--with-gallium-drivers=r300,r600,radeonsi,swrast
On 01/24/2013 08:40 PM, Kenneth Graunke wrote:
On 01/22/2013 12:52 AM, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/glsl/glsl_types.cpp | 12 +++-
src/glsl/glsl_types.h| 6 ++
v2: Record texObj.BufferSize as -1 in TexBuffer(non-Range) instead
of the buffer's current size so we know we always have to use the
full size of the buffer object (i.e. even if it changes without the
user calling TexBuffer again) for the texture.
Clarify invalid offset alignment error message.
https://bugs.freedesktop.org/show_bug.cgi?id=59831
--- Comment #3 from Michel Dänzer mic...@daenzer.net ---
(In reply to comment #2)
without. Is that required now?
No, but I do wonder if we shouldn't drop support for linking LLVM statically.
--
You are receiving this mail because:
You are
On 01/23/2013 09:49 PM, Paul Berry wrote:
On 22 January 2013 00:52, Ian Romanick i...@freedesktop.org
mailto:i...@freedesktop.org wrote:
From: Ian Romanick ian.d.roman...@intel.com
mailto:ian.d.roman...@intel.com
If the block has an instance name, add the instance name to the
https://bugs.freedesktop.org/show_bug.cgi?id=59831
--- Comment #4 from Tom Stellard tstel...@gmail.com ---
The problem is that llvm_wrapper.cpp is being built without --enable-opencl or
--enable-r600-llvm-compiler, so the necessary libraries haven't been added to
LLVM_LIBS. The fix is to disable
On 01/24/2013 10:47 PM, Matt Turner wrote:
---
src/glsl/builtins/tools/generate_builtins.py |1 +
src/glsl/glcpp/glcpp-parse.y |3 +++
src/glsl/glsl_parser_extras.cpp |1 +
src/glsl/glsl_parser_extras.h|2 ++
https://bugs.freedesktop.org/show_bug.cgi?id=59851
Priority: medium
Bug ID: 59851
Assignee: mesa-dev@lists.freedesktop.org
Summary: AC_ARG_WITH misusage leading to mesa configure failure
Severity: normal
Classification: Unclassified
https://bugs.freedesktop.org/show_bug.cgi?id=59831
Johannes Obermayr johannesoberm...@gmx.de changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop.
On 01/24/2013 10:44 PM, Matt Turner wrote:
Following this email are eight patches that add the 4x8 pack/unpack
operations that are the difference between what GLSL ES 3.0 and
ARB_shading_language_packing require.
They require Chad's gles3-glsl-packing series and are available at
On 01/24/2013 10:47 PM, Matt Turner wrote:
Lower them to arithmetic and bit manipulation expressions.
---
src/glsl/ir_optimization.h |6 +
src/glsl/lower_packing_builtins.cpp | 279 +++
2 files changed, 285 insertions(+), 0 deletions(-)
diff
- Rename draw_x/y to tile_x/y in dri image struct. These are now used as
adjustment pixels from our stored aligned offset to the exported image
instead of the entire x/y offset from the base address.
- Take into consideration the offset from our bo so that sub-image functions
resolves
Add create image from texture extension and bump version.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
include/GL/internal/dri_interface.h | 14 +-
src/egl/drivers/dri2/egl_dri2.c | 85 +++
2 files changed, 98 insertions(+), 1
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/intel/intel_regions.h |6 ++
1 file changed, 6 insertions(+)
diff --git a/src/mesa/drivers/dri/intel/intel_regions.h
b/src/mesa/drivers/dri/intel/intel_regions.h
index 8737a6d..1eef3b5 100644
---
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 37
src/mesa/drivers/dri/intel/intel_mipmap_tree.h | 14 -
2 files changed, 31 insertions(+), 20 deletions(-)
diff --git
Add helper to calculate fine-grained x and y adjustment pixels
to an image within a miptree level for tiled regions.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 15 +++
If the offsets are present, this lets us specify a particular level and slice
in a shared region using the base level of an exported mip-map tree.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 12 +++-
We need to take account the offset from original bo when using glTexSubImage()
and other functions that manipulate the subregion of an exported texture.
Offsets are appended to mapped region address and when blitting from a source
region.
Signed-off-by: Abdiel Janulgue
Save miptree level info to DRIImage:
- Appropriately-aligned base offset pointing to the image
- Additional x/y adjustment offsets from above.
In non-tile-aligned surface cases where resolving back to the original image
located in mip-levels higher than the base level proves problematic due to
When binding a region to a texture image, re-create the miptree base-level
considering the offset and dimension information exported by DRIImage.
Signed-off-by: Abdiel Janulgue abdiel.janul...@linux.intel.com
---
src/mesa/drivers/dri/intel/intel_tex_image.c | 31 --
1
On 01/25/2013 08:54 AM, Christoph Bumiller wrote:
v2: Record texObj.BufferSize as -1 in TexBuffer(non-Range) instead
of the buffer's current size so we know we always have to use the
full size of the buffer object (i.e. even if it changes without the
user calling TexBuffer again) for the
On 24 January 2013 19:47, Matt Turner matts...@gmail.com wrote:
That is, evaluate constant expressions for the following functions:
packSnorm4x8, unpackSnorm4x8
packUnorm4x8, unpackUnorm4x8
---
src/glsl/ir_constant_expression.cpp | 162
+++
1 files
From: Tom Stellard thomas.stell...@amd.com
We were using the NEED_RADEON_GALLIUM conditional to decide whether or not
to build llvm_wrapper.cpp, which is required for using the LLVM backend.
llvm_wrapper.cpp needs to be linked against the LLVM IPO libary
and this library is only added to
From: Tom Stellard thomas.stell...@amd.com
This is required when LLVM is built with CMake, which creates one
shared library for each component.
---
configure.ac | 18 --
1 file changed, 8 insertions(+), 10 deletions(-)
diff --git a/configure.ac b/configure.ac
index
On 24 January 2013 19:47, Matt Turner matts...@gmail.com wrote:
Lower them to arithmetic and bit manipulation expressions.
---
src/glsl/ir_optimization.h |6 +
src/glsl/lower_packing_builtins.cpp | 279
+++
2 files changed, 285 insertions(+), 0
https://bugs.freedesktop.org/show_bug.cgi?id=59831
--- Comment #6 from Tom Stellard tstel...@gmail.com ---
This should be fixed by this patch:
http://lists.freedesktop.org/archives/mesa-dev/2013-January/033482.html
--
You are receiving this mail because:
You are the QA Contact for the bug.
On 24 January 2013 19:44, Matt Turner matts...@gmail.com wrote:
Following this email are eight patches that add the 4x8 pack/unpack
operations that are the difference between what GLSL ES 3.0 and
ARB_shading_language_packing require.
They require Chad's gles3-glsl-packing series and are
https://bugs.freedesktop.org/show_bug.cgi?id=59831
--- Comment #7 from Tom Stellard tstel...@gmail.com ---
(In reply to comment #5)
It was false to remove libr600_la_LDFLAGS in this patch:
http://cgit.freedesktop.org/mesa/mesa/commit/
?id=69d639ba8b3cfd95cfbb12b861dbe2eda53f2e25
And please
---
lib/Target/R600/R600Instructions.td | 9 +++--
lib/Target/R600/R600Intrinsics.td | 4 ++--
2 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/lib/Target/R600/R600Instructions.td
b/lib/Target/R600/R600Instructions.td
index 13293b6..3537906 100644
---
Fixes: https://bugs.freedesktop.org/show_bug.cgi?id=59588
---
src/gallium/drivers/r600/r600_llvm.c | 22 --
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/r600/r600_llvm.c
b/src/gallium/drivers/r600/r600_llvm.c
index 32b8e56..913dccc 100644
On Fri, Jan 25, 2013 at 10:43 AM, Tom Stellard t...@stellard.net wrote:
From: Tom Stellard thomas.stell...@amd.com
We were using the NEED_RADEON_GALLIUM conditional to decide whether or not
to build llvm_wrapper.cpp, which is required for using the LLVM backend.
llvm_wrapper.cpp needs to be
v2: Record texObj.BufferSize as -1 in TexBuffer(non-Range) instead
of the buffer's current size so we know we always have to use the
full size of the buffer object (i.e. even if it changes without the
user calling TexBuffer again) for the texture.
Clarify invalid offset alignment error message.
From: Jerome Glisse jgli...@redhat.com
Signed-off-by: Jerome Glisse jgli...@redhat.com
---
src/gallium/drivers/r600/evergreen_compute.c | 2 ++
src/gallium/drivers/r600/r600_hw_context.c | 1 +
src/gallium/drivers/r600/r600_pipe.c | 6 ++
src/gallium/drivers/r600/r600_pipe.h
Tapani Pälli tapani.pa...@intel.com writes:
commit 60894edeef973e86a73067276f658b72f84271b6 changed the way dri2
buffer pitch is interpreted in intel driver createImageFromName
implementation, caller must set pitch in bytes, not pixels.
Oops, I didn't mean to change behavior of the interface.
Kenneth Graunke kenn...@whitecape.org writes:
On 01/20/2013 02:59 PM, Eric Anholt wrote:
Before, we were keeping a CPU-only buffer to accumulate the batchbuffer in,
which was an improvement over mapping the batch through the GTT directly
(since any readback or other failure to stream through
To silence warnings about unhandled cases.
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 14 --
1 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 643a9bb..2c5ba41 100644
So design is mostly the same then previously. Few changes, first i use only
one thread to offload all cs submission wether gfx or dma. Reasons is that
using on thread for gfx and one for dma lead to more complex synchronization
with no gain ie when submitting gfx you would need to make sure
From: Jerome Glisse jgli...@redhat.com
Add ring support, you can create a cs for each ring. DMA ring is
bit special regarding relocation as you must emit as much relocation
as there is use of the buffer.
v2: - Improved comment on relocation changes
- Use a single thread to queue cs
From: Jerome Glisse jgli...@redhat.com
Signed-off-by: Jerome Glisse jgli...@redhat.com
---
src/gallium/drivers/r600/evergreen_hw_context.c | 44 ++
src/gallium/drivers/r600/evergreen_state.c | 197
src/gallium/drivers/r600/evergreend.h | 15 ++
On 25 January 2013 07:49, Paul Berry stereotype...@gmail.com wrote:
On 24 January 2013 19:44, Matt Turner matts...@gmail.com wrote:
Following this email are eight patches that add the 4x8 pack/unpack
operations that are the difference between what GLSL ES 3.0 and
ARB_shading_language_packing
From: Tom Stellard thomas.stell...@amd.com
---
configure.ac | 35 +++
1 file changed, 19 insertions(+), 16 deletions(-)
diff --git a/configure.ac b/configure.ac
index ccf95c5..9cc5c4a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1898,21 +1898,23 @@ dnl by
On 01/25/2013 05:43 AM, Ian Romanick wrote:
On 01/24/2013 08:40 PM, Kenneth Graunke wrote:
On 01/22/2013 12:52 AM, Ian Romanick wrote:
From: Ian Romanick ian.d.roman...@intel.com
Signed-off-by: Ian Romanick ian.d.roman...@intel.com
---
src/glsl/glsl_types.cpp | 12 +++-
On 01/25/2013 11:59 AM, Chad Versace wrote:
On 01/24/2013 07:47 PM, Matt Turner wrote:
Lower them to arithmetic and bit manipulation expressions.
---
src/glsl/ir_optimization.h |6 +
src/glsl/lower_packing_builtins.cpp | 279
+++
2 files
On 01/25/2013 11:38 AM, Chad Versace wrote:
On 01/24/2013 07:47 PM, Matt Turner wrote:
That is, evaluate constant expressions for the following functions:
packSnorm4x8, unpackSnorm4x8
packUnorm4x8, unpackUnorm4x8
---
src/glsl/ir_constant_expression.cpp | 162
Patches 1-3, 6-8 are
Reviewed-by: Chad Versace chad.vers...@linux.intel.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 01/24/2013 07:44 PM, Matt Turner wrote:
Following this email are eight patches that add the 4x8 pack/unpack
operations that are the difference between what GLSL ES 3.0 and
ARB_shading_language_packing require.
They require Chad's gles3-glsl-packing series and are available at
On 01/25/2013 09:55 AM, Paul Berry wrote:
On 25 January 2013 07:49, Paul Berry stereotype...@gmail.com wrote:
On 24 January 2013 19:44, Matt Turner matts...@gmail.com wrote:
Following this email are eight patches that add the 4x8 pack/unpack
operations that are the difference between what
https://bugs.freedesktop.org/show_bug.cgi?id=59835
--- Comment #2 from Chad Versace chad.vers...@linux.intel.com ---
Sorry about that. Next time I change the Android and Autotools system, I'll
remember to change Scons too.
--
You are receiving this mail because:
You are the assignee for the
On 01/24/2013 06:59 PM, Zack Rusin wrote:
Only drivers supporting DRI2 version=4 support GLX_INTEL_swap_event.
So lets mark it as such otherwise applications which use this extension
(i.e. everything based on Clutter, e.g. gnome-shell) break horribly on
drivers supporting DRI2 versions only up
On 01/24/2013 07:48 PM, srol...@vmware.com wrote:
From: Roland Scheideggersrol...@vmware.com
Split the sampler interface to use separate sampler and texture (sampler_view)
state. This is needed to support dx10-style sampling instructions.
This is not quite complete since both draw/llvmpipe
On Fri, Jan 25, 2013 at 9:55 AM, Paul Berry stereotype...@gmail.com wrote:
On 25 January 2013 07:49, Paul Berry stereotype...@gmail.com wrote:
On 24 January 2013 19:44, Matt Turner matts...@gmail.com wrote:
Following this email are eight patches that add the 4x8 pack/unpack
operations that
On Fri, Jan 25, 2013 at 11:59 AM, Chad Versace
chad.vers...@linux.intel.com wrote:
+ *x = unpack_1x8((uint8_t) (u 0xff));
+ *y = unpack_1x8((uint8_t) (u 8));
+ *z = unpack_1x8((uint8_t) (u 16));
+ *w = unpack_1x8((uint8_t) (u 24));
+}
The bitmask (u 0xff) confused me for a few
On 25 January 2013 13:18, Matt Turner matts...@gmail.com wrote:
On Fri, Jan 25, 2013 at 9:55 AM, Paul Berry stereotype...@gmail.com
wrote:
On 25 January 2013 07:49, Paul Berry stereotype...@gmail.com wrote:
On 24 January 2013 19:44, Matt Turner matts...@gmail.com wrote:
Following this
On 01/25/2013 09:31 AM, Eric Anholt wrote:
Kenneth Graunke kenn...@whitecape.org writes:
On 01/20/2013 02:59 PM, Eric Anholt wrote:
Before, we were keeping a CPU-only buffer to accumulate the batchbuffer in,
which was an improvement over mapping the batch through the GTT directly
(since any
From: Jerome Glisse jgli...@redhat.com
v2: Add virtual address to dma src/dst offset for cayman
Signed-off-by: Jerome Glisse jgli...@redhat.com
---
src/gallium/drivers/r600/evergreen_hw_context.c | 46 ++
src/gallium/drivers/r600/evergreen_state.c | 201
On Thu, Jan 24, 2013 at 7:44 PM, Matt Turner matts...@gmail.com wrote:
Following this email are eight patches that add the 4x8 pack/unpack
operations that are the difference between what GLSL ES 3.0 and
ARB_shading_language_packing require.
They require Chad's gles3-glsl-packing series and
https://bugs.freedesktop.org/show_bug.cgi?id=59851
Matt Turner matts...@gmail.com changed:
What|Removed |Added
CC||tstel...@gmail.com
--
I did not list the *_get_program_binary extensions since they're not
useful to anyone with their current implementation (that supports 0
binary formats).
---
We should also write something about ES3 and the float-texture S3TC
changes.
docs/relnotes-9.1.html | 12 +++-
1 files changed,
I'm quite sure there are g965 boards around which indeed support Pentium
4 (and P4-based Celerons) (but yes I guess cmov and at least sse2 are
safe - not that the p4 had a usable cmov implementation as it was
incredibly slow IIRC but it should at least work).
Roland
Am 25.01.2013 04:33, schrieb
---
lib/Target/R600/AMDGPUTargetMachine.cpp | 2 +-
lib/Target/R600/R600LowerConstCopy.cpp | 170 +---
2 files changed, 160 insertions(+), 12 deletions(-)
diff --git a/lib/Target/R600/AMDGPUTargetMachine.cpp
b/lib/Target/R600/AMDGPUTargetMachine.cpp
index
When trying glxgears the screen locks up, and SSH eventually stops
responding as well, but I was able to get these messages from kern.log:
[ 790.516059] radeon :01:05.0: GPU lockup CP stall for more than
1msec
[ 790.516076] radeon :01:05.0: GPU lockup (waiting for
0x215b
From: Alex Deucher alexander.deuc...@amd.com
Need to add the virtual address.
Signed-off-by: Alex Deucher alexander.deuc...@amd.com
---
src/gallium/drivers/r600/r600.h|4 ++--
src/gallium/drivers/r600/r600_hw_context.c | 11 +++
2 files changed, 9 insertions(+), 6
Contains a fix for Khronos bug 9557.
---
include/GLES3/gl3.h |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/GLES3/gl3.h b/include/GLES3/gl3.h
index b9399e9..09f2b53 100644
--- a/include/GLES3/gl3.h
+++ b/include/GLES3/gl3.h
@@ -2,7 +2,7 @@
#define __gl3_h_
On Thu, Jan 24, 2013 at 7:33 PM, Eric Anholt e...@anholt.net wrote:
While most of our development and testing is on x86-64, some of our
major consumers of the driver are on i386 still. This meant they aren't
taking advantage of SSE for floating point math or cmov instructions,
unless the user
https://bugs.freedesktop.org/show_bug.cgi?id=59872
Priority: medium
Bug ID: 59872
Keywords: regression
CC: cwo...@cworth.org
Assignee: mesa-dev@lists.freedesktop.org
Summary: [swrast] piglit depth_texture_mode_and_swizzle
From: Roland Scheidegger srol...@vmware.com
Make it obvious what unit this is (no change in functionality).
draw still uses unit in places where it changes the shader by adding
texture sampling itself - it seems like this can't work with shaders
using dx10-style sample opcodes (can't mix gl-style
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com
On Fri, Jan 25, 2013 at 4:07 PM, Matt Turner matts...@gmail.com wrote:
Contains a fix for Khronos bug 9557.
---
include/GLES3/gl3.h |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/GLES3/gl3.h
https://bugs.freedesktop.org/show_bug.cgi?id=59873
Priority: medium
Bug ID: 59873
Keywords: regression
CC: bri...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [swrast] piglit
https://bugs.freedesktop.org/show_bug.cgi?id=59876
Priority: medium
Bug ID: 59876
Assignee: mesa-dev@lists.freedesktop.org
Summary: glGetTexLevelParameteriv broken for indirect rendering
Severity: normal
Classification: Unclassified
https://bugs.freedesktop.org/show_bug.cgi?id=59877
Priority: medium
Bug ID: 59877
Assignee: mesa-dev@lists.freedesktop.org
Summary: Build fail since r600g: Don't build llvm_wrapper.cpp
when we aren't using LLVM
Severity:
https://bugs.freedesktop.org/show_bug.cgi?id=59879
Priority: medium
Bug ID: 59879
Assignee: mesa-dev@lists.freedesktop.org
Summary: reducing symbol visibility of shared objects / static
libstdc++
Severity: normal
https://bugs.freedesktop.org/show_bug.cgi?id=59877
--- Comment #1 from Tom Stellard tstel...@gmail.com ---
Created attachment 73664
-- https://bugs.freedesktop.org/attachment.cgi?id=73664action=edit
Possible fix
Does this patch help?
--
You are receiving this mail because:
You are the
The packet length may change at some point in the future. Specifying it
explicitly (rather than hardcoding it in the command #define) allows us
to change it much more easily in the future.
Signed-off-by: Kenneth Graunke kenn...@whitecape.org
---
src/mesa/drivers/dri/intel/intel_blit.c | 8
Roland Scheidegger srol...@vmware.com writes:
I'm quite sure there are g965 boards around which indeed support Pentium
4 (and P4-based Celerons) (but yes I guess cmov and at least sse2 are
safe - not that the p4 had a usable cmov implementation as it was
incredibly slow IIRC but it should at
On 01/25/2013 03:13 PM, Roland Scheidegger wrote:
I'm quite sure there are g965 boards around which indeed support Pentium
4 (and P4-based Celerons) (but yes I guess cmov and at least sse2 are
safe - not that the p4 had a usable cmov implementation as it was
incredibly slow IIRC but it should at
Fixes side effect in assertion defects reported by Coverity.
Signed-off-by: Vinson Lee v...@freedesktop.org
---
src/mesa/drivers/dri/i965/brw_fs_emit.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_emit.cpp
80 matches
Mail list logo