Re: [Mesa-dev] [PATCH 13/13] anv: Enable CCS for basic cases

2016-09-25 Thread Jason Ekstrand
On Sep 26, 2016 8:39 AM, "Pohjolainen, Topi" wrote: > > On Sun, Sep 25, 2016 at 10:11:55AM -0700, Jason Ekstrand wrote: > >On Sun, Sep 25, 2016 at 9:59 AM, Jason Ekstrand > ><[1]ja...@jlekstrand.net> wrote: > > > > This enables lossless render target compression but only for first > >

Re: [Mesa-dev] [PATCH 13/13] anv: Enable CCS for basic cases

2016-09-25 Thread Pohjolainen, Topi
On Sun, Sep 25, 2016 at 10:11:55AM -0700, Jason Ekstrand wrote: >On Sun, Sep 25, 2016 at 9:59 AM, Jason Ekstrand ><[1]ja...@jlekstrand.net> wrote: > > This enables lossless render target compression but only for first > mip level > of the first slice and only within a render

Re: [Mesa-dev] [PATCH 12/88] i965: add initial implementation of on disk shader cache

2016-09-25 Thread Kenneth Graunke
On Monday, September 26, 2016 1:28:35 PM PDT Timothy Arceri wrote: > On Sun, 2016-09-25 at 19:43 -0700, Kenneth Graunke wrote: > > On Saturday, September 24, 2016 3:24:53 PM PDT Timothy Arceri wrote: > > > > > > This uses the recently-added cache.c to write out the final linked > > > binary for ve

Re: [Mesa-dev] [PATCH 12/88] i965: add initial implementation of on disk shader cache

2016-09-25 Thread Timothy Arceri
On Sun, 2016-09-25 at 19:43 -0700, Kenneth Graunke wrote: > On Saturday, September 24, 2016 3:24:53 PM PDT Timothy Arceri wrote: > > > > This uses the recently-added cache.c to write out the final linked > > binary for vertex and fragment shader programs. > > > > This is based off the initial imp

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

2016-09-25 Thread Kenneth Graunke
On Saturday, September 24, 2016 3:26:09 PM PDT Timothy Arceri wrote: > Pointers will have different lengths so we simple create a different > sha1 for each platform. I don't understand...you're putting pointers in the blob? How can that work? You obviously can't reload things from the cache as a

Re: [Mesa-dev] [PATCH 12/88] i965: add initial implementation of on disk shader cache

2016-09-25 Thread Kenneth Graunke
On Saturday, September 24, 2016 3:24:53 PM PDT Timothy Arceri wrote: > This uses the recently-added cache.c to write out the final linked > binary for vertex and fragment shader programs. > > This is based off the initial implementation done by Carl. > --- > src/mesa/drivers/dri/i965/Makefile.sou

Re: [Mesa-dev] [PATCH 42/88] glsl: use reproducible name for lowered const arrays

2016-09-25 Thread Kenneth Graunke
On Saturday, September 24, 2016 3:25:23 PM PDT Timothy Arceri wrote: > Otherwise we can end up with mismatching names between the cached > binary and the cached metadata. > --- > src/compiler/glsl/ir_optimization.h | 2 +- > src/compiler/glsl/linker.cpp |

Re: [Mesa-dev] [PATCH 36/88] glsl: don't crash when dumping shaders if some come from cache

2016-09-25 Thread Kenneth Graunke
On Saturday, September 24, 2016 3:25:17 PM PDT Timothy Arceri wrote: > --- > src/mesa/drivers/dri/i965/brw_link.cpp | 6 +- > src/mesa/main/shaderapi.c | 6 +- > 2 files changed, 10 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_link.cpp > b/src

Re: [Mesa-dev] [PATCH 06/88] i965: make vs and fs key generation helpers available to shader cache

2016-09-25 Thread Kenneth Graunke
On Saturday, September 24, 2016 3:24:47 PM PDT Timothy Arceri wrote: > From: Carl Worth > > Signed-off-by: Timothy Arceri > --- > src/mesa/drivers/dri/i965/brw_vs.c | 3 +-- > src/mesa/drivers/dri/i965/brw_vs.h | 4 > src/mesa/drivers/dri/i965/brw_wm.c | 2 +- > src/mesa/drivers/dri/i965/b

Re: [Mesa-dev] [PATCH 66/88] i965: make gs key generation helper available to shader cache

2016-09-25 Thread Kenneth Graunke
On Saturday, September 24, 2016 3:25:47 PM PDT Timothy Arceri wrote: > --- > src/mesa/drivers/dri/i965/brw_gs.c | 2 +- > src/mesa/drivers/dri/i965/brw_gs.h | 4 > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/src/mesa/drivers/dri/i965/brw_gs.c > b/src/mesa/drivers/dri/i9

Re: [Mesa-dev] [PATCH 71/88] i965: create populate key functions for tcs and tes

2016-09-25 Thread Kenneth Graunke
On Saturday, September 24, 2016 3:25:52 PM PDT Timothy Arceri wrote: > These will be used by the on disk shader cache. > --- > src/mesa/drivers/dri/i965/brw_program.h | 10 ++-- > src/mesa/drivers/dri/i965/brw_state_upload.c | 18 +-- > src/mesa/drivers/dri/i965/brw_tcs.c | 70 >

Re: [Mesa-dev] [PATCH 07/88] i965: add new field for storing program size

2016-09-25 Thread Kenneth Graunke
On Saturday, September 24, 2016 3:24:48 PM PDT Timothy Arceri wrote: > From: Carl Worth > > This will be used by the on disk shader cache. > > Signed-off-by: Timothy Arceri > --- > src/mesa/drivers/dri/i965/brw_compiler.h | 4 > src/mesa/drivers/dri/i965/brw_vs.c | 2 ++ > src/mesa/

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

2016-09-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97879 Alexander Tsoy changed: What|Removed |Added CC||alexan...@tsoy.me -- You are receiving

[Mesa-dev] [Bug 97260] R9 290 low performance in Linux 4.7

2016-09-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97260 Alexander Tsoy changed: What|Removed |Added CC||alexan...@tsoy.me -- You are receiving

Re: [Mesa-dev] [PATCH 03/88] glsl: Add initial functions to

2016-09-25 Thread Timothy Arceri
On Sun, 2016-09-25 at 13:26 -0700, Eric Anholt wrote: > Timothy Arceri writes: > > > > > From: Carl Worth > > > > This code provides for an on-disk cache of objects. Objects are > > stored > > and retrieved via names that are arbitrary 20-byte sequences, > > (intended to be SHA-1 hashes of som

[Mesa-dev] [Bug 97921] Mesa-only glBlitFrameBuffer error

2016-09-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97921 --- Comment #3 from Maxqia --- https://www.maxqia.com/files/wine-preloader.trace.gz Here you go (I decided to use my own server), When glBlitFrameBuffer doesn't work, the aiming at the end blurs the whole entire frame instead of working correctl

Re: [Mesa-dev] [PATCH 58/88] glsl: disable on disk shader cache when running as root

2016-09-25 Thread Timothy Arceri
On Sun, 2016-09-25 at 12:39 -0700, Eric Anholt wrote: > Timothy Arceri writes: > > > > > --- > >  src/compiler/glsl/cache.c | 4 > >  1 file changed, 4 insertions(+) > > > > diff --git a/src/compiler/glsl/cache.c b/src/compiler/glsl/cache.c > > index a6a9e47..dd53f4e 100644 > > --- a/src/co

[Mesa-dev] [Bug 97921] Mesa-only glBlitFrameBuffer error

2016-09-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97921 --- Comment #2 from Maxqia --- Where's a good place I could upload it? It's half a gig compressed ... -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug._

Re: [Mesa-dev] [PATCH 03/88] glsl: Add initial functions to implement an on-disk cache

2016-09-25 Thread Eric Anholt
Timothy Arceri writes: > From: Carl Worth > > This code provides for an on-disk cache of objects. Objects are stored > and retrieved via names that are arbitrary 20-byte sequences, > (intended to be SHA-1 hashes of something identifying for the > content). The directory used for the cache can be

Re: [Mesa-dev] [PATCH] st/va Avoid VBR bitrate calculation overflow

2016-09-25 Thread Andy Furniss
Ignore this one un-mungled version sent. Andy Furniss wrote: VBR bitrate calc needs 64 bits at high rates. Signed-off-by: Andy Furniss --- src/gallium/state_trackers/va/picture.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/va/picture.c b/sr

Re: [Mesa-dev] [PATCH 58/88] glsl: disable on disk shader cache when running as root

2016-09-25 Thread Eric Anholt
Timothy Arceri writes: > --- > src/compiler/glsl/cache.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/src/compiler/glsl/cache.c b/src/compiler/glsl/cache.c > index a6a9e47..dd53f4e 100644 > --- a/src/compiler/glsl/cache.c > +++ b/src/compiler/glsl/cache.c > @@ -141,6 +141,10 @@ c

Re: [Mesa-dev] [PATCH] st/va: enable vbr rate control for vaapi encode

2016-09-25 Thread Andy Furniss
Andy Furniss wrote: Andy Furniss wrote: https://patchwork.freedesktop.org/patch/112040/ Hmm that got mungled I'll try again later going to be AFK for a while. This one worked. https://patchwork.freedesktop.org/patch/112069/ ___ mesa-dev mailing

Re: [Mesa-dev] [PATCH] intel/i965: fix resource leak

2016-09-25 Thread Kenneth Graunke
On Sunday, September 25, 2016 7:42:16 PM PDT Nayan Deshmukh wrote: > CovID: 1373370 > > Signed-off-by: Nayan Deshmukh > --- > src/intel/tools/disasm.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/intel/tools/disasm.c b/src/intel/tools/disasm.c > index e5276

[Mesa-dev] [PATCH] st/va Avoid VBR bitrate calculation overflow

2016-09-25 Thread Andy Furniss
VBR bitrate calc needs 64 bits at high rates. Signed-off-by: Andy Furniss --- src/gallium/state_trackers/va/picture.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/va/picture.c b/src/gallium/state_trackers/va/picture.c index 7f3d96d..cd2dabb 1006

Re: [Mesa-dev] [PATCH v2] intel/i965: fix resource leak

2016-09-25 Thread Nayan Deshmukh
Hi Lionel, Thanks for the review. Can you please push the patch. Regards, Nayan. On Sun, Sep 25, 2016 at 10:18 PM, Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Reviewed-by: Lionel Landwerlin > > > On 25/09/16 16:06, Nayan Deshmukh wrote: > >> CovID: 1373370 >> v2: fix indentatio

Re: [Mesa-dev] [PATCH mesa 4/4] nir/spirv: add spirv2nir binary to .gitignore

2016-09-25 Thread Jason Ekstrand
I hope you realize that this is the only truly useful change in the series. :-). Still, no reason why our silly little helpers shouldn't be correct. Series is Reviewed-by: Jason Ekstrand On Sep 25, 2016 6:50 PM, "Eric Engestrom" wrote: > Signed-off-by: Eric Engestrom > --- > src/compiler/.gi

Re: [Mesa-dev] [PATCH 13/13] anv: Enable CCS for basic cases

2016-09-25 Thread Jason Ekstrand
On Sun, Sep 25, 2016 at 9:59 AM, Jason Ekstrand wrote: > This enables lossless render target compression but only for first mip > level > of the first slice and only within a render pass. At the end of the render > pass it always does a full resolve. > > Signed-off-by: Jason Ekstrand > --- > s

[Mesa-dev] [PATCH 13/13] anv: Enable CCS for basic cases

2016-09-25 Thread Jason Ekstrand
This enables lossless render target compression but only for first mip level of the first slice and only within a render pass. At the end of the render pass it always does a full resolve. Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_blorp.c | 30 src

[Mesa-dev] [PATCH 11/13] anv: Move BindImageMemory to anv_image.c

2016-09-25 Thread Jason Ekstrand
Signed-off-by: Nanley Chery --- src/intel/vulkan/anv_device.c | 20 src/intel/vulkan/anv_image.c | 20 2 files changed, 20 insertions(+), 20 deletions(-) diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c index cda7324..5780894 1

[Mesa-dev] [PATCH 12/13] anv: Add some basic infrastructure for aux surfaces

2016-09-25 Thread Jason Ekstrand
This adds an aux_surface member to anv_image as well as the various bits of plumbing required to initialize them in BindImageMemory, keep track of the aux_usage, and add the surface state relocations. Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_cmd_buffer.c | 28 ++

[Mesa-dev] [PATCH 10/13] anv: Create render target surface states in CreateFramebuffer

2016-09-25 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_blorp.c | 4 ++-- src/intel/vulkan/anv_cmd_buffer.c | 14 -- src/intel/vulkan/anv_device.c | 38 +++--- src/intel/vulkan/anv_dump.c | 2 +- src/intel/vulkan/anv_image.c | 33 --

[Mesa-dev] [PATCH 08/13] anv/cmd_buffer: Refactor surface state reloction handling

2016-09-25 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_cmd_buffer.c | 34 +- 1 file changed, 21 insertions(+), 13 deletions(-) diff --git a/src/intel/vulkan/anv_cmd_buffer.c b/src/intel/vulkan/anv_cmd_buffer.c index 678166a..3d627d0 100644 --- a/src/intel/vulkan/

[Mesa-dev] [PATCH 09/13] intel/isl: Rework the asserts and fails in isl_surf_get_ccs

2016-09-25 Thread Jason Ekstrand
There are some invariants such as number of samples on which we should assert. However, most other things should silently return false since they're much easier for isl_surf_get_ccs to check than the caller. We also update the checking to be a bit more complete. Signed-off-by: Jason Ekstrand --

[Mesa-dev] [PATCH 07/13] anv/image: Stop force-disabling AUX

2016-09-25 Thread Jason Ekstrand
We're not creating any aux surfaces so force-disabling it does nothing whatsoever at the moment. Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_image.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index 4e22d64..fd8956b

[Mesa-dev] [PATCH 06/13] anv/image: Add an add_surface() helper

2016-09-25 Thread Jason Ekstrand
From: Chad Versace Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_image.c | 14 +++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index 93c1c06..4e22d64 100644 --- a/src/intel/vulkan/anv_image.c +++

[Mesa-dev] [PATCH 00/13] anv: Add initial CCS support

2016-09-25 Thread Jason Ekstrand
This little series adds the most basic of CCS support to the Vulkan driver. It's restricted to being entirely within a renderpass and only on the first mip level and first array slice. However, it does get us started and set up most of the infastructure we will need for both full compression and f

[Mesa-dev] [PATCH 04/13] anv/image: Add an isl_view to anv_image_view

2016-09-25 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_blorp.c | 4 ++-- src/intel/vulkan/anv_dump.c| 3 ++- src/intel/vulkan/anv_image.c | 37 ++--- src/intel/vulkan/anv_private.h | 4 ++-- src/intel/vulkan/genX_cmd_buffer.c | 4 ++-- 5

[Mesa-dev] [PATCH 02/13] anv/meta: Roll clear_image into CmdClearDepthStencilImage

2016-09-25 Thread Jason Ekstrand
It is now the only caller so there's no sense in keeping things split out. Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_meta_clear.c | 84 +-- 1 file changed, 28 insertions(+), 56 deletions(-) diff --git a/src/intel/vulkan/anv_meta_clear.c b/src/in

[Mesa-dev] [PATCH 05/13] intel/isl: Add some basic info about RENDER_SURFACE_STATE to isl_device

2016-09-25 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/blorp/blorp_genX_exec.h| 33 -- src/intel/isl/isl.c | 19 src/intel/isl/isl.h | 11 + src/intel/vulkan/anv_cmd_buffer.c| 8 +--- src/mesa

[Mesa-dev] [PATCH 03/13] anv/image: Rework our handling of 3-D image array ranges

2016-09-25 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_image.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/src/intel/vulkan/anv_image.c b/src/intel/vulkan/anv_image.c index f6e8672..f55c9b6 100644 --- a/src/intel/vulkan/anv_image.c +++ b/src/intel/vulkan/

[Mesa-dev] [PATCH 01/13] anv: Use blorp for VkCmdFillBuffer

2016-09-25 Thread Jason Ekstrand
Signed-off-by: Jason Ekstrand --- src/intel/vulkan/anv_blorp.c | 106 + src/intel/vulkan/anv_meta_clear.c | 120 -- 2 files changed, 96 insertions(+), 130 deletions(-) diff --git a/src/intel/vulkan/anv_blorp.c b/src/intel/v

Re: [Mesa-dev] [PATCH v2] intel/i965: fix resource leak

2016-09-25 Thread Lionel Landwerlin
Reviewed-by: Lionel Landwerlin On 25/09/16 16:06, Nayan Deshmukh wrote: CovID: 1373370 v2: fix indentation Signed-off-by: Nayan Deshmukh --- src/intel/tools/disasm.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/intel/tools/disasm.c b/src/intel/tools/disasm.c i

[Mesa-dev] [PATCH mesa 2/4] nir/spirv: improve lseek() error handling

2016-09-25 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- src/compiler/spirv/spirv2nir.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/src/compiler/spirv/spirv2nir.c b/src/compiler/spirv/spirv2nir.c index d151c2c..a725228 100644 --- a/src/compiler/spirv/spirv2nir.c +++ b/src/compiler/

[Mesa-dev] [PATCH mesa 4/4] nir/spirv: add spirv2nir binary to .gitignore

2016-09-25 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- src/compiler/.gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/src/compiler/.gitignore b/src/compiler/.gitignore index c0e6299..c4f17be 100644 --- a/src/compiler/.gitignore +++ b/src/compiler/.gitignore @@ -3,3 +3,4 @@ subtest-cr subtest-cr-lf subt

[Mesa-dev] [PATCH mesa 3/4] nir/spirv: improve mmap() error handling

2016-09-25 Thread Eric Engestrom
Signed-off-by: Eric Engestrom --- src/compiler/spirv/spirv2nir.c | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/src/compiler/spirv/spirv2nir.c b/src/compiler/spirv/spirv2nir.c index a725228..063e451 100644 --- a/src/compiler/spirv/spirv2nir.c +++ b/src/compiler/spir

[Mesa-dev] [PATCH mesa 1/4] nir/spirv: add some error checking to open()

2016-09-25 Thread Eric Engestrom
CovID: 1373369 Signed-off-by: Eric Engestrom --- src/compiler/spirv/spirv2nir.c | 9 + 1 file changed, 9 insertions(+) diff --git a/src/compiler/spirv/spirv2nir.c b/src/compiler/spirv/spirv2nir.c index c837186..d151c2c 100644 --- a/src/compiler/spirv/spirv2nir.c +++ b/src/compiler/spirv/

Re: [Mesa-dev] [PATCH 5/5] i965: stop passing stage as a function parameter

2016-09-25 Thread Jason Ekstrand
r-b On Sun, Sep 25, 2016 at 5:50 AM, Timothy Arceri < timothy.arc...@collabora.com> wrote: > We already pass the shader so we can just get the stage from this. > --- > src/mesa/drivers/dri/i965/brw_link.cpp | 8 +++- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/src/mesa

[Mesa-dev] [PATCH v2] intel/i965: fix resource leak

2016-09-25 Thread Nayan Deshmukh
CovID: 1373370 v2: fix indentation Signed-off-by: Nayan Deshmukh --- src/intel/tools/disasm.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/intel/tools/disasm.c b/src/intel/tools/disasm.c index e527615..4ac7b90 100644 --- a/src/intel/tools/disasm.c +++ b/src/intel/to

Re: [Mesa-dev] [PATCH] intel/i965: fix resource leak

2016-09-25 Thread Ilia Mirkin
On Sun, Sep 25, 2016 at 10:12 AM, Nayan Deshmukh wrote: > CovID: 1373370 > > Signed-off-by: Nayan Deshmukh > --- > src/intel/tools/disasm.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/intel/tools/disasm.c b/src/intel/tools/disasm.c > index e527615..a747a0a

[Mesa-dev] [PATCH] intel/i965: fix resource leak

2016-09-25 Thread Nayan Deshmukh
CovID: 1373370 Signed-off-by: Nayan Deshmukh --- src/intel/tools/disasm.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/intel/tools/disasm.c b/src/intel/tools/disasm.c index e527615..a747a0a 100644 --- a/src/intel/tools/disasm.c +++ b/src/intel/tools/disasm.c @@ -

[Mesa-dev] Various tidy ups

2016-09-25 Thread Timothy Arceri
I've sent these before as part of my work on calling the nir optimisations passes earlier however I've dropped the controversial changes in order to land these tidy ups. Patches 3 & 4 will help enabled us to call the nir passes earlier the others are just tidy ups. ___

[Mesa-dev] [PATCH 5/5] i965: stop passing stage as a function parameter

2016-09-25 Thread Timothy Arceri
We already pass the shader so we can just get the stage from this. --- src/mesa/drivers/dri/i965/brw_link.cpp | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_link.cpp b/src/mesa/drivers/dri/i965/brw_link.cpp index fc2e539..a0097ef 100644 -

[Mesa-dev] [PATCH 2/5] glsl: move some uniform linking code to new link_assign_uniform_storage()

2016-09-25 Thread Timothy Arceri
This makes link_assign_uniform_locations() easier to follow. --- src/compiler/glsl/link_uniforms.cpp | 132 +++- 1 file changed, 69 insertions(+), 63 deletions(-) diff --git a/src/compiler/glsl/link_uniforms.cpp b/src/compiler/glsl/link_uniforms.cpp index 11204fc.

[Mesa-dev] [PATCH 3/5] glsl: disable dead code removal of lowered ubos

2016-09-25 Thread Timothy Arceri
This lets us assign uniform storage for packed UBOs after they have been lowered otherwise the var is removed too early. --- src/compiler/glsl/glsl_parser_extras.cpp | 5 +++-- src/compiler/glsl/ir_optimization.h| 4 +++- src/compiler/glsl/link_varyings.cpp| 2 +- src/compiler/gl

[Mesa-dev] [PATCH 4/5] glsl: move update_uniform_buffer_variables() to lower UBO

2016-09-25 Thread Timothy Arceri
This make more sense here as its lowering that uses the results of this function. This allows us to call lower_ubo_reference() before assigning uniform locations which is useful for calling backend specific optimisations on the IR before assigning uniform and varying locations. While we are at it

[Mesa-dev] [PATCH 1/5] glsl: move some uniform linking code to new link_setup_uniform_remap_tables()

2016-09-25 Thread Timothy Arceri
This makes link_assign_uniform_locations() easier to follow. --- src/compiler/glsl/link_uniforms.cpp | 330 +++- src/compiler/glsl/linker.cpp| 4 +- src/compiler/glsl/linker.h | 5 +- 3 files changed, 177 insertions(+), 162 deletions(-) diff --

[Mesa-dev] [Bug 97921] Mesa-only glBlitFrameBuffer error

2016-09-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97921 --- Comment #1 from Timothy Arceri --- It would likely be useful if you created an apitrace of the problem, upload it somewhere and linked to it from here. https://github.com/apitrace/apitrace/blob/master/docs/USAGE.markdown -- You are receiv

[Mesa-dev] [PATCH] st/va: H.264 baseline profile is not supported

2016-09-25 Thread Mark Thompson
H.264 constrained baseline is, so use that instead. --- On decode, FMO doesn't work - it's commented out at , and I doubt it is supported by the hardware anyway. On encode, it seems to use constrained b

Re: [Mesa-dev] [PATCH] st/va: enable vbr rate control for vaapi encode

2016-09-25 Thread Andy Furniss
Andy Furniss wrote: https://patchwork.freedesktop.org/patch/112040/ Hmm that got mungled I'll try again later going to be AFK for a while. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-d

Re: [Mesa-dev] [PATCH] st/va: enable vbr rate control for vaapi encode

2016-09-25 Thread Andy Furniss
Leo Liu wrote: On 09/07/2016 03:02 AM, Christian König wrote: Am 06.09.2016 um 22:39 schrieb boyuan.zh...@amd.com: From: Boyuan Zhang This patch enables variable bit-rate for vaapi encoding. According to va.h, target bit-rate equals to maximum bit-rate multiplies by target_percentage. Sign

[Mesa-dev] [PATCH] st/va Avoid VBR bitrate calculation overflow

2016-09-25 Thread Andy Furniss
VBR bitrate calc needs 64 bits at high rates. Signed-off-by: Andy Furniss --- src/gallium/state_trackers/va/picture.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/va/picture.c b/src/gallium/state_trackers/va/picture.c index 7f3d96d..cd2dabb 10

[Mesa-dev] [Bug 97921] Mesa-only glBlitFrameBuffer error

2016-09-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=97921 Bug ID: 97921 Summary: Mesa-only glBlitFrameBuffer error Product: Mesa Version: git Hardware: x86-64 (AMD64) OS: Linux (All) Status: NEW Severity: normal

[Mesa-dev] [Bug 77449] Tracker bug for all bugs related to Steam titles

2016-09-25 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=77449 Jacek Konieczny changed: What|Removed |Added Depends on||97890 Referenced Bugs: https://bugs.