Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757040Ab0LIUJ7 (ORCPT ); Thu, 9 Dec 2010 15:09:59 -0500 Received: from rcsinet10.oracle.com ([148.87.113.121]:54141 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750985Ab0LIUJ5 (ORCPT ); Thu, 9 Dec 2010 15:09:57 -0500 Message-ID: <4D01377B.5070809@kernel.org> Date: Thu, 09 Dec 2010 12:09:31 -0800 From: Yinghai Lu User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.1.15) Gecko/20101026 SUSE/3.0.10 Thunderbird/3.0.10 MIME-Version: 1.0 To: Stanislaw Gruszka CC: "H. Peter Anvin" , Maxim Uvarov , kexec@lists.infradead.org, linux-kernel@vger.kernel.org, Neil Horman Subject: Re: kdump broken on 2.6.37-rc4 References: <20101203111623.GA2741@redhat.com> <20101203171147.GA2299@redhat.com> <20101203175401.GE28603@hmsreliant.think-freely.org> <20101207105053.GA2803@redhat.com> <4CFE89FE.1090901@kernel.org> <20101208141942.GA2335@redhat.com> <4D00823A.9050808@kernel.org> <20101209124117.GA6032@redhat.com> In-Reply-To: <20101209124117.GA6032@redhat.com> Content-Type: multipart/mixed; boundary="------------000404080907050301040201" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4287 Lines: 142 This is a multi-part message in MIME format. --------------000404080907050301040201 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit On 12/09/2010 04:41 AM, Stanislaw Gruszka wrote: > On Wed, Dec 08, 2010 at 11:16:10PM -0800, Yinghai Lu wrote: >>>> please check debug patches, and boot first kernel and kexec second kernel with "ignore_loglevel debug earlyprintk...." >>> >>> Second kernel does not print anything, so maybe it not even start. >>> Dmesg from primary kernel attached. >>> >>> Stanislaw >> >> >> please try attached debug patch. > > With debug patch kdump kernel boot. Dmesg's from kdump and > primary kernel in attachment. > thanks. please check if this one works. it only put crashkernel low. Yinghai --------------000404080907050301040201 Content-Type: text/x-patch; name="lmb_x86_back_9_x_x.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="lmb_x86_back_9_x_x.patch" Subject: [PATCH] x86, memblock: Add memblock_x86_find_in_range_low() Generic version is going from high to low, and it seems it can not find right area compact enough. the x86 version will go from goal to limit and just like the way We used for early_res to make crashkernel happy with 32bit kdump Signed-off-by: Yinghai Lu --- arch/x86/include/asm/memblock.h | 2 + arch/x86/kernel/setup.c | 2 - arch/x86/mm/memblock.c | 52 ++++++++++++++++++++++++++++++++++++++++ 3 files changed, 55 insertions(+), 1 deletion(-) Index: linux-2.6/arch/x86/mm/memblock.c =================================================================== --- linux-2.6.orig/arch/x86/mm/memblock.c +++ linux-2.6/arch/x86/mm/memblock.c @@ -346,3 +346,55 @@ u64 __init memblock_x86_hole_size(u64 st return end - start - ((u64)ram << PAGE_SHIFT); } + +/* Check for already reserved areas */ +static inline bool __init check_with_memblock_reserved(u64 *addrp, u64 size, u64 align) +{ + u64 addr = *addrp; + bool changed = false; + struct memblock_region *r; +again: + for_each_memblock(reserved, r) { + if ((addr + size) > r->base && addr < (r->base + r->size)) { + addr = round_up(r->base + r->size, align); + changed = true; + goto again; + } + } + + if (changed) + *addrp = addr; + + return changed; +} + +/* + * Find a free area with specified alignment in a specific range from bottom up + */ +u64 __init memblock_x86_find_in_range_low(u64 start, u64 end, u64 size, u64 align) +{ + struct memblock_region *r; + + for_each_memblock(memory, r) { + u64 ei_start = r->base; + u64 ei_last = ei_start + r->size; + u64 addr, last; + + addr = round_up(ei_start, align); + if (addr < start) + addr = round_up(start, align); + if (addr >= ei_last) + continue; + while (check_with_memblock_reserved(&addr, size, align) && addr+size <= ei_last) + ; + last = addr + size; + if (last > ei_last) + continue; + if (last > end) + continue; + + return addr; + } + + return MEMBLOCK_ERROR; +} Index: linux-2.6/arch/x86/include/asm/memblock.h =================================================================== --- linux-2.6.orig/arch/x86/include/asm/memblock.h +++ linux-2.6/arch/x86/include/asm/memblock.h @@ -20,4 +20,6 @@ u64 memblock_x86_find_in_range_node(int u64 memblock_x86_free_memory_in_range(u64 addr, u64 limit); u64 memblock_x86_memory_in_range(u64 addr, u64 limit); +u64 memblock_x86_find_in_range_low(u64 start, u64 end, u64 size, u64 align); + #endif Index: linux-2.6/arch/x86/kernel/setup.c =================================================================== --- linux-2.6.orig/arch/x86/kernel/setup.c +++ linux-2.6/arch/x86/kernel/setup.c @@ -522,7 +522,7 @@ static void __init reserve_crashkernel(v /* * kexec want bzImage is below DEFAULT_BZIMAGE_ADDR_MAX */ - crash_base = memblock_find_in_range(alignment, + crash_base = memblock_x86_find_in_range_low(alignment, DEFAULT_BZIMAGE_ADDR_MAX, crash_size, alignment); if (crash_base == MEMBLOCK_ERROR) { --------------000404080907050301040201-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/