From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH 13/13] memblock, x86: Replace memblock_x86_reserve/free_range() with generic ones Date: Tue, 26 Jul 2011 23:46:11 +0200 Message-ID: References: <1310462166-31469-1-git-send-email-tj@kernel.org> <1310462166-31469-14-git-send-email-tj@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Return-path: Received: from mail-vw0-f46.google.com ([209.85.212.46]:65080 "EHLO mail-vw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753844Ab1GZVqM (ORCPT ); Tue, 26 Jul 2011 17:46:12 -0400 In-Reply-To: Sender: linux-arch-owner@vger.kernel.org List-ID: To: Yinghai Lu Cc: mingo@redhat.com, hpa@zytor.com, tglx@linutronix.de, benh@kernel.crashing.org, davem@davemloft.net, linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, x86@kernel.org Hello, Yinghai. On Tue, Jul 26, 2011 at 11:06 PM, Yinghai Lu wrote: > Wonder if we have a new function like > > __memblock_reserve(addr, size, name) > > in mm/memblock.c > > so could keep string for debug purpose instead of using _RET_IP_. > > with that, we can figure out what is in exact reserved position with boot log. If you look at the current users, the caller function name is unique except for one or two cases, so we can already tell who's doing what even without consulting the exact IP. I can't see much point in providing debug strings explicitly. The only benefit would be that it would be prettier but these are purely debug messages. One thing which might make sense is preserving _RET_IP_ of callers of alloc functions so that reserve can show who's allocating instead of printing the alloc function each time. Thanks. -- tejun