[Mesa-dev] [PATCH 3/3] clover: Query drivers for max clock frequency
From: Tom Stellard thomas.stell...@amd.com v2: PIPE_COMPUTE_CAP_MAX_CLOCK_FREQUENCY instead of PIPE_COMPUTE_MAX_CLOCK_FREQUENCY Signed-off-by: Igor Gnatenko i.gnatenko.br...@gmail.com clover: Correct the returned value for max_clock_frequency According to OpenCL 1.1 spec, the returned value must be in MHz, and we were returning it in kHz v3: Now the conversion is done in patch 2. --- src/gallium/state_trackers/clover/api/device.cpp | 2 +- src/gallium/state_trackers/clover/core/device.cpp | 6 ++ src/gallium/state_trackers/clover/core/device.hpp | 1 + 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/api/device.cpp b/src/gallium/state_trackers/clover/api/device.cpp index 7bc8d0a..f980845 100644 --- a/src/gallium/state_trackers/clover/api/device.cpp +++ b/src/gallium/state_trackers/clover/api/device.cpp @@ -122,7 +122,7 @@ clGetDeviceInfo(cl_device_id d_dev, cl_device_info param, break; case CL_DEVICE_MAX_CLOCK_FREQUENCY: - buf.as_scalarcl_uint() = 0; + buf.as_scalarcl_uint() = dev.max_clock_frequency(); break; case CL_DEVICE_ADDRESS_BITS: diff --git a/src/gallium/state_trackers/clover/core/device.cpp b/src/gallium/state_trackers/clover/core/device.cpp index 2c5f9b7..2f84677 100644 --- a/src/gallium/state_trackers/clover/core/device.cpp +++ b/src/gallium/state_trackers/clover/core/device.cpp @@ -154,6 +154,12 @@ device::max_mem_alloc_size() const { PIPE_COMPUTE_CAP_MAX_MEM_ALLOC_SIZE)[0]; } +cl_uint +device::max_clock_frequency() const { + return get_compute_paramuint32_t(pipe, + PIPE_COMPUTE_CAP_MAX_CLOCK_FREQUENCY)[0]; +} + std::vectorsize_t device::max_block_size() const { auto v = get_compute_paramuint64_t(pipe, PIPE_COMPUTE_CAP_MAX_BLOCK_SIZE); diff --git a/src/gallium/state_trackers/clover/core/device.hpp b/src/gallium/state_trackers/clover/core/device.hpp index 433ac81..3662c6b 100644 --- a/src/gallium/state_trackers/clover/core/device.hpp +++ b/src/gallium/state_trackers/clover/core/device.hpp @@ -61,6 +61,7 @@ namespace clover { cl_uint max_const_buffers() const; size_t max_threads_per_block() const; cl_ulong max_mem_alloc_size() const; + cl_uint max_clock_frequency() const; std::vectorsize_t max_block_size() const; std::string device_name() const; -- 1.9.2 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Re: [Mesa-dev] [PATCH 3/3] clover: Query drivers for max clock frequency
Bruno Jiménez brunoji...@gmail.com writes: From: Tom Stellard thomas.stell...@amd.com v2: PIPE_COMPUTE_CAP_MAX_CLOCK_FREQUENCY instead of PIPE_COMPUTE_MAX_CLOCK_FREQUENCY Signed-off-by: Igor Gnatenko i.gnatenko.br...@gmail.com clover: Correct the returned value for max_clock_frequency According to OpenCL 1.1 spec, the returned value must be in MHz, and we were returning it in kHz v3: Now the conversion is done in patch 2. The commit message looks kind of funny, but other than that the patch looks good to me. Reviewed-by: Francisco Jerez curroje...@riseup.net --- src/gallium/state_trackers/clover/api/device.cpp | 2 +- src/gallium/state_trackers/clover/core/device.cpp | 6 ++ src/gallium/state_trackers/clover/core/device.hpp | 1 + 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/api/device.cpp b/src/gallium/state_trackers/clover/api/device.cpp index 7bc8d0a..f980845 100644 --- a/src/gallium/state_trackers/clover/api/device.cpp +++ b/src/gallium/state_trackers/clover/api/device.cpp @@ -122,7 +122,7 @@ clGetDeviceInfo(cl_device_id d_dev, cl_device_info param, break; case CL_DEVICE_MAX_CLOCK_FREQUENCY: - buf.as_scalarcl_uint() = 0; + buf.as_scalarcl_uint() = dev.max_clock_frequency(); break; case CL_DEVICE_ADDRESS_BITS: diff --git a/src/gallium/state_trackers/clover/core/device.cpp b/src/gallium/state_trackers/clover/core/device.cpp index 2c5f9b7..2f84677 100644 --- a/src/gallium/state_trackers/clover/core/device.cpp +++ b/src/gallium/state_trackers/clover/core/device.cpp @@ -154,6 +154,12 @@ device::max_mem_alloc_size() const { PIPE_COMPUTE_CAP_MAX_MEM_ALLOC_SIZE)[0]; } +cl_uint +device::max_clock_frequency() const { + return get_compute_paramuint32_t(pipe, + PIPE_COMPUTE_CAP_MAX_CLOCK_FREQUENCY)[0]; +} + std::vectorsize_t device::max_block_size() const { auto v = get_compute_paramuint64_t(pipe, PIPE_COMPUTE_CAP_MAX_BLOCK_SIZE); diff --git a/src/gallium/state_trackers/clover/core/device.hpp b/src/gallium/state_trackers/clover/core/device.hpp index 433ac81..3662c6b 100644 --- a/src/gallium/state_trackers/clover/core/device.hpp +++ b/src/gallium/state_trackers/clover/core/device.hpp @@ -61,6 +61,7 @@ namespace clover { cl_uint max_const_buffers() const; size_t max_threads_per_block() const; cl_ulong max_mem_alloc_size() const; + cl_uint max_clock_frequency() const; std::vectorsize_t max_block_size() const; std::string device_name() const; -- 1.9.2 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev pgpjt9vbV6Xcq.pgp Description: PGP signature ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
[Mesa-dev] [PATCH 3/3] clover: Query drivers for max clock frequency
From: Tom Stellard thomas.stell...@amd.com v2: PIPE_COMPUTE_CAP_MAX_CLOCK_FREQUENCY instead of PIPE_COMPUTE_MAX_CLOCK_FREQUENCY Signed-off-by: Igor Gnatenko i.gnatenko.br...@gmail.com clover: Correct the returned value for max_clock_frequency According to OpenCL 1.1 spec, the returned value must be in MHz, and we were returning it in kHz --- src/gallium/state_trackers/clover/api/device.cpp | 2 +- src/gallium/state_trackers/clover/core/device.cpp | 6 ++ src/gallium/state_trackers/clover/core/device.hpp | 1 + 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/clover/api/device.cpp b/src/gallium/state_trackers/clover/api/device.cpp index 7bc8d0a..f980845 100644 --- a/src/gallium/state_trackers/clover/api/device.cpp +++ b/src/gallium/state_trackers/clover/api/device.cpp @@ -122,7 +122,7 @@ clGetDeviceInfo(cl_device_id d_dev, cl_device_info param, break; case CL_DEVICE_MAX_CLOCK_FREQUENCY: - buf.as_scalarcl_uint() = 0; + buf.as_scalarcl_uint() = dev.max_clock_frequency(); break; case CL_DEVICE_ADDRESS_BITS: diff --git a/src/gallium/state_trackers/clover/core/device.cpp b/src/gallium/state_trackers/clover/core/device.cpp index 2c5f9b7..4ae575f 100644 --- a/src/gallium/state_trackers/clover/core/device.cpp +++ b/src/gallium/state_trackers/clover/core/device.cpp @@ -154,6 +154,12 @@ device::max_mem_alloc_size() const { PIPE_COMPUTE_CAP_MAX_MEM_ALLOC_SIZE)[0]; } +cl_uint +device::max_clock_frequency() const { + return get_compute_paramuint32_t(pipe, + PIPE_COMPUTE_CAP_MAX_CLOCK_FREQUENCY)[0] / 1000; +} + std::vectorsize_t device::max_block_size() const { auto v = get_compute_paramuint64_t(pipe, PIPE_COMPUTE_CAP_MAX_BLOCK_SIZE); diff --git a/src/gallium/state_trackers/clover/core/device.hpp b/src/gallium/state_trackers/clover/core/device.hpp index 433ac81..3662c6b 100644 --- a/src/gallium/state_trackers/clover/core/device.hpp +++ b/src/gallium/state_trackers/clover/core/device.hpp @@ -61,6 +61,7 @@ namespace clover { cl_uint max_const_buffers() const; size_t max_threads_per_block() const; cl_ulong max_mem_alloc_size() const; + cl_uint max_clock_frequency() const; std::vectorsize_t max_block_size() const; std::string device_name() const; -- 1.9.2 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev