Skip to content

Commit ad8ee5f

Browse files
committed
Merge remote-tracking branch 'upstream/master'
2 parents 8309caf + 8e141eb commit ad8ee5f

File tree

5 files changed

+13
-22
lines changed

5 files changed

+13
-22
lines changed

Source/GmmLib/CachePolicy/GmmGen10CachePolicy.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ uint32_t GmmLib::GmmGen10CachePolicy::BestMatchingPATIdx(GMM_CACHE_POLICY_ELEMEN
295295
WantedTC = GMM_GFX_TC_ELLC_ONLY; // Note: this overrides the MOCS target cache selection.
296296
}
297297

298-
for(i = 1; i < GMM_NUM_PAT_ENTRIES; i++)
298+
for(i = 1; i < NumPATRegisters; i++)
299299
{
300300
GMM_PRIVATE_PAT PAT1 = GetPrivatePATEntry(PATIdx);
301301
GMM_PRIVATE_PAT PAT2 = GetPrivatePATEntry(i);
@@ -359,7 +359,7 @@ GMM_STATUS GmmLib::GmmGen10CachePolicy::SetupPAT()
359359
}
360360

361361
// Set values for GmmGlobalInfo PrivatePATTable
362-
for(i = 0; i < GMM_NUM_PAT_ENTRIES; i++)
362+
for(i = 0; i < NumPATRegisters; i++)
363363
{
364364
GMM_PRIVATE_PAT PAT = {0};
365365

Source/GmmLib/CachePolicy/GmmGen12CachePolicy.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -424,7 +424,7 @@ uint32_t GmmLib::GmmGen12CachePolicy::BestMatchingPATIdx(GMM_CACHE_POLICY_ELEMEN
424424
WantedMemoryType = GMM_GFX_UC_WITH_FENCE;
425425
}
426426

427-
for(i = 1; i < GMM_NUM_PAT_ENTRIES; i++)
427+
for(i = 1; i < NumPATRegisters; i++)
428428
{
429429
GMM_PRIVATE_PAT PAT1 = GetPrivatePATEntry(PATIdx);
430430
GMM_PRIVATE_PAT PAT2 = GetPrivatePATEntry(i);
@@ -504,7 +504,7 @@ GMM_STATUS GmmLib::GmmGen12CachePolicy::SetupPAT()
504504
}
505505

506506
// Set values for GmmGlobalInfo PrivatePATTable
507-
for(i = 0; i < GMM_NUM_PAT_ENTRIES; i++)
507+
for(i = 0; i < NumPATRegisters; i++)
508508
{
509509
GMM_PRIVATE_PAT PAT = {0};
510510

Source/GmmLib/CachePolicy/GmmGen8CachePolicy.cpp

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -159,7 +159,7 @@ GMM_STATUS GmmLib::GmmGen8CachePolicy::SetupPAT()
159159
}
160160

161161
// Set values for GmmGlobalInfo PrivatePATTable
162-
for(i = 0; i < GMM_NUM_PAT_ENTRIES; i++)
162+
for(i = 0; i < NumPATRegisters; i++)
163163
{
164164
GMM_PRIVATE_PAT PAT = {0};
165165

@@ -334,7 +334,7 @@ uint32_t GmmLib::GmmGen8CachePolicy::BestMatchingPATIdx(GMM_CACHE_POLICY_ELEMENT
334334
WantedTC = GMM_GFX_TC_ELLC_ONLY; // Note: this overrides the MOCS target cache selection.
335335
}
336336

337-
for(i = 1; i < GMM_NUM_PAT_ENTRIES; i++)
337+
for(i = 1; i < NumPATRegisters; i++)
338338
{
339339
GMM_PRIVATE_PAT PAT1 = GetPrivatePATEntry(PATIdx);
340340
GMM_PRIVATE_PAT PAT2 = GetPrivatePATEntry(i);
@@ -368,19 +368,14 @@ uint32_t GmmLib::GmmGen8CachePolicy::BestMatchingPATIdx(GMM_CACHE_POLICY_ELEMENT
368368
/////////////////////////////////////////////////////////////////////////////////////
369369
bool GmmLib::GmmGen8CachePolicy::SetPrivatePATEntry(uint32_t PATIdx, GMM_PRIVATE_PAT Entry)
370370
{
371-
if(PATIdx >= GMM_NUM_PAT_ENTRIES)
371+
if(PATIdx >= NumPATRegisters)
372372
{
373373
GMM_ASSERTDPF(false, "CRITICAL ERROR: INVALID PAT IDX");
374374
return false;
375375
}
376-
#if(!defined(__GMM_KMD__))
377-
GMM_UNREFERENCED_PARAMETER(Entry);
378-
GMM_ASSERTDPF(false, "Should only be called from KMD");
379-
return false;
380-
#else
376+
381377
pGmmLibContext->GetPrivatePATTable()[PATIdx] = Entry;
382378
return true;
383-
#endif
384379
}
385380

386381
/////////////////////////////////////////////////////////////////////////////////////
@@ -394,17 +389,13 @@ GMM_PRIVATE_PAT GmmLib::GmmGen8CachePolicy::GetPrivatePATEntry(uint32_t PATIdx)
394389
{
395390
GMM_PRIVATE_PAT NullPAT = {0};
396391

397-
if(PATIdx >= GMM_NUM_PAT_ENTRIES)
392+
if(PATIdx >= NumPATRegisters)
398393
{
399394
GMM_ASSERTDPF(false, "CRITICAL ERROR: INVALID PAT IDX");
400395
return NullPAT;
401396
}
402397

403-
#if(!defined(__GMM_KMD__))
404-
return NullPAT;
405-
#else
406398
return pGmmLibContext->GetPrivatePATTable()[PATIdx];
407-
#endif
408399
}
409400

410401
/////////////////////////////////////////////////////////////////////////////////////

Source/GmmLib/CachePolicy/GmmGen9CachePolicy.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -286,7 +286,7 @@ GMM_STATUS GmmLib::GmmGen9CachePolicy::SetupPAT()
286286
}
287287

288288
// Set values for GmmGlobalInfo PrivatePATTable
289-
for(i = 0; i < GMM_NUM_PAT_ENTRIES; i++)
289+
for(i = 0; i < NumPATRegisters; i++)
290290
{
291291
GMM_PRIVATE_PAT PAT = {0};
292292

Source/GmmLib/inc/External/Common/GmmInfo.h

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,13 +105,13 @@ namespace GmmLib
105105

106106
#if(defined(__GMM_KMD__))
107107
uint64_t IA32ePATTable;
108-
GMM_PRIVATE_PAT PrivatePATTable[GMM_NUM_PAT_ENTRIES];
109-
int32_t PrivatePATTableMemoryType[GMM_NUM_GFX_PAT_TYPES];
108+
GMM_PRIVATE_PAT PrivatePATTable[GMM_NUM_PAT_ENTRIES];
109+
int32_t PrivatePATTableMemoryType[GMM_NUM_GFX_PAT_TYPES];
110110
#endif
111111

112112
// Padding Percentage limit on 64KB paged resource
113113
uint32_t AllowedPaddingFor64KbPagesPercentage;
114-
uint64_t InternalGpuVaMax;
114+
uint64_t InternalGpuVaMax;
115115
uint32_t AllowedPaddingFor64KBTileSurf;
116116
#ifdef GMM_LIB_DLL
117117
// Mutex Object used for synchronization of ProcessSingleton Context

0 commit comments

Comments
 (0)