ABI API merge - Githubissues

MerlijnWajer / tracy

tracy - a system call tracer and injector. Find us in #tracy on irc.freenode.net

http://hetgrotebos.org/wiki/Tracy

Do What The F*ck You Want To Public License

72 stars 26 forks source link

ABI API merge #35

Closed MerlijnWajer closed 11 years ago

MerlijnWajer commented 11 years ago

Okay, so to merge the ABI changes we need to do several things:

Create several def_syscall like files; make the syscall number to string function take not just a system call number but also an abi. We thus also need to modify the Makefile
Modify tracyarch.h accordingly. We also want constants for ABIs I guess? How do we do this in a cross platform way?
Add the abi to the tracy_event structure; do proper book keeping everywhere.
Possibly add (per child) what abis it can possibly use.
Add an ABI argument to the inject system calls. (Also store in tracy_inject_data?)
Add an ABI argument to the hook system calls? (Personally I think we should not do this, if someone wants different hooks for different ABIs of the same system call he can just do this him/herself)

Will add more as we figure this out.

MerlijnWajer commented 11 years ago

Apparently we can use the CS; http://old-list-archives.xen.org/archives/html/xen-devel/2005-06/msg00377.html Also, the other method didn't work well so I'm switching over to CS.

MerlijnWajer commented 11 years ago

Initial proposal in 2e6e38a

MerlijnWajer commented 11 years ago

I've been thinking, maybe we need a function that will tell you if a certain event is a certain system call, taking the abi etc into account. Macro should do, I guess.

MerlijnWajer commented 11 years ago

Implemented quite a bit now; see d2b0bef. Implemented AMD64 abi detection; ARM is currently WIP; x86 should be very straightforward. Lots of Makefile changes (they need to be cleaned up); there's some extra debug statements, but it mostly works. We also need to replace all getsyscall{name,number} instances that do not use the ABI variants.

MerlijnWajer commented 11 years ago

I think I want to merge this eventually, I need to make few more changes but then I think it's ready for merging to master. Especially if our (only) application soxy doesn't break. I'll make it abi-compatible as well.

MerlijnWajer commented 11 years ago

See 33e61f1718a4b83947cc2fa43a6634b9ac830d04 Added hooking per abi, register fixes and other stuff.

MerlijnWajer commented 11 years ago

IMHO work is now stable enough to be merged into master. So. Merged.