From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH 04/10] percpu: make 4k first chunk allocator map memory Date: Wed, 15 Jul 2009 12:17:13 +0900 Message-ID: <4A5D4A39.9020000@kernel.org> References: <1245850216-31653-5-git-send-email-tj@kernel.org> <1245850216-31653-1-git-send-email-tj@kernel.org> <24253.1247479973@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from hera.kernel.org ([140.211.167.34]:54716 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751108AbZGODSe (ORCPT ); Tue, 14 Jul 2009 23:18:34 -0400 In-Reply-To: <24253.1247479973@redhat.com> Sender: linux-arch-owner@vger.kernel.org List-ID: To: David Howells Cc: linux-kernel@vger.kernel.org, x86@kernel.org, linux-arch@vger.kernel.org, mingo@elte.hu, andi@firstfloor.org, hpa@zytor.com, tglx@linutronix.de, cl@linux-foundation.org, akpm@linux-foundation.org Hello, David Howells wrote: > Tejun Heo wrote: > >> + pr_info("PERCPU: %d 4k pages per cpu, static data %zu bytes\n", >> + pcpu4k_unit_pages, static_size); > > It occurs to me that this is may include a bad assumption. Page size need not > be 4K. I don't know that it matters, but looking at this: > >> + vm.size = num_possible_cpus() * pcpu4k_unit_pages << PAGE_SHIFT; >> + vm_area_register_early(&vm, PAGE_SIZE); > > It might. Yeah, I realized that after trying to convert powerpc64 and ia64 and renamed it to page. I'll soon post the patches. Thanks. -- tejun