https://bugs.freedesktop.org/show_bug.cgi?id=89088
Timothy Arceri changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Resolution|---
On Thu, Feb 11, 2016 at 03:49:16PM -0800, Ben Widawsky wrote:
> On Thu, Feb 11, 2016 at 08:34:01PM +0200, Topi Pohjolainen wrote:
> > Signed-off-by: Topi Pohjolainen
> > ---
> > src/mesa/drivers/dri/i965/gen8_surface_state.c | 62
> > --
> > 1 file changed, 29 insertions(
https://bugs.freedesktop.org/show_bug.cgi?id=69053
Timothy Arceri changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://bugs.freedesktop.org/show_bug.cgi?id=29272
Timothy Arceri changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Am 10.02.2016 um 19:10 schrieb Samuel Pitoiset:
> This state tracker implements DispatchCompute() and DispatchComputeIndirect().
>
> Signed-off-by: Samuel Pitoiset
> Reviewed-by: Marek Olšák
> Reviewed-by: Ilia Mirkin
> ---
> src/mesa/Makefile.sources | 2 +
> src/mesa/state_trac
On 02/13/2016 04:52 PM, Roland Scheidegger wrote:
Am 10.02.2016 um 19:10 schrieb Samuel Pitoiset:
This state tracker implements DispatchCompute() and DispatchComputeIndirect().
Signed-off-by: Samuel Pitoiset
Reviewed-by: Marek Olšák
Reviewed-by: Ilia Mirkin
---
src/mesa/Makefile.sources
Fixes MSVC build error which doesn't allow empty initializers.
Signed-off-by: Samuel Pitoiset
Cc: Roland Scheidegger
---
src/mesa/state_tracker/st_cb_compute.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_cb_compute.c
b/src/mesa/state_tracker/st
Am 13.02.2016 um 16:56 schrieb Samuel Pitoiset:
>
>
> On 02/13/2016 04:52 PM, Roland Scheidegger wrote:
>> Am 10.02.2016 um 19:10 schrieb Samuel Pitoiset:
>>> This state tracker implements DispatchCompute() and
>>> DispatchComputeIndirect().
>>>
>>> Signed-off-by: Samuel Pitoiset
>>> Reviewed-by
Reviewed-by: Roland Scheidegger
Am 13.02.2016 um 17:02 schrieb Samuel Pitoiset:
> Fixes MSVC build error which doesn't allow empty initializers.
>
> Signed-off-by: Samuel Pitoiset
> Cc: Roland Scheidegger
> ---
> src/mesa/state_tracker/st_cb_compute.c | 2 +-
> 1 file changed, 1 insertion(+),
On 02/13/2016 05:06 PM, Roland Scheidegger wrote:
Am 13.02.2016 um 16:56 schrieb Samuel Pitoiset:
On 02/13/2016 04:52 PM, Roland Scheidegger wrote:
Am 10.02.2016 um 19:10 schrieb Samuel Pitoiset:
This state tracker implements DispatchCompute() and
DispatchComputeIndirect().
Signed-off-by:
---
src/gallium/state_trackers/clover/core/kernel.cpp | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 41b3852..a4ef2b1 100644
--- a/src/gallium/state_trackers/clo
Except the minor comment below, this patch is:
Reviewed-by: Samuel Pitoiset
Sorry for this, the next time I'll build clover and learn from my
mistakes. :-)
On 02/13/2016 06:01 PM, Serge Martin wrote:
---
src/gallium/state_trackers/clover/core/kernel.cpp | 7 ---
1 file changed, 4 ins
On Sat, Feb 13, 2016 at 12:01 PM, Serge Martin wrote:
> ---
> src/gallium/state_trackers/clover/core/kernel.cpp | 7 ---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
> b/src/gallium/state_trackers/clover/core/kernel.cpp
On 02/12/2016 06:07 PM, Ilia Mirkin wrote:
Reviewed-by: Ilia Mirkin
Thanks, but this comes after the preceding 14 or so patches that I
posted before. Care to review those too?
-Brian
On Fri, Feb 12, 2016 at 8:07 PM, Brian Paul wrote:
---
src/mesa/state_tracker/st_cb_drawpixels.c | 1
This series is: Reviewed-by: Ilia Mirkin
On Fri, Feb 12, 2016 at 10:45 AM, Brian Paul wrote:
> Rename to 'tex_attr' to be a bit more clear.
> ---
> src/mesa/state_tracker/st_cb_drawtex.c | 20 ++--
> 1 file changed, 10 insertions(+), 10 deletions(-)
>
> diff --git a/src/mesa/sta
---
src/gallium/state_trackers/clover/core/kernel.cpp | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 41b3852..3daec3b 100644
--- a/src/gallium/state_trackers/c
I don't see the necessity to add SV_LBASE and SV_SBASE for now
because they are currently not used and I need to double-check
the ids first.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/
Reviewed-by: Ilia Mirkin
On Sat, Feb 13, 2016 at 12:27 PM, Serge Martin wrote:
> ---
> src/gallium/state_trackers/clover/core/kernel.cpp | 9 ++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
> b/src/gallium/state_tra
On Sat, Feb 13, 2016 at 12:32 PM, Samuel Pitoiset
wrote:
> I don't see the necessity to add SV_LBASE and SV_SBASE for now
> because they are currently not used and I need to double-check
> the ids first.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/codegen/nv50_ir_emit_
---
Placed under "Core Mesa environment variables" as I didn't find any
better place. The text is a summarized version of
MESA_GL_VERSION_OVERRIDE documentation. I hope not be forgetting
something specific of this envvar.
docs/envvars.html | 9 +
1 file changed, 9 insertions(+)
diff --g
On 02/13/2016 11:12 AM, Alejandro Piñeiro wrote:
---
Placed under "Core Mesa environment variables" as I didn't find any
better place. The text is a summarized version of
MESA_GL_VERSION_OVERRIDE documentation. I hope not be forgetting
something specific of this envvar.
docs/envvars.html | 9
Variable was previously always set to true. Accordingly, the later
assert() served no active purpose.
Found with GCC warning and code inspection:
mesa/src/gallium/drivers/vc4/vc4_qpu_emit.c: In function'vc4_generate_code':
mesa/src/gallium/drivers/vc4/vc4_qpu_emit.c:315:22: warning: variable
'ha
v2: Removed reference to FC not being an allowed suffix (Brian Paul)
---
docs/envvars.html | 8
1 file changed, 8 insertions(+)
diff --git a/docs/envvars.html b/docs/envvars.html
index ba83335..8c4597c 100644
--- a/docs/envvars.html
+++ b/docs/envvars.html
@@ -91,6 +91,14 @@ This is only
On 02/13/2016 11:24 AM, Alejandro Piñeiro wrote:
v2: Removed reference to FC not being an allowed suffix (Brian Paul)
---
docs/envvars.html | 8
1 file changed, 8 insertions(+)
diff --git a/docs/envvars.html b/docs/envvars.html
index ba83335..8c4597c 100644
--- a/docs/envvars.html
++
On 13/02/16 20:05, Brian Paul wrote:
> On 02/13/2016 11:24 AM, Alejandro Piñeiro wrote:
>> v2: Removed reference to FC not being an allowed suffix (Brian Paul)
>> ---
>> docs/envvars.html | 8
>> 1 file changed, 8 insertions(+)
>>
>> diff --git a/docs/envvars.html b/docs/envvars.html
Tested-By: Aaron Watry
On Sat, Feb 13, 2016 at 11:27 AM, Serge Martin wrote:
> ---
> src/gallium/state_trackers/clover/core/kernel.cpp | 9 ++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
> b/src/gallium/state_track
https://bugs.freedesktop.org/show_bug.cgi?id=94137
Bug ID: 94137
Summary: build error - incompatible types in assignment of
'unsigned int*' to 'uint [3] {aka unsigned int [3]}'
Product: Mesa
Version: git
Hardware: x86-64 (A
https://bugs.freedesktop.org/show_bug.cgi?id=94137
--- Comment #1 from Samuel Pitoiset ---
Hi,
Yes, I know that I did break clover with my compute shader series, sorry for
that.
A bugfix has been already sent by Serge Martin:
https://lists.freedesktop.org/archives/mesa-dev/2016-February/107540.
On Fri, Feb 12, 2016 at 10:43 AM, Brian Paul wrote:
> diff --git a/src/mesa/state_tracker/st_context.c
> b/src/mesa/state_tracker/st_context.c
> index 9016846..cb2c390 100644
> --- a/src/mesa/state_tracker/st_context.c
> +++ b/src/mesa/state_tracker/st_context.c
> @@ -241,16 +241,23 @@ st_create_
Signed-off-by: Rhys Kidd
---
docs/envvars.html | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/envvars.html b/docs/envvars.html
index ba83335..d7697e3 100644
--- a/docs/envvars.html
+++ b/docs/envvars.html
@@ -224,7 +224,7 @@ See src/mesa/state_tracker/st_debug.c for othe
brw_draw_upload.c uploads VertexID/InstanceID first, then DrawID.
So we need to assign the attribute mapping in that order as well.
Fixes the following Pigit tests with the vec4 backend:
- arb_shader_draw_parameters-drawid vertexid
- arb_shader_draw_parameters-drawid-indirect basevertex
Signed-of
On Sat, Feb 13, 2016 at 12:45 PM, Kenneth Graunke wrote:
> brw_draw_upload.c uploads VertexID/InstanceID first, then DrawID.
> So we need to assign the attribute mapping in that order as well.
>
> Fixes the following Pigit tests with the vec4 backend:
> - arb_shader_draw_parameters-drawid vertexid
Signed-off-by: Rhys Kidd
---
docs/envvars.html | 19 +++
1 file changed, 19 insertions(+)
diff --git a/docs/envvars.html b/docs/envvars.html
index ba83335..c0007a6 100644
--- a/docs/envvars.html
+++ b/docs/envvars.html
@@ -245,6 +245,25 @@ for details.
+VC4 driver environme
On Thu, Feb 11, 2016 at 4:41 PM, Matt Turner wrote:
> Gen4/5's SEL instruction cannot use conditional modifiers, so min/max
> are implemented as CMP + SEL. Handling that after optimization lets us
> CSE more.
>
> On Ironlake:
>
>total instructions in shared programs: 6426035 -> 6422753 (-0.05%
Per
http://help.appveyor.com/discussions/problems/3801-builds-failing-without-generating-any-log
this was a transitory issue. And in fact I never saw it again.
Builds have been going reliably, so I've just re-enabled notification
emails to this list.
Is there any admin here that could help
Ilia Mirkin writes:
> On Sat, Feb 13, 2016 at 12:01 PM, Serge Martin wrote:
>> ---
>> src/gallium/state_trackers/clover/core/kernel.cpp | 7 ---
>> 1 file changed, 4 insertions(+), 3 deletions(-)
>>
>> diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
>> b/src/gallium/state_tr
On 12/02/16 15:43, Brian Paul wrote:
---
src/mesa/state_tracker/st_cb_bitmap.c | 16 +++-
src/mesa/state_tracker/st_cb_clear.c | 13 ++---
src/mesa/state_tracker/st_cb_drawpixels.c | 14 ++
3 files changed, 7 insertions(+), 36 deletions(-)
diff --git
MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
introduced in be27f77 (mesa: do not use a constant for
MAX_COMPUTE_SHARED_SIZE).
Signed-off-by: Samuel Pitoiset
Cc: Mark Janes
---
This patch is untested because I don't have any Intel GPUs.
src/mesa/drivers/dri/i965/brw_
Commit be27f772e8b97031bf5177 added ctx->Const.MaxComputeSharedMemorySize,
and added a line to initialize it to a default value of 32768 in the
standalone GLSL compiler, but didn't initialize a default value in real
GL contexts.
This regressed arb_compute_shader-minmax on i965, as the value of the
On 2016-02-13 13:49:37, Samuel Pitoiset wrote:
> MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
> introduced in be27f77 (mesa: do not use a constant for
> MAX_COMPUTE_SHARED_SIZE).
>
> Signed-off-by: Samuel Pitoiset
> Cc: Mark Janes
> ---
>
> This patch is untested beca
On Sat, Feb 13, 2016 at 4:49 PM, Samuel Pitoiset
wrote:
> MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
> introduced in be27f77 (mesa: do not use a constant for
> MAX_COMPUTE_SHARED_SIZE).
>
> Signed-off-by: Samuel Pitoiset
> Cc: Mark Janes
Reviewed-by: Ilia Mirkin
P
On Saturday, February 13, 2016 10:49:37 PM PST Samuel Pitoiset wrote:
> MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
> introduced in be27f77 (mesa: do not use a constant for
> MAX_COMPUTE_SHARED_SIZE).
>
> Signed-off-by: Samuel Pitoiset
> Cc: Mark Janes
> ---
>
> This
On 02/13/2016 10:55 PM, Jordan Justen wrote:
On 2016-02-13 13:49:37, Samuel Pitoiset wrote:
MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
introduced in be27f77 (mesa: do not use a constant for
MAX_COMPUTE_SHARED_SIZE).
Signed-off-by: Samuel Pitoiset
Cc: Mark Janes
On 02/13/2016 10:57 PM, Kenneth Graunke wrote:
On Saturday, February 13, 2016 10:49:37 PM PST Samuel Pitoiset wrote:
MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
introduced in be27f77 (mesa: do not use a constant for
MAX_COMPUTE_SHARED_SIZE).
Signed-off-by: Samuel P
On 12/02/16 15:44, Brian Paul wrote:
---
src/mesa/state_tracker/st_cb_bitmap.c | 36
src/mesa/state_tracker/st_cb_clear.c | 46 +++
src/mesa/state_tracker/st_cb_drawpixels.c | 46 +++
src/mesa/state_tra
Matt Turner writes:
> On Thu, Feb 11, 2016 at 4:41 PM, Matt Turner wrote:
>> Gen4/5's SEL instruction cannot use conditional modifiers, so min/max
>> are implemented as CMP + SEL. Handling that after optimization lets us
>> CSE more.
>>
>> On Ironlake:
>>
>>total instructions in shared progr
Matt Turner writes:
> ---
> src/mesa/drivers/dri/i965/brw_vec4_builder.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_vec4_builder.h
> b/src/mesa/drivers/dri/i965/brw_vec4_builder.h
> index be1427c..5d4b452 100644
> --- a/src/mesa/driv
---
src/gallium/state_trackers/clover/core/program.cpp | 4 ++--
src/gallium/state_trackers/clover/core/program.hpp | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/program.cpp
b/src/gallium/state_trackers/clover/core/program.cpp
index
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/program.cpp | 31 +++---
src/gallium/state_trackers/clover/core/program.hpp | 4 +--
2 files changed, 29 insertions(+), 6 deletions(-)
diff --git a/src/gallium/state_trackers/clover/api/program.cpp
b/src/ga
---
.../state_trackers/clover/llvm/invocation.cpp | 234 ++---
1 file changed, 23 insertions(+), 211 deletions(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invocation.cpp
index 4d11c24..d3347de 100644
--- a/sr
---
src/gallium/state_trackers/clover/Makefile.sources | 3 +-
.../state_trackers/clover/llvm/ir_compiler.cpp | 282 +
.../state_trackers/clover/llvm/ir_compiler.hpp | 59 +
3 files changed, 343 insertions(+), 1 deletion(-)
create mode 100644 src/gallium/state_t
---
src/gallium/state_trackers/clover/api/program.cpp | 19 --
.../state_trackers/clover/core/compiler.hpp| 7 +-
src/gallium/state_trackers/clover/core/error.hpp | 7 ++
src/gallium/state_trackers/clover/core/program.cpp | 38 ++-
src/gallium/state_trackers/clover/core/pr
---
.../state_trackers/clover/llvm/invocation.cpp | 87 --
.../state_trackers/clover/llvm/ir_compiler.cpp | 46 +---
.../state_trackers/clover/llvm/ir_compiler.hpp | 4 +-
3 files changed, 85 insertions(+), 52 deletions(-)
diff --git a/src/gallium/state_t
This serie add clLinkProgram function needed for CL 1.2.
However, it lacks the binary type part that is mandatory for input validation
and also for CL_PROGRAM_BINARY_TYPE query. This will be adressed
in another serie once we agree on the way to store it.
Serge Martin (7):
clover: add a LLVM comp
Return an API object from an intrusive reference to a Clover object,
incrementing the reference count of the object.
Reviewed-by: Francisco Jerez
---
src/gallium/state_trackers/clover/api/util.hpp | 12
1 file changed, 12 insertions(+)
diff --git a/src/gallium/state_trackers/clover
On 12/02/16 15:44, Brian Paul wrote:
cso_save_state() takes a bitmask of state items to save. Calling
cso_restore_state() restores those states.
---
src/gallium/auxiliary/cso_cache/cso_context.c | 109 ++
src/gallium/auxiliary/cso_cache/cso_context.h | 24 ++
2 f
On 12/02/16 15:45, Brian Paul wrote:
Rename to 'tex_attr' to be a bit more clear.
---
src/mesa/state_tracker/st_cb_drawtex.c | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_drawtex.c
b/src/mesa/state_tracker/st_cb_drawtex
On Sat, Feb 13, 2016 at 4:45 PM, Francisco Jerez wrote:
> Ilia Mirkin writes:
>
>> On Sat, Feb 13, 2016 at 12:01 PM, Serge Martin wrote:
>>> ---
>>> src/gallium/state_trackers/clover/core/kernel.cpp | 7 ---
>>> 1 file changed, 4 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/src/galliu
Ilia Mirkin writes:
> On Sat, Feb 13, 2016 at 4:45 PM, Francisco Jerez
> wrote:
>> Ilia Mirkin writes:
>>
>>> On Sat, Feb 13, 2016 at 12:01 PM, Serge Martin wrote:
---
src/gallium/state_trackers/clover/core/kernel.cpp | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-
On 2016-02-13 13:57:17, Samuel Pitoiset wrote:
>
> On 02/13/2016 10:55 PM, Jordan Justen wrote:
> > On 2016-02-13 13:49:37, Samuel Pitoiset wrote:
> >> MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
> >> introduced in be27f77 (mesa: do not use a constant for
> >> MAX_COMPU
On 02/13/2016 11:18 PM, Jordan Justen wrote:
On 2016-02-13 13:57:17, Samuel Pitoiset wrote:
On 02/13/2016 10:55 PM, Jordan Justen wrote:
On 2016-02-13 13:49:37, Samuel Pitoiset wrote:
MAX_COMPUTE_SHARED_SIZE should be set to 32768. This fixes a regression
introduced in be27f77 (mesa: do not
On Sat, 2016-02-13 at 13:55 -0800, Kenneth Graunke wrote:
> Commit be27f772e8b97031bf5177 added ctx-
> >Const.MaxComputeSharedMemorySize,
> and added a line to initialize it to a default value of 32768 in the
> standalone GLSL compiler, but didn't initialize a default value in
> real
> GL contexts.
---
src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_trackers/clover/core/kernel.cpp
index 41b3852..8396be9 100644
--- a/src/gallium/state_trackers/clov
https://bugs.freedesktop.org/show_bug.cgi?id=94088
Vinson Lee changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
We already have one in the IR code that can be used everywhere its
needed in the AST code so remove the one from the AST.
---
src/compiler/glsl/ast.h | 12
src/compiler/glsl/ast_to_hir.cpp | 7 ++-
src/compiler/glsl/ast_type.cpp | 13 -
3 files changed, 2 i
On Sat, Feb 13, 2016 at 2:02 PM, Francisco Jerez wrote:
> Matt Turner writes:
>
>> On Thu, Feb 11, 2016 at 4:41 PM, Matt Turner wrote:
>>> Gen4/5's SEL instruction cannot use conditional modifiers, so min/max
>>> are implemented as CMP + SEL. Handling that after optimization lets us
>>> CSE more
Assuming this compiles, this is
Reviewed-by: Ilia Mirkin
On Sat, Feb 13, 2016 at 5:39 PM, Serge Martin wrote:
> ---
> src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/core/kernel.c
Thanks kindly.
Reviewed-by: Edward O'Callaghan
On 2016-02-14 09:39, Serge Martin wrote:
---
src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
b/src/gallium/state_tracker
Matt Turner writes:
> On Sat, Feb 13, 2016 at 2:02 PM, Francisco Jerez
> wrote:
>> Matt Turner writes:
>>
>>> On Thu, Feb 11, 2016 at 4:41 PM, Matt Turner wrote:
Gen4/5's SEL instruction cannot use conditional modifiers, so min/max
are implemented as CMP + SEL. Handling that after o
Without this NVF0_COMPUTE environment variable, compute support is
initialized by default and this is not what we want for now because
it might break 3D. Compute will be enabled by default on GK110+ once
we are sure that it won't break anything else.
Signed-off-by: Samuel Pitoiset
---
src/galliu
For some unknown reasons, the command 0x518, which is FIRMWARE[0x6]
from rnndb, totally hangs the GPU, and a reboot is needed. With
that removed, I can succesfully launch compute shaders and so the
compute support seems to work as expected.
Tested on GK208 and GM107.
Signed-off-by: Samuel Pitoise
Fortunately, compute support on GM107 is very close to GK110, except
the GK110_COMPUTE.UNK02C4 which is invalid and should not be used.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/nv_object.xml.h | 1 +
src/gallium/drivers/nouveau/nvc0/nve4_compute.c | 5 -
2 files cha
On Sat, Feb 13, 2016 at 6:31 PM, Samuel Pitoiset
wrote:
> For some unknown reasons, the command 0x518, which is FIRMWARE[0x6]
Please make a note that the blob calls it, along with the args passed
in (usually via scratch regs iirc). It's probably one of the context
register modifiers. Our firmware
Reviewed-by: Ilia Mirkin
On Sat, Feb 13, 2016 at 6:31 PM, Samuel Pitoiset
wrote:
> Fortunately, compute support on GM107 is very close to GK110, except
> the GK110_COMPUTE.UNK02C4 which is invalid and should not be used.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/nv
Serge Martin writes:
> ---
> src/gallium/state_trackers/clover/core/kernel.cpp | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/state_trackers/clover/core/kernel.cpp
> b/src/gallium/state_trackers/clover/core/kernel.cpp
> index 41b3852..8396be9 100644
>
Reviewed-by: Ilia Mirkin
On Sat, Feb 13, 2016 at 6:31 PM, Samuel Pitoiset
wrote:
> Without this NVF0_COMPUTE environment variable, compute support is
> initialized by default and this is not what we want for now because
> it might break 3D. Compute will be enabled by default on GK110+ once
> we
From: Nicolai Hähnle
Reviewed-by: Ilia Mirkin
---
src/mesa/state_tracker/st_atom_image.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/mesa/state_tracker/st_atom_image.c
b/src/mesa/state_tracker/st_atom_image.c
index 1025725..ff36537 100644
--- a/src/mesa/state_t
Make them akin to shader buffers, with no refcounting/etc. Just used to
pass data about the bound image in ->set_shader_images.
Signed-off-by: Ilia Mirkin
Reviewed-by: Dave Airlie
Reviewed-by: Marek Olšák
---
src/gallium/auxiliary/util/u_inlines.h| 11 ---
src/gallium/drivers/d
From: Nicolai Hähnle
Reviewed-by: Ilia Mirkin
---
src/mesa/state_tracker/st_atom_image.c | 24 +---
1 file changed, 17 insertions(+), 7 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_image.c
b/src/mesa/state_tracker/st_atom_image.c
index ff36537..4b48bc3 100644
-
Signed-off-by: Ilia Mirkin
Reviewed-by: Marek Olšák
Reviewed-by: Dave Airlie
---
src/gallium/auxiliary/tgsi/tgsi_dump.c | 2 +-
src/gallium/auxiliary/tgsi/tgsi_text.c | 13 ++---
2 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_dump.c
b/s
Signed-off-by: Ilia Mirkin
Reviewed-by: Marek Olšák
Reviewed-by: Dave Airlie
---
src/mesa/state_tracker/st_format.h | 8
1 file changed, 8 insertions(+)
diff --git a/src/mesa/state_tracker/st_format.h
b/src/mesa/state_tracker/st_format.h
index 90e00e8..3e10aa6 100644
--- a/src/mesa/s
OK... v2 is here, rebased across the recent compute support and Dave's
AoA changes (and probably other stuff). No significant changes to the
patches themselves.
Dave, your R-b was conditional on addressing your feedback re
writability of surfaces, but I believe you later decided it was OK
as-is, a
Signed-off-by: Ilia Mirkin
Reviewed-by: Marek Olšák
Reviewed-by: Dave Airlie
---
src/gallium/auxiliary/gallivm/lp_bld_limits.h| 1 +
src/gallium/auxiliary/tgsi/tgsi_exec.h | 1 +
src/gallium/docs/source/screen.rst | 1 +
src/gallium/drivers/freedreno/freedreno_screen
Signed-off-by: Ilia Mirkin
Reviewed-by: Marek Olšák
Reviewed-by: Dave Airlie
---
src/mesa/Makefile.sources | 1 +
src/mesa/state_tracker/st_atom.c | 6 +
src/mesa/state_tracker/st_atom.h | 6 +
src/mesa/state_tracker/st_atom_image.c | 222 +
This enables ARB_shader_image_load_store and ARB_shader_image_size when
the backend claims support for these. It will also implicitly enable the
image component of ARB_shader_texture_image_samples.
Signed-off-by: Ilia Mirkin
Reviewed-by: Marek Olšák
Reviewed-by: Dave Airlie
---
src/mesa/state_
Signed-off-by: Ilia Mirkin
Reviewed-by: Dave Airlie
---
src/mesa/main/mtypes.h | 1 +
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 214 -
2 files changed, 211 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtyp
This can happen with 0-sized buffers.
Signed-off-by: Ilia Mirkin
---
src/mesa/state_tracker/st_atom_atomicbuf.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_atomicbuf.c
b/src/mesa/state_tracker/st_atom_atomicbuf.c
index d83c396..a27d
On Sat, Feb 13, 2016 at 3:30 PM, Francisco Jerez wrote:
> Matt Turner writes:
>
>> On Sat, Feb 13, 2016 at 2:02 PM, Francisco Jerez
>> wrote:
>>> Matt Turner writes:
>>>
On Thu, Feb 11, 2016 at 4:41 PM, Matt Turner wrote:
> Gen4/5's SEL instruction cannot use conditional modifiers, s
Reviewed-by: Samuel Pitoiset
On 02/14/2016 01:25 AM, Ilia Mirkin wrote:
This can happen with 0-sized buffers.
Signed-off-by: Ilia Mirkin
---
src/mesa/state_tracker/st_atom_atomicbuf.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom
https://bugs.freedesktop.org/show_bug.cgi?id=77449
Bug 77449 depends on bug 87389, which changed state.
Bug 87389 Summary: [ILK] HL2 & CS:Source water shader broken
https://bugs.freedesktop.org/show_bug.cgi?id=87389
What|Removed |Added
--
The ImageAccess array is statically sized to MAX_IMAGE_UNIFORMS:
GLenum ImageAccess[MAX_IMAGE_UNIFORMS];
There was no bounds checking ensuring we don't overflow. Passing in a
shader with too many uniforms would cause writes to extend into other
fields, such as sh->NumImages.
Later linker che
On Sat, Feb 13, 2016 at 8:27 PM, Kenneth Graunke wrote:
> The ImageAccess array is statically sized to MAX_IMAGE_UNIFORMS:
>
>GLenum ImageAccess[MAX_IMAGE_UNIFORMS];
>
> There was no bounds checking ensuring we don't overflow. Passing in a
> shader with too many uniforms would cause writes to
On Saturday, February 13, 2016 8:30:51 PM PST Ilia Mirkin wrote:
> On Sat, Feb 13, 2016 at 8:27 PM, Kenneth Graunke
wrote:
> > The ImageAccess array is statically sized to MAX_IMAGE_UNIFORMS:
> >
> >GLenum ImageAccess[MAX_IMAGE_UNIFORMS];
> >
> > There was no bounds checking ensuring we don't
On Sat, 2016-02-13 at 17:27 -0800, Kenneth Graunke wrote:
> The ImageAccess array is statically sized to MAX_IMAGE_UNIFORMS:
>
> GLenum ImageAccess[MAX_IMAGE_UNIFORMS];
>
> There was no bounds checking ensuring we don't overflow. Passing in
> a
> shader with too many uniforms would cause writ
On 02/11/2016 12:58 PM, Miklós Máté wrote:
> On 02/09/2016 05:02 PM, Ian Romanick wrote:
>> On 02/08/2016 05:11 PM, Ian Romanick wrote:
>>> On 02/05/2016 01:11 PM, Miklós Máté wrote:
dri drawables must never be released when unbound from a context
as long as their corresponding glx object
This new pass lowers load/store_var intrinsics that act on indirect derefs
to if-ladder of direct load/store_var intrinsics. The if-ladders perform a
simple binary search on the indirect.
---
src/compiler/Makefile.sources| 1 +
src/compiler/nir/Makefile.sources| 1
Previously, nir_dominance.c didn't properly handle unreachable blocks.
This can happen if, for instance, you have something like this:
loop {
if (...) {
break;
} else {
break;
}
}
In this case, the block right after the if statement will be unreachable.
This commit makes two
---
src/util/bitset.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/util/bitset.h b/src/util/bitset.h
index c452819..2404ce7 100644
--- a/src/util/bitset.h
+++ b/src/util/bitset.h
@@ -98,7 +98,7 @@ __bitset_ffs(const BITSET_WORD *x, int n)
static inline unsigned
__bit
This patch series adds actual function support to NIR. Previously all NIR
usage relied on GLSL to have lowered functions away entirely. As a result,
any support for functions that was there was dead code and not really
tested. With SPIR-V on the horizon, NIR will need real function support.
The
---
src/compiler/nir/nir.c | 16
1 file changed, 16 insertions(+)
diff --git a/src/compiler/nir/nir.c b/src/compiler/nir/nir.c
index 0e4d981..cd78475 100644
--- a/src/compiler/nir/nir.c
+++ b/src/compiler/nir/nir.c
@@ -304,6 +304,22 @@ nir_function_impl_create(nir_function *funct
1 - 100 of 129 matches
Mail list logo
