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

Convert api.c into Rust code. #9

Merged
merged 13 commits into from
Aug 1, 2019
Merged

Conversation

efenniht
Copy link
Collaborator

@efenniht efenniht commented Jul 31, 2019

  • api.c 와 그와 관련된 코드들(abi, spci 등)을 Rust 로 옮겼습니다.
  • 테스트는, 빠른 컴퓨터 또는 time limit을 늘이면 모두 통과합니다. vcpu_state.concurrent_save_restore 라는 테스트 케이스가 문제인데, 앞서 말씀하신 PR을 반영해 보고 그래도 느리면 다시 알려드리겠습니다.

@jeehoonkang
Copy link
Member

느려지는건 따로 이슈를 만들어주세요.

@efenniht
Copy link
Collaborator Author

efenniht commented Jul 31, 2019

등록했습니다: #10

@@ -0,0 +1,137 @@
/*
Copy link
Member

Choose a reason for hiding this comment

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

이 파일은 원래 어디있던 건가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

inc/vmapi/hf/abi.h 에 있었습니다.

Copy link
Member

Choose a reason for hiding this comment

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

issue로 남깁시다: filename convention

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Issue를 만들었습니다: #15

hfo2/src/arch/mod.rs Show resolved Hide resolved
hfo2/src/cpu.rs Show resolved Hide resolved
hfo2/src/dlog.rs Show resolved Hide resolved
hfo2/src/spci.rs Show resolved Hide resolved
hfo2/src/std.rs Outdated Show resolved Hide resolved
src/dlog.c Show resolved Hide resolved
@jeehoonkang jeehoonkang merged commit 76e1fb4 into kaist-cp:hfo2 Aug 1, 2019
@efenniht efenniht deleted the hfo2-api-1b1 branch August 1, 2019 06:09
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