From fa1fd2413f5a8a24959664e284372135b3deac83 Mon Sep 17 00:00:00 2001 From: Benjamin Cheng Date: Fri, 28 Nov 2025 13:46:44 -0500 Subject: [PATCH] radv/video: Fix force_integer_mv=1 on intra frame According to AV1 spec, force_integer_mv=1 on intra frames. However, VCN FW does not expect integer mv to be set unless screen content tools are enabled. This also aligns the code to the radeonsi logic. Cc: mesa-stable Reviewed-by: David Rosca Part-of: --- src/amd/vulkan/radv_video_enc.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/amd/vulkan/radv_video_enc.c b/src/amd/vulkan/radv_video_enc.c index 032a2cee86d..ee761f61b02 100644 --- a/src/amd/vulkan/radv_video_enc.c +++ b/src/amd/vulkan/radv_video_enc.c @@ -840,12 +840,11 @@ radv_enc_spec_misc_av1(struct radv_cmd_buffer *cmd_buffer, const struct VkVideoE tile_config->tile_groups[0].end = tile_config->num_tile_cols * tile_config->num_tile_rows - 1; tile_config->tile_size_bytes_minus_1 = 3; - uint32_t precision = RENCODE_AV1_MV_PRECISION_ALLOW_HIGH_PRECISION; - - if (!pic->flags.allow_high_precision_mv) - precision = RENCODE_AV1_MV_PRECISION_DISALLOW_HIGH_PRECISION; - if (pic->flags.force_integer_mv) + uint32_t precision = RENCODE_AV1_MV_PRECISION_DISALLOW_HIGH_PRECISION; + if (pic->flags.allow_screen_content_tools && pic->flags.force_integer_mv) precision = RENCODE_AV1_MV_PRECISION_FORCE_INTEGER_MV; + else if (pic->flags.allow_high_precision_mv) + precision = RENCODE_AV1_MV_PRECISION_ALLOW_HIGH_PRECISION; av1_state->skip_mode_allowed = seq->flags.enable_order_hint &&