Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | radv: merge radv_shader_variant_info into radv_shader_info | Samuel Pitoiset | 2019-09-06 | 1 | -61/+61 |
| | | | | | | | Having two different structs is useless. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/radeonsi: Don't count read-only data when reporting code size | Connor Abbott | 2019-09-05 | 1 | -1/+1 |
| | | | | | | | | | | We usually use these counts as a simple way to figure out if a change reduces the number of instructions or shrinks an instruction. However, since .rodata sections aren't executed, we shouldn't be counting their size for this analysis. Make the linker return the total executable size, and use it to report the more useful size in both drivers. Reviewed-by: Marek Olšák <[email protected]> | ||||
* | radv: move lowering PS inputs/outputs at the right place | Samuel Pitoiset | 2019-08-30 | 1 | -0/+3 |
| | | | | | | | At shaders creation, just after NIR linking. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: gather info about PS inputs in the shader info pass | Samuel Pitoiset | 2019-08-30 | 1 | -5/+5 |
| | | | | | | | It's the right place to do that. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv/gfx10: compute the LDS size for exporting PrimID for VS | Samuel Pitoiset | 2019-08-29 | 1 | -0/+9 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | ac: add rbplus_allowed to ac_gpu_info | Samuel Pitoiset | 2019-08-27 | 1 | -1/+1 |
| | | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> Reviewed-by: Marek Olšák <[email protected]> | ||||
* | ac: add has_gfx9_scissor_bug to ac_gpu_info | Samuel Pitoiset | 2019-08-27 | 1 | -1/+1 |
| | | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> Reviewed-by: Marek Olšák <[email protected]> | ||||
* | ac: add has_rbplus to ac_gpu_info | Samuel Pitoiset | 2019-08-27 | 1 | -3/+3 |
| | | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> Reviewed-by: Marek Olšák <[email protected]> | ||||
* | ac: add has_distributed_tess to ac_gpu_info | Samuel Pitoiset | 2019-08-27 | 1 | -2/+2 |
| | | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> Reviewed-by: Marek Olšák <[email protected]> | ||||
* | radv/gfx10: do not use NGG with NAVI14 | Samuel Pitoiset | 2019-08-23 | 1 | -0/+1 |
| | | | | | | Cc: 19.2 <[email protected]> Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv: Disable NGG for geometry shaders. | Bas Nieuwenhuizen | 2019-08-22 | 1 | -0/+20 |
| | | | | | | | | A bunch of remaining issues including some that affect users. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=111248 Fixes: ee21bd7440c "radv/gfx10: implement NGG support (VS only)" Reviewed-by: Samuel Pitoiset <[email protected]> | ||||
* | radv: Emit VGT_GS_ONCHIP_CNTL for tess on GFX10. | Bas Nieuwenhuizen | 2019-08-21 | 1 | -0/+8 |
| | | | | | | | | Otherwise hangs are possible. This register was already set for GS and NGG. Fixes: 5eaed7ecfce "radv/gfx10: enable support for NAVI10, NAVI12 and NAVI14" Reviewed-by: Samuel Pitoiset <[email protected]> | ||||
* | radv: Implement radv_GetPipelineExecutableStatisticsKHR. | Bas Nieuwenhuizen | 2019-08-12 | 1 | -0/+103 |
| | | | | Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Implement radv_GetPipelineExecutableInternalRepresentationsKHR. | Bas Nieuwenhuizen | 2019-08-12 | 1 | -5/+104 |
| | | | | Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Implement radv_GetPipelineExecutablePropertiesKHR. | Bas Nieuwenhuizen | 2019-08-12 | 1 | -0/+111 |
| | | | | Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Keep shader info when needed. | Bas Nieuwenhuizen | 2019-08-12 | 1 | -9/+17 |
| | | | | | | | This allows enabling the shader info keeping on a per shader basis. Also disables the cache on a per shader basis. Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Use string for nir dumping. | Bas Nieuwenhuizen | 2019-08-12 | 1 | -2/+1 |
| | | | | | | Reviewed-by: Dave Airlie <[email protected]> Allows us to easily dump all nir shaders for combined variants in vega and simplifies ownership. | ||||
* | radv: Hash Wave32 settings in shader key. | Bas Nieuwenhuizen | 2019-08-12 | 1 | -0/+6 |
| | | | | | | | Can result in different shaders. Fixes: 8a86908e9a7 "radv/gfx10: add Wave32 support for vertex, tessellation and geometry shaders" Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Put wave size in shader options/info. | Bas Nieuwenhuizen | 2019-08-12 | 1 | -6/+22 |
| | | | | | | | Instead of having the three values everywhere. This is also more future proof if we want the driver to make those decisions eventually. Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Avoid VEGA/RAVEN scissor bug in binning. | Bas Nieuwenhuizen | 2019-08-08 | 1 | -1/+2 |
| | | | | | CC: <[email protected]> Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Avoid binning RAVEN hangs. | Bas Nieuwenhuizen | 2019-08-08 | 1 | -1/+2 |
| | | | | | | | Mirroring radeonsi. CC: <[email protected]> Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Do non-uniform lowering before bool lowering. | Bas Nieuwenhuizen | 2019-08-06 | 1 | -1/+1 |
| | | | | | | | | Since it can introduce comparisons. Fixes: 028ce527395 "radv: Add non-uniform indexing lowering." Reviewed-by: Dave Airlie <[email protected]> Reviewed-by: Samuel Pitoiset <[email protected]> | ||||
* | radv/gfx10: add Wave32 support for vertex, tessellation and geometry shaders | Samuel Pitoiset | 2019-08-02 | 1 | -1/+9 |
| | | | | | | | It can be enabled with RADV_PERFTEST=gewave32. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: add Wave32 support for fragment shaders | Samuel Pitoiset | 2019-08-02 | 1 | -1/+2 |
| | | | | | | | It can be enabled with RADV_PERFTEST=pswave32. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: implement a GE bug workaround | Samuel Pitoiset | 2019-07-31 | 1 | -4/+23 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: remove an obsolete VGT_REUSE_OFF workaround | Samuel Pitoiset | 2019-07-31 | 1 | -6/+0 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: implement a bug workaround for GE_PC_ALLOC | Samuel Pitoiset | 2019-07-31 | 1 | -17/+0 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: add Wave32 support for compute shaders | Samuel Pitoiset | 2019-07-31 | 1 | -1/+2 |
| | | | | | | | It can be enabled with RADV_PERFTEST=cswave32. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: only compile the GS copy shader on-demand | Samuel Pitoiset | 2019-07-30 | 1 | -1/+2 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: do not enable NGG if a pipeline uses XFB | Samuel Pitoiset | 2019-07-24 | 1 | -0/+27 |
| | | | | | | | | NGG GS for streamout requires a bunch of work, so enable it with the legacy path only for now. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: Implement bin size calculation. | Bas Nieuwenhuizen | 2019-07-23 | 1 | -2/+112 |
| | | | | | Reviewed-by: Dave Airlie <[email protected]> Reviewed-by: Samuel Pitoiset <[email protected]> | ||||
* | radv/gfx9: Select between depth/color bins based on area. | Bas Nieuwenhuizen | 2019-07-23 | 1 | -2/+2 |
| | | | | | | | Mirrors radeonsi. Reviewed-by: Samuel Pitoiset <[email protected]> Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv: Generalize binning settings. | Bas Nieuwenhuizen | 2019-07-23 | 1 | -10/+2 |
| | | | | | Reviewed-by: Samuel Pitoiset <[email protected]> Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv/gfx10: Use new scan converter. | Bas Nieuwenhuizen | 2019-07-23 | 1 | -9/+51 |
| | | | | | Reviewed-by: Dave Airlie <[email protected]> Reviewed-by: Samuel Pitoiset <[email protected]> | ||||
* | radv: Set FLUSH_ON_BINNING_TRANSITION. | Bas Nieuwenhuizen | 2019-07-23 | 1 | -10/+2 |
| | | | | | Reviewed-by: Dave Airlie <[email protected]> Reviewed-by: Samuel Pitoiset <[email protected]> | ||||
* | radv/gfx10: set BREAK_WAVE_AT_EOI if TES or GS enable the primitive ID | Samuel Pitoiset | 2019-07-18 | 1 | -0/+8 |
| | | | | Signed-off-by: Samuel Pitoiset <[email protected]> | ||||
* | radv/gfx10: move emitting VGT_PRIMITIVEID_EN into the NGG path | Samuel Pitoiset | 2019-07-18 | 1 | -6/+11 |
| | | | | | | And do not emit VGT_GS_MODE which is unnecessary on GFX10. Signed-off-by: Samuel Pitoiset <[email protected]> | ||||
* | radv: fix VGT_GS_MODE if VS uses the primitive ID | Samuel Pitoiset | 2019-07-18 | 1 | -5/+5 |
| | | | | | | | | Found by inspection. Cc: <[email protected]> Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv: use correct register setter for ngg hw addr | Dave Airlie | 2019-07-18 | 1 | -1/+1 |
| | | | | | | this shouldn't matter, but it's good to be correct. Reviewed-by: Samuel Pitoiset <[email protected]> | ||||
* | radv: add an option for disabling NGG on GFX10 | Samuel Pitoiset | 2019-07-17 | 1 | -1/+4 |
| | | | | | | | Will be useful for testing the legacy path. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: implement VK_EXT_post_depth_coverage | Samuel Pitoiset | 2019-07-17 | 1 | -0/+1 |
| | | | | | | | | I did implement this extension a while ago but it didn't work on pre GFX10 for some reasons. Now all CTS pass. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: fallback to the legacy path if tess and extreme geometry | Samuel Pitoiset | 2019-07-17 | 1 | -0/+12 |
| | | | | | | | | | | This is unsupported and hangs. This fixes GPU hangs with dEQP-VK.tessellation.geometry_interaction.limits.output_required_*. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: always build the GS copy shader but uses it on-demand | Samuel Pitoiset | 2019-07-17 | 1 | -3/+18 |
| | | | | | | | | | It should be possible to build it on-demand too but it requires more work. On GFX10, the GS copy shader is required when tess is enabled with extreme geometry. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: support pixel shaders without exports | Samuel Pitoiset | 2019-07-16 | 1 | -1/+7 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Dave Airlie <[email protected]> | ||||
* | radv/gfx10: emit GE_PC_ALLOC | Samuel Pitoiset | 2019-07-12 | 1 | -0/+17 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: enable vertex shaders without export parameters | Samuel Pitoiset | 2019-07-12 | 1 | -3/+15 |
| | | | | | | | GFX10 allows this. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: launch 2 compute waves per CU before going onto the next CU | Samuel Pitoiset | 2019-07-12 | 1 | -4/+9 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv: use ac_get_compute_resource_limits() | Samuel Pitoiset | 2019-07-12 | 1 | -17/+6 |
| | | | | | | | No behaviour change. Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv/gfx10: fix wrong emission of GE_CNTL | Samuel Pitoiset | 2019-07-12 | 1 | -1/+1 |
| | | | | | Signed-off-by: Samuel Pitoiset <[email protected]> Reviewed-by: Bas Nieuwenhuizen <[email protected]> | ||||
* | radv: only enable the GS copy shader stage if GS is enabled | Samuel Pitoiset | 2019-07-11 | 1 | -1/+1 |
| | | | | | | Ooops. Signed-off-by: Samuel Pitoiset <[email protected]> |