Skip to content
This repository was archived by the owner on Mar 28, 2023. It is now read-only.

Commit 6183ef2

Browse files
authored
[SYCL][ESIMD] Increase USM allocation size for Stencil test. (#189)
The Stencil test showed sporadic failures before last change. Investigation shows it works fine on Linux. The problem only showed up on Windows and could be voided by adjusting SLM allocation size. The GPU assembly dump is the same in the passing and failing case. Semenov, Sergey helped look into USM allocation issue on Windows vs. Linux and didn't find anything suspicious. So the problem could be still due to out-of-order block access. As discussed, in this patch the kernel SLM allocation size is increased by +16 to avoid OOB access.
1 parent 8bc731b commit 6183ef2

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

SYCL/ESIMD/Stencil.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
// test 8x16 block size
2323
//
2424
#define DIM_SIZE (1 << 13)
25-
#define SQUARE_SZ (DIM_SIZE * DIM_SIZE + 1)
25+
#define SQUARE_SZ (DIM_SIZE * DIM_SIZE + 16)
2626

2727
#define WIDTH 16
2828
#define HEIGHT 16

SYCL/ESIMD/stencil2.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
// test 8x16 block size
2323
//
2424
#define DIM_SIZE (1 << 13)
25-
#define SQUARE_SZ (DIM_SIZE * DIM_SIZE + 1)
25+
#define SQUARE_SZ (DIM_SIZE * DIM_SIZE + 16)
2626

2727
#define WIDTH 16
2828
#define HEIGHT 16

0 commit comments

Comments
 (0)