- 01 Dec, 2020 40 commits
-
-
Erik Faye-Lund authoredWe can risk having NULL samplers in this range, so let's ignore them while we're populating the shader-key. Reviewed-by:
Jesse Natalie <jenatali@microsoft.com> Part-of: <!7856>
-
Timur Kristóf authoredThere was a temp which was defined in a branch but used outside, without a phi. Fixes: 62b5012e Signed-off-by:
Timur Kristóf <timur.kristof@gmail.com>
Reviewed-by: 
Rhys Perry <pendingchaos02@gmail.com>
Part-of: <!7817> -
Marcin Ślusarz authoredSigned-off-by:
Marcin Ślusarz <marcin.slusarz@intel.com>
Reviewed-by: 
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <!7833> -
Marcin Ślusarz authored
Signed-off-by:
Marcin Ślusarz <marcin.slusarz@intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <!7833> -
Marcin Ślusarz authored
Signed-off-by:
Marcin Ślusarz <marcin.slusarz@intel.com>
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <!7833> -
Erik Faye-Lund authored
This wasn't implemented yet, because we hadn't encountered it yet. But now it seems we can trigger this, thanks to the nv_copy_depth_to_color piglit tests. This makes the test go from crash to fail, which isn't perfect, but it's better than nothing. Reviewed-by:
-
Samuel Pitoiset authored
It introduces regressions. This reverts commit 6fb4babf. Part-of: <!7852>
-
Samuel Pitoiset authored
Otherwise the CI sanity check fails if the commit title exceed 80 characters Signed-off-by:
Samuel Pitoiset <samuel.pitoiset@gmail.com> Part-of: <!7852>
-
Christian Gmeiner authored
Signed-off-by:
Christian Gmeiner <christian.gmeiner@gmail.com> Reviewed-by:
Lucas Stach <l.stach@pengutronix.de> Part-of: <!7832>
-
Christian Gmeiner authored
We do not need a full blown pthread mutex. Signed-off-by:
-
Christian Gmeiner authored
Cc: <mesa-stable@lists.freedesktop.org> Signed-off-by:
-
Iago Toral authored
Reviewed-by:
Alejandro Piñeiro <apinheiro@igalia.com>
Part-of: <!7848> -
Iago Toral authored
No point in computing this if debug is disabled. Reviewed-by:
Alejandro Piñeiro <apinheiro@igalia.com>
Part-of: <!7848> -
Marek Olšák authored
to merge both arrays and remove malloc because both arrays have the same size. Reviewed-by:
Zoltán Böszörményi <zboszor@gmail.com> Part-of: <!6946>
-
Marek Olšák authored
The rework of matrix state vars made the last element unused. Reviewed-by:
Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> Part-of: <!6946>
-
Marek Olšák authored
Fixes: 584f2732 - st/mesa: factor ucp-lowering logic into helper Reviewed-by:
-
Marek Olšák authored
Reviewed-by:
-
Marek Olšák authored
I don't see why this is needed and it's not used anywhere. As long as apps don't call glDeleteTextures, nothing will release them. And even if they do, we don't use the saved textures anywhere. Also, BindTexture will fail for deleted textures anyway, so they can't be popped. The existing code already binds the Name that was saved, not the texture object that was saved. Reviewed-by:
-
Marek Olšák authored
Acked-by:
Adam Jackson <ajax@redhat.com> Acked-by:
-
Marek Olšák authored
Reviewed-by:
-
Marek Olšák authored
Reviewed-by:
-
Marek Olšák authored
Let's get rid of the big memcpy. Acked-by:
-
Marek Olšák authored
Reviewed-by:
-
Marek Olšák authored
Put the fields saved by glPush/PopAttrib into the sub-structure declared as Attrib. This will make glPush/PopAttrib much faster because it will only save and restore that structure. Acked-by:
-
Marek Olšák authored
Acked-by:
-
Marek Olšák authored
Acked-by:
-
Marek Olšák authored
This was inevitable. This gets rid of the malloc/free calls and chaining each push group in a linked list. Now the whole attribute stack is declared statically in gl_context. Yes, this improves performance a lot. Acked-by:
-
Marek Olšák authored
Reviewed-by:
-
Marek Olšák authored
Reviewed-by:
-
Marek Olšák authored
If there are no changes to the matrix, we can move pop the stack but not update any state constants. Reviewed-by:
-
Marek Olšák authored
It doesn't change any states. Reviewed-by:
-
Marek Olšák authored
This happens a lot with viewperf and it causes unnecessary constant buffer updates. Reviewed-by:
-
Marek Olšák authored
I copied error conditions from validate_uniform_parameters. Reviewed-by:
-
Marek Olšák authored
Acked-by:
-
Marek Olšák authored
The old path copies state parameters into the parameter list, and then the driver copies them into a buffer. The optional new path loads state parameters into a buffer directly. This increases performance by 5% in one subtest of viewperf. Acked-by:
-
Marek Olšák authored
Acked-by:
-
Marek Olšák authored
Drivers should upload only UniformBytes of uniforms and constants, and then use _mesa_upload_state_parameters to upload state parameters. This allows removing one copy of state parameters. Acked-by:
-
Marek Olšák authored
Disabled because of CI failures. Invoke fetch_state only once for all lights in the best case instead of 8*8 times. Acked-by:
-
Marek Olšák authored
Disabled because of CI failures. Instead of separate state vars for each row and invoking fetch_state 4x: state.matrix.modelview.row[0] state.matrix.modelview.row[1] state.matrix.modelview.row[2] state.matrix.modelview.row[3] The rows are now merged and fetch_state is invoked once: state.matrix.modelview.row[0..3] Acked-by:
-
Marek Olšák authored
Acked-by:
-
