On Wednesday, April 26, 2017 3:45:00 PM PDT Francisco Jerez wrote:
> Kenneth Graunke writes:
>
> > The original Broadwater and Crestline platforms computed antialiased
> > line distances using "manhattan" distance, aka a + b = c. Eaglelake
> > and Cantiga added "true"
Make use of dispatch KHR_no_error support for varray functions.
---
src/mapi/glapi/gen/ARB_vertex_attrib_64bit.xml | 2 +-
src/mapi/glapi/gen/GL3x.xml| 3 +-
src/mapi/glapi/gen/es_EXT.xml | 3 +-
src/mapi/glapi/gen/gl_API.xml | 9 +
This will allows us to create no error versions of functions
noted by a _no_error suffix. We also need to set a no_error
attribute equal to "true" in the xml.
---
src/mapi/glapi/gen/gl_XML.py | 5 +
src/mapi/glapi/gen/gl_genexec.py | 19 ++-
2 files changed, 19
From: Michel Dänzer
Hardcode the OpenCL InputKind in compat::set_lang_defaults.
Signed-off-by: Michel Dänzer
---
src/gallium/state_trackers/clover/llvm/compat.hpp | 10 ++
src/gallium/state_trackers/clover/llvm/invocation.cpp | 2 +-
The majority of cache files are less than 1kb this resulted in us
greatly miscalculating the amount of disk space used by the cache.
Using the number of blocks allocated to the file is more
conservative and less likely to cause issues.
This change will result in cache sizes being miscalculated
Modern disks are extremely large and are only going to get bigger.
Usage has shown frequent Mesa upgrades can result in the cache
growing very fast i.e. wasting a lot of disk space unnecessarily.
5% seems like a more reasonable default.
Cc: "17.1"
---
On 26/04/17 07:11 PM, Gregory Hainaut wrote:
> On 4/26/17, Michel Dänzer wrote:
>> On 26/04/17 05:07 PM, Gregory Hainaut wrote:
>>>
>>> Note: those dri2* functions are typically called by gallium/mesa state
>>> tracker to handle new backbuffer allocation. When the old
On Monday, April 24, 2017 3:19:20 PM PDT Rafael Antognolli wrote:
[snip]
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
> b/src/mesa/drivers/dri/i965/genX_state_upload.c
> index cb3c2db..1b9dedf 100644
> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
> +++
On Monday, April 24, 2017 3:19:19 PM PDT Rafael Antognolli wrote:
> Emit 3DSTATE_SOL on Gen7+ using brw_batch_emit helper, that uses pack
> structs from genxml.
>
> v2:
>- Add helpers to assign struct brw_address (Kristian)
>
> Signed-off-by: Rafael Antognolli
>
Series is
Reviewed-by: Bas Nieuwenhuizen
On Wed, Apr 26, 2017 at 1:12 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This is ported from radeonsi, and I can see at least one
> Talos shader drops an export due to this, and saves
Reviewed-by: Bas Nieuwenhuizen
On Thu, Apr 27, 2017 at 1:35 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> Since the host pool changes,
>
> Fixes:
> dEQP-VK.api.descriptor_pool.out_of_pool_memory
>
> Fixes: 126d5ad "radv: Use
2017-04-27 0:37 GMT+02:00 Rob Herring :
>
> On Tue, Apr 25, 2017 at 3:12 PM, Rob Herring wrote:
> > On Tue, Apr 25, 2017 at 1:50 PM, Mauro Rossi wrote:
> >> 2017-04-25 18:21 GMT+02:00 Emil Velikov :
> >>>
> >>> On
On Monday, April 24, 2017 3:19:18 PM PDT Rafael Antognolli wrote:
> This function now lives inside genX_state_upload.c.
>
> Signed-off-by: Rafael Antognolli
Reviewed-by: Kenneth Graunke
signature.asc
Description: This is a digitally signed
On Monday, April 24, 2017 3:19:17 PM PDT Rafael Antognolli wrote:
[snip]
> diff --git a/src/mesa/drivers/dri/i965/genX_state_upload.c
> b/src/mesa/drivers/dri/i965/genX_state_upload.c
> index 68b7c29..ec85ec1 100644
> --- a/src/mesa/drivers/dri/i965/genX_state_upload.c
> +++
From: Dave Airlie
Since the host pool changes,
Fixes:
dEQP-VK.api.descriptor_pool.out_of_pool_memory
Fixes: 126d5ad "radv: Use host memory pool for non-freeable descriptors."
Signed-off-by: Dave Airlie
---
src/amd/vulkan/radv_descriptor_set.c | 2 +-
1
On Monday, April 24, 2017 3:19:16 PM PDT Rafael Antognolli wrote:
> Emit sf state on Gen6+ using brw_batch_emit helper, using pack structs
> from genxml.
>
> Signed-off-by: Rafael Antognolli
> ---
> src/mesa/drivers/dri/i965/brw_state.h | 3 +-
>
Kenneth Graunke writes:
> The original Broadwater and Crestline platforms computed antialiased
> line distances using "manhattan" distance, aka a + b = c. Eaglelake
> and Cantiga added "true" distance, aka a^2 + b^2 = c^2, which is
> obviously superior.
>
The hardware
Looks good to me :)
This series is: Reviewed-by: Plamena Manolova
On Wed, Apr 26, 2017 at 3:15 PM, Kenneth Graunke
wrote:
> We set this unconditionally on every other platform. Zero (Manhattan)
> isn't even listed as an option in the
On Tue, Apr 25, 2017 at 3:12 PM, Rob Herring wrote:
> On Tue, Apr 25, 2017 at 1:50 PM, Mauro Rossi wrote:
>> 2017-04-25 18:21 GMT+02:00 Emil Velikov :
>>>
>>> On 24 April 2017 at 22:49, Rob Herring wrote:
>>> >
Hello list,
The candidate for the Mesa 17.0.5 is now available. Currently we have:
- 38 queued
- 7 nominated (outstanding)
- and 2 rejected patch(es)
The current queue contains fixes in several fronts.
We have a fix for nouveau regarding the instructions emission with
GF100's ISA encoding.
The original Broadwater and Crestline platforms computed antialiased
line distances using "manhattan" distance, aka a + b = c. Eaglelake
and Cantiga added "true" distance, aka a^2 + b^2 = c^2, which is
obviously superior.
The G45 documentation indicates that the old manhattan distance setting
is
We set this unconditionally on every other platform. Zero (Manhattan)
isn't even listed as an option in the Sandybridge docs - only "true".
Cc: rafael.antogno...@intel.com
---
src/mesa/drivers/dri/i965/gen6_sf_state.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git
Looks good to me.
Reviewed-by: Timothy Arceri
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 27/04/17 01:01, Nicolai Hähnle wrote:
On 26.04.2017 12:16, Timothy Arceri wrote:
On 26/04/17 20:14, Timothy Arceri wrote:
On 26/04/17 19:50, Timothy Arceri wrote:
On 26/04/17 17:28, Nicolai Hähnle wrote:
On 26.04.2017 04:45, Timothy Arceri wrote:
Reviewed-by: Timothy Arceri
>On 2017-04-26 at 10:32, Emil Velikov wrote:
>No worries - we can always change/polish the theme as needed.
>Just checking that there's nothing 'fundamentally broken' there.
It's already fixed. Check the repo and the live demo.
>Right - I was skimming through 4.1
Looks good to me.
Reviewed-by: Neha Bhende
Regards,
Neha
From: Brian Paul
Sent: Wednesday, April 26, 2017 8:28:02 AM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Neha Bhende
Subject: [PATCH] st/mesa: minor
On 04/26/2017 09:40 AM, Nicolai Hähnle wrote:
On 24.04.2017 12:35, Samuel Pitoiset wrote:
The ARB_bindless_texture spec says:
"Interactions with GLSL 4.20"
"Without GLSL 4.20 support, sampler and image uniforms may only
be initialized through the OpenGL API. With GLSL 4.20,
>On 2017-04-26 at 10:32, Emil Velikov wrote:
>>>Thanks again for doing this Jean. A handful of, mostly trivial, questions.
>>>
>>> - Is there a website where one can check the output of specific
>>>Sphinx markdown.
>>
>> I have searched for such a thing too, but have not
https://bugs.freedesktop.org/show_bug.cgi?id=100741
--- Comment #2 from Bartosz Tomczyk ---
Can someone more experienced comment, which if any of above fixes is correct?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the
On 04/26/2017 09:30 AM, Nicolai Hähnle wrote:
On 24.04.2017 12:35, Samuel Pitoiset wrote:
The GL_ARB_bindless_texture allows to declare images inside
structures which means that qualifiers like writeonly should
be allowed.
I have a got a confirmation from Jeff Bolz (one author of the spec),
Strange ... for some reason, the prefix is wrong, and "kmscube" is
missing. Sorry about that. This set of 12 patches is to be applied
against kmscube.
Anyway, that's the last patchset version I'll post until it is merged
and/or reviewed.
On 2017-04-26 20:56, Carlos Rafael Giani wrote:
With synced playback, frames may be dropped if the OpenGL ES code does
not output the decoded frames in time, thereby making sure the timestamps
in the video aren't missed
Signed-off-by: Carlos Rafael Giani
---
common.h | 8
cube-video.c | 9 ++---
From: Nicolas Dufresne
This is done by setting a max-lateness and enable QoS messaging. This
will have the effect that buffer arriving late to the appsink won't will
be dropped instead of being queued. This is similar to what GstVideoSink
base class would do.
Signed-off-by: Carlos Rafael Giani
---
gst-decoder.c | 13 +
1 file changed, 9 insertions(+), 4 deletions(-)
diff --git a/gst-decoder.c b/gst-decoder.c
index d1cb18c..768aa1b 100644
--- a/gst-decoder.c
+++ b/gst-decoder.c
@@ -42,6 +42,8 @@
Signed-off-by: Carlos Rafael Giani
---
gst-decoder.c | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/gst-decoder.c b/gst-decoder.c
index 51304a2..05d73b7 100644
--- a/gst-decoder.c
+++ b/gst-decoder.c
@@ -115,15 +115,24 @@
* Make EGL image attribute specification code more generic, and not
specific to certain pixel formats, implicitely gaining support for YUY2
* Better handling of gstbuffers with multiple memory blocks
* Print out more information about the stream
* Use the GST_VIDEO_INFO_* macros instead of
This format is used for example by the i.MX6 CODA hardware video codec
Signed-off-by: Carlos Rafael Giani
---
gst-decoder.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/gst-decoder.c b/gst-decoder.c
index 22dc068..fd28201 100644
--- a/gst-decoder.c
+++
Signed-off-by: Carlos Rafael Giani
---
gst-decoder.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/gst-decoder.c b/gst-decoder.c
index fd28201..51304a2 100644
--- a/gst-decoder.c
+++ b/gst-decoder.c
@@ -63,24 +63,23 @@ static
The bus watch is useful for logging state changes, printing out
info/warning/error messages and handling common GStreamer activities like
latency redistribution and state change requests (that are sent by
elements since they are not allowed to directly change the state).
State changes and error
Signed-off-by: Carlos Rafael Giani
---
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure.ac b/configure.ac
index f564ef3..b5a0519 100644
--- a/configure.ac
+++ b/configure.ac
@@ -41,7 +41,7 @@ PKG_CHECK_MODULES(EGL, egl)
Signed-off-by: Carlos Rafael Giani
---
common.c | 17 +
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/common.c b/common.c
index 64c4f91..610ff87 100644
--- a/common.c
+++ b/common.c
@@ -125,9 +125,12 @@ int init_egl(struct egl *egl, const
From: Nicolas Dufresne
This way, upstream decoder the produce frame with special strides and
offsets won't have to copy the frames. This also ensure DMABuf can be
delivered to kmscube.
Signed-off-by: Nicolas Dufresne
---
gst-decoder.c | 25
Without this, the various GST_* log macros won't output anything.
To enable, add "kmscube:" to the GST_DEBUG environment variable.
Example: GST_DEBUG=kmscube:5
Signed-off-by: Carlos Rafael Giani
---
gst-decoder.c | 3 +++
kmscube.c | 4 +++-
2 files changed, 6
On 26 April 2017 at 09:38, Mun, Gwan-gyeong wrote:
> Hi Emil,
>
> Thanks for review my patch,
>
> 2017-04-26 0:58 GMT+09:00 Daniel Stone :
>> Hi,
>>
>> On 25 April 2017 at 17:03, Emil Velikov wrote:
>>> From a quick look most of
https://bugs.freedesktop.org/show_bug.cgi?id=100789
--- Comment #2 from calexil ---
(In reply to Michel Dänzer from comment #1)
> Is this a regression from older versions of Mesa? If so, can you bisect Mesa
> or at least narrow down the version where the problem first
On Monday, April 24, 2017 3:19:25 PM PDT Rafael Antognolli wrote:
> Upload blend states using GENX(BLEND_STATE_ENTRY_pack), generated from
> genxml.
>
> Signed-off-by: Rafael Antognolli
> ---
> src/mesa/drivers/dri/i965/Makefile.sources| 1 +-
>
---
src/gallium/drivers/swr/Makefile.sources | 1 +
src/gallium/drivers/swr/rasterizer/common/os.cpp | 107 +
src/gallium/drivers/swr/rasterizer/common/os.h | 5 +-
.../drivers/swr/rasterizer/core/threads.cpp| 15 ++-
4 files changed, 126
Misplaced #endif preventing depth and stencil hot tile pointers
from incrementing in SIMD16 8x2 configuration of BackendPixelRate.
---
src/gallium/drivers/swr/rasterizer/core/backend.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
---
src/gallium/drivers/swr/rasterizer/core/api.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/swr/rasterizer/core/api.h
b/src/gallium/drivers/swr/rasterizer/core/api.h
index d0f29dd..b9b994a 100644
--- a/src/gallium/drivers/swr/rasterizer/core/api.h
---
src/gallium/drivers/swr/rasterizer/core/api.cpp | 23 ---
src/gallium/drivers/swr/rasterizer/core/api.h | 5 +
src/gallium/drivers/swr/rasterizer/core/backend.h | 3 ---
src/gallium/drivers/swr/swr_context.cpp | 5 +
---
src/gallium/drivers/swr/rasterizer/common/os.h | 6 +
src/gallium/drivers/swr/rasterizer/core/api.cpp | 48
src/gallium/drivers/swr/rasterizer/core/api.h | 141
3 files changed, 151 insertions(+), 44 deletions(-)
diff --git
---
src/gallium/drivers/swr/rasterizer/core/knobs.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/swr/rasterizer/core/knobs.h
b/src/gallium/drivers/swr/rasterizer/core/knobs.h
index 640b672..7ad6fe3 100644
---
---
src/gallium/drivers/swr/rasterizer/core/api.cpp| 3 ++
src/gallium/drivers/swr/rasterizer/core/api.h | 45 ++
.../drivers/swr/rasterizer/memory/ClearTile.cpp| 2 +-
.../drivers/swr/rasterizer/memory/LoadTile.cpp | 2 +-
Small fetch performance optimization - use gather instruction
for odd format fetch instead of slow emulated code.
---
.../drivers/swr/rasterizer/jitter/fetch_jit.cpp| 55 --
1 file changed, 9 insertions(+), 46 deletions(-)
diff --git
Disabling buffer overrun warning for Assemble(uint32_t slot,
simdvector *verts) due to what looks like a MSVC compiler bug
when compiling the SIMD16 FE.
---
src/gallium/drivers/swr/rasterizer/core/pa.h | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git
---
src/gallium/drivers/swr/rasterizer/core/clip.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/swr/rasterizer/core/clip.h
b/src/gallium/drivers/swr/rasterizer/core/clip.h
index 28042d5..ad2745b 100644
---
Fixes gcc error for SIMD16 FE.
---
src/gallium/drivers/swr/rasterizer/core/binner.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/swr/rasterizer/core/binner.cpp
b/src/gallium/drivers/swr/rasterizer/core/binner.cpp
index 3c94004..958f0a2 100644
---
Frontend - reduce simdvertex/simd16vertex stack usage for VS output in
ProcessDraw, fixes stack overflow in some of the deeper call stacks under
SIMD16.
1. Move the vertex store out of PA_FACTORY, and off the stack
2. Allocate the vertex store out of the aligned heap (pointer is
temporarily
---
src/gallium/drivers/swr/rasterizer/core/api.cpp | 6 +-
src/gallium/drivers/swr/rasterizer/core/api.h | 7 ++-
src/gallium/drivers/swr/rasterizer/core/backend.cpp | 10 +-
src/gallium/drivers/swr/rasterizer/core/backend.h | 2 +-
Fixes MSVC errors with SIMD16 FE.
---
src/gallium/drivers/swr/rasterizer/core/binner.cpp | 6 +++---
src/gallium/drivers/swr/rasterizer/core/clip.cpp | 6 +++---
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/swr/rasterizer/core/binner.cpp
On 2017-04-26 at 10:32, Emil Velikov wrote:
No worries - we can always change/polish the theme as needed.
Just checking that there's nothing 'fundamentally broken' there.
It's already fixed. Check the repo and the live demo.
Right - I was skimming through 4.1
Not used yet.
---
.../drivers/swr/rasterizer/jitter/builder.cpp | 1 +
.../drivers/swr/rasterizer/jitter/builder.h| 1 +
.../drivers/swr/rasterizer/jitter/builder_misc.cpp | 69 +-
.../drivers/swr/rasterizer/jitter/builder_misc.h | 6 ++
4 files changed, 76
---
src/gallium/drivers/swr/rasterizer/core/binner.cpp | 8
src/gallium/drivers/swr/rasterizer/core/frontend.cpp | 7 ++-
src/gallium/drivers/swr/rasterizer/core/pa_avx.cpp | 1 -
3 files changed, 2 insertions(+), 14 deletions(-)
diff --git
Ability to allocate space for an arbitrary number (at compile time)
of positions in the vertex layout.
Removes KNOB_NUM_ATTRIBUTES from knobs.h, replaces the VTX slot
number #defines with the SWR_VTX_SLOTS enum (which contains
replacement for NUM_ATTRIBUTES: SWR_VTX_NUM_SLOTS)
---
v2:
Rewrite commit messages
Split some commits
No code/functional changes
Tim Rowley (17):
swr/rast: more flexible max attribute slots
swr/rast: add additional jit utility functions
swr/rast: add SIMDAPI decorators in binner/clipper
swr/rast: clean up clipper comments
swr/rast: disable
https://bugs.freedesktop.org/show_bug.cgi?id=100690
--- Comment #7 from Kenneth Graunke ---
Thanks :) I'm just concerned that since no drivers have been enforcing this
rule, there might be a lot of apps that get hit by this issue.
--
You are receiving this mail because:
https://bugs.freedesktop.org/show_bug.cgi?id=90264
Joseph Yasi changed:
What|Removed |Added
CC||joe.y...@gmail.com
--
Add logic for converting enums and also making sure stipple works.
v2: remove cc stable, and remove "not implemented" assert
---
src/gallium/drivers/swr/swr_state.cpp | 8 +++-
src/gallium/drivers/swr/swr_state.h | 20
2 files changed, 27 insertions(+), 1 deletion(-)
add support in the binner: Split BinPoints into BinPostSetupPoints,
so we can use it from BinTriangles, since setup is already done.
v2: remove cc stable
---
src/gallium/drivers/swr/rasterizer/core/binner.cpp | 175 ++---
1 file changed, 117 insertions(+), 58 deletions(-)
diff
On 24 April 2017 at 18:20, Eric Engestrom wrote:
> On Friday, 2017-04-21 13:08:26 +0100, Emil Velikov wrote:
>> From: Emil Velikov
>>
>> Signed-off-by: Emil Velikov
>> ---
>> .travis.yml | 39
For dual source blending, CB_COLOR1_INFO is required to be the same as
CB_COLOR0_INFO.
Other than that, as long as there are no piglit regressions, I'm OK
with any development going on in gallium/drivers/r600.
If you want me to commit stuff, please send me a pull request or you
can ask for an
On 21 April 2017 at 17:46, Jordan Justen wrote:
> On 2017-04-21 04:02:11, Emil Velikov wrote:
>> On 21 April 2017 at 07:18, Jordan Justen wrote:
>> > On 2017-04-20 18:49:57, Timothy Arceri wrote:
>> >> On 21/04/17 11:37, Jordan Justen wrote:
On Wed, Apr 26, 2017 at 1:12 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> This code can be shared by radv, we bump the max to
> VARYING_SLOT_MAX here, but that shouldn't have too
> much fallout.
>
> Signed-off-by: Dave Airlie
> ---
>
On 26.04.2017 17:51, Marc Dietrich wrote:
> Am Mittwoch, 26. April 2017, 16:36:39 CEST schrieb Constantine Kharlamov:
>> HEAD recently have triggered a bug in sb compiler, so, just for the safe
>> case, can you tell if reverting this commit
>>
>> git revert --no-commit
On Mon, Apr 24, 2017 at 11:42 PM, Samuel Pitoiset
wrote:
>
>
> On 04/24/2017 11:22 PM, Rob Clark wrote:
>>
>> On Mon, Apr 24, 2017 at 5:18 PM, Samuel Pitoiset
>> wrote:
>>>
>>>
>>>
>>> On 04/24/2017 11:12 PM, Rob Clark wrote:
For patches 1-3:
Reviewed-by: Marek Olšák
Marek
On Tue, Apr 25, 2017 at 12:31 AM, Samuel Pitoiset
wrote:
> 47109 shaders in 29632 tests
> Totals:
> SGPRS: 1917364 -> 1916620 (-0.04 %)
> VGPRS: 1165802 -> 1165202 (-0.05 %)
> Spilled SGPRs: 1880
Reviewed-by: Samuel Pitoiset
On 04/26/2017 03:21 PM, Boyan Ding wrote:
v2:
Also handle the the new format in indirect dispatch
Use compute class check instead of chipset check
Signed-off-by: Boyan Ding
---
Reviewed-by: Marek Olšák
Marek
On Tue, Apr 25, 2017 at 1:10 AM, Samuel Pitoiset
wrote:
> No longer used.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/auxiliary/Makefile.sources | 2 -
>
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 26, 2017 at 12:36 AM, Samuel Pitoiset
wrote:
> Because the buffer is new, it can't be referenced by any CS.
>
> This can save few CPU cycles by skipping the whole
> PIPE_TRANSFER_UNSYNCHRONIZED if in
The old algorithm worked fine assuming a constant block size. We're
about to break that assumption so we need an algorithm that's a bit more
robust against suddenly growing by a huge amount compared to the
currently allocated quantity of memory.
---
src/intel/vulkan/anv_allocator.c | 28
---
src/intel/vulkan/anv_allocator.c | 10 +++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/src/intel/vulkan/anv_allocator.c b/src/intel/vulkan/anv_allocator.c
index 8f3b31f..61d09f3 100644
--- a/src/intel/vulkan/anv_allocator.c
+++ b/src/intel/vulkan/anv_allocator.c
@@
For the series:
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 19, 2017 at 10:56 AM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> ---
> src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 4 ++--
> 1 file changed, 2 insertions(+), 2
Interface blocks don't allow to declare opaque types and atomic
counters/images are forbidden inside structures.
Fixes format-layout-with-non-image-type.frag and
memory-qualifier-with-non-image-type.frag.
Signed-off-by: Samuel Pitoiset
---
Signed-off-by: Samuel Pitoiset
---
src/compiler/glsl/ast_to_hir.cpp | 29 -
1 file changed, 20 insertions(+), 9 deletions(-)
diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/ast_to_hir.cpp
index e26bd3f047..a1463c38c8
Signed-off-by: Samuel Pitoiset
---
src/compiler/glsl/ast_to_hir.cpp | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/ast_to_hir.cpp
index aeb223db9e..e26bd3f047 100644
---
grrr... Wrong list obviously. Sorry for the noise.
On 04/26/2017 06:40 PM, Samuel Pitoiset wrote:
Signed-off-by: Samuel Pitoiset
---
.../memory-qualifier-with-non-image-type.frag | 25 ++
1 file changed, 25 insertions(+)
create mode
Signed-off-by: Samuel Pitoiset
---
.../memory-qualifier-with-non-image-type.frag | 25 ++
1 file changed, 25 insertions(+)
create mode 100644
tests/spec/arb_shader_image_load_store/compiler/memory-qualifier-with-non-image-type.frag
diff
On Wed, 2017-04-26 at 07:35 -0700, Jason Ekstrand wrote:
> ---
> src/intel/vulkan/anv_allocator.c | 71
> +--
> src/intel/vulkan/anv_cmd_buffer.c | 8 ++--
> src/intel/vulkan/anv_descriptor_set.c | 4 +-
> src/intel/vulkan/anv_private.h| 21
Reviewed-by: Marek Olšák
Marek
On Wed, Apr 26, 2017 at 5:28 PM, Brian Paul wrote:
> Remove unneeded parens. Add const qualifiers. Move var decls closer
> to where they're used.
> ---
> src/mesa/state_tracker/st_cb_fbo.c | 17 -
> 1
Hi Emil,
Thanks for the comments, I’ll work on improving my commit messages.
In this case, the single change to fix crashes yields all 3 improvements —
tests were crashing because our format types were limited. The additional
benefit of supporting these types is much increased performance.
Remove unneeded parens. Add const qualifiers. Move var decls closer
to where they're used.
---
src/mesa/state_tracker/st_cb_fbo.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/src/mesa/state_tracker/st_cb_fbo.c
b/src/mesa/state_tracker/st_cb_fbo.c
index
On 26.04.2017 12:16, Timothy Arceri wrote:
On 26/04/17 20:14, Timothy Arceri wrote:
On 26/04/17 19:50, Timothy Arceri wrote:
On 26/04/17 17:28, Nicolai Hähnle wrote:
On 26.04.2017 04:45, Timothy Arceri wrote:
Reviewed-by: Timothy Arceri
On 24/04/17 20:35, Samuel
Am Mittwoch, 26. April 2017, 16:36:39 CEST schrieb Constantine Kharlamov:
> HEAD recently have triggered a bug in sb compiler, so, just for the safe
> case, can you tell if reverting this commit
>
> git revert --no-commit eb8aa93c03ee89ffd3041d41b6293e4b282b6ce6
>
> but having my patch still
Now that we can allocate states larger than the block size, we no longer
need a block size of 1MB which can be rather wasteful.
---
src/intel/vulkan/anv_device.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
---
src/intel/vulkan/anv_allocator.c| 23 ++-
src/intel/vulkan/tests/block_pool_no_free.c | 7 ---
2 files changed, 14 insertions(+), 16 deletions(-)
diff --git a/src/intel/vulkan/anv_allocator.c b/src/intel/vulkan/anv_allocator.c
index fa7101b..e58734b
Previously, the maximum size of a state that could be allocated from a
state pool was a block. However, this has caused us various issues
particularly with shaders which are potentially very large. We've also
hit issues with render passes with a large number of attachments when we
go to allocate
HEAD recently have triggered a bug in sb compiler, so, just for the safe case,
can you tell if reverting this commit
git revert --no-commit eb8aa93c03ee89ffd3041d41b6293e4b282b6ce6
but having my patch still applied changes anything or not?
(I have this commit reverted locally)
On 26.04.2017
---
src/intel/vulkan/anv_allocator.c | 19 ---
1 file changed, 16 insertions(+), 3 deletions(-)
diff --git a/src/intel/vulkan/anv_allocator.c b/src/intel/vulkan/anv_allocator.c
index 0e33d1e..7a687b7 100644
--- a/src/intel/vulkan/anv_allocator.c
+++
Now that the state stream is allocating off of the state pool, there's
no reason why we need the block pool to be separate.
---
src/intel/vulkan/anv_allocator.c | 30 ---
src/intel/vulkan/anv_batch_chain.c | 35 +++---
---
src/intel/vulkan/anv_allocator.c | 30 --
1 file changed, 20 insertions(+), 10 deletions(-)
diff --git a/src/intel/vulkan/anv_allocator.c b/src/intel/vulkan/anv_allocator.c
index e58734b..0e33d1e 100644
--- a/src/intel/vulkan/anv_allocator.c
+++
1 - 100 of 202 matches
Mail list logo