Skip to content

Commit 0d2663e

Browse files
committed
remove invalid capability error condition for clGetSVMSuggestedTypeIndexKHR
1 parent 97a43e1 commit 0d2663e

1 file changed

Lines changed: 10 additions & 2 deletions

File tree

extensions/cl_khr_unified_svm.asciidoc

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -809,8 +809,6 @@ The suggested SVM type may be {CL_UINT_MAX}, indicating that there is no SVM all
809809
** if the value specified for a supported property name is not valid
810810
** if the same property name is specified more than once
811811
* {CL_INVALID_VALUE}
812-
** if _required_capabilities_ contains an invalid SVM capability
813-
** if _desired_capabilities_ contains an invalid SVM capability
814812
** if _suggested_svm_type_index_ is `NULL`
815813
* {CL_OUT_OF_RESOURCES}
816814
** if there is a failure to allocate resources required by the OpenCL
@@ -1282,6 +1280,16 @@ This extension does not support initialize an allocation with both the *HOST_NOW
12821280
If desired, a layered extension could add a new property to {clSVMAllocWithPropertiesKHR} that would specify a pointer with the initial contents of an SVM allocation with both the *HOST_NOWRITE* and *DEVICE_NOWRITE* access flags.
12831281
--
12841282

1283+
. What should happen if an unsupported or unknown capability is passed as a required capability to {clGetSVMSuggestedTypeIndexKHR}?
1284+
+
1285+
--
1286+
`RESOLVED`: If an unknown or unsupported capability is passed as a required capability to {clGetSVMSuggestedTypeIndexKHR}, then the returned suggested SVM type index must be {CL_UINT_MAX}, since there is no SVM allocation type that supports the capability.
1287+
Note that this is not an error, and it is also not an error to pass an unsupported or unknown capability as a desired capability.
1288+
If this were an error instead, then it would make {clGetSVMSuggestedTypeIndexKHR} harder to use, because it would require checking for the set of supported capabilities before performing the query to ensure no error is generated.
1289+
1290+
This behavior is tested by the CTS test `unified_svm_api_suggested_type_index`.
1291+
--
1292+
12851293
== Revision History
12861294

12871295
[cols="5,15,15,70"]

0 commit comments

Comments
 (0)