[Mesa-dev] [PATCH v4 5/6] util: port _mesa_strto[df] to C

2015-06-28 Thread Erik Faye-Lund
_mesa_strtod and _mesa_strtof are only used from the GLSL compiler and the ARB_[vertex|fragment]_program code, meaning that the locale doesn't need to be initialized before the first OpenGL context gets initialized. So let's use explicit initialization from the one-time init code instead of

[Mesa-dev] [PATCH v4 3/6] mesa/main: only call _mesa_destroy_shader_compiler once on exit

2015-06-28 Thread Erik Faye-Lund
There's no point in calling _mesa_destroy_shader_compiler multiple times on exit; the resources will only be released once anyway. So let's move the atexit-call into the part that is only called once. Signed-off-by: Erik Faye-Lund kusmab...@gmail.com Reviewed-by: Matt Turner matts...@gmail.com

[Mesa-dev] [PATCH v4 4/6] glsl: No need to lock in _mesa_glsl_release_types

2015-06-28 Thread Erik Faye-Lund
This function only gets called while mesa is unloading, so there's no potential of racing or multiple calls at the same time. So let's just get rid of the locking. Signed-off-by: Erik Faye-Lund kusmab...@gmail.com Reviewed-by: Matt Turner matts...@gmail.com Reviewed-by: Brian Paul

[Mesa-dev] [PATCH v4 6/6] mesa/main: free locale at exit

2015-06-28 Thread Erik Faye-Lund
In order to save a small leak if mesa is continously loaded and unloaded, let's free the locale when the shared object is unloaded. Signed-off-by: Erik Faye-Lund kusmab...@gmail.com Reviewed-by: Matt Turner matts...@gmail.com Reviewed-by: Brian Paul bri...@vmware.com --- src/mesa/main/context.c

[Mesa-dev] [PATCH v4 2/6] dri: don't touch the shader compiler

2015-06-28 Thread Erik Faye-Lund
This function is for deleting per-screen resources, and the shader compiler resources are not of such nature. Besides, dri shouldn't need to even know about the presence of a shader compiler. These resources will already be released when mesa gets unloaded, and that should be sufficient.

[Mesa-dev] [PATCH v4 1/6] mesa/main: Get rid of outdated GDB-hack

2015-06-28 Thread Erik Faye-Lund
All of these enums are now in use around in the code, so there's no need to explicitly use them here any more. Signed-off-by: Erik Faye-Lund kusmab...@gmail.com Reviewed-by: Matt Turner matts...@gmail.com Reviewed-by: Brian Paul bri...@vmware.com --- src/mesa/main/context.c | 27

[Mesa-dev] [PATCH v4 0/6] port _mesa_strto[df] to C

2015-06-28 Thread Erik Faye-Lund
Here's the fourth, and hopefully final version of this series. The only code-change this time is fixing up the definition of one_time_fini to have an explict void argument-list. Erik Faye-Lund (6): mesa/main: Get rid of outdated GDB-hack dri: don't touch the shader compiler mesa/main: only

Re: [Mesa-dev] [Nouveau] [RFC PATCH 3/8] nv50: allocate and map a notifier buffer object for PM

2015-06-28 Thread Samuel Pitoiset
On 06/26/2015 01:02 AM, Ilia Mirkin wrote: On Mon, Jun 22, 2015 at 4:53 PM, Samuel Pitoiset samuel.pitoi...@gmail.com wrote: This notifier buffer object will be used to read back global performance counters results written by the kernel. For each domain, we will store the handle of the

Re: [Mesa-dev] [Nouveau] [RFC PATCH 6/8] nv50: add support for compute/graphics global performance counters

2015-06-28 Thread Samuel Pitoiset
On 06/26/2015 01:09 AM, Ilia Mirkin wrote: What's with the \%'s everywhere? Maybe percent will be better ? On Mon, Jun 22, 2015 at 4:53 PM, Samuel Pitoiset samuel.pitoi...@gmail.com wrote: This commit adds support for both compute and graphics global performance counters which have been

[Mesa-dev] [Bug 90817] swrast fails to load with certain remote X servers

2015-06-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90817 Tom Hughes t...@compton.nu changed: What|Removed |Added Attachment #116242|0 |1 is obsolete|

[Mesa-dev] [Bug 90817] swrast fails to load with certain remote X servers

2015-06-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90817 --- Comment #7 from Tom Hughes t...@compton.nu --- Happy to try and improve the patch. I admit to having been a bit confused by the whole GLX_DONT_CARE thing when I was originally looking at this though because as I understand it config

[Mesa-dev] [Bug 90817] swrast fails to load with certain remote X servers

2015-06-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90817 --- Comment #9 from Tom Hughes t...@compton.nu --- I just looked, and there doesn't seem to be a mesa3d-dev list - did you mean mesa-dev? This bug is assigned there anyway, so presumably they are seeing all these comments... -- You are

Re: [Mesa-dev] [PATCH v2] glsls: Modify exec_list to avoid strict-aliasing violations

2015-06-28 Thread Davin McCall
On 26/06/15 14:53, Francisco Jerez wrote: [...] Your first approach seemed quite reasonable IMHO. Were you able to measure any performance regression from it? Thanks. When I run an apitrace replay of a Dota 2 trace [1] with LIBGL_ALWAYS_SOFTWARE and without the patch I get (averaged over

Re: [Mesa-dev] [PATCH 2/4] egl/wayland: handle NULL native_window in create_surface

2015-06-28 Thread Michel Dänzer
On 26.06.2015 21:04, Emil Velikov wrote: Rise EGL_BAD_NATIVE_WINDOW instead of crashing. Spelling: Raise -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer

Re: [Mesa-dev] [PATCH] i965: Switch on shader stage in nir_setup_outputs().

2015-06-28 Thread Jordan Justen
Reviewed-by: Jordan Justen jordan.l.jus...@intel.com On 2015-06-26 16:03:21, Kenneth Graunke wrote: Adding new shader stages to a switch statement is less confusing than an if-else-if ladder where all but the first case are fragment shader specific (but don't claim to be). Signed-off-by:

[Mesa-dev] [PATCH v2] glsl: use non-null context when cloning variable

2015-06-28 Thread Ilia Mirkin
ProgramResourceList might not yet have been initialized. In that case, parent the var to the program. Fixes: c2ff3485b3d (glsl: clone inputs and outputs during linking) Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: mesa-sta...@lists.freedesktop.org --- v1 - v2: parent to prog only if the

Re: [Mesa-dev] [PATCH v2] glsl: use non-null context when cloning variable

2015-06-28 Thread Tapani Pälli
On 06/29/2015 08:34 AM, Kenneth Graunke wrote: On Monday, June 29, 2015 01:13:30 AM Ilia Mirkin wrote: ProgramResourceList might not yet have been initialized. In that case, parent the var to the program. Fixes: c2ff3485b3d (glsl: clone inputs and outputs during linking) Signed-off-by: Ilia

[Mesa-dev] [PATCH] glsl: use correct context when cloning variable

2015-06-28 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin imir...@alum.mit.edu --- I'm not really clear on how ralloc, clone, etc all work. However experimentally, this makes the leak go away from: bin/fbo-generatemipmap-formats GL_ARB_depth_texture -auto (and likely others). In fact the ProgramResourceList is reralloc'd

Re: [Mesa-dev] [PATCH] i965: Don't try to print the GLSL IR if it has been freed

2015-06-28 Thread Tapani Pälli
Reviewed-by: Tapani Pälli tapani.pa...@intel.com On 06/26/2015 07:54 PM, Neil Roberts wrote: Since commit 104c8fc2c2aa5621261f8 the GLSL IR will be freed if NIR is being used. This was causing it to segfault if INTEL_DEBUG=wm is set. This patch just makes it avoid dumping the GLSL IR in that

Re: [Mesa-dev] [PATCH v4 3/6] mesa/es3.1: enable GL_ARB_texture_multisample for GLES 3.1

2015-06-28 Thread Tapani Pälli
On 06/26/2015 05:38 PM, Ilia Mirkin wrote: On Fri, Jun 26, 2015 at 4:18 AM, Tapani Pälli tapani.pa...@intel.com wrote: On 06/26/2015 01:06 AM, Ilia Mirkin wrote: On Thu, Jun 25, 2015 at 4:22 PM, Ilia Mirkin imir...@alum.mit.edu wrote: On Thu, Jun 25, 2015 at 5:08 AM, Marta Lofstedt

Re: [Mesa-dev] [PATCH v2] glsl: use non-null context when cloning variable

2015-06-28 Thread Kenneth Graunke
On Monday, June 29, 2015 01:13:30 AM Ilia Mirkin wrote: ProgramResourceList might not yet have been initialized. In that case, parent the var to the program. Fixes: c2ff3485b3d (glsl: clone inputs and outputs during linking) Signed-off-by: Ilia Mirkin imir...@alum.mit.edu Cc: