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

Minor fixes #209

Merged
merged 2 commits into from
Sep 14, 2020
Merged

Minor fixes #209

merged 2 commits into from
Sep 14, 2020

Conversation

efenniht
Copy link
Collaborator

  • panic 이 일어나면 멈추지 않고 kernel panic 이 계속해서 발생하는 버그를 수정.
  • Update Rust and use min_const_generics in RcPool #208 이후 release 모드로 빌드해도 잘 실행되는 것을 확인했습니다. Makefile 을 수정하여 dev/release 둘다 테스트 가능하도록 만들었습니다.
    • LLVM 버전이 업데이트되어서 그런 듯 합니다.
    • make 는 debug 모드로,
    • make RUST_MODE=release 는 release 모드로 실행됩니다.

Closes #146

@efenniht efenniht self-assigned this Sep 13, 2020
Copy link
Collaborator

@kimjungwow kimjungwow left a comment

Choose a reason for hiding this comment

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

@efenniht efenniht merged commit 4f44961 into kaist-cp:riscv Sep 14, 2020
@efenniht efenniht deleted the fix.spin_loop branch September 14, 2020 02:05
@efenniht
Copy link
Collaborator Author

@kimjungwow
Copy link
Collaborator

  • "fixup value out of range" error (LLVM bug?) #146 이슈 발생 당시, cargo.toml에서 release mode로 빌드해도 디버그 모드로 빌드되도록 설정되어 있었습니다 (Remove C-style strings #147 수정 전 참고)
  • stack size를 키우는 것은 적절하지 못한 해결책일까요? opt level이 0이면 디버깅이 유용한 경우가 있을 수 있어 질문드립니다 :)

@efenniht
Copy link
Collaborator Author

  • "fixup value out of range" error (LLVM bug?) #146 이슈 발생 당시, cargo.toml에서 release mode로 빌드해도 디버그 모드로 빌드되도록 설정되어 있었습니다 (Remove C-style strings #147 수정 전 참고)
  • stack size를 키우는 것은 적절하지 못한 해결책일까요? opt level이 0이면 디버깅이 유용한 경우가 있을 수 있어 질문드립니다 :)
  • debug option 은 디버깅 정보를 바이너리에 추가하는 옵션이고, optimization level 에 영향을 주는 것 같지 않습니다. "fixup value out of range" error (LLVM bug?) #146 은 high optimization level 에서 컴파일이 안 되는 문제가 아닌가요?
  • stack size 를 키우면 xv6 코드가 암묵적으로 가지고 있는 메모리 레이아웃에 대한 가정이 깨질 수 있습니다. 그리고 semantics 가 달라진다고 생각할 수 있어서 좋지 않은 방법으로 보입니다. 디버깅이 어려운 것은 ... 어쩔 수 없군요 😭

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.

"fixup value out of range" error (LLVM bug?)
3 participants