[Piglit] [PATCH v5] cl: add clEnqueue-read_write BufferRect releated tests
Serge Martin (EdB)
edb+piglit at sigluy.net
Tue Aug 18 03:54:58 PDT 2015
Hello
I was about to push it, be find that some of the errors tests return CL_SUCCESS
when they shouldn't have, sorry about that.
Does this test pass on INTEL OCL?
See comments below
On Monday 03 August 2015 01:59:44 Meng Mengmeng wrote:
> It's a simple function test for clEnqueueReadBufferRect and
> clEnqueueWriteBufferRect.
> ---
> tests/cl.py | 2 +
> tests/cl/api/CMakeLists.cl.txt | 1 +
> tests/cl/api/enqueue-read_write-buffer-rect.c | 338
> ++++++++++++++++++++++++++ 3 files changed, 341 insertions(+)
> create mode 100644 tests/cl/api/enqueue-read_write-buffer-rect.c
>
> diff --git a/tests/cl.py b/tests/cl.py
> index 4668ddc..572ccdc 100644
> --- a/tests/cl.py
> +++ b/tests/cl.py
> @@ -58,6 +58,8 @@ with profile.group_manager(PiglitCLTest, 'api') as g:
> g(['cl-api-enqueue-copy-buffer-rect'], 'clEnqueueCopyBufferRect')
> g(['cl-api-enqueue-read_write-buffer'],
> 'clEnqueueReadBuffer and clEnqueueWriteBuffer')
> + g(['cl-api-enqueue-read_write-buffer-rect'],
> + 'clEnqueueReadBufferRect and clEnqueueWriteBufferRect')
> g(['cl-api-get-mem-object-info'], 'clGetMemObjectInfo')
> g(['cl-api-get-image-info'], 'clGetImageInfo')
> g(['cl-api-retain_release-mem-object'],
> diff --git a/tests/cl/api/CMakeLists.cl.txt b/tests/cl/api/CMakeLists.cl.txt
> index b598528..f0a388f 100644
> --- a/tests/cl/api/CMakeLists.cl.txt
> +++ b/tests/cl/api/CMakeLists.cl.txt
> @@ -21,6 +21,7 @@ piglit_cl_add_api_test (enqueue-copy-buffer
> enqueue-copy-buffer.c) piglit_cl_add_api_test (enqueue-map-buffer
> enqueue-map-buffer.c)
> piglit_cl_add_api_test (enqueue-copy-buffer-rect
> enqueue-copy-buffer-rect.c) piglit_cl_add_api_test
> (enqueue-read_write-buffer enqueue-read_write-buffer.c)
> +piglit_cl_add_api_test (enqueue-read_write-buffer-rect
> enqueue-read_write-buffer-rect.c) piglit_cl_add_api_test
> (retain_release-mem-object retain_release-mem-object.c)
> piglit_cl_add_api_test (get-mem-object-info get-mem-object-info.c)
> piglit_cl_add_api_test (get-image-info get-image-info.c)
> diff --git a/tests/cl/api/enqueue-read_write-buffer-rect.c
> b/tests/cl/api/enqueue-read_write-buffer-rect.c new file mode 100644
> index 0000000..6f7d2ca
> --- /dev/null
> +++ b/tests/cl/api/enqueue-read_write-buffer-rect.c
> @@ -0,0 +1,338 @@
> +/*
> + * Copyright © 2015 Intel Corporation
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the
> "Software"), + * to deal in the Software without restriction, including
> without limitation + * the rights to use, copy, modify, merge, publish,
> distribute, sublicense, + * and/or sell copies of the Software, and to
> permit persons to whom the + * Software is furnished to do so, subject to
> the following conditions: + *
> + * The above copyright notice and this permission notice (including the
> next + * paragraph) shall be included in all copies or substantial portions
> of the + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS
> OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
> IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
> CLAIM, DAMAGES OR OTHER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
> TORT OR OTHERWISE, ARISING + * FROM, OUT OF OR IN CONNECTION WITH THE
> SOFTWARE OR THE USE OR OTHER + * DEALINGS IN THE SOFTWARE.
> + *
> + * Authors: Meng Mengmeng <mengmeng.meng at intel.com>
> + *
> + */
> +
> +#include "piglit-framework-cl-api.h"
> +#include "piglit-util-cl.h"
> +
> +
> +PIGLIT_CL_API_TEST_CONFIG_BEGIN
> +
> + config.name = "clEnqueueReadBufferRect and clEnqueueWriteBufferRect";
> + config.version_min = 11;
> +
> + config.run_per_platform = true;
> + config.create_context = true;
> +
> +PIGLIT_CL_API_TEST_CONFIG_END
> +
> +static bool
> +test_read_rect(cl_command_queue command_queue,
> + cl_mem buffer,
> + cl_bool blocking_read,
> + const size_t * buffer_origin,
> + const size_t * host_origin,
> + const size_t * region,
> + size_t buffer_row_pitch,
> + size_t buffer_slice_pitch,
> + size_t host_row_pitch,
> + size_t host_slice_pitch,
> + void *ptr,
> + cl_uint num_events_in_wait_list,
> + const cl_event *event_wait_list,
> + cl_event *event,
> + cl_int expected_error,
> + enum piglit_result* result,
> + const char* test_str) {
> + cl_int errNo;
> +
> + errNo = clEnqueueReadBufferRect(command_queue,
> + buffer,
> + blocking_read,
> + buffer_origin,
> + host_origin,
> + region,
> + buffer_row_pitch,
> + buffer_slice_pitch,
> + host_row_pitch,
> + host_slice_pitch,
> + ptr,
> + num_events_in_wait_list,
> + event_wait_list,
> + event);
> +
> + if(!piglit_cl_check_error(errNo, expected_error)) {
Please add space after if: if (!pigli...
> + fprintf(stderr, "clEnqueueReadBufferRect: Failed (error code: %s):
> %s.\n", + piglit_cl_get_error_name(errNo), test_str);
> + piglit_merge_result(result, PIGLIT_FAIL);
> + return false;
> + }
> +
> + return true;
> +}
> +
> +static bool
> +test_write_rect(cl_command_queue command_queue,
> + cl_mem buffer,
> + cl_bool blocking_write,
> + const size_t * buffer_origin,
> + const size_t * host_origin,
> + const size_t * region,
> + size_t buffer_row_pitch,
> + size_t buffer_slice_pitch,
> + size_t host_row_pitch,
> + size_t host_slice_pitch,
> + void *ptr,
> + cl_uint num_events_in_wait_list,
> + const cl_event *event_wait_list,
> + cl_event *event,
> + cl_int expected_error,
> + enum piglit_result* result,
> + const char* test_str) {
> + cl_int errNo;
> +
> + errNo = clEnqueueWriteBufferRect(command_queue,
> + buffer,
> + blocking_write,
> + buffer_origin,
> + host_origin,
> + region,
> + buffer_row_pitch,
> + buffer_slice_pitch,
> + host_row_pitch,
> + host_slice_pitch,
> + ptr,
> + num_events_in_wait_list,
> + event_wait_list,
> + event);
> +
> + if(!piglit_cl_check_error(errNo, expected_error)) {
same here
> + fprintf(stderr, "clEnqueueWiteBufferRect: Failed (error code: %s):
> %s.\n", + piglit_cl_get_error_name(errNo), test_str);
> + piglit_merge_result(result, PIGLIT_FAIL);
> + return false;
> + }
> +
> + return true;
> +}
> +enum piglit_result
> +piglit_cl_test(const int argc,
> + const char **argv,
> + const struct piglit_cl_api_test_config* config,
> + const struct piglit_cl_api_test_env* env)
> +{
> +
> + enum piglit_result result = PIGLIT_PASS;
> + piglit_cl_context context;
> +
> + cl_event valid_event;
> + cl_event invalid_event = NULL;
> +
> + cl_int i, j;
> + cl_int k = 0;
> + float full_matrix[81], result_matrix[9];
> + const size_t buffer_origin[3] = {3 * sizeof(float), 3, 0};
> + const size_t host_origin[3] = {3 * sizeof(float), 3, 0};
> + const size_t host_new_origin[3] = {0 * sizeof(float), 0, 0};
> + const size_t region[3] = {3 * sizeof(float), 3, 1};
> + cl_mem matrix_buffer;
> +
> + for(i=0; i<81; i++) {
as for the if, please put a space here
> + full_matrix[i] = i * 1.0f;
> + }
> + for(i=0; i<9; i++) {
same here
> + result_matrix[i] = 0.0f;
> + }
> +
> + float reference_matrix[9] = {30.0f, 31.0f, 32.0f,
> + 39.0f, 40.0f, 41.0f, 48.0f, 49.0f, 50.0f};
> +
> + matrix_buffer = piglit_cl_create_buffer(env->context, CL_MEM_READ_WRITE ,
> sizeof(full_matrix)); +
> + /*
> + * Write to a buffer object from host memory(9*9 matrix), then read
> + * rectangular the region 3*3 from that buffer object to host memory,
> + * finally compare the read result with reference matrix.
> + */
> + test_write_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region, 9 * sizeof(float), 0,
> + 9 * sizeof(float), 0, full_matrix, 0, NULL, NULL,
> + CL_SUCCESS, &result,
> + "clEnqueueWriteBufferRec: Failed when write to a buffer
> object 9*9 from host memor");
> + test_read_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_new_origin, region, 9 *
> sizeof(float), 0, + 3 * sizeof(float), 0, result_matrix, 0,
> NULL, NULL, + CL_SUCCESS, &result,
> + "clEnqueueReadBufferRect: Failed when read from a
> rectangular region 3*3 to host memory"); +
> + for(i = 0; i < 3; ++i){
> + for(j = 0; j < 3; ++j){
> + if(result_matrix[j+i*3] != reference_matrix[k]){
> + fprintf(stderr,
> + "clEnqueueWriteBufferRect and
clEnqueueReadBufferRect: Failed
> (expected %f, but got %f)\n", +
> reference_matrix[k],result_matrix[j+i*3]);
> + piglit_merge_result(&result, PIGLIT_FAIL);
> + }else
Please put a space and add an extra {
> + k++;
> + }
> + }
> +
> + /*
> + * CL_INVALID_COMMAND_QUEUE if command_queue is not a valid command-queue.
> + */
> + test_write_rect(NULL, matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region, 9 * sizeof(float), 0,
> + 9 * sizeof(float), 0, full_matrix, 0, NULL, NULL,
> + CL_INVALID_COMMAND_QUEUE, &result,
> + "Trigger CL_INVALID_COMMAND_QUEUE when command_queue is
> not a valid command-queue"); + test_read_rect(NULL, matrix_buffer,
CL_TRUE,
> + buffer_origin, host_new_origin, region, 9 * sizeof(float),
> 0, + 3 * sizeof(float), 0, result_matrix, 0, NULL, NULL, +
> CL_INVALID_COMMAND_QUEUE, &result,
> + "Trigger CL_INVALID_COMMAND_QUEUE when command_queue is not
> a valid command-queue"); +
> + /*
> + * CL_INVALID_CONTEXT if the context associated with command_queue
> + * and buffer are not the same or if the context associated with
> command_queue + * and events in event_wait_list are not the same.
> + */
> + context = piglit_cl_create_context(env->platform_id,
> + env->context->device_ids,
> + 1);
> + if(context != NULL){
same here + space before {
> + test_write_rect(context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region, 9 * sizeof(float),
0,
> + 9 * sizeof(float), 0, full_matrix, 0, NULL, NULL, +
> CL_INVALID_CONTEXT, &result,
> + "Trigger CL_INVALID_CONTEXT when context associated
with
> command_queue and buffer are not the same");
> + test_read_rect(context->command_queues[0], matrix_buffer, CL_TRUE, +
> buffer_origin, host_new_origin, region, 9 * sizeof(float), 0,
> + 3 * sizeof(float), 0, result_matrix, 0, NULL, NULL, +
> CL_INVALID_CONTEXT, &result,
> + "Trigger CL_INVALID_CONTEXT when context associated
with
> command_queue and buffer are not the same");
add piglit_cl_release_context(context)
> + }else {
a space here
> + fprintf(stderr, "Could not test triggering CL_INVALID_CONTEXT.\n");
> + piglit_merge_result(&result, PIGLIT_WARN);
> + }
> +
> + /*
> + * CL_INVALID_MEM_OBJECT if buffer is not a valid buffer object.
> + */
> + test_write_rect(env->context->command_queues[0], NULL, CL_TRUE,
> + buffer_origin, host_origin, region, 9 * sizeof(float), 0,
> + 9 * sizeof(float), 0, full_matrix, 0, NULL, NULL,
> + CL_INVALID_MEM_OBJECT, &result,
> + "Trigger CL_INVALID_MEM_OBJECT when buffer is not a valid
> buffer object"); + test_read_rect(env->context->command_queues[0], NULL,
> CL_TRUE,
> + buffer_origin, host_new_origin, region, 9 * sizeof(float),
> 0, + 3 * sizeof(float), 0, result_matrix, 0, NULL, NULL, +
> CL_INVALID_MEM_OBJECT, &result,
> + "Trigger CL_INVALID_MEM_OBJECT when buffer is not a valid
> buffer object"); +
> + /*
> + * CL_INVALID_VALUE if the region being read specified by
> + * (buffer_origin, region) is out of bounds or if ptr is a NULL value.
> + */
> + test_write_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin + 1, host_origin, region, 9 * sizeof(float),0,
You are doing pointer arithmetic here, I guess it's not what you wanted
> + 9 * sizeof(float), 0, full_matrix, 0, NULL, NULL, +
> CL_INVALID_VALUE, &result,
> + "Trigger CL_INVALID_VALUE if the region being write
> specified by (buffer_origin, region) is out of bounds");
> + test_read_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin + 1, host_new_origin, region, 9 *
same here and probably after
> sizeof(float), 0, + 3 * sizeof(float), 0, result_matrix, 0,
> NULL, NULL, + CL_INVALID_VALUE, &result,
> + "Trigger CL_INVALID_VALUE if the region being read
> specified by (buffer_origin, region) is out of bounds");
> + test_write_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region + 1, 9 *
> sizeof(float), 0, + 9 * sizeof(float), 0, full_matrix, 0,
> NULL, NULL, + CL_INVALID_VALUE, &result,
> + "Trigger CL_INVALID_VALUE if the region being write
> specified by (buffer_origin, region) is out of bounds");
> + test_read_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin , host_new_origin, region + 1, 9 *
> sizeof(float), 0, + 3 * sizeof(float), 0, result_matrix, 0,
> NULL, NULL, + CL_INVALID_VALUE, &result,
> + "Trigger CL_INVALID_VALUE if the region being read
> specified by (buffer_origin, region) is out of bounds");
> + test_write_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region , 9 * sizeof(float),
> 0, + 9 * sizeof(float), 0, NULL, 0, NULL, NULL,
> + CL_INVALID_VALUE, &result,
> + "Trigger CL_INVALID_VALUE if ptr is a NULL value");
> + test_read_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin , host_new_origin, region, 9 *
> sizeof(float), 0, + 3 * sizeof(float), 0, NULL, 0, NULL,
> NULL,
> + CL_INVALID_VALUE, &result,
> + "Trigger CL_INVALID_VALUE if ptr is a NULL value");
> +
> + /*
> + * CL_INVALID_EVENT_WAIT_LIST if event_wait_list is NULL and
> + * num_events_in_wait_list greater than 0,or event_wait_list
> + * is not NULL and num_events_in_wait_list is 0, or if event
> + * objects in event_wait_list are not valid events.
> + */
> + test_write_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region, 9 * sizeof(float), 0,
> + 9 * sizeof(float), 0, full_matrix, 0, NULL, &valid_event,
> + CL_SUCCESS, &result,
> + "Create an event");
> +
> + test_write_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region, 9 * sizeof(float), 0,
> + 9 * sizeof(float), 0, full_matrix, 1, NULL, NULL,
> + CL_INVALID_EVENT_WAIT_LIST, &result,
> + "Trigger CL_INVALID_EVENT_WAIT_LIST when event_wait_list
> is NULL and num_events_in_wait_list greater than 0");
> + test_read_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin , host_new_origin, region, 9 *
> sizeof(float), 0, + 3 * sizeof(float), 0, result_matrix, 1,
> NULL, NULL, + CL_INVALID_EVENT_WAIT_LIST, &result,
> + "Trigger CL_INVALID_EVENT_WAIT_LIST when event_wait_list is
> NULL and num_events_in_wait_list greater than 0");
> + test_write_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region, 9 * sizeof(float), 0,
> + 9 * sizeof(float), 0, full_matrix, 0, &valid_event, NULL,
> + CL_INVALID_EVENT_WAIT_LIST, &result,
> + "Trigger CL_INVALID_EVENT_WAIT_LIST when vent_wait_list is
> not NULL and num_events_in_wait_list is 0");
> + test_read_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin , host_new_origin, region, 9 *
> sizeof(float), 0, + 3 * sizeof(float), 0, result_matrix, 0,
> &valid_event, NULL, + CL_INVALID_EVENT_WAIT_LIST, &result,
> + "Trigger CL_INVALID_EVENT_WAIT_LIST when vent_wait_list is
> not NULL and num_events_in_wait_list is 0");
> + test_write_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin, host_origin, region, 9 * sizeof(float), 0,
> + 9 * sizeof(float), 0, full_matrix, 1, &invalid_event,
> NULL, + CL_INVALID_EVENT_WAIT_LIST, &result,
> + "Trigger CL_INVALID_EVENT_WAIT_LIST when event objects in
> event_wait_list are not valid events");
> + test_read_rect(env->context->command_queues[0], matrix_buffer, CL_TRUE,
> + buffer_origin , host_new_origin, region, 9 *
> sizeof(float), 0, + 3 * sizeof(float), 0, result_matrix, 0,
> &invalid_event, NULL, + CL_INVALID_EVENT_WAIT_LIST,
&result,
> + "Trigger CL_INVALID_EVENT_WAIT_LIST when event objects in
> event_wait_list are not valid event"); +
> + clReleaseEvent(valid_event);
> +
> + /*
> + * CL_MISALIGNED_SUB_BUFFER_OFFSET if buffer is a sub-buffer
> + * object and offset specified when the sub-buffer object is
> + * created is not aligned to CL_DEVICE_MEM_BASE_ADDR_ALIGN
> + * value for device associated with queue.
> + */
> +
> + /*
> + * CL_EXEC_STATUS_ERROR_FOR_EVENTS_IN_WAIT_LIST if the read
> + * and write operations are blocking and the execution status of any
> + * of the events in event_wait_list is a negative integer value.
> + */
> +
clReleaseMemObject(matrix_buffer)
EdB
> + return result;
> +}
More information about the Piglit
mailing list