Skip to content

Conversation

@danbugs
Copy link
Contributor

@danbugs danbugs commented Dec 9, 2025

Gate hyperlight-guest-tracing to x86_64 since it uses inline assembly that doesn't work on 32-bit. Add compile_error when trace_guest is enabled on non-x86_64 targets. Add CI check to verify hyperlight-guest builds for i686-unknown-linux-gnu.

Move hyperlight-guest-tracing dependency to target-specific section for
x86_64 architecture only. This allows i686 and other architectures to
build without x86_64-specific tracing functionality.

Signed-off-by: danbugs <danilochiarlone@gmail.com>
Add CI check to ensure hyperlight-guest library builds for 32-bit x86
architecture (i686-unknown-linux-gnu) to maintain compatibility with
Nanvix's custom 32-bit x86 target.

Signed-off-by: danbugs <danilochiarlone@gmail.com>
@danbugs danbugs added the kind/refactor For PRs that restructure or remove code without adding new functionality. label Dec 9, 2025
Copy link
Contributor

@dblnz dblnz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Maybe open a sub-issue to #964 to enable the tracing work for 32bit x86 also.

@simongdavies simongdavies merged commit 827f913 into hyperlight-dev:main Dec 9, 2025
117 of 167 checks passed
@danbugs danbugs deleted the danbugs/nanvix-compat branch December 10, 2025 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactor For PRs that restructure or remove code without adding new functionality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants