From 6e985845b32393f6175406a4dda3f2da7d36c770 Mon Sep 17 00:00:00 2001 From: Vlad Sytchenko Date: Wed, 6 May 2020 12:04:34 -0400 Subject: [PATCH] Take into account dynamic LDS size when validating the launch parameters. Bottom layers don't error check this value, so we might and up writing a bad value to a register and cause the SPI to hang. Change-Id: I6da4ae71c66a25c63ebb804da4afe4ca7fb831b7 --- rocclr/platform/command.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rocclr/platform/command.cpp b/rocclr/platform/command.cpp index db0cce08bf..1e15aebead 100644 --- a/rocclr/platform/command.cpp +++ b/rocclr/platform/command.cpp @@ -457,7 +457,7 @@ int32_t NDRangeKernelCommand::captureAndValidate() { int32_t error; uint64_t lclMemSize = kernel().getDeviceKernel(device)->workGroupInfo()->localMemSize_; - parameters_ = kernel().parameters().capture(device, lclMemSize, &error); + parameters_ = kernel().parameters().capture(device, sharedMemBytes_ + lclMemSize, &error); return error; }