diff options
author | Dave Airlie <[email protected]> | 2016-10-12 08:37:14 +1000 |
---|---|---|
committer | Dave Airlie <[email protected]> | 2016-10-12 08:57:09 +1000 |
commit | 8cc9f89d2613da51db8f4d8516d450addf11c63f (patch) | |
tree | ece47bd6f2fd471526e3c8de614d0d994c0b5446 | |
parent | 014ec78fb26305b0d6ed1a9ae5e74915860df934 (diff) |
radv: remove the validation layer and some related bits.
As pointed out by Emil this isn't used in anv anymore,
and it was totally unused in radv anyways.
Signed-off-by: Dave Airlie <[email protected]>
-rw-r--r-- | src/amd/vulkan/radv_entrypoints_gen.py | 28 | ||||
-rw-r--r-- | src/amd/vulkan/radv_private.h | 11 |
2 files changed, 2 insertions, 37 deletions
diff --git a/src/amd/vulkan/radv_entrypoints_gen.py b/src/amd/vulkan/radv_entrypoints_gen.py index d6c49ce54cf..a6e832a0ab5 100644 --- a/src/amd/vulkan/radv_entrypoints_gen.py +++ b/src/amd/vulkan/radv_entrypoints_gen.py @@ -128,7 +128,6 @@ if opt_header: for type, name, args, num, h in entrypoints: print_guard_start(name) print "%s radv_%s%s;" % (type, name, args) - print "%s radv_validate_%s%s;" % (type, name, args) print_guard_end(name) exit() @@ -181,7 +180,7 @@ for type, name, args, num, h in entrypoints: i += 2 + len(name) + 1 print " ;" -# Now generate the table of all entry points and their validation functions +# Now generate the table of all entry points print "\nstatic const struct radv_entrypoint entrypoints[] = {" for type, name, args, num, h in entrypoints: @@ -196,7 +195,7 @@ print """ */ """ -for layer in [ "radv", "validate" ]: +for layer in [ "radv" ]: for type, name, args, num, h in entrypoints: print_guard_start(name) print "%s %s_%s%s __attribute__ ((weak));" % (type, layer, name, args) @@ -209,33 +208,10 @@ for layer in [ "radv", "validate" ]: 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); -} void * __attribute__ ((noinline)) radv_resolve_entrypoint(uint32_t index) { - if (enable_validate && validate_layer.entrypoints[index]) - return validate_layer.entrypoints[index]; - return radv_layer.entrypoints[index]; } """ diff --git a/src/amd/vulkan/radv_private.h b/src/amd/vulkan/radv_private.h index 4fd4848c9e8..6db8c7acfe3 100644 --- a/src/amd/vulkan/radv_private.h +++ b/src/amd/vulkan/radv_private.h @@ -39,7 +39,6 @@ #include <valgrind.h> #include <memcheck.h> #define VG(x) x -#define __gen_validate_value(x) VALGRIND_CHECK_MEM_IS_DEFINED(&(x), sizeof(x)) #else #define VG(x) #endif @@ -224,16 +223,6 @@ void radv_loge_v(const char *format, va_list va); #define radv_assert(x) #endif -/** - * If a block of code is annotated with radv_validate, then the block runs only - * in debug builds. - */ -#ifdef DEBUG -#define radv_validate if (1) -#else -#define radv_validate if (0) -#endif - void radv_abortf(const char *format, ...) radv_noreturn radv_printflike(1, 2); void radv_abortfv(const char *format, va_list va) radv_noreturn; |