On Tue, Mar 15, 2016 at 10:13:57AM +0100, Iago Toral wrote:
> On Tue, 2016-03-15 at 11:04 +0200, Pohjolainen, Topi wrote:
> > On Tue, Mar 15, 2016 at 07:44:43AM +0100, Iago Toral wrote:
> > > On Mon, 2016-03-14 at 11:15 -0700, Mark Janes wrote:
> > > > Iago Toral writes:
> > > >
> > > > > On Wed,
https://bugs.freedesktop.org/show_bug.cgi?id=94564
Bug ID: 94564
Summary: [swrast] piglit attribs regression
Product: Mesa
Version: 11.2
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Keywords: bisec
On Tue, Mar 15, 2016 at 8:41 PM, Eric Anholt wrote:
> I copy and pasted this code before finding the helper.
> ---
> src/compiler/nir/nir_opt_constant_folding.c | 10 +++---
> 1 file changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/src/compiler/nir/nir_opt_constant_folding.c
> b/src
On Tue, Mar 15, 2016 at 8:41 PM, Eric Anholt wrote:
> I found this in a shader that was doing an alpha test when alpha is fixed
> at 1.0.
>
> instructions in affected programs: 16 -> 15 (-6.25%)
> total uniforms in shared programs: 28703 -> 28703 (0.00%)
> ---
> src/compiler/nir/nir_opt_cons
Sorry please ignore this for the moment I also need to include rules
for not packing tess amoung other things now that this is based off
master.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-d
I copy and pasted this code before finding the helper.
---
src/compiler/nir/nir_opt_constant_folding.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/src/compiler/nir/nir_opt_constant_folding.c
b/src/compiler/nir/nir_opt_constant_folding.c
index 8a353c2..190a509 10
I found this in a shader that was doing an alpha test when alpha is fixed
at 1.0.
instructions in affected programs: 16 -> 15 (-6.25%)
total uniforms in shared programs: 28703 -> 28703 (0.00%)
---
src/compiler/nir/nir_opt_constant_folding.c | 8
1 file changed, 8 insertions(+)
diff
From: Nicolai Hähnle
Empirically, both the EXEC_ON_* flags and LATE_Z are necessary.
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/rade
From: Nicolai Hähnle
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 12
src/gallium/auxiliary/tgsi/tgsi_scan.h | 1 +
2 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index 65bdab5..d32c3a1
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_descriptors.c | 22 +-
1 file changed, 21 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
b/src/gallium/drivers/radeonsi/si_descriptors.c
index 37e49e5..10d4499 100644
--- a/src
Fixes the following piglit tests:
- shaders/complex-loop-analysis-bug
- shaders/glsl-fs-discard-04
Converts the following piglit tests from crash to fail:
- shaders/glsl-fs-continue-inside-do-while
- shaders/glsl-fs-loop
- shaders/glsl-fs-loop-continue
- shaders/glsl-fs-loop-nested
- shaders/glsl-
In GL 4.4+ there is no guarantee that interpolation qualifiers will
match between stages so we cannot safely pack varyings using the
current packing pass in Mesa.
We also disable packing on outward facing SSO as these too are outside
the rule that guarantees the interpolation qualifiers match.
We
This will allow us to choose to ignore the disable which will be
useful for more fine grained control over when to enable or disable
packing.
Reviewed-by: Anuj Phogat
Reviewed-by: Edward O'Callaghan
Reviewed-by: Kenneth Graunke
---
src/compiler/glsl/ir_optimization.h | 3 ++-
src/comp
On Wed, Mar 16, 2016 at 12:05 AM, Nicolai Hähnle wrote:
> If I understand your patch correctly, the only effect is that a larger HTILE
> buffer is allocated, right? I'm surprised that only hangs were reported and
> not VM faults. We should really introduce guard pages / gaps between mapped
> buffe
https://bugs.freedesktop.org/show_bug.cgi?id=94561
--- Comment #3 from Roland Scheidegger ---
FWIW even if the number reported wouldn't overflow, reporting 16GB on 32bit
probably isn't really the right answer. If you'd try to use that much memory,
it would not quite succeed - maybe llvmpipe shoul
https://bugs.freedesktop.org/show_bug.cgi?id=94561
--- Comment #2 from Roland Scheidegger ---
At a very quick glance I would suspect it's os_get_total_physical_memory()
which isn't working. This has phys_pages * page_size which are both just long
which would overflow. If you use a int64_t for one
If I understand your patch correctly, the only effect is that a larger
HTILE buffer is allocated, right? I'm surprised that only hangs were
reported and not VM faults. We should really introduce guard pages /
gaps between mapped buffers.
It seems like htile.{pitch, height, xalign, yalign} are
On Tuesday, March 15, 2016 8:39:49 PM PDT Plamena Manolova wrote:
> When a user defines a point size array and enables it, the point
> size value set via glPointSize should be ignored. To achieve this,
> we can simply omit point size when creating a batch inside
> upload_sf_state for brw, gen6, gen
https://bugs.freedesktop.org/show_bug.cgi?id=94561
--- Comment #1 from Axel Davy ---
Actually, my test was 32 bits, and I just checked 64 bits and the issue is not
there.
I'm going to adjust the name of the bug to indicate this information.
--
You are receiving this mail because:
You are the Q
https://bugs.freedesktop.org/show_bug.cgi?id=94561
Axel Davy changed:
What|Removed |Added
Summary|[llvmpipe] |[llvmpipe]
|PIPE_CAP_VIDEO
On 03/04/2016 12:11 AM, Miklós Máté wrote:
On 02/25/2016 08:20 PM, Ian Romanick wrote:
On 02/25/2016 07:48 AM, Brian Paul wrote:
On 02/25/2016 08:26 AM, Miklós Máté wrote:
On 02/25/2016 02:37 AM, Brian Paul wrote:
On 02/24/2016 04:35 PM, Miklós Máté wrote:
Now it follows the GLX 1.4 specific
https://bugs.freedesktop.org/show_bug.cgi?id=94561
Bug ID: 94561
Summary: [llvmpipe] PIPE_CAP_VIDEO_MEMORY reports negative
value
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NEW
From: Marek Olšák
Cc: 11.1 11.2
---
src/gallium/drivers/radeon/r600_texture.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index 115c728..3957e71 100644
--- a/src/gallium/dri
This weird thing is there since 2013 and doesn't seem to be used
(according to the codegent part). I have carefully read the code to
make sure this is really unused but I will double-check with piglit.
This frees the first 32 bytes of the driver constant buffer.
Signed-off-by: Samuel Pitoiset
--
Instead make use of constants to improve readability.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nvc0/nvc0_compute.c| 13 +-
src/gallium/drivers/nouveau/nvc0/nvc0_context.h| 22 +
src/gallium/drivers/nouveau/nvc0/nvc0_program.c| 12 +---
The first 32 bytes of the driver constant buffer are currently unused.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nvc0/nvc0_context.h | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_context.h
b/src/gallium/
Hi,
This new series improves things related to the driver constant buffer which
are useful for compute shaders on gk104+. I have still not tested the series.
Please review,
Thanks!
Samuel Pitoiset (5):
nv50,nvc0: replace resInfoCBSlot by auxCBSlot
nv50/ir: make use of auxCBSlot instead of ma
Having two different variables for the driver constant buffer slot
is confusing and really useless.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_driver.h | 3 +--
.../drivers/nouveau/codegen/nv50_ir_lowering_nv50.cpp| 4 ++--
.../drivers/nouvea
This avoids using magic numbers for the driver constbuf slot which
is always 15 except for compute shaders on gk104+ where the slot 0
is used.
For gk104+, some special compute-related values like the thread
index are uploaded to screen->parm which is currently bound on c0.
Signed-off-by: Samuel P
Matt Turner writes:
> These printed the cycle count the last basic block (sched.time is set
> per basic block!). We have accurate, full program, data printed
> elsewhere.
Reviewed-by: Francisco Jerez
> ---
> src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp | 10 --
> 1 file cha
Matt Turner writes:
> All of these were simply code for "architecture register file" (and in
> the case of destinations, "not the null register").
> ---
> .../drivers/dri/i965/brw_schedule_instructions.cpp | 35
> +-
> 1 file changed, 8 insertions(+), 27 deletions(-)
>
> dif
On Mon, Mar 14, 2016 at 11:57 PM, Jordan Justen
wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 1 +
> src/mesa/drivers/dri/i965/brw_nir.c | 8
> src/mesa/drivers/dri/i965/brw_nir.h | 1 +
> 3 files changed, 10 insertions(+)
>
> diff --git a/src/m
Matt Turner writes:
> Missing this causes an assertion failure in the scheduler with the next
> patch.
> ---
> src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 1 -
> src/mesa/drivers/dri/i965/brw_vec4_tcs.cpp | 4 +++-
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/
On Mon, Mar 14, 2016 at 11:57 PM, Jordan Justen
wrote:
> Signed-off-by: Jordan Justen
> ---
> src/compiler/nir/nir_lower_io.c | 86
> -
> 1 file changed, 84 insertions(+), 2 deletions(-)
>
> diff --git a/src/compiler/nir/nir_lower_io.c
> b/src/compiler/ni
Rhys Kidd writes:
> Fixes the following piglit tests:
> - shaders/complex-loop-analysis-bug
> - shaders/glsl-fs-discard-04
>
> Converts the following piglit tests from crash to fail:
> - shaders/glsl-fs-continue-inside-do-while
> - shaders/glsl-fs-loop
> - shaders/glsl-fs-loop-continue
> - shader
Thanks for catching this! It came up before when I reworked function calls
but, thanks to the order in which things got merged, it had to be done in
your series.
On Mon, Mar 14, 2016 at 11:56 PM, Jordan Justen
wrote:
> Prevents:
>
> ../../../../../src/compiler/nir/nir.h:176:30: warning:
> 'nir_
I sent a v2 of the patches with your proposition Nicolai.
I think I don't have the rights to push patches, so it would be great
if you could do it.
2016-03-15 16:14 GMT-04:00 Romain Failliot :
> This fixes some exceptions I have to deal with in mesamatrix.net.
> The extensions GL_ARB_texture_buff
Kenneth Graunke writes:
> On Monday, March 14, 2016 5:25:58 PM PDT Matt Turner wrote:
>> On Mon, Mar 14, 2016 at 5:22 PM, Matt Turner wrote:
>> > On Mon, Mar 14, 2016 at 5:10 PM, Francisco Jerez
> wrote:
>> >> Matt Turner writes:
>> >>
>> >>> Missing this causes an assertion failure in the sc
Matt Turner writes:
> On Mon, Mar 14, 2016 at 9:32 PM, Kenneth Graunke
> wrote:
>> On Monday, March 14, 2016 5:25:58 PM PDT Matt Turner wrote:
>>> On Mon, Mar 14, 2016 at 5:22 PM, Matt Turner wrote:
>>> > On Mon, Mar 14, 2016 at 5:10 PM, Francisco Jerez
>> wrote:
>>> >> Matt Turner writes:
>
The "Status" column was misaligned in some GL sections.
This is a lot of diffs, but it's only spaces in the end.
---
docs/GL3.txt | 278 +--
1 file changed, 139 insertions(+), 139 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index 1
This fixes some exceptions I have to deal with in mesamatrix.net.
The extensions GL_ARB_texture_buffer_object had a comment between "DONE"
and the brackets.
And the extension GL_KHR_robustness (in GL 4.5 and GLES 3.1) was using
"90% done" instead of "in progress". The "90% done" is still here
thoug
For older extensions, there is an explanation first and the extension
name in brackets, like that:
Clamping controls (GL_ARB_color_buffer_float)
I inverted that so we have the extension first and then the explanation
in brackets, like that:
GL_ARB_color_buffer_float (Clamping controls)
It
Added a small guide on how to read and edit GL3.txt.
I think this would help as much the devs as the users reading this file.
---
docs/GL3.txt | 25 -
1 file changed, 20 insertions(+), 5 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index ee7faca..1ed5c1a 100644
--
From: Samuel Pitoiset
This makes use of the new state validation interface to be consistent
with 3d.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nv50/nv50_compute.c | 25 +
1 file changed, 13 insertions(+), 12 deletions(-)
diff --git a/src/gallium/dr
From: Samuel Pitoiset
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nv50/nv50_context.c| 24 +++---
src/gallium/drivers/nouveau/nv50/nv50_context.h| 20 +-
.../drivers/nouveau/nv50/nv50_shader_state.c | 12 +--
src/gallium/d
From: Samuel Pitoiset
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nv50/nv50_compute.c| 90 +++---
.../drivers/nouveau/nv50/nv50_query_hw_sm.c| 10 +--
src/gallium/drivers/nouveau/nv50/nv50_winsys.h | 4 +-
3 files changed, 52 insertions(+), 52
From: Samuel Pitoiset
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nv50/nv50_compute.c| 2 +-
src/gallium/drivers/nouveau/nv50/nv50_context.c| 10 +--
src/gallium/drivers/nouveau/nv50/nv50_context.h| 44 +--
.../drivers/nouveau/nv50/nv50_shader_state.c
From: Samuel Pitoiset
Reduce the amount of duplicated code by re-using
nv50_program_validate(). While we are at it, change the prototype to
return void. We don't check anymore if the translation fails but
improving the state validation is a long process.
Signed-off-by: Samuel Pitoiset
---
src/
From: Samuel Pitoiset
This exposes an interface for state validation that will be also used
to rework the compute validation path.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nv50/nv50_context.h| 10 +-
.../drivers/nouveau/nv50/nv50_state_validate.c | 36
From: Samuel Pitoiset
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nv50/nv50_compute.c| 2 +-
src/gallium/drivers/nouveau/nv50/nv50_context.c| 10 +++---
src/gallium/drivers/nouveau/nv50/nv50_context.h| 2 +-
.../drivers/nouveau/nv50/nv50_shader_state.c | 1
From: Samuel Pitoiset
Hi,
This is loosely based on what I did for nvc0 few weeks ago. I have not tested
this series because I don't have access to a Tesla card, but this should not
break anything. By the way, doing almost the same series twice is not so cool
but... refactoring nv50 and nvc0 driv
Thanks for taking the time to clean up the patches!
On 15.03.2016 07:45, Romain Failliot wrote:
Hmm... I thought that was what it meant.
I can add a new status like "paused" for instance.
"not started" is probably more accurate. AFAIU, ARB_robustness is
implemented, and that represents 90% of
Indeed, none of nouveau_vieux, radeon, or r200 support GLSL. Of
course, no classic driver except i965 supports #version 130, so those
are clear.
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.
On Mon, Mar 14, 2016 at 9:32 PM, Kenneth Graunke wrote:
> On Monday, March 14, 2016 5:25:58 PM PDT Matt Turner wrote:
>> On Mon, Mar 14, 2016 at 5:22 PM, Matt Turner wrote:
>> > On Mon, Mar 14, 2016 at 5:10 PM, Francisco Jerez
> wrote:
>> >> Matt Turner writes:
>> >>
>> >>> Missing this causes
Now that we implement tessellation shaders, the TES might be the last
stage enabled. If it's outputting points, then the primitive type
reaching the SF is points. We need to account for this.
Caught by Ilia Mirkin.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen6_sf_state.c |
This is cleaner than using glBindAttribLocation().
Not all drivers support the extension, but I don't think those drivers
use GLSL in the first place. Apparently some Meta shaders already use
GL_ARB_explicit_attrib_location, so I think it should be okay.
Honestly, I'm not sure how the old code w
https://bugs.freedesktop.org/show_bug.cgi?id=94522
--- Comment #11 from Roland Scheidegger ---
FWIW the bug Stephane mentioned (which you'd hit near certainly too, unless you
built without sse2 enabled, and the compositor itself might not hit it) is
fixed by bb2c5e657b5f4c55bcec49a8d96f352ed4c1e0
https://bugs.freedesktop.org/show_bug.cgi?id=94193
Roland Scheidegger changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
https://bugs.freedesktop.org/show_bug.cgi?id=94481
Roland Scheidegger changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
Patches 1-4 are
Reviewed-by: Ilia Mirkin
On Tue, Mar 15, 2016 at 2:28 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> ---
> src/gallium/auxiliary/tgsi/tgsi_scan.c | 3 +++
> src/gallium/auxiliary/tgsi/tgsi_scan.h | 4
> 2 files changed, 7 insertions(+)
>
> diff --git a/src/gallium/a
When a user defines a point size array and enables it, the point
size value set via glPointSize should be ignored. To achieve this,
we can simply omit point size when creating a batch inside
upload_sf_state for brw, gen6, gen7 and gen8.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=42187
From: Nicolai Hähnle
Whether DCC is disabled depends on the access flags with which the image
is bound: image_load supports DCC, but store and atomic don't.
---
src/gallium/drivers/radeonsi/si_descriptors.c | 206 --
src/gallium/drivers/radeonsi/si_pipe.h| 7 +
From: Nicolai Hähnle
Prevent loads from being re-ordered or coalesced.
Atomics don't need special handling by definition, and stores don't need
special handling because LLVM is unable to detect dead image or buffer
stores.
---
src/gallium/drivers/radeonsi/si_shader.c | 4
1 file changed, 4
From: Nicolai Hähnle
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 18 --
src/gallium/auxiliary/tgsi/tgsi_scan.h | 5 +
2 files changed, 21 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index 8e24c
From: Nicolai Hähnle
This will allow it to be reused for RESQ.
---
src/gallium/drivers/radeonsi/si_shader.c | 55
1 file changed, 35 insertions(+), 20 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_shader.c | 83 ++--
1 file changed, 80 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index e2ae39e..374afac 100644
--- a/src/gal
From: Edward O'Callaghan
This enables ARB_shader_image_load_store and ARB_shader_image_size.
Signed-off-by: Edward O'Callaghan
[allow the same number of images for all shader stages and require LLVM 3.9]
---
src/gallium/drivers/radeonsi/si_pipe.c | 3 ++-
1 file changed, 2 insertions(+), 1 del
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_shader.c | 31 +++
1 file changed, 31 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index deeda30..fcbab5f 100644
--- a/src/gallium/drivers/rade
From: Nicolai Hähnle
Re-order flags in the order in which they appear in the OpenGL spec in the
description of MemoryBarrier().
---
src/mesa/state_tracker/st_cb_texturebarrier.c | 21 ++---
1 file changed, 18 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_c
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_shader.c | 26 +++---
1 file changed, 19 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 373d4b4..b655e34 100644
--- a/src/gallium/d
From: Nicolai Hähnle
---
docs/GL3.txt | 4 ++--
docs/relnotes/11.3.0.html | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/docs/GL3.txt b/docs/GL3.txt
index ee7faca..08ca8db 100644
--- a/docs/GL3.txt
+++ b/docs/GL3.txt
@@ -139,7 +139,7 @@ GL 4.2, GLSL 4.20:
From: Nicolai Hähnle
---
src/gallium/include/pipe/p_defines.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/include/pipe/p_defines.h
b/src/gallium/include/pipe/p_defines.h
index bdd76ab..90af7a7 100644
--- a/src/gallium/include/pipe/p_defines.h
+++ b/src/gallium/include
From: Nicolai Hähnle
This avoids a lockup at least on Tonga.
---
src/gallium/drivers/radeonsi/si_shader.c | 53 +++-
1 file changed, 45 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
ind
From: Nicolai Hähnle
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 3 +++
src/gallium/auxiliary/tgsi/tgsi_scan.h | 4
2 files changed, 7 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index dee6884..65bdab5 100644
--- a/src/galliu
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_state.c | 36 +
1 file changed, 36 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
b/src/gallium/drivers/radeonsi/si_state.c
index 0c3fbdc..6dcd532 100644
--- a/src/gallium/drivers/radeo
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_shader.c | 121 +--
1 file changed, 113 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index 374afac..deeda30 100644
--- a/src/ga
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_shader.c | 17 +
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index fcbab5f..22296fe 100644
--- a/src/gallium/drivers/ra
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_blit.c | 36 +++---
1 file changed, 33 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_blit.c
b/src/gallium/drivers/radeonsi/si_blit.c
index f9a6de4..e0dbec5 100644
--- a/src/gallium
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_shader.c | 141 +++
1 file changed, 141 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index b655e34..e2ae39e 100644
--- a/src/gallium/drivers/ra
Hi,
finally: ARB_shader_image_load_store (and ARB_shader_image_size) for radeonsi!
You can also find this series at
https://cgit.freedesktop.org/~nh/mesa/log/?h=images-radeonsi
You will need bleeding edge LLVM for this to work correctly, and not all
required LLVM changes have been pushed upstream
From: Nicolai Hähnle
---
src/gallium/drivers/radeonsi/si_shader.c | 129 +++
1 file changed, 129 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index a4352f1..373d4b4 100644
--- a/src/gallium/drivers/ra
From: Nicolai Hähnle
We will need it in radeonsi for shader images.
---
src/gallium/drivers/radeon/r600_pipe_common.h | 2 ++
src/gallium/drivers/radeon/r600_texture.c | 4 ++--
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_pipe_common.h
b/sr
On Tue, Mar 15, 2016 at 12:41 AM, Iago Toral wrote:
> On Mon, 2016-03-14 at 17:33 -0700, Jason Ekstrand wrote:
> >
> >
> > On Mon, Mar 7, 2016 at 12:46 AM, Samuel Iglesias Gonsálvez
> > wrote:
> > From: Connor Abbott
> >
> > When we replace an expresion we have to compute bitsiz
On Sat, Mar 12, 2016 at 3:08 PM, Jason Ekstrand
wrote:
>
> On Mar 11, 2016 12:33 PM, "Alejandro Piñeiro"
> wrote:
> >
> > On 11/03/16 20:15, Anuj Phogat wrote:
> > > yoffset is also applicable to 1d array textures.
> > >
> > > Signed-off-by: Anuj Phogat
> > > ---
> > > I don't know if it fixes
https://bugs.freedesktop.org/show_bug.cgi?id=94522
--- Comment #10 from Roland Scheidegger ---
Actually the asm snipped might be something different. You could try printing
the whole jit prog (from the address from the caller, or use
GALLIVM_DEBUG=asm).
Using valgrind could possibly tell you some
Not really an answer to your question, but you may be interested in
this model of the NVIDIA Tesla FPU that mwk RE'd, including fp64
(which was available on the G200 only in that series). I have no
reason to believe that Fermi+ are substantially different:
https://github.com/envytools/envytools/bl
https://bugs.freedesktop.org/show_bug.cgi?id=94522
--- Comment #9 from Roland Scheidegger ---
(In reply to comicfans44 from comment #8)
> (In reply to Roland Scheidegger from comment #4)
> > So, it crashed in the jit fragment shader code.
> > Could you tell what the fb size was (the values from s
Hi,
as complementary work to the one done to "Add FP64 support to the i965
shader backends" at:
https://bugs.freedesktop.org/show_bug.cgi?id=92760
I've been working to add piglit tests that would check the new features
added by this addition.
One of the topics we have been creating tests for is
Am 15.03.2016 um 08:20 schrieb Jose Fonseca:
> On 13/03/16 20:40, srol...@vmware.com wrote:
>> From: Roland Scheidegger
>>
>> The logic was comparing actual ints, not true/false values.
>> This meant that it was emitting always multiple line segments instead
>> of just
>> one even if the stipple t
Hi,
I haven't had any input on this and I will welcome opinions from more
knowledgeable developers :)
Do you think my approach in this case would be the best?
* To enforce the usage of the "flat" qualifier when in a member of
an input block interface in a fragment shader for (unsign
On Tue, Mar 15, 2016 at 3:53 PM, Ilia Mirkin wrote:
>
> On Mar 15, 2016 7:23 AM, "Nicolai Hähnle" wrote:
>>
>> From: Nicolai Hähnle
>>
>> The bitcasting which is possible with shader images (and texture views?)
>> requires that when the user specifies a sized internal format for a
>> texture, we
On Mar 15, 2016 7:48 AM, "Connor Abbott" wrote:
>
> On Tue, Mar 15, 2016 at 10:43 AM, Connor Abbott
wrote:
> > On Tue, Mar 15, 2016 at 5:53 AM, Iago Toral wrote:
> >> On Mon, 2016-03-14 at 16:48 -0700, Jason Ekstrand wrote:
> >>>
> >>>
> >>> On Mon, Mar 7, 2016 at 12:46 AM, Samuel Iglesias Gonsá
On Mar 15, 2016 7:23 AM, "Nicolai Hähnle" wrote:
>
> From: Nicolai Hähnle
>
> The bitcasting which is possible with shader images (and texture views?)
> requires that when the user specifies a sized internal format for a
> texture, we really allocate that format. To this end:
>
> (1) find_exact_f
On Tue, Mar 15, 2016 at 10:43 AM, Connor Abbott wrote:
> On Tue, Mar 15, 2016 at 5:53 AM, Iago Toral wrote:
>> On Mon, 2016-03-14 at 16:48 -0700, Jason Ekstrand wrote:
>>>
>>>
>>> On Mon, Mar 7, 2016 at 12:46 AM, Samuel Iglesias Gonsálvez
>>> wrote:
>>> From: Connor Abbott
>>>
>>>
On Tue, Mar 15, 2016 at 5:53 AM, Iago Toral wrote:
> On Mon, 2016-03-14 at 16:48 -0700, Jason Ekstrand wrote:
>>
>>
>> On Mon, Mar 7, 2016 at 12:46 AM, Samuel Iglesias Gonsálvez
>> wrote:
>> From: Connor Abbott
>>
>> v2: Use the bit-size information from the opcode information
>>
From: Nicolai Hähnle
The bitcasting which is possible with shader images (and texture views?)
requires that when the user specifies a sized internal format for a
texture, we really allocate that format. To this end:
(1) find_exact_format should ignore sized internal formats and
(2) some of the
---
src/compiler/glsl/link_varyings.cpp | 1 +
src/mesa/main/mtypes.h | 2 ++
src/mesa/main/program_resource.c| 16 +++-
src/mesa/main/shader_query.cpp | 35 ++-
4 files changed, 52 insertions(+), 2 deletions(-)
diff --git a/src/
---
src/compiler/glsl/linker.cpp | 14 +-
src/compiler/glsl/program.h| 3 ++-
src/mesa/drivers/dri/i965/brw_link.cpp | 2 +-
src/mesa/program/ir_to_mesa.cpp| 2 +-
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 2 +-
5 files changed, 18 i
---
src/compiler/glsl/link_varyings.cpp | 34 --
src/compiler/glsl/link_varyings.h | 13 +
2 files changed, 37 insertions(+), 10 deletions(-)
diff --git a/src/compiler/glsl/link_varyings.cpp
b/src/compiler/glsl/link_varyings.cpp
index ac14815..c3f6c3
---
src/compiler/glsl/link_varyings.cpp | 1 +
src/mesa/main/mtypes.h | 1 +
src/mesa/main/shader_query.cpp | 14 +++---
3 files changed, 13 insertions(+), 3 deletions(-)
diff --git a/src/compiler/glsl/link_varyings.cpp
b/src/compiler/glsl/link_varyings.cpp
index a97d
1 - 100 of 147 matches
Mail list logo