Hi Mauro,
On Mon, Apr 3, 2017 at 2:48 AM, Mauro Rossi wrote:
>
>
> 2017-03-31 13:05 GMT+02:00 Tapani Pälli :
>>
>>
>>
>> On 03/31/2017 10:12 AM, Tapani Pälli wrote:
>>>
>>>
>>>
>>> On 03/31/2017 09:06 AM, Tapani Pälli wrote:
On 03/31/2017 08:24 AM, Rob Clark wrote:
>
>
Am 03.04.2017 um 06:35 schrieb Shaleen Jain:
---
src/gallium/state_trackers/omx/vid_dec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/omx/vid_dec.c
b/src/gallium/state_trackers/omx/vid_dec.c
index 9a6efb8e28..94664eba04 100644
--- a/src/galli
On 31.03.2017 20:14, Brian Paul wrote:
On 03/31/2017 12:01 AM, Nicolai Hähnle wrote:
On 31.03.2017 05:21, Brian Paul wrote:
To avoid/fix a segmentation fault when running the stand-alone GLSL
compiler utility for cases such as the Mesa demos toyball test:
glsl_compiler --dump-builder --version
On 03/31/2017 04:04 PM, Rob Clark wrote:
On Fri, Mar 31, 2017 at 2:06 AM, Tapani Pälli wrote:
On 03/31/2017 08:24 AM, Rob Clark wrote:
On Fri, Mar 31, 2017 at 12:22 AM, Tapani Pälli
wrote:
On 03/30/2017 05:57 PM, Emil Velikov wrote:
On 30 March 2017 at 15:30, Tomasz Figa wrote:
On 01.04.2017 12:48, Mauro Rossi wrote:
Fixes the following build error:
external/mesa/src/amd/addrlib/gfx9/gfx9addrlib.cpp:36:10: fatal error:
'gfx9_gb_reg.h' file not found
^
1 error generated.
Fixes: 7f160ef "amd/addrlib: import gfx9 support"
Reviewed-by: Nicolai Hähnle
---
s
Some of these are a bit subtle, but I think they're fine. Series is:
Reviewed-by: Nicolai Hähnle
On 02.04.2017 20:00, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/auxiliary/indices/u_primconvert.c | 10 --
src/gallium/include/pipe/p_state.h| 2 +-
2 files change
On 02.04.2017 18:02, Ilia Mirkin wrote:
Signed-off-by: Ilia Mirkin
Reviewed-by: Nicolai Hähnle
---
src/gallium/docs/source/tgsi.rst | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/src/gallium/docs/source/tgsi.rst b/src/gallium/docs/sou
On 03.04.2017 02:01, Marek Olšák wrote:
From: Marek Olšák
Don't clear state[7] on SI-CIK, and only do the meta stuff on VI+.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100531
Makes sense. Please add
Fixes: 5abf60076ce4 ("radeonsi/gfx9: image descriptor changes in mutable
fields
Hi Jason,
On 1 April 2017 at 06:37, Jason Ekstrand wrote:
> @@ -594,7 +595,19 @@ wsi_wl_swapchain_queue_present(struct wsi_swapchain
> *wsi_chain,
>
> assert(image_index < chain->base.image_count);
> wl_surface_attach(chain->surface, chain->images[image_index].buffer, 0,
> 0);
> - wl_
On 03.04.2017 02:01, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 108 ++-
1 file changed, 48 insertions(+), 60 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
ind
On Mon, Apr 3, 2017 at 11:00 AM, Nicolai Hähnle wrote:
> On 03.04.2017 02:01, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> ---
>> src/gallium/drivers/radeonsi/si_shader.c | 108
>> ++-
>> 1 file changed, 48 insertions(+), 60 deletions(-)
>>
>> diff --git a/src/gal
On 04/01/2017 09:11 AM, Nicolai Hähnle wrote:
On 31.03.2017 16:16, Samuel Pitoiset wrote:
This will be used by bindless to initialize the descriptor for
both samplers and images.
Signed-off-by: Samuel Pitoiset
I'd prefer to see these patches in a larger context, to be honest.
As for this
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 88 ++
.../drivers/radeonsi/si_shader_tgsi_setup.c| 14 ++--
2 files changed, 47 insertions(+), 55 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/rad
From: Marek Olšák
even vec_type is f32.
---
src/gallium/drivers/radeonsi/si_shader.c | 28 ++
.../drivers/radeonsi/si_shader_tgsi_setup.c| 16 ++---
2 files changed, 21 insertions(+), 23 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shad
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 116 ++---
src/gallium/drivers/radeonsi/si_shader_tgsi_alu.c | 4 +-
.../drivers/radeonsi/si_shader_tgsi_setup.c| 46
3 files changed, 79 insertions(+), 87 deletions(-)
diff --git a/s
Yes, that is correct. I wasn't thinking clearly.
On Mon, Apr 3, 2017 at 1:57 PM Christian König
wrote:
> Am 03.04.2017 um 06:35 schrieb Shaleen Jain:
> > ---
> > src/gallium/state_trackers/omx/vid_dec.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/gallium
From: Marek Olšák
It silences the following radeonsi LLVM warning due to a previous
commit adding an LLVM workaround:
"mesa: for the -simplifycfg-sink-common option: may only occur zero or one
times!"
Cc: 17.0
---
src/gallium/targets/omx/omx.sym | 5 +
src/gallium/targets/pip
I've changed the commit message to "targets: export radeon
winsys_create functions to silence LLVM warning".
Marek
On Mon, Apr 3, 2017 at 12:08 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> It silences the following radeonsi LLVM warning due to a previous
> commit adding an LLVM workaround:
>
For the series:
Reviewed-by: Marek Olšák
Marek
On Sun, Apr 2, 2017 at 7:33 PM, Constantine Kharlamov
wrote:
> Specifically, non-line primitives skipped, and defaulting to reset on
> each packet.
>
> The skip of non-line primitives saves ≈110 resetting of
> PA_SC_LINE_STIPPLE register per frame
Am 03.04.2017 um 12:08 schrieb Marek Olšák:
From: Marek Olšák
It silences the following radeonsi LLVM warning due to a previous
commit adding an LLVM workaround:
"mesa: for the -simplifycfg-sink-common option: may only occur zero or one
times!"
Cc: 17.0
Reviewed-by: Christian König
https://bugs.freedesktop.org/show_bug.cgi?id=100402
Vedran Miletić changed:
What|Removed |Added
CC||ved...@miletic.net
--
You are receivi
v2 (Andreas Boll):
- Mark GL 4.1 as supported by i965/gen7+
- Mark GL_ARB_shader_precision as supported by i965/gen7+
- Update release notes
---
docs/features.txt | 4 ++--
docs/relnotes/17.1.0.html | 3 +++
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/docs/features.txt b
On Wed, 2017-03-29 at 12:06 -0700, Jason Ekstrand wrote:
> Looking over the patch, I think I've convinced myself that it's correct. (I
> honestly wasn't expecting to come to that conclusion without more iteration.)
> That said, this raises some interesting questions. I added Kristian to the
>
I need to test this series on Windows and with MinGW first. I'm worried
about enums with bitfields.
-Brian
On Mon, Apr 3, 2017 at 2:46 AM, Nicolai Hähnle wrote:
> Some of these are a bit subtle, but I think they're fine. Series is:
>
> Reviewed-by: Nicolai Hähnle
>
>
> On 02.04.2017 20:00, M
On Mon, Apr 3, 2017 at 5:02 AM, Juan A. Suarez Romero
wrote:
> On Wed, 2017-03-29 at 12:06 -0700, Jason Ekstrand wrote:
> > Looking over the patch, I think I've convinced myself that it's
> correct. (I honestly wasn't expecting to come to that conclusion without
> more iteration.) That said, th
On Mon, Apr 3, 2017 at 1:54 AM, Daniel Stone wrote:
> Hi Jason,
>
> On 1 April 2017 at 06:37, Jason Ekstrand wrote:
> > @@ -594,7 +595,19 @@ wsi_wl_swapchain_queue_present(struct
> wsi_swapchain *wsi_chain,
> >
> > assert(image_index < chain->base.image_count);
> > wl_surface_attach(chai
Sorry. I thought I had.
Reviewed-by: Jason Ekstrand
On Sun, Apr 2, 2017 at 11:02 PM, Iago Toral wrote:
> Can anyone review this one?
>
> On Wed, 2017-03-29 at 08:58 +0200, Iago Toral Quiroga wrote:
> > Writing and testing are two different things and they can be set
> > separately
> > by the
Hi,
On 3 April 2017 at 15:45, Jason Ekstrand wrote:
> On Mon, Apr 3, 2017 at 1:54 AM, Daniel Stone wrote:
>> Very scrupulous version check, but you forgot to actually use
>> wl_surface_damage_buffer. ;)
>
> Gah!!! I'll get that fixed. Assuming that change, now that you've looked
> at it all, w
On 04/02/2017 12:00 PM, Marek Olšák wrote:
From: Marek Olšák
Also:
pipe_transfer: 48 -> 40 bytes.
pipe_blit_info = 176 -> 160 bytes.
---
src/gallium/include/pipe/p_state.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/include/pipe/p_state.h
b/src/g
On Sun, Apr 2, 2017 at 2:00 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> Also:
>
> pipe_transfer: 48 -> 40 bytes.
> pipe_blit_info = 176 -> 160 bytes.
> ---
> src/gallium/include/pipe/p_state.h | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/gallium/include
On Mon, Apr 03, 2017 at 08:02:54AM +0200, Iago Toral wrote:
> Can anyone review this one?
>
> On Wed, 2017-03-29 at 08:58 +0200, Iago Toral Quiroga wrote:
> > Writing and testing are two different things and they can be set
> > separately
> > by the application. If an application wants to record d
From: Karol Herbst
With the shader cache, compilation time matters less.
As a side effect we can write more optimizations to produce better optimized
code.
total instructions in shared programs : 3931743 -> 3917512 (-0.36%)
total gprs used in shared programs: 481460 -> 481680 (0.05%)
total
From: Karol Herbst
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.
Slowly we are getting to the point, that we miss enough optimization
opportunities as the result of our own passes.
For this we need to fix AlgebraicOpt to be able to handle mods on sources
without creating new issues.
The last patch enables looping opts.
Karol Herbst (3):
nv50/ir: fix Algebra
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
index bd60a84998..0de84fe9fc 1006
Slowly we are getting to the point, that we miss enough optimization
opportunities as the result of our own passes.
For this we need to fix AlgebraicOpt to be able to handle mods on sources
without creating new issues.
The last patch enables looping opts.
v2: update commit author
Karol Herbst (
With the shader cache, compilation time matters less.
As a side effect we can write more optimizations to produce better optimized
code.
total instructions in shared programs : 3931743 -> 3917512 (-0.36%)
total gprs used in shared programs: 481460 -> 481680 (0.05%)
total local used in shared
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
index bd60a84998..0de84fe9fc 1006
Signed-off-by: Karol Herbst
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
index 4c92a1efb5.
On Wed, Mar 22, 2017 at 03:13:56PM -0700, Jason Ekstrand wrote:
> That pass hasn't existed since dd4db84640bbb694f180dd50850c3388f67228be
> but the prototype stuck around for no reason.
> ---
> src/intel/vulkan/anv_nir.h | 3 ---
> 1 file changed, 3 deletions(-)
>
This patch is
Reviewed-by: Nanl
Otherwise, we were getting the definition for 'inline' by chance from
some other preceeding #include.
---
src/util/list.h | 1 +
src/util/mesa-sha1.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/util/list.h b/src/util/list.h
index 07eb9f3..6edb750 100644
--- a/src/util/list.h
+++
To follow the convention of other header include guards.
---
src/util/mesa-sha1.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/util/mesa-sha1.h b/src/util/mesa-sha1.h
index a81aba9..d3f7aff 100644
--- a/src/util/mesa-sha1.h
+++ b/src/util/mesa-sha1.h
@@ -20,8 +20,8 @
To silence
C:\Users\Brian\projects\mesa\src\util/u_vector.h(41) : warning C4146: unary
minus operator applied to unsigned type, result still unsigned
---
src/util/u_vector.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/util/u_vector.h b/src/util/u_vector.h
index f97a8b4.
On Mon, Apr 3, 2017 at 8:55 AM, Nanley Chery wrote:
> On Mon, Apr 03, 2017 at 08:02:54AM +0200, Iago Toral wrote:
> > Can anyone review this one?
> >
> > On Wed, 2017-03-29 at 08:58 +0200, Iago Toral Quiroga wrote:
> > > Writing and testing are two different things and they can be set
> > > separ
On 03/04/17 17:09, Brian Paul wrote:
Otherwise, we were getting the definition for 'inline' by chance from
some other preceeding #include.
---
src/util/list.h | 1 +
src/util/mesa-sha1.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/util/list.h b/src/util/list.h
index 07eb9f3..6
On Apr 3, 2017 4:04 PM, "Brian Paul" wrote:
I need to test this series on Windows and with MinGW first. I'm worried
about enums with bitfields.
Hopefully it'll work. Enums without bitfields always occupy 4 bytes. If
bitfields don't work with those, we'll have to stop using enums in these
cases
https://bugs.freedesktop.org/show_bug.cgi?id=100259
--- Comment #14 from Emil Velikov ---
Ouch that is some nasty bug in Slackware packaging.
Note you want /usr/local/ and /usr/ in the same order across
--with-pkg-config-dir and --with-system-libdir.
--
You are receiving this mail because:
You
On Wed, Mar 29, 2017 at 12:06 PM, Jason Ekstrand wrote:
> Looking over the patch, I think I've convinced myself that it's correct. (I
> honestly wasn't expecting to come to that conclusion without more
> iteration.) That said, this raises some interesting questions. I added
> Kristian to the Cc
On Apr 3, 2017 5:11 PM, "Alex Deucher" wrote:
On Sun, Apr 2, 2017 at 2:00 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> Also:
>
> pipe_transfer: 48 -> 40 bytes.
> pipe_blit_info = 176 -> 160 bytes.
> ---
> src/gallium/include/pipe/p_state.h | 8
> 1 file changed, 4 insertions(+), 4 d
Replace "nore" by "more".
---
bin/get-fixes-pick-list.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/bin/get-fixes-pick-list.sh b/bin/get-fixes-pick-list.sh
index 59bcae4..75242a2 100755
--- a/bin/get-fixes-pick-list.sh
+++ b/bin/get-fixes-pick-list.sh
@@ -27,7 +27,7 @@ do
On Mon, Apr 3, 2017 at 9:40 AM, Kristian Høgsberg
wrote:
> On Wed, Mar 29, 2017 at 12:06 PM, Jason Ekstrand
> wrote:
> > Looking over the patch, I think I've convinced myself that it's
> correct. (I
> > honestly wasn't expecting to come to that conclusion without more
> > iteration.) That said
On Mon, Apr 3, 2017 at 7:44 AM, Jason Ekstrand wrote:
> On Mon, Apr 3, 2017 at 5:02 AM, Juan A. Suarez Romero > wrote:
>
>> On Wed, 2017-03-29 at 12:06 -0700, Jason Ekstrand wrote:
>> > Looking over the patch, I think I've convinced myself that it's
>> correct. (I honestly wasn't expecting to c
Hi, Rob,
On 03/24/2017 10:21 PM, Rob Clark wrote:
> It's kinda sad that (a) we don't have debug_backtrace support on !X86
> and that (b) we re-invent our own crude backtrace support in the first
> place. If available, use libunwind instead. The backtrace format is
> based on what xserver and wes
On Mon, Apr 3, 2017 at 12:56 PM, Thomas Hellstrom wrote:
> Hi, Rob,
>
> On 03/24/2017 10:21 PM, Rob Clark wrote:
>> It's kinda sad that (a) we don't have debug_backtrace support on !X86
>> and that (b) we re-invent our own crude backtrace support in the first
>> place. If available, use libunwind
On Mon, Apr 3, 2017 at 10:13 AM, Rob Clark wrote:
> On Mon, Apr 3, 2017 at 12:56 PM, Thomas Hellstrom
> wrote:
>> Hi, Rob,
>>
>> On 03/24/2017 10:21 PM, Rob Clark wrote:
>>> It's kinda sad that (a) we don't have debug_backtrace support on !X86
>>> and that (b) we re-invent our own crude backtrac
On 04/03/2017 07:13 PM, Rob Clark wrote:
> On Mon, Apr 3, 2017 at 12:56 PM, Thomas Hellstrom
> wrote:
>> Hi, Rob,
>>
>> On 03/24/2017 10:21 PM, Rob Clark wrote:
>>> It's kinda sad that (a) we don't have debug_backtrace support on !X86
>>> and that (b) we re-invent our own crude backtrace support
---
src/compiler/glsl/blob.h | 11 +++
src/compiler/glsl/shader_cache.cpp | 2 +-
src/compiler/glsl/tests/blob_test.c | 8
src/mesa/state_tracker/st_shader_cache.c | 2 +-
4 files changed, 17 insertions(+), 6 deletions(-)
diff --git a/src/compiler/gl
We supported more generally. Decreased the dynamic buffers though, as
we only support 16 for uniform+storage.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_device.c | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
diff --git a/src/amd/vulkan/radv_d
On Thu, Mar 30, 2017 at 3:47 PM, Matt Turner wrote:
> On Thu, Mar 30, 2017 at 3:26 PM, Grazvydas Ignotas
wrote:
>> There are still some distributions trying to support unfortunate people
>> with old or exotic CPUs that don't have 64bit atomic operations. When
>> compiling for such a machine, gcc
Reviewed-by: Bas Nieuwenhuizen
On Mon, Apr 3, 2017 at 7:17 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This fixes an old bug that seems to get triggered by
> dEQP-VK.memory.requirements.image.sparse_tiling_optimal
>
> We return early when allocating S8_UINT due to there being
> no color or d
On 3 April 2017 at 17:09, Brian Paul wrote:
> To silence
> C:\Users\Brian\projects\mesa\src\util/u_vector.h(41) : warning C4146: unary
> minus operator applied to unsigned type, result still unsigned
For the series:
Reviewed-by: Emil Velikov
-Emil
___
Reviewed-by: Emil Velikov
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
---
src/mesa/main/glthread.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/glthread.c b/src/mesa/main/glthread.c
index 3f07c420d4..aa14292e59 100644
--- a/src/mesa/main/glthread.c
+++ b/src/mesa/main/glthread.c
@@ -53,7 +53,8 @@ glthread_allocate
Both are
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 03.04.2017 20:38, Bartosz Tomczyk wrote:
---
src/mesa/main/glthread.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/glthread.c b/src/mesa/main/glthread.c
index 3f07c420d4..aa14292e59 100644
--- a/src/mesa/main/glthread.c
+++ b/src/mesa/main/
Hi Rob,
On 24 March 2017 at 21:21, Rob Clark wrote:
> It's kinda sad that (a) we don't have debug_backtrace support on !X86
> and that (b) we re-invent our own crude backtrace support in the first
> place. If available, use libunwind instead. The backtrace format is
> based on what xserver and
On 2 April 2017 at 21:48, Rhys Kidd wrote:
> Per comments by Travis-CI, the apt addon is only really needed for the
> container-based Precise builds, as they don't yet support Trusty on that
> platform.
>
> Mesa currently uses Trusty fully-virtualized environment (due to sudo:
> required).
>
> S
On 3 April 2017 at 19:52, Marek Olšák wrote:
> From: Marek Olšák
radv uses i32zero and i32one, it might be nice to be consistent, but I
don't mind which way.
Dave.
>
> ---
> src/gallium/drivers/radeonsi/si_shader.c | 88
> ++
> .../drivers/radeonsi/si_shader_tgsi
Actually, I can just set only batch->used to 0, but it seems to error
prone. When someone adds some fields to batch struct, it will be easy to
miss that it should be initialized in glthread_unmarshal_batch.
Anyway I can change it if you want to.
On Mon, Apr 3, 2017 at 8:42 PM, Nicolai Hähnle wro
From: Dave Airlie
Get rid of usage of SI.vs.load.input.
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 13 +
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 520e4cf..da38331 1006
On 03.04.2017 20:53, Bartosz Tomczyk wrote:
Actually, I can just set only batch->used to 0, but it seems to error
prone. When someone adds some fields to batch struct, it will be easy to
miss that it should be initialized in glthread_unmarshal_batch.
Better to have it fail early and loudly with
Address sanitizer reports lot of misaligned access:
SUMMARY: AddressSanitizer: undefined-behavior main/marshal.c:276:31 in
main/marshal.c:276:31: runtime error: load of misaligned address 0x631000104866
for type
'const GLuint' (aka 'const unsigned int'), which requires 4 byte alignment
0x631000104
Reviewed-by: Bas Nieuwenhuizen
On Mon, Apr 3, 2017 at 8:57 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> Get rid of usage of SI.vs.load.input.
>
> Signed-off-by: Dave Airlie
> ---
> src/amd/common/ac_nir_to_llvm.c | 13 +
> 1 file changed, 5 insertions(+), 8 deletions(-)
>
> diff
---
src/mesa/main/glthread.c | 15 +--
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/src/mesa/main/glthread.c b/src/mesa/main/glthread.c
index 3f07c420d4..c4d3f4a434 100644
--- a/src/mesa/main/glthread.c
+++ b/src/mesa/main/glthread.c
@@ -53,7 +53,8 @@ glthread_allocate
Series is:
Reviewed-by: Bas Nieuwenhuizen
On Mon, Apr 3, 2017 at 5:46 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This fixes:
> dEQP-VK.glsl.texture_functions.texture.samplercubearray*
>
> Signed-off-by: Dave Airlie
> ---
> src/amd/common/ac_nir_to_llvm.c | 6 +-
> 1 file changed, 5 i
On Fri 31 Mar 2017, Chad Versace wrote:
> On Wed 15 Mar 2017, Jason Ekstrand wrote:
> > This cache allows us to easily ensure that we have a unique anv_bo for
> > each gem handle. We'll need this in order to support multiple-import of
> > memory objects and semaphores.
> >
> > v2 (Jason Ekstrand)
Just enabling the driver-independent implementation that Jason did.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_device.c | 4
src/amd/vulkan/radv_entrypoints_gen.py | 1 +
src/amd/vulkan/radv_wsi.c | 11 ++-
3 files changed, 15 insertions(+), 1
Series is
Reviewed-by: Bas Nieuwenhuizen
Not really a fan of yet another flag in the command buffer, but not
sure what would be the best solution.
On Mon, Apr 3, 2017 at 5:44 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> The current code was broken, and I decided to redesign it instead.
>
>
On Mar 30, 2017 16:16, "Lionel Landwerlin"
wrote:
While exercising reading report with moderate load, we might have to
wait for all the reports to land in the OA buffer, otherwise we might
miss some reports. That means we need to keep on reading the OA stream
until the last report we read has a t
On 04/03/2017 07:33 PM, Thomas Hellstrom wrote:
> On 04/03/2017 07:13 PM, Rob Clark wrote:
>> On Mon, Apr 3, 2017 at 12:56 PM, Thomas Hellstrom
>> wrote:
>>> Hi, Rob,
>>>
>>> On 03/24/2017 10:21 PM, Rob Clark wrote:
It's kinda sad that (a) we don't have debug_backtrace support on !X86
a
This is pretty much the same tool as what i-g-t has, only with a more
fancy decoding of the instructions/registers. It also doesn't support
anything before gen4.
v2 (from Matt): Drop authors
Remove undefined automake variable
v3: Fix incorrect offsets for dword > 1 (Jordan)
Signe
On Mon, Apr 3, 2017 at 4:06 PM, Thomas Hellstrom wrote:
> On 04/03/2017 07:33 PM, Thomas Hellstrom wrote:
>> On 04/03/2017 07:13 PM, Rob Clark wrote:
>>> On Mon, Apr 3, 2017 at 12:56 PM, Thomas Hellstrom
>>> wrote:
Hi, Rob,
On 03/24/2017 10:21 PM, Rob Clark wrote:
> It's kinda
Reviewed-by: Jason Ekstrand
On April 3, 2017 12:36:16 PM Bas Nieuwenhuizen wrote:
Just enabling the driver-independent implementation that Jason did.
Signed-off-by: Bas Nieuwenhuizen
---
src/amd/vulkan/radv_device.c | 4
src/amd/vulkan/radv_entrypoints_gen.py | 1 +
src/a
This commit appears to intermittently provoke gpu hangs on 32-bit Intel
systems when running
piglit.shaders.glsl-max-varyings >max_varying_components
Since the behavior is intermittent, I may have identified the wrong
patch. Please let me know if this patch seems unlikely to affect the
test.
Ma
On Mon, Apr 3, 2017 at 4:06 PM, Thomas Hellstrom wrote:
> On 04/03/2017 07:33 PM, Thomas Hellstrom wrote:
>> On 04/03/2017 07:13 PM, Rob Clark wrote:
>>> On Mon, Apr 3, 2017 at 12:56 PM, Thomas Hellstrom
>>> wrote:
Hi, Rob,
On 03/24/2017 10:21 PM, Rob Clark wrote:
> It's kinda
On Mon, Apr 3, 2017 at 12:31 PM, Chad Versace
wrote:
> On Fri 31 Mar 2017, Chad Versace wrote:
> > On Wed 15 Mar 2017, Jason Ekstrand wrote:
> > > This cache allows us to easily ensure that we have a unique anv_bo for
> > > each gem handle. We'll need this in order to support multiple-import
> o
On Mon, Apr 3, 2017 at 4:57 PM, Rob Clark wrote:
> On Mon, Apr 3, 2017 at 4:06 PM, Thomas Hellstrom
> wrote:
>> On 04/03/2017 07:33 PM, Thomas Hellstrom wrote:
>>> On 04/03/2017 07:13 PM, Rob Clark wrote:
On Mon, Apr 3, 2017 at 12:56 PM, Thomas Hellstrom
wrote:
> Hi, Rob,
>
>
On Mon, Apr 3, 2017 at 1:37 PM, Mark Janes wrote:
>
> This commit appears to intermittently provoke gpu hangs on 32-bit Intel
> systems when running
>
> piglit.shaders.glsl-max-varyings >max_varying_components
>
> Since the behavior is intermittent, I may have identified the wrong
> patch. Please
Am 03.04.2017 um 17:11 schrieb Alex Deucher:
> On Sun, Apr 2, 2017 at 2:00 PM, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> Also:
>>
>> pipe_transfer: 48 -> 40 bytes.
>> pipe_blit_info = 176 -> 160 bytes.
>> ---
>> src/gallium/include/pipe/p_state.h | 8
>> 1 file changed, 4 insertions
On Mon 03 Apr 2017, Jason Ekstrand wrote:
> On Mon, Apr 3, 2017 at 12:31 PM, Chad Versace
> wrote:
>
> > On Fri 31 Mar 2017, Chad Versace wrote:
> > > On Wed 15 Mar 2017, Jason Ekstrand wrote:
> > > > This cache allows us to easily ensure that we have a unique anv_bo for
> > > > each gem handle.
Hi Tomasz,
2017-04-03 10:00 GMT+02:00 Tomasz Figa :
> Hi Mauro,
>
> On Mon, Apr 3, 2017 at 2:48 AM, Mauro Rossi wrote:
> >
> >
> > 2017-03-31 13:05 GMT+02:00 Tapani Pälli :
> >>
> >>
> >>
> >> On 03/31/2017 10:12 AM, Tapani Pälli wrote:
> >>>
> >>>
> >>>
> >>> On 03/31/2017 09:06 AM, Tapani Päll
Signed-off-by: Jordan Justen
---
src/intel/tools/aubinator.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/intel/tools/aubinator.c b/src/intel/tools/aubinator.c
index a64bce7a536..0d33c392384 100644
--- a/src/intel/tools/aubinator.c
+++ b/src/intel/tools/aubinator.c
@
From BDW PRM, Volume 6: Command Stream Programming, 'Render Command
Header Format'.
Signed-off-by: Jordan Justen
---
src/intel/common/gen_decoder.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/intel/common/gen_decoder.c b/src/intel/common/gen_decoder.c
in
Pushed, Thanks!
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Decoding with aubinator encountered a command of 0x. With the
previous code, it caused aubinator to jump 255 + 2 dwords to start
decoding again.
Instead we can attempt to detect the known instruction formats. If the
format is not recognized, then we can advance just 1 dword.
Signed-off-by
---
src/intel/compiler/brw_vec4_nir.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/src/intel/compiler/brw_vec4_nir.cpp
b/src/intel/compiler/brw_vec4_nir.cpp
index 2384265..613c695 100644
--- a/src/intel/compiler/brw_vec4_nir.cpp
+++ b/src/intel/compiler/brw_vec4_nir.cpp
@@ -1310,6 +1
On Mon, Apr 3, 2017 at 3:45 PM, Chad Versace
wrote:
> On Mon 03 Apr 2017, Jason Ekstrand wrote:
> > On Mon, Apr 3, 2017 at 12:31 PM, Chad Versace
> > wrote:
> >
> > > On Fri 31 Mar 2017, Chad Versace wrote:
> > > > On Wed 15 Mar 2017, Jason Ekstrand wrote:
> > > > > This cache allows us to easil
On Wed 15 Mar 2017, Jason Ekstrand wrote:
> This cache allows us to easily ensure that we have a unique anv_bo for
> each gem handle. We'll need this in order to support multiple-import of
> memory objects and semaphores.
>
> v2 (Jason Ekstrand):
> - Reject BO imports if the size doesn't match t
On Mon, Apr 3, 2017 at 4:26 PM, Chad Versace
wrote:
> On Wed 15 Mar 2017, Jason Ekstrand wrote:
> > This cache allows us to easily ensure that we have a unique anv_bo for
> > each gem handle. We'll need this in order to support multiple-import of
> > memory objects and semaphores.
> >
> > v2 (Ja
On 04/04/17 05:12, Bartosz Tomczyk wrote:
Address sanitizer reports lot of misaligned access:
SUMMARY: AddressSanitizer: undefined-behavior main/marshal.c:276:31 in
main/marshal.c:276:31: runtime error: load of misaligned address 0x631000104866
for type
'const GLuint' (aka 'const unsigned int')
1 - 100 of 118 matches
Mail list logo
