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 X-Spam-Level: X-Spam-Status: No, score=-15.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C07C8C4338F for ; Mon, 26 Jul 2021 14:40:30 +0000 (UTC) Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 5D7E160EB2 for ; Mon, 26 Jul 2021 14:40:30 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 5D7E160EB2 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=me.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=lists.ozlabs.org Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4GYMzK0vwMz3dDd for ; Tue, 27 Jul 2021 00:40:29 +1000 (AEST) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=me.com header.i=@me.com header.a=rsa-sha256 header.s=1a1hai header.b=HUlQhDYK; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=me.com (client-ip=17.58.6.48; helo=pv50p00im-hyfv10021501.me.com; envelope-from=sxwjean@me.com; receiver=) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=me.com header.i=@me.com header.a=rsa-sha256 header.s=1a1hai header.b=HUlQhDYK; dkim-atps=neutral Received: from pv50p00im-hyfv10021501.me.com (pv50p00im-hyfv10021501.me.com [17.58.6.48]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4GYMxs41RFz2xfT for ; Tue, 27 Jul 2021 00:39:13 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=me.com; s=1a1hai; t=1627309903; bh=5LoKxpiOaCX8VNLa2cka/PO0i9FkUE10bIa3XTPRxU0=; h=From:To:Subject:Date:Message-Id:MIME-Version; b=HUlQhDYKtl9TNTp0Zk5JJ/B1nfOdSdSzFeHisHn3T86rFTUs4ZY34P/yLeK2STaVv x7/zhdkitjTgm6lRyI0MmAydyaLcRXi1tAwvJ6Rqt36Qv/9woaRvNAONWCtW589bfX 1NLTnUtC9tzmwSLsXYHO+lb4DmbAyiGwUL0Cgu+4nHnl/f71/vHpPKOBVPp2vxToja q8IQO/ErUA+SdNk9Y2QDb7zAZm7T8tkX0OUZvCH9PwOmkyFsa2Z4th9vSjebMmocX3 bb1ivHLkWTHYb4pFh+t/ZCwYXfpQYoPZtyHnSHXFOBeSyZm1j+bJxKYXkIqd3cwyJq 6vXF6o9T6ncLg== Received: from xiongwei.. (unknown [120.245.2.8]) by pv50p00im-hyfv10021501.me.com (Postfix) with ESMTPSA id 6F94BB4031D; Mon, 26 Jul 2021 14:31:38 +0000 (UTC) From: sxwjean@me.com To: linuxppc-dev@lists.ozlabs.org Subject: [RFC PATCH 3/4] powerpc: Optimize register usage for dear register Date: Mon, 26 Jul 2021 22:30:52 +0800 Message-Id: <20210726143053.532839-3-sxwjean@me.com> X-Mailer: git-send-email 2.30.2 In-Reply-To: <20210726143053.532839-1-sxwjean@me.com> References: <20210726143053.532839-1-sxwjean@me.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.391, 18.0.790 definitions=2021-07-26_06:2021-07-26, 2021-07-26 signatures=0 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 malwarescore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 mlxscore=0 mlxlogscore=522 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-2009150000 definitions=main-2107260082 X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: ravi.bangoria@linux.ibm.com, Xiongwei Song , aneesh.kumar@linux.ibm.com, oleg@redhat.com, linux-kernel@vger.kernel.org, efremov@linux.com, paulus@samba.org, npiggin@gmail.com, peterx@redhat.com, akpm@linux-foundation.org, sandipan@linux.ibm.com Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" From: Xiongwei Song Create an anonymous union for dar and dear regsiters, we can reference dear to get the effective address when CONFIG_4xx=y or CONFIG_BOOKE=y. Otherwise, reference dar. This makes code more clear. Signed-off-by: Xiongwei Song --- arch/powerpc/include/asm/ptrace.h | 5 ++++- arch/powerpc/include/uapi/asm/ptrace.h | 5 ++++- arch/powerpc/kernel/process.c | 2 +- arch/powerpc/kernel/ptrace/ptrace.c | 2 ++ arch/powerpc/kernel/traps.c | 5 ++++- arch/powerpc/mm/fault.c | 2 +- 6 files changed, 16 insertions(+), 5 deletions(-) diff --git a/arch/powerpc/include/asm/ptrace.h b/arch/powerpc/include/asm/ptrace.h index c252d04b1206..fa725e3238c2 100644 --- a/arch/powerpc/include/asm/ptrace.h +++ b/arch/powerpc/include/asm/ptrace.h @@ -43,7 +43,10 @@ struct pt_regs unsigned long mq; #endif unsigned long trap; - unsigned long dar; + union { + unsigned long dar; + unsigned long dear; + }; union { unsigned long dsisr; unsigned long esr; diff --git a/arch/powerpc/include/uapi/asm/ptrace.h b/arch/powerpc/include/uapi/asm/ptrace.h index e357288b5f34..9ae150fb4c4b 100644 --- a/arch/powerpc/include/uapi/asm/ptrace.h +++ b/arch/powerpc/include/uapi/asm/ptrace.h @@ -52,7 +52,10 @@ struct pt_regs unsigned long trap; /* Reason for being here */ /* N.B. for critical exceptions on 4xx, the dar and dsisr fields are overloaded to hold srr0 and srr1. */ - unsigned long dar; /* Fault registers */ + union { + unsigned long dar; /* Fault registers */ + unsigned long dear; + }; union { unsigned long dsisr; /* on Book-S used for DSISR */ unsigned long esr; /* on 4xx/Book-E used for ESR */ diff --git a/arch/powerpc/kernel/process.c b/arch/powerpc/kernel/process.c index f74af8f9133c..50436b52c213 100644 --- a/arch/powerpc/kernel/process.c +++ b/arch/powerpc/kernel/process.c @@ -1499,7 +1499,7 @@ static void __show_regs(struct pt_regs *regs) trap == INTERRUPT_DATA_STORAGE || trap == INTERRUPT_ALIGNMENT) { if (IS_ENABLED(CONFIG_4xx) || IS_ENABLED(CONFIG_BOOKE)) - pr_cont("DEAR: "REG" ESR: "REG" ", regs->dar, regs->esr); + pr_cont("DEAR: "REG" ESR: "REG" ", regs->dear, regs->esr); else pr_cont("DAR: "REG" DSISR: %08lx ", regs->dar, regs->dsisr); } diff --git a/arch/powerpc/kernel/ptrace/ptrace.c b/arch/powerpc/kernel/ptrace/ptrace.c index 00789ad2c4a3..969dca8b0718 100644 --- a/arch/powerpc/kernel/ptrace/ptrace.c +++ b/arch/powerpc/kernel/ptrace/ptrace.c @@ -373,6 +373,8 @@ void __init pt_regs_check(void) offsetof(struct user_pt_regs, trap)); BUILD_BUG_ON(offsetof(struct pt_regs, dar) != offsetof(struct user_pt_regs, dar)); + BUILD_BUG_ON(offsetof(struct pt_regs, dear) != + offsetof(struct user_pt_regs, dear)); BUILD_BUG_ON(offsetof(struct pt_regs, dsisr) != offsetof(struct user_pt_regs, dsisr)); BUILD_BUG_ON(offsetof(struct pt_regs, esr) != diff --git a/arch/powerpc/kernel/traps.c b/arch/powerpc/kernel/traps.c index 2164f5705a0b..0796630d3d23 100644 --- a/arch/powerpc/kernel/traps.c +++ b/arch/powerpc/kernel/traps.c @@ -1609,7 +1609,10 @@ DEFINE_INTERRUPT_HANDLER(alignment_exception) } bad: if (user_mode(regs)) - _exception(sig, regs, code, regs->dar); + if (IS_ENABLED(CONFIG_4xx) || IS_ENABLED(CONFIG_BOOKE)) + _exception(sig, regs, code, regs->dear); + else + _exception(sig, regs, code, regs->dar); else bad_page_fault(regs, sig); } diff --git a/arch/powerpc/mm/fault.c b/arch/powerpc/mm/fault.c index 62953d4e7c93..3db6b39a1178 100644 --- a/arch/powerpc/mm/fault.c +++ b/arch/powerpc/mm/fault.c @@ -542,7 +542,7 @@ static __always_inline void __do_page_fault(struct pt_regs *regs) long err; if (IS_ENABLED(CONFIG_4xx) || IS_ENABLED(CONFIG_BOOKE)) - err = ___do_page_fault(regs, regs->dar, regs->esr); + err = ___do_page_fault(regs, regs->dear, regs->esr); else err = ___do_page_fault(regs, regs->dar, regs->dsisr); -- 2.30.2