Re: [Mesa-dev] [PATCH libdrm v2 5/5] xf86drm: implement an OpenBSD specific drmGetDevice2

2016-12-07 Thread Emil Velikov
On 6 December 2016 at 05:12, Jonathan Gray wrote: > On Mon, Dec 05, 2016 at 05:56:40PM +, Emil Velikov wrote: >> On 1 December 2016 at 04:18, Jonathan Gray wrote: >> > DRI devices on OpenBSD are not in their own directory. They reside in >> > /dev with a

Re: [Mesa-dev] [PATCH shader-db] run: don't use alloca to avoid random crashes in the GLSL compiler

2016-12-07 Thread Ilia Mirkin
What's wrong with feeding it stack memory? On Wed, Dec 7, 2016 at 12:33 PM, Marek Olšák wrote: > From: Marek Olšák > > --- > run.c | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/run.c b/run.c > index 08fd543..ded224a 100644

Re: [Mesa-dev] [PATCH shader-db] run: don't use alloca to avoid random crashes in the GLSL compiler

2016-12-07 Thread Ian Romanick
On 12/07/2016 09:44 AM, Marek Olšák wrote: > On Wed, Dec 7, 2016 at 6:42 PM, Ilia Mirkin wrote: >> What's wrong with feeding it stack memory? > > I don't really have an answer to that. Does valgrind provide any clues? The usual problems are running out of stack (is that

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Ilia Mirkin
On Wed, Dec 7, 2016 at 10:31 AM, Ilia Mirkin wrote: > On Wed, Dec 7, 2016 at 10:25 AM, Lionel Landwerlin > wrote: >> On 07/12/16 14:55, Ilia Mirkin wrote: >>> >>> On Thu, Dec 1, 2016 at 10:56 AM, Lionel Landwerlin >>>

Re: [Mesa-dev] [PATCH 1/3] cso: don't release sampler states that are bound

2016-12-07 Thread Nicolai Hähnle
On 07.12.2016 17:26, Marek Olšák wrote: Optimizing the CSO cache isn't exactly on the top of my list, so I can't really do that right now. I think that varying the LOD bias is starting to be common. It's used for smooth LOD transitions when loading textures during rendering. Games with lots of

Re: [Mesa-dev] [PATCH] i965/blit: Remove Yf/Ys tiled check under a FIXME at can_fast_copy_blit

2016-12-07 Thread Anuj Phogat
On Wed, Dec 7, 2016 at 3:38 AM, Alejandro Piñeiro wrote: > > > On 07/12/16 09:36, Alejandro Piñeiro wrote: >> On 06/12/16 22:26, Anuj Phogat wrote: >>> On Tue, Dec 6, 2016 at 10:58 AM, Alejandro Piñeiro >>> wrote: The FIXME suggest that the check

[Mesa-dev] [PATCH 1/7] mesa: Add types for ARB_transform_feedback_oveflow_query.

2016-12-07 Thread Rafael Antognolli
Add some basic types and storage for the queries of this extension. Signed-off-by: Rafael Antognolli --- src/mesa/main/extensions_table.h | 1 + src/mesa/main/mtypes.h | 5 + 2 files changed, 6 insertions(+) diff --git

[Mesa-dev] [PATCH 0/7] Add support for ARB_transform_feedback_overflow_query.

2016-12-07 Thread Rafael Antognolli
This patch series implements the ARB_transform_feedback_overflow_query extension for i965. It already addresses the issues raised by Ian Romanick on the RFC. The series is available on github here: https://github.com/rantogno/mesa/tree/review/overflow_query-v01 There are also piglit tests

[Mesa-dev] [PATCH 6/7] i965: Enable arb_transform_feedback_overflow_query.

2016-12-07 Thread Rafael Antognolli
This extension adds new query types which can be used to detect overflow of transform feedback buffers. The new query types are also accepted by conditional rendering commands. Signed-off-by: Rafael Antognolli --- docs/features.txt| 2 +-

[Mesa-dev] [PATCH 3/7] i965: add plumbing for ARB_transform_feedback_overflow_query.

2016-12-07 Thread Rafael Antognolli
When querying for transform feedback overflow on one or all of the streams, store information about number of generated and written primitives. Then check whether generated == written. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/brw_queryobj.c |

[Mesa-dev] [PATCH 4/7] i965: Add support for xfb overflow on query buffer objects.

2016-12-07 Thread Rafael Antognolli
Enable getting the results of a transform feedback overflow query with a buffer object. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/hsw_queryobj.c | 108 +++ 1 file changed, 108 insertions(+) diff --git

[Mesa-dev] [PATCH 7/7] i965: Enable predicate support on gen >= 8.

2016-12-07 Thread Rafael Antognolli
Predication needs cmd parser only on gen7. For newer platforms, it should be available without it. Signed-off-by: Rafael Antognolli --- src/mesa/drivers/dri/i965/intel_extensions.c | 1 + 1 file changed, 1 insertion(+) diff --git

[Mesa-dev] [PATCH 2/7] mesa: Track transform feedback overflow query objects.

2016-12-07 Thread Rafael Antognolli
Also update checks on conditional rendering. Signed-off-by: Rafael Antognolli --- src/mesa/main/condrender.c | 4 +++- src/mesa/main/queryobj.c| 21 + src/mesa/state_tracker/st_cb_queryobj.c | 6 ++ 3 files

[Mesa-dev] [PATCH 5/7] i965: Add support for xfb overflow query on conditional render.

2016-12-07 Thread Rafael Antognolli
Enable the use of a transform feedback overflow query with glBeginConditionalRender. The render commands will only execute if the query is true (i.e. if there was an overflow). Use ARB_conditional_render_inverted to change this behavior. Signed-off-by: Rafael Antognolli

[Mesa-dev] [Bug 98563] Xorg segfaults with displaylink attached and mesa version >= 13.0

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98563 --- Comment #21 from Mark Janes --- David, thank you for your input on this bug. Even though the root cause of your crash was different, I would have come to the same initial conclusion as you based on the different

[Mesa-dev] [PATCH shader-db] run: don't use alloca to avoid random crashes in the GLSL compiler

2016-12-07 Thread Marek Olšák
From: Marek Olšák --- run.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/run.c b/run.c index 08fd543..ded224a 100644 --- a/run.c +++ b/run.c @@ -656,28 +656,32 @@ main(int argc, char **argv) /* If there's only one GLSL shader, mark

Re: [Mesa-dev] [PATCH v2 7/7] i965: Increase max texture to 16k for gen7+

2016-12-07 Thread Jason Ekstrand
This has some interesting implications... >From the Sky Lake PRM, Vol 7, section "Vertex XY Clamping and Quantization", > In addition, in order to be correctly rendered, objects must have a screenspace > bounding box not exceeding 8K in the X or Y direction. This additional restriction > must

Re: [Mesa-dev] [PATCH 1/3] cso: don't release sampler states that are bound

2016-12-07 Thread Roland Scheidegger
Am 07.12.2016 um 17:26 schrieb Marek Olšák: > Optimizing the CSO cache isn't exactly on the top of my list, so I > can't really do that right now. > > I think that varying the LOD bias is starting to be common. It's used > for smooth LOD transitions when loading textures during rendering. > Games

[Mesa-dev] [Bug 98563] Xorg segfaults with displaylink attached and mesa version >= 13.0

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98563 --- Comment #20 from David Rosenstrauch --- BTW, is there a separate bug report for that opendir issue that I can track to know when a fix gets released? -- You are receiving this mail because: You are the assignee for the

Re: [Mesa-dev] [PATCH 1/2] mesa: add support for GL_INTEL_conservative_rasterization

2016-12-07 Thread Ilia Mirkin
On Wed, Dec 7, 2016 at 12:04 PM, Ilia Mirkin wrote: > On Wed, Dec 7, 2016 at 10:31 AM, Ilia Mirkin wrote: >> On Wed, Dec 7, 2016 at 10:25 AM, Lionel Landwerlin >> wrote: >>> On 07/12/16 14:55, Ilia Mirkin wrote:

Re: [Mesa-dev] [PATCH shader-db] run: don't use alloca to avoid random crashes in the GLSL compiler

2016-12-07 Thread Marek Olšák
On Wed, Dec 7, 2016 at 6:42 PM, Ilia Mirkin wrote: > What's wrong with feeding it stack memory? I don't really have an answer to that. Marek ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH] [rfc] glsl: allow invariant on fragment shader outputs.

2016-12-07 Thread Andres Gomez
Hei Dave, recently, khronos bug 16140 was opened to address my comments regarding the "#pragma STDGL invariant(all)" sentence in GLSL specs since 1.30. The action to be taken will be removing such sentence. In other words, the "invariant" keyword should be also legal for FS outputs. Therefore,

Re: [Mesa-dev] [PATCH shader-db] run: don't use alloca to avoid random crashes in the GLSL compiler

2016-12-07 Thread Jan Vesely
On Wed, 2016-12-07 at 18:44 +0100, Marek Olšák wrote: > On Wed, Dec 7, 2016 at 6:42 PM, Ilia Mirkin wrote: > > What's wrong with feeding it stack memory? > > I don't really have an answer to that. what happens when you malloc shader[i].length + 1 instead of +1000? looks

[Mesa-dev] [Bug 98563] Xorg segfaults with displaylink attached and mesa version >= 13.0

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=98563 --- Comment #19 from David Rosenstrauch --- Well my stack trace is actually almost identical to one someone posted later in that Arch forum thread (see https://bbs.archlinux.org/viewtopic.php?pid=1672878#p1672878) which is how

[Mesa-dev] [Bug 99014] clover is broken in 12.0.4

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99014 --- Comment #8 from Jan Vesely --- 12.0.4 is working fine* on r600 (turks). Looks like SI compute problem. *fine: within limits of what I know was broken in LLVM 3.8 -- You are receiving this mail because: You

Re: [Mesa-dev] [PATCH 1/3] gallium: add renderonly library

2016-12-07 Thread Alexandre Courbot
On 11/30/2016 10:44 PM, Christian Gmeiner wrote: > This a very lightweight library to add basic support for > renderonly GPUs. It does all the magic regarding in/exporting > buffers etc. This library will likely break android support and > hopefully will get replaced with a better solution based

Re: [Mesa-dev] [PATCH v2 13/25] radeonsi: load the streamout buf descriptors closer to their use

2016-12-07 Thread Marek Olšák
Other than the comment on patch 2, patches 1-13 are: Reviewed-by: Marek Olšák Marek On Tue, Dec 6, 2016 at 11:48 AM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > LLVM can still decide to hoist the loads since they're

[Mesa-dev] [PATCH 24/84] st/nine: Back User Clip Planes to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 5 - src/gallium/state_trackers/nine/nine_state.c | 30

[Mesa-dev] [PATCH 20/84] st/nine: Refactor LightEnable

2016-12-07 Thread Axel Davy
Call a helper function. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 25 +--- src/gallium/state_trackers/nine/nine_state.c | 35 src/gallium/state_trackers/nine/nine_state.h | 4 3 files changed,

[Mesa-dev] [PATCH 08/84] st/nine: Move vtxbuf to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 33 -- src/gallium/state_trackers/nine/nine_state.c

[Mesa-dev] [PATCH 25/84] st/nine: Track dirty state groups in nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 3 +- src/gallium/state_trackers/nine/device9.c | 2 +-

[Mesa-dev] [PATCH 22/84] st/nine: Back ds to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 7 +++--- src/gallium/state_trackers/nine/nine_state.c | 33

[Mesa-dev] [PATCH 12/84] st/nine: Back sampler states to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 18 +++--- src/gallium/state_trackers/nine/nine_state.c | 83

[Mesa-dev] [PATCH 17/84] st/nine: Back viewport to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 4 ++- src/gallium/state_trackers/nine/nine_ff.c| 2 +-

[Mesa-dev] [PATCH 01/84] st/nine: Introduce nine_context

2016-12-07 Thread Axel Davy
nine_context is a new structure which goal will be to contain all internal states. It will be the states of the second thread in the to-be-introduced CSMT mode. This patch moves several internal states to nine_context, while the next patches add the other fields. Signed-off-by: Axel Davy

[Mesa-dev] [PATCH 30/84] st/nine: Rename cso in nine_context to cso_shader

2016-12-07 Thread Axel Davy
This patch it to avoid name conflict when device->cso is moved to nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c| 12 ++-- src/gallium/state_trackers/nine/nine_state.h| 2 +-

[Mesa-dev] [PATCH 03/84] st/nine: Move draw calls to nine_state

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 209 +++--

[Mesa-dev] [PATCH 21/84] st/nine: Back all ff states in nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 60 -- src/gallium/state_trackers/nine/nine_ff.c |

[Mesa-dev] [PATCH 37/84] st/nine: Decompose nine_context_set_indices

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 32 +--- 1 file changed, 24 insertions(+),

[Mesa-dev] [PATCH 62/84] st/nine: Add CSMT_NO_WAIT_WITH_COUNTER

2016-12-07 Thread Axel Davy
From: Patrick Rudolph Similar to the other macros, but introduces a counter, which enables to know when the instructions has been executed. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/nine_csmt_helper.h | 54 ++

[Mesa-dev] [PATCH 60/84] st/nine: Optimize ColorFill

2016-12-07 Thread Axel Davy
When we lock the whole surface to overwrite it, we can use DISCARD. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c

[Mesa-dev] [PATCH 26/84] st/nine: Use atomics for nine_bind

2016-12-07 Thread Axel Davy
nine_bind didn't need atomics up to now, because it's use what always within a protected mutex. We need to use atomics because with the next patches several threads may use nine_bind. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/iunknown.h | 5 +++-- 1 file

[Mesa-dev] [PATCH 65/84] st/nine: Fix NineUnknown_Detach

2016-12-07 Thread Axel Davy
We don't bind the container in AddRef. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/iunknown.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gallium/state_trackers/nine/iunknown.h b/src/gallium/state_trackers/nine/iunknown.h index

[Mesa-dev] [PATCH 10/84] st/nine: Back vdecl to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 21 +--

[Mesa-dev] [PATCH 34/84] st/nine: Move Managed Pool handling out of nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 57

[Mesa-dev] [PATCH 28/84] st/nine: Remove NineDevice9_GetCSO

2016-12-07 Thread Axel Davy
Was useless. Remove useless usage in swapchain9. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 6 -- src/gallium/state_trackers/nine/device9.h| 3 --- src/gallium/state_trackers/nine/swapchain9.c | 1 -

[Mesa-dev] [PATCH 15/84] st/nine: Back RT to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 16 +++- src/gallium/state_trackers/nine/nine_state.c | 54

[Mesa-dev] [PATCH 09/36] st/nine: Remove useless check in surface9 ctor

2016-12-07 Thread Axel Davy
Textures already have the check in BaseTexture9. Non-Textures cannot be in the MANAGED Pool. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/surface9.c | 4 1 file changed, 4 deletions(-) diff --git a/src/gallium/state_trackers/nine/surface9.c

[Mesa-dev] [PATCH 24/36] driconf: Fix missing gettext

2016-12-07 Thread Axel Davy
DRI_CONF_NINE_OVERRIDEVENDOR was missing gettext for the description. Signed-off-by: Axel Davy --- src/mesa/drivers/dri/common/xmlpool/t_options.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/common/xmlpool/t_options.h

[Mesa-dev] [PATCH 06/36] st/nine: Fix ps multisample check

2016-12-07 Thread Axel Davy
We want to use centroid for nonmaskable multisampling as well. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/pixelshader9.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/pixelshader9.h

[Mesa-dev] [PATCH 15/36] st/nine: Correctly release sw cursor image

2016-12-07 Thread Axel Davy
cursor.image is used for software cursor emulation. It wasn't released. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/nine/device9.c

[Mesa-dev] [PATCH 13/36] st/nine: Avoid crash on empty Draw*Up

2016-12-07 Thread Axel Davy
Ignore empty draw calls. Avoid assertion fault when such draw calls happen in u_upload_mgr. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/state_trackers/nine/device9.c

[Mesa-dev] [PATCH 03/36] st/nine: Remove useless call to nine_update_state

2016-12-07 Thread Axel Davy
The call was not needed. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/device9.c b/src/gallium/state_trackers/nine/device9.c index 055e71d..7bdf19f 100644 ---

[Mesa-dev] [Bug 99010] --disable-gallium-llvm no longer recognized

2016-12-07 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=99010 --- Comment #13 from Vinson Lee --- I also get this error when trying to use softpipe. libGL: dlopen lib/gallium/swrast_dri.so failed (lib/gallium/swrast_dri.so: undefined symbol: draw_gs_llvm_destroy_variant) -- You are

[Mesa-dev] [PATCH 67/84] st/nine: Comment and simplify iunknown

2016-12-07 Thread Axel Davy
The behaviour is a bit less obscure now. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/iunknown.c | 9 ++--- src/gallium/state_trackers/nine/iunknown.h | 30 -- 2 files changed, 18 insertions(+), 21 deletions(-) diff --git

[Mesa-dev] [PATCH 63/84] st/nine: Avoid flushing the queue for queries GetData

2016-12-07 Thread Axel Davy
Use the newly introduced counter to know when we don't need synchronization. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 33 +++- src/gallium/state_trackers/nine/nine_state.h | 6 ++---

[Mesa-dev] [PATCH 33/84] st/nine: Integrate nine_pipe_context_clear to nine_context_clear

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 7 ++- src/gallium/state_trackers/nine/device9ex.c | 3 +--

[Mesa-dev] [PATCH 55/84] st/nine: Add nine_context_get_pipe_acquire/release

2016-12-07 Thread Axel Davy
See commit for description. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_queue.c | 9 src/gallium/state_trackers/nine/nine_queue.h | 3 ++ src/gallium/state_trackers/nine/nine_state.c | 67

[Mesa-dev] [PATCH 41/84] st/nine: Back swvp in nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 3 ++- src/gallium/state_trackers/nine/nine_state.c| 14

[Mesa-dev] [PATCH 57/84] st/nine: Implement Fast path for dynamic buffers and csmt

2016-12-07 Thread Axel Davy
Use the secondary pipe for DISCARD/NOOVERWRITE, which avoids stalling to get the pipe from the worker thread. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/buffer9.c | 62 ++ src/gallium/state_trackers/nine/buffer9.h | 7 ++-

[Mesa-dev] [PATCH 39/84] st/nine: Reimplement nine_context_apply_stateblock

2016-12-07 Thread Axel Davy
The new version uses nine_context functions instead of applying the changes directly to nine_context. This will enable it to work with CSMT. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 205 +++ 1 file changed, 205

[Mesa-dev] [PATCH 73/84] st/nine: Implement nine_context_box_upload

2016-12-07 Thread Axel Davy
This function will be used for surface and volume uploads Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 39 src/gallium/state_trackers/nine/nine_state.h | 11 2 files changed, 50 insertions(+) diff --git

[Mesa-dev] [PATCH 42/84] st/nine: Create pipe_surfaces on resource creation.

2016-12-07 Thread Axel Davy
Create the pipe_surfaces on renderable resources creation. This enables to avoid creating them on the fly. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/surface9.c | 9 + src/gallium/state_trackers/nine/surface9.h | 5 ++--- 2 files changed, 11

[Mesa-dev] [PATCH 79/84] st/nine: Idem for nine_context_gen_mipmap

2016-12-07 Thread Axel Davy
Will enable to use the bind count as an information for whether the surface/volume is used in the worker thread. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 3 ++- src/gallium/state_trackers/nine/nine_state.c | 4

[Mesa-dev] [PATCH 47/84] st/nine: Print threadid in debug log

2016-12-07 Thread Axel Davy
From: Patrick Rudolph To ease debugging. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/nine_debug.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git

[Mesa-dev] [PATCH 78/84] st/nine: Bind destination for surface/volume uploads

2016-12-07 Thread Axel Davy
Will enable to use the bind count as an information for whether the surface/volume is used in the worker thread. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 4 src/gallium/state_trackers/nine/nine_state.h | 1 +

[Mesa-dev] [PATCH 49/84] st/nine: Use nine_context for blit

2016-12-07 Thread Axel Davy
From: Patrick Rudolph Enables to offload it with CSMT. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c| 5 +++-- src/gallium/state_trackers/nine/nine_state.c | 8 src/gallium/state_trackers/nine/nine_state.h | 4

[Mesa-dev] [PATCH 80/84] st/nine: Add arguments to context's blit and copy_region

2016-12-07 Thread Axel Davy
The new arguments enable to reference the objects while the function hasn't run. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 14 -- src/gallium/state_trackers/nine/nine_state.c | 10 ++

[Mesa-dev] [PATCH 61/84] st/nine: Use nine_context_clear_render_target

2016-12-07 Thread Axel Davy
Enables to not wait for the worker thread for ColorFill. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 7 +-- src/gallium/state_trackers/nine/nine_state.c | 17 + src/gallium/state_trackers/nine/nine_state.h | 9 +

[Mesa-dev] [PATCH 83/84] st/nine: Allow non-zero resource offset for vertex buffers

2016-12-07 Thread Axel Davy
Next patches will introduce an offset. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c| 15 +-- src/gallium/state_trackers/nine/vertexbuffer9.c | 3 ++- src/gallium/state_trackers/nine/vertexbuffer9.h | 2 +- 3 files changed, 12

[Mesa-dev] [PATCH 70/84] st/nine: Optimize managed buffer upload

2016-12-07 Thread Axel Davy
Do the upload in the other thread. Usually managed buffers are used once per frame. It is then very likely pending_upload is 0 at Lock time. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/buffer9.c | 3 +++ src/gallium/state_trackers/nine/buffer9.h | 12

[Mesa-dev] [PATCH 84/84] st/nine: Implement new buffer upload path

2016-12-07 Thread Axel Davy
This new buffer upload path enables to lock faster than the normal path when using DISCARD/NOOVERWRITE. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/Makefile.sources | 2 + src/gallium/state_trackers/nine/buffer9.c | 75 +-

[Mesa-dev] [PATCH 66/84] st/nine: Detach buffers in swapchain dtor.

2016-12-07 Thread Axel Davy
BackBuffers can survive swapchain dtor if the user has a reference on them. The swapchain itself has no reference on the buffer. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/swapchain9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 31/84] st/nine: Rename pipe to pipe_data in nine_context

2016-12-07 Thread Axel Davy
This patch it to avoid name conflict when device->pipe will be moved to nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_ff.c| 20 +++ src/gallium/state_trackers/nine/nine_state.c | 90 ++--

[Mesa-dev] [PATCH 77/84] st/nine: Use nine_context_box_upload for volumes

2016-12-07 Thread Axel Davy
Use nine_context_box_upload for uploads: . systemmem volume to default volume . managed volume internal content to its resource. Check the uploads are executed before any action that can alter the data, that is LockBox and volume destruction. Signed-off-by: Axel Davy ---

[Mesa-dev] [PATCH 58/84] st/nine: use get_pipe_acquire/release when possible

2016-12-07 Thread Axel Davy
Use the acquire/release semantic when we don't need to wait for any pending command. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 8 ++-- src/gallium/state_trackers/nine/buffer9.c | 4 +++-

[Mesa-dev] [PATCH 69/84] st/nine: Implement nine_context_range_upload

2016-12-07 Thread Axel Davy
Will be used to upload buffers. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 11 +++ src/gallium/state_trackers/nine/nine_state.h | 8 2 files changed, 19 insertions(+) diff --git a/src/gallium/state_trackers/nine/nine_state.c

[Mesa-dev] [PATCH 82/84] st/nine: Do not wait for DEFAULT lock for volumes when we can

2016-12-07 Thread Axel Davy
If the volumes (and the texture container) are not referenced, then they are no pending operations on them. We can lock directly. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/volume9.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git

[Mesa-dev] [PATCH 81/84] st/nine: Do not wait for DEFAULT lock for surfaces when we can

2016-12-07 Thread Axel Davy
If the surfaces (and the texture container) are not referenced, then they are no pending operations on them. We can lock directly. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/surface9.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff

[Mesa-dev] [PATCH 74/84] st/nine: Use nine_context_box_upload for surfaces

2016-12-07 Thread Axel Davy
Use nine_context_box_upload for uploads: . systemmem surface to default surface . managed surface internal content to its resource. Check the uploads are executed before any action that can alter the data, that is LockRect, NineSurface9_CopyDefaultToMem and surface destruction. Signed-off-by:

[Mesa-dev] [PATCH 71/84] st/nine: Implement nine_context_gen_mipmap

2016-12-07 Thread Axel Davy
To offload mipmap generation as well. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 15 +++ src/gallium/state_trackers/nine/nine_state.h | 7 +++ 2 files changed, 22 insertions(+) diff --git

[Mesa-dev] [PATCH 76/84] st/nine: Fix leak with volume dtor

2016-12-07 Thread Axel Davy
The last level was not released. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/volumetexture9.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/nine/volumetexture9.c

[Mesa-dev] [PATCH 56/84] st/nine: Add secondary pipe for device

2016-12-07 Thread Axel Davy
The secondary pipe will be used for operations that don't need synchronization. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c | 4 +++- src/gallium/state_trackers/nine/device9.h | 2 ++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git

Re: [Mesa-dev] [PATCH] radeonsi: fix an off-by-one error in the bounds check for max_vertices

2016-12-07 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Tue, Dec 6, 2016 at 9:06 PM, Nicolai Hähnle wrote: > From: Nicolai Hähnle > > The spec actually says that calling EmitStreamVertex is undefined when > you exceed max_vertices. But we do need to

[Mesa-dev] [PATCH 46/84] st/nine: Implement gallium nine CSMT

2016-12-07 Thread Axel Davy
From: Patrick Rudolph Use an offloading thread for all nine_context functions. Macros are used to ease the reading of the code. Signed-off-by: Patrick Rudolph Signed-off-by: Axel Davy --- src/gallium/auxiliary/os/os_thread.h

[Mesa-dev] [PATCH 36/84] st/nine: Decompose nine_context_set_stream_source

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 34 1 file changed, 25 insertions(+),

[Mesa-dev] [PATCH 23/84] st/nine: Back ps to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 29 src/gallium/state_trackers/nine/nine_ff.c

[Mesa-dev] [PATCH 32/84] st/nine: Move pipe and cso to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 32 +++ src/gallium/state_trackers/nine/device9.h| 2 -

[Mesa-dev] [PATCH 35/84] st/nine: Do not use NineBaseTexture9 in nine_context

2016-12-07 Thread Axel Davy
Some fields are subject to modification outside of nine_context (SetLod, etc). Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.c | 21 src/gallium/state_trackers/nine/nine_ff.c | 6 +-- src/gallium/state_trackers/nine/nine_state.c |

[Mesa-dev] [PATCH 11/84] st/nine: Back vs to nine_context

2016-12-07 Thread Axel Davy
And move programmable_vs storage and computation. Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 26 +--

[Mesa-dev] [PATCH 27/84] st/nine: Move query9 pipe calls to nine_context

2016-12-07 Thread Axel Davy
This will enable to use threading for them. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/nine_state.c | 33 src/gallium/state_trackers/nine/nine_state.h | 17 ++ src/gallium/state_trackers/nine/query9.c | 32

[Mesa-dev] [PATCH 51/84] st/nine: Simplify the logic to bind textures

2016-12-07 Thread Axel Davy
This makes the code more readable. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/basetexture9.h | 20 ++ src/gallium/state_trackers/nine/device9.c | 11 +- src/gallium/state_trackers/nine/stateblock9.c | 55 -- 3 files

[Mesa-dev] [PATCH 45/84] st/nine: Pass size of memory to nine_state

2016-12-07 Thread Axel Davy
From: Patrick Rudolph Required for CSMT. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/device9.c| 20 ++-- src/gallium/state_trackers/nine/nine_state.c | 18 ++

[Mesa-dev] [PATCH 16/84] st/nine: Back scissor to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 7 +- src/gallium/state_trackers/nine/nine_state.c | 32

[Mesa-dev] [PATCH 53/84] st/nine: Track bindings for buffers

2016-12-07 Thread Axel Davy
Similar code than for textures. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/buffer9.h | 18 ++ src/gallium/state_trackers/nine/device9.c | 9 +-- src/gallium/state_trackers/nine/stateblock9.c | 34 +++ 3 files

[Mesa-dev] [PATCH 68/84] st/nine: Do not bind the container if forward is false

2016-12-07 Thread Axel Davy
This doesn't make sense to bind the container in that specific case. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/iunknown.h | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/src/gallium/state_trackers/nine/iunknown.h

[Mesa-dev] [PATCH 14/84] st/nine: Back current index buffer to nine_context

2016-12-07 Thread Axel Davy
Part of the refactor to move all gallium calls to nine_state.c, and have all internal states required for those calls in nine_context. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/device9.c| 16 ++- src/gallium/state_trackers/nine/nine_state.c |

[Mesa-dev] [PATCH 54/84] st/nine: Upload Managed buffers just before draw call using them

2016-12-07 Thread Axel Davy
Previously we were uploading Managed buffers at the next draw call after they were set dirty. This is not the expected behaviour. Instead upload just before draw call needing the content. Signed-off-by: Axel Davy --- src/gallium/state_trackers/nine/buffer9.c | 19

[Mesa-dev] [PATCH 48/84] st/nine: Add NINE_DEBUG=tid to turn threadid on or off

2016-12-07 Thread Axel Davy
From: Patrick Rudolph To ease debugging. Signed-off-by: Patrick Rudolph --- src/gallium/state_trackers/nine/nine_debug.c | 14 +- src/gallium/state_trackers/nine/nine_debug.h | 1 + 2 files changed, 10 insertions(+), 5 deletions(-) diff

Re: [Mesa-dev] [PATCH 1/3] cso: don't release sampler states that are bound

2016-12-07 Thread Roland Scheidegger
Am 07.12.2016 um 21:46 schrieb Marek Olšák: > On Wed, Dec 7, 2016 at 6:00 PM, Roland Scheidegger wrote: >> Am 07.12.2016 um 17:26 schrieb Marek Olšák: >>> Optimizing the CSO cache isn't exactly on the top of my list, so I >>> can't really do that right now. >>> >>> I think

  1   2   3   >