| Age | Commit message (Collapse) | Author |
|
Change-Id: I80770b7ab14795316400279dc4e7c954fa5ad374
|
|
Change reinterpret_cast<uint64_t> to reinterpret_cast<uint64_t&>.
Change-Id: I87b8479df6ba7f93b922bab11edbbf6c4fbfe7f6
|
|
|
|
Changed the type of the string validation constants to from char to
uint8_t to fix warning C4309: 'initializing' : truncation of constant
value, due to an implicit unsgined char to signed char conversion.
Change-Id: I86cca4b1a8c3f818a2e7a9a7e38f05ee8cd01150
|
|
Fixes the following MSVS warnings:
- C4800 - forcing value to bool 'true' or 'false'
- C4244 - conversion from type1 to type2, possible loss of data
- C4267 - conversion from size_t to type, possible loss of data
Change-Id: I78576c48c5d42e4c9116a83c5f4a4f4a5035e0fc
|
|
Much of this was eventually going to go away anyway, but getting rid of
this thread of it was trivial.
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
mem_tracker layer had specific maps to track memory binding for images
and buffers. This change merges the data from those maps into IMAGE_NODE
and BUFFER_NODE respectively and eliminates the binding maps.
|
|
Correct comment for VkLayer*CreateInfo's sType
|
|
dEQP-VK.api.object_management.alloc_callback_fail.device
When the loader vkCreateDevice trampoline code calls down to
vkCreateDevice don't update the dev->device field with a bogus device
handle.
Change-Id: I703acb5283650ab0013483bda0f478da9b46e3ea
|
|
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
|
|
- Track whether we've seen anything disqualifying -- non-stripped
arrays, matrices, structures, etc.
- Add relaxed vec/scalar and vec/vec matches if not disqualified.
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
Add explicit VK_FALSE/VK_TRUE conditional checks.
Change-Id: Idb5c82ce53d54c9f2c378dc557152efc695ae2b1
|
|
Fixes the MSVS warnings regarding unsafe mix of VkBool32 and bool for the
device_limits, image, parameter_validation, and threading layers.
Converts all Boolean usage internal to the layers to bool/true/false. The
VK_TRUE and VK_FALSE values continue to be used with the Vulkan API
components.
Change-Id: I612d3f26050fadcd77ffca4d8723a7c734613816
|
|
Fixes the MSVS warnings regarding unsafe mix of VkBool32 and bool for the
core_validation layer. Converts all Boolean values internal to the layer
to bool/true/false. The VK_TRUE and VK_FALSE values continue to be used
with the Vulkan API components.
Change-Id: I9a298e2a9f5f92a388a2b7c158451d237fdb4fc0
|
|
Change-Id: I67ebce2e4e46a5a1439bb35fb966ed522c80ee52
|
|
Also added execute permission to CreateInstallerRT.sh
|
|
This problem was introduced when logging was added.
|
|
Change-Id: I6c9e9b7650a7599797e93e60eb2191345c3e6ec8
|
|
Change-Id: Ibb9ffa80bbb71d15296e39730f48f33c23bd331a
|
|
These functions were returning VK_ERROR_VALIDATION_FAILED_EXT,
which was giving a warning in Visual Studio. The app shouldn't be
checking for non-bool return values of these functions.
Change-Id: I9924a25b1c49ad6973c8915be1dd7aea4cc0edf4
|
|
This fixes a bug where on Windows, windows.h would include
winsock.h before winsock2.h, causing numerous warnings.
Change-Id: I8eba1497385dcee6ebb201c74fe48268e6663b9c
|
|
warning C4267: conversion, possible loss of data
warning C4389: signed/unsigned mismatch
warning C4701: potentially uninitialized local variable used
warning C4702: unreachable code
Change-Id: Ibc2ce3825a986c3701f6e0e6244a0e30fbf7e87c
|
|
PIPELINE_NODE was storing ptrs from the app and has some ugly initialization code
for the gfx pipeline create struct. This commit switches pipeline node to use
safe_struct wrappers for Gfx and Compute create info.
There were also some bugs found in the safe_struct wrappers making this migration
so those are cleaned up in this commit as well.
|
|
Also add group id check to getenv suid check in loader.
Change-Id: Icbc08258498f893ee5fce144c043bdc6bd8e5423
|
|
|
|
|
|
warning 4457: declaration of 'fence' hides function parameter
|
|
warning 4577: 'noexcept' used with no exception handling mode specified
|
|
Fixes an issue where the generated parameter_validation code could
index a NULL pointer to an array of structs.
Change-Id: If78f1c97ea256911d857b0b8009b38e6ca6f2581
|
|
This reverts commit dcce8143110568353a6217e5b885da407edd29be.
Change-Id: I0882b499b38e3cf7670846a9ffae48a0fbf015a3
|
|
Change-Id: I155ee469ef98d1ad9b1bfc8813a1cff43f30f8ca
|
|
We have an existing check to make sure that a set has been updated, but there was
still a hole where individual slots may not have an update. This adds an additional
check to make sure that we flag any slots that are used by not updated at draw time.
|
|
Change-Id: Ic35eb444c34004343e5fb19cc54f12518512c6be
|
|
Change-Id: I809110e8483d1877f5106551d9f066ed6c25d8bc
|
|
|
|
There were two places where this script called super, one with
Python 2 and 3 compatible syntax and one with Python 3 only. Fix
it to use the compatible syntax in both locations.
|
|
Missed this from a rebase conflict. Kill top-level cmd buffer node
dynamicOffsets vector and only use vector based off of lastBound data.
|
|
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
|
|
|
|
|
|
Typo allowed object access for invalid QueryToStateMap objects
Change-Id: I50845ca09564ea8b77173d2165a5318160dd8d7e
|
|
Fix buffer/descriptor offset alignment checks in device_limits:
- vkUpdateDescriptorSets offset check was not performed with
the correct physical device alignments.
- vkBindBufferMemory offset check was always performed with the
UBO alignment. This check has been moved to core_validation,
where the offset is checked against the alignment associated
with its usage type and the alignment retrieved from
vkGetBufferMemoryRequirements.
Change-Id: I4f64e06c44972a224d1930cb0798a9f472b768f2
|
|
Fix parameter validation issue with arrays of structs, where only
the first item in the array was processed. Validation messages
for arrays of structs will currently print the array variable name
with the letter 'i' as the array index instead of the actual index
number, producing a value like 'pCreateInfos[i]'. This will be
fixed by a future update.
Change-Id: I2cb773c23c6515a20ac8cb066e7cc61028e569f8
|
|
VkResult validation now prints a description of the error
code being returned.
Change-Id: Ia6dc4e71ef6d1b27365e3c0499b24a3fc1d332ac
|