| Age | Commit message (Collapse) | Author |
|
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
Change-Id: I8df42b71a40318b6d46a6b65cc9d8060a8f912a8
|
|
Layer and ICD libraries are opened and closed as needed. No ref counting,
no knowledge of if a library contains multiple layers or ICDs.
Change-Id: Ie88b671cd1671187a42d3d838d20e3af1afc67cc
|
|
In previous commit we modified build.gradle to be cross platform,
so we can stop using the windows specific version.
|
|
Use gcc instead if building on Windows.
|
|
This commit cleans up the platform detection code that was emitting some
(but not all) ifdef wrappers around platform specific code. It changes
the logic to wrap all the functions that are platform specific. This
allows us to use these generated files on multiple platforms, not just
the platform is was generated on.
|
|
|
|
Add support for validating that required handle parameters are not
specified as VK_NULL_HANDLE to the parameter validation layer's code
generator.
- Add new parameter validation utility functions to validate required
handles and arrays of handles.
- Add new parameter validation layer entrypoints for functions that
previously had no parameters to validate.
- Add handle validation logic to the parameter validation code generator.
Change-Id: I7a5680954245db4c1b12587f78e30e17c3903d6c
|
|
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
Fixes GH #330.
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
Fixes GH #377.
Signed-off-by: Chris Forbes <chrisforbes@google.com>
|
|
If a descriptor is an immutable sampler of type VK_DESCRIPTOR_TYPE_SAMPLER
then it doesn't need to be explicitly updated so don't flag errors in this case.
|
|
|
|
|
|
|
|
|
|
|
|
Change-Id: Id447f4c8c53d33f9192940bdd3ae727b7b738a97
|
|
Properly track commandBuffer/FrameBuffer references.
Change-Id: Ibbc078b582b204bee26cdc510575c57023cf9d6b
|
|
|
|
A couple of bugs in object_tracker hand-coded sections that MarkY tracked down.
In DestroyInstance case we were attempting to delete wrong map key.
Then, for vkDestroyCommandPool we were using wrong function to clean-up the
command buffers in that pool which would result in occassional crashes.
|
|
|
|
- Only validate that VkPipelineColorBlendStateCreateInfo::logicOp is
a valid VkLogicOp enumeration value when
VkPipelineColorBlendStateCreateInfo::logicOpEnable is set to VK_TRUE.
- Remove redundant VkPipelineColorBlendStateCreateInfo validation (hand
written code replaced by generated code).
Change-Id: I075c58342678219c5ed1dd5c401e0da2cea53232
|
|
Improvements to the parameter validation code generator, primarily to
eliminate string concatenations performed when processing structs. A
validation function was being produced for each struct, which received a
string specifying the name of the struct parameter. This string was then
concatenated with the names of the struct members. The struct validation
code is no longer in a separate function, and a single string containing
the full names of each struct member (eg. pCreateInfo->pApplicationInfo)
is created at code generation time.
Change-Id: I2800bfbc26247e4e4c6f8c885c0e7c4b8b3776a2
|
|
- Remove unnecessary conditional checks for input vs. output parameters
from the generated C++ code.
- Split some generator code into smaller functions.
Change-Id: I32e47d417ab884e5cc0fc7af40cb5657b39c176d
|
|
Change-Id: I88af1e86c0ef93dbeab303977b8190ad1968c00d
|
|
Change-Id: I2c5584a5d36edc097b061db2e49fba2505bd1d65
|
|
Change-Id: I5f9ed9dc92daccaada895808e3fea3189ef81212
|
|
Change-Id: I773ea148807e5b634ad021ea58b001cd3d1bad87
|
|
Change-Id: Ie132289c0e601397f54b06c1ef758c65b712f906
|
|
Secondary command buffers are considered executing, or "in-flight"
at the time they're bound into a primary command buffer. This leads
to a couple of corner case bugs if the primary command buffer is
never submitted on a queue. This change fixes those bugs:
1. If a secondary cmd buffer is being freed and is in the global
in-flight set, remove it if associated primary is not in-flight
2. When a primary cmd buffer is reset, remove its secondary
cmd buffers from in-flight
|
|
When we reset descriptor pool, need to remove the allocated
descriptors from setMap and free them.
|
|
This reverts commit 884065fcd89540dca9a710389641bf1c39cedce0.
|
|
This reverts commit 23b248d32c943c136589e679e1fcc3444488d906.
|
|
Change-Id: I2020999bac6090fe40688f977f15c9818b828cf4
|
|
No functional changes.
Change-Id: Ia57a172a2ab0c7e082ad30aef64354cd1427b528
|
|
These are not needed since rely on OSes library loader to figure out default path.
Change-Id: Ide5eec4ed4301c51de7dc9171f7e040deb1878e7
|
|
|
|
Last several commits to loader weren't run through it.
Change-Id: I512b2df75c7831ee6433e4c60c6664db85184584
|
|
Change reinterpret_cast<uint64_t> to reinterpret_cast<uint64_t&> for
unique_objects generated and static code.
Change-Id: I8be5680063ce8a26b58c444e59cb325c4cf6e98b
|
|
This makes it more like Linux which leaves it up to the system
LoadLibrary() call to find the .dll as specified in the .json files.
Hard-coding "C:" or any other search path creates the possibility of
picking up the wrong .dll. The LoadLibrary() implementation does the
right thing.
Change-Id: I58cbdf230a1e86481067efb4aec03447ee55f1ff
|
|
Change-Id: I68ac2681b7c4923213c65dc720f54d81a36c99ef
|
|
Check only the valid dimensions for the imageType provided in
the create info.
Change-Id: I11d484caf1085b7b0ae4a402453b5fdcadd7b67d
|
|
Unique objects now uses a 64-bit counter as the unique handle returned
to the app. It stores an internal mapping of handles to actual object
ptrs that are used for all non-dispatchable objects. This virtually
eliminates the possibility of having a repeat handle as objects
are created and destroyed.
All counter increments and map accesses are protected by a mutex.
|
|
|
|
|
|
Previously were only validating *_ONE_TIME_SUBMIT_BIT for primary cmd buffers.
After clarification on spec, moving the check so it will also hit secondary
command buffers.
|
|
Tobin discovered an issue with my previous change. This resolves
the issue with the blend state targeting only a specific subpass
and verifying the subpass attachment count matches the blend
attachment count.
Change-Id: Iacdd880dde32267da095a3e9baa9e8eb66fdc760
|
|
Add a drawtime check to validate that storage or uniform buffers
have valid memory bound to them at draw time before using.
Change-Id: Ia0a07f221dfcb2ea4315e4f52ee04d7cdda22cf7
|
|
If sub-resource data is present, print out that info as part of
the error message when the image layout is incorrect.
Change-Id: Ibd4d95e22d846b823b5c76986e8646624c109bc6
|