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

Kvm support when opt-level > 1 #606

Merged
merged 3 commits into from
Feb 25, 2022
Merged

Kvm support when opt-level > 1 #606

merged 3 commits into from
Feb 25, 2022

Conversation

junsooo
Copy link
Contributor

@junsooo junsooo commented Feb 25, 2022

Related context:
#604
#605

@junsooo junsooo marked this pull request as ready for review February 25, 2022 06:35
@@ -40,7 +40,10 @@ pub static mut stack0: Stack = Stack::new();
pub unsafe fn start() {
// launch other cores
if cpu_id() == 0 {
let kernel_entry = unsafe { _entry.as_mut_ptr() as usize } as u64;
// XXX(https://github.com/kaist-cp/rv6/issues/605): rustc bug?
Copy link
Collaborator

Choose a reason for hiding this comment

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

// TODO(https://github.com/kaist-cp/rv6/issues/517): safety?
주석 형식을 이 부분처럼 TODO로 해주실 수 있을까요? (XXX -> TODO). rv6에서 원래 사용하던 형식이라 일관성을 유지하는 것이 좋아보입니다.

Copy link
Contributor Author

@junsooo junsooo Feb 25, 2022

Choose a reason for hiding this comment

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

3474b86 여기서 todo로 수정 완료했습니다!

@jeehoonkang jeehoonkang merged commit 66badc7 into kaist-cp:main Feb 25, 2022
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.

3 participants