From e345b03c89486d803b2e4a9ccf3cc0d04c389580 Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Fri, 16 May 2025 13:10:29 +0200 Subject: [PATCH] ac/gpu_info: rename has_zerovram_support to has_default_zerovram_support To avoid confusion between RADEON_FLAG_ZERO_VRAM and default VRAM allocation clears since AMDGPU 3.59+. Signed-off-by: Samuel Pitoiset Part-of: --- src/amd/common/ac_gpu_info.c | 5 +++-- src/amd/common/ac_gpu_info.h | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/amd/common/ac_gpu_info.c b/src/amd/common/ac_gpu_info.c index 34ed02b1b07..c5343371ac4 100644 --- a/src/amd/common/ac_gpu_info.c +++ b/src/amd/common/ac_gpu_info.c @@ -1447,7 +1447,8 @@ ac_query_gpu_info(int fd, void *dev_p, struct radeon_info *info, info->has_stable_pstate = info->drm_minor >= 45; - info->has_zerovram_support = info->drm_minor >= 59; + /* AMDGPU 3.59+ clears VRAM on allocations by default. */ + info->has_default_zerovram_support = info->drm_minor >= 59; if (info->gfx_level >= GFX12) { /* Gfx12 doesn't use pc_lines and pbb_max_alloc_count. */ @@ -2099,7 +2100,7 @@ void ac_print_gpu_info(const struct radeon_info *info, FILE *f) info->fw_based_mcbp.csa_alignment); } - fprintf(f, " has_zerovram_support = %u\n", info->has_zerovram_support); + fprintf(f, " has_default_zerovram_support = %u\n", info->has_default_zerovram_support); fprintf(f, " has_tmz_support = %u\n", info->has_tmz_support); fprintf(f, " has_trap_handler_support = %u\n", info->has_trap_handler_support); for (unsigned i = 0; i < AMD_NUM_IP_TYPES; i++) { diff --git a/src/amd/common/ac_gpu_info.h b/src/amd/common/ac_gpu_info.h index 06a2228bf6a..fa6c0c02e30 100644 --- a/src/amd/common/ac_gpu_info.h +++ b/src/amd/common/ac_gpu_info.h @@ -241,7 +241,7 @@ struct radeon_info { bool has_stable_pstate; /* Whether SR-IOV is enabled or amdgpu.mcbp=1 was set on the kernel command line. */ bool register_shadowing_required; - bool has_zerovram_support; + bool has_default_zerovram_support; bool has_tmz_support; bool has_trap_handler_support; bool kernel_has_modifiers;