From 3c4885ecb8ca3297a04b170361c98016f70c6bea Mon Sep 17 00:00:00 2001 From: Rafal Maziejuk Date: Tue, 14 Sep 2021 14:57:43 +0000 Subject: [PATCH] Add allocation size adjustment check Signed-off-by: Rafal Maziejuk --- opencl/test/unit_test/os_interface/hw_info_config_tests.cpp | 5 +++++ shared/source/os_interface/hw_info_config.h | 2 ++ shared/source/os_interface/hw_info_config.inl | 5 +++++ shared/test/common/mocks/mock_hw_info_config.cpp | 5 +++++ 4 files changed, 17 insertions(+) diff --git a/opencl/test/unit_test/os_interface/hw_info_config_tests.cpp b/opencl/test/unit_test/os_interface/hw_info_config_tests.cpp index 3b73bf7779..a04da84d52 100644 --- a/opencl/test/unit_test/os_interface/hw_info_config_tests.cpp +++ b/opencl/test/unit_test/os_interface/hw_info_config_tests.cpp @@ -263,3 +263,8 @@ HWTEST_F(HwInfoConfigTest, givenVariousDebugKeyValuesWhenGettingLocalMemoryAcces DebugManager.flags.ForceLocalMemoryAccessMode.set(3); EXPECT_EQ(LocalMemoryAccessMode::CpuAccessDisallowed, hwInfoConfig.getLocalMemoryAccessMode(pInHwInfo)); } + +HWTEST_F(HwInfoConfigTest, givenHwInfoConfigWhenAskedIfAllocationSizeAdjustmentIsRequiredThenFalseIsReturned) { + const auto &hwInfoConfig = *HwInfoConfig::get(pInHwInfo.platform.eProductFamily); + EXPECT_FALSE(hwInfoConfig.isAllocationSizeAdjustmentRequired(pInHwInfo)); +} diff --git a/shared/source/os_interface/hw_info_config.h b/shared/source/os_interface/hw_info_config.h index 544c79349a..1cbf6c11e5 100644 --- a/shared/source/os_interface/hw_info_config.h +++ b/shared/source/os_interface/hw_info_config.h @@ -59,6 +59,7 @@ class HwInfoConfig { virtual bool allowRenderCompression(const HardwareInfo &hwInfo) const = 0; virtual bool allowStatelessCompression(const HardwareInfo &hwInfo) const = 0; virtual LocalMemoryAccessMode getLocalMemoryAccessMode(const HardwareInfo &hwInfo) const = 0; + virtual bool isAllocationSizeAdjustmentRequired(const HardwareInfo &hwInfo) const = 0; protected: virtual LocalMemoryAccessMode getDefaultLocalMemoryAccessMode(const HardwareInfo &hwInfo) const = 0; @@ -103,6 +104,7 @@ class HwInfoConfigHw : public HwInfoConfig { bool allowRenderCompression(const HardwareInfo &hwInfo) const override; bool allowStatelessCompression(const HardwareInfo &hwInfo) const override; LocalMemoryAccessMode getLocalMemoryAccessMode(const HardwareInfo &hwInfo) const override; + bool isAllocationSizeAdjustmentRequired(const HardwareInfo &hwInfo) const override; protected: HwInfoConfigHw() = default; diff --git a/shared/source/os_interface/hw_info_config.inl b/shared/source/os_interface/hw_info_config.inl index 084707bd7a..97571bd313 100644 --- a/shared/source/os_interface/hw_info_config.inl +++ b/shared/source/os_interface/hw_info_config.inl @@ -190,4 +190,9 @@ LocalMemoryAccessMode HwInfoConfigHw::getLocalMemoryAccessMode(const } return getDefaultLocalMemoryAccessMode(hwInfo); } + +template +bool HwInfoConfigHw::isAllocationSizeAdjustmentRequired(const HardwareInfo &hwInfo) const { + return false; +} } // namespace NEO diff --git a/shared/test/common/mocks/mock_hw_info_config.cpp b/shared/test/common/mocks/mock_hw_info_config.cpp index 24eeb7da72..4a0a5365e4 100644 --- a/shared/test/common/mocks/mock_hw_info_config.cpp +++ b/shared/test/common/mocks/mock_hw_info_config.cpp @@ -163,4 +163,9 @@ std::vector HwInfoConfigHw::getKernelSupportedThreadArbi return std::vector(); } +template <> +bool HwInfoConfigHw::isAllocationSizeAdjustmentRequired(const HardwareInfo &hwInfo) const { + return false; +} + } //namespace NEO