From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753178Ab0CEGSI (ORCPT ); Fri, 5 Mar 2010 01:18:08 -0500 Received: from hera.kernel.org ([140.211.167.34]:45236 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752111Ab0CEGSE (ORCPT ); Fri, 5 Mar 2010 01:18:04 -0500 Message-ID: <4B90A214.70301@kernel.org> Date: Fri, 05 Mar 2010 15:17:56 +0900 From: Tejun Heo User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.5) Gecko/20091130 SUSE/3.0.0-1.1.1 Thunderbird/3.0 MIME-Version: 1.0 To: Sachin Sant CC: linux-next@vger.kernel.org, LKML Subject: Re: -next March 3: Boot failure on x86 (Oops) References: <20100303174603.5be197ba.sfr@canb.auug.org.au> <4B8E83D4.6090507@in.ibm.com> <4B8F0CD0.1040507@kernel.org> <4B8F43DD.10002@in.ibm.com> <4B909FC5.8020800@kernel.org> <4B90A031.8090306@kernel.org> In-Reply-To: <4B90A031.8090306@kernel.org> X-Enigmail-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.2.3 (hera.kernel.org [127.0.0.1]); Fri, 05 Mar 2010 06:17:59 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, On 03/05/2010 03:09 PM, Tejun Heo wrote: > On 03/05/2010 03:08 PM, Tejun Heo wrote: >> Hmmm... this means that on one of the chunks, chunk->list.next was >> NULL (BTW, the disassembly is from unlinked object, right?). The main >> allocation code hasn't seen much change lately. The only changes are, >> >> 22b737f4c75197372d64afc6ed1bccd58c00e549 : just refactoring >> 833af8427be4b217b5bc522f61afdbd3f1d282c2 : possible but isn't very new > > Can you also please try reverting the above two commits? Sorry about all the fuss but I think this could be it. It looks like I forgot to update need_to_extend logic while adding simultaneous head/tail split for alignment, so the array might be overrun by one entry. Can you please try this one first? Thanks. diff --git a/mm/percpu.c b/mm/percpu.c index 768419d..f1ed9ea 100644 --- a/mm/percpu.c +++ b/mm/percpu.c @@ -373,11 +373,11 @@ static int pcpu_need_to_extend(struct pcpu_chunk *chunk) { int new_alloc; - if (chunk->map_alloc >= chunk->map_used + 2) + if (chunk->map_alloc >= chunk->map_used + 3) return 0; new_alloc = PCPU_DFL_MAP_ALLOC; - while (new_alloc < chunk->map_used + 2) + while (new_alloc < chunk->map_used + 3) new_alloc *= 2; return new_alloc; -- tejun