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.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,MSGID_FROM_MTA_HEADER,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_2 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 A58CDC47082 for ; Tue, 8 Jun 2021 07:25:04 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 5A8F561073 for ; Tue, 8 Jun 2021 07:25:04 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5A8F561073 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=synaptics.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org 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:MIME-Version:References:In-Reply-To: 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=SfXe3Z/5RDWLtVDqB8sfFVedOUQIAldohKT32VwQr0s=; b=gxGNjUWdbQt5o8 b+6VBQ7qEFQpYYHziXm8h/MA35QWUwYIsJIIfWJY+iyA94AdAQs62kYqbybpCSKmfxsCDPQs5KvMS /2yd8z1iLHHsyFTMtHtlkxIZKcNLVM2D4IYlOiUTVYG77D6ejzVBQ/82jflgU4ALF1CRwxTGPi6mq S7le/Ttrcf2W0IyQHCuQwqGQXk0Cp38ACfAUh5i2qrnko5pLkdM3ngfiLD7NBV3yfUOGquFpw5/HN 4hEGYeVuWE6wZ3166f6OHdF9CnPtaUZ+JS1UAJMuOfTZ6Pxcc8q6r4JQi4W25YNHtZRQKsV9O7oTv nM1uTqvCRkAGdQXMzX9g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1lqW6N-006qh8-Jt; Tue, 08 Jun 2021 07:24:43 +0000 Received: from mail-dm6nam10on2048.outbound.protection.outlook.com ([40.107.93.48] helo=NAM10-DM6-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1lqW6K-006qgA-Ln for linux-riscv@lists.infradead.org; Tue, 08 Jun 2021 07:24:42 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ZqlqRD2fTJByTY7dol34WBo2An/PyfWzOlepXuoKuhMpKqz3Up/cPsVR88PqByoZEyzAcOsQiMQxFCAAH7Rl5iaGQTarxe1fOHjzxpIYtX5ahqEIrX0X/X/x7EsiG7T4j8WzVW3jCbPuoMdxfpsgpqx/Co5snr9ragDwI77pxS6ZPtuOfW/mNf+GYStUSozp5drpuKMZqHEYCg3r2aYKMDEusYq+aBh0trbJ4Vb632E14V7hL+Dhg9W0x8iT3FSndoIW++f877bfLQJ96ntbzEkjqOqwd9ogG95UgHPnbfmQ1HYJC88s34waOm775RZPQWMj+3RCthqRvgY53J3atw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=K9iufL7HbpDDcJY1BA63yvDZ6PVmiejL3UYYj+309Yk=; b=cVPrl8IUdK3Y94lGzCnWj5Y1HFicypQ0afUyc1yrgWK3tprjmETGobaKx7gLIgSsdE2RQnPOdhsDqsl3tBiZ9UsMqwAPb23QbZKhtSP20UXtwhabTevJln4dVw2SBiGd3spdUrpVLCEKl0h0A65lYQsNYocSSl/o1OuiduaM2diKZai/UFRF2guHxsgz551JSwLbj3Zf0LdJO3OZk59FfNauAooNLW9eixt9DZlbs/38mqKNkflJGmQreG+hm/4vemvzcidZ9jqa/EPwxkvmyy2BDc+iU+pSP5pEyj0cOMbiGV9A/1aWlBTxPy+4TiXnLc/f9MkWKKsNXUqbzW/2/Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=synaptics.com; dmarc=pass action=none header.from=synaptics.com; dkim=pass header.d=synaptics.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Synaptics.onmicrosoft.com; s=selector2-Synaptics-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=K9iufL7HbpDDcJY1BA63yvDZ6PVmiejL3UYYj+309Yk=; b=ksKD1V6HXu3j8V6DPE00TYUIqm7kXc+vH9vRT+eQgpayihsukuprpC2JsovCfHb64AOgIvLMWXOswRSFmrzakVdfvd8Vhpq941f2Ddjp627h3Y+TLQyxFbBLRAcTohZKrXqjoGXNd0E9zuHS6P7NZ6cydr+wrCblIkaTHl9M3SM= Authentication-Results: konsulko.com; dkim=none (message not signed) header.d=none; konsulko.com; dmarc=none action=none header.from=synaptics.com; Received: from BN9PR03MB6058.namprd03.prod.outlook.com (2603:10b6:408:137::15) by BN9PR03MB6138.namprd03.prod.outlook.com (2603:10b6:408:11b::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4195.24; Tue, 8 Jun 2021 07:24:34 +0000 Received: from BN9PR03MB6058.namprd03.prod.outlook.com ([fe80::502a:5487:b3ee:f61c]) by BN9PR03MB6058.namprd03.prod.outlook.com ([fe80::502a:5487:b3ee:f61c%4]) with mapi id 15.20.4219.021; Tue, 8 Jun 2021 07:24:34 +0000 Date: Tue, 8 Jun 2021 15:24:21 +0800 From: Jisheng Zhang To: Vitaly Wool Cc: Palmer Dabbelt , Jisheng Zhang , Paul Walmsley , Albert Ou , linux-riscv , LKML Subject: Re: [PATCH] riscv: code patching only works on !XIP_KERNEL Message-ID: <20210608152406.36a973d0@xhacker.debian> In-Reply-To: References: X-Mailer: Claws Mail 3.17.8 (GTK+ 2.24.33; x86_64-pc-linux-gnu) X-Originating-IP: [192.147.44.204] X-ClientProxiedBy: SJ0PR03CA0206.namprd03.prod.outlook.com (2603:10b6:a03:2ef::31) To BN9PR03MB6058.namprd03.prod.outlook.com (2603:10b6:408:137::15) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from xhacker.debian (192.147.44.204) by SJ0PR03CA0206.namprd03.prod.outlook.com (2603:10b6:a03:2ef::31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4195.22 via Frontend Transport; Tue, 8 Jun 2021 07:24:31 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d8bc3d18-e134-4ae9-6cae-08d92a4e76e6 X-MS-TrafficTypeDiagnostic: BN9PR03MB6138: X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:9508; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: RyRVrn8STPFExruiM6LgplmoF+jWq0N7p1470o9z17zgCxKRfQriDdMs1aMD8yxHrmWm5uUB20Z4hFQx4Gbk25fAqZI5/xcwoqKElZokHoRNUNph0jWQnJ2vebHgmFpsSgfox1+FPlFRUfGnS367SZ38EZAkZMN2b7Ff3jiEWF/3rThOYp0jP+FKf5+ENWL1lacvVLrbyFIiSBL1XllE0/a0ixVSw6WW4h/eHfc8c5FzIEY5GaYAwZi+BYCAd25eje7NRIgiLE3j565RWs4TDggouBDPK13PoRbMRjSbw7BfTUhOMy5SuWRNLNTWobCnYN0krLm3i3xKlcKOUwFcr6m/n72BwyqYAiNV49Sg/buhnAC/aEr3mTis7kSnJh2eMTIbT0mgNieftdaGQv9d2w0mMJTrQy9acAHSQggrKhdzby1BnYw54rWYP1nfIcpVJAthXNPqpMSU2YU1o8ABQCrD3ngpqBDlsp1BznOI+w+ZvwOgiyZAKGUguFmtcg0eEQpCkX9GwT0R5i6jxuZ4cym5Zc/OAnFwA8my0W+tQEIMYqmmsNeRtS9rTFGdAg9nidqLxcL+pfU5mlnPZ50RaSoWqPQ4MUCkw6sGDfO3V4NEVUYsajP69GO+60X7N2nfAGs79l47iUo/x1ex3KuFDpUd0YTKRALAS32gWkxvhRM= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BN9PR03MB6058.namprd03.prod.outlook.com; PTR:; CAT:NONE; SFS:(346002)(366004)(396003)(376002)(136003)(39850400004)(66476007)(9686003)(66556008)(5660300002)(2906002)(1076003)(38100700002)(956004)(38350700002)(7696005)(8676002)(6506007)(55016002)(53546011)(52116002)(66946007)(8936002)(186003)(54906003)(6916009)(16526019)(86362001)(316002)(6666004)(478600001)(4326008)(83380400001)(26005); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ZJ+7NUhSZCzqu+C+urZZfGqYxIXwYU3Dq1GthzjDbaYcBkHOLJCPTobDuNqZ?= =?us-ascii?Q?8Zgd5CqkxG1AARy8AXnJv6ifPsq7Gqh7XxjFSORvOOZx26YUvAY+mRiTL8Vr?= =?us-ascii?Q?tp5E1N3FUkc+75p+UruvVhkMmhviAMLSk6x+zf1RS/EfmRfRMZXzZCojr2Ee?= =?us-ascii?Q?HHWwUDlZNu/8a2R0ydT1wA3zyZOC/YxRlyFJLxG2Ux2cUtTBxG5WD93BwBCW?= =?us-ascii?Q?QkpAqUYxp3pdEhZtr8bIHzngkmTuM0UjScSFSZOyvrW/LJacgt0Y5OWcycgK?= =?us-ascii?Q?xbkmUlfFuz5sLWBUbj8B13Oi5cOqpOUtwRarlhHJhEExoCFi1D9gVXD5jmq7?= =?us-ascii?Q?jb09JgVD3lzMJZrqnGNhq6TkdXnoCBWGf7RnopgMyKDFf23eqZcqO8uo4fTg?= =?us-ascii?Q?TGIxa1ET6y7WAXIVgeNNWNzukWFh1xTlhwjosM9fc0Zh1+qXzVEXX7OuDjdA?= =?us-ascii?Q?/Mx51QkwTFvkBWI6wgdd5i6z2OPmv8uswxmOjMAz8WN62d/cOhXc/LnAuZ21?= =?us-ascii?Q?9mkcTV5bh/odRuWDA6RPTHmSga8K1Vhxw+B46sDSJEuCDf5gwIZe0y0lRGSX?= =?us-ascii?Q?rxn4h0JKdLNtYwh9VE4RGtOgat2hwD1NgaVpy942bwCvR56Q34P2jsA1jndV?= =?us-ascii?Q?QJ6+bHic88qHnapH53m7X7gGn6f+pXSMsrcEoXhFddk+ZEBfp38Agi0/2Tv1?= =?us-ascii?Q?XcfE20yxRCLb8sty74cnGo7W5fqXSoZVqtrN5BdoHkZtFfwAxGrzh5xa2JF8?= =?us-ascii?Q?q6qIWaLNyhZ8yie8RUabGtl9qcznIQEZSLw+cU5MJO0orNMj0OeDmHUyd8/b?= =?us-ascii?Q?7OgaOfnwFa/5omaN9zdW/GsUeyugIsas9Z1oz2MR4oy2bdc0mcrbOTzHAzzF?= =?us-ascii?Q?Ntio5zAgqtQoWm/LQ93DE8/DshlCisoIGcJLjgV4i8pwczB4MzHdHq9cjk7V?= =?us-ascii?Q?Xwtm6W7XWQCMZs5HsP0pw35ib0wHApPpDXQCo7n1jaQKATkU4M8yyJd/gnvd?= =?us-ascii?Q?idOmROdpViOm1hDNLlhtDDWaZVJKsbiDS4cVLeb5DhyDpYlLm11kN8mIbvxM?= =?us-ascii?Q?H0ZusjUIIahOMmfpzdigubccFyQGPYrBIuJdvCXJEYMVfxN5Wea+zK+0a/K5?= =?us-ascii?Q?cwe1qK2Q9dlGi2FMc9EmTwPfx0FRey40hvUfMmJcsyYXBCAqrjxuhlAB4XpK?= =?us-ascii?Q?y+y4p2WuDKP+kgIuodJuSJkRUCNS2DI799pXuuHiWxPtDJnNEevml6dJbD/6?= =?us-ascii?Q?sdZ+q7covb5asFJcQgaj7vLKOWvCqkPpFiyJvS6tvhRtm1lCzGFY+NTyrv85?= =?us-ascii?Q?0dgtXWBduRPll9tKV9VNdbcT?= X-OriginatorOrg: synaptics.com X-MS-Exchange-CrossTenant-Network-Message-Id: d8bc3d18-e134-4ae9-6cae-08d92a4e76e6 X-MS-Exchange-CrossTenant-AuthSource: BN9PR03MB6058.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Jun 2021 07:24:34.1548 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 335d1fbc-2124-4173-9863-17e7051a2a0e X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: Dc9Fm6cOMqHrDlOLWXrg6X/Hh6HR8MvMns1/VOnP0+mbM70Wyqoef0Y3HvuLjcjdoyymhG92j9CR/KIrKN+jlQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR03MB6138 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210608_002440_802235_21DC9B15 X-CRM114-Status: GOOD ( 32.64 ) X-BeenThere: linux-riscv@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-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org On Mon, 24 May 2021 21:32:38 +0200 Vitaly Wool wrote: > > > On Sat, May 22, 2021 at 7:15 PM Palmer Dabbelt wrote: > > > > On Mon, 10 May 2021 11:19:30 PDT (-0700), vitaly.wool@konsulko.com wrote: > > > On Mon, May 10, 2021 at 6:35 PM Jisheng Zhang wrote: > > >> > > >> From: Jisheng Zhang > > >> > > >> Some features which need code patching such as KPROBES, DYNAMIC_FTRACE > > >> KGDB can only work on !XIP_KERNEL. Add dependencies for these features > > >> that rely on code patching. > > > > > > Since we can define extra RW sections to place some tiny code bits in > > > RAM, I would suggest that you hold back this patch for a while. > > > I am not going to support the idea of e. g. compiling KGDB support > > > into a XIP kernel, but disabling the whole HAVE_ARCH_JUMP_LABEL is > > > hardly the way to go. > > > > Do you have any updates on this? The SOC_SIFIVE disable thread floated > > back to the top of my inbox and I didn't see anything there. I'd prefer > > to avoid chasing around Kconfig disables for a bunch of stuff that we're > > just going to fix soon, but if this is going to be a big change then > > it's likely for next cycle anyway so in that case I'll start picking up > > these compile disable patches. > > Sorry, was caught up in fixing other stuff but now I'm on it so expect > some updates soon. Hi Vitaly, Based on your "[PATCH] riscv: xip: support runtime trap patching", IMHO, it solves ERRATA patching issue(And it makes use of the fact that the ERRATA only touches traps at the moment), but can't make code patching tech such as KPROBES, DYNAMIC_FTRACE etc. work properly, so my patch is still needed. What do you think? Regards > > Best regards, > Vitaly > > > Not trying to rush any thing here, just don't want to have a bunch of > > broken configs. > > > > > > > > Best regards, > > > Vitaly > > > > > >> Signed-off-by: Jisheng Zhang > > >> --- > > >> Hi, > > >> > > >> Currently, the RISCV_ERRATA_ALTERNATIVE also only works on !XIP_KERNEL > > >> but from the maillist it seem there's effort to make the co-exist, so > > >> I drop RISCV_ERRATA_ALTERNATIVE dependency on !XIP_KERNEL. > > >> > > >> Thanks > > >> > > >> arch/riscv/Kconfig | 18 +++++++++--------- > > >> 1 file changed, 9 insertions(+), 9 deletions(-) > > >> > > >> diff --git a/arch/riscv/Kconfig b/arch/riscv/Kconfig > > >> index a8ad8eb76120..61320b94ef97 100644 > > >> --- a/arch/riscv/Kconfig > > >> +++ b/arch/riscv/Kconfig > > >> @@ -60,11 +60,11 @@ config RISCV > > >> select GENERIC_TIME_VSYSCALL if MMU && 64BIT > > >> select HANDLE_DOMAIN_IRQ > > >> select HAVE_ARCH_AUDITSYSCALL > > >> - select HAVE_ARCH_JUMP_LABEL > > >> - select HAVE_ARCH_JUMP_LABEL_RELATIVE > > >> + select HAVE_ARCH_JUMP_LABEL if !XIP_KERNEL > > >> + select HAVE_ARCH_JUMP_LABEL_RELATIVE if !XIP_KERNEL > > >> select HAVE_ARCH_KASAN if MMU && 64BIT > > >> select HAVE_ARCH_KASAN_VMALLOC if MMU && 64BIT > > >> - select HAVE_ARCH_KGDB > > >> + select HAVE_ARCH_KGDB if !XIP_KERNEL > > >> select HAVE_ARCH_KGDB_QXFER_PKT > > >> select HAVE_ARCH_MMAP_RND_BITS if MMU > > >> select HAVE_ARCH_SECCOMP_FILTER > > >> @@ -79,9 +79,9 @@ config RISCV > > >> select HAVE_GCC_PLUGINS > > >> select HAVE_GENERIC_VDSO if MMU && 64BIT > > >> select HAVE_IRQ_TIME_ACCOUNTING > > >> - select HAVE_KPROBES > > >> - select HAVE_KPROBES_ON_FTRACE > > >> - select HAVE_KRETPROBES > > >> + select HAVE_KPROBES if !XIP_KERNEL > > >> + select HAVE_KPROBES_ON_FTRACE if !XIP_KERNEL > > >> + select HAVE_KRETPROBES if !XIP_KERNEL > > >> select HAVE_PCI > > >> select HAVE_PERF_EVENTS > > >> select HAVE_PERF_REGS > > >> @@ -230,11 +230,11 @@ config ARCH_RV64I > > >> bool "RV64I" > > >> select 64BIT > > >> select ARCH_SUPPORTS_INT128 if CC_HAS_INT128 && GCC_VERSION >= 50000 > > >> - select HAVE_DYNAMIC_FTRACE if MMU && $(cc-option,-fpatchable-function-entry=8) > > >> + select HAVE_DYNAMIC_FTRACE if !XIP_KERNEL && MMU && $(cc-option,-fpatchable-function-entry=8) > > >> select HAVE_DYNAMIC_FTRACE_WITH_REGS if HAVE_DYNAMIC_FTRACE > > >> - select HAVE_FTRACE_MCOUNT_RECORD > > >> + select HAVE_FTRACE_MCOUNT_RECORD if !XIP_KERNEL > > >> select HAVE_FUNCTION_GRAPH_TRACER > > >> - select HAVE_FUNCTION_TRACER > > >> + select HAVE_FUNCTION_TRACER if !XIP_KERNEL > > >> select SWIOTLB if MMU > > >> > > >> endchoice > > >> -- > > >> 2.31.0 > > >> > > >> > > >> > > >> _______________________________________________ _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv