Code

Adding support for sanitised builds#182

Code

Merged

csegarragonz merged 50 commits intomasterfaasm/faabric:masterfrom sanitisersfaasm/faabric:sanitisersCopy head branch name to clipboardNov 30, 2021

+755-314

Commits

Commits on Nov 25, 2021

• adding sanitiser flags and workflow file

  csegarragonz

  committed
  285d89b

  Copy full SHA for 285d89b

• adding files to diff against

  csegarragonz

  committed
  949cb34

  Copy full SHA for 949cb34

• cleaning up workflow file

  csegarragonz

  committed
  1e98852

  Copy full SHA for 1e98852

• run only on workflow dispatch

  csegarragonz

  committed
  aa45f75

  Copy full SHA for aa45f75

• properly use caching

  csegarragonz

  committed
  7e9a4f2

  Copy full SHA for 7e9a4f2

• setting flag as default to false

  csegarragonz

  committed
  1c8cc22

  Copy full SHA for 1c8cc22

Commits on Nov 26, 2021

• cache the right folder

  csegarragonz

  committed
  51b7fac

  Copy full SHA for 51b7fac

• Fix ASAN_OPTIONS typo

  eigenraven

  committed
  43b71d4

  Copy full SHA for 43b71d4

• Try using cachev2 instead of artifact for conan dependencies

  eigenraven

  committed
  64bcf5e

  Copy full SHA for 64bcf5e

• Use a local conan cache folder when using the cli, just like in faasm

  eigenraven

  committed
  b95e2b6

  Copy full SHA for b95e2b6

• UBSan fix: Make sure created int pointer are 4-aligned

  eigenraven

  committed
  a5b8c51

  Copy full SHA for a5b8c51

• ASan,TSan: Latch destruction race-condition fix

  eigenraven

  committed
  815c4ec

  Copy full SHA for 815c4ec

• ASan: MPI World test buffer too small

  eigenraven

  committed
  f5e077b

  Copy full SHA for f5e077b

• ASan: fix mismatched new[]-delete

  eigenraven

  committed
  be1b08d

  Copy full SHA for be1b08d

• ASan: out-of-bound read, this was a simple copy-paste error

  eigenraven

  committed
  45bf978

  Copy full SHA for 45bf978

• ASan: Copy temporary strings in RapidJSON

  eigenraven

  committed
  11c3a77

  Copy full SHA for 11c3a77

• fix #183: Use unique_ptr for all redis reply objects so they get cleaned up on exceptions and any scope exit

  eigenraven

  committed
  d75741c

  Copy full SHA for d75741c

• ASan/leak: use unique_ptr for pulled and dirtyMask to free the arrays automatically in the destructor

  eigenraven

  committed
  f43fa32

  Copy full SHA for f43fa32

• Change the BYTES macro to a template to catch potential type errors

  eigenraven

  committed
  bbc1db2

  Copy full SHA for bbc1db2

• Replace all other uses of new/delete with unique_ptr/vector.

  Show description for 04bdc9b
  eigenraven

  committed
  04bdc9b

  Copy full SHA for 04bdc9b

• Merge remote-tracking branch 'upstream/master' into sanitisers

  eigenraven

  committed
  cc5af1d

  Copy full SHA for cc5af1d

• Add a TSan ignore list

  eigenraven

  committed
  5b855f5

  Copy full SHA for 5b855f5

• TSan: Fix data race in Scheduler testing code

  eigenraven

  committed
  0a921ad

  Copy full SHA for 0a921ad

• Ignore zeromq races

  eigenraven

  committed
  487791e

  Copy full SHA for 487791e

• TSan: fix shared_ptr data races (in a shared_ptr the counter is atomic, but ptr is not, and requires explicit atomic accesses)

  Show description for aef1cb3
  eigenraven

  committed
  aef1cb3

  Copy full SHA for aef1cb3

• Ignore config change data races, as they could only affect tests

  eigenraven

  committed
  680e55d

  Copy full SHA for 680e55d

• TSan: Missing lock on scheduler reset

  eigenraven

  committed
  37f1eac

  Copy full SHA for 37f1eac

• Ignore catch2 bad signal handler errors in tsan

  eigenraven

  committed
  ea910c6

  Copy full SHA for ea910c6

• TSan: Fix racy executor cleanup/deadlock

  eigenraven

  committed
  745c8d6

  Copy full SHA for 745c8d6

• Missing locks in SnapshotClient mock code

  eigenraven

  committed
  72b9b3a

  Copy full SHA for 72b9b3a

• MockMode needs to be atomic to work across threads without a mutex

  eigenraven

  committed
  935dbe5

  Copy full SHA for 935dbe5

• Missing mutexes in FunctionCallClient mock code

  eigenraven

  committed
  feba949

  Copy full SHA for feba949

• More missing locks on reads in the scheduler

  eigenraven

  committed
  ad6265e

  Copy full SHA for ad6265e

• Missing locks on *ThisHostResources in Scheduler

  eigenraven

  committed
  e87360c

  Copy full SHA for e87360c

• Skip mpi::allGather as I don't know what this exactly does

  eigenraven

  committed
  0d7d73b

  Copy full SHA for 0d7d73b

• Catch2 REQUIRE is not thread-safe, use assert

  eigenraven

  committed
  54ece01

  Copy full SHA for 54ece01

• TSan snapshot fixes

  eigenraven

  committed
  12b3d3b

  Copy full SHA for 12b3d3b

• Run TSan with maximum history size so it doesn't lose stack traces

  eigenraven

  committed
  0d896a5

  Copy full SHA for 0d896a5

• Missing shared lock in KV State

  eigenraven

  committed
  7e2e81f

  Copy full SHA for 7e2e81f

• Disable two tests that break in TSan builds

  eigenraven

  committed
  7e0a55a

  Copy full SHA for 7e0a55a

• Stop gid generation from tripping TSan using relaxed atomics

  eigenraven

  committed
  a6ae467

  Copy full SHA for a6ae467

• Avoid racy destruction on FlagWaiter just like the Latch

  eigenraven

  committed
  f7436b5

  Copy full SHA for f7436b5

• Missing lock on a queue push

  eigenraven

  committed
  d77f4ae

  Copy full SHA for d77f4ae

• Fix formatting

  eigenraven

  committed
  e8ac90c

  Copy full SHA for e8ac90c

Commits on Nov 29, 2021

• Address review comments

  eigenraven

  committed
  d1ac721

  Copy full SHA for d1ac721

• Remove mode checking from python as it's already done by cmake

  eigenraven

  committed
  279c9a8

  Copy full SHA for 279c9a8

• Fix gha cache key, add restore-keys to fall back on cache misses and try a different key

  eigenraven

  committed
  eb58bba

  Copy full SHA for eb58bba

• Another attempt at fixing checksum hashes, this time use bash instead of GHA builtins

  eigenraven

  committed
  fd98afe

  Copy full SHA for fd98afe

Commits on Nov 30, 2021

• Merge remote-tracking branch 'upstream/master' into sanitisers

  eigenraven

  committed
  d0f3ff8

  Copy full SHA for d0f3ff8

• Show full test and sanitiser logs, it halts on errors so still should be readable

  eigenraven

  committed
  d5341e8

  Copy full SHA for d5341e8