Re: [PATCH 0/9] sched: Migrate disable support

2020-10-02 Thread Peter Zijlstra
On Fri, Sep 25, 2020 at 08:32:24PM +0100, Valentin Schneider wrote: > The IRC handout so far is: I meant to go post a new version today; but of course cleanup took longer than expected. In any case, there's a slightly updated version here: git://git.kernel.org/pub/scm/linux/kernel/git/peterz/

Re: [PATCH 0/9] sched: Migrate disable support

2020-09-29 Thread Dietmar Eggemann
On 25/09/2020 19:49, Valentin Schneider wrote: > > On 25/09/20 13:19, Valentin Schneider wrote: >> On 25/09/20 12:58, Dietmar Eggemann wrote: >>> With Valentin's print_rq() inspired test snippet I always see one of the >>> RT user tasks as the second guy? BTW, it has to be RT tasks, never >>> trig

Re: [PATCH 0/9] sched: Migrate disable support

2020-09-25 Thread Valentin Schneider
On 25/09/20 19:17, Sebastian Andrzej Siewior wrote: > On 2020-09-21 18:35:57 [+0200], Peter Zijlstra wrote: >> Hi, > Hi, > >> Here's my take on migrate_disable(). It avoids growing a second means of > > I have here: > > |005: numa_remove_cpu cpu 5 node 0: mask now 0,3-4,6-7 > |007: smpboot: CPU 5

Re: [PATCH 0/9] sched: Migrate disable support

2020-09-25 Thread Sebastian Andrzej Siewior
On 2020-09-21 18:35:57 [+0200], Peter Zijlstra wrote: > Hi, Hi, > Here's my take on migrate_disable(). It avoids growing a second means of I have here: |005: numa_remove_cpu cpu 5 node 0: mask now 0,3-4,6-7 |007: smpboot: CPU 5 is now offline |006: [ cut here ] |006: rq->

Re: [PATCH 0/9] sched: Migrate disable support

2020-09-25 Thread Valentin Schneider
On 25/09/20 13:19, Valentin Schneider wrote: > On 25/09/20 12:58, Dietmar Eggemann wrote: >> With Valentin's print_rq() inspired test snippet I always see one of the >> RT user tasks as the second guy? BTW, it has to be RT tasks, never >> triggered with CFS tasks. >> >> [ 57.849268] CPU2 nr_run

Re: [PATCH 0/9] sched: Migrate disable support

2020-09-25 Thread Valentin Schneider
On 25/09/20 12:58, Dietmar Eggemann wrote: > On 25/09/2020 12:10, Peter Zijlstra wrote: >> On Fri, Sep 25, 2020 at 11:12:09AM +0200, Dietmar Eggemann wrote: >> >>> I get this when running 6 (periodic) RT50 tasks with CPU hp stress on my >>> 6 CPU JUNO board (!CONFIG_PREEMPT_RT). >>> >>> [ 55.49

Re: [PATCH 0/9] sched: Migrate disable support

2020-09-25 Thread Dietmar Eggemann
On 25/09/2020 12:10, Peter Zijlstra wrote: > On Fri, Sep 25, 2020 at 11:12:09AM +0200, Dietmar Eggemann wrote: > >> I get this when running 6 (periodic) RT50 tasks with CPU hp stress on my >> 6 CPU JUNO board (!CONFIG_PREEMPT_RT). >> >> [ 55.490263] [ cut here ] >> [ 55

Re: [PATCH 0/9] sched: Migrate disable support

2020-09-25 Thread Peter Zijlstra
On Fri, Sep 25, 2020 at 11:12:09AM +0200, Dietmar Eggemann wrote: > I get this when running 6 (periodic) RT50 tasks with CPU hp stress on my > 6 CPU JUNO board (!CONFIG_PREEMPT_RT). > > [ 55.490263] [ cut here ] > [ 55.505261] Modules linked in: > [ 55.508322] CPU: 3

Re: [PATCH 0/9] sched: Migrate disable support

2020-09-25 Thread Dietmar Eggemann
On 21/09/2020 18:35, Peter Zijlstra wrote: > Hi, > > Here's my take on migrate_disable(). It avoids growing a second means of > changing the affinity, documents how the things violates locking rules but > still mostly works. > > It also avoids blocking completely, so no more futex band-aids requi