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

i#7303: Integrate Drsyscall into DynamoRIO. #7396

Merged
merged 18 commits into from
Mar 26, 2025

Conversation

ivankyluk
Copy link
Contributor

@ivankyluk ivankyluk commented Mar 24, 2025

The following changes are made to integrate DrSyscall into DynamoRIO:

  • Update copyright.
  • Remove drsyscall.dox.
  • Add drcontext as the first parameter to LOG() to be consistent with DynamoRIO.
  • Add drcontext as the first parameter to name2num_entry_add(), name2num_record(), wingdi_add_usercall(), and wingdi_get_secondary_syscall_num() to pass it along to LOG().
  • Change the include file path for wininc/* to drmf/wininc/*.
  • Remove rlimit64 declaration from linux_defines.h.
  • Update drsyscall/CMakeLists.txt to include files in framework and common.
  • Copy the DRMF_VERSION and set_library_version logic from drmemory/CMakeLists.txt to ext/drsyscall/CMakeList.txt.
  • Skip Alpine, ANDROID and RISCV64 build in drsycall/CMakeLists.txt
  • Change make/resources.rc to core/win32/resources.rc in drsyscall/CMakeLists.txt.

Issue: #7303

@ivankyluk ivankyluk changed the title i7303: Build and link drsyscall. i#7303: Integrate Drsyscall into DynamoRIO. Mar 25, 2025
@ivankyluk ivankyluk marked this pull request as ready for review March 25, 2025 00:26
@ivankyluk ivankyluk requested a review from derekbruening March 25, 2025 00:26
@ivankyluk ivankyluk requested a review from derekbruening March 25, 2025 23:27
@ivankyluk ivankyluk merged commit 50b1a73 into master Mar 26, 2025
24 checks passed
@ivankyluk ivankyluk deleted the i7303-build-and-link-drsyscall branch March 26, 2025 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants