This will be useful for fixing phi srcs when cloning a loop body
during loop unrolling.
---
src/compiler/nir/nir_clone.c | 36 +---
1 file changed, 21 insertions(+), 15 deletions(-)
diff --git a/src/compiler/nir/nir_clone.c b/src/compiler/nir/nir_clone.c
index 0e39
---
src/compiler/Makefile.sources | 1 +
src/compiler/nir/nir.h | 2 +
src/compiler/nir/nir_opt_loop_unroll.c | 443 +
3 files changed, 446 insertions(+)
create mode 100644 src/compiler/nir/nir_opt_loop_unroll.c
diff --git a/src/compil
---
src/compiler/glsl/glsl_parser_extras.cpp | 12 +++-
src/compiler/nir/nir_metadata.c | 2 ++
src/mesa/drivers/dri/i965/brw_compiler.c | 6 +-
src/mesa/drivers/dri/i965/brw_nir.c | 4
4 files changed, 18 insertions(+), 6 deletions(-)
diff --git a/src/compiler/g
---
src/compiler/nir/nir.h | 3 +++
src/compiler/nir/nir_clone.c | 64 +++-
2 files changed, 60 insertions(+), 7 deletions(-)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index c20f46b..0f35a9e 100644
--- a/src/compiler/nir/nir.h
+++
This makes stitch_blocks() available for use else where, and adds
a new helper that extracts a cf list without worrying about
validation.
---
src/compiler/nir/nir_control_flow.c | 34 --
src/compiler/nir/nir_control_flow.h | 5 +
2 files changed, 37 insertions(
This will be useful for cleaning up phis when unrolling loops.
---
src/compiler/nir/nir.h | 1 +
src/compiler/nir/nir_opt_remove_phis.c | 95 +++---
2 files changed, 53 insertions(+), 43 deletions(-)
diff --git a/src/compiler/nir/nir.h b/src/compiler/n
We generate these in each optimisation pass, counting them
as progress would cause the loop to run forever.
---
src/compiler/nir/nir_opt_remove_phis.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/compiler/nir/nir_opt_remove_phis.c
b/src/compiler/nir/nir_opt_remove_phis.c
index ee92f
This will be used by the loop unroll and lcssa passes.
---
src/compiler/nir/nir.h | 8
1 file changed, 8 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index f85e829..6b0a73f 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -2614,6 +2614,14
V2: Do a "depth first search" to convert to LCSSA
V3: Small comment fixup
V4: Rebase, adapt to removal of function overloads
V5: Rebase, adapt to relocation of nir to compiler/nir
Still need to adapt to potential if-uses
Work around nir_validate issue
V6 (Timothy):
- tidy lcssa and sto
This series does the equivalent of the simple unroll in GLSL IR. The
main difference is that currently it unrolls everything with an
iteration count < 32, the GLSL IR pass also tries to limit unrolling
based on the number of nodes it contains but this seemed a little simplistic
so I have left this
This pass detects induction variables and calculates the
trip count of loops to be used for loop unrolling.
I've removed support for float induction values for now, for the
simple reason that they don't appear in my shader-db collection,
and so I don't see it as common enough that we want to pollu
Don't do copy propagation inside loops until after we try
unrolling them.
This helps avoid propagating everything to the phis which
makes loop unrolling more difficult.
For example without this:
loop {
block block_1:
/* preds: block_0 block_4 */
vec1 32 ssa_10 = phi block_0:
This moves the nir_lower_indirect_derefs() call into
brw_preprocess_nir() so thats is called by both OpenGL and Vulkan
and removes that call to the old GLSL IR pass
lower_variable_index_to_cond_assign()
We want to do this pass in nir to be able to move loop unrolling
to nir.
There is a increase o
This fixes a numerical precision issue that was causing two CTS
failures:
ES31-CTS.blend_equation_advanced.blend_specific.GL_COLORBURN_KHR
ES31-CTS.blend_equation_advanced.blend_all.GL_COLORBURN_KHR_all_qualifier
When blending with GL_COLORDODGE_KHR and these colors:
dst = <0.372549027, 0.372
Hi,
I'd like to propose a conversion of Mesa's documentation to
reStructuredText (RST) and hosting the result on readthedocs.org. The
intent is to make Mesa's documentation more accessible, searchable, and
easier to edit.
I put together a quick proof-of-concept here:
http://mesa-docs.readthe
The range from ANV_MIN_STATE_SIZE_LOG2 to ANV_MAX_STATE_SIZE_LOG2 should
be inclusive and we have asserts that ensure that you never try to allocate
a state larger than (1 << ANV_MAX_STATE_SIZE_LOG2). However, without
adding 1 to the difference, we allocate 1 too few bucckts and so, even
though we
This sanitizes blorp's access to the i965 driver's shader cache by patching
it through the blorp_context. When we start using blorp in Vulkan, we will
simply have to implement such a caching interface in the Vulkan driver.
Note: In my first attempt at this, I simplified it down to a single
upload
On Thu, Aug 25, 2016 at 10:15 PM, Jason Ekstrand
wrote:
> On Aug 25, 2016 9:41 PM, "Pohjolainen, Topi"
> wrote:
> >
> > On Thu, Aug 25, 2016 at 02:34:37PM -0700, Jason Ekstrand wrote:
> > >On Thu, Aug 25, 2016 at 1:10 AM, Pohjolainen, Topi
> > ><[1]topi.pohjolai...@gmail.com> wrote:
> >
If you wanted to be extra spiffy, you could add a discard helper to
nir_builder and make it do it automatically :)
In any case,
Reviewed-by: Jason Ekstrand
On Fri, Aug 26, 2016 at 6:13 PM, Eric Anholt wrote:
> vc4 is about to start using the shader info field to set up discard
> handling.
> -
The original pipeline cache the Kristian wrote was based on a now-false
premise that the shaders can be stored in the pipeline cache. The Vulkan
1.0 spec explicitly states that the pipeline cache object is transiant and
you are allowed to delete it after using it to create a pipeline with no
ill e
This new anv_shader_bin struct stores the compiled kernel (as an anv_state)
as well as all of the metadata that is generated at shader compile time.
The struct is very similar to the old cache_entry struct except that it
is reference counted and stores the actual pipeline_bind_map. Similarly to
ca
vc4 is about to start using the shader info field to set up discard
handling.
---
src/compiler/nir/nir_lower_bitmap.c | 2 ++
src/compiler/nir/nir_lower_clip.c | 2 ++
2 files changed, 4 insertions(+)
diff --git a/src/compiler/nir/nir_lower_bitmap.c
b/src/compiler/nir/nir_lower_bitmap.c
index
On Fri 19 Aug 2016, Sirisha Gandikota wrote:
> From: Sirisha Gandikota
>
> This is a patch series for adding the aubinator tool to the codebase.
>
> The aubinator tool is designed to help the driver developers to debug
> the driver functionality by decoding the data in the .aub files. This
> too
Patches 2, 3 are
Reviewed-by: Chad Versace
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Aug 26, 2016 2:47 PM, "Chad Versace" wrote:
>
> On Fri 26 Aug 2016, Kenneth Graunke wrote:
> > Gen6 only has one additional restriction over Gen7+, so we just add it
> > to the existing gen7 function (which actually covers later gens too).
> >
> > This should stop FINISHME spew when running GL
On Fri 26 Aug 2016, Pohjolainen, Topi wrote:
> On Thu, Aug 25, 2016 at 03:58:24PM -0700, Jason Ekstrand wrote:
> > This probably isn't the only thing that needs to be done to get
> > multisampled array textures working in Vulkan but I think this is all that
> > ISL really needs and it does fix 8 of
On Fri 26 Aug 2016, Nicholas Bishop wrote:
> From: Nicholas Bishop
>
> * Changed "Mesa mailing list" to "mesa-dev mailing list" to clarify
> which list patches should be sent to
>
> * Added an explicit link to
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev to show
> where to sub
On Fri 26 Aug 2016, Kenneth Graunke wrote:
> Gen6 only has one additional restriction over Gen7+, so we just add it
> to the existing gen7 function (which actually covers later gens too).
>
> This should stop FINISHME spew when running GL on Sandybridge.
>
> Signed-off-by: Kenneth Graunke
> ---
Gen6 only has one additional restriction over Gen7+, so we just add it
to the existing gen7 function (which actually covers later gens too).
This should stop FINISHME spew when running GL on Sandybridge.
Signed-off-by: Kenneth Graunke
---
src/intel/isl/isl.c | 2 +-
src/intel/isl/isl_gen7.
On 26/08/16 16:27, Brian Paul wrote:
Move computation of zslice, layer inside the conditional where they're
used.
---
src/gallium/drivers/svga/svga_surface.c | 37 +
1 file changed, 19 insertions(+), 18 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_surf
From: Nicholas Bishop
* Changed "Mesa mailing list" to "mesa-dev mailing list" to clarify
which list patches should be sent to
* Added an explicit link to
https://lists.freedesktop.org/mailman/listinfo/mesa-dev to show
where to subscribe to the list
* Added a link to https://git-scm.com/d
Hi Kai,
Am 26.08.2016 um 15:23 schrieb Kai Wasserbäch:
> v1 → v2:
> - Fixed indentation (noted by Brian Paul)
> - Removed second assert from nouveau's switch statements (suggested by
>Brian Paul)
>
> Signed-off-by: Kai Wasserbäch
> ---
> src/gallium/auxiliary/cso_cache/cso_context.c |
We were allocating global variables for the maximum LDS size
which made the compiler think we were using all of LDS, which
isn't the case.
---
src/gallium/drivers/radeonsi/si_shader.c | 15 ++-
1 file changed, 6 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/s
Iago Toral Quiroga writes:
> This hack was introduced in commit 03ac2c7223f7645e3:
> i965/gs: Fix up gl_PointSize input swizzling for DUAL_INSTANCED gs
>
> Specifically to fixup the code we emitted to deal with gl_PointSize inputs
> in dual instance mode, where we were emitting a MOV to copy the
On Fri, Aug 26, 2016 at 3:08 PM, Miklós Máté wrote:
> On 08/26/2016 07:47 PM, Ilia Mirkin wrote:
>>
>> On Fri, Aug 26, 2016 at 1:39 PM, Miklós Máté wrote:
>>>
>>> On 08/26/2016 06:46 PM, Ilia Mirkin wrote:
On Fri, Aug 26, 2016 at 12:42 PM, Miklós Máté wrote:
>
> Tomb Raider 201
On Fri, Aug 26, 2016 at 3:01 PM, Ryan Houdek wrote:
> Most of the Tegra devices (K1 and above) provide desktop GL, except for the
> Nexus devices which cut out that functionality.
> Not sure how front buffers differ there, never checked.
> Dolphin relies on a large amount of extensions, both for p
On 08/26/2016 07:47 PM, Ilia Mirkin wrote:
On Fri, Aug 26, 2016 at 1:39 PM, Miklós Máté wrote:
On 08/26/2016 06:46 PM, Ilia Mirkin wrote:
On Fri, Aug 26, 2016 at 12:42 PM, Miklós Máté wrote:
Tomb Raider 2013 uses #version 420 in compute shaders, and current Mesa
rejects them, because the loc
Most of the Tegra devices (K1 and above) provide desktop GL, except for the
Nexus devices which cut out that functionality.
Not sure how front buffers differ there, never checked.
Dolphin relies on a large amount of extensions, both for performance and
proper emulation standpoint..
For performance,
On 08/26/2016 01:06 AM, Ilia Mirkin wrote:
Although "silences" is more accurate than "fixes".
Yes, I will slightly improve the description before pushing.
Thanks.
On Thu, Aug 25, 2016 at 7:05 PM, Ilia Mirkin wrote:
Reviewed-by: Ilia Mirkin
On Thu, Aug 25, 2016 at 12:41 PM, Samuel Pitoi
Needed to successfully link llvmpipe or swr when using shared llvm libs
built with inteljitevents enabled.
v2: Make adding inteljitevents component global rather than just
llvmpipe/swr, since libgallium will have a symbol dependency.
---
configure.ac | 4
1 file changed, 4 insertions(+)
dif
On 08/26/2016 04:17 PM, Kai Wasserbäch wrote:
Hey Samuel,
Samuel Pitoiset wrote on 26.08.2016 15:54:
On 08/26/2016 01:58 PM, Kai Wasserbäch wrote:
[...]
diff --git a/src/gallium/drivers/nouveau/nv30/nv30_texture.c
b/src/gallium/drivers/nouveau/nv30/nv30_texture.c
index 4f4f87e..dc1a476 100644
Applied. Thanks!
Marek
On Fri, Aug 26, 2016 at 6:59 PM, Mario Kleiner
wrote:
> This is a direct port of Marek Olšáks patch
> "radeonsi: increase performance for DRI PRIME
> offloading if 2nd GPU is CIK or VI" to r600.
>
> It uses SDMA for the detiling blit from renderoffload VRAM
> to GTT, as SD
On Fri, Aug 26, 2016 at 1:39 PM, Miklós Máté wrote:
> On 08/26/2016 06:46 PM, Ilia Mirkin wrote:
>>
>> On Fri, Aug 26, 2016 at 12:42 PM, Miklós Máté wrote:
>>>
>>> Tomb Raider 2013 uses #version 420 in compute shaders, and current Mesa
>>> rejects them, because the local size qualifiers require 4
On 08/26/2016 06:46 PM, Ilia Mirkin wrote:
On Fri, Aug 26, 2016 at 12:42 PM, Miklós Máté wrote:
Tomb Raider 2013 uses #version 420 in compute shaders, and current Mesa
rejects them, because the local size qualifiers require 430.
Signed-off-by: Miklós Máté
---
src/compiler/glsl/glsl_parser.y
This is a direct port of Marek Olšáks patch
"radeonsi: increase performance for DRI PRIME
offloading if 2nd GPU is CIK or VI" to r600.
It uses SDMA for the detiling blit from renderoffload VRAM
to GTT, as SDMA is much faster for tiled->linear blits from
VRAM to GTT.
Testing on a dual Radeon HD-57
> On Aug 26, 2016, at 8:30 AM, Emil Velikov wrote:
>
> On 2 August 2016 at 18:54, Tim Rowley wrote:
>> Needed to successfully link llvmpipe or swr when using shared llvm libs.
>> ---
>> configure.ac | 5 +
>> 1 file changed, 5 insertions(+)
>>
>> diff --git a/configure.ac b/configure.ac
>>
On Fri, Aug 26, 2016 at 12:42 PM, Miklós Máté wrote:
> Tomb Raider 2013 uses #version 420 in compute shaders, and current Mesa
> rejects them, because the local size qualifiers require 430.
>
> Signed-off-by: Miklós Máté
> ---
> src/compiler/glsl/glsl_parser.yy | 5 ++---
> 1 file changed, 2 ins
Tomb Raider 2013 uses #version 420 in compute shaders, and current Mesa
rejects them, because the local size qualifiers require 430.
Signed-off-by: Miklós Máté
---
src/compiler/glsl/glsl_parser.yy | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/compiler/glsl/glsl_par
On Fri, Aug 26, 2016 at 04:21:14PM +0200, Kai Wasserbäch wrote:
> Hey Eric,
> Eric Engestrom wrote on 26.08.2016 15:49:
> > On Fri, Aug 26, 2016 at 03:14:57PM +0200, Kai Wasserbäch wrote:
> >> Brian Paul wrote on 26.08.2016 14:50:
> >>> On 08/26/2016 05:58 AM, Kai Wasserbäch wrote:
> Cc: Brian
Series looks good me.
Reviewed-by: Neha Bhende
Regards,
Neha
From: Brian Paul
Sent: Friday, August 26, 2016 8:27:18 AM
To: mesa-dev@lists.freedesktop.org
Cc: Neha Bhende; Jose Fonseca
Subject: [PATCH 14/14] svga: move some code in svga_propagate_surface()
Move computation of zslice, layer inside the conditional where they're
used.
---
src/gallium/drivers/svga/svga_surface.c | 37 +
1 file changed, 19 insertions(+), 18 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_surface.c
b/src/gallium/drivers/svga/svga_
---
src/gallium/drivers/svga/svga_sampler_view.h | 4
src/gallium/drivers/svga/svga_state_sampler.c | 33 ++-
2 files changed, 36 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/svga/svga_sampler_view.h
b/src/gallium/drivers/svga/svga_sampler_view.h
i
Use the tex variable instead of using svga_texture() again.
---
src/gallium/drivers/svga/svga_resource_texture.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_resource_texture.c
b/src/gallium/drivers/svga/svga_resource_texture.c
index aff6fb1.
Use local vars instead of jumping through a pointer.
---
src/gallium/drivers/svga/svga_resource_texture.c | 18 +-
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_resource_texture.c
b/src/gallium/drivers/svga/svga_resource_texture.c
inde
Rewrite the comment too.
---
src/gallium/drivers/svga/svga_pipe_misc.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_pipe_misc.c
b/src/gallium/drivers/svga/svga_pipe_misc.c
index a26e577..7160154 100644
--- a/src/gallium/driver
---
src/gallium/drivers/svga/svga_pipe_sampler.c | 21 +++--
1 file changed, 3 insertions(+), 18 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_pipe_sampler.c
b/src/gallium/drivers/svga/svga_pipe_sampler.c
index 59609b8..42e4fa5 100644
--- a/src/gallium/drivers/svga/svga
We don't do this for other cast wrappers. And this will simplify some
code at call sites.
---
src/gallium/drivers/svga/svga_surface.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_surface.h
b/src/gallium/drivers/svga/svga_surface.h
index 0e5794b..c166791 1006
Give more info about backing resources/surfaces.
---
src/gallium/drivers/svga/svga_surface.h | 16
1 file changed, 16 insertions(+)
diff --git a/src/gallium/drivers/svga/svga_surface.h
b/src/gallium/drivers/svga/svga_surface.h
index c166791..2f003b2 100644
--- a/src/gallium/driv
---
src/gallium/drivers/svga/svga_resource_texture.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_resource_texture.c
b/src/gallium/drivers/svga/svga_resource_texture.c
index 450ed2b..aff6fb1 100644
--- a/src/gallium/drivers/svga/svga_reso
Put near other assignments to the svga_transfer variable.
---
src/gallium/drivers/svga/svga_resource_texture.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_resource_texture.c
b/src/gallium/drivers/svga/svga_resource_texture.c
index 243b
---
src/gallium/drivers/svga/svga_resource_texture.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/svga/svga_resource_texture.h
b/src/gallium/drivers/svga/svga_resource_texture.h
index e779f19..ffd5fea 100644
--- a/src/gallium/drivers/svga/svga_resource
tex was already declared at the function body scope.
---
src/gallium/drivers/svga/svga_resource_texture.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/drivers/svga/svga_resource_texture.c
b/src/gallium/drivers/svga/svga_resource_texture.c
index 9104b0a..450ed2b 100644
--- a/src/
To simplify the code a bit.
---
src/gallium/drivers/svga/svga_surface.c | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_surface.c
b/src/gallium/drivers/svga/svga_surface.c
index 43a00c3..4b0b56c 100644
--- a/src/gallium/drivers/svga/sv
To make it symmetric with the svga_texture() cast wrapper.
---
src/gallium/drivers/svga/svga_resource_buffer.h | 10 --
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_resource_buffer.h
b/src/gallium/drivers/svga/svga_resource_buffer.h
index 059
Hey Eric,
Eric Engestrom wrote on 26.08.2016 15:49:
> On Fri, Aug 26, 2016 at 03:14:57PM +0200, Kai Wasserbäch wrote:
>> Brian Paul wrote on 26.08.2016 14:50:
>>> On 08/26/2016 05:58 AM, Kai Wasserbäch wrote:
Cc: Brian Paul
Signed-off-by: Kai Wasserbäch
---
Hi Brian,
Hey Samuel,
Samuel Pitoiset wrote on 26.08.2016 15:54:
> On 08/26/2016 01:58 PM, Kai Wasserbäch wrote:
>> [...]
>> diff --git a/src/gallium/drivers/nouveau/nv30/nv30_texture.c
>> b/src/gallium/drivers/nouveau/nv30/nv30_texture.c
>> index 4f4f87e..dc1a476 100644
>> --- a/src/gallium/drivers/nouveau/
On 08/26/2016 07:48 AM, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 4
src/gallium/auxiliary/tgsi/tgsi_scan.h | 1 +
2 files changed, 5 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
ind
Build mesa 2025 completed
Commit 0035f7f136 by Charmaine Lee on 8/26/2016 1:58 PM:
svga: add guest statistic gathering interface\n\nThis file was supposed to be added with the previous "svga: add guest\nstatistic gathering interface" patch but went MIA for some
On 08/26/2016 01:58 PM, Kai Wasserbäch wrote:
Cc: Brian Paul
Signed-off-by: Kai Wasserbäch
---
Hi Brian,
is this what you had in mind? If so, I was wondering whether virgl_encode.c
would need to be updated as well. Doesn't seem like it, since the functions
there map everything to uint32_t or
On Fri, Aug 26, 2016 at 03:14:57PM +0200, Kai Wasserbäch wrote:
> Hey Brian,
> Brian Paul wrote on 26.08.2016 14:50:
> > On 08/26/2016 05:58 AM, Kai Wasserbäch wrote:
> >> Cc: Brian Paul
> >> Signed-off-by: Kai Wasserbäch
> >> ---
> >>
> >> Hi Brian,
> >> is this what you had in mind? If so, I wa
From: Marek Olšák
---
src/gallium/auxiliary/tgsi/tgsi_scan.c | 4
src/gallium/auxiliary/tgsi/tgsi_scan.h | 1 +
2 files changed, 5 insertions(+)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_scan.c
b/src/gallium/auxiliary/tgsi/tgsi_scan.c
index 0167e22..a3b0d9f 100644
--- a/src/gallium/aux
On Fri, Aug 26, 2016 at 12:42 PM, Bas Nieuwenhuizen
wrote:
> I would prefer it if the function could be split in two functions
> instead of using the initialized flag. I think we know whether it is
> an initialization or a reinitialization per call site.
OK. The updated patch will be sent as part
Build mesa 2023 failed
Commit b9ac72b511 by Miklós Máté on 8/26/2016 12:48 PM:
vbo: set draw_id\n\nFixes conditional jump depending on uninitialized value\nin si_state_draw.c:593\n\nCc: \nSigned-off-by: Miklós Máté \nReviewed-by: Brian Paul
Configure
On Fri, Aug 26, 2016 at 06:50:52AM -0600, Brian Paul wrote:
> On 08/26/2016 05:58 AM, Kai Wasserbäch wrote:
> > Cc: Brian Paul
> > Signed-off-by: Kai Wasserbäch
> > ---
> >
> > Hi Brian,
> > is this what you had in mind? If so, I was wondering whether virgl_encode.c
> > would need to be updated
On 2 August 2016 at 18:54, Tim Rowley wrote:
> Needed to successfully link llvmpipe or swr when using shared llvm libs.
> ---
> configure.ac | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index fb4a12a..edbc95b 100644
> --- a/configure.ac
> +++ b/confi
v1 → v2:
- Fixed indentation (noted by Brian Paul)
- Removed second assert from nouveau's switch statements (suggested by
Brian Paul)
Signed-off-by: Kai Wasserbäch
---
src/gallium/auxiliary/cso_cache/cso_context.c | 7 ---
src/gallium/auxiliary/cso_cache/cso_context.h | 5 +++-
Hey Brian,
Brian Paul wrote on 26.08.2016 14:50:
> On 08/26/2016 05:58 AM, Kai Wasserbäch wrote:
>> Cc: Brian Paul
>> Signed-off-by: Kai Wasserbäch
>> ---
>>
>> Hi Brian,
>> is this what you had in mind? If so, I was wondering whether virgl_encode.c
>> would need to be updated as well. Doesn't se
On 26 August 2016 at 08:50, Tapani Pälli wrote:
> Reviewed-by: Tapani Pälli
>
What happened with my suggestion about getting things fixed as opposed
to adding tape over things, namely these thread [1] ?
Can someone please look into that one instead or give me some tips how
I can get things into A
On 08/26/2016 02:53 PM, Brian Paul wrote:
On 08/26/2016 06:48 AM, Miklós Máté wrote:
Fixes conditional jump depending on uninitialized value
in si_state_draw.c:593
Cc:
Signed-off-by: Miklós Máté
---
src/mesa/vbo/vbo_exec_array.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa
On 08/26/2016 06:48 AM, Miklós Máté wrote:
Fixes conditional jump depending on uninitialized value
in si_state_draw.c:593
Cc:
Signed-off-by: Miklós Máté
---
src/mesa/vbo/vbo_exec_array.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_ex
On 08/26/2016 05:58 AM, Kai Wasserbäch wrote:
Cc: Brian Paul
Signed-off-by: Kai Wasserbäch
---
Hi Brian,
is this what you had in mind? If so, I was wondering whether virgl_encode.c
would need to be updated as well. Doesn't seem like it, since the functions
there map everything to uint32_t or s
On 26 August 2016 at 12:27, Tapani Pälli wrote:
> Android.mk files were referring to LOCAL_PATH instead of path
> where wanted Makefile.sources was existing. Also minor cleanups.
>
This will break the automake build. Please port commit
bebc1a1d995e33f173ea207848bcddd81f6dd19a to the Android build.
Fixes conditional jump depending on uninitialized value
in si_state_draw.c:593
Cc:
Signed-off-by: Miklós Máté
---
src/mesa/vbo/vbo_exec_array.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/vbo/vbo_exec_array.c b/src/mesa/vbo/vbo_exec_array.c
index 1997039..46543f8 100644
--- a/
Cc: Brian Paul
Signed-off-by: Kai Wasserbäch
---
Hi Brian,
is this what you had in mind? If so, I was wondering whether virgl_encode.c
would need to be updated as well. Doesn't seem like it, since the functions
there map everything to uint32_t or some other standard type.
Another point are the
2016-08-26 12:27 GMT+02:00 Tapani Pälli :
> Hi;
>
> This is patch is causing build issues for me, how was this tested?
>
Hi,
thanks for highlighting the issue,
My patch was tested and reviewed, the current android build breakage due to
following commit, upstreamed 13 hours ago:
https://cgit.fre
On Fri, Aug 26, 2016 at 12:22 PM, Bas Nieuwenhuizen
wrote:
> Patch 3, 4 are
>
> Reviewed-by: Bas Nieuwenhuizen
Thanks.
>
> For patch 1 and 2 I'm not familiar with the difference between the SI
> and CIK+ tiling modes to be sure they are good, but the other parts of
> the patch look good to me.
Android.mk files were referring to LOCAL_PATH instead of path
where wanted Makefile.sources was existing. Also minor cleanups.
Signed-off-by: Tapani Pälli
---
src/intel/Makefile.sources | 62 ++---
src/intel/genxml/Android.mk | 4 +--
src/intel/isl/Andro
On 08/26/2016 01:27 PM, Tapani Pälli wrote:
Hi;
This is patch is causing build issues for me, how was this tested?
(I'll send a patch proposal to fix the issues I encountered in a sec)
See below for 2 questions:
On 05/31/2016 01:20 AM, Mauro Rossi wrote:
isl library is needed to build i96
On Thu, Aug 25, 2016 at 05:18:22PM +0100, Emil Velikov wrote:
> Hi all,
>
> With the resent noise in the egl area I decided to do some of the long
> planned cleanup in the area. It spans across the following:
>
> - glapi missing glFlush and non-shared glapi are not an option
> - encapsulate/se
Patch 1-7 of this series are
Reviewed-by: Bas Nieuwenhuizen
- Bas
(Adding mesa-dev ML this time)
On Thu, Aug 18, 2016 at 9:46 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> If the kernel driver doesn't support it, it returns 0.
> ---
> src/gallium/drivers/radeon/r600_query.c | 8 +
On 25 August 2016 at 18:53, Gurchetan Singh wrote:
>>> unsigned int format_count[ARRAY_SIZE(visuals)] = {};
>
> Isn't this invalid in C?
> http://stackoverflow.com/questions/17589533/is-an-empty-initializer-list-valid-c-code
>
Since we have a bunch of these around (here, xserver, libdrm, IGT...)
I
On Wed, Aug 24, 2016 at 12:50:33PM +0100, Emil Velikov wrote:
> From: Emil Velikov
>
> Ported from the i965 commit e7ab358e8186dd8651cf920d4db1500c60ccd2fc.
>
> Cc: 11.2 12.0
> Cc: Tomasz Figa
> Signed-off-by: Emil Velikov
Reviewed-by: Eric Engestrom
> ---
> src/mesa/drivers/dri/i915/inte
On 25 August 2016 at 18:16, Kristian Høgsberg wrote:
> On Thu, Aug 25, 2016 at 9:18 AM, Emil Velikov
> wrote:
>> From: Emil Velikov
>>
>> Signed-off-by: Emil Velikov
>> ---
>> src/egl/drivers/dri2/egl_dri2.c | 28 ++--
>> 1 file changed, 10 insertions(+), 18 deletions(
I would prefer it if the function could be split in two functions
instead of using the initialized flag. I think we know whether it is
an initialization or a reinitialization per call site.
- Bas
On Thu, Aug 18, 2016 at 9:46 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> Invalidated buffers don
On Thu, Aug 25, 2016 at 04:58:58PM +0100, Emil Velikov wrote:
> From: Emil Velikov
>
> Function was never part of the API/ABI and the final user was removed
> with commit a73c6540d9a7f6e26d8568ba2fc522cb865f0a6c, back in 2010.
>
> Signed-off-by: Emil Velikov
This series is
Reviewed-by: Eric En
On 25 August 2016 at 18:06, Kristian Høgsberg wrote:
> On Thu, Aug 25, 2016 at 9:18 AM, Emil Velikov
> wrote:
>> From: Emil Velikov
>>
>> The dri3 version of commits 60e9c35b3a0 and 6de9a03bed4.
>>
>> CC:
>> Signed-off-by: Emil Velikov
>> ---
>> src/egl/drivers/dri2/platform_x11.c | 14 +
Hi;
This is patch is causing build issues for me, how was this tested?
See below for 2 questions:
On 05/31/2016 01:20 AM, Mauro Rossi wrote:
isl library is needed to build i965, libmesa_isl static library is added
to fix related Android building errors.
Any attempt to build libmesa_genxml as
Patch 3, 4 are
Reviewed-by: Bas Nieuwenhuizen
For patch 1 and 2 I'm not familiar with the difference between the SI
and CIK+ tiling modes to be sure they are good, but the other parts of
the patch look good to me.
btw, I'm not sure what you tried wrt the CE hang, but it may be worth
trying with
On Fri, Aug 26, 2016 at 6:48 AM, Rob Clark wrote:
> On Thu, Aug 25, 2016 at 8:00 PM, Brian Paul wrote:
>> If someone is looking for a simple task in gallium...
>>
>> In commit 0135bd44 Marek introduced some new enum types in p_defines.h
>>
>> In p_context.h we should replace several instances of
This hack was introduced in commit 03ac2c7223f7645e3:
i965/gs: Fix up gl_PointSize input swizzling for DUAL_INSTANCED gs
Specifically to fixup the code we emitted to deal with gl_PointSize inputs
in dual instance mode, where we were emitting a MOV to copy the point
size from .w (where the hardware
1 - 100 of 112 matches
Mail list logo
