Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753724AbYJQDG6 (ORCPT ); Thu, 16 Oct 2008 23:06:58 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752162AbYJQDGu (ORCPT ); Thu, 16 Oct 2008 23:06:50 -0400 Received: from gate.crashing.org ([63.228.1.57]:36190 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752105AbYJQDGt (ORCPT ); Thu, 16 Oct 2008 23:06:49 -0400 Subject: Re: [PATCH] Fix broken debug output reserve_region_with_split() From: Benjamin Herrenschmidt Reply-To: benh@kernel.crashing.org To: Linus Torvalds Cc: Geert.Uytterhoeven@sonycom.com, hpa@zytor.com, linux-kernel@vger.kernel.org, mingo@elte.hu, yhlu.kernel@gmail.com, David Miller , Andrew Morton In-Reply-To: <1224212533.7654.60.camel@pasglop> References: <48F409E9.3030501@zytor.com> <20081013.223811.174710738.davem@davemloft.net> <20081016.013850.85183428.davem@davemloft.net> <1224212533.7654.60.camel@pasglop> Content-Type: text/plain Date: Fri, 17 Oct 2008 14:06:31 +1100 Message-Id: <1224212791.7654.63.camel@pasglop> Mime-Version: 1.0 X-Mailer: Evolution 2.22.3.1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3536 Lines: 95 From: Linus Torvalds Implement %pR to print struct resource content Add a %pR option to the kernel vsnprintf that prints the range of addresses inside a struct resource passed by pointer. Signed-off-by: Linus Torvalds Signed-off-by: Benjamin Herrenschmidt --- And this one has updated comments. Index: linux-work/lib/vsprintf.c =================================================================== --- linux-work.orig/lib/vsprintf.c 2008-10-17 13:30:18.000000000 +1100 +++ linux-work/lib/vsprintf.c 2008-10-17 14:04:58.000000000 +1100 @@ -24,6 +24,7 @@ #include #include #include +#include #include /* for PAGE_SIZE */ #include @@ -550,18 +551,38 @@ static char *symbol_string(char *buf, ch #endif } +static char *resource_string(char *buf, char *end, struct resource *res, int field_width, int precision, int flags) +{ + /* room for the actual numbers, the two "0x", -, [, ] and the final zero */ + char sym[4*sizeof(resource_size_t) + 8]; + char *p = sym, *pend = sym + sizeof(sym); + + *p++ = '['; + p = number(p, pend, res->start, 16, -1, -1, SPECIAL | SMALL); + *p++ = '-'; + p = number(p, pend, res->end, 16, -1, -1, SPECIAL | SMALL); + *p++ = ']'; + *p = 0; + + return string(buf, end, sym, field_width, precision, flags); +} + /* * Show a '%p' thing. A kernel extension is that the '%p' is followed * by an extra set of alphanumeric characters that are extended format * specifiers. * - * Right now we just handle 'F' (for symbolic Function descriptor pointers) - * and 'S' (for Symbolic direct pointers), but this can easily be - * extended in the future (network address types etc). - * - * The difference between 'S' and 'F' is that on ia64 and ppc64 function - * pointers are really function descriptors, which contain a pointer the - * real address. + * Right now we handle: + * + * - 'F' For symbolic Function descriptor pointers + * - 'S' For Symbolic direct pointers), but this can easily be + * extended in the future (network address types etc). + * - 'R' For a struct resouce pointer, it prints the range of + * addresses (not the name nor the flags) + * + * Note: The difference between 'S' and 'F' is that on ia64 and ppc64 + * function pointers are really function descriptors, which contain a + * pointer the real address. */ static char *pointer(const char *fmt, char *buf, char *end, void *ptr, int field_width, int precision, int flags) { @@ -571,6 +592,8 @@ static char *pointer(const char *fmt, ch /* Fallthrough */ case 'S': return symbol_string(buf, end, ptr, field_width, precision, flags); + case 'R': + return resource_string(buf, end, ptr, field_width, precision, flags); } flags |= SMALL; if (field_width == -1) { @@ -590,6 +613,7 @@ static char *pointer(const char *fmt, ch * This function follows C99 vsnprintf, but has some extensions: * %pS output the name of a text symbol * %pF output the name of a function pointer + * %pR output the address range in a struct resource * * The return value is the number of characters which would * be generated for the given input, excluding the trailing -- 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/