feature: reenable RelaxedOrdering

Related-To: NEO-7458

Signed-off-by: Dunajski, Bartosz <bartosz.dunajski@intel.com>
This commit is contained in:
Dunajski, Bartosz
2023-06-23 16:07:04 +00:00
committed by Compute-Runtime-Automation
parent c6404e6626
commit ecb415bf62
8 changed files with 3 additions and 24 deletions

View File

@@ -35,10 +35,6 @@ bool ApiSpecificConfig::isDynamicPostSyncAllocLayoutEnabled() {
return (NEO::DebugManager.flags.EnableDynamicPostSyncAllocLayout.get() != 0);
}
bool ApiSpecificConfig::isRelaxedOrderingEnabled() {
return false;
}
ApiSpecificConfig::ApiType ApiSpecificConfig::getApiType() {
return ApiSpecificConfig::L0;
}

View File

@@ -43,10 +43,6 @@ TEST(ApiSpecificConfigL0Tests, WhenCheckingIfDeviceAllocationCacheIsEnabledThenR
EXPECT_FALSE(ApiSpecificConfig::isDeviceAllocationCacheEnabled());
}
TEST(ApiSpecificConfigL0Tests, WhenCheckingIfRelaxedOrderingIsEnabledThenReturnFalse) {
EXPECT_FALSE(ApiSpecificConfig::isRelaxedOrderingEnabled());
}
TEST(ApiSpecificConfigL0Tests, GivenDebugFlagSetWhenCheckingIfDynamicPostSyncAllocLayoutEnabledThenReturnFalse) {
DebugManagerStateRestore restore;

View File

@@ -35,10 +35,6 @@ bool ApiSpecificConfig::isDynamicPostSyncAllocLayoutEnabled() {
return false;
}
bool ApiSpecificConfig::isRelaxedOrderingEnabled() {
return true;
}
ApiSpecificConfig::ApiType ApiSpecificConfig::getApiType() {
return ApiSpecificConfig::OCL;
}

View File

@@ -45,10 +45,6 @@ TEST(ApiSpecificConfigOclTests, WhenCheckingIfDeviceAllocationCacheIsEnabledThen
EXPECT_FALSE(ApiSpecificConfig::isDeviceAllocationCacheEnabled());
}
TEST(ApiSpecificConfigOclTests, WhenCheckingIfRelaxedOrderingIsEnabledThenReturnTrue) {
EXPECT_TRUE(ApiSpecificConfig::isRelaxedOrderingEnabled());
}
TEST(ApiSpecificConfigOclTests, WhenCheckingIfDynamicPostSyncAllocLayoutEnabledThenReturnFalse) {
DebugManagerStateRestore restore;

View File

@@ -18,7 +18,6 @@ struct ApiSpecificConfig {
static bool getBindlessMode();
static bool isDeviceAllocationCacheEnabled();
static bool isDynamicPostSyncAllocLayoutEnabled();
static bool isRelaxedOrderingEnabled();
static ApiType getApiType();
static std::string getName();
static uint64_t getReducedMaxAllocSize(uint64_t maxAllocSize);

View File

@@ -426,7 +426,7 @@ uint32_t GfxCoreHelperHw<Family>::getAmountOfAllocationsToFill() const {
template <>
bool GfxCoreHelperHw<Family>::isRelaxedOrderingSupported() const {
return ApiSpecificConfig::isRelaxedOrderingEnabled();
return true;
}
} // namespace NEO

View File

@@ -56,10 +56,6 @@ bool ApiSpecificConfig::isDynamicPostSyncAllocLayoutEnabled() {
return (NEO::DebugManager.flags.EnableDynamicPostSyncAllocLayout.get() == 1);
}
bool ApiSpecificConfig::isRelaxedOrderingEnabled() {
return true;
}
ApiSpecificConfig::ApiType ApiSpecificConfig::getApiType() {
return apiTypeForUlts;
}

View File

@@ -94,9 +94,9 @@ XE_HPC_CORETEST_F(GfxCoreHelperTest, givenGfxCoreHelperWhenCallGetAmountOfAlloca
HWTEST_EXCLUDE_PRODUCT(GfxCoreHelperTest, givenGfxCoreHelperWhenAskingForRelaxedOrderingSupportThenReturnFalse, IGFX_XE_HPC_CORE);
XE_HPC_CORETEST_F(GfxCoreHelperXeHpcCoreTest, givenGfxCoreHelperWhenAskingForRelaxedOrderingSupportThenCheckApiSpecificConfig) {
XE_HPC_CORETEST_F(GfxCoreHelperXeHpcCoreTest, givenGfxCoreHelperWhenAskingForRelaxedOrderingSupportThenReturnTrue) {
MockExecutionEnvironment mockExecutionEnvironment{};
auto &gfxCoreHelper = mockExecutionEnvironment.rootDeviceEnvironments[0]->getHelper<GfxCoreHelper>();
EXPECT_EQ(ApiSpecificConfig::isRelaxedOrderingEnabled(), gfxCoreHelper.isRelaxedOrderingSupported());
EXPECT_TRUE(gfxCoreHelper.isRelaxedOrderingSupported());
}