| Age | Commit message (Collapse) | Author |
|
Comment out a couple of lines from the stats script that were
printing out debug information. Also update "str" var name to be
non-keyword "sub_str" and fix a broken print debug statement.
|
|
Have stats script count the un-implemented implicit checks and report
them out. Also report out the potential coverage number if all
un-implemented implicit checks are handled by parameter_validation.
|
|
Updated all implicit checks in database file to include the text
'implicit' in their note field.
spec.py will add 'implicit' to the note field for any new implicit
checks that are added, and will also print out a count of implicit
checks.
|
|
Change-Id: I37cfdb4b290dc8d5be7c606e32eb8d369ea43c6a
|
|
Modify the loader's trampoline and terminator
vkEnumeratePhysicalDevice functions to utilize helper functions.
Also, fix the failure in CTS.
Change-Id: Id9356fffd42f878c660ac8ee831e7527cce608bd
|
|
Change-Id: I71916c5300c24158901123f775b7b2fec3293e5b
|
|
Add exit code after failed mallocs.
Dynamically allocate gpu info and handle arrays, getting
rid of artificial limit. This also moves a large allocation
from the stack to the heap.
There are still some false positives from the MSFT analyzer
that aren't worth fixing or suppressing.
|
|
Running smoketest with --flush will call vkFlushMappedMemoryRanges so
that the trace layer is aware of modified memory. This is not
necessary with the --PMB option in vktrace.
Change-Id: If72c0ed0c651cf82f2f2c4c78ec2534a0d475a9c
|
|
Change-Id: I153a68206db8d5f346755f0691bdca777908d748
|
|
PreCallValidateCreateSwapchainKHR is now used to validate
CreateSwapchain as well as CreateSharedSwapchains. Plumbed the
functionName down through the validation code so users can see the
faulting API and the bad swapchain index.
Change-Id: Ieb885704bef8a73c09e6670594e96494b62cb1a4
|
|
Added PreCallValidate and PostCallRecord functions for this API.
Change-Id: Iebf8eb980dadbb29cceffd6a9fefb5340939eb0f
|
|
Added a PostCallRecord function for this API.
Change-Id: I6f9e67ea5fbcf1ee4f54043c5269ad282339a9db
|
|
Change-Id: I1e382aefdd13ed6c575d3654962e09e9558af97f
|
|
Add more logging messages on Error cases to help debugging.
Change-Id: I78b6bd7c73d7234283bf94d745c8733d59752422
|
|
Change-Id: I95b1c346d3611c6d6b3a22f108b4b5ff9c313e0c
|
|
This functionality is now generated entirely from vk.xml.
Change-Id: Ie3385f924b6ec9a8d74f2ad6fed7064ab0dc0534
|
|
Change-Id: Ia89cfec2b836437c2f2009608795d1ee38b811ad
|
|
Change-Id: Idc0e76aa479d2c9f3e096a31f620eb96d843849d
|
|
Change-Id: I6adc6a8ac07f2ac11814a6f8cd06b7d5b0c49797
|
|
Change-Id: I1f3c28b7737e58731206e93af50fb1d18cd4f937
|
|
Change-Id: Ie4171bb18ff499e54bdbd8ff7237e7026e6cf3e9
|
|
Change-Id: I87de80173ff414202f6e876db3da688466f3598d
|
|
Change-Id: Ie972f2d8b6bca633fbfb4745f29eab183cde7acb
|
|
Change-Id: I060e8bb0eddda0c94bdaed7e04fddac8bf9d5822
|
|
Change-Id: Iffd19369976ac15ad54251e15a7f49e061a807e8
|
|
Unnecessary lock was grabbed just before an API call.
Change-Id: I0ef561d9b620f42333ba034dcbc87cba6daea776
|
|
Following the audit of existing Valid Usage enums in the LVL codebase,
we can claim that there are no Valid Usage checks in an unknown
state; the existing VU checks are either implemented or not.
This commit resets the check_implemented flags in the VU database,
such that any remaining "U" (Unknown) entries now become "N" (Not
implemented).
Going forward, the check_implemented status flag must be either "Y"
or "N"; "U" is no longer permitted.
Change-Id: Iaf1807cadf02d57e3fa5676108e290d3bab9b271
|
|
Change-Id: I31c32e56ae09134b7b91c610b546a9d46c20c007
|
|
|
|
This fixes the possibility of spurious validation errors like the following,
when attempting to transition multiple aspects of an image after only a subset
of those aspects have previously been used in the command buffer:
"Cannot query for VkImage 0x599 layout when combined aspect mask 6 has
multiple initial layout types: VK_IMAGE_LAYOUT_TRANSFER_DST_OPTIMAL and
VK_IMAGE_LAYOUT_DEPTH_STENCIL_ATTACHMENT_OPTIMAL"
The specific case where this bug was encountered was where both the depth and
stencil aspects of an image were initially in the TRANSFER_DST_OPTIMAL layout.
The first command in the command buffer to reference the image was a
vkCmdClearDepthStencilImage on only the depth aspect of the image, followed
later by a vkCmdPipelineBarrier to transition both aspects to the
DEPTH_STENCIL_ATTACHMENT_OPTIMAL layout.
Since TransitionImageLayouts tries to look up the initial state for all
aspects at the same time, it was picking up the state based on the depth
aspect because of the previous use of that (which had both initialLayout
and layout set to TRANSFER_DST_OPTIMAL), and then calling SetLayout for all
aspects with only the new layout. Since the stencil aspect didn't have any
currently recorded state, it was being added with both initialLayout and
layout set to the new layout, causing a mismatch between the initialLayout for
the two aspects and therefore the spurious error above.
Fix by updating the state for each aspect individually.
|
|
Struct size helper files are now generated by the helper_file_generator
script.
Change-Id: Ifcb0cceb9621f3f4bb41d948e9f0bb3fca7a0ea1
|
|
Change-Id: I264d7ecca323089bda1583dbcf9eb751736146a7
|
|
Changed enum_string_helper target to be generic for multiple files.
Change-Id: I760eb634839228211554686aa92c9771602786bb
|
|
Remove old references, add calls to new generator.
Change-Id: I64ab859e4ae6893870697f16ee6fd596b8a7908b
|
|
This header is unused by the module.
Change-Id: Ife7a1b8eb344e2a670e1cfc9ace3e0697c97cd7e
|
|
The vk.xml file contains latex-formatted metadata which must be
parsed for codegen.
Change-Id: Ie64c9974371d2bb66ba460165a0c23505991ebd7
|
|
Change-Id: I1afcee1ab90ad18d758fdb476d2d0012a891bdfc
|
|
This file will now output a complete vk_struct_size_helper.c file.
Change-Id: Ic87fd7d3c9a95daa666e8e4da63a21d31e8bfb86
|
|
First level code-gen, move some code into functions.
Change-Id: Idf8ee44a1f74c4117aed74b8df5b548df618e790
|
|
Added generator-side plumbing to create the vk_struct_size_helper.h
file.
Change-Id: I5ff226131b8db2d773e4422877bdb2993ed76cf4
|
|
Added struct processing guts for building the struct_size_helper
header and source files.
Change-Id: I8172b921b670b47b8fd66499de6d76ddb3ef934d
|
|
Added parsing of option, conditional file generation to helper file
generator.
Change-Id: Ia7c51d50fb1785f3002dc2868f98d8e384d0e36e
|
|
The helper file generator will be used to output multiple helper
files. Added passing of parameter and test call for this feature
to the generator options block.
Change-Id: I1e417ff50628f395c589282ed6f96f0340758f23
|
|
Update valid usage enums in subsection of core validation from
validate_and_capture_pipeline_shader_state() to
checkGraphicsOrComputeBit() inclusive.
Change-Id: I8b71c63de26e42eb073a1eac1abd121d5e57f5c7
|
|
This is already being checked in the object tracker layer.
Change-Id: Ie1d5776810c3f8a3df7b9ef920d02e90c0d239b4
|
|
Change-Id: I9aba0480aad5b565eb59f642def7a11e46822566
|
|
Update Valid Usage enums in parameter_validation.cpp.
Update VU status in the database.
Completes Jira task VL-69.
Change-Id: Ie16c41dd0ad8f264b62b3d5b60d100d7b9b3f229
|
|
Reduce the scope of internal helper functions by
declaring them static.
This is only changes lines related to VL-69,
rather than a full sweep of the code.
Change-Id: I3593e519d11066cd8631f56131de39e6e3b73f50
|
|
For consistancy, Vulkan function names in log_msg strings should
include the vk prefix and end with parens.
Also fix some typos and cut/paste errors.
This only touches lines related to VL-69; it is not a sweep
of all instances in the code.
Change-Id: I41a8f0c9d3b93203f67aef3fa570eb8d95725d31
|
|
Update Valid Usage enums in a subsection of core_validation.cpp.
Refactored some helper functions to add parameter for msgCode.
Update VU status in the database.
Completes Jira task VL-63.
Change-Id: Iba2cfb978a044d6affb21df9b02a2ac852dfba4e
|