Re: [PATCH v3 2/2] valgrind: Disable on Risc-V

On 5/27/22 16:03, Sean Anderson wrote:

There are no defined instruction sequences in include/valgrind.h for Risc-V, so CONFIG_VALGRIND will do nothing on this arch (and possibly won't compile?). Update Kconfig accordingly.

RISC-V support for valgrind has not been merged upstream yet but is under development.

Reviewed-by: Heinrich Schuchardt xypron.glpk@gmx.de

Signed-off-by: Sean Anderson seanga2@gmail.com

(no changes since v2)

Changes in v2:

• New

  Kconfig | 1 + 1 file changed, 1 insertion(+)

diff --git a/Kconfig b/Kconfig index bdae59e06f..a83b534b34 100644 --- a/Kconfig +++ b/Kconfig @@ -307,6 +307,7 @@ config TPL_SYS_MALLOC_F_LEN

config VALGRIND bool "Inform valgrind about memory allocations"

• depends on !RISCV help Valgrind is an instrumentation framework for building dynamic analysis tools. In particular, it may be used to detect memory management bugs