From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753910AbdLFIpp (ORCPT ); Wed, 6 Dec 2017 03:45:45 -0500 Received: from mail-pl0-f44.google.com ([209.85.160.44]:35654 "EHLO mail-pl0-f44.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750989AbdLFIpn (ORCPT ); Wed, 6 Dec 2017 03:45:43 -0500 X-Google-Smtp-Source: AGs4zMZhYNjig7vWOHelg8rZviiCDCaz5a0TINp2mng7tQZcrPl3/siXAaKRoB3UgPH9MuvwHwrtIg== Date: Wed, 6 Dec 2017 17:45:37 +0900 From: Sergey Senozhatsky To: Geert Uytterhoeven Cc: Linus Torvalds , Sergey Senozhatsky , Randy Dunlap , David Laight , Kees Cook , "Tobin C. Harding" , "kernel-hardening@lists.openwall.com" , "Jason A. Donenfeld" , "Theodore Ts'o" , Paolo Bonzini , Tycho Andersen , "Roberts, William C" , Tejun Heo , Jordan Glover , Greg KH , Petr Mladek , Joe Perches , Ian Campbell , Sergey Senozhatsky , Catalin Marinas , Will Deacon , Steven Rostedt , Chris Fries , Dave Weinstein , Daniel Micay , Djalal Harouni , Radim =?iso-8859-1?Q?Krcm=E1r?= , Linux Kernel Mailing List , Network Development , David Miller , Stephen Rothwell , Andrey Ryabinin , Alexander Potapenko , Dmitry Vyukov , Andrew Morton Subject: Re: [PATCH V11 4/5] vsprintf: add printk specifier %px Message-ID: <20171206084537.GA462@jagdpanzerIV> References: <1511921105-3647-5-git-send-email-me@tobin.cc> <324abe19f1ee4177911d2fc48899ac8a@AcuMS.aculab.com> <596c917e-3183-d80a-89d4-97e0eb5dc077@infradead.org> <20171206013657.GA479@jagdpanzerIV> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On (12/06/17 09:32), Geert Uytterhoeven wrote: [..] > >> show_fault_oops(struct pt_regs *regs, unsigned long error_code, > >> unsigned long address) > >> ... > >> printk(KERN_CONT " at %p\n", (void *) address); > >> printk(KERN_ALERT "IP: %pS\n", (void *)regs->ip); > > > > So %pS isn't %p, and shows the symbolic name. > > If the symbolic name is available. > Else it prints the non-hashed pointer value (FTR). hm, indeed. and !CONFIG_KALLSYMS config turns %pS/%ps into special_hex_number(). -ss