On Wed, Jun 20, 2018 at 11:26 AM, Dylan Baker <dy...@pnwbakers.com> wrote:
> Quoting Marek Olšák (2018-06-08 20:16:54)
>> From: Marek Olšák <marek.ol...@amd.com>
>>
>> This improves performance for certain games.
>>
>> Cc: 18.1 <mesa-sta...@lists.freedesktop.org>
>> ---
>> src/gallium/drivers/radeonsi/si_buffer.c | 6 +++++-
>> 1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/src/gallium/drivers/radeonsi/si_buffer.c
>> b/src/gallium/drivers/radeonsi/si_buffer.c
>> index 2d68edc3404..0546fa9d336 100644
>> --- a/src/gallium/drivers/radeonsi/si_buffer.c
>> +++ b/src/gallium/drivers/radeonsi/si_buffer.c
>> @@ -144,22 +144,26 @@ void si_init_resource_fields(struct si_screen *sscreen,
>> if (res->b.b.target == PIPE_BUFFER &&
>> res->b.b.flags & (PIPE_RESOURCE_FLAG_MAP_PERSISTENT |
>> PIPE_RESOURCE_FLAG_MAP_COHERENT)) {
>> /* Use GTT for all persistent mappings with older
>> * kernels, because they didn't always flush the HDP
>> * cache before CS execution.
>> *
>> * Write-combined CPU mappings are fine, the kernel
>> * ensures all CPU writes finish before the GPU
>> * executes a command stream.
>> + *
>> + * radeon doesn't have good BO move throttling, so put all
>> + * persistent buffers into GTT to prevent VRAM CPU page
>> faults.
>> */
>> - if (!sscreen->info.kernel_flushes_hdp_before_ib)
>> + if (!sscreen->info.kernel_flushes_hdp_before_ib ||
>> + sscreen->info.drm_major == 2)
>> res->domains = RADEON_DOMAIN_GTT;
>> }
>>
>> /* Tiled textures are unmappable. Always put them in VRAM. */
>> if ((res->b.b.target != PIPE_BUFFER && !rtex->surface.is_linear) ||
>> res->b.b.flags & SI_RESOURCE_FLAG_UNMAPPABLE) {
>> res->domains = RADEON_DOMAIN_VRAM;
>> res->flags |= RADEON_FLAG_NO_CPU_ACCESS |
>> RADEON_FLAG_GTT_WC;
>> }
>> --
>> 2.17.1
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
> Hi Marek,
>
> This doesn't apply cleanly to 18.1, since
> b81149e258a492ed0c81058fb535f6bfdacb36da isn't in 18.1; would you like me to
> pull that commit as well, or do something else?
Yes, you can pull that commit too. Thanks!
Marek
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
