test: Make valgrind.supp work on aarch64 #19159

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2006-valgrindAarch64 changing 1 files +6 −6

MarcoFalke commented at 3:00 PM on June 3, 2020: member

Was easy to fix by simply removing a line

test: Make valgrind.supp work on aarch64

The equivalent suppression on aarch64 looks like:

{
   <insert_a_suppression_name_here>
   Memcheck:Param
   pwrite64(buf)
   fun:__libc_pwrite64
   fun:pwrite
   fun:__os_io
   obj:/usr/lib/aarch64-linux-gnu/libdb_cxx-5.3.so
   fun:__log_flush_int
   fun:__log_flush
   fun:__memp_sync_int
   fun:__db_sync
   fun:__db_refresh
   fun:__db_close
   fun:__fop_subdb_setup
   fun:__db_open
   fun:__db_open_pp
}

fab7d954b2

MarcoFalke added the label Tests on Jun 3, 2020
MarcoFalke commented at 3:03 PM on June 3, 2020: member
If someone wants to try this at home, I had to crank up the timeout factor:
```
./test/functional/test_runner.py --valgrind -j 1 --timeout-factor=10
```
practicalswift commented at 7:00 PM on June 3, 2020: contributor

ACK fab7d954b261d74b369fe2a3c1785540c4f056b4 -- diff looks correct
fanquake merged this on Jun 4, 2020
fanquake closed this on Jun 4, 2020
MarcoFalke deleted the branch on Jun 4, 2020
sidhujag referenced this in commit a82437c1cd on Jun 4, 2020
DrahtBot locked this on Feb 15, 2022

Contributors

MarcoFalke

practicalswift

Labels

Tests