Messages in this thread |  | | | Date | Fri, 15 Apr 2011 16:42:13 -0700 (PDT) | | From | David Rientjes <> | | Subject | Re: [PATCH v2] cpusets: randomize node rotor used in cpuset_mem_spread_node() |
| |
On Fri, 15 Apr 2011, Michal Hocko wrote:
> You are right. I was thinking about lazy approach and initialize those > values when they are used for the first time. What about the patch > below? > > Change from v1: > - initialize cpuset_{mem,slab}_spread_rotor lazily >
The difference between this v2 patch and what is already in the -mm tree (http://userweb.kernel.org/~akpm/mmotm/broken-out/cpusets-randomize-node-rotor-used-in-cpuset_mem_spread_node.patch) is the lazy initialization by adding cpuset_{mem,slab}_spread_node()? It'd probably be better to just make an incremental patch on top of mmotm-2011-04-14-15-08 with a new changelog and then propose with with your list of reviewed-by lines.
Andrew could easily drop the earlier version and merge this v2, but I'm asking for selfish reasons: please use NUMA_NO_NODE instead of -1.
Thanks!
|  |