beehive-lab / mambo

A low-overhead dynamic binary instrumentation and modification tool for ARM (both AArch32 and AArch64 support) and RISC-V (RV64GC).
Apache License 2.0
320 stars 69 forks source link

Move PIE from submodule into MAMBO repository #121

Closed IgWod closed 4 months ago

IgWod commented 5 months ago

This is something still very much to be discussed, but I think the current sentiment is to merge PIE and MAMBO together into a single repo. This has a number of benefits:

  1. Simplifies process of merging new instructions into MAMBO
  2. Reduces a possibility of MAMBO and PIE getting out of sync
  3. Simplifies repo management and access rights

I don't see any potential downside of such move, but I'm very much opened for comments.

Once that's done we need to port following PRs into this repo:

  1. https://github.com/beehive-lab/pie/pull/8
  2. https://github.com/beehive-lab/pie/pull/9
ijan1 commented 4 months ago

Might be worth putting in the path to https://github.com/beehive-lab/pie in a commit message for archival purposes, otherwise LGTM.