[Mesa-dev] [PATCH v2 02/29] mesa/main: rename format-check function

2018-11-23 Thread Erik Faye-Lund
_mesa_es3_error_check_format_and_type isn't specific to OpenGL ES 3.x, it applies to all versions of OpenGL ES. So let's rename it to reflect this. While we're at it, let's also rename a helper function it uses similarly. As the helper is static, we can also remove the namespacing-prefix from the

[Mesa-dev] [PATCH v2 05/29] mesa/main: clean up ES2_compatibility check

2018-11-23 Thread Erik Faye-Lund
This makes the logic a little bit easier to follow; this is *either* about ES2 compatibility *or* about gles. GL_RGB565 was added already in OpenGL ES 1.0. Signed-off-by: Erik Faye-Lund --- src/mesa/main/glformats.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/mai

[Mesa-dev] [PATCH v2 07/29] mesa/main: use _mesa_has_FOO_bar for compressed format checks

2018-11-23 Thread Erik Faye-Lund
_mesa_has_FOO_bar() knows about the APIs these extensions should be supported under, so let's use that to simplify these checks a bit. Signed-off-by: Erik Faye-Lund --- src/mesa/main/glformats.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/src/mesa/main

Re: [Mesa-dev] [PATCH 29/30] mesa/main: do not require float-texture filtering for es3

2018-11-23 Thread Erik Faye-Lund
On Thu, 2018-11-22 at 13:26 +0100, Mathias Fröhlich wrote: > Hi Erik, > > The series looks very reasonable and I could not spot loosing any > negating ! in the > query logic. Even if I have not been able time wise to double checked > every move when > which texture format got introduced in which E

[Mesa-dev] [Bug 108848] 3d image broken in Dragon age: origins

2018-11-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108848 --- Comment #3 from ilia --- Created attachment 142590 --> https://bugs.freedesktop.org/attachment.cgi?id=142590&action=edit Wine backtrace of game segfault In doubt is this relevant, but game also segfaults with this backtrace when pressing

[Mesa-dev] [Bug 108848] 3d image broken in Dragon age: origins

2018-11-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108848 --- Comment #2 from ilia --- Created attachment 142588 --> https://bugs.freedesktop.org/attachment.cgi?id=142588&action=edit Log with nine disabled Same env variables as with nine enabled. -- You are receiving this mail because: You are the

Re: [Mesa-dev] [PATCH v2 00/13] query validation fixes

2018-11-23 Thread Erik Faye-Lund
On Fri, 2018-11-23 at 07:45 +0200, Tapani Pälli wrote: > Ping, this series has not landed yet, planning to land it soon? > Sorry for the delay, pushed now! > On 11/9/18 7:58 AM, Tapani Pälli wrote: > > Thanks, _mesa_has makes things a lot cleaner and simpler to manage; > > > > Reviewed-by: Tapa

[Mesa-dev] [Bug 108848] 3d image broken in Dragon age: origins

2018-11-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108848 --- Comment #1 from ilia --- Created attachment 142587 --> https://bugs.freedesktop.org/attachment.cgi?id=142587&action=edit Log with nine enabled Game running with GALLIUM_HUD="fps,requested-VRAM" LIBGL_DEBUG=verbose GALLIUM_PRINT_OPTIONS=al

[Mesa-dev] [Bug 108848] 3d image broken in Dragon age: origins

2018-11-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=108848 Bug ID: 108848 Summary: 3d image broken in Dragon age: origins Product: Mesa Version: 18.2 Hardware: Other OS: All Status: NEW Severity: normal

Re: [Mesa-dev] [PATCH] meson: Bump version to 0.46 for python module

2018-11-23 Thread Eero Tamminen
Hi, On 21.11.2018 20.13, Dylan Baker wrote: From: Arfrever Frehtes Taifersar Arahesis Meson has two modules for finding python, the python3 module and the python module. Python3 is older, and has some corner cases, python is newer, has no known corner cases and can detect python2. Things have

[Mesa-dev] [PATCH 1/2] radv: remove useless before CmdClear{Color, DepthStencil}Image()

2018-11-23 Thread Samuel Pitoiset
We don't need to flush anything before these two commands as well. This is because they have to be externally synchronized, so the app should have called CmdPipelineBarrier() prior to that and the driver should have flushed the caches. Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_meta_

[Mesa-dev] [PATCH 2/2] radv: ignore subpass self-dependencies for CreateRenderPass() too

2018-11-23 Thread Samuel Pitoiset
We really need to refactor this... Signed-off-by: Samuel Pitoiset --- src/amd/vulkan/radv_pass.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/src/amd/vulkan/radv_pass.c b/src/amd/vulkan/radv_pass.c index f8e5ea40954..b41ae89deec 100644 --- a/src/amd/vulkan/radv_pass.c +++ b/sr

Re: [Mesa-dev] [PATCH 2/2] glsl: free or reuse memory allocated for TF varying

2018-11-23 Thread Gert Wollny
Hello Tapani, since you wrote the original code, could you have a look at this patch? Many thanks, Gert Am Freitag, den 16.11.2018, 19:12 +0100 schrieb Gert Wollny: > From: Gert Wollny > > When a shader program is de-serialized the gl_shader_program passed > in > may actually still hold memo

[Mesa-dev] [PATCH 1/2] radv: add device->instance extension dependencies

2018-11-23 Thread Niklas Haas
From the vulkan spec 33.3 "Extension Dependencies": "Any device extension that has an instance extension dependency that is not enabled by vkCreateInstance is considered to be unsupported, hence it must not be returned by vkEnumerateDeviceExtensionProperties for any VkPhysicalDevice child of the i

[Mesa-dev] [PATCH 2/2] radv: correctly use vulkan 1.0 by default

2018-11-23 Thread Niklas Haas
From the vulkan spec 3.2 "Instances": "Providing a NULL VkInstanceCreateInfo::pApplicationInfo or providing an apiVersion of 0 is equivalent to providing an apiVersion of VK_MAKE_VERSION(1,0,0)." --- src/amd/vulkan/radv_device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/

[Mesa-dev] [PATCH 2/6] mesa/st: Only unmap the uploader that was actually used.

2018-11-23 Thread Mathias . Froehlich
From: Mathias Fröhlich In st_atom_array, we only need to unmap the upload buffer that was actually used. Signed-off-by: Mathias Fröhlich --- src/mesa/state_tracker/st_atom_array.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_atom_array.c b/sr

[Mesa-dev] [PATCH 5/6] mesa/st: Use binding information from the VAO in feedback rendering.

2018-11-23 Thread Mathias . Froehlich
From: Mathias Fröhlich Use VAO binding information in feedback rendering. In theory it should reduce the amount of buffer objects scheduled for rendering. Feedback rendering is implemented in a crude way anyhow, so I do not expect much gain here. But for the sake of code reuse we should use the s

[Mesa-dev] [PATCH 3/6] mesa/st: Factor out array and buffer setup from st_atom_array.c.

2018-11-23 Thread Mathias . Froehlich
From: Mathias Fröhlich Factor out vertex array setup routines from the array state atom. The factored functions will be used in feedback rendering in the next change. Signed-off-by: Mathias Fröhlich --- src/mesa/state_tracker/st_atom.h | 17 ++ src/mesa/state_tracker/st_atom_array.c

[Mesa-dev] [PATCH 4/6] mesa/st: Avoid extra references in the feedback draw function scope.

2018-11-23 Thread Mathias . Froehlich
From: Mathias Fröhlich The change removes the reference that is held on the entries of the vbuffers[] array. The new code does not do that anymore as following the code into draw_set_vertex_buffers() the draw context holds an other reference as long as it is reset down the function again. So it s

[Mesa-dev] [PATCH 6/6] mesa/st: Make st_pipe_vertex_format static.

2018-11-23 Thread Mathias . Froehlich
From: Mathias Fröhlich Signed-off-by: Mathias Fröhlich --- src/mesa/state_tracker/st_atom.h | 3 --- src/mesa/state_tracker/st_atom_array.c | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/src/mesa/state_tracker/st_atom.h b/src/mesa/state_tracker/st_atom.h index d36f0

[Mesa-dev] [PATCH 0/6] Use common code for gallium array setup.

2018-11-23 Thread Mathias . Froehlich
From: Mathias Fröhlich Hi, The series unifies functions for array setup into gallium. Instead of just walking all VAO attributes we can make use of the enabled and required attribute arrays also for feedback rendering. Along that way cleanup duplicate references and unneeded calls to the uploade

[Mesa-dev] [PATCH 1/6] mesa/st: Only care about the uploader if it was used.

2018-11-23 Thread Mathias . Froehlich
From: Mathias Fröhlich In st_atom_array, we only need to care for unmapping the upload buffer if we actually used it. Signed-off-by: Mathias Fröhlich --- src/mesa/state_tracker/st_atom_array.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mesa/state_tracker/st_a

<    1   2