summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* softpipe: tie in new code path for lod evaluationGert Wollny2019-05-012-46/+21
| | | | | | | | | This enables the use of explicit gradients. Also remove an unused parameter when changing the interfaces. Signed-off-by: Gert Wollny <[email protected]> Reviewed-by: Roland Scheidegger <[email protected]>
* softpipe: Move selection of shadow values up and clean parameter listGert Wollny2019-05-011-29/+43
| | | | | | | | | The shadow evaluation compare parameter is stored in different locations, depending on the texture type. Move the values to a common location free the lod storage and to be able to reduce the number of parameters. Signed-off-by: Gert Wollny <[email protected]> Reviewed-by: Roland Scheidegger <[email protected]>
* softpipe: Pipe gather_comp through from st_tgsi_get_samplesGert Wollny2019-05-012-5/+18
| | | | | | | | The value is stored in the lod components and this will be overwritten when swithcing to the new code path. Signed-off-by: Gert Wollny <[email protected]> Reviewed-by: Roland Scheidegger <[email protected]>
* softpipe: Prepare handling explicit gradientsGert Wollny2019-05-013-1/+42
| | | | | | | This only adds corde that is not yet enabled. Signed-off-by: Gert Wollny <[email protected]> Reviewed-by: Roland Scheidegger <[email protected]>
* softpipe: Factor gradient evaluation out of the lambda evaluationGert Wollny2019-05-011-19/+119
| | | | | | | this is useful when we want to use explicit gradients. Signed-off-by: Gert Wollny <[email protected]> Reviewed-by: Roland Scheidegger <[email protected]>
* egl: return correct error code for a case req ver < 3 with forward-compatibleAndrii Simiklit2019-05-011-1/+4
| | | | | | | | | | | | | | | | | | | | The EGL_KHR_create_context spec says: "If an OpenGL context is requested and the values for attributes EGL_CONTEXT_MAJOR_VERSION_KHR and EGL_CONTEXT_MINOR_VERSION_KHR, when considered together with the value for attribute EGL_CONTEXT_OPENGL_FORWARD_COMPATIBLE_BIT_KHR, specify an OpenGL version and feature set that are not defined, than an EGL_BAD_MATCH error is generated." This case is already correctly handled a bit below in the same source file. The correct handling was added by commit: 63beb3df Reported-by: Ian Romanick <[email protected]> Here: https://bugzilla.freedesktop.org/show_bug.cgi?id=92552#c9 Fixes: 11cabc45b712 "egl: rework handling EGL_CONTEXT_FLAGS" Reviewed-by: Ian Romanick <[email protected]> Signed-off-by: Andrii Simiklit <[email protected]>
* radeonsi/nir: call radeonsi nir opts before the scan passTimothy Arceri2019-05-012-0/+2
| | | | | | | | | | | | | | Some of the opts are not called in the general optimastion loop in the state trackers glsl -> nir conversion. We need to call the radeonsi specific optimisation once before scanning over the nir otherwise we can end up gathering info on code that is later removed. Fixes an assert in the piglit test: ./bin/varying-struct-centroid_gles3 Reviewed-by: Marek Olšák <[email protected]>
* radeonsi/nir: create si_nir_opts() helperTimothy Arceri2019-05-012-36/+43
| | | | | | We will make use of this in the following commit. Reviewed-by: Marek Olšák <[email protected]>
* swr/rast: early exit on empty triangle maskAlok Hota2019-04-301-0/+6
| | | | Reviewed-by: Bruce Cherniak <[email protected]>
* swr/rast: add guards for cpuid on LinuxAlok Hota2019-04-301-0/+2
| | | | Reviewed-by: Bruce Cherniak <[email protected]>
* swr/rast: add flat shadingAlok Hota2019-04-301-0/+13
| | | | Reviewed-by: Bruce Cherniak <[email protected]>
* swr/rast: add SWR_STATIC_ASSERT() macroAlok Hota2019-04-301-1/+8
| | | | Reviewed-by: Bruce Cherniak <[email protected]>
* swr/rast: update guardband rects at draw setupAlok Hota2019-04-301-2/+3
| | | | | | It's dependent on other state fields Reviewed-by: Bruce Cherniak <[email protected]>
* swr/rast: add more llvm intrinsicsAlok Hota2019-04-301-0/+3
| | | | Reviewed-by: Bruce Cherniak <[email protected]>
* st/va: properly set stride and offset in vlVaDeriveImageJulien Isorce2019-04-301-10/+25
| | | | | | | | Using the new resource_get_info function. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110443 Signed-off-by: Julien Isorce <[email protected]> Reviewed-by: Leo Liu <[email protected]>
* radeonsi: implement resource_get_infoJulien Isorce2019-04-301-8/+35
| | | | | | | | Re-use existing si_texture_get_offset. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110443 Signed-off-by: Julien Isorce <[email protected]> Reviewed-by: Marek Olšák <[email protected]>
* gallium: add resource_get_info to pipe_screenJulien Isorce2019-04-304-0/+51
| | | | | | | | Generic plumbing. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=110443 Signed-off-by: Julien Isorce <[email protected]> Reviewed-by: Marek Olšák <[email protected]>
* freedreno/ir3: fixes for half reg in/outRob Clark2019-04-304-13/+39
| | | | | | | Needs to update max_half_reg, or be remapped to full reg and update max_reg accordingly, depending on generation.. Signed-off-by: Rob Clark <[email protected]>
* st/nine: Check discard_delayed_release is set before allocating moreAxel Davy2019-04-301-0/+2
| | | | | | | | | | When discard_delayed_release is set (default), we allocate more buffers and use a different buffer wait path. Check if it is set, and use the old paths if not (the alternative buffer wait path could still be used, but there is no advantage to using it in this case). Signed-off-by: Axel Davy <[email protected]>
* st/nine: Throttle rendering similarly for thread_submitAxel Davy2019-04-302-22/+22
| | | | | | | | | thread_submit's throttling depending on the number of internal back buffers, and wasn't affected by the driver requested throttling value. Now it is. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Optimize a bit writeonly buffersAxel Davy2019-04-301-1/+6
| | | | | | | | | | | Optimize writeonly by passing PIPE_TRANSFER_WRITE for these buffers instead of the safer PIPE_TRANSFER_READ_WRITE. This seems to improve the performance of d3d8 games using d3d8to9. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Use TGSI_SEMANTIC_GENERIC for fogAxel Davy2019-04-302-28/+29
| | | | | | | | | | | | | | We used TGSI_SEMANTIC_FOG for fog, however on vs/ps 3, fog is allowed to have 4 components (even on the ff pipeline according to a wine test). Since gallium's TGSI_SEMANTIC_FOG has only one component, use TGSI_SEMANTIC_GENERIC instead. Fixes: https://github.com/iXit/Mesa-3D/issues/346 Signed-off-by: Axel Davy <[email protected]>
* st/nine: Enable computing const_rangesAxel Davy2019-04-301-1/+1
| | | | | | | All the pieces for constant compact are ready, thus enable the path. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Handle const_ranges in nine_stateAxel Davy2019-04-301-2/+26
| | | | | | Handle slot mapping if there is one. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Cache constant buffer sizeAxel Davy2019-04-307-23/+52
| | | | | | | | | | | | The shader constant buffer size with the constant compaction code can vary depending on the shader variant compiled (for example if fog constants are required, etc). Thus instead of using fixed size for the shader, add in the variant cache the size required, pass it to the context, and use this value. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Propagate const_range to contextAxel Davy2019-04-307-15/+37
| | | | | | | | | As with the constant compaction we map the constant slots to new slots, we need to pass that information to the context which is in charge of uploading the constants. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Prepare constant compaction in nine_shaderAxel Davy2019-04-302-28/+118
| | | | | | | | | | | | | | When indirect addressing is not used, we know exactly which constants are accessed, and thus can have them located in consecutive slots. We thus parse again the shader with a slot map for compaction. The path contains the work inside nine_shader.c for this path, but it needs some other commits to work, and thus is not enabled yet by this commit. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Refactor counting of constantsAxel Davy2019-04-301-25/+21
| | | | | | Track the number of slots used Signed-off-by: Axel Davy <[email protected]>
* st/nine: Track constant slots usedAxel Davy2019-04-301-2/+11
| | | | | | This tracking will be useful for constant compaction Signed-off-by: Axel Davy <[email protected]>
* st/nine: Refactor ct_ctorAxel Davy2019-04-301-89/+95
| | | | | | | The refactoring will make it easier to parse the shader twice for the constant compaction path. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Make swvp_on imply IS_VSAxel Davy2019-04-302-6/+10
| | | | | | | swvp cannot happen with ps, thus it makes sense to force it to false with ps. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Refactor shader constants ureg_src computationAxel Davy2019-04-302-95/+99
| | | | | | | Put the shader constant code in one place to better change that code in future commits. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Manually upload vs and ps constantsAxel Davy2019-04-301-0/+46
| | | | | | | In future commits we will introduce more fine-grained uploads Signed-off-by: Axel Davy <[email protected]>
* st/nine: use helper ureg_DECL_sampler everywhereAxel Davy2019-04-301-1/+1
| | | | Signed-off-by: Axel Davy <[email protected]>
* st/nine: Compact pixel shader keyAxel Davy2019-04-302-15/+31
| | | | | | | Compact the shader key to make room for new elements. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Compact nine_ff_get_projected_keyAxel Davy2019-04-304-8/+24
| | | | | | | | Only the first four sampler slots can be used by ff ps < 0x14, thus the size of the key can be reduced. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Refactor param->relAxel Davy2019-04-301-4/+6
| | | | | | | Refactor param->rel to enable different paths for constants and inputs relative addressing. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Regroup param->rel testsAxel Davy2019-04-301-8/+3
| | | | | | | | | Regroup all the param->rel assertions into one assertion for better clarity and better covering. param->rel on an input can only happen with float constants for vs, or with inputs on vs/ps 3.0. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Control shader constant inlining with drircAxel Davy2019-04-304-1/+14
| | | | | | | Until we use async shader compilation for constant inlining, don't enable it unless user asks for it. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Recompile optimized shaders based on b/i constsAxel Davy2019-04-309-5/+211
| | | | | | | | | | | | | | | | | | | | | | | | | Boolean and Integer constants are used in d3d9 for flow control. Boolean are used for if/then/else and Integer constants for loops. The compilers can generate better code if these values are known at compilation. I haven't met so far a game that would change the values of these constants frequently (and when they do, they set to the values used for the previous draw call, and thus the changes get filtered out). Thus it makes sense to inline these constants and recompile the shaders. The commit sets a bound to the number of variants for a given shader to avoid too many shaders to be generated. One drawback is it means more shader compilations. It would probably make sense to compile these shaders asynchronously or let the user control the behaviour with an env var, but this is not done here. The games I tested hit very few shader variants, and the performance impact was negligible, but it could help for games with uber shaders. Signed-off-by: Axel Davy <[email protected]>
* drirc: Add Gallium nine workaround for Rayman LegendsAxel Davy2019-04-301-0/+5
| | | | | | The game requires it to display many textures properly. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Add drirc option to use data_internal for dynamic texturesAxel Davy2019-04-307-2/+28
| | | | | | | | | | | | | | dynamic textures seem to have predictable stride. This stride should be the same as for a ram buffer. It seems some game don't check the actual stride value, assuming it to be the expected one. Thus this workaround (protected by drirc option) is to use an intermediate ram buffer. Fixes Rayman Legends texture issues when enabled. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Support internal compressed format for volumesAxel Davy2019-04-301-13/+16
| | | | | | Reuse the generic path to support compressed formats. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Support internal compressed format for surfacesAxel Davy2019-04-301-12/+15
| | | | | | Reuse the generic path to support compressed formats. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Refactor volume GetSystemMemPointerAxel Davy2019-04-301-8/+11
| | | | | | It will make it easier to reuse in another place. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Refactor surface GetSystemMemPointerAxel Davy2019-04-301-10/+12
| | | | | | It will make it easier to reuse in another place. Signed-off-by: Axel Davy <[email protected]>
* st/nine: rename *_conversion to *_internalAxel Davy2019-04-304-59/+59
| | | | | | | Rename these variables to a new name which will fit new usages introduced in later commits. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Optimize volume upload with conversionAxel Davy2019-04-301-27/+25
| | | | | | | | | Use nine_context_box_upload instead of locking the pipe for volume upload with format conversion. nine_context_box_upload already handles format conversion. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Optimize surface upload with conversionAxel Davy2019-04-301-28/+27
| | | | | | | | | Use nine_context_box_upload instead of locking the pipe for surface upload with format conversion. nine_context_box_upload already handles format conversion. Signed-off-by: Axel Davy <[email protected]>
* st/nine: Fix SINCOS inputAxel Davy2019-04-301-2/+7
| | | | | | | | | | | | | SINCOS takes an input with replicated swizzle. the swizzle can be on any component, not just x. Enable it to read from any component, but also use a temporary register to avoid dst/src aliasing. No known game is fixed by this change as it seems the input swizzle is commonly on x for this instruction, and src and dst don't alias. Signed-off-by: Axel Davy <[email protected]>