Re: [Mesa-dev] [PATCH] st/clover: Define __OPENCL_VERSION__ on the device side

2016-08-31 Thread Jan Vesely
On Wed, 2016-08-31 at 15:53 +0200, Serge Martin wrote: > On Wednesday 31 August 2016 12:39:23 Vedran Miletić wrote: > > > > On 08/28/2016 04:42 PM, Niels Ole Salscheider wrote: > > > > > > This is required by the OpenCL standard. > > > > > > Signed-off-by: Niels Ole Salscheider

[Mesa-dev] [PATCH 11/12] i965/blorp: Sanity check all layers before actual clear

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_blorp.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c index 6339932..ce8b848 100644 ---

[Mesa-dev] i965: Hardware assisted layered clears

2016-08-31 Thread Topi Pohjolainen
This series requests hardware to generate as many copies of drawing primitives as there are layers to be cleared instead of submitting separate render passes for the layers. In case of patches 2, 3, 8 and 9 I'm open to suggestions. There probably are other (better?) alternatives. CC: Jason

[Mesa-dev] [PATCH 04/12] i965/rbc: Allocate mcs directly

2016-08-31 Thread Topi Pohjolainen
From: Topi Pohjolainen such as we do for compressed msaa. In case of non-compressed simgle sampled buffers the allocation of mcs is deferred until there is actually a clear operation that needs the mcs. In case of render buffer compression the mcs buffer always needed

[Mesa-dev] [PATCH 08/12] intel/blorp: Allow multiple layers

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/intel/blorp/blorp.c b/src/intel/blorp/blorp.c index 4dbba01..17c1ff4 100644 --- a/src/intel/blorp/blorp.c +++

[Mesa-dev] [PATCH 05/12] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-08-31 Thread Topi Pohjolainen
Originally re-clears where skipped but when lossless compression was introduced the re-clears where errorneously enabled also for non-compressed fast clears. Signed-off-by: Topi Pohjolainen CC: Ben Widawsky CC: Kenneth Graunke

[Mesa-dev] [PATCH 12/12] i965/blorp: Use hw generetad primitive copies for layered clears

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_blorp.c | 36 --- 1 file changed, 12 insertions(+), 24 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c index

[Mesa-dev] [PATCH 09/12] intel/blorp: Allow single slice converter to suppress number of layers

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp_blit.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/intel/blorp/blorp_blit.c b/src/intel/blorp/blorp_blit.c index 170c381..026061b 100644 --- a/src/intel/blorp/blorp_blit.c +++

[Mesa-dev] [PATCH 06/12] i965/meta: Split conversion of color and setting it

2016-08-31 Thread Topi Pohjolainen
And fix a mangled comment while at it. Signed-off-by: Topi Pohjolainen CC: Ben Widawsky CC: Jason Ekstrand --- src/mesa/drivers/dri/i965/brw_blorp.c | 7 +++- src/mesa/drivers/dri/i965/brw_meta_util.c | 56

[Mesa-dev] [PATCH 04/12] i965/rbc: Allocate mcs directly

2016-08-31 Thread Topi Pohjolainen
such as we do for compressed msaa. In case of non-compressed simgle sampled buffers the allocation of mcs is deferred until there is actually a clear operation that needs the mcs. In case of render buffer compression the mcs buffer always needed and there is no real reason to defer the allocation.

[Mesa-dev] [PATCH 06/12] i965/meta: Split conversion of color and setting it

2016-08-31 Thread Topi Pohjolainen
And fix a mangled comment while at it. Signed-off-by: Topi Pohjolainen CC: Ben Widawsky CC: Jason Ekstrand --- src/mesa/drivers/dri/i965/brw_blorp.c | 7 +++- src/mesa/drivers/dri/i965/brw_meta_util.c | 56

[Mesa-dev] [PATCH 02/12] i965/rbc: Set aux surface unconditionally for sampling engine

2016-08-31 Thread Topi Pohjolainen
Once mcs buffer gets allocated without delay for lossless compression (same as we do for msaa), one gets regression in: GL45-CTS.texture_barrier_ARB.same-texel-rw Setting the auxiliary surface for both sampling engine and data port seems to fix this. I haven't found any hardware documentation

[Mesa-dev] [PATCH 11/12] i965/blorp: Sanity check all layers before actual clear

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_blorp.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c index 6339932..ce8b848 100644 ---

[Mesa-dev] [PATCH 03/12] isl/gen8+: Allow 3D auxiliary surfaces

2016-08-31 Thread Topi Pohjolainen
Otherwise once mcs buffer gets allocated without delay for lossless compression (same as we do for msaa), assert starts to fire in piglit case: tex3d. The test uses depth of one which is in fact supported even now. Signed-off-by: Topi Pohjolainen ---

[Mesa-dev] [PATCH 12/12] i965/blorp: Use hw generetad primitive copies for layered clears

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/mesa/drivers/dri/i965/brw_blorp.c | 36 --- 1 file changed, 12 insertions(+), 24 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp.c b/src/mesa/drivers/dri/i965/brw_blorp.c index

[Mesa-dev] [PATCH 07/12] i965/blorp: Instruct vertex fetcher to provide prim instance id

2016-08-31 Thread Topi Pohjolainen
This will indicate target layer (Render Target Array Index) needed for layered clears. v2: Use 3DSTATE_VF_SGVS for gen8+ Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp_genX_exec.h | 26 ++ 1 file changed, 22 insertions(+), 4

[Mesa-dev] [PATCH 08/12] intel/blorp: Allow multiple layers

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/intel/blorp/blorp.c b/src/intel/blorp/blorp.c index 4dbba01..17c1ff4 100644 --- a/src/intel/blorp/blorp.c +++

[Mesa-dev] [PATCH 10/12] intel/blorp: Add plumbing for setting color clear layer count

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp.h | 6 +++--- src/intel/blorp/blorp_clear.c | 16 +--- src/mesa/drivers/dri/i965/brw_blorp.c | 6 +++--- 3 files changed, 15 insertions(+), 13 deletions(-) diff --git

[Mesa-dev] [PATCH 01/12] i965/rbc: Allow integer formats as advertised in isl_format.c

2016-08-31 Thread Topi Pohjolainen
From: Topi Pohjolainen Blorp consults brw_is_color_fast_clear_compatible() to see if any restrictions apply for fast clear in addition to the capablities advertised in isl_format.c::format_info[]. On Gen8+ integer formats are backlisted for plain old fast clear but

[Mesa-dev] [PATCH 10/12] intel/blorp: Add plumbing for setting color clear layer count

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp.h | 6 +++--- src/intel/blorp/blorp_clear.c | 16 +--- src/mesa/drivers/dri/i965/brw_blorp.c | 6 +++--- 3 files changed, 15 insertions(+), 13 deletions(-) diff --git

[Mesa-dev] [PATCH 09/12] intel/blorp: Allow single slice converter to suppress number of layers

2016-08-31 Thread Topi Pohjolainen
Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp_blit.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/intel/blorp/blorp_blit.c b/src/intel/blorp/blorp_blit.c index 170c381..026061b 100644 --- a/src/intel/blorp/blorp_blit.c +++

[Mesa-dev] [PATCH 02/12] i965/rbc: Set aux surface unconditionally for sampling engine

2016-08-31 Thread Topi Pohjolainen
Once mcs buffer gets allocated without delay for lossless compression (same as we do for msaa), one gets regression in: GL45-CTS.texture_barrier_ARB.same-texel-rw Setting the auxiliary surface for both sampling engine and data port seems to fix this. I haven't found any hardware documentation

[Mesa-dev] [PATCH 05/12] i965/blorp: Skip redundant re-fast clear for non-compressed

2016-08-31 Thread Topi Pohjolainen
Originally re-clears where skipped but when lossless compression was introduced the re-clears where errorneously enabled also for non-compressed fast clears. Signed-off-by: Topi Pohjolainen CC: Ben Widawsky CC: Kenneth Graunke

[Mesa-dev] [PATCH 07/12] i965/blorp: Instruct vertex fetcher to provide prim instance id

2016-08-31 Thread Topi Pohjolainen
This will indicate target layer (Render Target Array Index) needed for layered clears. v2: Use 3DSTATE_VF_SGVS for gen8+ Signed-off-by: Topi Pohjolainen --- src/intel/blorp/blorp_genX_exec.h | 26 ++ 1 file changed, 22 insertions(+), 4

[Mesa-dev] [PATCH 03/12] isl/gen8+: Allow 3D auxiliary surfaces

2016-08-31 Thread Topi Pohjolainen
Otherwise once mcs buffer gets allocated without delay for lossless compression (same as we do for msaa), assert starts to fire in piglit case: tex3d. The test uses depth of one which is in fact supported even now. Signed-off-by: Topi Pohjolainen ---

[Mesa-dev] i965: Hardware assisted layered clears

2016-08-31 Thread Topi Pohjolainen
This series requests hardware to generate as many copies of drawing primitives as there are layers to be cleared instead of submitting separate render passes for the layers. In case of patches 2, 3, 8 and 9 I'm open to suggestions. There probably are other (better?) alternatives. CC: Jason

[Mesa-dev] [PATCH 01/12] i965/rbc: Allow integer formats as advertised in isl_format.c

2016-08-31 Thread Topi Pohjolainen
Blorp consults brw_is_color_fast_clear_compatible() to see if any restrictions apply for fast clear in addition to the capablities advertised in isl_format.c::format_info[]. On Gen8+ integer formats are backlisted for plain old fast clear but there is no reason why lossless compression shouldn't

[Mesa-dev] [PATCH] gallium/docs: clarify render_condition_enabled parameter to clear functions

2016-08-31 Thread Brian Paul
If false, it means do the clear unconditionally. --- src/gallium/docs/source/context.rst | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/docs/source/context.rst b/src/gallium/docs/source/context.rst index 1b045fa..13dd606 100644 --- a/src/gallium/docs/source/context.rst +++

[Mesa-dev] [PATCH 1/2] nv50: remove unused nv50_program::immd_size field

2016-08-31 Thread Samuel Pitoiset
Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nv50/nv50_program.h | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_program.h b/src/gallium/drivers/nouveau/nv50/nv50_program.h index fc9ada4..009d41f 100644 ---

[Mesa-dev] [PATCH 2/2] nvc0: remove an attempt at uploading all IMMD into a CB

2016-08-31 Thread Samuel Pitoiset
This has never been used because info->immd.bufSize is always 0 and anyways this is an experimental code which has never been completed. This gets rid of some unused code in the program validation process. Signed-off-by: Samuel Pitoiset ---

Re: [Mesa-dev] [PATCH 2/2] nvc0: remove an attempt at uploading all IMMD into a CB

2016-08-31 Thread Ilia Mirkin
Series is Reviewed-by: Ilia Mirkin I think this was an attempt to feed immediates via a constbuf. Not sure what the advantage is, although at least on nv50 many instructions can't take an immediate. Like you said, this was never completed, I think it's fine to remove. On

Re: [Mesa-dev] [PATCH] loader: fix sysfs uevent file parsing

2016-08-31 Thread Eric Engestrom
On Mon, Jun 20, 2016 at 11:46:49AM +0100, Eric Engestrom wrote: > On Fri, Jun 17, 2016 at 06:29:40PM +0100, Frank Binns wrote: > > When trying to get a device name for an fd using sysfs, it would always fail > > as it was expecting key/value pairs to be delimited by '\0', which is not > > the > >

Re: [Mesa-dev] [PATCH] i965: Pass start_offset to brw_set_uip_jip().

2016-08-31 Thread Iago Toral
On Tue, 2016-08-30 at 12:08 -0700, Matt Turner wrote: > Without this, we would pass over the instructions in the SIMD8 > program > (which is located earlier in the buffer) when brw_set_uip_jip() is > called to handle the SIMD16 program. oh, right. > The assertion about compacted control flow was

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Eero Tamminen
Hi, On 31.08.2016 16:37, Marek Olšák wrote: ... OK. I'm afraid malloc/calloc/realloc/free from libjemalloc.so will conflict with libc and Steam. > When building jemalloc from source, there is an option to add a function prefix. There are basically two options at the moment: - my allocator:

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Emil Velikov
On 31 August 2016 at 14:37, Marek Olšák wrote: > On Wed, Aug 31, 2016 at 3:02 PM, Emil Velikov > wrote: >> On 31 August 2016 at 13:40, Marek Olšák wrote: >>> On Aug 31, 2016 5:02 AM, "Michel Dänzer" wrote:

Re: [Mesa-dev] [PATCH] loader/dri3: Overhaul dri3_update_num_back

2016-08-31 Thread Jason Ekstrand
On Aug 19, 2016 12:07 AM, "Michel Dänzer" wrote: > > From: Michel Dänzer > > Always use 3 buffers when flipping. With only 2 buffers, we have to wait > for a flip to complete (which takes non-0 time even with asynchronous > flips) before we can start

[Mesa-dev] [PATCH 7/7] st/omx/dec: enable hevc omx decode support

2016-08-31 Thread Leo Liu
Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/vid_dec.c | 23 ++- src/gallium/state_trackers/omx/vid_dec.h | 3 +++ 2 files changed, 25 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/omx/vid_dec.c

[Mesa-dev] [PATCH 1/7] st/omx/dec: add initial omx hevc support

2016-08-31 Thread Leo Liu
Mainly based on the h264 implementation. Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/Makefile.sources | 1 + src/gallium/state_trackers/omx/vid_dec.h| 11 ++ src/gallium/state_trackers/omx/vid_dec_h265.c | 148 3 files

[Mesa-dev] [PATCH 3/7] st/omx/dec/h265: add picture parameter sets

2016-08-31 Thread Leo Liu
Specified by subclause 7.3.2.3 Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/vid_dec_h265.c | 98 +++ 1 file changed, 98 insertions(+) diff --git a/src/gallium/state_trackers/omx/vid_dec_h265.c

[Mesa-dev] [PATCH 4/7] st/omx/dec/h265: add slice header

2016-08-31 Thread Leo Liu
Specified by subclause 7.3.6.1 Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/vid_dec.h | 3 + src/gallium/state_trackers/omx/vid_dec_h265.c | 215 +- 2 files changed, 216 insertions(+), 2 deletions(-) diff --git

[Mesa-dev] [PATCH 5/7] st/omx/dec/h265: add short term reference picture sets

2016-08-31 Thread Leo Liu
Specified by subclause 7.3.7 Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/vid_dec_h265.c | 114 +- 1 file changed, 110 insertions(+), 4 deletions(-) diff --git a/src/gallium/state_trackers/omx/vid_dec_h265.c

[Mesa-dev] [PATCH 2/7] st/omx/dec/h265: add sequence parameter sets

2016-08-31 Thread Leo Liu
Specified by subclause 7.3.2.2 Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/vid_dec.h | 2 + src/gallium/state_trackers/omx/vid_dec_h265.c | 295 ++ 2 files changed, 297 insertions(+) diff --git

[Mesa-dev] [PATCH 6/7] st/omx/dec/h265: get the reference list for uvd

2016-08-31 Thread Leo Liu
Signed-off-by: Leo Liu --- src/gallium/state_trackers/omx/vid_dec_h265.c | 39 +++ 1 file changed, 39 insertions(+) diff --git a/src/gallium/state_trackers/omx/vid_dec_h265.c b/src/gallium/state_trackers/omx/vid_dec_h265.c index df2dc05..7c0f75d 100644

[Mesa-dev] [PATCH 0/7] omx hevc decode support

2016-08-31 Thread Leo Liu
This set implements hevc decode for omx, it includes basic structures from h264 implementation and hevc specific sps, pps, slice header and reference picture sets, as well as what's required by uvd. Leo Liu (7): st/omx/dec: add initial omx hevc support st/omx/dec/h265: add sequence parameter

Re: [Mesa-dev] [PATCH 3/2] i965: Merge gen7_clip_state atom into gen6_clip_state atom.

2016-08-31 Thread Iago Toral
On Tue, 2016-08-30 at 21:51 -0700, Kenneth Graunke wrote: > The original motivation was that gen6_clip_state ignored _NEW_POLYGON > as it didn't care about early culling.  The only other change was > that > Gen6 ignored BRW_NEW_TES_PROG_DATA as it doesn't have tessellation > shaders, but listening

Re: [Mesa-dev] [PATCH] st/clover: Define __OPENCL_VERSION__ on the device side

2016-08-31 Thread Serge Martin
On Wednesday 31 August 2016 12:39:23 Vedran Miletić wrote: > On 08/28/2016 04:42 PM, Niels Ole Salscheider wrote: > > This is required by the OpenCL standard. > > > > Signed-off-by: Niels Ole Salscheider > Reviewed-by: Vedran Miletić > >

[Mesa-dev] [Bug 97549] [SNB, BXT] up to 40% perf drop from "loader/dri3: Overhaul dri3_update_num_back" commit

2016-08-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97549 Bug ID: 97549 Summary: [SNB, BXT] up to 40% perf drop from "loader/dri3: Overhaul dri3_update_num_back" commit Product: Mesa Version: git Hardware: x86-64 (AMD64)

Re: [Mesa-dev] [PATCH v2 1/2] i965: Fix missing dirty bits related to is_drawing_points/lines.

2016-08-31 Thread Iago Toral
On Tue, 2016-08-30 at 21:50 -0700, Kenneth Graunke wrote: > calculate_attr_overrides() uses is_drawing_points(), which depends > on tessellation and geometry program state, as well as polygon state. > > v2: Add missing _NEW_POLYGON as well.  Caught by Iago Toral. > > Signed-off-by: Kenneth

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Marek Olšák
On Wed, Aug 31, 2016 at 3:02 PM, Emil Velikov wrote: > On 31 August 2016 at 13:40, Marek Olšák wrote: >> On Aug 31, 2016 5:02 AM, "Michel Dänzer" wrote: >>> >>> On 30/08/16 06:51 PM, Marek Olšák wrote: >>> > >>> > If we don't care

Re: [Mesa-dev] [PATCH] gallium/postprocess: Fix resource freeing

2016-08-31 Thread Alex Deucher
On Wed, Aug 31, 2016 at 2:22 AM, Thomas Hellstrom wrote: > The code was triggering asserts in DEBUG builds of the SVGA driver since > the reference count of the resource was never decremented before destroy. > > Signed-off-by: Thomas Hellstrom >

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Emil Velikov
On 31 August 2016 at 13:40, Marek Olšák wrote: > On Aug 31, 2016 5:02 AM, "Michel Dänzer" wrote: >> >> On 30/08/16 06:51 PM, Marek Olšák wrote: >> > >> > If we don't care about memory usage, let's use my allocator. If we do, >> > let's import jemalloc into

Re: [Mesa-dev] [PATCH] clover: Introduce CLOVER_EXTRA_COMPILER_OPTIONS

2016-08-31 Thread Serge Martin
On Wednesday 31 August 2016 12:11:05 Vedran Miletić wrote: > Options specified via the CLOVER_EXTRA_COMPILER_OPTIONS shell variable > are appended to the compiler and linker options specified by the OpenCL > program (if any). > > v2: > * rename to CLOVER_EXTRA_COMPILER_OPTIONS > * use

Re: [Mesa-dev] What happened to the stable releases?

2016-08-31 Thread Emil Velikov
On 30 August 2016 at 23:38, Matt Turner wrote: > The biweekly stable releases have stopped without any indication why. > 11.2.2 was nearly four months ago, and there effectively hasn't been a > stable release in the 12.0 series. > > Can we resume these as soon as possible?

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Eero Tamminen
HI, On 31.08.2016 15:40, Marek Olšák wrote: On Aug 31, 2016 5:02 AM, "Michel Dänzer" wrote: On 30/08/16 06:51 PM, Marek Olšák wrote: If we don't care about memory usage, let's use my allocator. If we do, let's import jemalloc into the Mesa tree and use it for ralloc.

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Marek Olšák
On Aug 31, 2016 5:02 AM, "Michel Dänzer" wrote: > > On 30/08/16 06:51 PM, Marek Olšák wrote: > > > > If we don't care about memory usage, let's use my allocator. If we do, > > let's import jemalloc into the Mesa tree and use it for ralloc. > > Instead of importing jemalloc

Re: [Mesa-dev] [PATCH] clover: Introduce CLOVER_EXTRA_COMPILER_OPTIONS

2016-08-31 Thread Vedran Miletić
On 08/31/2016 12:11 PM, Vedran Miletić wrote: > Options specified via the CLOVER_EXTRA_COMPILER_OPTIONS shell variable > are appended to the compiler and linker options specified by the OpenCL > program (if any). > > v2: > * rename to CLOVER_EXTRA_COMPILER_OPTIONS > * use debug_get_option > *

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Marek Olšák
NS2 causes mmap failures on radeonsi. My guess is that there is a memory leak somewhere. I have no evidence that the compiler is responsible for that. Marek On Aug 31, 2016 11:31 AM, "Eero Tamminen" wrote: > Hi, > > On 31.08.2016 08:03, Kenneth Graunke wrote: > >> On

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Tapani Pälli
On 08/31/2016 08:03 AM, Kenneth Graunke wrote: On Tuesday, August 30, 2016 3:52:12 PM PDT Ian Romanick wrote: [snip] I don't recall any native Linux games encountering this, but my memory may have faded. There were some Windows DirectX games that would exhaust VMA when run inside VMware. Once

Re: [Mesa-dev] [PATCH] st/clover: Define __OPENCL_VERSION__ on the device side

2016-08-31 Thread Vedran Miletić
On 08/28/2016 04:42 PM, Niels Ole Salscheider wrote: > This is required by the OpenCL standard. > > Signed-off-by: Niels Ole Salscheider Reviewed-by: Vedran Miletić Good catch. Do we miss more defines from [1]? Regards, Vedran [1]

[Mesa-dev] [PATCH] clover: Introduce CLOVER_EXTRA_COMPILER_OPTIONS

2016-08-31 Thread Vedran Miletić
Options specified via the CLOVER_EXTRA_COMPILER_OPTIONS shell variable are appended to the compiler and linker options specified by the OpenCL program (if any). v2: * rename to CLOVER_EXTRA_COMPILER_OPTIONS * use debug_get_option * append to linker options as well Signed-off-by: Vedran

[Mesa-dev] [PATCH 2/2] st/vdapu: use lanczos filter for scaling v4

2016-08-31 Thread Nayan Deshmukh
HIGH_QUALITY_SCALING_L2 to HIGH_QUALTIY_SCALING_L7 uses lanczos filter with number representing the size of the sinc window. v3: avoid L8 and L9 to allow for other scaling algrithms v4 (chk): rebased on master Signed-off-by: Nayan Deshmukh ---

[Mesa-dev] [PATCH 1/2] vl: add a lanczos interpolation filter v4

2016-08-31 Thread Nayan Deshmukh
v2: avoid dividing by zero when calculating lanczos v4 (chk): rebase on master Signed-off-by: Nayan Deshmukh --- src/gallium/auxiliary/Makefile.sources | 2 + src/gallium/auxiliary/vl/vl_lanczos_filter.c | 445 +++

[Mesa-dev] [Bug 93723] vblank_mode=3 segfaults SynMark2 EGL versions to: intelDestroyContext() -> _mesa_meta_free() -> _mesa_make_current()

2016-08-31 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=93723 Eero Tamminen changed: What|Removed |Added Summary|vblank_mode=3 segfaults |vblank_mode=3

Re: [Mesa-dev] [PATCH 2/2] st/vdapu: use lanczos filter for scaling v4

2016-08-31 Thread Nayan Deshmukh
Hi Christian, You missed a few things in the merge. I will send in the updated patch. I looked at the code in vl_winsys.c and vl_winsys3.c and also at the code in presentation.c where delayed rendering is used. I was trying to compile using vl_dri3_screen_create() but its giving me some error. I

Re: [Mesa-dev] [PATCH] clover: Introduce CLOVER_COMPILER_OPTIONS

2016-08-31 Thread Vedran Miletić
On 08/30/2016 09:27 AM, Serge Martin wrote: > On Tuesday 30 August 2016 01:20:55 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ć

Re: [Mesa-dev] Improving ralloc performance for the GLSL compiler

2016-08-31 Thread Eero Tamminen
Hi, On 31.08.2016 08:03, Kenneth Graunke wrote: On Tuesday, August 30, 2016 3:52:12 PM PDT Ian Romanick wrote: [snip] I don't recall any native Linux games encountering this, but my memory may have faded. There were some Windows DirectX games that would exhaust VMA when run inside VMware.

[Mesa-dev] Lanczos filter V4

2016-08-31 Thread Christian König
Hi Nayan, picking this topic up once more. Following this mail are your patches rebased on top of current master. I also played around with them a bit. One bug I've found which should also be reproducible for you is when switching mpv from windowed to fullscreen and back the video seems to

[Mesa-dev] [PATCH 2/2] st/vdapu: use lanczos filter for scaling v4

2016-08-31 Thread Christian König
From: Nayan Deshmukh HIGH_QUALITY_SCALING_L2 to HIGH_QUALTIY_SCALING_L7 uses lanczos filter with number representing the size of the sinc window. v3: avoid L8 and L9 to allow for other scaling algrithms v4 (chk): rebased on master Signed-off-by: Nayan Deshmukh

[Mesa-dev] [PATCH 1/2] vl: add a lanczos interpolation filter v4

2016-08-31 Thread Christian König
From: Nayan Deshmukh v2: avoid dividing by zero when calculating lanczos v4 (chk): rebase on master Signed-off-by: Nayan Deshmukh --- src/gallium/auxiliary/Makefile.sources | 2 + src/gallium/auxiliary/vl/vl_lanczos_filter.c | 445

Re: [Mesa-dev] [PATCH] i965: Allow creating image from different dma_buf fds

2016-08-31 Thread Eric Engestrom
On Tue, Aug 30, 2016 at 04:49:53PM -0700, k...@bitplanet.net wrote: > From: "Kristian H. Kristensen" > > As long as the dma_buf fds import to the same drm_intel_bo, we're fine. > > Reviewed-by: Stéphane Marchesin Looks good to me. Reviewed-by:

Re: [Mesa-dev] [PATCH] mesa/st: Standardize some asserts to use !"foo"

2016-08-31 Thread Eric Engestrom
On Tue, Aug 30, 2016 at 12:09:52PM +0200, Michael Schellenberger Costa wrote: > These are the final asserts in the tree that do not follow the pattern > assert(!"foo"). To simplify spotting of (mostly) incorrect asserts via git > grep "assert(\"" rewrite them into the standard form. > >

Re: [Mesa-dev] [PATCH] intel/blorp: Inline get_vs_entry_size into emit_urb_config

2016-08-31 Thread Pohjolainen, Topi
On Tue, Aug 30, 2016 at 07:34:57AM -0700, Jason Ekstrand wrote: >On Mon, Aug 29, 2016 at 11:39 PM, Pohjolainen, Topi ><[1]topi.pohjolai...@gmail.com> wrote: > > On Mon, Aug 29, 2016 at 09:00:33AM -0700, Jason Ekstrand wrote: > > Topi asked to have the prefix removed because

Re: [Mesa-dev] [PATCH] mesa/st: Standardize some asserts to use !"foo"

2016-08-31 Thread Edward O'Callaghan
Reviewed-by: Edward O'Callaghan On 08/30/2016 08:09 PM, Michael Schellenberger Costa wrote: > These are the final asserts in the tree that do not follow the pattern > assert(!"foo"). To simplify spotting of (mostly) incorrect asserts via git > grep "assert(\""

[Mesa-dev] [PATCH] gallium/postprocess: Fix resource freeing

2016-08-31 Thread Thomas Hellstrom
The code was triggering asserts in DEBUG builds of the SVGA driver since the reference count of the resource was never decremented before destroy. Signed-off-by: Thomas Hellstrom Reviewed-by: Brian Paul --- src/gallium/auxiliary/postprocess/pp_mlaa.c |

<    1   2