On Thu, Aug 20, 2015 at 05:22:31PM +0300, Kirill Tkhai wrote:
>
>
> On 20.08.2015 15:59, Vladimir Davydov wrote:
> > On Thu, Aug 20, 2015 at 02:27:05PM +0300, Kirill Tkhai wrote:
> > ...
> >> +static void pre_schedule_fair(struct rq *rq, struct task_struct *prev)
> >> +{
> >> + trigger_cpulimit_balance(prev);
> >> +}
> >> +
> >
> > You don't remove trigger_cpulimit_balance call from put_prev_task_fair.
> > Do we still need it there? [The same question for PCS6 version of this
> > patch]
>
> I simulated the same behaviour as it used to be before. If we remove the
> trigger from put_prev_task_fair(), there will be a later cpulimit balancing
> in some cases.
I don't see any point in it. You call pre_schedule ->
trigger_cpulimit_balance, then immediately after it put_prev_task ->
trigger_cpulimit_balance again.
_______________________________________________
Devel mailing list
[email protected]
https://lists.openvz.org/mailman/listinfo/devel
