From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergey Senozhatsky Subject: Re: [PATCH V11 0/5] hash addresses printed with %p Date: Thu, 30 Nov 2017 19:26:38 +0900 Message-ID: <20171130102638.GA434@jagdpanzerIV> References: <1511921105-3647-1-git-send-email-me@tobin.cc> <20171129152040.ed5b28c198093de8968aac9b@linux-foundation.org> <28ab447e24684c58a5e03af44edd6d5a@AcuMS.aculab.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: 'Andrew Morton' , "Tobin C. Harding" , "kernel-hardening@lists.openwall.com" , Linus Torvalds , "Jason A. Donenfeld" , Theodore Ts'o , Kees Cook , 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 Return-path: Content-Disposition: inline In-Reply-To: <28ab447e24684c58a5e03af44edd6d5a@AcuMS.aculab.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On (11/30/17 10:23), David Laight wrote: [..] > > Maybe I'm being thick, but... if we're rendering these addresses > > unusable by hashing them, why not just print something like > > "" in their place? That loses the uniqueness thing but I > > wonder how valuable that is in practice? > > My worry is that is you get a kernel 'oops' print with actual register > values you have no easy way of tying an address or address+offset to > the corresponding hash(address) printed elsewhere. print the existing hash:pointer mappings in panic()? [if we can do that] -ss