Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Do not merge] CI test, cmake option to enable address santitizer. #2890

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Commits on Sep 10, 2023

  1. Configuration menu
    Copy the full SHA
    0b1a774 View commit details
    Browse the repository at this point in the history
  2. buildkite: Add per variant timeout value and increase aarch asan and …

    …force32bit to 55 minutes.
    bernhardu committed Sep 10, 2023
    Configuration menu
    Copy the full SHA
    ea7cb3c View commit details
    Browse the repository at this point in the history
  3. Attempt to avoid assert in force32bit.

    Error message:
      [FATAL src/MemoryRange.h:20:MemoryRange()] start_ <= end_
    bernhardu committed Sep 10, 2023
    Configuration menu
    Copy the full SHA
    ac0227e View commit details
    Browse the repository at this point in the history
  4. ASan: Increase timeout value.

    CI showed timeouts with test:
      madvise_fracture_flags (exceeded 120s, aarch64)
      nested_release (exceeded 120s)
      record_replay (exceeded 600s)
    bernhardu committed Sep 10, 2023
    Configuration menu
    Copy the full SHA
    930ce5b View commit details
    Browse the repository at this point in the history
  5. ASan: Remove tests failing with executed strex instructions detected.

    Remove those tests for now.
    No idea how that could be avoided?
    
    Error message:
      [FATAL src/PerfCounters.cc:785:read_ticks()] 1 (speculatively) executed strex instructions detected.
    
    Example:
      https://buildkite.com/julialang/rr/builds/1141
      BUILDKITE_AGENT_NAME="default-armageddon.3"
    bernhardu committed Sep 10, 2023
    Configuration menu
    Copy the full SHA
    720e885 View commit details
    Browse the repository at this point in the history
  6. ASan: Run with verify_asan_link_order=0

    Avoids error message:
      ASan runtime does not come first in initial library list; you should either link runtime to your application or manually preload it with LD_PRELOAD.
    
    Affected tests:
      nested_detach
      nested_detach_kill
      nested_detach_kill_stuck
      nested_detach_stop
      nested_detach_wait
    bernhardu committed Sep 10, 2023
    Configuration menu
    Copy the full SHA
    cc0d479 View commit details
    Browse the repository at this point in the history