Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • D dynamorio
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1,467
    • Issues 1,467
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 44
    • Merge requests 44
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • DynamoRIO
  • dynamorio
  • Merge requests
  • !3003

i#2042: signal interrupting ibl jump

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Derek Bruening requested to merge i2042-ibl-signal into master May 10, 2018
  • Overview 1
  • Commits 1
  • Pipelines 0
  • Changes 12

Adds handling of a signal interrupting the ibl hit path's jump, where the target tag is not conveniently sitting in a register like it is in the rest of the ibl. We have to instead look at the jump's cache target and find the fragment to unlink from there.

Also fixes up the i#2328 support for a signal interrupting clean_call_{save,restore} to work for AArch64.

Adds a new test, client.cleancallsig, which tries to trigger both of these cases: the client adds a lot of clean calls while the app runs a loop with a lot of indirect branches along with an itimer. I managed to hit each of the new features at least once running this test in a loop on each platform except the ibl jump on ARM and AArch64. I left an ASSERT_NOT_TESTED for those.

Issue: #2328 (closed) Fixes #2042 (closed)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: i2042-ibl-signal