Initial implementation of ARMv8 support

GDATASoftwareAG / smartvmi

Virtual Machine Introspection (VMI) for memory forensics and machine-learning.

http://www.smartvmi.org

MIT License

23 stars 4 forks source link

Initial implementation of ARMv8 support #35

Open thomasdangl opened 2 years ago

thomasdangl commented 2 years ago

This PR provides preliminary support of all current features for ARMv8 on Linux.

What remains to be done:

Implement support for Windows on ARM
Create consistent naming between ARMv8 and AMD64, in particular, avoid the usage of architecture specific names such as CR3

thomasdangl commented 2 years ago

Please note that I also turned off include sorting in clang format in this PR.

Otherwise it will sort the libvmi includes in src/vmi/VmiInitError.h lexicographically, which breaks the include order and prevents the project from compiling...

rageagainsthepc commented 2 years ago

Please note that I also turned off include sorting in clang format in this PR.

Otherwise it will sort the libvmi includes in src/vmi/VmiInitError.h lexicographically, which breaks the include order and prevents the project from compiling...

This has been fixed some time ago: https://github.com/libvmi/libvmi/pull/995 It's not necessary to include libvmi.h at all anymore when events.h is included.

thomasdangl commented 2 years ago

Thanks for the information. Unfortunately, the libvmi port for ARM is based off this version https://github.com/libvmi/libvmi/commit/88d281db5a4312574f65ff547b0675ec491f175d.

Maybe it would make sense to rebase it onto the current master branch before we continue merging this?

thomasdangl commented 2 years ago

Alright, I have rebased to current libvmi upstream. So we should be able to keep the sorting in clang-tidy : )

rageagainsthepc commented 1 year ago

@thomasdangl At the moment it is not possible to run CI builds from forks. You need to push your branch to our upstream repo.

rageagainsthepc commented 1 year ago

Is this PR still relevant? :thinking: