-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Add LoongArch64 support #2529
base: dev
Are you sure you want to change the base?
Add LoongArch64 support #2529
Conversation
9a94347
to
9500d33
Compare
4c9660a
to
ec4a299
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very cool, thank you for the contribution!
To fully support shellcraft syscall generation the pushstr
shellcode is required too. Please consider adding that too if you'd like.
$ shellcraft loong64.linux.execve "/bin/sh" 0 0
...
AttributeError: 'module' object has no attribute 'pushstr'
@@ -113,6 +113,7 @@ jobs: | |||
- name: Install Linux dependencies | |||
run: | | |||
sudo apt-get update | |||
# binutils-loongarch64-linux-gnu is not in Ubuntu 22.04 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can switch to only testing on Ubuntu 24.04 after #2519 is merged.
Signed-off-by: Bingwu Zhang <[email protected]>
pushstr and pushstr_array has been added (: |
@@ -3,7 +3,7 @@ | |||
# Based on Ubuntu | |||
############################################################ | |||
|
|||
FROM ubuntu:jammy | |||
FROM ubuntu:noble |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd expect building the Docker images to fail now due to removal of python2 packages in 24.04. That won't be an issue when we drop Python2 support ourselves in #2519
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah. I am waiting for that PR to be merged (:
No description provided.