From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261673AbULNVkX (ORCPT ); Tue, 14 Dec 2004 16:40:23 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261672AbULNVkX (ORCPT ); Tue, 14 Dec 2004 16:40:23 -0500 Received: from e34.co.us.ibm.com ([32.97.110.132]:31369 "EHLO e34.co.us.ibm.com") by vger.kernel.org with ESMTP id S261669AbULNVkO (ORCPT ); Tue, 14 Dec 2004 16:40:14 -0500 Date: Tue, 14 Dec 2004 12:10:57 -0800 From: "Martin J. Bligh" To: Brent Casavant cc: linux-kernel@vger.kernel.org, linux-mm@kvack.org, linux-ia64@vger.kernel.org, ak@suse.de Subject: Re: [PATCH 0/3] NUMA boot hash allocation interleaving Message-ID: <19950000.1103055057@flay> In-Reply-To: References: <9250000.1103050790@flay> X-Mailer: Mulberry/2.1.2 (Linux/x86) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org >> Yup, makes a lot of sense to me to stripe these, for the caches that >> are global (ie inodes, dentries, etc). Only question I'd have is >> didn't Manfred or someone (Andi?) do this before? Or did that never >> get accepted? I know we talked about it a while back. > > Are you thinking of the 2006-06-05 patch from Andi about using > the NUMA policy API for boot time allocation? > > If so, that patch was accepted, but affects neither allocations > performed via alloc_bootmem nor __get_free_pages, which are > currently used to allocate these hashes. vmalloc, however, does > behave as desired with Andi's patch. Nope, was for the hashes, but I think maybe it was all vapourware. > Which is why vmalloc was chosen to solve this problem. There were > other more complicated possible solutions (e.g. multi-level hash tables, > with the bottommost/largest level being allocated across all nodes), > however those would have been so intrusive as to be unpalatable. > So the vmalloc solution seemed reasonable, as long as it is used > only on architectures with plentiful vmalloc space. Yup, seems like a reasonable approach. M.