- Oct 16, 2019
-
-
Signed-off-by:
Andreas Baierl <ichgeh@imkreisrum.de> Reviewed-by:
Vasily Khoruzhick <anarsoul@gmail.com>
-
'struct lima_context' has to be declared before usage in lima_program.h Signed-off-by:
-
Rhys Perry authoredSigned-off-by:
Rhys Perry <pendingchaos02@gmail.com> Reviewed-by:
Daniel Schürmann <daniel@schuermann.dev>
-
Rhys Perry authored
Signed-off-by:
-
Rhys Perry authored
Signed-off-by:
-
Fixes build errors of: In file included from ../src/intel/vulkan/anv_private.h:48, from ../src/intel/vulkan/genX_blorp_exec.c:26: ../src/intel/common/gen_gem.h: In function ‘gen_ioctl’: ../src/intel/common/gen_gem.h:68:15: error: implicit declaration of function ‘ioctl’ [-Werror=implicit-function-declaration] 68 | ret = ioctl(fd, request, arg); | ^~~~~ In file included from ../include/c11/threads_posix.h:35, from ../include/c11/threads.h:66, from ../src/mesa/main/mtypes.h:39, from ../src/intel/compiler/brw_compiler.h:30, from ../src/intel/vulkan/anv_private.h:51, from ../src/intel/vulkan/genX_blorp_exec.c:26: /usr/include/unistd.h: At top level: /usr/include/unistd.h:471:12: error: conflicting types for ‘ioctl’ 471 | extern int ioctl(int, int, ...); | ^~~~~ /usr/include/unistd.h:471:1: note: a parameter list with an ellipsis can’t match an empty parameter name list declaration 471 | extern int ioctl(int, int, ...); | ^~~~~~ In file included from ../src/intel/vulkan/anv_private.h:48, from ../src/intel/vulkan/genX_blorp_exec.c:26: ../src/intel/common/gen_gem.h:68:15: note: previous implicit declaration of ‘ioctl’ was here 68 | ret = ioctl(fd, request, arg); | ^~~~~ Signed-off-by:Alan Coopersmith <alan.coopersmith@oracle.com> Acked-by:
Eric Engestrom <eric.engestrom@intel.com>
-
Signed-off-by:
Dylan Baker <dylan@pnwbakers.com>
-
Fixes: dcf9d91a ("util: Handle differences in pthread_setname_np") Signed-off-by:
-
v2: Replace autoconf check for flock() with meson check Signed-off-by:
-
gcc is very particular about where you place the (void) cast The previous placement made it error out with: In file included from disk_cache.c:40:0: ../../src/util/u_atomic.h:203:29: error: void value not ignored as it ought to be #define p_atomic_add(v, i) ((void) \ ^ disk_cache.c:658:4: note: in expansion of macro ‘p_atomic_add’ p_atomic_add(cache->size, size); ^ Signed-off-by: -
Fixes build failures on Solaris in C++ files using gcc: ../src/util/u_math.h:628:41: error: expected ‘,’ or ‘...’ before ‘dest’ 628 | util_memcpy_cpu_to_le32(void * restrict dest, const void * restrict src, size_t n) | ^~~~ ../src/util/u_math.h: In function ‘void* util_memcpy_cpu_to_le32(void*)’: ../src/util/u_math.h:641:18: error: ‘dest’ was not declared in this scope 641 | return memcpy(dest, src, n); | ^~~~ ../src/util/u_math.h:641:24: error: ‘src’ was not declared in this scope 641 | return memcpy(dest, src, n); | ^~~ ../src/util/u_math.h:641:29: error: ‘n’ was not declared in this scope; did you mean ‘yn’? 641 | return memcpy(dest, src, n); | ^ | yn Signed-off-by: -
Alyssa Rosenzweig authored
It doesn't make sense. You already spilled it once, and it didn't help. Don't try again, or you'll end up in a loop. Signed-off-by:Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
-
Alyssa Rosenzweig authored
Essentially an off-by-one error ... bit of an edge case, but seems to occur in some glamor shaders. Signed-off-by: -
Alyssa Rosenzweig authored
Fixes: ada752af ("panfrost: Extend the panfrost_batch_add_bo() API to pass access flags") Signed-off-by:
-
Alyssa Rosenzweig authored
We'll want liveness per-byte, so we need to accomodate up to 16 bytes. Signed-off-by: -
Alyssa Rosenzweig authored
The new frame throttling implemention interacts unfortunately with pipelining, leading to fence fds leaking like crazy and ultimately apps crashing quickly. With this patch, apps still crash but not as quickly. We need to either figure out the real cause or revert the core changes. Nevertheless, we don't want frame throttling in the first place, so. Fixes: a65e29cc ("gallium: simplify throttle implementation") Signed-off-by:
-
Pierre-Eric Pelloux-Prayer authored
This commit moves the target check before using _mesa_get_current_tex_object to fix a "Mesa implementation error: bad target in _mesa_get_current_tex_object()" error. Fixes: 9dd1f7ce ("mesa: pass gl_texture_object as arg to not depend on state") Reviewed-by:
Marek Olšák <marek.olsak@amd.com>
-
Marek Olšák authored
just a cleanup
-
Alyssa Rosenzweig authored
Signed-off-by: -
Alyssa Rosenzweig authored
I really need to just make this a table.. Signed-off-by: -
Alyssa Rosenzweig authored
Signed-off-by: -
Alyssa Rosenzweig authored
We don't really need to impose this condition, but we do need to cope with the slightly more general case. Signed-off-by: -
Alyssa Rosenzweig authored
Conditionals in particular read values. Signed-off-by:
-
- Oct 15, 2019
-
-
A buffer and its aux are imported separately, if the aux import is not completed yet when resource_get_param is called, merge the separate aux a.k.a the 2nd image into the main image. Fixes: 246eebba ("iris: Export and import surfaces with modifiers that have aux data") Signed-off-by:
James Xiong <james.xiong@intel.com> Reviewed-by:
Kenneth Graunke <kenneth@whitecape.org>
Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
-
Kenneth Graunke authoredOnce again, we were handling back-to-front in the GLES3 case, but not the desktop GL case. Fixes GTF-GL46.gtf30.GL3Tests.framebuffer_srgb.framebuffer_srgb_default_encoding when run with --deqp-surface-type=pbuffer --deqp-gl-context-type=egl. Reviewed-by:
Kristian H. Kristensen <hoegsberg@google.com> Reviewed-by:
-
Kenneth Graunke authored
So I can use it in fbobject.c in the next commit. Reviewed-by:
-
Kenneth Graunke authored
We were looking at ctx->DrawBuffer when asking about the read buffer, which was good enough for CTS purposes, but definitely not right. Reviewed-by:
-
Lionel Landwerlin authoredFound out by accident this was clashing with another driver. Signed-off-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com> Reviewed-by:
Christian Gmeiner <christian.gmeiner@gmail.com> Reviewed-by:
-
Marek Olšák authored
Acked-by:Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>
-
The first time you call glXMakeCurrent, current != ctx. As a result we would never look up whether the drawable already had an XMesaDrawable, and would instead always create one. Then XMesaBufferList would have two different buffers for the same XID, and you'd be reading and drawing to different places, and that's not what you want at all. Instead just always look up the drawable. Fixes: db8be355 (gallium/xlib: Remove drawable caching from the MakeCurrent path) Closes: mesa/mesa#1196 Reviewed-by:
Michel Dänzer <mdaenzer@redhat.com>
-
Eric Engestrom authored
Signed-off-by:
-
Eric Engestrom authored
Signed-off-by:
-
Eric Engestrom authored
While at it, rename to singular "container" for consistency. Signed-off-by:
-
Samuel Pitoiset authored
This reverts commit 2ca8629f. This was initially ported from RadeonSI, but in the meantime it has been reverted because it might hang. Be conservative and re-introduce this packet emission. Unfortunately this doesn't fix anything known. Cc: 19.2 <mesa-stable@lists.freedesktop.org> Signed-off-by:
Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by:
Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
-
Jonathan Marek authored
Signed-off-by:
Jonathan Marek <jonathan@marek.ca> Reviewed-by:
Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
-
Jonathan Marek authored
Signed-off-by:
-
Jonathan Marek authored
Signed-off-by:
-
Jonathan Marek authored
Signed-off-by:
-
Jonathan Marek authored
Not perfect but gets through some tests. Signed-off-by:
-
Jonathan Marek authored
Signed-off-by:
-
