ahmed/OpenCL-CTS
1
0
Fork 0
mirror of https://github.com/KhronosGroup/OpenCL-CTS.git synced 2026-03-19 06:09:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Tighten AHB buffer row pitch in test_cl_khr_external_memory_ahb test (#2594)

Browse Source 
What was wrong:
enqueue_copy_buffer_to_image (and the related write/fill tests) mis-set
imageInfo.rowPitch to width*height*pixelSize. Because get_image_size
multiplies row pitch by height, this wrongly calculates the intended
buffer size.

How it’s fixed: set rowPitch to the true per-line pitch
(width*pixelSize) for all buffer-backed image cases so the calculated
sizes match the actual data layout and stay within the expected memory
footprint.

Signed-off-by: Xin Jin <xin.jin@arm.com>
This commit is contained in:
Xin Jin
2025-12-09 18:06:57 +00:00
committed by GitHub
parent 68c3eec051
commit afb6f6519c
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
test_conformance/extensions/cl_khr_external_memory_ahb/test_ahb.cpp
View File

@@ -1239,8 +1239,8 @@ REGISTER_TEST(enqueue_copy_buffer_to_image)
imageInfo.type = format.clMemObjectType;
imageInfo.width = resolution.width;
imageInfo.height = resolution.height;
imageInfo.rowPitch = resolution.width * resolution.height
* pixelSize; // data is tightly packed in buffer
// data is tightly packed in buffer
imageInfo.rowPitch = resolution.width * pixelSize;
test_assert_error(imageInfo.rowPitch
>= pixelSize * imageInfo.width,
"Row pitch is smaller than width");
@@ -1446,8 +1446,8 @@ REGISTER_TEST(enqueue_write_image)
imageInfo.type = format.clMemObjectType;
imageInfo.width = resolution.width;
imageInfo.height = resolution.height;
imageInfo.rowPitch = resolution.width * resolution.height
* pixelSize; // Data is tightly packed
// Data is tightly packed
imageInfo.rowPitch = resolution.width * pixelSize;
test_assert_error(imageInfo.rowPitch
>= pixelSize * imageInfo.width,
"Row pitch is smaller than width");
@@ -1637,8 +1637,8 @@ REGISTER_TEST(enqueue_fill_image)
imageInfo.type = format.clMemObjectType;
imageInfo.width = resolution.width;
imageInfo.height = resolution.height;
imageInfo.rowPitch = resolution.width * resolution.height
* pixelSize; // Data is tightly packed
imageInfo.rowPitch = resolution.width * pixelSize;
// Data is tightly packed
test_assert_error(imageInfo.rowPitch
>= pixelSize * imageInfo.width,
"Row pitch is smaller than width");