aboutsummaryrefslogtreecommitdiff
path: root/layers/parameter_validation.cpp
diff options
context:
space:
mode:
authorKarl Schultz <karl@lunarg.com>2016-10-06 17:53:48 -0600
committerKarl Schultz <karl@lunarg.com>2016-10-11 08:49:33 -0600
commit8788c712a463c269066fb6486fe2831b83cc03ce (patch)
treed6dfa25944f15a53c7c3811152742ebf9adc2e2c /layers/parameter_validation.cpp
parent82e3c33710a6d8f48943c5c787e6a05a40a47d7f (diff)
downloadusermoji-8788c712a463c269066fb6486fe2831b83cc03ce.tar.xz
layers: Updating some CreateBuffer checks to use unique enums
Small batch of updates to use unique validation error enums and messages for a few checks. Updated the database file and flagged some other checks that are missing with the error enums and TODO notes. Change-Id: I2020a88bbdcf7ea6d725f667523e89b160fe7ab8
Diffstat (limited to 'layers/parameter_validation.cpp')
-rw-r--r--layers/parameter_validation.cpp25
1 files changed, 19 insertions, 6 deletions
diff --git a/layers/parameter_validation.cpp b/layers/parameter_validation.cpp
index 59df9d85..1306217f 100644
--- a/layers/parameter_validation.cpp
+++ b/layers/parameter_validation.cpp
@@ -2361,6 +2361,17 @@ VKAPI_ATTR VkResult VKAPI_CALL CreateBuffer(VkDevice device, const VkBufferCreat
assert(device_data != nullptr);
debug_report_data *report_data = device_data->report_data;
+ // TODO: Add check for VALIDATION_ERROR_00660
+ // TODO: Add check for VALIDATION_ERROR_00661
+ // TODO: Add check for VALIDATION_ERROR_00662
+ // TODO: Add check for VALIDATION_ERROR_00670
+ // TODO: Add check for VALIDATION_ERROR_00671
+ // TODO: Add check for VALIDATION_ERROR_00672
+ // TODO: Add check for VALIDATION_ERROR_00673
+ // TODO: Add check for VALIDATION_ERROR_00674
+ // TODO: Add check for VALIDATION_ERROR_00675
+ // TODO: Note that the above errors need to be generated from the next function, which is codegened.
+ // TODO: Add check for VALIDATION_ERROR_00663
skip_call |= parameter_validation_vkCreateBuffer(report_data, pCreateInfo, pAllocator, pBuffer);
if (pCreateInfo != nullptr) {
@@ -2368,20 +2379,22 @@ VKAPI_ATTR VkResult VKAPI_CALL CreateBuffer(VkDevice device, const VkBufferCreat
if (pCreateInfo->sharingMode == VK_SHARING_MODE_CONCURRENT) {
// If sharingMode is VK_SHARING_MODE_CONCURRENT, queueFamilyIndexCount must be greater than 1
if (pCreateInfo->queueFamilyIndexCount <= 1) {
- skip_call |=
- log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0, __LINE__,
- INVALID_USAGE, LayerName, "vkCreateBuffer: if pCreateInfo->sharingMode is VK_SHARING_MODE_CONCURRENT, "
- "pCreateInfo->queueFamilyIndexCount must be greater than 1");
+ skip_call |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
+ __LINE__, VALIDATION_ERROR_00665, LayerName,
+ "vkCreateBuffer: if pCreateInfo->sharingMode is VK_SHARING_MODE_CONCURRENT, "
+ "pCreateInfo->queueFamilyIndexCount must be greater than 1. %s",
+ validation_error_map[VALIDATION_ERROR_00665]);
}
// If sharingMode is VK_SHARING_MODE_CONCURRENT, pQueueFamilyIndices must be a pointer to an array of
// queueFamilyIndexCount uint32_t values
if (pCreateInfo->pQueueFamilyIndices == nullptr) {
skip_call |= log_msg(report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_UNKNOWN_EXT, 0,
- __LINE__, REQUIRED_PARAMETER, LayerName,
+ __LINE__, VALIDATION_ERROR_00664, LayerName,
"vkCreateBuffer: if pCreateInfo->sharingMode is VK_SHARING_MODE_CONCURRENT, "
"pCreateInfo->pQueueFamilyIndices must be a pointer to an array of "
- "pCreateInfo->queueFamilyIndexCount uint32_t values");
+ "pCreateInfo->queueFamilyIndexCount uint32_t values. %s",
+ validation_error_map[VALIDATION_ERROR_00664]);
}
// Ensure that the queue family indices were specified at device creation