[Mesa-dev] [Bug 91254] (regresion) video using VA-API on Intel slow and freeze system with mesa 10.6 or 10.6.1

2015-07-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91254 --- Comment #4 from Tomasz C. --- I made a test the bisection method, here's the result: 0e0e23ef537c9add672ff322f34e129a07edc55e is the first bad commit commit 0e0e23ef537c9add672ff322f34e129a07edc55e Author: Jordan Justen Date: Wed Apr 22 1

Re: [Mesa-dev] [PATCH 04/18] i965: Introduce a context-local batch manager

2015-07-08 Thread Chris Wilson
On Tue, Jul 07, 2015 at 10:03:09PM -0700, Kenneth Graunke wrote: > Hi Chris, > > I made a genuine effort to review this patch, hoping to better understand > the various changes and what you were trying to accomplish. I spent many > hours reading and trying to enumerate changes - or potential chan

[Mesa-dev] [PATCH] nir: Fix comment above nir_convert_from_ssa() prototype.

2015-07-08 Thread Kenneth Graunke
Presumably Connor renamed the parameter, inverting the sense. Update the comment accordingly. Cc: Connor Abbott Signed-off-by: Kenneth Graunke --- src/glsl/nir/nir.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/glsl/nir/nir.h b/src/glsl/nir/nir.h index 4cb7d2f..

Re: [Mesa-dev] [PATCH 05/18] i965: Reuse our VBO for streaming fast-clear vertices

2015-07-08 Thread Chris Wilson
On Tue, Jul 07, 2015 at 01:42:47PM +0300, Martin Peres wrote: > On 06/07/15 19:43, Kenneth Graunke wrote: > >On Monday, July 06, 2015 11:33:10 AM Chris Wilson wrote: > >>Rather than allocating a fresh page every time we clear a buffer, keep > >>that page around between invocations by tracking the l

[Mesa-dev] [Bug 91254] (regresion) video using VA-API on Intel slow and freeze system with mesa 10.6 or 10.6.1

2015-07-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91254 --- Comment #5 from Tomasz C. --- This confirmed yet by compiling 10.6.1 of reverse patch: https://git.thm.de/tjkl80/mesa/commit/0e0e23ef537c9add672ff322f34e129a07edc55e.patch It works correctly. Do you expect the correction in the next version?

Re: [Mesa-dev] [PATCH 01/18] i965: Query whether we have kernel support for the TIMESTAMP register once

2015-07-08 Thread Chris Wilson
On Mon, Jul 06, 2015 at 10:01:21AM -0700, Kenneth Graunke wrote: > On Monday, July 06, 2015 05:12:10 PM Chris Wilson wrote: > > On Mon, Jul 06, 2015 at 04:19:36PM +0300, Martin Peres wrote: > > > > > > > > > On 06/07/15 16:15, Martin Peres wrote: > > > >On 06/07/15 16:13, Chris Wilson wrote: > >

Re: [Mesa-dev] [Mesa-stable] [PATCH] util: Don't link to SHA1 library if shader-cache is disabled.

2015-07-08 Thread Emil Velikov
On 08/07/15 07:33, Matt Turner wrote: > Cc: "10.6" Thanks for that I've completely forgot about this. Seems like we never got to using mesa_sha. Can we just remove it for now and revive it as a user comes along ? Cheers, Emil ___ mesa-dev mailing list

Re: [Mesa-dev] [PATCH 03/18] i965: Share the workaround bo between all contexts

2015-07-08 Thread Chris Wilson
On Mon, Jul 06, 2015 at 09:34:10AM -0700, Kenneth Graunke wrote: > On Monday, July 06, 2015 11:33:08 AM Chris Wilson wrote: > > Since the workaround bo is used strictly as a write-only buffer, we need > > only allocate one per screen and use the same one from all contexts. > > > > (The caveat here

Re: [Mesa-dev] [PATCH 17/18] loader: Look for any version of currently linked libudev.so

2015-07-08 Thread Chris Wilson
On Tue, Jul 07, 2015 at 08:40:12PM +0100, Emil Velikov wrote: > On 06/07/15 11:33, Chris Wilson wrote: > > Since there was an ABI break and linking twice against libudev.so.0 and > > libudev.so.1 causes the application to quickly crash, we first check if > > the application is currently linked agai

Re: [Mesa-dev] [PATCH 02/18] i965: Move pipecontrol workaround bo to brw_pipe_control

2015-07-08 Thread Chris Wilson
On Mon, Jul 06, 2015 at 09:30:48AM -0700, Kenneth Graunke wrote: > On Monday, July 06, 2015 11:33:07 AM Chris Wilson wrote: > > With the exception of gen8, the sole user of the workaround bo are for > > emitting pipe controls. Move it out of the purview of the batchbuffer > > and into the pipecontr

Re: [Mesa-dev] [PATCH] nir: Fix comment above nir_convert_from_ssa() prototype.

2015-07-08 Thread Connor Abbott
Whoops! Reviewed-by: Connor Abbott On Wed, Jul 8, 2015 at 1:57 AM, Kenneth Graunke wrote: > Presumably Connor renamed the parameter, inverting the sense. > Update the comment accordingly. > > Cc: Connor Abbott > Signed-off-by: Kenneth Graunke > --- > src/glsl/nir/nir.h | 6 +++--- > 1 file c

[Mesa-dev] [PATCH 3/5] i965: Move the pipelined test for SO register access to the screen

2015-07-08 Thread Chris Wilson
Moving the test to the screen places it alongside the other global HW feature tesst that want to be shared between contexts. Signed-off-by: Chris Wilson Cc: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_context.c | 1 + src/mesa/drivers/dri/i965/intel_extensions.c | 69 ---

[Mesa-dev] [PATCH 2/5] i965: Share the workaround bo between all contexts

2015-07-08 Thread Chris Wilson
Since the workaround bo is used strictly as a write-only buffer, we need only allocate one per screen and use the same one from all contexts. (The caveat here is during extension initialisation, where we write into and read back register values from the buffer, but that is performed only once for

[Mesa-dev] [PATCH 1/5] i965: Only flush the batchbuffer if we need to zero the SO offsets

2015-07-08 Thread Chris Wilson
If we don't have pipelined register access (e.g. Haswell before kernel v4.2), then we can only implement EXT_transform_feedback by reseting the SO offsets *between* batches. However, if we do have pipelined access to the SO registers on gen7, we can simply emit an inline reset of the SO registers w

[Mesa-dev] [PATCH 5/5] i965: Combine the multiple pipelined register detection into one round-trip

2015-07-08 Thread Chris Wilson
Combining the multiple access checks into a few batches and a single serialising read can reduce detection times from around 100us to 70us on a fast Haswell system. Signed-off-by: Chris Wilson Cc: Kenneth Graunke --- src/mesa/drivers/dri/i965/intel_screen.c | 177 +++

[Mesa-dev] [PATCH 4/5] i965: Move the OACONTROL pipelined access check from context to screen

2015-07-08 Thread Chris Wilson
Similarly to the pipelined SO_OFFSET check, this moves the global HW compatability check to the screen next to the other global checks. Signed-off-by: Chris Wilson --- src/mesa/drivers/dri/i965/brw_context.c | 1 + src/mesa/drivers/dri/i965/brw_context.h | 1 + src/mesa/drivers/dri/i

Re: [Mesa-dev] [PATCH 04/18] i965: Introduce a context-local batch manager

2015-07-08 Thread Chris Wilson
On Wed, Jul 08, 2015 at 09:51:07AM +0100, Chris Wilson wrote: > On Tue, Jul 07, 2015 at 10:03:09PM -0700, Kenneth Graunke wrote: > > * Gen4-5 structure changes. Did you mean brw_structs.h? diff --git a/src/mesa/drivers/dri/i965/brw_structs.h b/src/mesa/drivers/dri/i965/brw_structs.h index 55338c

Re: [Mesa-dev] [PATCH 05/18] i965: Reuse our VBO for streaming fast-clear vertices

2015-07-08 Thread Martin Peres
On 08/07/15 12:01, Chris Wilson wrote: On Tue, Jul 07, 2015 at 01:42:47PM +0300, Martin Peres wrote: On 06/07/15 19:43, Kenneth Graunke wrote: On Monday, July 06, 2015 11:33:10 AM Chris Wilson wrote: Rather than allocating a fresh page every time we clear a buffer, keep that page around betwee

[Mesa-dev] [Bug 90264] [Regression, bisected] Tooltip corruption in Chrome

2015-07-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=90264 --- Comment #33 from Jose P. --- (In reply to Jose P. from comment #21) > This seems to have been fixed, I can't see the corruption anymore. I'm using > Chromium 43.0.2357.81 on Ubuntu 14.04 (64-bit) and Mesa 10.7.0-devel > (git-20dca37 2015-06-2

Re: [Mesa-dev] [Mesa-stable] [PATCH] util: Don't link to SHA1 library if shader-cache is disabled.

2015-07-08 Thread Matt Turner
On Wed, Jul 8, 2015 at 3:00 AM, Emil Velikov wrote: > On 08/07/15 07:33, Matt Turner wrote: >> Cc: "10.6" > Thanks for that I've completely forgot about this. > > Seems like we never got to using mesa_sha. Can we just remove it for now > and revive it as a user comes along ? I don't have a probl

[Mesa-dev] [PATCH] i965: bump libdrm requirement to 2.4.61 and drop in-tree workaround

2015-07-08 Thread Emil Velikov
Signed-off-by: Emil Velikov --- configure.ac | 2 +- src/mesa/drivers/dri/i965/intel_screen.c | 5 - 2 files changed, 1 insertion(+), 6 deletions(-) diff --git a/configure.ac b/configure.ac index d819bef..eb7180b 100644 --- a/configure.ac +++ b/configure.ac @@ -68

[Mesa-dev] [PATCH 03/11] radeon: remove dri_mirror state

2015-07-08 Thread Emil Velikov
Most of the data stored(duplicated) was unused, and for the one that is follow the approach set by other drivers. This eliminates the use of legacy (dri1) types. XXX: The radeon code is the only user of __DRIscreen::drm_version (the only __DRIversion outside of dri1 land). Should we move it into r

[Mesa-dev] [PATCH 01/11] configure.ac: do not set HAVE_DRI(23) when libdrm is missing

2015-07-08 Thread Emil Velikov
These conditionals are used to guard both dri modules and loader(s). Currently if we try to build the gallium swrast dri module (without glx) on a system that's missing libdrm the build will fail. Cc: 10.6 Signed-off-by: Emil Velikov --- configure.ac | 4 ++-- 1 file changed, 2 insertions(+),

[Mesa-dev] [PATCH 10/11] android: don't build the kms-dri winsys

2015-07-08 Thread Emil Velikov
GBM (the only user of kms-dri) is currently not available under Android. Considering we have no way of testing/using this let's not bother building it for now. Cc: Chih-Wei Huang Cc: Eric Anholt Signed-off-by: Emil Velikov --- src/gallium/Android.mk| 2 +- src/gallium/stat

[Mesa-dev] [PATCH 06/11] loader: use HAVE_LIBDRM instead of ! __NOT_HAVE_DRM_H

2015-07-08 Thread Emil Velikov
Double negatives in English language are normally avoided, plus the former seems cleaner and more consistent. Signed-off-by: Emil Velikov --- src/loader/Android.mk | 6 ++ src/loader/Makefile.am | 5 + src/loader/SConscript | 2 -- src/loader/loader.c

[Mesa-dev] [PATCH 00/11] Bye bye __NOT_HAVE_DRM_H

2015-07-08 Thread Emil Velikov
Hi all, As double-negatives are not too appealing in the English language I've decided to nuke the common ifndef __NOT_HAVE_DRM_H. The first three patches are trivial bugfix + cleanups which I've noticed while browsing through, 04-09 remove all occurrences of the lastly macro (HAVE_LIBDRM is us

[Mesa-dev] [PATCH 05/11] st/dri: unwrap/remove __NOT_HAVE_DRM_H magic

2015-07-08 Thread Emil Velikov
With the dri_interface.h clean of the macro, we can remove the final only st/dri specific use of the very same. Seemingly it was incorrectly used, as the build-time presence of dri2 is not libdrm specific. At run-time, the code is already limited to dri2 use-cases plus returning true, when the ext

[Mesa-dev] [PATCH 08/11] swrast: remove unneeded __NOT_HAVE_DRM_H define

2015-07-08 Thread Emil Velikov
No longer applicable since the cleanup of dri_interface.h. Signed-off-by: Emil Velikov --- src/mesa/drivers/dri/swrast/Makefile.am | 1 - 1 file changed, 1 deletion(-) diff --git a/src/mesa/drivers/dri/swrast/Makefile.am b/src/mesa/drivers/dri/swrast/Makefile.am index bfc3c10..b1ba34d 100644 -

[Mesa-dev] [PATCH 04/11] dri_interface: drop __NOT_HAVE_DRM_H magic

2015-07-08 Thread Emil Velikov
Signed-off-by: Emil Velikov --- include/GL/internal/dri_interface.h | 11 --- 1 file changed, 11 deletions(-) diff --git a/include/GL/internal/dri_interface.h b/include/GL/internal/dri_interface.h index c827bb6..c0545b1 100644 --- a/include/GL/internal/dri_interface.h +++ b/include/GL/i

[Mesa-dev] [PATCH 09/11] android: dri: correctly set HAVE_LIBDRM

2015-07-08 Thread Emil Velikov
Set the macro if we're not building swrast alone. Cc: Chih-Wei Huang Cc: Eric Anholt Signed-off-by: Emil Velikov --- src/gallium/targets/dri/Android.mk | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/gallium/targets/dri/Android.mk b/src/gallium/targets/dri/Andr

[Mesa-dev] [PATCH 07/11] dri/common: use HAVE_LIBDRM over __NOT_HAVE_DRM_H

2015-07-08 Thread Emil Velikov
See previous commit message for details. Signed-off-by: Emil Velikov --- src/mesa/drivers/dri/common/Android.mk | 13 ++--- src/mesa/drivers/dri/common/Makefile.am | 2 +- src/mesa/drivers/dri/common/SConscript | 4 src/mesa/drivers/dri/common/dri_util.c | 4 ++-- 4 files chan

[Mesa-dev] [PATCH 11/11] scons: don't build the kms-dri winsys

2015-07-08 Thread Emil Velikov
Same as previous commit - unused (gbm is not a thing outside the autotools build). Signed-off-by: Emil Velikov --- src/gallium/SConscript| 1 - src/gallium/state_trackers/dri/SConscript | 1 - src/gallium/targets/dri/SConscript| 4 src/gallium/winsys/sw/kms-dr

[Mesa-dev] [PATCH 02/11] i915; remove unused driFd variable

2015-07-08 Thread Emil Velikov
Signed-off-by: Emil Velikov --- src/mesa/drivers/dri/i915/intel_context.c | 1 - src/mesa/drivers/dri/i915/intel_context.h | 2 -- 2 files changed, 3 deletions(-) diff --git a/src/mesa/drivers/dri/i915/intel_context.c b/src/mesa/drivers/dri/i915/intel_context.c index 5618dcd..c780103 100644 ---

Re: [Mesa-dev] [PATCH 02/11] i915; remove unused driFd variable

2015-07-08 Thread Matt Turner
On Wed, Jul 8, 2015 at 10:07 AM, Emil Velikov wrote: > Signed-off-by: Emil Velikov > --- > src/mesa/drivers/dri/i915/intel_context.c | 1 - > src/mesa/drivers/dri/i915/intel_context.h | 2 -- > 2 files changed, 3 deletions(-) > > diff --git a/src/mesa/drivers/dri/i915/intel_context.c > b/src/me

Re: [Mesa-dev] [RFC] loader: libudev vs sysfs vs libdrm

2015-07-08 Thread Eric Anholt
Emil Velikov writes: > Hello all, > > A recent patch by Chris, fixing some libudev fun in our loader, made > me think if we can clear it up a bit. > > Having three different ways of retrieving the vendor/device ID does > feel a bit excessive. Plus as one gets fixed others are likely to > break -

[Mesa-dev] [PATCH] xa: don't leak fences

2015-07-08 Thread Rob Clark
From: Rob Clark XA was never unref'ing last_fence in the various call paths to pipe->flush(). Add this to xa_context_flush() and update the other open-coded calls to pipe->flush() to use xa_context_flush() instead. This fixes a memory leak reported with xf86-video-freedreno. Reported-by: Nicol

[Mesa-dev] [Bug 91254] (regresion) video using VA-API on Intel slow and freeze system with mesa 10.6 or 10.6.1

2015-07-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91254 Ian Romanick changed: What|Removed |Added CC||i...@freedesktop.org QA Contact|i

Re: [Mesa-dev] [PATCH 04/18] i965: Introduce a context-local batch manager

2015-07-08 Thread Kenneth Graunke
On Wednesday, July 08, 2015 03:17:35 PM Chris Wilson wrote: > On Wed, Jul 08, 2015 at 09:51:07AM +0100, Chris Wilson wrote: > > On Tue, Jul 07, 2015 at 10:03:09PM -0700, Kenneth Graunke wrote: > > > * Gen4-5 structure changes. > > Did you mean brw_structs.h? > > diff --git a/src/mesa/drivers/dri/

Re: [Mesa-dev] [PATCH 04/18] i965: Introduce a context-local batch manager

2015-07-08 Thread Chris Wilson
On Wed, Jul 08, 2015 at 10:49:24AM -0700, Kenneth Graunke wrote: > On Wednesday, July 08, 2015 03:17:35 PM Chris Wilson wrote: > > On Wed, Jul 08, 2015 at 09:51:07AM +0100, Chris Wilson wrote: > > > On Tue, Jul 07, 2015 at 10:03:09PM -0700, Kenneth Graunke wrote: > > > > * Gen4-5 structure changes.

Re: [Mesa-dev] [RFC] loader: libudev vs sysfs vs libdrm

2015-07-08 Thread Axel Davy
On 08/07/2015 00:15, Emil Velikov wrote : Can anyone shed a light/cast their 2c ? Thanks Emil ___ The DRI3/Wayland/Gallium Nine DRI_PRIME path uses libudev, and I'm not sure it could work without. It is based on the ID_PATH_TAG advertised by udev

Re: [Mesa-dev] [Mesa-stable] [PATCH] util: Don't link to SHA1 library if shader-cache is disabled.

2015-07-08 Thread Emil Velikov
On 8 July 2015 at 17:47, Matt Turner wrote: > On Wed, Jul 8, 2015 at 3:00 AM, Emil Velikov wrote: >> On 08/07/15 07:33, Matt Turner wrote: >>> Cc: "10.6" >> Thanks for that I've completely forgot about this. >> >> Seems like we never got to using mesa_sha. Can we just remove it for now >> and re

[Mesa-dev] [PATCH 1/4] gallium: clarify reference counting for fence

2015-07-08 Thread Rob Clark
From: Rob Clark No where was it spelled out that the state tracker may expect the pipe driver to unref the old fence. Signed-off-by: Rob Clark --- src/gallium/include/pipe/p_context.h | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/gallium/include/pipe/p_context.

[Mesa-dev] [PATCH 4/4] vc4: unref old fence

2015-07-08 Thread Rob Clark
From: Rob Clark Some, but not all, state trackers will explicitly unref (and set to NULL) the previous *fence before calling pipe->flush(). So driver should use fence_ref() which will unref the old fence if not NULL. Signed-off-by: Rob Clark --- src/gallium/drivers/vc4/vc4_context.c | 3 ++-

[Mesa-dev] [PATCH 0/4] fence refcnting fixes

2015-07-08 Thread Rob Clark
From: Rob Clark This isn't at all clear for pipe driver writers currently, since it is not documented anywhere. But radeon/nouveau/llvmpipe seem to drop the ref on the **fence passed in to pipe->flush() (if *fence!=NULL). Freedreno/ilo/vc4 where not doing this. Some state trackers do call scree

[Mesa-dev] [PATCH 2/4] freedreno: unref old fence

2015-07-08 Thread Rob Clark
From: Rob Clark Some, but not all, state trackers will explicitly unref (and set to NULL) the previous *fence before calling pipe->flush(). So driver should use fence_ref() which will unref the old fence if not NULL. Signed-off-by: Rob Clark --- src/gallium/drivers/freedreno/freedreno_context

[Mesa-dev] [PATCH 3/4] ilo: unref old fence

2015-07-08 Thread Rob Clark
From: Rob Clark Some, but not all, state trackers will explicitly unref (and set to NULL) the previous *fence before calling pipe->flush(). So driver should use fence_ref() which will unref the old fence if not NULL. Signed-off-by: Rob Clark --- src/gallium/drivers/ilo/ilo_context.c | 5 -

[Mesa-dev] [PATCH 2/2] i965: Optimize intel_batchbuffer_emit_dword().

2015-07-08 Thread Matt Turner
By keeping a pointer to the next available location, we reduce the number of memory accesses needed to write to the batchbuffer. A net ~7k reduction of .text size, 7.5k of which is from the change to intel_batchbuffer_emit_dword(). text data bss dec hex filename 4943740 1

[Mesa-dev] [PATCH 1/2] i965: Set brw->batch.emit only #ifdef DEBUG.

2015-07-08 Thread Matt Turner
It's only used inside #ifdef DEBUG. Cuts ~1.7k of .text, and more importantly prevents a larger code size regression in the next commit when the .used field is replaced and calculated on demand. text data bss dec hex filename 4945468 19515226192 5166812 4ed6dc i965

Re: [Mesa-dev] [PATCH 2/2] i965: Optimize intel_batchbuffer_emit_dword().

2015-07-08 Thread Chris Wilson
On Wed, Jul 08, 2015 at 02:00:02PM -0700, Matt Turner wrote: > By keeping a pointer to the next available location, we reduce the > number of memory accesses needed to write to the batchbuffer. > > A net ~7k reduction of .text size, 7.5k of which is from the change to > intel_batchbuffer_emit_dwor

[Mesa-dev] [PATCH] mesa: Implement faster streaming memcpy

2015-07-08 Thread Ben Widawsky
WARNING: No perf data, please keep reading though) This implements the suggestion provided by the paper, "Fast USWC to WB Memory Copy" (https://software.intel.com/en-us/articles/copying-accelerated-video-decode-frame-buffers). This is described throughout the paper, but the sample code lives in Fi

Re: [Mesa-dev] [PATCH 0/4] fence refcnting fixes

2015-07-08 Thread Rob Clark
Bleh, assert(pipe_is_referenced(ptr)) in pipe_reference_described() is non-pleased in debug builds about the trick of starting out the fences w/ refcnt=0 (so that it doesn't end up getting upref'd to 2 in pipe->flush().. so I'll have to re-work the fd/ilo/vc4 patches slightly.. BR, -R On Wed, Jul

Re: [Mesa-dev] [PATCHv2] i965/gen9: Use custom MOCS entries set up by the kernel.

2015-07-08 Thread Ben Widawsky
On Tue, Jul 07, 2015 at 10:21:28PM +0300, Francisco Jerez wrote: > Instead of relying on hardware defaults the i915 kernel driver is > going program custom MOCS tables system-wide on Gen9 hardware. The > "WT" entry previously used for renderbuffers had a number of problems: > It disabled caching o

Re: [Mesa-dev] [PATCH] i965/chv|skl: Apply sampler bypass w/a

2015-07-08 Thread Ben Widawsky
On Thu, Jul 02, 2015 at 12:58:33PM -0700, Matt Turner wrote: > On Thu, Jul 2, 2015 at 12:57 PM, Matt Turner wrote: > > On Wed, Jul 1, 2015 at 4:03 PM, Ben Widawsky > > wrote: > >> Certain compressed formats require this setting. The docs don't go into > >> much > >> detail as to why it's needed

Re: [Mesa-dev] [PATCH 2/2] i965: Optimize intel_batchbuffer_emit_dword().

2015-07-08 Thread Matt Turner
On Wed, Jul 8, 2015 at 2:07 PM, Chris Wilson wrote: > On Wed, Jul 08, 2015 at 02:00:02PM -0700, Matt Turner wrote: >> By keeping a pointer to the next available location, we reduce the >> number of memory accesses needed to write to the batchbuffer. >> >> A net ~7k reduction of .text size, 7.5k of

Re: [Mesa-dev] [PATCH] mesa: Implement faster streaming memcpy

2015-07-08 Thread Matt Turner
On Wed, Jul 8, 2015 at 2:07 PM, Ben Widawsky wrote: > WARNING: No perf data, please keep reading though) > > This implements the suggestion provided by the paper, "Fast USWC to WB Memory > Copy" > (https://software.intel.com/en-us/articles/copying-accelerated-video-decode-frame-buffers). > This is

Re: [Mesa-dev] [PATCH] util: Don't link to SHA1 library if shader-cache is disabled.

2015-07-08 Thread Kenneth Graunke
On Tuesday, July 07, 2015 11:33:57 PM Matt Turner wrote: > Cc: "10.6" > --- > src/util/Makefile.am | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/util/Makefile.am b/src/util/Makefile.am > index 2e7542e..1e087b4 100644 > --- a/src/util/Makefile.am > +++ b/src/util/Ma

Re: [Mesa-dev] [PATCH 2/2] i965: Optimize intel_batchbuffer_emit_dword().

2015-07-08 Thread Chris Wilson
On Wed, Jul 08, 2015 at 03:33:17PM -0700, Matt Turner wrote: > On Wed, Jul 8, 2015 at 2:07 PM, Chris Wilson wrote: > > On Wed, Jul 08, 2015 at 02:00:02PM -0700, Matt Turner wrote: > >> By keeping a pointer to the next available location, we reduce the > >> number of memory accesses needed to write

Re: [Mesa-dev] [PATCH 2/2] i965: Optimize intel_batchbuffer_emit_dword().

2015-07-08 Thread Chris Wilson
On Thu, Jul 09, 2015 at 12:53:23AM +0100, Chris Wilson wrote: > This is what I expected to see >0x025e <+62>: movl $0x780d1c02,(%rcx) >0x0264 <+68>: mov0x22f08(%rdi),%rax >0x026b <+75>: mov0x24320(%rdi),%edx >0x0271 <+81>:

[Mesa-dev] [PATCH] [v2] i965/chv|skl: Apply sampler bypass w/a

2015-07-08 Thread Ben Widawsky
Certain compressed formats require this setting. The docs don't go into much detail as to why it's needed exactly. This patch introduces no piglit regressions on gen9 (bsw is untested). Note that the SKL "regressions" are fixed tests, and the egl_khr_gl_colorspace tests are WTF. The patch also fix

Re: [Mesa-dev] [PATCH 2/2] i965: Optimize intel_batchbuffer_emit_dword().

2015-07-08 Thread Matt Turner
On Wed, Jul 8, 2015 at 4:53 PM, Chris Wilson wrote: > On Wed, Jul 08, 2015 at 03:33:17PM -0700, Matt Turner wrote: >> On Wed, Jul 8, 2015 at 2:07 PM, Chris Wilson >> wrote: >> > On Wed, Jul 08, 2015 at 02:00:02PM -0700, Matt Turner wrote: >> >> By keeping a pointer to the next available location

[Mesa-dev] [PATCH 3/4] ilo: unref old fence

2015-07-08 Thread Rob Clark
From: Rob Clark Some, but not all, state trackers will explicitly unref (and set to NULL) the previous *fence before calling pipe->flush(). So driver should use fence_ref() which will unref the old fence if not NULL. Signed-off-by: Rob Clark --- src/gallium/drivers/ilo/ilo_context.c | 2 ++ 1

[Mesa-dev] [PATCH 2/4] freedreno: unref old fence

2015-07-08 Thread Rob Clark
From: Rob Clark Some, but not all, state trackers will explicitly unref (and set to NULL) the previous *fence before calling pipe->flush(). So driver should use fence_ref() which will unref the old fence if not NULL. Signed-off-by: Rob Clark --- src/gallium/drivers/freedreno/freedreno_context

[Mesa-dev] [PATCH 4/4] vc4: unref old fence

2015-07-08 Thread Rob Clark
From: Rob Clark Some, but not all, state trackers will explicitly unref (and set to NULL) the previous *fence before calling pipe->flush(). So driver should use fence_ref() which will unref the old fence if not NULL. Signed-off-by: Rob Clark --- src/gallium/drivers/vc4/vc4_context.c | 2 ++ 1

Re: [Mesa-dev] [PATCH 1/4] gallium: clarify reference counting for fence

2015-07-08 Thread Michel Dänzer
On 09.07.2015 05:34, Rob Clark wrote: > From: Rob Clark > > No where was it spelled out that the state tracker may expect the pipe "Nowhere"? > driver to unref the old fence. > > Signed-off-by: Rob Clark Either way, the series is Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer

Re: [Mesa-dev] [PATCH 03/11] radeon: remove dri_mirror state

2015-07-08 Thread Michel Dänzer
On 09.07.2015 02:07, Emil Velikov wrote: > Most of the data stored(duplicated) was unused, and for the one that is > follow the approach set by other drivers. > This eliminates the use of legacy (dri1) types. The commentary below should have been after the --- separator, not in the actual Git comm

[Mesa-dev] [Bug 91259] FS compile failed: Register spilling not supported with m14 used

2015-07-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91259 Kenneth Graunke changed: What|Removed |Added Component|Other |Drivers/DRI/i965 Assignee|me

Re: [Mesa-dev] [PATCH 4/4] vc4: unref old fence

2015-07-08 Thread Eric Anholt
Rob Clark writes: > From: Rob Clark > > Some, but not all, state trackers will explicitly unref (and set to > NULL) the previous *fence before calling pipe->flush(). So driver > should use fence_ref() which will unref the old fence if not NULL. > > Signed-off-by: Rob Clark Acked-by: Eric Anho

[Mesa-dev] [PATCH] r600g: fix sampler/ubo indexing on cayman

2015-07-08 Thread Dave Airlie
From: Dave Airlie Cayman needs a different method to upload the CF IDX0/1 This fixes 31 piglits when ARB_gpu_shader5 is forced on with cayman. Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/eg_asm.c | 17 +++-- src/gallium/drivers/r600/eg_sq.h | 11 +++ 2 files c

[Mesa-dev] [Bug 91222] lp_test_format regression on CentOS 7

2015-07-08 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91222 Luke changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [PATCH] i965: Replace illegal compacted NOP with valid compact instruction

2015-07-08 Thread Zhenyu Wang
NOP actually has no compact version, but we use it for instruction alignment for compact kernel. Although it seems working on HW, it is illegal and might not be valid for any future one. This trys to get a temporary compact instruction with no effect for alignment to replace compacted NOP. G45 spe

[Mesa-dev] [PATCH] r600g: move sampler/ubo index registers before temp reg

2015-07-08 Thread Dave Airlie
From: Dave Airlie temp_reg needs to be last, as we increment things away from it, otherwise on cayman some tests were overwriting the index regs. Fixes 2 piglit with ARB_gpu_shader5 forced on cayman. Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/r600_shader.c | 13 ++--- 1 f

[Mesa-dev] [PATCH 4/6] r600g/sb: add support for multiple streams to SB backend

2015-07-08 Thread Dave Airlie
From: Glenn Kennard This adds a peephole and removes an assert that isn't actually valid with some of the stream emit instructions. Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/sb/sb_bc_finalize.cpp | 2 -- src/gallium/drivers/r600/sb/sb_bc_parser.cpp | 18 +++--- 2 f

[Mesa-dev] [PATCH 6/6] r600g: enable ARB_gpu_shader5 on evergreen and up

2015-07-08 Thread Dave Airlie
From: Dave Airlie Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/r600_pipe.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/r600/r600_pipe.c b/src/gallium/drivers/r600/r600_pipe.c index 67caa69..0db1c1c 100644 --- a/src/gallium/drivers/r600/r60

[Mesa-dev] [RFC] r600 geometry streams (and ARB_gpu_shader5 support)

2015-07-08 Thread Dave Airlie
This applies on top of the two patches I've sent already, and enables the geometry streams, which is the final piece missing for ARB_gpu_shader5 on evergreen and cayman. (I'll do doc update patches later) Glenn wrote most of this, I just spent some time making it work and cleaning up the code. Tho

[Mesa-dev] [PATCH 1/6] r600g: add support for streams to the assembler.

2015-07-08 Thread Dave Airlie
From: Glenn Kennard This just adds support to the assembler dumper and allows stream instructions to be generated. Also fix up the stream debugging to add stream info. Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/eg_asm.c | 1 + src/gallium/drivers/r600/r600_asm.c|

[Mesa-dev] [PATCH 2/6] radeon: add streamout status 1-3 queries.

2015-07-08 Thread Dave Airlie
From: Glenn Kennard This adds support for queries against the non-0 vertex streams. Signed-off-by: Dave Airlie --- src/gallium/drivers/radeon/r600_query.c | 18 -- src/gallium/drivers/radeon/r600d_common.h | 3 +++ 2 files changed, 19 insertions(+), 2 deletions(-) diff --gi

[Mesa-dev] [PATCH 5/6] r600g: add streamout support

2015-07-08 Thread Dave Airlie
From: Glenn Kennard This adds the main chunk of the geometry shader multiple stream support to the r600 driver. Glenn wrote the original pass, and I took his code and hacked it into a working state. Signed-off-by: Dave Airlie --- src/gallium/drivers/r600/evergreen_state.c | 29 ++-- src/ga

[Mesa-dev] [PATCH 3/6] radeon: add support for streams to the common streamout code.

2015-07-08 Thread Dave Airlie
From: Glenn Kennard This just adds to the common radeon streamout code, support for multiple streams. Signed-off-by: Dave Airlie --- src/gallium/drivers/radeon/r600_pipe_common.h | 1 + src/gallium/drivers/radeon/r600_streamout.c | 23 +-- 2 files changed, 18 insertions(

Re: [Mesa-dev] [PATCH 16/78] i965/nir/vec4: Implement store_output intrinsic

2015-07-08 Thread Eduardo Lima Mitev
On 06/30/2015 06:51 PM, Jason Ekstrand wrote: > On Fri, Jun 26, 2015 at 1:06 AM, Eduardo Lima Mitev wrote: >> The index into the output_reg array where to store the destination register >> is >> fetched from the nir_outputs map built during nir_setup_outputs stage. >> >> Bugzilla: https://bugs.fr