修复jiffy时钟过快问题，启用gettimeofday测试，修改mount测试#680
Merged
fslongjin merged 27 commits intoDragonOS-Community:masterfrom Apr 4, 2024
xiaolin2004:fix_time
Merged
修复jiffy时钟过快问题，启用gettimeofday测试，修改mount测试#680fslongjin merged 27 commits intoDragonOS-Community:masterfrom xiaolin2004:fix_time
fslongjin merged 27 commits intoDragonOS-Community:masterfrom
xiaolin2004:fix_time
Conversation
Contributor
|
@xiaolin2004: no appropriate reviewer found, use |
fslongjin
reviewed
Apr 1, 2024
kernel/src/time/jiffies.rs
Outdated
| pub static ref DEFAULT_CLOCK: Arc<ClocksourceJiffies> = ClocksourceJiffies::new(); | ||
| } | ||
| pub const CLOCK_TICK_RATE: u32 = HZ as u32 * 100000; | ||
| pub const CLOCK_TICK_RATE: u32 = 1193182; |
Collaborator
Author
There was a problem hiding this comment.
Member
There was a problem hiding this comment.
不应该写成一个死的数字，而是应该定义应该定义一个全局变量。然后将其与这个地方 ，还有hpet之类的时钟源的频率关联起来。
你这里写成固定值的话，要是我们想修改jiffies频率，就会出现错误。
Collaborator
Author
There was a problem hiding this comment.
修改了HZ之后，这个修改会通过LATCH传递，完成对jiffy频率的控制
Member
|
@dragonosbot author |
dragonosbot
added
S-等待作者修改
Member
|
这样还没跟HPET关联上。 |
fslongjin
requested changes
Apr 4, 2024
kernel/src/time/clocksource.rs
Outdated
| /// 最大能接受的误差大小 | ||
| pub const WATCHDOG_THRESHOLD: u32 = NSEC_PER_SEC >> 4; | ||
|
|
||
| pub const PIT_TICK_RATE: u32 = 1193182; |
Member
There was a problem hiding this comment.
我看了linux相关的内容，应该是错误的这样写。因为我们没有使用PIT，这个会导致vruntime有问题。
2. 修复墙上时间同步错误的问题 3. 启用时间watch dog.
dragonosbot
added
O-riscv64
fslongjin
approved these changes
Apr 4, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.