Skip to content

Commit

Permalink
Apply review feedback to remove unnecessary noise
Browse files Browse the repository at this point in the history
  • Loading branch information
GeorgeWeb committed Apr 2, 2024
1 parent d96bd03 commit 68b68b1
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions source/adapters/hip/enqueue.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1618,6 +1618,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urEnqueueUSMMemcpy2D(
hipPointerAttribute_t srcAttribs{};
hipPointerAttribute_t dstAttribs{};

hipError_t hipRes{};
// Determine if pSrc and/or pDst are system allocated pageable host memory.
bool srcIsSystemAlloc{false};
bool dstIsSystemAlloc{false};
Expand All @@ -1630,11 +1631,11 @@ UR_APIEXPORT ur_result_t UR_APICALL urEnqueueUSMMemcpy2D(
// the pointer handle as system allocated pageable host memory.
// The HIP runtime can handle the registering/unregistering of the memory
// as long as the right copy-kind (direction) is provided to hipMemcpy2D*.
hipError_t hipRet = hipPointerGetAttributes(&srcAttribs, pSrc);
if (pSrc && hipRet == hipErrorInvalidValue)
hipRes = hipPointerGetAttributes(&srcAttribs, pSrc);
if (hipRes == hipErrorInvalidValue && pSrc)
srcIsSystemAlloc = true;
hipRet = hipPointerGetAttributes(&dstAttribs, (const void *)pDst);
if (pDst && hipRet == hipErrorInvalidValue)
hipRes = hipPointerGetAttributes(&dstAttribs, (const void *)pDst);
if (hipRes == hipErrorInvalidValue && pDst)
dstIsSystemAlloc = true;
#if HIP_VERSION_MAJOR >= 6
srcIsSystemAlloc |= srcAttribs.type == hipMemoryTypeUnregistered;
Expand Down

0 comments on commit 68b68b1

Please sign in to comment.