On Tue, Aug 30, 2016 at 12:40 AM, Matt Turner wrote:
> On Mon, Aug 29, 2016 at 9:06 PM, Timothy Arceri
> wrote:
>> Can't the phi have more than one source from before the loop? e.g
>>
>>int i = 0;
>>if (somthing)
>> i = 1;
>>else
On 08/30/2016 07:59 AM, Tapani Pälli wrote:
On 08/29/2016 07:11 PM, Emil Velikov wrote:
On 29 August 2016 at 05:37, Tapani Pälli wrote:
On 08/26/2016 03:58 PM, Emil Velikov wrote:
On 26 August 2016 at 08:50, Tapani Pälli
wrote:
On 08/30/2016 09:20 AM, Vedran Miletić wrote:
> Options specified via the CLOVER_COMPILER_OPTIONS shell variable are
> appended to the compiler options specified by the OpenCL program (if
> any).
>
> Signed-off-by: Vedran Miletić
Reviewed-by: Edward O'Callaghan
On 08/29/2016 07:11 PM, Emil Velikov wrote:
On 29 August 2016 at 05:37, Tapani Pälli wrote:
On 08/26/2016 03:58 PM, Emil Velikov wrote:
On 26 August 2016 at 08:50, Tapani Pälli wrote:
Reviewed-by: Tapani Pälli
On Mon, Aug 29, 2016 at 9:06 PM, Timothy Arceri
wrote:
> Can't the phi have more than one source from before the loop? e.g
>
>int i = 0;
>if (somthing)
> i = 1;
>else
> i = 2;
>
>for ( ; i < 5; i++)
> do_stuff(i);
In fact, no. :)
On Mon, 2016-08-29 at 21:06 -0400, Connor Abbott wrote:
> So, you've noticed that your method of handling phi's while cloning
> doesn't handle phi's that point to other phi's in the same block.
> Particularly, something like:
>
> a = phi(b, ...)
> b = phi(a, ...)
>
> which is supposed to swap a
On Tue, 2016-08-30 at 14:16 +1000, Timothy Arceri wrote:
> On Tue, 2016-08-30 at 14:06 +1000, Timothy Arceri wrote:
> >
> > On Mon, 2016-08-29 at 20:34 -0400, Connor Abbott wrote:
> > >
> > >
> > > On Mon, Aug 29, 2016 at 12:59 AM, Timothy Arceri
> > > wrote:
> >
On Tue, 2016-08-30 at 14:06 +1000, Timothy Arceri wrote:
> On Mon, 2016-08-29 at 20:34 -0400, Connor Abbott wrote:
> >
> > On Mon, Aug 29, 2016 at 12:59 AM, Timothy Arceri
> > wrote:
> > >
> > >
> > > ---
> > > src/compiler/Makefile.sources | 1 +
> > >
On Mon, 2016-08-29 at 20:34 -0400, Connor Abbott wrote:
> On Mon, Aug 29, 2016 at 12:59 AM, Timothy Arceri
> wrote:
> >
> > ---
> > src/compiler/Makefile.sources | 1 +
> > src/compiler/nir/nir.h | 2 +
> >
On Mon, 2016-08-29 at 20:42 -0400, Connor Abbott wrote:
> I already noted in patch 12/13 that you can get rid of your use of
> stitch_blocks(). I also don't get why you need a special
> nir_cf_loop_list_extract() here... why
>
> On Mon, Aug 29, 2016 at 12:59 AM, Timothy Arceri
>
Caught by Coverity. Likely fixes real issues if an output component
is not present.
CID: 1372278
Signed-off-by: Kenneth Graunke
---
src/compiler/glsl/lower_blend_equation_advanced.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Hi Dave and all,
This version is based on Dave's review comments. Could you please
review it?
Thanks!
-Original Message-
From: Weng, Chuanbo
Sent: Friday, August 26, 2016 1:51 AM
To: mesa-dev@lists.freedesktop.org; airl...@gmail.com
Cc: emil.l.veli...@gmail.com; Weng,
On Mon, Aug 29, 2016 at 2:30 AM, Timothy Arceri
wrote:
> On Mon, 2016-08-29 at 01:30 -0400, Connor Abbott wrote:
>> On Mon, Aug 29, 2016 at 12:54 AM, Timothy Arceri
>> wrote:
>> >
>> > Don't do copy propagation inside loops until after
So, you've noticed that your method of handling phi's while cloning
doesn't handle phi's that point to other phi's in the same block.
Particularly, something like:
a = phi(b, ...)
b = phi(a, ...)
which is supposed to swap a and b each iteration. Here's a better
strategy which should be simpler
I already noted in patch 12/13 that you can get rid of your use of
stitch_blocks(). I also don't get why you need a special
nir_cf_loop_list_extract() here... why
On Mon, Aug 29, 2016 at 12:59 AM, Timothy Arceri
wrote:
> This makes stitch_blocks() available for use
On Mon, Aug 29, 2016 at 12:59 AM, Timothy Arceri
wrote:
> ---
> src/compiler/Makefile.sources | 1 +
> src/compiler/nir/nir.h | 2 +
> src/compiler/nir/nir_opt_loop_unroll.c | 394
> +
> 3 files changed,
On Tuesday, August 30, 2016 9:53:35 AM PDT Timothy Arceri wrote:
> Fixes misleading indentation warning in gcc.
>
> Cc: Kristian Høgsberg Kristensen
> Cc: Kenneth Graunke
> ---
> src/intel/tools/disasm.c | 7 ---
> 1 file changed, 4 insertions(+),
Fixes misleading indentation warning in gcc.
Cc: Kristian Høgsberg Kristensen
Cc: Kenneth Graunke
---
src/intel/tools/disasm.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/intel/tools/disasm.c
From: Dave Airlie
At least set this to not be uninitialised memory.
Signed-off-by: Dave Airlie
---
src/intel/vulkan/anv_wsi_x11.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/intel/vulkan/anv_wsi_x11.c b/src/intel/vulkan/anv_wsi_x11.c
index
Options specified via the CLOVER_COMPILER_OPTIONS shell variable are
appended to the compiler options specified by the OpenCL program (if
any).
Signed-off-by: Vedran Miletić
---
docs/envvars.html | 2 ++
On Mon, Aug 29, 2016 at 10:40 PM, Dave Airlie wrote:
>>
>> There are also some cosmetic changes.
>
> Did I miss 0/2? is there some future use for this in mesa?
>
> I assume there is, just wanting more info :)
It was for an experiment that turned out to be a disaster. There is
calculate_attr_overrides() uses is_drawing_points(), which depends
on tessellation and geometry program state.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen6_clip_state.c | 4
src/mesa/drivers/dri/i965/gen7_sf_state.c | 13 ++---
State upload code should use prog_data rather than poking at core
Mesa shader data structures wherever possible.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen6_clip_state.c | 16
src/mesa/drivers/dri/i965/gen6_sf_state.c | 8 +---
On Mon, Aug 29, 2016 at 11:44 AM, Brian Paul wrote:
>
> We need to set the need_convert flag with each loop iteration, not
> just when the rgba pointer is null.
>
> Bug reported by Markus Müller on mesa-users list.
> Fixes new piglit
On Monday 29 August 2016 22:30:58 Eric Engestrom wrote:
> A few weeks ago, Jose Fonseca suggested [0] we use .editorconfig files
> to try and enforce the formatting of the code, to which Michel Dänzer
> suggested [1] we start by importing the existing .dir-locals.el
> settings. The first draft was
A few weeks ago, Jose Fonseca suggested [0] we use .editorconfig files
to try and enforce the formatting of the code, to which Michel Dänzer
suggested [1] we start by importing the existing .dir-locals.el
settings. The first draft was discussed in the RFC [2].
These .editorconfig are a first
>
> There are also some cosmetic changes.
Did I miss 0/2? is there some future use for this in mesa?
I assume there is, just wanting more info :)
Dave.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
On 30 August 2016 at 01:28, Marek Olšák wrote:
> From: Marek Olšák
>
> This fixes: GL43-CTS.texture_view.view_sampling
Reviewed-by: Dave Airlie
>
> Cc: mesa-sta...@lists.freedesktop.org
> ---
> src/gallium/drivers/radeonsi/si_state.c
On 30 August 2016 at 01:28, Marek Olšák wrote:
> From: Marek Olšák
>
> The closed compiler does the same thing.
>
> This fixes: GL45-CTS.texture_gather.*-int-* (18 tests)
Also reused in radv, and works there to fix a bunch of gather int tests.
On 30 August 2016 at 01:28, Marek Olšák wrote:
> From: Marek Olšák
>
> Sometimes it was f32, other times it was i32. Now it's always i32.
I think this is pretty much what radv does in the same situation.
Reviewed-by: Dave Airlie
>
>
On Mon, Aug 29, 2016 at 5:28 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> Calculate depth ranges from viewport states and
> pipe_rasterizer_state::clip_halfz.
>
> The evergreend.h change is required to silence a warning.
>
> This fixes this recently
On Aug 29, 2016 12:06 PM, "Matt Turner" wrote:
>
> On Sun, Aug 28, 2016 at 7:13 PM, Timothy Arceri
> wrote:
> > Fixes uninitialised warning for coord_components.
> > ---
> > src/compiler/spirv/spirv_to_nir.c | 2 +-
> > 1 file changed, 1
On 08/29/2016 09:30 PM, Jan Vesely wrote:
On Mon, 2016-08-29 at 20:13 +0200, Vedran Miletić wrote:
Clover presently reports 32 as CL_DEVICE_ADDRESS_BITS, which is not
correct for AMD SI and newer chip generations. This patch introduces
the PIPE_COMPUTE_CAP_ADDRESS_BITS pipe capability queried
On Mon, 2016-08-29 at 20:13 +0200, Vedran Miletić wrote:
> Clover presently reports 32 as CL_DEVICE_ADDRESS_BITS, which is not
> correct for AMD SI and newer chip generations. This patch introduces
> the PIPE_COMPUTE_CAP_ADDRESS_BITS pipe capability queried by Clover
> to r600 pipe and sets the
On Mon, Aug 29, 2016 at 8:29 PM, Bas Nieuwenhuizen
wrote:
> Hi Marek,
>
> I don't think this accounts for the fast clear bits? unorm->uint and
> snorm<->sint should have compatible clear values, but otherwise we may
> need to eliminate the fast clears.
That's a good
On Sun, Aug 28, 2016 at 7:13 PM, Timothy Arceri
wrote:
> Fixes uninitialised warning for coord_components.
> ---
> src/compiler/spirv/spirv_to_nir.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/compiler/spirv/spirv_to_nir.c
>
On Mon, Aug 29, 2016 at 2:45 PM, Matt Turner wrote:
> On Sun, Aug 28, 2016 at 7:10 PM, Ilia Mirkin wrote:
>> Signed-off-by: Ilia Mirkin
>> ---
>> src/compiler/glsl/glsl_parser_extras.cpp | 57
>> +++-
On Sun, Aug 28, 2016 at 7:10 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/compiler/glsl/glsl_parser_extras.cpp | 57
> +++-
> src/compiler/glsl/glsl_parser_extras.h | 2 ++
>
We need to set the need_convert flag with each loop iteration, not
just when the rgba pointer is null.
Bug reported by Markus Müller on mesa-users list.
Fixes new piglit arb_texture_float-get-tex3d test.
Cc:
---
Hi Marek,
I don't think this accounts for the fast clear bits? unorm->uint and
snorm<->sint should have compatible clear values, but otherwise we may
need to eliminate the fast clears.
Yours sincerely,
Bas Nieuwenhuizen
On Mon, Aug 29, 2016 at 5:28 PM, Marek Olšák wrote:
>
On Sun, Aug 28, 2016 at 10:10 PM, Ilia Mirkin wrote:
> This also exposes them for ARB_ES3_2_compatibility.
>
> Signed-off-by: Ilia Mirkin
> ---
> src/mesa/main/context.h | 10 ++
> src/mesa/main/get.c | 26
On Mon, Aug 29, 2016 at 2:19 PM, Jan Vesely wrote:
> On Mon, 2016-08-29 at 14:47 +0200, Marek Olšák wrote:
>> On Sun, Aug 28, 2016 at 7:57 PM, Jan Vesely
>> wrote:
>> >
>> > v2: document the new cap
>> >
>> > Signed-off-by: Jan Vesely
On Mon, 2016-08-29 at 14:47 +0200, Marek Olšák wrote:
> On Sun, Aug 28, 2016 at 7:57 PM, Jan Vesely
> wrote:
> >
> > v2: document the new cap
> >
> > Signed-off-by: Jan Vesely
> > ---
> > src/gallium/docs/source/screen.rst | 1 +
> >
On Mon, Aug 29, 2016 at 7:23 PM, Emil Velikov wrote:
> On 29 August 2016 at 16:29, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> ---
>> src/gallium/drivers/noop/noop_state.c | 56
>> +--
>> 1 file
Echelon9 writes:
> From: Rhys Kidd
>
> Noticed this error in a debug message whilst reviewing
> https://bugs.freedesktop.org/show_bug.cgi?id=97477
>
> This patch doesn't go towards fixing that bug, but at
> least may clarify future debug output.
>
>
Rhys Kidd writes:
> Having run Mesa through Clang on Eric Anholt's Travis harness, these small
> code clean ups improve readability of TGSI code in r600g and may avoid
> future problems.
>
> Series also can be found at:
>
Clover presently reports 32 as CL_DEVICE_ADDRESS_BITS, which is not
correct for AMD SI and newer chip generations. This patch introduces
the PIPE_COMPUTE_CAP_ADDRESS_BITS pipe capability queried by Clover
to r600 pipe and sets the value to 32 for AMD EG/NI chips and 64 for
SI and newer (chips
Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Monday, August 29, 2016 9:16:09 AM
To: mesa-dev@lists.freedesktop.org
Cc: Neha Bhende; Charmaine Lee
Subject: [PATCH] svga: s/unsigned/enum pipe_shader_type/
On 29 August 2016 at 16:29, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/drivers/noop/noop_state.c | 56
> +--
> 1 file changed, 7 insertions(+), 49 deletions(-)
>
> diff --git
---
src/gallium/drivers/svga/svga_draw.c| 4 ++--
src/gallium/drivers/svga/svga_pipe_sampler.c| 2 +-
src/gallium/drivers/svga/svga_sampler_view.h| 2 +-
src/gallium/drivers/svga/svga_shader.c | 3 ++-
src/gallium/drivers/svga/svga_shader.h | 5 +++--
On 26 August 2016 at 19:24, Tim Rowley wrote:
> 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
On 29 August 2016 at 05:37, Tapani Pälli wrote:
>
>
> On 08/26/2016 03:58 PM, Emil Velikov wrote:
>>
>> On 26 August 2016 at 08:50, Tapani Pälli wrote:
>>>
>>> Reviewed-by: Tapani Pälli
>>>
>> What happened with my
Topi asked to have the prefix removed because there's nothing gen7 about
it. However, now that everything is in a single file, there is no good
reason to have it split out into a helper function anyway. Let's just put
the contents in emit_urb_config and call it a day.
---
On Sun, Aug 28, 2016 at 10:29 PM, Pohjolainen, Topi <
topi.pohjolai...@gmail.com> wrote:
> On Fri, Aug 19, 2016 at 09:55:43AM -0700, Jason Ekstrand wrote:
> > ---
> > src/mesa/drivers/dri/i965/genX_blorp_exec.c | 6 ++
> > 1 file changed, 2 insertions(+), 4 deletions(-)
> >
> > diff --git
On Mon, Aug 29, 2016 at 11:49 AM, Marek Olšák wrote:
> On Mon, Aug 29, 2016 at 5:43 PM, Ilia Mirkin wrote:
>> On Mon, Aug 29, 2016 at 11:29 AM, Marek Olšák wrote:
>>> From: Marek Olšák
>>>
>>> ---
>>>
On Fri, Aug 26, 2016 at 7:58 AM, Kai Wasserbäch
wrote:
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_state.c
> b/src/gallium/drivers/nouveau/nvc0/nvc0_state.c
> index b9ac9f4..48aaa46 100644
> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_state.c
> +++
On Mon, Aug 29, 2016 at 5:41 PM, Ilia Mirkin wrote:
> On Mon, Aug 29, 2016 at 11:35 AM, Marek Olšák wrote:
>> On Sat, Aug 27, 2016 at 11:53 PM, Ilia Mirkin wrote:
>>> Signed-off-by: Ilia Mirkin
>>> ---
>>>
On Mon, Aug 29, 2016 at 5:43 PM, Ilia Mirkin wrote:
> On Mon, Aug 29, 2016 at 11:29 AM, Marek Olšák wrote:
>> From: Marek Olšák
>>
>> ---
>> src/gallium/drivers/noop/noop_pipe.c | 51
>> +++
>>
On Mon, Aug 29, 2016 at 11:35 AM, Marek Olšák wrote:
> On Sat, Aug 27, 2016 at 11:53 PM, Ilia Mirkin wrote:
>> Signed-off-by: Ilia Mirkin
>> ---
>> docs/features.txt | 4 ++--
>> docs/relnotes/12.1.0.html
On Mon, Aug 29, 2016 at 11:29 AM, Marek Olšák wrote:
> From: Marek Olšák
>
> ---
> src/gallium/drivers/noop/noop_pipe.c | 51
> +++
> src/gallium/drivers/noop/noop_state.c | 24 +
> 2 files changed, 75
From: Marek Olšák
num-cs-flushes will mean compute shader flushes
---
src/gallium/drivers/radeon/r600_query.c | 8
src/gallium/drivers/radeon/r600_query.h | 2 +-
2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_query.c
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_pipe_common.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c
b/src/gallium/drivers/radeon/r600_pipe_common.c
index b1da22f..32486c8 100644
---
https://bugs.freedesktop.org/show_bug.cgi?id=97250
Vedran Miletić changed:
What|Removed |Added
CC||ved...@miletic.net
From: Marek Olšák
For coherency with the current context.
---
src/gallium/drivers/radeon/r600_texture.c | 17 +++--
1 file changed, 11 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
From: Marek Olšák
---
src/gallium/auxiliary/Makefile.sources | 2 -
src/gallium/auxiliary/util/u_slab.c| 171 -
src/gallium/auxiliary/util/u_slab.h| 96
On Sat, Aug 27, 2016 at 11:53 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> docs/features.txt | 4 ++--
> docs/relnotes/12.1.0.html | 4 ++--
> src/mesa/state_tracker/st_extensions.c | 14
From: Marek Olšák
X+DRI3 locks up if the returned handle is invalid.
---
src/gallium/drivers/noop/noop_pipe.c | 16 ++--
1 file changed, 14 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/noop/noop_pipe.c
b/src/gallium/drivers/noop/noop_pipe.c
From: Marek Olšák
---
src/gallium/drivers/noop/noop_pipe.c | 51 +++
src/gallium/drivers/noop/noop_state.c | 24 +
2 files changed, 75 insertions(+)
diff --git a/src/gallium/drivers/noop/noop_pipe.c
From: Marek Olšák
---
src/gallium/drivers/noop/noop_state.c | 56 +--
1 file changed, 7 insertions(+), 49 deletions(-)
diff --git a/src/gallium/drivers/noop/noop_state.c
b/src/gallium/drivers/noop/noop_state.c
index 0c0ad9f..01538bfe27
From: Marek Olšák
for less noise in the HUD
---
src/gallium/drivers/radeonsi/si_compute.c| 1 +
src/gallium/drivers/radeonsi/si_pipe.h | 1 +
src/gallium/drivers/radeonsi/si_state_draw.c | 4 +++-
3 files changed, 5 insertions(+), 1 deletion(-)
diff --git
From: Marek Olšák
It's part of the viewport state now.
---
src/gallium/drivers/r600/evergreen_state.c | 14 +-
src/gallium/drivers/r600/r600_state.c | 6 --
src/gallium/drivers/radeonsi/si_state.c| 6 --
3 files changed, 1 insertion(+), 25
From: Marek Olšák
There are also some cosmetic changes.
---
src/util/Makefile.sources | 2 +
src/util/slab.c | 180 ++
src/util/slab.h | 62
3 files changed, 244 insertions(+)
create mode
From: Marek Olšák
Calculate depth ranges from viewport states and
pipe_rasterizer_state::clip_halfz.
The evergreend.h change is required to silence a warning.
This fixes this recently updated piglit: arb_depth_clamp/depth-clamp-range
---
From: Marek Olšák
The closed compiler does the same thing.
This fixes: GL45-CTS.texture_gather.*-int-* (18 tests)
---
src/gallium/drivers/radeonsi/si_shader.c | 99 +++-
1 file changed, 96 insertions(+), 3 deletions(-)
diff --git
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_viewport.c | 30 --
1 file changed, 16 insertions(+), 14 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_viewport.c
b/src/gallium/drivers/radeon/r600_viewport.c
index
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_shader.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index a5b566e..2863faa 100644
---
From: Marek Olšák
---
src/gallium/drivers/radeon/r600_pipe_common.h | 3 +++
src/gallium/drivers/radeon/r600_query.c | 21 +
src/gallium/drivers/radeon/r600_query.h | 3 +++
src/gallium/drivers/radeonsi/si_state_draw.c | 8
4
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_state_draw.c | 62 ++--
1 file changed, 31 insertions(+), 31 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
b/src/gallium/drivers/radeonsi/si_state_draw.c
index
From: Marek Olšák
ported from Vulkan
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/radeonsi/si_state_shaders.c
index 394afaa..b4f19fe
From: Marek Olšák
DCC is limited in how texture formats can be reinterpreted using texture
views. If we get a view format that is incompatible with the initial
texture format with respect to DCC, disable DCC.
There is a new piglit which tests all format combinations.
What
From: Marek Olšák
This fixes:
GL45-CTS.tessellation_shader.tessellation_control_to_tessellation_evaluation
.gl_PatchVerticesIn
---
src/gallium/drivers/radeonsi/si_shader.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git
From: Marek Olšák
just do what the comment says
---
src/gallium/drivers/radeon/r600_texture.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index
From: Marek Olšák
For coherency with the current context.
---
src/gallium/drivers/radeon/r600_texture.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_texture.c
b/src/gallium/drivers/radeon/r600_texture.c
index
From: Marek Olšák
Sometimes it was f32, other times it was i32. Now it's always i32.
This fixes:
GL45-CTS.texture_cube_map_array.image_texture_size.texture_size_compute_sh
---
src/gallium/drivers/radeonsi/si_shader.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
From: Marek Olšák
Invalidated buffers don't have to go through it.
Split r600_init_resource into r600_init_resource_fields and
r600_alloc_resource.
---
src/gallium/drivers/r600/r600_state_common.c| 5 +-
src/gallium/drivers/radeon/r600_buffer_common.c | 84
From: Marek Olšák
Limit it to geometry shaders and Hawaii.
---
src/gallium/drivers/radeonsi/si_state_draw.c | 21 +
1 file changed, 13 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_draw.c
From: Marek Olšák
This fixes: GL43-CTS.texture_view.view_sampling
Cc: mesa-sta...@lists.freedesktop.org
---
src/gallium/drivers/radeonsi/si_state.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/gallium/drivers/radeonsi/si_state.c
Hi,
This series contains mostly fixes, i.e. for DCC, cubemaps, tessellation,
texture views, Gather4, viewport depth range, etc.
There are also some new HUD queries.
Please review.
Marek
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
On Sun, Aug 28, 2016 at 6:33 PM, Mauro Rossi wrote:
> > While you're at it, I've got another build-breaking branch here:
> >
> > https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/blorp-vulkan
> >
> > It's almost reviewed so I'll be pushing soon. If you could provide
https://bugs.freedesktop.org/show_bug.cgi?id=77449
Vedran Miletić changed:
What|Removed |Added
Depends on||91969
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=77449
Vedran Miletić changed:
What|Removed |Added
Depends on||97340, 95308
On 08/27/2016 11:18 AM, Jan Vesely wrote:
On Fri, 2016-08-26 at 17:25 +0100, Eric Engestrom wrote:
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
https://bugs.freedesktop.org/show_bug.cgi?id=97260
Vedran Miletić changed:
What|Removed |Added
CC||ved...@miletic.net
Reviewed-by: Marek Olšák
Marek
On Mon, Aug 29, 2016 at 4:16 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This should be all that is required for cull distances to work
> on radeonsi.
>
> v1.1: whitespace cleanup, add docs fix
Hi Rhys,
I ran piglit on my Evergreen HD5850 with your patches.
No regressions here.
Tested-by: James Harvey
Thanks,
James
On 08/27/2016 09:05 AM, Rhys Kidd wrote:
Having run Mesa through Clang on Eric Anholt's Travis harness, these small
code clean ups improve
On Monday, August 29, 2016, Marek Olšák wrote:
> For the series:
>
> Reviewed-by: Marek Olšák >
>
> Marek
Thanks Marek.
As I don't have commit access, when you have a moment I would appreciate if
you could please push to master.
Regards,
For the series:
Reviewed-by: Marek Olšák
Marek
On Sat, Aug 27, 2016 at 6:05 PM, Rhys Kidd wrote:
> Small code clean up that removes magic numbers where a TGSI
> opcode has been defined.
>
> No functional change expected as each opcode is unsupported on
On Sun, Aug 28, 2016 at 7:57 PM, Jan Vesely wrote:
> v2: document the new cap
>
> Signed-off-by: Jan Vesely
> ---
> src/gallium/docs/source/screen.rst | 1 +
> src/gallium/drivers/ilo/ilo_screen.c | 6 ++
>
On Mon, Aug 29, 2016 at 2:21 AM, Edward O'Callaghan
wrote:
> Missing Signoff-by line but otherwise with that fix,
Signed-off-by isn't required in Mesa.
The patch is:
Reviewed-by: Marek Olšák
Marek
>
> Reviewed-By: Edward O'Callaghan
Hi Nayan,
yeah, that sounds like a good idea to me. In the meantime I'm taking a
look at your lanczos filter once more and try to figure out where those
artifacts come from.
The problem with using multiple output surfaces is that DRI2 didn't
supported that. E.g. in DRI2 you can only get two
1 - 100 of 110 matches
Mail list logo