On Fri, 21 Dec 2018 at 18:15, Valentin Schneider
wrote:
>
> On 21/12/2018 14:49, Vincent Guittot wrote:
> [...]
> > After looking at shed.c at this sha1, (sd->nr_balance_failed >
> > sd->cache_nice_tries+2) was the only condition for doing active
> > migration and as a result it was the only reas
On 21/12/2018 14:49, Vincent Guittot wrote:
[...]
> After looking at shed.c at this sha1, (sd->nr_balance_failed >
> sd->cache_nice_tries+2) was the only condition for doing active
> migration and as a result it was the only reason for doubling
> sd->balance_interval.
> My patch keeps exactly the
On Thu, 20 Dec 2018 at 18:24, Valentin Schneider
wrote:
>
> On 20/12/2018 14:50, Vincent Guittot wrote:
> [...]
> >> So now we reset the interval for all active balances (expect last active
> >> balance case), even when it is done as a last resort because all other
> >> tasks were pinned.
> >>
>
On 20/12/2018 14:50, Vincent Guittot wrote:
[...]
>> So now we reset the interval for all active balances (expect last active
>> balance case), even when it is done as a last resort because all other
>> tasks were pinned.
>>
>> Arguably the current code isn't much better (always increase the inter
On Thu, 20 Dec 2018 at 12:22, Valentin Schneider
wrote:
>
> On 20/12/2018 07:55, Vincent Guittot wrote:
> > In case of active balance, we increase the balance interval to cover
> > pinned tasks cases not covered by all_pinned logic. Neverthless, the
> > active migration triggered by asym packing s
On 20/12/2018 07:55, Vincent Guittot wrote:
> In case of active balance, we increase the balance interval to cover
> pinned tasks cases not covered by all_pinned logic. Neverthless, the
> active migration triggered by asym packing should be treated as the normal
> unbalanced case and reset the inte
On Thu, 13 Dec 2018 at 14:52, Peter Zijlstra wrote:
>
> On Tue, Aug 07, 2018 at 05:56:27PM +0200, Vincent Guittot wrote:
> > +static inline bool
> > +asym_active_balance(enum cpu_idle_type idle, unsigned int flags, int dst,
> > int src)
> > {
> > + if (idle != CPU_NOT_IDLE) {
> >
> >
On Tue, Aug 07, 2018 at 05:56:27PM +0200, Vincent Guittot wrote:
> +static inline bool
> +asym_active_balance(enum cpu_idle_type idle, unsigned int flags, int dst,
> int src)
> {
> + if (idle != CPU_NOT_IDLE) {
>
> /*
>* ASYM_PACKING needs to force migrate task
8 matches
Mail list logo
