From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pg0-x243.google.com (mail-pg0-x243.google.com [IPv6:2607:f8b0:400e:c05::243]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 3xyHZn2qKfzDqXn for ; Thu, 21 Sep 2017 10:30:41 +1000 (AEST) Received: by mail-pg0-x243.google.com with SMTP id u18so2499090pgo.1 for ; Wed, 20 Sep 2017 17:30:41 -0700 (PDT) Date: Thu, 21 Sep 2017 09:30:34 +0900 From: Sergey Senozhatsky To: Helge Deller Cc: Sergey Senozhatsky , Tony Luck , Fenghua Yu , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , James Bottomley , Petr Mladek , Steven Rostedt , Andrew Morton , Jessica Yu , Alexei Starovoitov , linux-ia64@vger.kernel.org, linux-parisc@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [RFC][PATCH v2 0/7] printk/ia64/ppc64/parisc64: let's deprecate %pF/%pf printk specifiers Message-ID: <20170921003034.GB773@jagdpanzerIV.localdomain> References: <20170920162910.32053-1-sergey.senozhatsky@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On (09/20/17 22:14), Helge Deller wrote: > On 20.09.2017 18:29, Sergey Senozhatsky wrote: > > This patch set attempts to move ia64/ppc64/parisc64 C function > > pointer ABI details out of printk() to arch code. Function dereference > > code now checks if a pointer belongs to a .opd ELF section and dereferences > > that pointer only if it does. The kernel and modules have their own .opd > > sections that's why I use two different ARCH functions: for kernel and > > for module pointer dereference. > > ...> *** A BIG NOTE *** > > I don't own ia64/ppc64/parisc64 hardware, so the patches are not > > tested. Sorry about that! > > > I just now tested your patch series successfully on parisc64. > > You may add to the whole series: > Tested-by: Helge Deller # parisc64 thanks, Helge! > > Another note: > > I need to check what is BPF symbol lookup and do we need to > > do any dereference there. > > Not relevant for parisc, since we don't support it yet. so that was my suspicion as well. at glance it didn't look like bpf symbol resolution would work on platforms that do description dereference. -ss