lkml.org 
[lkml]   [2013]   [Jul]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 13/13] sched: Account for the number of preferred tasks running on a node when selecting a preferred node
>  static void task_numa_placement(struct task_struct *p)
> {
> int seq, nid, max_nid = 0;
> @@ -897,7 +924,7 @@ static void task_numa_placement(struct task_struct *p)
>
> /* Find maximum private faults */
> faults = p->numa_faults[task_faults_idx(nid, 1)];
> - if (faults > max_faults) {
> + if (faults > max_faults && !sched_numa_overloaded(nid)) {

Should we take the other approach of setting the preferred nid but not
moving the task to the node?

So if some task moves out of the preferred node, then we should still be
able to move this task there.

However your current approach has an advantage that it atleast runs on
second preferred choice if not the first.

Also should sched_numa_overloaded() also consider pinned tasks?

> max_faults = faults;
> max_nid = nid;
> }
> @@ -923,9 +950,7 @@ static void task_numa_placement(struct task_struct *p)
> max_nid);
> }
>
> - /* Update the preferred nid and migrate task if possible */
> - p->numa_preferred_nid = max_nid;
> - p->numa_migrate_seq = 0;
> + sched_setnuma(p, max_nid, preferred_cpu);
> migrate_task_to(p, preferred_cpu);
>
> /*



\
 
 \ /
  Last update: 2013-07-04 16:01    [W:0.336 / U:0.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site