From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 59494C00140 for ; Mon, 8 Aug 2022 09:45:22 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=t3BU8WoAb28DxGX0RIswqzwd59iFCMAKdXMJNKNQqXY=; b=L8uLFe9TqvZVIZ VKnrvGvlQVgoE6wDDC6vqQhW5nY+6UwG9XYXV+BEut4rVSPg1p+7wqqg/CYJlbkYfmXFcYNVaNuHA X4Z0oe2kXdxvAFR2VmmA0ACtfWO/Wyi0E3/iwy2yEbU37ldM2RcgVFlpVqwzoFD59QBdx7lnlAVBn JMGImCWfFBL6ClV+uodArb04dMjZyo7Mu4CrVkbsyh+57HSHSH/l/CT+ZBR6s+Ep9EnKTXe4mpz/P RNZUDn7US5IOBlqZDVhDK11ezCK/OXq2QPnNRBPv3GN1a+kNEheJkvQT60shE0zC1MITtnCmREKI0 0r6urGb1468aRsjQm4Bw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oKzIx-00CjVm-WF; Mon, 08 Aug 2022 09:44:12 +0000 Received: from pandora.armlinux.org.uk ([2001:4d48:ad52:32c8:5054:ff:fe00:142]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oKzIu-00Cj1H-7O for linux-arm-kernel@lists.infradead.org; Mon, 08 Aug 2022 09:44:09 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=armlinux.org.uk; s=pandora-2019; h=Sender:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id: List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive; bh=jabxQNmKugX49duLCRQ1WxMQ73IJlWHxcJdvAYi/ZBE=; b=i59R3fYUn+GkU9gh8J01B+Jci2 8u6oRdOJM59SLJ9LoeHMpXvk6LWZcrP48cIqymCTEFH3Q4/xWKwd8iPqqJABo20JHYDnP576C/AUD MqxdUvz9EkoN8qAUskgdRpw7cOrrs33snGkeSqBpfghmx6lh24pJMs5lKsFK/2W1WAmH/hYUUA36q CR5fwThXeAq8bX0xWv5jYu9PMpF1yiKgMdVpcOhQWVx3dkjyW156fq1GczQxEmFFinsoHPIFecsDE OYRieMMqGBnXUrtI+zuhi1q/+Yj09hNaJJ0ZXCkQiXv4d+oZfpPFRX3Ns/Vq35oXEIJd9eAd91GMr xO1LBn8A==; Received: from shell.armlinux.org.uk ([fd8f:7570:feb6:1:5054:ff:fe00:4ec]:33716) by pandora.armlinux.org.uk with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1oKzGm-0007tv-Ay; Mon, 08 Aug 2022 10:41:56 +0100 Received: from linux by shell.armlinux.org.uk with local (Exim 4.94.2) (envelope-from ) id 1oKzGj-0005Kl-N4; Mon, 08 Aug 2022 10:41:54 +0100 Date: Mon, 8 Aug 2022 10:41:53 +0100 From: "Russell King (Oracle)" To: Zhen Lei Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, patches@armlinux.org.uk Subject: Re: [PATCH] ARM: Remove the special printing format of pc and lr in __show_regs() Message-ID: References: <20220801032016.1524-1-thunder.leizhen@huawei.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220801032016.1524-1-thunder.leizhen@huawei.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220808_024408_283787_6E235FF4 X-CRM114-Status: GOOD ( 19.18 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Aug 01, 2022 at 11:20:16AM +0800, Zhen Lei wrote: > Currently, instruction pointers are printed in [<%08lx>] format to make > them more visible. However, it is not necessary in __show_regs() because > they have the prefix 'pc :' or 'lr :', and it is also inconsistent with > that of other registers, which causes misalignment. The formatting is not "to make them more visible" - it was to mark the addresses that we wanted the ksymoops utility to translate to kernel symbols before we had kallsyms in the kernel. If one disables kallsyms, then we still need a way to translate kernel addresses to symbols. I notice there is a script which helps with this that is part of the kernel source - scripts/decode_stacktrace.sh. I haven't tried this on arm32 since I always use kallsyms - and I suspect that is rather universally true as it avoids needing System.map files etc to decode the oops. That said, if you're building a kernel for small systems, you probably don't want the overhead of kallsyms. So, there's an argument for keeping it - it's an API in that it provides hints to scripting to identify which values need to be converted to symbols. There's also the argument for getting rid of it, which is that hardly anyone does that anymore. The question is, which is the more important argument, and I don't think there's a definite answer. So I'm inclined to leave this as-is. -- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 40Mbps down 10Mbps up. Decent connectivity at last! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel