Received: by 2002:a6b:500f:0:0:0:0:0 with SMTP id e15csp3076593iob; Mon, 16 May 2022 12:32:06 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx9/i4AFmKqX59FNERYaZ0vd3FWsJVQdgZ9BKZ6rk5r4qvsgwhGql2t7Q5ULuic5drWKp0v X-Received: by 2002:a17:90b:164f:b0:1dc:199c:56cc with SMTP id il15-20020a17090b164f00b001dc199c56ccmr21179854pjb.177.1652729526760; Mon, 16 May 2022 12:32:06 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1652729526; cv=none; d=google.com; s=arc-20160816; b=YVByi4poPQizjkZsZM2s3OJPZEN8wDLQbAGzK/9R5gSmzgG1EsDwkgnhxRv7C3nDZI t7VkdU2/K/hzCCCpAcG9cSUZMr3cDLOjjjk/r5v48YYjf0Go3orvUDg8Iuezjzq+ZBRf 0PXCHXbQ7h9rit4Mpq/qhLg458jSO2gGGOzODVVP3m46NAYx4usZ5a5O5UsRRLeI0Yh+ oqz0id+mJ+zuEbfXPbCoLOB146PVmz+UPtMBAldFEmplgN1rQH2OcdpRzfFZYNpVwjDM mgQRAkRUvbOTVPd+Ve8yzme46L4SbvO8gsDPi3qPqH0wg8qJpAnq4/cp+FPOB/pI7dxo cdAA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=+4y1lUQE4jcfytaW0Wa1mHb+7bWWQjJjq9fZCketHKQ=; b=r8iSkc7kiuo38mF/eraqd4M8Ty4QC7F2PQpOa5FT27Mrro3MHk49wBpfzaKvdzA+uf +aX7Vltov8Q2E/SogFQL1Xz44ZGs6q0Lkd8eXJvb+EM37rpCwkEzp6nHfVw8/CyC0XUQ fYUbXraGtfJNwJF3Ak3hwBzoO+gdXRnNyxp+LatIjYzfOV9q645zfmkl64jDxdmOOHZ+ rMPlYruKAbL5NxpXsHYdzhAykzyJcHp1cHcKtWdZ2X9CKbteGU2JomTHWOhhkbW4jQm5 h4MiqZ7t9/Lg8RRfc+fCPzy/Nh4NWaMlYmEN96xJuWzMaPfBQk1KE5GxlJGFI26zJejX MXWg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@vrull.eu header.s=google header.b=X1DNS3iz; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id nd10-20020a17090b4cca00b001bf35bf24d7si196522pjb.63.2022.05.16.12.31.55; Mon, 16 May 2022 12:32:06 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@vrull.eu header.s=google header.b=X1DNS3iz; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242144AbiEPJJm (ORCPT + 99 others); Mon, 16 May 2022 05:09:42 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38504 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S241928AbiEPJJ2 (ORCPT ); Mon, 16 May 2022 05:09:28 -0400 Received: from mail-wm1-x32a.google.com (mail-wm1-x32a.google.com [IPv6:2a00:1450:4864:20::32a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3D57FE038 for ; Mon, 16 May 2022 02:09:25 -0700 (PDT) Received: by mail-wm1-x32a.google.com with SMTP id 1-20020a05600c248100b00393fbf11a05so10479392wms.3 for ; Mon, 16 May 2022 02:09:25 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vrull.eu; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=+4y1lUQE4jcfytaW0Wa1mHb+7bWWQjJjq9fZCketHKQ=; b=X1DNS3izGDmG2ZzNwvKwxubD4dAzvtune5mqBpUfMaC9AZjnraMTJnXgdFUIyF2x8l LpSUaDNEShEjSSelQzC1RgUbQEVH4tRAKa8t5gVFugBLXmG8gwhO+xsowghGh1Z86GSI h68cyD6r7+NpmD0LUWLywlB0zmSOm5GJxaT8l/5sYrnNyYCUtNR16XtgoBxDfZQQMSY7 mmDv4thUTnvZkUNzp8v5ULMPpXBqIlv4QwUiL7c6NFJ+DNVEeXBm7z0Z2imtZrkK/Wq+ Z4gDfU9HjbF9BLHILP8lNkrvw7yNct7x6tCww/34XQf+x906GIwQASqqcESIo1cu6LBE 4UHw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=+4y1lUQE4jcfytaW0Wa1mHb+7bWWQjJjq9fZCketHKQ=; b=temrkMxjXqbppDwCWY43N6QZuE3qAJu2Y8IuF8tD4kfINOTtYLrpKXRMP3iIXM8i6P BYoX4XXJA2vBmLxmZWHsf8EbA+jr87Np+XDtLSGjuPEM9uhVLuLowpOZOLLYb9Lb4gMB JLAhNyBsoCzOG7Gv6uOZXF10U+UWhxux5A2cTMMtHzL73Orw/Yk0K1h4P+B69WM+VAL5 fceAQTg1/Fm4QcPGFp7tJ35SVgujsR8N8TMiDnmx2+NXFRYlqHV0m+Cm8C602TyFcuGW EzaKaCA9/bTobNYomQpsVqNByvzPoiCEfQAjwknP/UZ/STYRhhBM7r3w+rbEXbRT5B1Y zFgw== X-Gm-Message-State: AOAM532rTj50Ww01DGJkeNFDpIVDoJ2LxmF8zQhMZRnjmWhssOuEOgzr fJEfySGYRQSmgkLI8vDXy/Ou0k16LfCJ9aJxUkMG2w== X-Received: by 2002:a05:600c:19c9:b0:394:766a:318d with SMTP id u9-20020a05600c19c900b00394766a318dmr26005246wmq.199.1652692163753; Mon, 16 May 2022 02:09:23 -0700 (PDT) MIME-Version: 1.0 References: <20220511192921.2223629-1-heiko@sntech.de> <20220511192921.2223629-10-heiko@sntech.de> <20220516061057.GL12339@lst.de> In-Reply-To: <20220516061057.GL12339@lst.de> From: Philipp Tomsich Date: Mon, 16 May 2022 11:09:12 +0200 Message-ID: Subject: Re: [PATCH 09/12] riscv: add RISC-V Svpbmt extension support To: Christoph Hellwig Cc: Heiko Stuebner , palmer@dabbelt.com, paul.walmsley@sifive.com, aou@eecs.berkeley.edu, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, wefu@redhat.com, liush@allwinnertech.com, guoren@kernel.org, atishp@atishpatra.org, anup@brainfault.org, drew@beagleboard.org, arnd@arndb.de, wens@csie.org, maxime@cerno.tech, gfavor@ventanamicro.com, andrea.mondelli@huawei.com, behrensj@mit.edu, xinhaoqu@huawei.com, mick@ics.forth.gr, allen.baum@esperantotech.com, jscheid@ventanamicro.com, rtrauben@gmail.com, samuel@sholland.org, cmuellner@linux.com, Wei Wu , Daniel Lustig , Bill Huffman Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 16 May 2022 at 08:11, Christoph Hellwig wrote: > > > +config RISCV_ISA_SVPBMT > > + bool "SVPBMT extension support" > > I don't think this prompt is very useful as it doesn't describe > what it does. But do we even want people to disable it as it is > really essentially for a fully functioning kernel and a pity that > it took RISC-V so long to get there? Given that RISC-V is (in some ways) an ISA construction set, there will be valid use cases for embedded users to disable this (e.g. if they have their own non-standard way to configure these). So while kernels for binary distributions (and desktop, server, or general-purpose embedded) will always enable these, I would fully expect some users to want to turn these off. @Heiko: I would request that we have a longer help text on this, which explains what it is and ends with the usual "When in doubt, say Y." > > + depends on 64BIT && MMU > > + select RISCV_ALTERNATIVE > > + default y > > + help > > + Adds support to dynamically detect the presence of the SVPBMT extension > > overly long line here. > > > index 5f1046e82d9f..dbfcd9b72bd8 100644 > > --- a/arch/riscv/include/asm/errata_list.h > > +++ b/arch/riscv/include/asm/errata_list.h > > @@ -14,6 +14,9 @@ > > #define ERRATA_SIFIVE_NUMBER 2 > > #endif > > > > +#define CPUFEATURE_SVPBMT 0 > > +#define CPUFEATURE_NUMBER 1 > > is errata_list.h really the right place for architectural features? > > Otherwise looks good: > > Reviewed-by: Christoph Hellwig