[Mesa-dev] [PATCH] vl/dri3: fix warning about incompatible pointer type

Signed-off-by: Nayan Deshmukh --- src/gallium/auxiliary/vl/vl_winsys_dri3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri3.c b/src/gallium/auxiliary/vl/vl_winsys_dri3.c index 191a64b..2929928 100644 ---

Re: [Mesa-dev] [PATCH 1/5] st/mesa: simplify some code in get_texture_format_swizzle()

This series is, Reviewed-by: Edward O'Callaghan On 10/01/2016 08:53 AM, Brian Paul wrote: > There's no need to cast to st_texture_image. Just use gl_texture_image. > --- > src/mesa/state_tracker/st_atom_texture.c | 10 +- > 1 file changed, 5 insertions(+),

Re: [Mesa-dev] [PATCH] vl/dri3: handle the case of different GPU(v4.1)

Hi Axel, On 1 Oct 2016 02:34, "Axel Davy" wrote: > > On 16/09/2016 14:51, Nayan Deshmukh wrote: >> >> >> bool flushed; >> + bool is_different_gpu; >> }; >> > > I get a warning when I build about incompatible pointer type at the line > > fd =

[Mesa-dev] [PATCH] glsl: add missing headers to blob.h

--- src/compiler/glsl/blob.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/compiler/glsl/blob.h b/src/compiler/glsl/blob.h index ec903ec..0765bf3 100644 --- a/src/compiler/glsl/blob.h +++ b/src/compiler/glsl/blob.h @@ -29,6 +29,8 @@ extern "C" { #endif +#include +#include

Re: [Mesa-dev] mesa/llvmpipe select alpha visual

Am 30.09.2016 um 01:49 schrieb Thomas Søndergaard: > My application window becomes transparent when I run my application with > llvmpipe by setting LD_LIBRARY_PATH instead of the proprietary NVIDIA > driver that I otherwise use. I've come to the understanding, after > digging into the code for a

Re: [Mesa-dev] [PATCH 6/9] anv/gen7_pipeline: Set few missing fields in 3DSTATE_VS

On Sep 30, 2016 3:56 PM, "Jason Ekstrand" wrote: > > Flyby comments... > > On Sep 30, 2016 3:34 PM, "Anuj Phogat" wrote: > > > > Signed-off-by: Anuj Phogat > > --- > > src/intel/vulkan/gen7_pipeline.c | 6 ++ > > 1 file

Re: [Mesa-dev] [PATCH 1/9] intel/genxml: Fix typo in gen75.xml

Patches 1, 2, 3, 5, and 7 are Reviewed-by: Jason Ekstrand On Sep 30, 2016 3:33 PM, "Anuj Phogat" wrote: > Signed-off-by: Anuj Phogat > --- > src/intel/genxml/gen75.xml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [Mesa-dev] [PATCH 9/9] anv/gen7_pipeline: Check for ksp0 when setting BarycentricInterpolationMode in 3DSTATE_WM

On Sep 30, 2016 3:34 PM, "Anuj Phogat" wrote: > > Signed-off-by: Anuj Phogat > --- > src/intel/vulkan/gen7_pipeline.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/src/intel/vulkan/gen7_pipeline.c

Re: [Mesa-dev] [PATCH 8/9] anv/gen7_pipeline: Set few missing fields in 3DSTATE_PS

On Sep 30, 2016 3:34 PM, "Anuj Phogat" wrote: > > Signed-off-by: Anuj Phogat > --- > src/intel/vulkan/gen7_pipeline.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c >

Re: [Mesa-dev] [PATCH] mesa: use uint32_t rather than unsigned for xfb struct members

Seems reasonable. I'm generally a fan of explicitly sized types anyway. R-B On Sep 30, 2016 4:20 PM, "Timothy Arceri" wrote: > These structs will be written to disk as part of the shader cache > so use uint32_t just to be safe. > --- > src/mesa/main/mtypes.h | 20

[Mesa-dev] [PATCH mesa] egl: add missing error-checking to eglReleaseTexImage()

Signed-off-by: Eric Engestrom --- src/egl/main/eglsurface.c | 43 --- 1 file changed, 40 insertions(+), 3 deletions(-) diff --git a/src/egl/main/eglsurface.c b/src/egl/main/eglsurface.c index 231a5f0..2762503 100644 ---

[Mesa-dev] [PATCH] mesa: use uint32_t rather than unsigned for xfb struct members

These structs will be written to disk as part of the shader cache so use uint32_t just to be safe. --- src/mesa/main/mtypes.h | 20 ++-- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h index d1b0560..efcd50e 100644

Re: [Mesa-dev] [PATCH] anv/formats: Fix build on gcc-4 and earlier

I remember the first tone you fixed this but when I was going the ISL_SWIZZLE stuff, I couldn't find it in the git log so I went ahead and pushed the change. Thanks for fixing it again. Reviewed-by: Jason Ekstrand On Sep 30, 2016 1:00 PM,

Re: [Mesa-dev] [PATCH 6/9] anv/gen7_pipeline: Set few missing fields in 3DSTATE_VS

Flyby comments... On Sep 30, 2016 3:34 PM, "Anuj Phogat" wrote: > > Signed-off-by: Anuj Phogat > --- > src/intel/vulkan/gen7_pipeline.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/src/intel/vulkan/gen7_pipeline.c

[Mesa-dev] [PATCH 1/5] st/mesa: simplify some code in get_texture_format_swizzle()

There's no need to cast to st_texture_image. Just use gl_texture_image. --- src/mesa/state_tracker/st_atom_texture.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/mesa/state_tracker/st_atom_texture.c b/src/mesa/state_tracker/st_atom_texture.c index

[Mesa-dev] [PATCH 5/5] st/mesa: move all sampler view code into new st_sampler_view.[ch] files

Previously, the sampler view code was scattered across several different files. Note, the previous REALLOC(), FREE() for st_texture_object::sampler_views are replaced by realloc(), free() to avoid conflicting macros in Mesa vs. Gallium. --- src/mesa/Makefile.sources | 2 +

[Mesa-dev] [PATCH 2/5] st/mesa: add some const qualifiers in st_atom_texture.c

And minor code reformatting. --- src/mesa/state_tracker/st_atom_texture.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/src/mesa/state_tracker/st_atom_texture.c b/src/mesa/state_tracker/st_atom_texture.c index 113c0ed..86f1c62 100644 ---

[Mesa-dev] [PATCH 3/5] st/mesa: consolidate view format setup code

Before, we had code to compute the sampler view's format spread across two different functions: in update_single_texture() and st_get_texture_sampler_view_from_stobj(). Now it's all in one new function. Also, use _mesa_texture_base_format() to simplify the code. ---

[Mesa-dev] [PATCH 4/5] st/mesa: optimize pipe_sampler_view validation

Before, st_get_texture_sampler_view_from_stobj() did a lot of work to check if the texture parameters matched the sampler view (format, swizzle, min/max lod, first/last layer, etc). We did this every time we validated the texture state. Now, we use a ctx->Driver.TexParameter() callback and a

Re: [Mesa-dev] [PATCH 3/3] mesa: make _mesa_texture_buffer_range() static

On Fri, Sep 30, 2016 at 3:37 PM, Brian Paul wrote: > > Not called from any other file. Also, add a comment. > --- > src/mesa/main/teximage.c | 34 +++--- > src/mesa/main/teximage.h | 8 > 2 files changed, 19 insertions(+), 23 deletions(-)

[Mesa-dev] [PATCH 2/3] mesa: add const qualifier, comment on can_avoid_reallocation()

--- src/mesa/main/teximage.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c index 3f3d430..1d0a6a2 100644 --- a/src/mesa/main/teximage.c +++ b/src/mesa/main/teximage.c @@ -3571,8 +3571,15 @@

[Mesa-dev] [PATCH 1/3] mesa: add comment/assertion on get_tex_level_parameter_buffer()

--- src/mesa/main/texparam.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c index bdd3fcb..a814778 100644 --- a/src/mesa/main/texparam.c +++ b/src/mesa/main/texparam.c @@ -1486,6 +1486,9 @@ invalid_pname: } +/** + * Handle a

[Mesa-dev] [PATCH 3/3] mesa: make _mesa_texture_buffer_range() static

Not called from any other file. Also, add a comment. --- src/mesa/main/teximage.c | 34 +++--- src/mesa/main/teximage.h | 8 2 files changed, 19 insertions(+), 23 deletions(-) diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c index

[Mesa-dev] [PATCH 4/9] anv/gen8_pipeline: Use Alternate floating point mode when use_alt_mode is set in prog_data

Signed-off-by: Anuj Phogat --- src/intel/vulkan/gen8_pipeline.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/intel/vulkan/gen8_pipeline.c b/src/intel/vulkan/gen8_pipeline.c index 1abb97c..7a9a3cf 100644 --- a/src/intel/vulkan/gen8_pipeline.c

[Mesa-dev] [PATCH 8/9] anv/gen7_pipeline: Set few missing fields in 3DSTATE_PS

Signed-off-by: Anuj Phogat --- src/intel/vulkan/gen7_pipeline.c | 4 1 file changed, 4 insertions(+) diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c index 9411631..4bab0bc 100644 --- a/src/intel/vulkan/gen7_pipeline.c +++

[Mesa-dev] [PATCH 9/9] anv/gen7_pipeline: Check for ksp0 when setting BarycentricInterpolationMode in 3DSTATE_WM

Signed-off-by: Anuj Phogat --- src/intel/vulkan/gen7_pipeline.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c index 4bab0bc..e804002 100644 --- a/src/intel/vulkan/gen7_pipeline.c

[Mesa-dev] [PATCH 7/9] anv/gen7_pipeline: Move ksp{1, 2} state setting next to ksp0

Signed-off-by: Anuj Phogat --- src/intel/vulkan/gen7_pipeline.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c index f98aef5..9411631 100644 --- a/src/intel/vulkan/gen7_pipeline.c

[Mesa-dev] [PATCH 1/9] intel/genxml: Fix typo in gen75.xml

Signed-off-by: Anuj Phogat --- src/intel/genxml/gen75.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/genxml/gen75.xml b/src/intel/genxml/gen75.xml index 1debc3a..8e18007 100644 --- a/src/intel/genxml/gen75.xml +++

[Mesa-dev] [PATCH 6/9] anv/gen7_pipeline: Set few missing fields in 3DSTATE_VS

Signed-off-by: Anuj Phogat --- src/intel/vulkan/gen7_pipeline.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c index 8d133b6..f98aef5 100644 --- a/src/intel/vulkan/gen7_pipeline.c +++

[Mesa-dev] [PATCH 5/9] anv/gen7: Make use of local variable prog_data

Signed-off-by: Anuj Phogat --- src/mesa/drivers/dri/i965/gen7_vs_state.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/gen7_vs_state.c b/src/mesa/drivers/dri/i965/gen7_vs_state.c index 57e0419..d3bd9fd 100644 ---

[Mesa-dev] [PATCH 3/9] anv/gen8_pipeline: Fix typo in semicolon

Signed-off-by: Anuj Phogat --- src/intel/vulkan/gen8_pipeline.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/intel/vulkan/gen8_pipeline.c b/src/intel/vulkan/gen8_pipeline.c index 030f71e..1abb97c 100644 --- a/src/intel/vulkan/gen8_pipeline.c +++

[Mesa-dev] [PATCH 2/9] intel/genxml: Keep the value name 'Alternate' uniform across gen75.xml

Signed-off-by: Anuj Phogat --- src/intel/genxml/gen75.xml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/intel/genxml/gen75.xml b/src/intel/genxml/gen75.xml index 8e18007..b8f5a14 100644 --- a/src/intel/genxml/gen75.xml +++

Re: [Mesa-dev] [PATCH] Revert "st/vdpau: use linear layout for output surfaces"

On Fri, Sep 30, 2016 at 11:23 PM, Dave Airlie wrote: > On 1 October 2016 at 06:11, Marek Olšák wrote: >> On Fri, Sep 30, 2016 at 4:42 PM, Ilia Mirkin wrote: >>> So is this getting pushed, or should I push my "don't build vdpau for >>>

Re: [Mesa-dev] [PATCH] mesa: move _mesa_valid_to_render() to api_validate.c

On Fri, Sep 30, 2016 at 2:21 PM, Brian Paul wrote: > Almost all of the other drawing validation code is in api_validate.c > so put this function there as well. > --- > src/mesa/main/api_validate.c | 191 ++ > - >

Re: [Mesa-dev] [PATCH] Revert "st/vdpau: use linear layout for output surfaces"

On 1 October 2016 at 06:11, Marek Olšák wrote: > On Fri, Sep 30, 2016 at 4:42 PM, Ilia Mirkin wrote: >> So is this getting pushed, or should I push my "don't build vdpau for >> nouveau" patch? > > Please wait. > > Christian, your commit says it fixes a

Re: [Mesa-dev] [PATCH v2 4/5] gallium/radeon: emit relocations for query fences

Fixed all Blender issues on r600g/NI/Turks XT. Apart from that that it's already committed you have my Tb Thank you Nicolai! Dieter. Am 30.09.2016 11:52, schrieb Nicolai Hähnle: From: Nicolai Hähnle This is only needed for r600 which doesn't have

[Mesa-dev] [PATCH] mesa: move _mesa_valid_to_render() to api_validate.c

Almost all of the other drawing validation code is in api_validate.c so put this function there as well. --- src/mesa/main/api_validate.c | 191 ++- src/mesa/main/api_validate.h | 3 + src/mesa/main/context.c | 188

Re: [Mesa-dev] [PATCH] vl/dri3: handle the case of different GPU(v4.1)

On 16/09/2016 14:51, Nayan Deshmukh wrote: bool flushed; + bool is_different_gpu; }; I get a warning when I build about incompatible pointer type at the line fd = loader_get_user_preferred_fd(fd, >is_different_gpu); When compiling on 32 bits. I get this should be int. Axel

Re: [Mesa-dev] [PATCH] Revert "st/vdpau: use linear layout for output surfaces"

On Fri, Sep 30, 2016 at 4:42 PM, Ilia Mirkin wrote: > So is this getting pushed, or should I push my "don't build vdpau for > nouveau" patch? Please wait. Christian, your commit says it fixes a radeonsi issue, but Ilia says it breaks nouveau. Can we do something to

[Mesa-dev] [PATCH] anv/formats: Fix build on gcc-4 and earlier

From: Ville Syrjälä gcc-4 and earlier don't allow compound literals where a constant is required in -std=c99/gnu99 mode, so we can't use ISL_SWIZZLE() when populating the anv_formats[] array. There are a few ways around it: First one would be -std=c89/gnu89, but

Re: [Mesa-dev] [PATCH] Revert "gallium/hud: automatically print % if max_value == 100"

On 09/30/2016 12:17 PM, Marek Olšák wrote: From: Marek Olšák This reverts commit dbfeb0ec12d6550e68de1bcd164e422e79bccf2d. With max_value being rounded to 100, it's often wrong. Yeah, I noticed that too. Reviewed-by: Brian Paul ---

Re: [Mesa-dev] [Mesa-stable] [PATCH v2] egl: stop claiming support for pbuffer + msaa

On 09/30/2016 06:00 AM, Emil Velikov wrote: > On 30 September 2016 at 07:32, Tapani Pälli wrote: >> This fixes a crash in egl-create-msaa-pbuffer-surface Piglit test >> and same crash in many dEQP EGL tests. >> >> I also found that some Qt example did a workaround because

[Mesa-dev] [PATCH] autoconf: Make header install distinct for various APIs

This fixes a problem where GL headers would only get installed if glx was enabled. So if osmesa was enabled but not glx, then the GL headers required by osmesa would be missing from the install. Signed-off-by: Chuck Atkins --- configure.ac| 2 ++

[Mesa-dev] [PATCH] Revert "gallium/hud: automatically print % if max_value == 100"

From: Marek Olšák This reverts commit dbfeb0ec12d6550e68de1bcd164e422e79bccf2d. With max_value being rounded to 100, it's often wrong. --- src/gallium/auxiliary/hud/hud_context.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git

Re: [Mesa-dev] [PATCH] swr: Removed stalling SwrWaitForIdle from queries.

Reviewed-by: George Kyriazis > From: Bruce Cherniak > Subject: [PATCH] swr: Removed stalling SwrWaitForIdle from queries. Date: September 27, 2016 at 1:27:08 PM CDT To:

Re: [Mesa-dev] [PATCH] HUD: Add support for block I/O, network I/O and lmsensor stats

Hello Emil, >> +AC_ARG_ENABLE([gallium-extra-hud], >> +[AS_HELP_STRING([--enable-gallium-extra-hud], >> +[enable HUD block/NIC I/O HUD stats support >> @<:@default=disabled@:>@])], >> +[enable_gallium_extra_hud="$enableval"], >> +[enable_gallium_extra_hud=no]) > Looking at

Re: [Mesa-dev] [PATCH 3/3] glx: rename choose_visual(), drop const argument

On 09/30/2016 03:01 AM, Emil Velikov wrote: > From: Emil Velikov > > The function deals with fb (style) configs, thus using "visual" > in the name is misleading. Which in itself had led to the use of > fbconfig_style_tags argument. > > Rename the function to reflect

Re: [Mesa-dev] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

On Sep 30, 2016 7:57 AM, "Ian Romanick" wrote: > > On 09/30/2016 06:23 AM, Brian Paul wrote: > > On 09/30/2016 04:59 AM, Emil Velikov wrote: > >> On 30 September 2016 at 03:31, Timothy Arceri > >> wrote: > >>> On Thu, 2016-09-29 at 19:17 -0700,

Re: [Mesa-dev] [PATCH] gallium/hud: Add support for CPU frequency monitoring

Looks OK to me. Reviewed-by: Brian Paul Will push soon. -Brian On 09/30/2016 05:58 AM, Steven Toth wrote: Detect all of the CPUs in the system. Expose metrics for min, max and current frequency in Hz. Signed-off-by: Steven Toth ---

Re: [Mesa-dev] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

On 30 September 2016 at 15:57, Ian Romanick wrote: > On 09/30/2016 06:23 AM, Brian Paul wrote: >> On 09/30/2016 04:59 AM, Emil Velikov wrote: >>> On 30 September 2016 at 03:31, Timothy Arceri >>> wrote: On Thu, 2016-09-29 at 19:17 -0700,

Re: [Mesa-dev] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

On Fri, Sep 30, 2016 at 5:14 PM, Karol Herbst wrote: > 2016-09-30 16:57 GMT+02:00 Ian Romanick : >> On 09/30/2016 06:23 AM, Brian Paul wrote: >>> On 09/30/2016 04:59 AM, Emil Velikov wrote: On 30 September 2016 at 03:31, Timothy Arceri

Re: [Mesa-dev] [PATCH] docs: update the list of Mesa major versions and API support

On 09/30/2016 07:47 AM, Emil Velikov wrote: On 30 September 2016 at 14:23, Brian Paul wrote: --- docs/intro.html | 25 + 1 file changed, 25 insertions(+) diff --git a/docs/intro.html b/docs/intro.html index 4b36562..6c91e3f 100644 ---

Re: [Mesa-dev] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

2016-09-30 16:57 GMT+02:00 Ian Romanick : > On 09/30/2016 06:23 AM, Brian Paul wrote: >> On 09/30/2016 04:59 AM, Emil Velikov wrote: >>> On 30 September 2016 at 03:31, Timothy Arceri >>> wrote: On Thu, 2016-09-29 at 19:17 -0700, Jason

[Mesa-dev] [PATCH v3] glsl: optimize copy_propagation_elements pass

From: Tapani Pälli Changes make copy_propagation_elements pass faster, reducing link time spent in test case of bug 94477. Does not fix the actual issue but brings down the total time. No regressions seen in CI. v2 (idr): Formatting / whitespace fixes. Embed the acp_ref

Re: [Mesa-dev] [PATCH] i915/i965: remove commented out warning

Reviewed-by: Ian Romanick On 09/29/2016 11:37 PM, Timothy Arceri wrote: > The warning was also the wrong location, it should have been > in the else. > --- > src/mesa/drivers/dri/i915/intel_fbo.h | 4 +--- > src/mesa/drivers/dri/i965/intel_fbo.h | 4 +--- > 2 files

[Mesa-dev] [PATCH] glsl: optimize copy_propagation_elements pass

From: Tapani Pälli Changes make copy_propagation_elements pass faster, reducing link time spent in test case of bug 94477. Does not fix the actual issue but brings down the total time. No regressions seen in CI. v2 (idr): Formatting / whitespace fixes. Embed the acp_ref

Re: [Mesa-dev] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

On 09/30/2016 06:23 AM, Brian Paul wrote: > On 09/30/2016 04:59 AM, Emil Velikov wrote: >> On 30 September 2016 at 03:31, Timothy Arceri >> wrote: >>> On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote: >>> >>> On Sep 29, 2016 5:14 PM, "Timothy Arceri"

Re: [Mesa-dev] [PATCH] Revert "st/vdpau: use linear layout for output surfaces"

So is this getting pushed, or should I push my "don't build vdpau for nouveau" patch? On Wed, Sep 14, 2016 at 11:58 PM, Dave Airlie wrote: > From: Dave Airlie > > This reverts commit d180de35320eafa3df3d76f0e82b332656530126. > > This is a radeon specific

Re: [Mesa-dev] [PATCH] HUD: Add support for block I/O, network I/O and lmsensor stats

Hi Steven, On 28 September 2016 at 19:58, Steven Toth wrote: > V8: Feedback based on peer review > convert if block into a switch > Constify some func args > > V7: Increase precision when measuring lmsensors volts > Flatten patch series. > > V6: Feedback based

Re: [Mesa-dev] [PATCH v2] util/slab: re-design to allow migration between pools (v2)

On Fri, Sep 30, 2016 at 3:08 PM, Bas Nieuwenhuizen wrote: > On Fri, Sep 30, 2016 at 2:13 PM, Marek Olšák wrote: >> intptr_t reads and writes aren't atomic. p_atomic_set and >> p_atomic_read functions don't do anything for atomicity. See: >> >> #define

Re: [Mesa-dev] [PATCH] docs: update the list of Mesa major versions and API support

On 30 September 2016 at 14:23, Brian Paul wrote: > --- > docs/intro.html | 25 + > 1 file changed, 25 insertions(+) > > diff --git a/docs/intro.html b/docs/intro.html > index 4b36562..6c91e3f 100644 > --- a/docs/intro.html > +++ b/docs/intro.html > @@

Re: [Mesa-dev] [PATCH 7/7] radeonsi: optionally run the LLVM IR verifier pass

I'm not so familiar with the LLVM basic block API, but this series makes sense to me. For the series: Reviewed-by: Marek Olšák Marek On Thu, Sep 29, 2016 at 3:15 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > This is

Re: [Mesa-dev] [Mesa-stable] [PATCH] egl/drm, gallium: replace all calls to dup(2) with fcntl(F_DUPFD_CLOEXEC)

On 30 September 2016 at 06:47, Matt Whitlock wrote: > Without this fix, duplicated file descriptors leak into child processes. > See commit aaac913e901229d11a1894f6aaf646de6b1a542c for one instance > where the same fix was employed. > > Cc:

Re: [Mesa-dev] [PATCH 6/6] virgl: use the new parent/child pools for transfers

On Fri, Sep 30, 2016 at 3:00 PM, Nicolai Hähnle wrote: > On 30.09.2016 14:58, Marek Olšák wrote: >> >> For patches 2-6: >> >> Reviewed-by: Marek Olšák > > > Thanks. > >> For patch 1, you'll have to decide whether atomicity is necessary, but >> at least

Re: [Mesa-dev] [Mesa-announce] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

On Fri, Sep 30, 2016 at 3:23 PM, Brian Paul wrote: > On 09/30/2016 04:59 AM, Emil Velikov wrote: >> >> On 30 September 2016 at 03:31, Timothy Arceri >> wrote: >>> >>> On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote: >>> >>> On Sep 29, 2016

Re: [Mesa-dev] Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?)

On Fri, Sep 30, 2016 at 3:31 AM, Dave Airlie wrote: > On 30 September 2016 at 01:07, Jason Ekstrand wrote: >> On Sep 29, 2016 7:56 AM, "Emil Velikov" wrote: >>> >>> On 28 September 2016 at 19:53, Marek Olšák

Re: [Mesa-dev] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

On 09/30/2016 04:59 AM, Emil Velikov wrote: On 30 September 2016 at 03:31, Timothy Arceri wrote: On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote: On Sep 29, 2016 5:14 PM, "Timothy Arceri" wrote: On Thu, 2016-09-29 at 15:56

[Mesa-dev] [PATCH] docs: update the list of Mesa major versions and API support

--- docs/intro.html | 25 + 1 file changed, 25 insertions(+) diff --git a/docs/intro.html b/docs/intro.html index 4b36562..6c91e3f 100644 --- a/docs/intro.html +++ b/docs/intro.html @@ -173,6 +173,27 @@ of the OpenGL specification is implemented. +Version 12.x

Re: [Mesa-dev] [PATCH v2] util/slab: re-design to allow migration between pools (v2)

On Fri, Sep 30, 2016 at 2:13 PM, Marek Olšák wrote: > intptr_t reads and writes aren't atomic. p_atomic_set and > p_atomic_read functions don't do anything for atomicity. See: > > #define p_atomic_set(_v, _i) (*(_v) = (_i)) > #define p_atomic_read(_v) (*(_v)) That

[Mesa-dev] [Bug 97879] [amdgpu] Rocket League: long hangs (several seconds) when loading assets (models/textures/shaders?)

https://bugs.freedesktop.org/show_bug.cgi?id=97879 Vedran Rodic changed: What|Removed |Added CC||vro...@gmail.com --

Re: [Mesa-dev] [PATCH 6/6] virgl: use the new parent/child pools for transfers

On 30.09.2016 14:58, Marek Olšák wrote: For patches 2-6: Reviewed-by: Marek Olšák Thanks. For patch 1, you'll have to decide whether atomicity is necessary, but at least p_atomic_read and p_atomic_set shouldn't be used on intptr_t. I do believe atomicity is

Re: [Mesa-dev] [PATCH v2] egl: stop claiming support for pbuffer + msaa

On 30 September 2016 at 07:32, Tapani Pälli wrote: > This fixes a crash in egl-create-msaa-pbuffer-surface Piglit test > and same crash in many dEQP EGL tests. > > I also found that some Qt example did a workaround because of this > crash:

Re: [Mesa-dev] [PATCH 6/6] virgl: use the new parent/child pools for transfers

For patches 2-6: Reviewed-by: Marek Olšák For patch 1, you'll have to decide whether atomicity is necessary, but at least p_atomic_read and p_atomic_set shouldn't be used on intptr_t. Marek On Tue, Sep 27, 2016 at 8:21 PM, Nicolai Hähnle wrote: >

[Mesa-dev] [PATCH] egl: remove explicit config_id management from dri2_add_config()

From: Emil Velikov Currently we only saved the id to memcpy the whole _EGLConfig to write back the exact same id value. Remove the unneeded and confusing/misleading code. Signed-off-by: Emil Velikov --- src/egl/drivers/dri2/egl_dri2.c |

Re: [Mesa-dev] [PATCH v2] util/slab: re-design to allow migration between pools (v2)

intptr_t reads and writes aren't atomic. p_atomic_set and p_atomic_read functions don't do anything for atomicity. See: #define p_atomic_set(_v, _i) (*(_v) = (_i)) #define p_atomic_read(_v) (*(_v)) Marek On Wed, Sep 28, 2016 at 9:47 AM, Nicolai Hähnle wrote: > From: Nicolai

Re: [Mesa-dev] [PATCH] gallium/hud: Add power sensor support

> > Reviewed-by: Brian Paul > > and pushed to master. Thanks. Thank you Brian. -- Steven Toth - Kernel Labs http://www.kernellabs.com ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

[Mesa-dev] [PATCH] gallium/hud: Add support for CPU frequency monitoring

Detect all of the CPUs in the system. Expose metrics for min, max and current frequency in Hz. Signed-off-by: Steven Toth --- src/gallium/auxiliary/Makefile.sources | 1 + src/gallium/auxiliary/hud/hud_context.c | 13 ++ src/gallium/auxiliary/hud/hud_cpufreq.c | 266

Re: [Mesa-dev] [PATCH] glsl: optimize copy_propagation_elements pass

On 09/30/2016 03:18 AM, Ian Romanick wrote: On 09/29/2016 12:17 AM, Tapani Pälli wrote: On 09/28/2016 06:14 PM, Ian Romanick wrote: On 09/16/2016 06:21 PM, Tapani Pälli wrote: Changes make copy_propagation_elements pass faster, reducing link time spent in test case of bug 94477. Does not

Re: [Mesa-dev] [PATCH 88/88] i965: handle 32bit and 64bit version of shader cache objects

On Wed, 2016-09-28 at 11:42 -0700, Kenneth Graunke wrote: > On Wednesday, September 28, 2016 9:00:50 AM PDT Eric Anholt wrote: > > > > Timothy Arceri writes: > > > > > > > > On Sun, 2016-09-25 at 19:51 -0700, Kenneth Graunke wrote: > > > > > > > > On Saturday,

Re: [Mesa-dev] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

On 30 September 2016 at 03:31, Timothy Arceri wrote: > On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote: > > On Sep 29, 2016 5:14 PM, "Timothy Arceri" > wrote: >> >> On Thu, 2016-09-29 at 15:56 +0100, Emil Velikov wrote: >> > On

Re: [Mesa-dev] Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?)

On 30 September 2016 at 02:31, Dave Airlie wrote: > On 30 September 2016 at 01:07, Jason Ekstrand wrote: >> On Sep 29, 2016 7:56 AM, "Emil Velikov" wrote: >>> >>> On 28 September 2016 at 19:53, Marek Olšák

Re: [Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

Well, it depends on both the size and the type. Sub-allocation can be used up to 16KB with the current setting, but the override is clearly the right solution here :) Cheers, Nicolai On 30.09.2016 12:24, Christian König wrote: Oh, so you use the allocation type instead of the size to

Re: [Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

Oh, so you use the allocation type instead of the size to determine if a buffer can be sub-allocated or not? I wasn't expecting that, in this case even the DPB would have ended up in the sub-allocator and that would clearly blow up immediately. Good that we stumbled over this. Regards,

Re: [Mesa-dev] [PATCH 3/3] glx: rename choose_visual(), drop const argument

For the series: Reviewed-by: Marek Olšák Marek On Fri, Sep 30, 2016 at 12:01 PM, Emil Velikov wrote: > From: Emil Velikov > > The function deals with fb (style) configs, thus using "visual" > in the name is

Re: [Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

Patches #1-#3 of this series are Reviewed-by: Christian König . Regards, Christian. Am 30.09.2016 um 11:52 schrieb Nicolai Hähnle: From: Nicolai Hähnle Cc: Christian König Bugzilla:

Re: [Mesa-dev] [PATCH] gallium/radeon: fix crash/regression in performance counters

Reviewed-by: Marek Olšák Marek On Fri, Sep 30, 2016 at 12:04 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > Cc: Michel Dänzer > --- > src/gallium/drivers/radeon/r600_perfcounter.c | 9 + > 1

Re: [Mesa-dev] Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?)

On 30.09.2016 03:31, Dave Airlie wrote: On 30 September 2016 at 01:07, Jason Ekstrand wrote: On Sep 29, 2016 7:56 AM, "Emil Velikov" wrote: On 28 September 2016 at 19:53, Marek Olšák wrote: Hi, It's been almost 4 months

[Mesa-dev] [PATCH] gallium/radeon: fix crash/regression in performance counters

From: Nicolai Hähnle Cc: Michel Dänzer --- src/gallium/drivers/radeon/r600_perfcounter.c | 9 + 1 file changed, 9 insertions(+) diff --git a/src/gallium/drivers/radeon/r600_perfcounter.c b/src/gallium/drivers/radeon/r600_perfcounter.c

[Mesa-dev] [PATCH 3/3] glx: rename choose_visual(), drop const argument

From: Emil Velikov The function deals with fb (style) configs, thus using "visual" in the name is misleading. Which in itself had led to the use of fbconfig_style_tags argument. Rename the function to reflect what it does and drop the unneeded argument.

[Mesa-dev] [PATCH 2/3] glx: return GL_FALSE from glx_screen_init where applicable.

From: Emil Velikov Return GL_FALSE if we fail to find any fb/visual configs, otherwise we end up with all sorts of chaos further down the GLX stack. Signed-off-by: Emil Velikov --- src/glx/glxext.c | 7 +-- 1 file changed, 5

[Mesa-dev] [PATCH 1/3] glx: correctly mask the drawableType for GLX_ARB_fbconfig_float

From: Emil Velikov The comment/spec says - only for pbuffer drawables, while the code clears the window/pixmap bit. Practise what you preach and apply the trivial tweak. In practise this should not cause functional change. Signed-off-by: Emil Velikov

Re: [Mesa-dev] [PATCH v2 4/5] gallium/radeon: emit relocations for query fences

Reviewed-by: Marek Olšák Marek On Fri, Sep 30, 2016 at 11:52 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > This is only needed for r600 which doesn't have ARB_query_buffer_object and > therefore wouldn't really need the

Re: [Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

I should add that textures are already never sub-allocated, so with this patch all the buffers used with UVD and VCE should be covered. Nicolai On 30.09.2016 11:52, Nicolai Hähnle wrote: From: Nicolai Hähnle Cc: Christian König Bugzilla:

[Mesa-dev] [PATCH v2 4/5] gallium/radeon: emit relocations for query fences

From: Nicolai Hähnle This is only needed for r600 which doesn't have ARB_query_buffer_object and therefore wouldn't really need the fences, but let's be optimistic about filling in this feature gap eventually. Cc: Dieter Nützel ---

[Mesa-dev] [PATCH v2 3/5] radeon/uvd: adjust the buffer offset when relocation is used

From: Nicolai Hähnle We don't plan to use sub-allocated buffers with UVD, but just in case one slips through, this increases the chances of things working out anyway. --- src/gallium/drivers/radeon/radeon_uvd.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Mesa-dev] [PATCH v2 2/5] radeon/vce: adjust the buffer offset when relocation is used

From: Nicolai Hähnle We don't plan to use sub-allocated buffers with VCE, but just in case one slips through, this increases the chances of things working out anyway. --- src/gallium/drivers/radeon/radeon_vce.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Mesa-dev] [PATCH v2 1/5] radeon/video: don't use sub-allocated buffers

From: Nicolai Hähnle Cc: Christian König Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97976 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97969 --- src/gallium/drivers/radeon/r600_buffer_common.c | 3 +++

[Mesa-dev] [PATCH v2 5/5] gallium/radeon: update documentation of buffer_get_virtual_address

From: Nicolai Hähnle Reviewed-by: Marek Olšák --- src/gallium/drivers/radeon/radeon_winsys.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/gallium/drivers/radeon/radeon_winsys.h b/src/gallium/drivers/radeon/radeon_winsys.h index

Re: [Mesa-dev] Mesa 13.0.0 release plan (Was Re: Mesa 12.1.0 release plan (Was Re: Next Mesa release, anyone?))

On Fri, Sep 30, 2016 at 4:17 AM, Jason Ekstrand wrote: > On Sep 29, 2016 5:14 PM, "Timothy Arceri" > wrote: >> >> On Thu, 2016-09-29 at 15:56 +0100, Emil Velikov wrote: >> > On 28 September 2016 at 19:53, Marek Olšák wrote:

Re: [Mesa-dev] [PATCH v2] egl: stop claiming support for pbuffer + msaa

Reviewed-by: Marek Olšák Marek On Fri, Sep 30, 2016 at 8:32 AM, Tapani Pälli wrote: > This fixes a crash in egl-create-msaa-pbuffer-surface Piglit test > and same crash in many dEQP EGL tests. > > I also found that some Qt example did a workaround

  1   2   >