Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030260AbbENRKn (ORCPT ); Thu, 14 May 2015 13:10:43 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:29243 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933455AbbENRKf (ORCPT ); Thu, 14 May 2015 13:10:35 -0400 From: Sasha Levin To: linux-mm@kvack.org Cc: akpm@linux-foundation.org, linux-kernel@vger.kernel.org, kirill@shutemov.name, Sasha Levin Subject: [PATCH 02/11] mm: debug: deal with a new family of MM pointers Date: Thu, 14 May 2015 13:10:05 -0400 Message-Id: <1431623414-1905-3-git-send-email-sasha.levin@oracle.com> X-Mailer: git-send-email 2.1.0 In-Reply-To: <1431623414-1905-1-git-send-email-sasha.levin@oracle.com> References: <1431623414-1905-1-git-send-email-sasha.levin@oracle.com> X-Source-IP: aserv0022.oracle.com [141.146.126.234] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2011 Lines: 63 This teaches our printing functions about a new family of MM pointer that it could now print. I've picked %pZ because %pm and %pM were already taken, so I figured it doesn't really matter what we go with. We also have the option of stealing one of those two... Signed-off-by: Sasha Levin --- lib/vsprintf.c | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) diff --git a/lib/vsprintf.c b/lib/vsprintf.c index 8243e2f..9350904 100644 --- a/lib/vsprintf.c +++ b/lib/vsprintf.c @@ -1375,6 +1375,21 @@ char *comm_name(char *buf, char *end, struct task_struct *tsk, return string(buf, end, name, spec); } +static noinline_for_stack +char *mm_pointer(char *buf, char *end, struct task_struct *tsk, + struct printf_spec spec, const char *fmt) +{ + switch (fmt[1]) { + default: + spec.base = 16; + spec.field_width = sizeof(unsigned long) * 2 + 2; + spec.flags |= SPECIAL | SMALL | ZEROPAD; + return number(buf, end, (unsigned long) ptr, spec); + } + + return buf; +} + int kptr_restrict __read_mostly; /* @@ -1463,6 +1478,7 @@ int kptr_restrict __read_mostly; * (legacy clock framework) of the clock * - 'Cr' For a clock, it prints the current rate of the clock * - 'T' task_struct->comm + * - 'Z' Outputs a readable version of a type of memory management struct. * * Note: The difference between 'S' and 'F' is that on ia64 and ppc64 * function pointers are really function descriptors, which contain a @@ -1615,6 +1631,8 @@ char *pointer(const char *fmt, char *buf, char *end, void *ptr, spec, fmt); case 'T': return comm_name(buf, end, ptr, spec, fmt); + case 'Z': + return mm_pointer(buf, end, ptr, spec, fmt); } spec.flags |= SMALL; if (spec.field_width == -1) { -- 1.7.10.4 -- 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/