[VP] refactor fallBackScalingToRender

move WA to uerFeatureControl
intel · Feb 28, 2025 · 58876d5 · 58876d5
1 parent f96eb02
commit 58876d5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/media_softlet/agnostic/common/vp/hal/feature_manager/policy.cpp b/media_softlet/agnostic/common/vp/hal/feature_manager/policy.cpp
@@ -1362,8 +1362,7 @@ MOS_STATUS Policy::GetScalingExecutionCaps(SwFilter *feature, bool isHdrEnabled,
         return MOS_STATUS_SUCCESS;
     }
 
-    if (MEDIA_IS_WA(m_vpInterface.GetHwInterface()->m_waTable, Wa_16025683853) &&
-        fallbackScalingToRender8K == true                     &&
+    if (fallbackScalingToRender8K == true                     &&
         scalingParams->input.dwHeight > 3072                  &&
         isScalingNeeded)
     {

diff --git a/media_softlet/agnostic/common/vp/hal/utils/vp_user_feature_control.cpp b/media_softlet/agnostic/common/vp/hal/utils/vp_user_feature_control.cpp
@@ -473,6 +473,7 @@ MOS_STATUS VpUserFeatureControl::CreateUserSettingForDebug()
 #endif
     {
         auto *waTable = m_osInterface->pfnGetWaTable(m_osInterface);
+        VP_PUBLIC_CHK_NULL_RETURN(waTable);
         // Default value
         m_ctrlValDefault.enableSFCLinearOutputByTileConvert = MEDIA_IS_WA(waTable, Wa_15016458807);
     }
@@ -494,11 +495,12 @@ MOS_STATUS VpUserFeatureControl::CreateUserSettingForDebug()
     else
 #endif
     {
-        // WA ID need be added before code merge.
         // Do not set fallbackScalingToRender8K if render is disabled. This is for internal vesfc usage (e.g., 8k preenc) that should not fallback to render.
         if (m_vpPlatformInterface && !m_vpPlatformInterface->IsRenderDisabled())
         {
-            m_ctrlValDefault.fallbackScalingToRender8K = true;
+            auto *waTable = m_osInterface->pfnGetWaTable(m_osInterface);
+            VP_PUBLIC_CHK_NULL_RETURN(waTable);
+            m_ctrlValDefault.fallbackScalingToRender8K = MEDIA_IS_WA(waTable, Wa_16025683853);
         }
     }
     VP_PUBLIC_NORMALMESSAGE("fallbackScalingToRender8K %d", m_ctrlValDefault.fallbackScalingToRender8K);