- Sep 23, 2019
-
-
Andreas Baierl authored
lt and le is lowered by swapping the arguments. They should be lowered to their opposite ops, which means that lt -> ge and le -> gt. Reorder the enum to fix this. Signed-off-by:Andreas Baierl <ichgeh@imkreisrum.de>
-
Daniel Schürmann authored
Reviewed-by:Rhys Perry <pendingchaos02@gmail.com>
-
Karol Herbst authoredSigned-off-by:
Karol Herbst <kherbst@redhat.com>
Reviewed-by: Rhys Kidd <rhyskidd@gmail.com>
-
Karol Herbst authored
Signed-off-by:
Karol Herbst <kherbst@redhat.com>
Reviewed-by: -
Erico Nunes authored
pipe->clear() is not called for partial clears, which mesa emulates by drawing a quad. Furthermore, drivers should not use rasterizer state information for scissor information (which was being used to handle the partial clears). So, remove the partial clear support since it was not supposed to be handled by pipe->clear() anyway. This fixes issues with clearing after switching to different sized framebuffers. Signed-off-by:
Erico Nunes <nunes.erico@gmail.com> Reviewed-by:
Vasily Khoruzhick <anarsoul@gmail.com> Reviewed-by:
Qiang Yu <yuq825@gmail.com>
-
Boris Brezillon authored
Signed-off-by:Boris Brezillon <boris.brezillon@collabora.com>
-
Boris Brezillon authored
->padded_count should be large enough to cover all vertices pointed by the index array. Use the local vertex_count variable that contains the updated vertex_count value for the indexed draw case. Signed-off-by:
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>
-
fixes "sorry, unimplemented: non-trivial designated initializers not supported" Fixes: deb04adf ("clover: add support for passing kernels as nir to the driver") Signed-off-by:
Karol Herbst <kherbst@redhat.com>
Reviewed-by: Eric Engestrom <eric@engestrom.ch>
-
Kenneth Graunke authoredIncomplete attachments don't have an associated pipe_surface, so this would crash. Fixes a WebGL conformance test that uses incomplete attachments: https://www.khronos.org/registry/webgl/sdk/tests/conformance2/renderbuffers/invalidate-framebuffer.html?webglVersion=2&quiet=0&quick=1 Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=111756 Reviewed-By:
Tapani Pälli <tapani.palli@intel.com>
-
Vasily Khoruzhick authored
Allocating BOs is expensive, so we should avoid doing that by caching freed BOs. BO cache is modelled after one in v3d driver and works as follows: - in lima_bo_create() check if we have matching BO in cache and return it if there's one, allocate new BO otherwise. - in lima_bo_unreference() (renamed from lima_bo_free()): put BO in cache instead of freeing it and remove all stale BOs from cache Reviewed-by:
-
Vasily Khoruzhick authored
os_time_get_absolute_timeout(0) returns current time, while kernel driver expects 0 as value to poll BO status and return immediately. Fix it by setting abs_timeout to 0 if timeout_ns is 0 Reviewed-by:
-
Qiang Yu authored
Reviewed-and-Tested-by:
-
Qiang Yu authored
Some time weston set full damage region. It is more effient to use the cached pp stream instead of dynamically create one. Reviewed-and-Tested-by:
-
Qiang Yu authored
This extension set a damage region for each buffer swap which can be used to reduce buffer reload cost by only feed damage region's tile buffer address for PP. Reviewed-and-Tested-by:
-
- Sep 22, 2019
-
-
Icenowy Zheng authoredThe PLBU expects the viewport's 4 borders' coordinates, however currently we're feeding the coordinate of the left-bottom point and the size to it, which leads to misrendering when the left-bottom point is not (0,0). Change the macros for the viewport PLBU command, and the data feed to it. The code to calculate the 4 borders is ported from Panfrost. Signed-off-by:
Icenowy Zheng <icenowy@aosc.io> Reviewed-by:
-
- Sep 21, 2019
-
-
Bas Nieuwenhuizen authored
ACO depends on C++14, but radeonsi/radv with LLVM 8,9 do not. Let us only require it for RADV, since that is the only user. Fixes: a70a9987 "radv/aco: Setup alternate path in RADV to support the experimental ACO compiler" Reviewed-by:
Marek Olšák <marek.olsak@amd.com>
-
required for OpenCL v2: adjust to changes in previous commits v3: properly convert to NIR in nvc0_cp_state_create Signed-off-by:Karol Herbst <kherbst@redhat.com>
Reviewed-by: Pierre Moreau <pierre.morrow@free.fr> (v1) -
v2: minor formatting fixes v3: call glsl_type_singleton_init_or_ref and glsl_type_singleton_decref v4: capitalize and punctuate comments fix text_executable -> text_intermediate in TODO make glsl_type_singleton wrapper static v5: rewrite how we run the nir passes v6: fix unhandled case switch warning in st/mesa Signed-off-by:Karol Herbst <kherbst@redhat.com>
Reviewed-by: Francisco Jerez <currojerez@riseup.net> (v4) -
v2: rework arguments to compiler::compile_program add assert to device::ir_format v3: remove PIPE_SHADER_IR_SPIRV change title Signed-off-by:Karol Herbst <kherbst@redhat.com>
Reviewed-by: Francisco Jerez <currojerez@riseup.net> (v2)
Reviewed-by: Pierre Moreau <pierre.morrow@free.fr>
-
Most drivers have actually no binary format and just store the IR directly as a single entry point blob. v2: add a cap to switch between single or multi entry point binaries v3: remove the entry_point field v4: remove PIPE_CAP_MULTI_ENTRY_POINT_BINARIES v5: remove supports_multiple_entry_points Signed-off-by:
Karol Herbst <kherbst@redhat.com>
Reviewed-by: -
v2: pass argument by value Signed-off-by:
Karol Herbst <kherbst@redhat.com>
Reviewed-by: Francisco Jerez <currojerez@riseup.net> Reviewed-by:
-
We want to use it for other formats as well, so give it a more generic name Signed-off-by:
Karol Herbst <kherbst@redhat.com>
Reviewed-by: -
makes it easier to consume a IR_NATIVE binary Signed-off-by:
Karol Herbst <kherbst@redhat.com>
Reviewed-by: -
Reviewed-by:
Karol Herbst <kherbst@redhat.com>
Reviewed-by: -
Reviewed-by:
Karol Herbst <kherbst@redhat.com>
Acked-by: -
v2 (Karol Herbst): silence warnings about unhandled enum values v3 (Karol Herbst): added back array size parsing (needed for structs passed by value) Acked-by: Francisco Jerez <currojerez@riseup.net> (v2)
-
Changes since: * v12: - remove autotools (Karol Herbst) - Remove the callback in format_validation_msg. (Francisco Jerez) - Removed is_binary_spirv. (Francisco Jerez) - Pass a string reference to is_valid_spirv instead of the notification callback. (Francisco Jerez) * v11: Fix compilation error introduced in v11. * v10: - Reuse format_validation_msg in is_valid_spirv. - Remove LVL2STR macro in format_validation_msg. * v9: Add `clover_cpp_std` to the overrides of the `libclspirv` target in Meson. * v7: Add DEFINES to libclspirv and libclover, in autotools, as they would otherwise never know whether CLOVER_ALLOW_SPIRV has been defined (Dave Airlie) * v6: Update the dependency name (meson) and the libs variable (Makefile) due to the replacement of llvm-spirv to the new official SPIRV-LLVM-Translator. * v5: Changed to match the updated “clover/llvm: Allow translating from SPIR-V to LLVM IR” in the v6. Reviewed-by:Karol Herbst <kherbst@redhat.com>
Reviewed-by: -
Changes since: * v12 (Karol Herbst): - rename CLOVER_ALLOW_SPIRV to HAVE_CLOVER_SPIRV * v11 (Karol Herbst): - only set new defines for clover to speed up recompilation - remove autotools * v10: - Add a new flag (`--enable-opencl-spirv` for autotools, and `-Dopencl-spirv=true` for meson) for enabling SPIR-V support in clover, and never automagically enable it without that flag. (Dylan Baker) - When enabling the SPIR-V support, the SPIRV-Tools and SPIRV-LLVM-Translator libraries are now required dependencies. * v7: - Properly align LLVMSPIRVLib comment (Dylan Baker) - Only define CLOVER_ALLOW_SPIRV when **both** dependencies are found: autotools was only requiring one or the other. * v6: Replace the llvm-spirv repository by the new official SPIRV-LLVM-Translator. * v4: Add a comment saying where to find llvm-spirv (Karol Herbst). * v3: - make SPIRV-Tools and llvm-spirv optional (Francisco Jerez); - bump requirement for llvm-spirv to version 0.2 * v2: - Bump the required version of SPIRV-Tools to the latest release; - Add a dependency on llvm-spirv. Reviewed-by: Dylan Baker <dylan@pnwbakers.com> (v10) Reviewed-by:Karol Herbst <kherbst@redhat.com>
Reviewed-by:
-
- Sep 20, 2019
-
-
Kenneth Graunke authored
Gen11 doesn't require us to bypass the L2 cache for BC* images anymore. The documentation is a bit hard to follow on this point, but the Windows driver clearly only applies this workaround on Gen9, and their commit history indicates that this was an intentional change to drop the workaround for Gen11+. Reviewed-by:Jason Ekstrand <jason@jlekstrand.net>
-
Freya Gentz authoredCurrently there is no way to make no context current w/gallium + osmesa. The non-gallium version of osmesa does this if the context and buffer passed to `OSMesaMakeCurrent` are both null. This small change makes it so that this is also the case with the gallium version. Cc: mesa-stable@lists.freedesktop.org Signed-off-by:
Hal Gentz <zegentzy@protonmail.com> Reviewed-by:
Eric Anholt <eric@anholt.net>
-
-
Reviewed-by:Kevin Strasser <kevin.strasser@intel.com>
-
Reviewed-by: -
Reviewed-by: -
No idea how these ended up with 3-then-2-space indents. Reviewed-by: -
Faith Ekstrand authoredReviewed-by:Paulo Zanoni <paulo.r.zanoni@intel.com>
-
Faith Ekstrand authored
Reviewed-by: -
Faith Ekstrand authored
We can't really handle it in the little-core 64-bit case but it's not really needed there. Where we really want this is for when we need to do 16 -> 8-bit conversions. Reviewed-by: -
Faith Ekstrand authored
Because byte immediates aren't a thing on GEN hardware, we return a signed or unsigned word immediate in the byte case. Reviewed-by: -
Paulo Zanoni authoredDuring generate_shuffle(), when we use byte sized registers we end up with a destination stride of 2. We don't take the stride into consideration when selecting the group offset for the last MOV operation, which means we end up moving things to the wrong place, leaving the last few channels untouched. Take the destination stride in consideration so we don't miss the last channels. v2: Assert this is not necessary for the IVB special case (Jason). Reviewed-by:
-
