-
-
Notifications
You must be signed in to change notification settings - Fork 137
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
feat(sched): PELT #812
base: master
Are you sure you want to change the base?
feat(sched): PELT #812
Conversation
@zwb0x00: no appropriate reviewer found, use |
#[cfg(CONFIG_FAIR_GROUP_SCHED)] | ||
{ | ||
if self.is_task() { | ||
return self.on_rq as u64; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个地方与Linux似乎语意不一致,Linux对应语意为:on_rq为0则返回0否则返回1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -297,6 +316,42 @@ impl FairSchedEntity { | |||
self.avg.load_avg = LoadWeight::scale_load_down(self.load.weight) as usize; | |||
} | |||
} | |||
|
|||
#[allow(dead_code)] | |||
fn attach_entity_cfs_rq(se: Arc<FairSchedEntity>) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
函数应该加注释解释函数的主体逻辑~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok,随着后续跨核部分一起提交
rv的编译不过,可能需要相应的对rv部分补一点TODO。部分地方可能需要加一点注释,以便后续维护 |
@dragonosbot author |
ping @zwb0x00 xd,目前这个进度怎么样呀~ |
前置条件:完成#811,并通过完整测试