Skip to content

Conversation

@woshiluo
Copy link
Member

本 PR 提供以下修改:

  • 修正了文档中的代码的未对齐的问题。
  • 统一结构体对齐到 8,因为 64 位下 core::mem::align_of::<usize>() 可能为 8。
  • examples/hifive-unmatched-a00.rs 改写而来了的测试。
  • 增加了 CI 用于自动测试。

feat: add test from example

Signed-off-by: Woshiluo Luo <woshiluo.luo@outlook.com>
Signed-off-by: Woshiluo Luo <woshiluo.luo@outlook.com>
Signed-off-by: Woshiluo Luo <woshiluo.luo@outlook.com>
Signed-off-by: Woshiluo Luo <woshiluo.luo@outlook.com>
Copy link
Member

@luojia65 luojia65 left a comment

Choose a reason for hiding this comment

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

LGTM, there is one small change we would like to apply, though. Please explain the code changes in more detail on the pull request title.

Signed-off-by: Woshiluo Luo <woshiluo.luo@outlook.com>
@woshiluo woshiluo changed the title Add ci Add tests & a GitHub workflow to automatically run tests & fix alignment issues Oct 27, 2024
Copy link
Member

@luojia65 luojia65 left a comment

Choose a reason for hiding this comment

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

LGTM

@luojia65 luojia65 merged commit 7c88139 into rustsbi:main Oct 27, 2024
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