Skip to content

Commit aaf0810

Browse files
Bensuocallumfare
authored andcommitted
[CMDBUF] Fix coverity issue in command buffers
- Fix incorrect conditions for copy engine usage that were reported on coverity.
1 parent a1c9e89 commit aaf0810

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

source/adapters/level_zero/command_buffer.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -885,7 +885,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urCommandBufferAppendMemBufferCopyExp(
885885
UR_CALL(DstBuffer->getZeHandle(ZeHandleDst, ur_mem_handle_t_::write_only,
886886
CommandBuffer->Device));
887887

888-
bool PreferCopyEngine = (SrcBuffer->OnHost || SrcBuffer->OnHost);
888+
bool PreferCopyEngine = (SrcBuffer->OnHost || DstBuffer->OnHost);
889889

890890
PreferCopyEngine |= UseCopyEngineForD2DCopy;
891891

@@ -917,7 +917,7 @@ UR_APIEXPORT ur_result_t UR_APICALL urCommandBufferAppendMemBufferCopyRectExp(
917917
UR_CALL(DstBuffer->getZeHandle(ZeHandleDst, ur_mem_handle_t_::write_only,
918918
CommandBuffer->Device));
919919

920-
bool PreferCopyEngine = (SrcBuffer->OnHost || SrcBuffer->OnHost);
920+
bool PreferCopyEngine = (SrcBuffer->OnHost || DstBuffer->OnHost);
921921

922922
PreferCopyEngine |= UseCopyEngineForD2DCopy;
923923

0 commit comments

Comments
 (0)