From 6bf1babb516d3ea3cec115eb6e56efdb0d2cc1d4 Mon Sep 17 00:00:00 2001 From: Philip Yang Date: Mon, 12 Jun 2023 12:39:31 -0400 Subject: [PATCH] kfdtest: Fix KFDSVMEvictTest.QueueTest OOM Typo to calculate bufferSize from vramBufSizeInPages. The OOM shows up only with HSA_XNACK=1 because HSA_XNACK=0 doesn't support VRAM oversubscription. We changed to run SVM tests with both XNACK off and on. Change-Id: I3949959288fd92f4e7f4a87115a5f1547e225042 Signed-off-by: Philip Yang [ROCm/ROCR-Runtime commit: 29b04c25349a409aa153c550eed08a658efd23c3] --- projects/rocr-runtime/tests/kfdtest/src/KFDSVMEvictTest.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/projects/rocr-runtime/tests/kfdtest/src/KFDSVMEvictTest.cpp b/projects/rocr-runtime/tests/kfdtest/src/KFDSVMEvictTest.cpp index e9ae0ca867..fb7a2e9c7e 100644 --- a/projects/rocr-runtime/tests/kfdtest/src/KFDSVMEvictTest.cpp +++ b/projects/rocr-runtime/tests/kfdtest/src/KFDSVMEvictTest.cpp @@ -98,7 +98,7 @@ HSAint64 KFDSVMEvictTest::GetBufferSize(HSAuint64 vramSize, HSAuint32 count) { sizeInPages = size >> PAGE_SHIFT; vramBufSizeInPages = sizeInPages / (count * N_PROCESSES); - return vramBufSizeInPages << 20; + return vramBufSizeInPages << PAGE_SHIFT; } void KFDSVMEvictTest::AllocBuffers(HSAuint32 defaultGPUNode, HSAuint32 count, HSAuint64 vramBufSize,