On Wed, 2017-10-25 at 16:25 -0700, Jason Ekstrand wrote:
> ---
> src/intel/compiler/brw_fs_nir.cpp | 33 +--
> --
> 1 file changed, 21 insertions(+), 12 deletions(-)
>
> diff --git a/src/intel/compiler/brw_fs_nir.cpp
> b/src/intel/compiler/brw_fs_nir.cpp
> index e008e2
Valgrind shows that leak is caused by gen6_upload_push_constant, add
unref push_const_bo per stage to destructor to fix this (like done for
scratch_bo).
==10952== 144 bytes in 1 blocks are definitely lost in loss record 44 of 66
==10952==at 0x4C30A1E: calloc (vg_replace_malloc.c:711)
On Wed, 2017-10-25 at 16:25 -0700, Jason Ekstrand wrote:
> Stop retyping the output of shuffle_64bit_data_for_32bit_write. It's
> always BRW_REGISTER_TYPE_D which is perfectly fine for writing out.
> Also, when we change get_nir_src to return something with a 64-bit
> type
> for 64-bit values, the
Signed-off-by: Jordan Justen
Reviewed-by: Jason Ekstrand
Cc: Jason Ekstrand
Cc: Kenneth Graunke
---
* Add comment (Ken)
* No typedef (Jason)
src/intel/compiler/brw_compiler.h | 20
1 file changed, 20 insertions(+)
diff --git a/src/intel/compiler/brw_compiler.h
b/src/
Reviewed-by: Tapani Pälli
On 10/26/2017 11:26 PM, Kenneth Graunke wrote:
According to the ARB_ES3_1_compatibility specification,
glGetFramebufferAttachmentParameteriv is supposed to accept BACK,
and it behaves exactly like BACK_LEFT.
Fixes a GL error in GFXBench 5 Aztec Ruins.
Cc: mesa-sta...
Hi Eric,
Eric Engestrom writes:
> On Monday, 2017-10-23 16:20:54 +0530, Harish Krupo wrote:
>> This passes 33/37 deqp tests related to partial_update, 4 are not
>> supported. Tests not supported:
>> dEQP-EGL.functional.negative_partial_update.not_postable_surface
>> dEQP-EGL.functional.negative_
Also, NIR doesn't (yet) have matrix SSA values so load_var/store_var
wouldn't really make sense on them.
On Fri, Oct 27, 2017 at 12:25 AM, Timothy Arceri wrote:
> On second thoughts I'm splitting the matrix as well so I probably don't need
> this.
>
>
> On 27/10/17 14:58, Timothy Arceri wrote:
>>
On second thoughts I'm splitting the matrix as well so I probably don't
need this.
On 27/10/17 14:58, Timothy Arceri wrote:
For some reason when we convert the following from GLSL IR to NIR:
out mat4 var;
The derefence type ends up as vec4 so we don't currently end up
asserting here. However
For some reason when we convert the following from GLSL IR to NIR:
out mat4 var;
The derefence type ends up as vec4 so we don't currently end up
asserting here. However I hit this with a NIR array splitting pass
I'm working on.
---
src/compiler/nir/nir_validate.c | 2 ++
1 file changed, 2 insert
Rb to both
On October 26, 2017 18:38:52 Kenneth Graunke wrote:
Asserting slot >= 2 made sense when the URB read offset was always 1
(pair of slots). Commit 566a0c43f0b9fbf5106161471dd5061c7275f761 made
it possible to read from the VUE header in slot 0, by adjusting the
offset to be 0. So, t
Am 27.10.2017 um 04:50 schrieb Marek Olšák:
> On Fri, Oct 27, 2017 at 1:44 AM, Roland Scheidegger
> wrote:
>> Am 26.10.2017 um 17:31 schrieb Marek Olšák:
>>> From: Marek Olšák
>>>
>>> Uploaded data must start at (stride * start), because we can't modify
>>> start in all cases.
>> I'm probably mi
On Fri, Oct 27, 2017 at 1:44 AM, Roland Scheidegger wrote:
> Am 26.10.2017 um 17:31 schrieb Marek Olšák:
>> From: Marek Olšák
>>
>> Uploaded data must start at (stride * start), because we can't modify
>> start in all cases.
> I'm probably missing something here, but why not actually?
1) Mixed V
Commit 566a0c43f0b9fbf5106161471dd5061c7275f761 started setting the
3DSTATE_SBE bit to override these values with the one calculated there.
So, they're dead. Stop setting them.
---
src/intel/vulkan/genX_pipeline.c | 26 --
1 file changed, 26 deletions(-)
diff --git a/src
Asserting slot >= 2 made sense when the URB read offset was always 1
(pair of slots). Commit 566a0c43f0b9fbf5106161471dd5061c7275f761 made
it possible to read from the VUE header in slot 0, by adjusting the
offset to be 0. So, this assert is now bogus. Use the one from GL.
---
src/intel/vulkan/
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Sun, Oct 22, 2017 at 1:01 PM, Jordan Justen
wrote:
> Signed-off-by: Jordan Justen
> ---
> src/intel/compiler/brw_compiler.h | 18 ++
> 1 file changed, 18 insertions(+)
>
> diff --git a/src/intel/compiler/brw_compiler.h b/src/intel/compiler/brw_
> compiler.h
> index 701b4a80bf
+idr
On Thu, Oct 19, 2017 at 11:04 AM, Jason Ekstrand
wrote:
> Not to be confused with variablePointersStorageBuffer which is the
> subset of VK_KHR_variable_pointers required to enable the extension.
> This gives us "full" support for variable pointers.
>
> The approach chosen here was to do th
Am 26.10.2017 um 17:31 schrieb Marek Olšák:
> From: Marek Olšák
>
> Uploaded data must start at (stride * start), because we can't modify
> start in all cases.
I'm probably missing something here, but why not actually?
Roland
If it's the first allocation, it's also the amount
> of memory wast
This is build tested only
Signed-off-by: Dylan Baker
---
meson.build | 4 +-
meson_options.txt| 2 +-
src/gallium/drivers/r300/meson.build | 156 +++
src/gallium/meson.build | 4 +-
src/gallium/target
Build tested only.
cc: Dave Airlie
Signed-off-by: Dylan Baker
---
meson.build| 12 ++---
meson_options.txt | 2 +-
src/gallium/drivers/virgl/meson.build | 39 ++
src/gallium/meson.build
Which allows the code to be compiled with c99 instead of gnu99.
A little history. This code is guarded by #ifdef __GNUC__, so it's only
compiled with autotools on *nix, SCons with MSVC wont hit that code.
However, meson is going to build both MSVC and GCC/Clang paths. As such
it makes sense to not
This series implements the rest of the remaining gallium drivers except SWR. I
have a branch for SWR, but it needs some upstream meson patches that haven't
landed, but will hopefully be in 0.44.0
Most of this is pretty straight forward, except for one change to svga.
I have no hardware/software t
Signed-off-by: Dylan Baker
---
meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/meson.build b/meson.build
index 1da1bb06798..64084d686e3 100644
--- a/meson.build
+++ b/meson.build
@@ -647,7 +647,7 @@ if with_gallium_freedreno
endif
llvm_modules = ['bitwriter', '
Build tested only.
cc: Brian Paul
Signed-off-by: Dylan Baker
---
meson.build | 2 +
meson_options.txt | 2 +-
src/gallium/drivers/svga/meson.build| 88 +
src/gallium/meson.build | 7 ++-
src
This has been build tested only.
Signed-off-by: Dylan Baker
---
meson.build | 20 --
meson_options.txt| 2 +-
src/gallium/drivers/r600/meson.build | 128 +++
src/gallium/meson.build | 4 +-
src/gal
Build tested only.
Signed-off-by: Dylan Baker
---
meson.build | 7 +++-
src/gallium/drivers/i915/meson.build| 70 +
src/gallium/meson.build | 7 +++-
src/gallium/targets/dri/meson.build | 5 +++
src/gallium/wi
On Thu, Oct 26, 2017 at 1:39 AM, Iago Toral wrote:
> On Wed, 2017-10-25 at 16:25 -0700, Jason Ekstrand wrote:
> > Before, we were careful to place the zip after the last of the split
> > instructions but did unzip on-demand. This changes things so that
> > the
> > unzips go before all of the spl
---
src/intel/compiler/brw_compiler.h | 1 -
src/intel/compiler/brw_vec4.cpp | 1 -
2 files changed, 2 deletions(-)
diff --git a/src/intel/compiler/brw_compiler.h
b/src/intel/compiler/brw_compiler.h
index 014202d36ca..6a330537a5b 100644
--- a/src/intel/compiler/brw_compiler.h
+++ b/src/intel/c
Having moved gallium_dri.so library to /vendor/lib/dri
also symlinks need to be coherently created using TARGET_OUT_VENDOR insted of
TARGET_OUT
or all non Intel drivers will not be loaded with Android N and earlier,
thus causing SurfaceFlinger SIGABRT
Fixes: c3f75d483c ("Android: move libraries t
On Thu, Oct 26, 2017 at 11:37 PM, Andres Gomez wrote:
> Nicolai, this patch depends on 759526813be which didn't make it for
> 17.2
>
> Hence, I could recover that if you want to nominate it or we would need
> a backport. Otherwise, I'm dropping this patch.
>
> Let me know what you think.
Yes, ple
Nicolai, this patch depends on 759526813be which didn't make it for
17.2
Hence, I could recover that if you want to nominate it or we would need
a backport. Otherwise, I'm dropping this patch.
Let me know what you think.
On Thu, 2017-10-12 at 11:24 +0200, Nicolai Hähnle wrote:
> From: Nicolai Hä
Quoting Eric Engestrom (2017-10-26 02:40:20)
> On Wednesday, 2017-10-25 15:58:23 -0700, Dylan Baker wrote:
> > This has been tested with the osdemo from mesa-demos
> >
> > Signed-off-by: Dylan Baker
> > ---
> > meson.build | 3 ++
> > meson_options.txt
Please "clear_db_cache_before_clear" and the option too. With that,
the patch is:
Reviewed-by: Marek Olšák
Thanks,
Marek
On Thu, Oct 26, 2017 at 6:08 PM, Samuel Pitoiset
wrote:
> It appears that flushing the DB metadata is actually not sufficient
> since the driver uses the new VS blit shaders
According to the ARB_ES3_1_compatibility specification,
glGetFramebufferAttachmentParameteriv is supposed to accept BACK,
and it behaves exactly like BACK_LEFT.
Fixes a GL error in GFXBench 5 Aztec Ruins.
Cc: mesa-sta...@lists.freedesktop.org
---
src/mesa/main/fbobject.c | 9 +
1 file ch
On Thu, Oct 26, 2017 at 7:23 PM, Eric Anholt wrote:
> Marek Olšák writes:
>
>> From: Marek Olšák
>
> Could you hit vc5, as well?
>
> I'm not clear on why this is a pipe_cap instead of just updating type or
> comments on the pipe structs to say what the new behavior is -- aren't
> most of us usin
In subject and comments: s/aera/area/
I'm not sure about DX line rasterization, but IIRC for GL, if a line's
endpoints are identical, nothing should be drawn.
I can see how this is relevant for point mode though.
Reviewed-by: Brian Paul
On 10/26/2017 01:24 PM, srol...@vmware.com wrote:
Fr
From: Roland Scheidegger
Culling tris with zero aera seems like a great idea, but apparently with
fill mode line (and point) we're supposed to draw them, at least some tests
for some other state tracker complained otherwise.
Such tris also always seem to be back facing (not sure if this can be
in
system/window.h is no longer available by default and is part of
libnativewindow, so add it to the shared libraries. It has to be conditional
because the library is only present in O and later.
Really, we should only be depending on vndk/window.h now, but that's only
in O and changing would be pre
26.10.2017 u 01:09, Jan Vesely je napisao/la:
> On Tue, 2017-10-24 at 15:32 +0200, Vedran Miletić wrote:
>> On 10/23/2017 05:24 AM, Jan Vesely wrote:
>>> From: Jan Vesely
>>>
>>> v2: use a more generic compat function
>>>
>>> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103388
>>> Signed
On Thu, Oct 26, 2017 at 3:49 AM, Iago Toral wrote:
> The subject line is incomplete, it misses the '64-bit types' at the
> end.
>
That must be a git-send-email fail. The commit message I have says:
Fix broadcast instruction for 64-bit values on little-core
> On Wed, 2017-10-25 at 16:25 -0700
On Thu, Oct 26, 2017 at 3:18 AM, Iago Toral wrote:
> I am not sure I get the purpose of this, there is nothing wrong with
> the change, but the subject suggests that was so that we modified that
> offset only inside brw_broadcast()... but that was already happening
> and in fact this patch only c
On Thu, Oct 26, 2017 at 5:15 AM, Iago Toral wrote:
> I left a few minor comments in patches 1, 2, 8 and 14. Otherwise
> patches 1-2, 4-5 and 7-14 (3 and 6 already have Rb) are:
>
> Reviewed-by: Iago Toral Quiroga
>
> I feel like patches 10, 11 could maybe use another extra review if
> there is s
On Thu, Oct 26, 2017 at 9:10 AM, Pohjolainen, Topi <
topi.pohjolai...@gmail.com> wrote:
> On Wed, Oct 25, 2017 at 02:58:40PM -0700, Jason Ekstrand wrote:
> > On Mon, Oct 23, 2017 at 11:07 PM, Pohjolainen, Topi <
> > topi.pohjolai...@gmail.com> wrote:
> >
> > > On Mon, Oct 23, 2017 at 05:23:08PM -0
Reviewed-by: Bas Nieuwenhuizen
for the series.
On Thu, Oct 26, 2017 at 6:03 PM, Samuel Pitoiset
wrote:
> Only needed when the CS path is used.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_cmd_buffer.c | 46 +--
> src/amd/vulkan/radv_meta_buff
Gwan-gyeong Mun writes:
> Hi,
>
> These Patch v5 series modified with new helper function series [1].
>
> These series only have mesa for tizen feature.
>
> [1] https://patchwork.freedesktop.org/series/32577/
Rather than have another giant pile of window system code in the tree,
I'd like to see
I couldn't find in the SPIR-V spec what the actual difference is
supposed by the two? Seems like in both cases the compiler can assume
no aliasing.
Reviewed-by: Bas Nieuwenhuizen
On Thu, Oct 26, 2017 at 7:11 PM, Jason Ekstrand wrote:
> It's rather surprising that we've never actually hit this b
On Thu, Oct 26, 2017 at 1:23 PM, Eric Anholt wrote:
> Marek Olšák writes:
>
>> From: Marek Olšák
>
> Could you hit vc5, as well?
>
> I'm not clear on why this is a pipe_cap instead of just updating type or
> comments on the pipe structs to say what the new behavior is -- aren't
> most of us usin
Quoting Eric Anholt (2017-10-26 10:33:11)
> Dylan Baker writes:
>
> > [ Unknown signature status ]
> > ping
>
> You had my r-b other than one comment. Are you pinging for something
> else?
Ilia and Timothy had expressed that they wanted to look at it, I wanted to make
sure they didn't have com
Dylan Baker writes:
> [ Unknown signature status ]
> ping
You had my r-b other than one comment. Are you pinging for something
else?
signature.asc
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.free
Marek Olšák writes:
> From: Marek Olšák
Could you hit vc5, as well?
I'm not clear on why this is a pipe_cap instead of just updating type or
comments on the pipe structs to say what the new behavior is -- aren't
most of us using 32-bit math that will wrap the same on addition?
signature.asc
Quoting Gert Wollny (2017-10-26 02:49:56)
>
> Am Mittwoch, den 25.10.2017, 10:24 +0200 schrieb Erik Faye-Lund:
> > If we don't want to use these deps, there's no good reason to search
> > for them in the first place. This should shave a bit of time for the
> > initial build.
> > ---
> >
> > This
one small nit, otherwise,
Reviewed-by: Dylan Baker
Quoting Eric Engestrom (2017-10-26 08:40:57)
> From: Eric Engestrom
>
> Meson's vcs_tag() uses the output of `git describe`, eg.
> 17.3-branchpoint-5-gfbf29c3cd15ae831e249+
>
> Whereas the other build systems used a script that outputs only
Both Reviewed-by: Jordan Justen
Cc: Dylan
On 2017-10-26 08:40:56, Eric Engestrom wrote:
> Suggested-by: Jordan Justen
> Signed-off-by: Eric Engestrom
> ---
> bin/git_sha1_gen.py | 13 -
> src/Makefile.am | 15 ---
> src/SConscript
It's rather surprising that we've never actually hit this before.
Aparently, Ian's SPIR-V generator currently claims the Simple when you
don't do anything complex. We really shouldn't assert-fail on it.
Reviewed-by: Ian Romanick
Cc: mesa-sta...@lists.freedesktop.org
---
src/compiler/spirv/spirv
I have a few tiny nits, but otherwise this seems fine:
Reviewed-by: Dylan Baker
Quoting Eric Engestrom (2017-10-26 08:40:56)
> Suggested-by: Jordan Justen
> Signed-off-by: Eric Engestrom
> ---
> bin/git_sha1_gen.py | 13 -
> src/Makefile.am | 1
Eric Engestrom writes:
> Mesa's DEBUG and assert's NDEBUG are not tied to each other, so we need
> to explicitly compile this code out.
>
> Fixes: 3df78928786134874eafa "vc4: Drop reloc_count tracking for debug
>asserts on non-debug builds."
> Cc: Eric Anholt
> Signed-off-by: Eric Engest
On 10/26/17 6:25 PM, Eric Engestrom wrote:
> Fixes: 80bfff5c4f1d4d8c842a0 "wayland-egl: adds CFLAGS for wayland.egl.h
> include"
> Suggested-by: Daniel Stone
> Signed-off-by: Eric Engestrom
> ---
> src/egl/wayland/wayland-egl/Makefile.am | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
I don't have an selinux enabled system to test with either, but this looks
correct and doesn't break the build.
Reviewed-by: Dylan Baker
Quoting Eric Engestrom (2017-10-26 09:30:24)
> Signed-off-by: Eric Engestrom
> ---
> build-tested only
> ---
> meson.build | 5 -
> meson_options.t
On Thursday, October 26, 2017 4:24:10 AM PDT Manolova, Plamena wrote:
[snip]
> After taking another look you're absolutely right. For some reason
> propagating the predicates like this didn't occur to me.
> If we use this approach couldn't we get away with just having one predicate
> per draw call?
On 26 October 2017 at 17:25, Eric Engestrom wrote:
> Fixes: 80bfff5c4f1d4d8c842a0 "wayland-egl: adds CFLAGS for wayland.egl.h
> include"
> Suggested-by: Daniel Stone
> Signed-off-by: Eric Engestrom
> ---
> src/egl/wayland/wayland-egl/Makefile.am | 2 +-
> 1 file changed, 1 insertion(+), 1 dele
Signed-off-by: Eric Engestrom
---
build-tested only
---
meson.build | 5 -
meson_options.txt | 6 ++
2 files changed, 10 insertions(+), 1 deletion(-)
diff --git a/meson.build b/meson.build
index 2a89b6482f966293282f..d245fbcea2ee9b02baf5 100644
--- a/meson.build
+++ b/meson.build
@
On 26 October 2017 at 17:25, Eric Engestrom wrote:
> Fixes: 80bfff5c4f1d4d8c842a0 "wayland-egl: adds CFLAGS for wayland.egl.h
> include"
> Suggested-by: Daniel Stone
Acked-by: Daniel Stone
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http
On 25 October 2017 at 10:51, Gert Wollny wrote:
> Dear all,
>
> this is a minor update to the patch set that adds enhanced tracking of IF/ELSE
> branches and tracking of reladdr* registers for the register_merge step.
>
> So far patches 1 & 5 (now 8) are
>
> Reviewed-by: Nicolai Hähnle
>
> Chan
Fixes: 80bfff5c4f1d4d8c842a0 "wayland-egl: adds CFLAGS for wayland.egl.h
include"
Suggested-by: Daniel Stone
Signed-off-by: Eric Engestrom
---
src/egl/wayland/wayland-egl/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/egl/wayland/wayland-egl/Makefile.am
b/s
On 18 October 2017 at 16:05, Gert Wollny wrote:
> Add a check that tests whether the c++ compiler supports c++11, either by
> default,
> by adding the compiler flag -std=c++11, or by adding a compiler flag that the
> user
> has specified via the environment variable CXX11_CXXFLAGS.
>
> The test
On 10/25/2017 10:58 PM, Bas Nieuwenhuizen wrote:
On Wed, Oct 25, 2017 at 4:03 PM, Samuel Pitoiset
wrote:
On 10/25/2017 02:20 PM, Alex Smith wrote:
On 25 October 2017 at 12:46, Samuel Pitoiset mailto:samuel.pitoi...@gmail.com>> wrote:
I have something similar on my local tree (starte
On Wed, Oct 25, 2017 at 02:58:40PM -0700, Jason Ekstrand wrote:
> On Mon, Oct 23, 2017 at 11:07 PM, Pohjolainen, Topi <
> topi.pohjolai...@gmail.com> wrote:
>
> > On Mon, Oct 23, 2017 at 05:23:08PM -0700, Jason Ekstrand wrote:
> > > ---
> > > src/mesa/drivers/dri/i965/brw_blorp.c | 15 +++
It appears that flushing the DB metadata is actually not sufficient
since the driver uses the new VS blit shaders. This looks quite
strange though, but it seems like we need to flush DB for fixing
the corruption.
v2: rename the drirc option
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=1
On 10/26/2017 04:26 PM, Marek Olšák wrote:
Hi Samuel,
Can you also rename the drirc option and the bool?
Hi Marek,
Yes, you are right. I will send a v2.
Thanks.
Thanks,
Marek
On Tue, Oct 24, 2017 at 11:59 AM, Samuel Pitoiset
wrote:
It appears that flushing the DB metadata is actually
Only needed when the CS path is used.
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_cmd_buffer.c | 46 +--
src/amd/vulkan/radv_meta_buffer.c | 13 +++---
src/amd/vulkan/radv_meta_clear.c | 50 +--
src/amd/vulkan/r
Serious Sam Fusion 2017 uses a huge number of occlusion queries,
and the allocated query pool buffer is greater than 4096 bytes.
This slightly improves performance (tested in Ultra) from
117.2 FPS to 119.7 FPS (~+2%) on my RX480.
This also improves Talos, from 69 FPS to 72/73 FPS (~+5%).
v2: - a
On 17 October 2017 at 16:54, Chuck Atkins wrote:
>> I also think adding a test for each C++11 feature used in the code is
>>
>> too tedious, regardless of the build system, and it would really need a
>> dedicated maintainer.
>
>
> Certainly. Rather than checking for everything, I think a code sni
That fixed many issues, but:
On deqp EGL tests, we get:
dEQP-EGL.functional.create_context_ext.gl_42.rgba_no_depth_no_stencil
deqp-egl: main/api_exec.c:135: _mesa_initialize_exec_table: Assertion
`ctx->Version > 0' failed.
On ES31 CTS tests:
ES31-CTS.functional.texture.texture_buffer.render
From: Eric Engestrom
Meson's vcs_tag() uses the output of `git describe`, eg.
17.3-branchpoint-5-gfbf29c3cd15ae831e249+
Whereas the other build systems used a script that outputs only the sha1
of the HEAD commit, eg.
fbf29c3cd1
Given that this information is used by printing it next to the
On 26 October 2017 at 15:37, Greg V wrote:
> On 10/26/2017 17:02, Emil Velikov wrote:
>
>> On 24 October 2017 at 18:21, Greg V wrote:
>>>
>>>
>>> On 10/24/2017 20:16, Dylan Baker wrote:
Quoting Greg V (2017-10-23 13:20:39)
>
> FreeBSD only has ETIMEDOUT, not ETIME
> ---
Suggested-by: Jordan Justen
Signed-off-by: Eric Engestrom
---
bin/git_sha1_gen.py | 13 -
src/Makefile.am | 15 ---
src/SConscript | 22 ++
src/mesa/Android.libmesa_git_sha1.mk | 4 ++--
4 fi
Mesa's DEBUG and assert's NDEBUG are not tied to each other, so we need
to explicitly compile this code out.
Fixes: 3df78928786134874eafa "vc4: Drop reloc_count tracking for debug
asserts on non-debug builds."
Cc: Eric Anholt
Signed-off-by: Eric Engestrom
---
Changes in v2:
- change DEBUG
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_descriptors.c | 25 +++--
src/gallium/drivers/radeonsi/si_pipe.c| 2 +-
2 files changed, 12 insertions(+), 15 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c
b/src/gallium/drivers/radeons
Hi,
This is a necessary feature to prevent out-of-memory errors with
SPECviewperf for drivers relying on u_vbuf for vertex uploads.
Only RadeonSI supports it at the moment.
I don't think we need to change pipe_vertex_buffer::buffer_offset
to int64_t just yet.
Please review.
Thanks,
Marek
__
From: Marek Olšák
Uploaded data must start at (stride * start), because we can't modify
start in all cases. If it's the first allocation, it's also the amount
of memory wasted. If the starting offset is larger than the size of
the upload buffer, the buffer is re-created, used for 1 upload, and th
From: Marek Olšák
---
src/gallium/docs/source/screen.rst | 4
src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 +
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
src/gallium/drivers/i915/i915_screen.c | 1 +
src/gallium/drivers/llvmpipe/lp_screen.c
Signed-off-by: Eric Engestrom
---
build-tested only
---
meson.build | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/meson.build b/meson.build
index 485889590041545c4de0..738ebc510ace26fd7917 100644
--- a/meson.build
+++ b/meson.build
@@ -697,8 +697,11 @@ endif
prog_bison
On 26 October 2017 at 15:40, Eric Engestrom wrote:
> While at it, drop the duplicate return.
>
The dup return is a left-over from the times we had three different,
all optional, codepaths.
Reviewed-by: Emil Velikov
Thanks
Emil
___
mesa-dev mailing lis
On 10/26/2017 17:07, Emil Velikov wrote:
On 23 October 2017 at 21:20, Greg V wrote:
Obtained from: FreeBSD ports
---
src/mesa/main/compiler.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
index 43a06b4313..8652aac36c 100644
--- a/
On 10/26/2017 17:02, Emil Velikov wrote:
On 24 October 2017 at 18:21, Greg V wrote:
On 10/24/2017 20:16, Dylan Baker wrote:
Quoting Greg V (2017-10-23 13:20:39)
FreeBSD only has ETIMEDOUT, not ETIME
---
src/intel/vulkan/anv_device.c | 4
src/intel/vulkan/anv_gem.c| 4
s
On 10/25/2017 10:55 PM, Jose Fonseca wrote:
On 23/10/17 20:15, Emil Velikov wrote:
Hi Nicolai,
On 22 October 2017 at 20:07, Nicolai Hähnle wrote:
From: Nicolai Hähnle
C11 threads were changed to use struct timespec instead of xtime, and
thrd_sleep got a second argument.
As xtime was repla
https://bugs.freedesktop.org/show_bug.cgi?id=103283
Greg V changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Thu, Oct 19, 2017 at 11:52 AM, Christian Gmeiner
wrote:
>>> + if ((usage & PIPE_TRANSFER_WRITE) &&
>>> + prsc->target == PIPE_BUFFER &&
>>> + !util_ranges_intersect(&rsc->valid_buffer_range,
>>> +box->x, box->x + box->width)) {
>>> + /*
While at it, drop the duplicate return.
Signed-off-by: Eric Engestrom
---
src/loader/loader.c | 13 ++---
1 file changed, 2 insertions(+), 11 deletions(-)
diff --git a/src/loader/loader.c b/src/loader/loader.c
index f7f85e37c2094f52175e..3c67f9dc2448a34dbf8e 100644
--- a/src/loader/load
On Thu, Oct 26, 2017 at 4:16 PM, PrasannaKumar Muralidharan
wrote:
>
> GCW Zero gaming console uses etnaviv for GC860. But they use very old
> version from 2013. It is not based on upstream version I think, does
> that count?
They use my initial version of etnaviv, which *only* had pre-GC2000 sup
Hi Samuel,
Can you also rename the drirc option and the bool?
Thanks,
Marek
On Tue, Oct 24, 2017 at 11:59 AM, Samuel Pitoiset
wrote:
> It appears that flushing the DB metadata is actually not sufficient
> since the driver uses the new VS blit shaders. This looks quite
> strange though, but it s
On 24 October 2017 at 14:28, Eric Engestrom wrote:
> On Monday, 2017-10-23 23:20:43 +0300, Greg V wrote:
>> Obtained from: FreeBSD ports
>> ---
>> src/util/u_endian.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/src/util/u_endian.h b/src/util/u_endian.h
>> index b9d
2017-10-26 15:51 GMT+02:00 Wladimir :
> On Thu, Oct 26, 2017 at 9:45 AM, Lucas Stach wrote:
>> Am Donnerstag, den 26.10.2017, 03:17 +0200 schrieb Christian Gmeiner:
>>> Signed-off-by: Christian Gmeiner
>>
>> Has this been tested on older GPU cores like the GC600?
>
Not yet.
> To be fair, I doub
On 26 October 2017 at 19:21, Wladimir wrote:
> On Thu, Oct 26, 2017 at 9:45 AM, Lucas Stach wrote:
>> Am Donnerstag, den 26.10.2017, 03:17 +0200 schrieb Christian Gmeiner:
>>> Signed-off-by: Christian Gmeiner
>>
>> Has this been tested on older GPU cores like the GC600?
>
> To be fair, I doubt a
2017-10-26 9:45 GMT+02:00 Lucas Stach :
> Am Donnerstag, den 26.10.2017, 03:17 +0200 schrieb Christian Gmeiner:
>> Signed-off-by: Christian Gmeiner
>
> Has this been tested on older GPU cores like the GC600?
>
Not yet... only looked what binary blob is using.
>> ---
>> src/gallium/drivers/etnav
On 23 October 2017 at 21:20, Greg V wrote:
> Obtained from: FreeBSD ports
> ---
> src/mesa/main/compiler.h | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
> index 43a06b4313..8652aac36c 100644
> --- a/src/mesa/main/compiler.h
> +++ b
On 24 October 2017 at 18:21, Greg V wrote:
>
>
> On 10/24/2017 20:16, Dylan Baker wrote:
>>
>> Quoting Greg V (2017-10-23 13:20:39)
>>>
>>> FreeBSD only has ETIMEDOUT, not ETIME
>>> ---
>>> src/intel/vulkan/anv_device.c | 4
>>> src/intel/vulkan/anv_gem.c| 4
>>> src/intel/vulkan
On Thu, Oct 26, 2017 at 9:45 AM, Lucas Stach wrote:
> Am Donnerstag, den 26.10.2017, 03:17 +0200 schrieb Christian Gmeiner:
>> Signed-off-by: Christian Gmeiner
>
> Has this been tested on older GPU cores like the GC600?
To be fair, I doubt any of the recent changes have been tested on GC600.
Is
On 24 October 2017 at 17:11, Jerome Duval wrote:
> +libswpipe_la_SOURCES = \
> + GalliumContext.cpp \
> + SoftwareRenderer.cpp
> +
Please enhance this and move to Makefile.sources. Bonus points for
reusing it in SConscript (at a later stage)
GalliumContext.cpp
Gallium
On 24 October 2017 at 17:11, Jerome Duval wrote:
> +libhgl_la_LIBADD = -ltranslation
New library - please document or remove, if not needed.
> +libhgl_la_SOURCES = $(common_SOURCES)
> +
> +
> +EXTRA_DIST = SConscript
> diff --git a/src/gallium/state_trackers/hgl/Makefile.sources
> b/src/gallium
1 - 100 of 141 matches
Mail list logo