linux-kernel - Re: [patch -mm] cpusets: add memory_slab

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: <alpine.DEB.2.00.0903091853000.25296@chino.kir.corp.google.com>
Date:	Mon, 9 Mar 2009 19:01:18 -0700 (PDT)
From:	David Rientjes <rientjes@...gle.com>
To:	KOSAKI Motohiro <kosaki.motohiro@...fujitsu.com>
cc:	Christoph Lameter <cl@...ux-foundation.org>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Pekka Enberg <penberg@...helsinki.fi>,
	Matt Mackall <mpm@...enic.com>,
	Paul Menage <menage@...gle.com>,
	Randy Dunlap <randy.dunlap@...cle.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [patch -mm] cpusets: add memory_slab_hardwall flag

On Tue, 10 Mar 2009, KOSAKI Motohiro wrote:

> That's pointless.
> Again, any fastpath modification should have reasonable reason.
> We are looking for your explanation.
> 

The fastpath modification simply checks if the hardwall bit is set in the 
allocating task's cpuset flags.  If it's disabled, there is no additional 
overhead.

This requirement was mandated during the first review of the patch by 
Christoph, who requested that it be configurable.  Before that it was 
possible to simply check if the global `number_of_cpusets' count was > 1.  
If not, cpuset_node_allowed_hardwall() would always return true.  If the 
system had more than one cpuset, it would have reduced to checking

	return in_interrupt() || (gfp_mask & __GFP_THISNODE) ||
	       node_isset(node, current->mems_allowed);

As I already mentioned, if fastpath optimization is your only concern, 
that we could simply add PF_SLAB_HARDWALL task flags that would simply 
make this

	return current->flags & PF_SLAB_HARDWALL;

So the fastpath cost can be mitigated at the expense of an additional task 
flag.
--
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/