Mesa (master): intel/compiler: Validate fuzzed instructions
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Wed Jan 22 00:51:23 UTC 2020
Module: Mesa
Branch: master
Commit: 22462ba242d4b1ca63fb867668d5ef19ef1dff6e
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=22462ba242d4b1ca63fb867668d5ef19ef1dff6e
Author: Matt Turner <mattst88 at gmail.com>
Date: Mon Nov 11 16:11:34 2019 -0800
intel/compiler: Validate fuzzed instructions
... before giving them to the instruction compactor.
Reviewed-by: Caio Marcelo de Oliveira Filho <caio.oliveira at intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/merge_requests/2635>
---
src/intel/compiler/brw_eu.h | 3 +++
src/intel/compiler/brw_eu_validate.c | 2 +-
src/intel/compiler/test_eu_compact.cpp | 3 +++
3 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/intel/compiler/brw_eu.h b/src/intel/compiler/brw_eu.h
index ce1368a7fd3..ea4ee7748f6 100644
--- a/src/intel/compiler/brw_eu.h
+++ b/src/intel/compiler/brw_eu.h
@@ -1243,6 +1243,9 @@ void brw_debug_compact_uncompact(const struct gen_device_info *devinfo,
brw_inst *orig, brw_inst *uncompacted);
/* brw_eu_validate.c */
+bool brw_validate_instruction(const struct gen_device_info *devinfo,
+ const brw_inst *inst, int offset,
+ struct disasm_info *disasm);
bool brw_validate_instructions(const struct gen_device_info *devinfo,
const void *assembly, int start_offset, int end_offset,
struct disasm_info *disasm);
diff --git a/src/intel/compiler/brw_eu_validate.c b/src/intel/compiler/brw_eu_validate.c
index e2bb552c720..3b802d3b355 100644
--- a/src/intel/compiler/brw_eu_validate.c
+++ b/src/intel/compiler/brw_eu_validate.c
@@ -1937,7 +1937,7 @@ instruction_restrictions(const struct gen_device_info *devinfo,
return error_msg;
}
-static bool
+bool
brw_validate_instruction(const struct gen_device_info *devinfo,
const brw_inst *inst, int offset,
struct disasm_info *disasm)
diff --git a/src/intel/compiler/test_eu_compact.cpp b/src/intel/compiler/test_eu_compact.cpp
index ec93d2bd142..4cb33aefba2 100644
--- a/src/intel/compiler/test_eu_compact.cpp
+++ b/src/intel/compiler/test_eu_compact.cpp
@@ -160,6 +160,9 @@ test_fuzz_compact_instruction(struct brw_codegen *p, brw_inst src)
clear_pad_bits(p->devinfo, &instr);
+ if (!brw_validate_instruction(p->devinfo, &instr, 0, NULL))
+ continue;
+
if (!test_compact_instruction(p, instr)) {
printf(" twiddled bits for fuzzing %d, %d\n", bit0, bit1);
return false;
More information about the mesa-commit
mailing list