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

忽略 AArch32 的 syscall,因为不支持。 #45

Merged
merged 1 commit into from
Mar 2, 2024

Conversation

yxsra
Copy link
Contributor

@yxsra yxsra commented Mar 2, 2024

No description provided.

@SeeFlowerX
Copy link
Owner

首先在追踪64位的syscall的时候,本来就会根据系统调用号自动排除掉没有预设配置的syscall

其次 --arch arm -c tests/xxx_aarch32.json 是可以追踪32位的,只是目前对结构体解析没做完

所以这个pr就没多少必要了,或许能提升一点性能?

@yxsra
Copy link
Contributor Author

yxsra commented Mar 2, 2024

首先在追踪64位的syscall的时候,本来就会根据系统调用号自动排除掉没有预设配置的syscall

其次 --arch arm -c tests/xxx_aarch32.json 是可以追踪32位的,只是目前对结构体解析没做完

所以这个pr就没多少必要了,或许能提升一点性能?

同一个调用号32和64位代表不一样的意思,比如64的 execve(221),在32位是fcntl64,全混在一起按64位的来解析会出问题的。

@SeeFlowerX
Copy link
Owner

确实是没有考虑到这个情况,谢谢。

至于AArch32,后面另外开一个分支

@SeeFlowerX SeeFlowerX closed this Mar 2, 2024
@SeeFlowerX SeeFlowerX reopened this Mar 2, 2024
@SeeFlowerX SeeFlowerX merged commit 3485aa0 into SeeFlowerX:dev Mar 2, 2024
1 check passed
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