From f67ed6630edc6675a2dea906fdcdf68c0ff31f61 Mon Sep 17 00:00:00 2001 From: foreman Date: Tue, 19 Jun 2018 13:28:44 -0400 Subject: [PATCH] P4 to Git Change 1570186 by gandryey@gera-w8 on 2018/06/19 13:20:24 SWDEV-150518 - [IQE][DG2018][I+eA][SSG] System hangs when trying to preview cache video to AMD SSG - Disable optimization for "disabled deferred allocations" in SSG path. There is an extra logic to support double copy transfers with allocations in the invisible heap. It's not an official requirement for the SSG extension and hasn't been tested, but Adobe utilizes that path. Affected files ... ... //depot/stg/opencl/drivers/opencl/runtime/platform/command.cpp#89 edit --- rocclr/runtime/platform/command.cpp | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/rocclr/runtime/platform/command.cpp b/rocclr/runtime/platform/command.cpp index 0ebdad31ba..1d7d11254d 100644 --- a/rocclr/runtime/platform/command.cpp +++ b/rocclr/runtime/platform/command.cpp @@ -502,11 +502,6 @@ void TransferBufferFileCommand::submit(device::VirtualDevice& device) { } bool TransferBufferFileCommand::validateMemory() { - // Runtime disables deferred memory allocation for single device. - // Hence ignore memory validations - if (queue()->context().devices().size() == 1) { - return true; - } // Check if the destination buffer has direct host access if (!(memory_->getMemFlags() & (CL_MEM_USE_HOST_PTR | CL_MEM_ALLOC_HOST_PTR | CL_MEM_USE_PERSISTENT_MEM_AMD))) { @@ -534,11 +529,6 @@ bool TransferBufferFileCommand::validateMemory() { } bool CopyMemoryP2PCommand::validateMemory() { - // Runtime disables deferred memory allocation for single device. - // Hence ignore memory validations - if (queue()->context().devices().size() == 1) { - return true; - } const std::vector& devices = memory1_->getContext().devices(); if (devices.size() != 1) { LogError("Can't allocate memory object for P2P extension");