Skip to content

Commit 0c8e0f9

Browse files
fix: ReturnSubDevicesAsApiDevices flag being ignored
Proper subdevice count being returned now in GfxCoreHelper path, as previous method ignored the usage of the ReturnSubDevicesAsApiDevices flag. Related-To: LOCI-4859 Signed-off-by: Latif, Raiyan <[email protected]> Source: dee5ecf
1 parent 9e3d38d commit 0c8e0f9

File tree

2 files changed

+22
-0
lines changed

2 files changed

+22
-0
lines changed

shared/source/helpers/gfx_core_helper.cpp

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,8 @@ uint32_t GfxCoreHelper::getMaxThreadsForVfe(const HardwareInfo &hwInfo) {
7272
uint32_t GfxCoreHelper::getSubDevicesCount(const HardwareInfo *pHwInfo) {
7373
if (DebugManager.flags.CreateMultipleSubDevices.get() > 0) {
7474
return DebugManager.flags.CreateMultipleSubDevices.get();
75+
} else if (DebugManager.flags.ReturnSubDevicesAsApiDevices.get() > 0) {
76+
return 1u;
7577
} else if (pHwInfo->gtSystemInfo.MultiTileArchInfo.IsValid && pHwInfo->gtSystemInfo.MultiTileArchInfo.TileCount > 0u) {
7678
return pHwInfo->gtSystemInfo.MultiTileArchInfo.TileCount;
7779
} else {

shared/test/unit_test/aub/aub_helper_tests.cpp

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,26 @@ TEST(AubHelper, GivenMultipleSubDevicesWhenGettingDeviceCountThenCorrectValueIsR
5151
EXPECT_EQ(devicesCount, 1u);
5252
}
5353

54+
TEST(AubHelper, GivenReturnSubDevicesAsApiDevicesWhenGettingDeviceCountThenCorrectValueIsReturned) {
55+
DebugManagerStateRestore stateRestore;
56+
FeatureTable featureTable = {};
57+
WorkaroundTable workaroundTable = {};
58+
RuntimeCapabilityTable capTable = {};
59+
GT_SYSTEM_INFO sysInfo = {};
60+
PLATFORM platform = {};
61+
62+
sysInfo.MultiTileArchInfo.IsValid = true;
63+
sysInfo.MultiTileArchInfo.TileCount = 2;
64+
HardwareInfo hwInfo{&platform, &featureTable, &workaroundTable, &sysInfo, capTable};
65+
66+
uint32_t devicesCount = GfxCoreHelper::getSubDevicesCount(&hwInfo);
67+
EXPECT_EQ(devicesCount, 2u);
68+
69+
DebugManager.flags.ReturnSubDevicesAsApiDevices.set(1);
70+
devicesCount = GfxCoreHelper::getSubDevicesCount(&hwInfo);
71+
EXPECT_EQ(devicesCount, 1u);
72+
}
73+
5474
TEST(AubHelper, WhenGetMemTraceIsCalledWithLocalMemoryPDEntryBitsThenTraceLocalIsReturned) {
5575
int hint = AubHelper::getMemTrace(BIT(PageTableEntry::localMemoryBit));
5676
EXPECT_EQ(AubMemDump::AddressSpaceValues::TraceLocal, hint);

0 commit comments

Comments
 (0)