linux-kernel - Re: [RFC][PATCH 4/4] sched, numa: Ignore pinned tasks

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150516093148.GZ21418@twins.programming.kicks-ass.net>
Date:	Sat, 16 May 2015 11:31:48 +0200
From:	Peter Zijlstra <peterz@...radead.org>
To:	mingo@...nel.org, riel@...hat.com
Cc:	dedekind1@...il.com, linux-kernel@...r.kernel.org, mgorman@...e.de,
	rostedt@...dmis.org, juri.lelli@....com
Subject: Re: [RFC][PATCH 4/4] sched, numa: Ignore pinned tasks

On Fri, May 15, 2015 at 05:43:37PM +0200, Peter Zijlstra wrote:
>  static void account_numa_enqueue(struct rq *rq, struct task_struct *p)
>  {
> +	if (p->nr_cpus_allowed == 1) {
> +		p->numa_preferred_nid = -1;
> +		rq->nr_pinned_running++;
> +	}
>  	rq->nr_numa_running += (p->numa_preferred_nid != -1);
>  	rq->nr_preferred_running += (p->numa_preferred_nid == task_node(p));
>  }

>  static inline enum fbq_type fbq_classify_rq(struct rq *rq)
>  {
> +	unsigned int nr_migratable = rq->cfs.h_nr_running - rq->nr_pinned_running;
> +

FWIW, there's a problem there with CFS bandwidth muck. When we throttle
groups we update cfs.h_nr_running properly, but we do not hierarchically
account the pinned, preferred and numa counts.

So that above subtraction can end up negative.

I've not yet decided what to do about this; ideally we'd do the
hierarchical accounting of the numa stats -- but that's a little bit
more expensive than I'd like.

A well. for monday that.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/