Skip to content

Commit

Permalink
disable USM memory allocation types other than Shared
Browse files Browse the repository at this point in the history
enabling Device/Host USM types make some Texture tests fail
  • Loading branch information
franz committed May 12, 2023
1 parent 26175de commit 25d2f83
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions src/backend/OpenCL/SVMemoryRegion.cc
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,10 @@ void *SVMemoryRegion::allocate(size_t Size, size_t Alignment,
int Err;
if (SupportsIntelUSM) {
switch (MemType) {

// TODO: investigate. Uncommenting this code makes
// a bunch of Unit_hipTexture tests fail with segfault.
/*
case hipMemoryTypeHost:
Ptr = USM.clHostMemAllocINTEL(Context_(), NULL, Size, Alignment, &Err);
break;
Expand All @@ -60,6 +64,7 @@ void *SVMemoryRegion::allocate(size_t Size, size_t Alignment,
break;
case hipMemoryTypeManaged:
case hipMemoryTypeUnified:
*/
default:
Ptr = USM.clSharedMemAllocINTEL(Context_(), Device_(), NULL, Size,
Alignment, &Err);
Expand Down

0 comments on commit 25d2f83

Please sign in to comment.