anv: remove internal 'validate' layer
Presently the layer has only a single entry point. As mentioned by Jason the function does not validate anything that isn't checked elsewhere, thus we can drop the whole thing. Cc: "12.0" <mesa-stable@lists.freedesktop.org> Cc: Jason Ekstrand <jason@jlekstrand.net> Suggested-by: Jason Ekstrand <jason@jlekstrand.net> Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
This commit is contained in:
parent
3a9e6102b4
commit
b0d56f2f4f
|
@ -134,7 +134,6 @@ if opt_header:
|
|||
print "%s gen75_%s%s;" % (type, name, args)
|
||||
print "%s gen8_%s%s;" % (type, name, args)
|
||||
print "%s gen9_%s%s;" % (type, name, args)
|
||||
print "%s anv_validate_%s%s;" % (type, name, args)
|
||||
print_guard_end(name)
|
||||
exit()
|
||||
|
||||
|
@ -185,23 +184,24 @@ for type, name, args, num, h in entrypoints:
|
|||
print " \"vk%s\\0\"" % name
|
||||
offsets.append(i)
|
||||
i += 2 + len(name) + 1
|
||||
print """ ;
|
||||
print " ;"
|
||||
|
||||
/* Weak aliases for all potential validate functions. These will resolve to
|
||||
* NULL if they're not defined, which lets the resolve_entrypoint() function
|
||||
* either pick a validate wrapper if available or just plug in the actual
|
||||
* entry point.
|
||||
*/
|
||||
"""
|
||||
|
||||
# Now generate the table of all entry points and their validation functions
|
||||
# Now generate the table of all entry points
|
||||
|
||||
print "\nstatic const struct anv_entrypoint entrypoints[] = {"
|
||||
for type, name, args, num, h in entrypoints:
|
||||
print " { %5d, 0x%08x }," % (offsets[num], h)
|
||||
print "};\n"
|
||||
|
||||
for layer in [ "anv", "validate", "gen7", "gen75", "gen8", "gen9" ]:
|
||||
print """
|
||||
|
||||
/* Weak aliases for all potential implementations. These will resolve to
|
||||
* NULL if they're not defined, which lets the resolve_entrypoint() function
|
||||
* either pick the correct entry point.
|
||||
*/
|
||||
"""
|
||||
|
||||
for layer in [ "anv", "gen7", "gen75", "gen8", "gen9" ]:
|
||||
for type, name, args, num, h in entrypoints:
|
||||
print_guard_start(name)
|
||||
print "%s %s_%s%s __attribute__ ((weak));" % (type, layer, name, args)
|
||||
|
@ -214,27 +214,6 @@ for layer in [ "anv", "validate", "gen7", "gen75", "gen8", "gen9" ]:
|
|||
print "};\n"
|
||||
|
||||
print """
|
||||
#ifdef DEBUG
|
||||
static bool enable_validate = true;
|
||||
#else
|
||||
static bool enable_validate = false;
|
||||
#endif
|
||||
|
||||
/* We can't use symbols that need resolving (like, oh, getenv) in the resolve
|
||||
* function. This means that we have to determine whether or not to use the
|
||||
* validation layer sometime before that. The constructor function attribute asks
|
||||
* the dynamic linker to invoke determine_validate() at dlopen() time which
|
||||
* works.
|
||||
*/
|
||||
static void __attribute__ ((constructor))
|
||||
determine_validate(void)
|
||||
{
|
||||
const char *s = getenv("ANV_VALIDATE");
|
||||
|
||||
if (s)
|
||||
enable_validate = atoi(s);
|
||||
}
|
||||
|
||||
static const struct brw_device_info *dispatch_devinfo;
|
||||
|
||||
void
|
||||
|
@ -246,9 +225,6 @@ anv_set_dispatch_devinfo(const struct brw_device_info *devinfo)
|
|||
void * __attribute__ ((noinline))
|
||||
anv_resolve_entrypoint(uint32_t index)
|
||||
{
|
||||
if (enable_validate && validate_layer.entrypoints[index])
|
||||
return validate_layer.entrypoints[index];
|
||||
|
||||
if (dispatch_devinfo == NULL) {
|
||||
return anv_layer.entrypoints[index];
|
||||
}
|
||||
|
|
|
@ -332,81 +332,6 @@ void anv_GetImageSubresourceLayout(
|
|||
}
|
||||
}
|
||||
|
||||
VkResult
|
||||
anv_validate_CreateImageView(VkDevice _device,
|
||||
const VkImageViewCreateInfo *pCreateInfo,
|
||||
const VkAllocationCallbacks *pAllocator,
|
||||
VkImageView *pView)
|
||||
{
|
||||
ANV_FROM_HANDLE(anv_image, image, pCreateInfo->image);
|
||||
const VkImageSubresourceRange *subresource;
|
||||
|
||||
/* Validate structure type before dereferencing it. */
|
||||
assert(pCreateInfo);
|
||||
assert(pCreateInfo->sType == VK_STRUCTURE_TYPE_IMAGE_VIEW_CREATE_INFO);
|
||||
subresource = &pCreateInfo->subresourceRange;
|
||||
|
||||
/* Validate viewType is in range before using it. */
|
||||
assert(pCreateInfo->viewType >= VK_IMAGE_VIEW_TYPE_BEGIN_RANGE);
|
||||
assert(pCreateInfo->viewType <= VK_IMAGE_VIEW_TYPE_END_RANGE);
|
||||
|
||||
/* Validate format is in range before using it. */
|
||||
assert(pCreateInfo->format >= VK_FORMAT_BEGIN_RANGE);
|
||||
assert(pCreateInfo->format <= VK_FORMAT_END_RANGE);
|
||||
|
||||
/* Validate channel swizzles. */
|
||||
assert(pCreateInfo->components.r >= VK_COMPONENT_SWIZZLE_BEGIN_RANGE);
|
||||
assert(pCreateInfo->components.r <= VK_COMPONENT_SWIZZLE_END_RANGE);
|
||||
assert(pCreateInfo->components.g >= VK_COMPONENT_SWIZZLE_BEGIN_RANGE);
|
||||
assert(pCreateInfo->components.g <= VK_COMPONENT_SWIZZLE_END_RANGE);
|
||||
assert(pCreateInfo->components.b >= VK_COMPONENT_SWIZZLE_BEGIN_RANGE);
|
||||
assert(pCreateInfo->components.b <= VK_COMPONENT_SWIZZLE_END_RANGE);
|
||||
assert(pCreateInfo->components.a >= VK_COMPONENT_SWIZZLE_BEGIN_RANGE);
|
||||
assert(pCreateInfo->components.a <= VK_COMPONENT_SWIZZLE_END_RANGE);
|
||||
|
||||
/* Validate subresource. */
|
||||
assert(subresource->aspectMask != 0);
|
||||
assert(subresource->levelCount > 0);
|
||||
assert(subresource->layerCount > 0);
|
||||
assert(subresource->baseMipLevel < image->levels);
|
||||
assert(subresource->baseMipLevel + anv_get_levelCount(image, subresource) <= image->levels);
|
||||
assert(subresource->baseArrayLayer < image->array_size);
|
||||
assert(subresource->baseArrayLayer + anv_get_layerCount(image, subresource) <= image->array_size);
|
||||
assert(pView);
|
||||
|
||||
MAYBE_UNUSED const VkImageAspectFlags view_format_aspects =
|
||||
vk_format_aspects(pCreateInfo->format);
|
||||
|
||||
const VkImageAspectFlags ds_flags = VK_IMAGE_ASPECT_DEPTH_BIT
|
||||
| VK_IMAGE_ASPECT_STENCIL_BIT;
|
||||
|
||||
/* Validate format. */
|
||||
if (subresource->aspectMask & VK_IMAGE_ASPECT_COLOR_BIT) {
|
||||
assert(subresource->aspectMask == VK_IMAGE_ASPECT_COLOR_BIT);
|
||||
assert(image->aspects == VK_IMAGE_ASPECT_COLOR_BIT);
|
||||
assert(view_format_aspects == VK_IMAGE_ASPECT_COLOR_BIT);
|
||||
} else if (subresource->aspectMask & ds_flags) {
|
||||
assert((subresource->aspectMask & ~ds_flags) == 0);
|
||||
|
||||
assert(pCreateInfo->format == image->vk_format);
|
||||
|
||||
if (subresource->aspectMask & VK_IMAGE_ASPECT_STENCIL_BIT) {
|
||||
assert(image->aspects & VK_IMAGE_ASPECT_DEPTH_BIT);
|
||||
assert(view_format_aspects & VK_IMAGE_ASPECT_DEPTH_BIT);
|
||||
}
|
||||
|
||||
if (subresource->aspectMask & VK_IMAGE_ASPECT_STENCIL_BIT) {
|
||||
/* FINISHME: Is it legal to have an R8 view of S8? */
|
||||
assert(image->aspects & VK_IMAGE_ASPECT_STENCIL_BIT);
|
||||
assert(view_format_aspects & VK_IMAGE_ASPECT_STENCIL_BIT);
|
||||
}
|
||||
} else {
|
||||
assert(!"bad VkImageSubresourceRange::aspectFlags");
|
||||
}
|
||||
|
||||
return anv_CreateImageView(_device, pCreateInfo, pAllocator, pView);
|
||||
}
|
||||
|
||||
static struct anv_state
|
||||
alloc_surface_state(struct anv_device *device,
|
||||
struct anv_cmd_buffer *cmd_buffer)
|
||||
|
|
Loading…
Reference in New Issue