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 54E25C433EF for ; Wed, 1 Dec 2021 10:21:29 +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:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=+MVFoyVeDv7jsHwKfzI1GcD+e5/KV88WWja05qY5l/g=; b=oFC0mYKg/P2XCO Ikz87K3OoHOk8rTyGWZ/y5jxzW7/5yqZ5+icDRf1QKU4EMBW/hDenpXPlfQAnJQSLY+Xo3kGrTNX2 3bfyk3Bm1HHVXi71rouft354t6Tee9K9JDXroJz43/fDyiUvQlWhyepo7buTEvZbZFS5/XXUoX+Jk Mumyvue4uZwXg8XjjkZStDfqO/mNF81YmVDOgHKNQRHsY16dh+Fsp0kyAy/wO+UezLVH1t8TztL81 AXLN+QVrlHR3QgdMIAcgCiONVg3TmOT/6rYdWKBKnB+69ca8iiJhiUJxcDC7I0SQcQ/JczVq6bGzW 94pHQKMaa3Ws3S0MSqHg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1msMji-0085ET-Th; Wed, 01 Dec 2021 10:21:14 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1msMje-0085Cq-Ro for linux-riscv@lists.infradead.org; Wed, 01 Dec 2021 10:21:13 +0000 Received: from ip4d17a2ab.dynamic.kabel-deutschland.de ([77.23.162.171] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1msMjT-0001jO-Fq; Wed, 01 Dec 2021 11:20:59 +0100 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: atish patra Cc: Tsukasa OI , Atish Patra , Jessica Clarke , Philipp Tomsich , linux-riscv , Wei Fu , "linux-kernel@vger.kernel.org List" , taiten.peng@canonical.com, Aniket Ponkshe , Gordan Markus , Guo Ren , Arnd Bergmann , Chen-Yu Tsai , Maxime Ripard , Dan Lustig , Greg Favor , Andrea Mondelli , Jonathan Behrens , Xinhaoqu , Bill Huffman , Nick Kossifidis , Allen Baum , Josh Scheid , Richard Trauben , Anup Patel , Rob Herring , Anup Patel , Palmer Dabbelt , Guo Ren , Christoph =?ISO-8859-1?Q?M=FCllner?= , Christoph Hellwig , liush , Wei Wu , Drew Fustini , Heinrich Schuchardt Subject: Re: [PATCH V4 1/2] dt-bindings: riscv: add MMU Standard Extensions support for Svpbmt Date: Wed, 01 Dec 2021 11:20:53 +0100 Message-ID: <2562321.AMx6Y6Rosd@diego> In-Reply-To: References: <20211129014007.286478-1-wefu@redhat.com> <7587334.dMCaUTs8Vl@diego> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20211201_022111_125909_EDD041A5 X-CRM114-Status: GOOD ( 70.07 ) 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="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org QW0gTWl0dHdvY2gsIDEuIERlemVtYmVyIDIwMjEsIDA5OjQxOjQ4IENFVCBzY2hyaWViIGF0aXNo IHBhdHJhOgo+IE9uIFdlZCwgRGVjIDEsIDIwMjEgYXQgMTI6MzAgQU0gSGVpa28gU3TDvGJuZXIg PGhlaWtvQHNudGVjaC5kZT4gd3JvdGU6Cj4gCj4gPiBBbSBNaXR0d29jaCwgMS4gRGV6ZW1iZXIg MjAyMSwgMDk6MTU6MTggQ0VUIHNjaHJpZWIgQXRpc2ggUGF0cmE6Cj4gPiA+IE9uIFR1ZSwgTm92 IDMwLCAyMDIxIGF0IDc6MDYgUE0gVHN1a2FzYSBPSSA8cmVzZWFyY2hfdHJhc2lvQGlycS5hNGxn LmNvbT4KPiA+IHdyb3RlOgo+ID4gPiA+Cj4gPiA+ID4gT24gMjAyMS8xMi8wMSAxMDoyMSwgQXRp c2ggUGF0cmEgd3JvdGU6Cj4gPiA+ID4gPiBPbiBUdWUsIE5vdiAzMCwgMjAyMSBhdCA4OjEzIEFN IEplc3NpY2EgQ2xhcmtlIDxqcnRjMjdAanJ0YzI3LmNvbT4KPiA+IHdyb3RlOgo+ID4gPiA+ID4+ Cj4gPiA+ID4gPj4gT24gMzAgTm92IDIwMjEsIGF0IDE1OjAxLCBQaGlsaXBwIFRvbXNpY2ggPHBo aWxpcHAudG9tc2ljaEB2cnVsbC5ldT4KPiA+IHdyb3RlOgo+ID4gPiA+ID4+Pgo+ID4gPiA+ID4+ PiBXZSBkaWQgdG91Y2ggb24gdGhpcyBpbiBvdXIgY29vcmRpbmF0aW9uIGNhbGwgYSBmZXcgd2Vl a3MgYWdvOiB0aGUKPiA+ID4gPiA+Pj4gZ3JvdXBpbmcgdW5kZXIgbW11IGFuZCB0aGUgYm9vbC1l bnRyaWVzIHdlcmUgY2hvc2VuIGJlY2F1c2Ugb2YKPiA+IHRoZWlyCj4gPiA+ID4gPj4+IHNpbWls YXJpdHkgdG8gb3RoZXIgZXh0ZW5zaW9ucyAoaS5lLiBmb3IgWmJbYWJjc10gdGhlcmUKPiA+IGNv dWxkL3Nob3VsZAo+ID4gPiA+ID4+PiBiZSBhIGJvb2wtZW50cnkgdW5kZXIgZWFjaCBjcHUtbm9k ZSDigJQgZm9yIHNvbWUgWnYqIGVudHJpZXMgYQo+ID4gc3Vibm9kZQo+ID4gPiA+ID4+PiBtaWdo dCBiZSBuZWVkZWQgd2l0aCBmdXJ0aGVyIHBhcmFtZXRlcnMpLgo+ID4gPiA+ID4+Pgo+ID4gPiA+ ID4+PiBUaGUgc3RyaW5nLWJhc2VkIGFwcHJvYWNoIChhcyBpbiB0aGUgb3JpZ2luYWxseSBwcm9w b3NlZAo+ID4gIm1tdS10eXBlPSIpCj4gPiA+ID4gPj4+IHdvdWxkIGxpa2Ugbm90IHNjYWxlIHdp dGggdGhlIHByb2xpZmVyYXRpb24gb2Ygc21hbGwgJiBtb2R1bGFyCj4gPiA+ID4gPj4+IGV4dGVu c2lvbnMuCj4gPiA+ID4gPj4KPiA+ID4gPiA+PiBJIGRvbuKAmXQgc2VlIHdoeSB0aGUgU3YqIGV4 dGVuc2lvbnMgbmVlZCB0byBiZSB1bmRlciBhbiBtbXUgbm9kZQo+ID4gdGhlbiwKPiA+ID4gPiA+ PiB1bmxlc3MgdGhlIGludGVudCBpcyB0aGF0IGV2ZXJ5IGV4dGVuc2lvbiBiZSBncm91cGVkIHVu ZGVyIGEKPiA+IHN1Yi1ub2RlCj4gPiA+ID4gPj4gKHdoaWNoIGRvZXNu4oCZdCBzZWVtIHZpYWJs ZSBkdWUgdG8gZXh0ZW5zaW9ucyBsaWtlIFpiayosIHVubGVzcyB5b3UKPiA+ID4gPiA+PiBncm91 cCBieSBTcywgU3YgYW5kIFopPwo+ID4gPiA+ID4+Cj4gPiA+ID4gPgo+ID4gPiA+ID4gSXQgc2hv dWxkbid0IGJlLiBBbGwgdGhlIElTQSBleHRlbnNpb25zIChpLmUuIHN0YW5kYXJkLCBzdXBlcnZp c29yICYKPiA+IGh5cGVydmlzb3IpCj4gPiA+ID4gPiB3aXRoIHByZWZpeCBTLFosSCBzaG91bGQg YmUga2VwdCBzZXBhcmF0ZSBpbiBhIHNlcGFyYXRlIG5vZGUgZm9yIGVhc3kKPiA+ID4gPiA+IHBh cnNpbmcuCj4gPiA+ID4KPiA+ID4gPiAiRWFzeSBwYXJzaW5nIiBpcyBub3QgcXVpdGUgY29udmlu Y2luZy4KPiA+ID4KPiA+ID4gVGhlIGRldmljZSB0cmVlIG5lZWQgdG8gY2FycnkgYSB2ZXJ5IGxv bmcgInJpc2N2LGlzYSIgc3RyaW5nLiBUaGUKPiA+ID4gcGFyc2VyIG5lZWQgdG8gcGFyc2UKPiA+ ID4gdGhhdCBzdHJpbmcgaW4gbWVtb3J5IGFzIHdlbGwuCj4gPiA+Cj4gPiA+ID4KPiA+ID4gPiBU aGVyZSdzIGEgcmVhc29uIG90aGVyIHRoYW4gdGhhdCBJIG1hZGUgUkZDIFBBVENIIHRvIHBhcnNl Cj4gPiA+ID4gbXVsdGktbGV0dGVyIGV4dGVuc2lvbnM6Cj4gPiA+ID4KPiA+ID4gPiB2MTogPAo+ ID4gaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvcGlwZXJtYWlsL2xpbnV4LXJpc2N2LzIwMjEt Tm92ZW1iZXIvMDEwMjUyLmh0bWwKPiA+ID4KPiA+ID4gPiB2MjogPAo+ID4gaHR0cDovL2xpc3Rz LmluZnJhZGVhZC5vcmcvcGlwZXJtYWlsL2xpbnV4LXJpc2N2LzIwMjEtTm92ZW1iZXIvMDEwMzUw Lmh0bWwKPiA+ID4KPiA+ID4gPgo+ID4gPgo+ID4gPiBJdCdzIG9uIG15IHRvZG8gbGlzdCB0byBy ZXZpZXcgdGhlIHNlcmllcy4gSSB0aGluayB3ZSBjYW4gd29yawo+ID4gPiB0b2dldGhlciB0byBw cm9wb3NlIGEgYmV0dGVyIGZyYW1ld29yayBmb3IgcmlzY3YgaXNhIGV4dGVuc2lvbnMuCj4gPiA+ Cj4gPiA+ID4gKG5vdGU6IHRob3NlIHBhdGNoZXMgd2lsbCBicmVhayBSSVNDLVYgS1ZNIGJlY2F1 c2Ugb2YgcG9zc2libGUgSVNBCj4gPiA+ID4gICAgICAgIE1hbnVhbCBpbmNvbnNpc3RlbmN5IGFu ZCBkaXNjdXNzaW9uL3Jlc29sdXRpb24gbmVlZGVkKQo+ID4gPiA+Cj4gPiA+ID4gKC4uLmNvbnRp bnVlZCBiZWxvdy4uLikKPiA+ID4gPgo+ID4gPiA+ID4KPiA+ID4gPiA+ICJyaXNjdixpc2EiIGR0 IHByb3BlcnR5IHdpbGwgbm90IHNjYWxlIGF0IGFsbC4gSnVzdCBsb29rIGF0IHRoZSBmZXcKPiA+ ID4gPiA+IGV4dGVuc2lvbnMgdGhhdCB3ZXJlIHJhdGlmaWVkIHRoaXMgeWVhcgo+ID4gPiA+ID4g YW5kIExpbnV4IGtlcm5lbCBuZWVkcyB0byBzdXBwb3J0IHRoZW0uCj4gPiA+ID4gPgo+ID4gPiA+ ID4gIlNzY29mcG1mIiwgIlN2cGJtdCIsICJaaWNib20iCj4gPiA+ID4gPgo+ID4gPiA+ID4+IEFs c28sIHdoYXQgaXMgZ29pbmcgdG8gaGFwcGVuIHRvIHRoZSBjdXJyZW50IHJpc2N2LGlzYT8gV2ls bCB0aGF0Cj4gPiA+ID4gPj4gY29udGludWUgdG8gZXhpc3QgYW5kIGR1cGxpY2F0ZSB0aGUgaW5m bywgb3Igd2lsbCBrZXJuZWxzIGJlCj4gPiByZXF1aXJlZAo+ID4gPiA+ID4+IHRvIHJlY29uc3Ry dWN0IHRoZSBzdHJpbmcgdGhlbXNlbHZlcyBpZiB0aGV5IHdhbnQgdG8gZGlzcGxheSBpdCB0bwo+ ID4gPiA+ID4+IHVzZXJzPwo+ID4gPiA+ID4+Cj4gPiA+Cj4gPiA+IFNvcnJ5LiBJIG1pc3NlZCB0 aGlzIHF1ZXN0aW9uIGVhcmxpZXIuIFNlZSBteSBhbnN3ZXIgYmVsb3cuCj4gPiA+Cj4gPiA+ID4g Pgo+ID4gPiA+ID4gVGhpcyBpcyBteSBwZXJzb25hbCBwcmVmZXJlbmNlOgo+ID4gPiA+ID4gcmlz Y3YsaXNhIHdpbGwgY29udGludWUgdG8gYmFzZSBTdGFuZGFyZCBJU0EgZXh0ZW5zaW9ucyB0aGF0 IGhhdmUKPiA+ID4gPiA+IHNpbmdsZSBsZXR0ZXIgZXh0ZW5zaW9ucy4KPiA+ID4gPiA+Cj4gPiA+ ID4gPiBUaGlzIG5ldyBEVCBub2RlIHdpbGwgZW5jb2RlIGFsbCB0aGUgbm9uLXNpbmdsZSBsZXR0 ZXIgZXh0ZW5zaW9ucy4KPiA+ID4gPiA+IEkgYW0gbm90IHN1cmUgaWYgaXQgc2hvdWxkIGluY2x1 ZGUgc29tZSBwcm92aXNpb25zIGZvciBjdXN0b20KPiA+ID4gPiA+IGV4dGVuc2lvbnMgc3RhcnRp bmcgd2l0aCBYIGJlY2F1c2UKPiA+ID4gPiA+IHRoYXQgd2lsbCBiZSBwbGF0Zm9ybSBzcGVjaWZp Yy4KPiA+ID4gPiA+Cj4gPiA+ID4gPiBBZ2FpbiwgdGhpcyBpcyBqdXN0IG15IHBlcnNvbmFsIHBy ZWZlcmVuY2UuIEkgd2lsbCB0cnkgdG8gc2VuZCBhCj4gPiBwYXRjaAo+ID4gPiA+ID4gc29vbiBz byB0aGF0IHdlIGNhbiBpbml0aWF0ZSBhIGJyb2FkZXIKPiA+ID4gPiA+IGRpc2N1c3Npb24gb2Yg dGhlIHNjaGVtZSBhbmQgYWdyZWUvZGlzYWdyZWUgb24gc29tZXRoaW5nLgo+ID4gPiA+Cj4gPiA+ ID4gRm9yIHN1cGVydmlzb3Itb25seSBleHRlbnNpb25zIGxpa2UgIlN2cGJtdCIsIG5ldyBEVCBu b2RlIHdvdWxkIGJlIGEKPiA+ID4gPiByZWFzb25hYmxlIHNvbHV0aW9uIChhbmQgSSB3b3VsZCBu b3QgZGlyZWN0bHkgb2JqZWN0IGFib3V0IHRoYXQgbm9kZSkuCj4gPiA+ID4KPiA+ID4gPiBIb3dl dmVyLCB0aGVyZSdzIG1hbnkgbXVsdGktbGV0dGVyIGV4dGVuc2lvbnMgdGhhdCBhcmUgdXNlZnVs IGZvcgo+ID4gPiA+IHVzZXIgbW9kZS4gIEJlY2F1c2UgInJpc2N2LGlzYSIgaXMgZXhwb3NlZCB2 aWEgc3lzZnMgYW5kIHByb2Nmcwo+ID4gPiA+ICgvcHJvYy9jcHVpbmZvKSwgaXQgY2FuIGJlIHJl YWxseSBoZWxwZnVsIHRvIGhhdmUgbXVsdGktbGV0dGVyCj4gPiA+Cj4gPiA+IElycmVzcGVjdGl2 ZSBvZiB0aGUgbWV0aG9kIGNob3NlbiB0byBwYXJzZSB0aGUgZGV2aWNlIHRyZWUgaW4ga2VybmVs LAo+ID4gPiB3ZSBuZWVkIHRvIHByb3ZpZGUgdGhlIGV4dGVuc2lvbiBpbmZvcm1hdGlvbiB0byB0 aGUgdXNlcnNwYWNlLgo+ID4gPgo+ID4gPiBUaGlzIGlzIHdoYXQgSSBoYXZlIGluIG1pbmQuIEFu IGluZGl2aWR1YWwgcm93IHdpdGggY29tbWEgc2VwYXJhdGVkCj4gPiA+IGV4dGVuc2lvbiBuYW1l cyBmb3IgZWFjaCB0eXBlIG9mIGV4dGVuc2lvbnMgKFNzLCBTdiwgU2gpCj4gPiA+IGFmdGVyIHRo ZSBiYXNlIGV4dGVuc2lvbiAocnY2NGltYWZkYykgaW4gL3Byb2MvY3B1aW5mbyBvdXRwdXQuIEkg YW0KPiA+ID4gb3BlbiB0byBvdGhlciBpZGVhcyBhcyB3ZWxsLgo+ID4gPgo+ID4gPiBpc2EgICBy djY0aW1hZmRjCj4gPiA+IGlzYS1leHQtU3YgU3ZwYm10Cj4gPiA+IGlzYS1leHQtU3MgU3Njb2Zw bWYKPiA+ID4gaXNhLWV4dC1TaCA8aHlwZXJ2aXNvciByZWxhdGVkIGV4dGVuc2lvbnM+Cj4gPiA+ IGlzYS1leHQtWiAgIFppY2JvbQo+ID4gPgo+ID4gPiBXZSBjYW4gZXZlbiBleHBsaWNpdGx5IG5h bWUgdGhlIGV4dGVuc2lvbnMgYWZ0ZXIgaXNhLWV4dC4gSG93ZXZlciwgaXQKPiA+ID4gbWF5IGJl IG5lY2Vzc2FyeSBhbmQgdG9vIGxvbmcuCj4gPgo+ID4gQXJlbid0IG90aGVyIGFyY2hpdGVjdHVy ZXMganVzdCB1c2luZyBhIGZsYWdzIFt4ODZdIG9yIGZlYXR1cmVzIFthcm02NF0KPiA+IGxpbmUg aW4gY3B1aW5mbyB0byBleHBvc2UgdGhlIGF2YWlsYWJsZSBhZGRpdGlvbmFsIGNwdSBmZWF0dXJl cwo+ID4gYXMgYSBzcGFjZS1zZXBhcmF0ZWQgbGlzdD8KPiA+Cj4gPiBTbyB5b3UgY291bGQgYWxz byBqdXN0IGRvIHNvbWV0aGluZyBzaW1pbGFyIGxpa2UKPiA+IGlzYTogICBydjY0aW1hZmRjCj4g PiBpc2EtZXh0OiBTdnBibXQgU3Njb2ZwbWYgZm9vIGJhcgo+ID4KPiA+Cj4gQSBzcGFjZSBzZXBh cmF0ZWQgbGlzdCBpcyBhbHNvIGZpbmUgYnkgbWUuCj4gU2hvdWxkIHdlIGtlZXAgYWxsIHRoZSBl eHRlbnNpb25zIGFzIG9uZSByb3cgb3Igc3BsaXQgYmFzZWQgb24gdGhlIHR5cGUgb2YKPiBleHRl bnNpb25zIChTcywgU3YsIFNoLCk/Cj4gCj4gV2hlbiBJIGxvb2sgYXQgdGhlIGZsYWdzIGluIHg4 NiwgbXkgZXllcyBodXJ0IGJhZGx5IDspCgpPbiBhcm02NApGZWF0dXJlcyAgICAgICAgOiBmcCBh c2ltZCBldnRzdHJtIGFlcyBwbXVsbCBzaGExIHNoYTIgY3JjMzIgY3B1aWQKCm9yIG9uIGFybTMy CkZlYXR1cmVzICAgICAgICA6IGhhbGYgdGh1bWIgZmFzdG11bHQgdmZwIGVkc3AgdGh1bWJlZSBu ZW9uIHZmcHYzIHRscyB2ZnB2NCBpZGl2YSBpZGl2dCB2ZnBkMzIgbHBhZSBldnRzdHJtCgoKPiBU aGF0J3Mgd2h5IEkgc3VnZ2VzdGVkIHNwbGl0dGluZyBieSB0eXBlIG9mIGV4dGVuc2lvbnMgdG8g aW1wcm92ZQo+IHJlYWRhYmlsaXR5LgoKVGhvdWdoIEkgZ3Vlc3Mgd2l0aCB0aGF0IHNwbGl0IHlv dSBpbnRyb2R1Y2UgbW9yZSByZXF1aXJlbWVudHMgb24gdXNlcnNwYWNlPwpCZWNhdXNlIHRoaW5n cyB0aGF0IHBhcnNlIGNwdWluZm8gKHRoaW5rIHNvbWUgcHl0aG9uIGxpYnJhcnkpIHdpbGwgbmVl ZCB0bwpiZSB1cGRhdGVkIHdoZW4gc29tZSBuZXcgZXh0ZW5zaW9uIGNhdGVnb3J5IHN1cmZhY2Vz PwoKCgo+ID4gVGhhdCB3b3VsZCBtYWtlIGEgbmljZSBjb21wcm9taXNlIGJldHdlZW4gbGVuZ3Ro IGFuZCByZWFkYWJpbGl0eQo+ID4gYnkgdXNlcnMgSSBndWVzcz8KPiA+Cj4gPgo+ID4gSGVpa28K PiA+Cj4gPiA+IEkgZ3Vlc3MgeW91IHByZWZlciB0byBkaXJlY3RseSBwcmludCB0aGUgZW50aXJl ICJyaXNjdixpc2EiIHN0cmluZyBpbgo+ID4gPiAiaXNhIiByb3cgaW4gL3Byb2MvY3B1aW5mbyBv dXRwdXQuCj4gPiA+IEl0IGlzIHByb2JhYmx5IG9rYXkgd2l0aCB0aGUgY3VycmVudCBudW1iZXIg b2YgZXh0ZW5zaW9ucyBhdmFpbGFibGUKPiA+ID4gdG9kYXkuIEhvd2V2ZXIsIGl0IHdpbGwgYmVj b21lIHNvIGxvbmcgc3RyaW5nCj4gPiA+IGluIHRoZSBmdXR1cmUgdGhhdCBpdCBoYXMgdG8gYmUg YnJva2VuIGludG8gbXVsdGlwbGUgbGluZXMuCj4gPiA+Cj4gPiA+ID4gZXh0ZW5zaW9ucy4gIEFs c28sIGN1cnJlbnQgdmVyc2lvbiBvZiBTcGlrZSwgYSBSSVNDLVYgSVNBIFNpbXVsYXRvcgo+ID4g PiA+IHB1dHMgYWxsIG11bHRpLWxldHRlciBleHRlbnNpb25zIGluICJyaXNjdixpc2EiIGFuZCBJ IHRob3VnaHQgdGhpcyBpcwo+ID4gPiA+IGludGVuZGVkLgo+ID4gPiA+Cj4gPiA+ID4gTXkgcHJl ZmVyZW5jZToKPiA+ID4gPiAoMSkgQWxsb3cgaGF2aW5nIG11bHRpLWxldHRlciBleHRlbnNpb25z IGFuZCB2ZXJzaW9ucyBpbiAicmlzY3YsaXNhIgo+ID4gPiA+ICgyKSBBZGRpbmcgbmV3IERUIG5v ZGUgZm9yIHN1cGVydmlzb3ItcmVsYXRlZCBleHRlbnNpb25zIHdvdWxkIGJlCj4gPiA+ID4gICAg IHJlYXNvbmFibGUgKGJ1dCBJIGRvbid0IHN0cm9uZ2x5IGFncmVlL2Rpc2FncmVlKS4KPiA+ID4g Pgo+ID4gPiA+IFRoYW5rcywKPiA+ID4gPiBUc3VrYXNhCj4gPiA+ID4KPiA+ID4gPiA+Cj4gPiA+ ID4gPgo+ID4gPiA+ID4KPiA+ID4gPiA+PiBBcyBhIEZyZWVCU0QgZGV2ZWxvcGVyIEnigJltIG9i dmlvdXNseSBub3QgYSBwYXJ0IG9mIG1hbnkgb2YgdGhlc2UKPiA+ID4gPiA+PiBkaXNjdXNzaW9u cywgYnV0IHdoYXQgdGhlIExpbnV4IGNvbW11bml0eSBpbXBvc2VzIGFzIHRoZSBkZXZpY2UgdHJl ZQo+ID4gPiA+ID4+IGJpbmRpbmdzIGhhcyBhIHJlYWwgaW1wYWN0IG9uIHVzLgo+ID4gPiA+ID4+ Cj4gPiA+ID4gPj4gSmVzcwo+ID4gPiA+ID4+Cj4gPiA+ID4gPj4+IE9uIFR1ZSwgMzAgTm92IDIw MjEgYXQgMTQ6NTksIEplc3NpY2EgQ2xhcmtlIDxqcnRjMjdAanJ0YzI3LmNvbT4KPiA+IHdyb3Rl Ogo+ID4gPiA+ID4+Pj4KPiA+ID4gPiA+Pj4+IE9uIDMwIE5vdiAyMDIxLCBhdCAxMzoyNywgSGVp a28gU3TDvGJuZXIgPGhlaWtvQHNudGVjaC5kZT4gd3JvdGU6Cj4gPiA+ID4gPj4+Pj4KPiA+ID4g PiA+Pj4+PiBIaSwKPiA+ID4gPiA+Pj4+Pgo+ID4gPiA+ID4+Pj4+IEFtIERpZW5zdGFnLCAzMC4g Tm92ZW1iZXIgMjAyMSwgMTQ6MTc6NDEgQ0VUIHNjaHJpZWIgSmVzc2ljYQo+ID4gQ2xhcmtlOgo+ ID4gPiA+ID4+Pj4+PiBPbiAzMCBOb3YgMjAyMSwgYXQgMTI6MDcsIEhlaWtvIFN0w7xibmVyIDxo ZWlrb0BzbnRlY2guZGU+Cj4gPiB3cm90ZToKPiA+ID4gPiA+Pj4+Pj4+Cj4gPiA+ID4gPj4+Pj4+ PiBBbSBNb250YWcsIDI5LiBOb3ZlbWJlciAyMDIxLCAxMzowNjoyMyBDRVQgc2NocmllYiBIZWlr bwo+ID4gU3TDvGJuZXI6Cj4gPiA+ID4gPj4+Pj4+Pj4gQW0gTW9udGFnLCAyOS4gTm92ZW1iZXIg MjAyMSwgMDk6NTQ6MzkgQ0VUIHNjaHJpZWIgSGVpbnJpY2gKPiA+IFNjaHVjaGFyZHQ6Cj4gPiA+ ID4gPj4+Pj4+Pj4+IE9uIDExLzI5LzIxIDAyOjQwLCB3ZWZ1QHJlZGhhdC5jb20gd3JvdGU6Cj4g PiA+ID4gPj4+Pj4+Pj4+PiBGcm9tOiBXZWkgRnUgPHdlZnVAcmVkaGF0LmNvbT4KPiA+ID4gPiA+ Pj4+Pj4+Pj4+Cj4gPiA+ID4gPj4+Pj4+Pj4+PiBQcmV2aW91cyBwYXRjaCBoYXMgYWRkZWQgc3Zw Ym10IGluIGFyY2gvcmlzY3YgYW5kIGFkZAo+ID4gInJpc2N2LHN2cG1idCIKPiA+ID4gPiA+Pj4+ Pj4+Pj4+IGluIHRoZSBEVCBtbXUgbm9kZS4gVXBkYXRlIGR0LWJpbmRpbmdzIHJlbGF0ZWQgcHJv cGVydHkKPiA+IGhlcmUuCj4gPiA+ID4gPj4+Pj4+Pj4+Pgo+ID4gPiA+ID4+Pj4+Pj4+Pj4gU2ln bmVkLW9mZi1ieTogV2VpIEZ1IDx3ZWZ1QHJlZGhhdC5jb20+Cj4gPiA+ID4gPj4+Pj4+Pj4+PiBD by1kZXZlbG9wZWQtYnk6IEd1byBSZW4gPGd1b3JlbkBrZXJuZWwub3JnPgo+ID4gPiA+ID4+Pj4+ Pj4+Pj4gU2lnbmVkLW9mZi1ieTogR3VvIFJlbiA8Z3VvcmVuQGtlcm5lbC5vcmc+Cj4gPiA+ID4g Pj4+Pj4+Pj4+PiBDYzogQW51cCBQYXRlbCA8YW51cEBicmFpbmZhdWx0Lm9yZz4KPiA+ID4gPiA+ Pj4+Pj4+Pj4+IENjOiBQYWxtZXIgRGFiYmVsdCA8cGFsbWVyQGRhYmJlbHQuY29tPgo+ID4gPiA+ ID4+Pj4+Pj4+Pj4gQ2M6IFJvYiBIZXJyaW5nIDxyb2JoK2R0QGtlcm5lbC5vcmc+Cj4gPiA+ID4g Pj4+Pj4+Pj4+PiAtLS0KPiA+ID4gPiA+Pj4+Pj4+Pj4+IERvY3VtZW50YXRpb24vZGV2aWNldHJl ZS9iaW5kaW5ncy9yaXNjdi9jcHVzLnlhbWwgfCAxMAo+ID4gKysrKysrKysrKwo+ID4gPiA+ID4+ Pj4+Pj4+Pj4gMSBmaWxlIGNoYW5nZWQsIDEwIGluc2VydGlvbnMoKykKPiA+ID4gPiA+Pj4+Pj4+ Pj4+Cj4gPiA+ID4gPj4+Pj4+Pj4+PiBkaWZmIC0tZ2l0Cj4gPiBhL0RvY3VtZW50YXRpb24vZGV2 aWNldHJlZS9iaW5kaW5ncy9yaXNjdi9jcHVzLnlhbWwKPiA+IGIvRG9jdW1lbnRhdGlvbi9kZXZp Y2V0cmVlL2JpbmRpbmdzL3Jpc2N2L2NwdXMueWFtbAo+ID4gPiA+ID4+Pj4+Pj4+Pj4gaW5kZXgg YWE1ZmI2NGQ1N2ViLi45ZmY5Y2JkZDhhODUgMTAwNjQ0Cj4gPiA+ID4gPj4+Pj4+Pj4+PiAtLS0g YS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvcmlzY3YvY3B1cy55YW1sCj4gPiA+ ID4gPj4+Pj4+Pj4+PiArKysgYi9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3Mvcmlz Y3YvY3B1cy55YW1sCj4gPiA+ID4gPj4+Pj4+Pj4+PiBAQCAtNjMsNiArNjMsMTYgQEAgcHJvcGVy dGllczoKPiA+ID4gPiA+Pj4+Pj4+Pj4+ICAgICAgLSByaXNjdixzdjQ4Cj4gPiA+ID4gPj4+Pj4+ Pj4+PiAgICAgIC0gcmlzY3Ysbm9uZQo+ID4gPiA+ID4+Pj4+Pj4+Pj4KPiA+ID4gPiA+Pj4+Pj4+ Pj4+ICsgIG1tdToKPiA+ID4gPiA+Pj4+Pj4+Pj4KPiA+ID4gPiA+Pj4+Pj4+Pj4gU2hvdWxkbid0 IHdlIGtlZXAgdGhlIGl0ZW1zIGJlIGluIGFscGhhYmV0aWMgb3JkZXIsIGkuZS4gbW11Cj4gPiBi ZWZvcmUKPiA+ID4gPiA+Pj4+Pj4+Pj4gbW11LXR5cGU/Cj4gPiA+ID4gPj4+Pj4+Pj4+Cj4gPiA+ ID4gPj4+Pj4+Pj4+PiArICAgIGRlc2NyaXB0aW9uOgo+ID4gPiA+ID4+Pj4+Pj4+Pj4gKyAgICAg IERlc2NyaWJlcyB0aGUgQ1BVJ3MgTU1VIFN0YW5kYXJkIEV4dGVuc2lvbnMgc3VwcG9ydC4KPiA+ ID4gPiA+Pj4+Pj4+Pj4+ICsgICAgICBUaGVzZSB2YWx1ZXMgb3JpZ2luYXRlIGZyb20gdGhlIFJJ U0MtViBQcml2aWxlZ2VkCj4gPiA+ID4gPj4+Pj4+Pj4+PiArICAgICAgU3BlY2lmaWNhdGlvbiBk b2N1bWVudCwgYXZhaWxhYmxlIGZyb20KPiA+ID4gPiA+Pj4+Pj4+Pj4+ICsgICAgICBodHRwczov L3Jpc2N2Lm9yZy9zcGVjaWZpY2F0aW9ucy8KPiA+ID4gPiA+Pj4+Pj4+Pj4+ICsgICAgJHJlZjog Jy9zY2hlbWFzL3R5cGVzLnlhbWwjL2RlZmluaXRpb25zL3N0cmluZycKPiA+ID4gPiA+Pj4+Pj4+ Pj4+ICsgICAgZW51bToKPiA+ID4gPiA+Pj4+Pj4+Pj4+ICsgICAgICAtIHJpc2N2LHN2cG1idAo+ ID4gPiA+ID4+Pj4+Pj4+Pgo+ID4gPiA+ID4+Pj4+Pj4+PiBUaGUgcHJpdmlsZWdlZCBzcGVjaWZp Y2F0aW9uIGhhcyBtdWx0aXBsZSBNTVUgcmVsYXRlZAo+ID4gZXh0ZW5zaW9uczoKPiA+ID4gPiA+ Pj4+Pj4+Pj4gU3ZuYXBvdCwgU3ZwYm10LCBTdmludmFsLiBTaGFsbCB0aGV5IGFsbCBiZSBtb2Rl bGVkIGluIHRoaXMKPiA+IGVudW0/Cj4gPiA+ID4gPj4+Pj4+Pj4KPiA+ID4gPiA+Pj4+Pj4+PiBJ IHJlbWVtYmVyIGluIHNvbWUgZWFybGllciB2ZXJzaW9uIHNvbWUgd2F5IGJhY2sgdGhlcmUgd2Fz IHRoZQo+ID4gPiA+ID4+Pj4+Pj4+IHN1Z2dlc3Rpb24gb2YgdXNpbmcgYSBzdWItbm9kZSBpbnN0 ZWFkIGFuZCB0aGVuIGFkZGluZyBib29sZWFuCj4gPiA+ID4gPj4+Pj4+Pj4gcHJvcGVydGllcyBm b3IgdGhlIHN1cHBvcnRlZCBleHRlbnNpb25zLgo+ID4gPiA+ID4+Pj4+Pj4+Cj4gPiA+ID4gPj4+ Pj4+Pj4gQWthIHNvbWV0aGluZyBsaWtlCj4gPiA+ID4gPj4+Pj4+Pj4gICBtbXUgewo+ID4gPiA+ ID4+Pj4+Pj4+ICAgICAgICAgICByaXNjdixzdnBibXQ7Cj4gPiA+ID4gPj4+Pj4+Pj4gICB9Owo+ ID4gPiA+ID4+Pj4+Pj4KPiA+ID4gPiA+Pj4+Pj4+IEZvciB0aGUgcmVjb3JkLCBJJ20gdGFsa2lu ZyBhYm91dCB0aGUgbWFpbCBmcm9tIHNlcHRlbWJlcgo+ID4gPiA+ID4+Pj4+Pj4KPiA+IGh0dHBz Oi8vbG9yZS5rZXJuZWwub3JnL2xpbnV4LXJpc2N2L0NBQWVMdFVDaGpqekcrUDh5ZzQ1R0xaTUp5 NVVSMks1UlJCb0xGVlpodE9hWjVwUHRFQUBtYWlsLmdtYWlsLmNvbS8KPiA+ID4gPiA+Pj4+Pj4+ Cj4gPiA+ID4gPj4+Pj4+PiBTbyBoYXZpbmcgYSBzdWItbm9kZSB3b3VsZCBtYWtlIGFkZGluZyBm dXR1cmUgZXh0ZW5zaW9ucwo+ID4gPiA+ID4+Pj4+Pj4gd2F5IG5pY2VyLgo+ID4gPiA+ID4+Pj4+ Pgo+ID4gPiA+ID4+Pj4+PiBTdnBibXQgaXMganVzdCBhbiBJU0EgZXh0ZW5zaW9uLCBhbmQgc2hv dWxkIGJlIHRyZWF0ZWQgbGlrZSBhbnkKPiA+IG90aGVyLgo+ID4gPiA+ID4+Pj4+PiBMZXTigJlz IG5vdCBpbnZlbnQgdHdvIGRpZmZlcmVudCB3YXlzIG9mIHJlcHJlc2VudGluZyB0aGF0IGluIHRo ZQo+ID4gZGV2aWNlCj4gPiA+ID4gPj4+Pj4+IHRyZWUuCj4gPiA+ID4gPj4+Pj4KPiA+ID4gPiA+ Pj4+PiBIZWlucmljaCBhc2tlZCBob3cgdGhlIG90aGVyIGV4dGVuc2lvbnMgc2hvdWxkIGJlIGhh bmRsZWQKPiA+ID4gPiA+Pj4+PiAoU3ZuYXBvdCwgU3ZwYm10LCBTdmludmFsKSwgc28gd2hhdCBk byB5b3Ugc3VnZ2VzdCB0byBkbyB3aXRoCj4gPiB0aGVzZT8KPiA+ID4gPiA+Pj4+Cj4gPiA+ID4g Pj4+PiBXaGF0ZXZlciBpcyBkb25lIGZvciBaYlthYmNzXSwgWmsqLCBadiosIFppY2JvKiwgZXRj LiBUaGVyZSBtYXkKPiA+IG5vdCBiZQo+ID4gPiA+ID4+Pj4gYSBjb25jcmV0ZSBwbGFuIGZvciB0 aGF0IHlldCwgYnV0IHRoYXQgbWVhbnMgeW91IHNob3VsZCBzcGVhawo+ID4gd2l0aCB0aGUKPiA+ ID4gPiA+Pj4+IHBlb3BsZSBpbnZvbHZlZCB3aXRoIHN1Y2ggZXh0ZW5zaW9ucyBhbmQgY29tZSB1 cCB3aXRoIHNvbWV0aGluZwo+ID4gPiA+ID4+Pj4gYXBwcm9wcmlhdGUgdG9nZXRoZXIuCj4gPiA+ ID4gPj4+Pgo+ID4gPiA+ID4+Pj4gSmVzcwo+ID4gPiA+ID4+Pj4KPiA+ID4gPiA+Pgo+ID4gPiA+ ID4+Cj4gPiA+ID4gPj4gX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX18KPiA+ID4gPiA+PiBsaW51eC1yaXNjdiBtYWlsaW5nIGxpc3QKPiA+ID4gPiA+PiBsaW51 eC1yaXNjdkBsaXN0cy5pbmZyYWRlYWQub3JnCj4gPiA+ID4gPj4gaHR0cDovL2xpc3RzLmluZnJh ZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5mby9saW51eC1yaXNjdgo+ID4gPiA+ID4KPiA+ID4gPiA+ Cj4gPiA+ID4gPgo+ID4gPiA+ID4gLS0KPiA+ID4gPiA+IFJlZ2FyZHMsCj4gPiA+ID4gPiBBdGlz aAo+ID4gPiA+ID4KPiA+ID4gPiA+IF9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fCj4gPiA+ID4gPiBsaW51eC1yaXNjdiBtYWlsaW5nIGxpc3QKPiA+ID4gPiA+ IGxpbnV4LXJpc2N2QGxpc3RzLmluZnJhZGVhZC5vcmcKPiA+ID4gPiA+IGh0dHA6Ly9saXN0cy5p bmZyYWRlYWQub3JnL21haWxtYW4vbGlzdGluZm8vbGludXgtcmlzY3YKPiA+ID4gPiA+Cj4gPiA+ Cj4gPiA+Cj4gPiA+Cj4gPiA+Cj4gPgo+ID4KPiA+Cj4gPgo+ID4KPiAKPiAKCgoKCgpfX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpsaW51eC1yaXNjdiBtYWls aW5nIGxpc3QKbGludXgtcmlzY3ZAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5m cmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LXJpc2N2Cg== 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1674AC433F5 for ; Wed, 1 Dec 2021 10:21:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1348620AbhLAKYg convert rfc822-to-8bit (ORCPT ); Wed, 1 Dec 2021 05:24:36 -0500 Received: from gloria.sntech.de ([185.11.138.130]:53924 "EHLO gloria.sntech.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1348612AbhLAKYf (ORCPT ); Wed, 1 Dec 2021 05:24:35 -0500 Received: from ip4d17a2ab.dynamic.kabel-deutschland.de ([77.23.162.171] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1msMjT-0001jO-Fq; Wed, 01 Dec 2021 11:20:59 +0100 From: Heiko =?ISO-8859-1?Q?St=FCbner?= To: atish patra Cc: Tsukasa OI , Atish Patra , Jessica Clarke , Philipp Tomsich , linux-riscv , Wei Fu , "linux-kernel@vger.kernel.org List" , taiten.peng@canonical.com, Aniket Ponkshe , Gordan Markus , Guo Ren , Arnd Bergmann , Chen-Yu Tsai , Maxime Ripard , Dan Lustig , Greg Favor , Andrea Mondelli , Jonathan Behrens , Xinhaoqu , Bill Huffman , Nick Kossifidis , Allen Baum , Josh Scheid , Richard Trauben , Anup Patel , Rob Herring , Anup Patel , Palmer Dabbelt , Guo Ren , Christoph =?ISO-8859-1?Q?M=FCllner?= , Christoph Hellwig , liush , Wei Wu , Drew Fustini , Heinrich Schuchardt Subject: Re: [PATCH V4 1/2] dt-bindings: riscv: add MMU Standard Extensions support for Svpbmt Date: Wed, 01 Dec 2021 11:20:53 +0100 Message-ID: <2562321.AMx6Y6Rosd@diego> In-Reply-To: References: <20211129014007.286478-1-wefu@redhat.com> <7587334.dMCaUTs8Vl@diego> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am Mittwoch, 1. Dezember 2021, 09:41:48 CET schrieb atish patra: > On Wed, Dec 1, 2021 at 12:30 AM Heiko Stübner wrote: > > > Am Mittwoch, 1. Dezember 2021, 09:15:18 CET schrieb Atish Patra: > > > On Tue, Nov 30, 2021 at 7:06 PM Tsukasa OI > > wrote: > > > > > > > > On 2021/12/01 10:21, Atish Patra wrote: > > > > > On Tue, Nov 30, 2021 at 8:13 AM Jessica Clarke > > wrote: > > > > >> > > > > >> On 30 Nov 2021, at 15:01, Philipp Tomsich > > wrote: > > > > >>> > > > > >>> We did touch on this in our coordination call a few weeks ago: the > > > > >>> grouping under mmu and the bool-entries were chosen because of > > their > > > > >>> similarity to other extensions (i.e. for Zb[abcs] there > > could/should > > > > >>> be a bool-entry under each cpu-node — for some Zv* entries a > > subnode > > > > >>> might be needed with further parameters). > > > > >>> > > > > >>> The string-based approach (as in the originally proposed > > "mmu-type=") > > > > >>> would like not scale with the proliferation of small & modular > > > > >>> extensions. > > > > >> > > > > >> I don’t see why the Sv* extensions need to be under an mmu node > > then, > > > > >> unless the intent is that every extension be grouped under a > > sub-node > > > > >> (which doesn’t seem viable due to extensions like Zbk*, unless you > > > > >> group by Ss, Sv and Z)? > > > > >> > > > > > > > > > > It shouldn't be. All the ISA extensions (i.e. standard, supervisor & > > hypervisor) > > > > > with prefix S,Z,H should be kept separate in a separate node for easy > > > > > parsing. > > > > > > > > "Easy parsing" is not quite convincing. > > > > > > The device tree need to carry a very long "riscv,isa" string. The > > > parser need to parse > > > that string in memory as well. > > > > > > > > > > > There's a reason other than that I made RFC PATCH to parse > > > > multi-letter extensions: > > > > > > > > v1: < > > http://lists.infradead.org/pipermail/linux-riscv/2021-November/010252.html > > > > > > > v2: < > > http://lists.infradead.org/pipermail/linux-riscv/2021-November/010350.html > > > > > > > > > > > > > It's on my todo list to review the series. I think we can work > > > together to propose a better framework for riscv isa extensions. > > > > > > > (note: those patches will break RISC-V KVM because of possible ISA > > > > Manual inconsistency and discussion/resolution needed) > > > > > > > > (...continued below...) > > > > > > > > > > > > > > "riscv,isa" dt property will not scale at all. Just look at the few > > > > > extensions that were ratified this year > > > > > and Linux kernel needs to support them. > > > > > > > > > > "Sscofpmf", "Svpbmt", "Zicbom" > > > > > > > > > >> Also, what is going to happen to the current riscv,isa? Will that > > > > >> continue to exist and duplicate the info, or will kernels be > > required > > > > >> to reconstruct the string themselves if they want to display it to > > > > >> users? > > > > >> > > > > > > Sorry. I missed this question earlier. See my answer below. > > > > > > > > > > > > > This is my personal preference: > > > > > riscv,isa will continue to base Standard ISA extensions that have > > > > > single letter extensions. > > > > > > > > > > This new DT node will encode all the non-single letter extensions. > > > > > I am not sure if it should include some provisions for custom > > > > > extensions starting with X because > > > > > that will be platform specific. > > > > > > > > > > Again, this is just my personal preference. I will try to send a > > patch > > > > > soon so that we can initiate a broader > > > > > discussion of the scheme and agree/disagree on something. > > > > > > > > For supervisor-only extensions like "Svpbmt", new DT node would be a > > > > reasonable solution (and I would not directly object about that node). > > > > > > > > However, there's many multi-letter extensions that are useful for > > > > user mode. Because "riscv,isa" is exposed via sysfs and procfs > > > > (/proc/cpuinfo), it can be really helpful to have multi-letter > > > > > > Irrespective of the method chosen to parse the device tree in kernel, > > > we need to provide the extension information to the userspace. > > > > > > This is what I have in mind. An individual row with comma separated > > > extension names for each type of extensions (Ss, Sv, Sh) > > > after the base extension (rv64imafdc) in /proc/cpuinfo output. I am > > > open to other ideas as well. > > > > > > isa rv64imafdc > > > isa-ext-Sv Svpbmt > > > isa-ext-Ss Sscofpmf > > > isa-ext-Sh > > > isa-ext-Z Zicbom > > > > > > We can even explicitly name the extensions after isa-ext. However, it > > > may be necessary and too long. > > > > Aren't other architectures just using a flags [x86] or features [arm64] > > line in cpuinfo to expose the available additional cpu features > > as a space-separated list? > > > > So you could also just do something similar like > > isa: rv64imafdc > > isa-ext: Svpbmt Sscofpmf foo bar > > > > > A space separated list is also fine by me. > Should we keep all the extensions as one row or split based on the type of > extensions (Ss, Sv, Sh,)? > > When I look at the flags in x86, my eyes hurt badly ;) On arm64 Features : fp asimd evtstrm aes pmull sha1 sha2 crc32 cpuid or on arm32 Features : half thumb fastmult vfp edsp thumbee neon vfpv3 tls vfpv4 idiva idivt vfpd32 lpae evtstrm > That's why I suggested splitting by type of extensions to improve > readability. Though I guess with that split you introduce more requirements on userspace? Because things that parse cpuinfo (think some python library) will need to be updated when some new extension category surfaces? > > That would make a nice compromise between length and readability > > by users I guess? > > > > > > Heiko > > > > > I guess you prefer to directly print the entire "riscv,isa" string in > > > "isa" row in /proc/cpuinfo output. > > > It is probably okay with the current number of extensions available > > > today. However, it will become so long string > > > in the future that it has to be broken into multiple lines. > > > > > > > extensions. Also, current version of Spike, a RISC-V ISA Simulator > > > > puts all multi-letter extensions in "riscv,isa" and I thought this is > > > > intended. > > > > > > > > My preference: > > > > (1) Allow having multi-letter extensions and versions in "riscv,isa" > > > > (2) Adding new DT node for supervisor-related extensions would be > > > > reasonable (but I don't strongly agree/disagree). > > > > > > > > Thanks, > > > > Tsukasa > > > > > > > > > > > > > > > > > > > > > > > >> As a FreeBSD developer I’m obviously not a part of many of these > > > > >> discussions, but what the Linux community imposes as the device tree > > > > >> bindings has a real impact on us. > > > > >> > > > > >> Jess > > > > >> > > > > >>> On Tue, 30 Nov 2021 at 14:59, Jessica Clarke > > wrote: > > > > >>>> > > > > >>>> On 30 Nov 2021, at 13:27, Heiko Stübner wrote: > > > > >>>>> > > > > >>>>> Hi, > > > > >>>>> > > > > >>>>> Am Dienstag, 30. November 2021, 14:17:41 CET schrieb Jessica > > Clarke: > > > > >>>>>> On 30 Nov 2021, at 12:07, Heiko Stübner > > wrote: > > > > >>>>>>> > > > > >>>>>>> Am Montag, 29. November 2021, 13:06:23 CET schrieb Heiko > > Stübner: > > > > >>>>>>>> Am Montag, 29. November 2021, 09:54:39 CET schrieb Heinrich > > Schuchardt: > > > > >>>>>>>>> On 11/29/21 02:40, wefu@redhat.com wrote: > > > > >>>>>>>>>> From: Wei Fu > > > > >>>>>>>>>> > > > > >>>>>>>>>> Previous patch has added svpbmt in arch/riscv and add > > "riscv,svpmbt" > > > > >>>>>>>>>> in the DT mmu node. Update dt-bindings related property > > here. > > > > >>>>>>>>>> > > > > >>>>>>>>>> Signed-off-by: Wei Fu > > > > >>>>>>>>>> Co-developed-by: Guo Ren > > > > >>>>>>>>>> Signed-off-by: Guo Ren > > > > >>>>>>>>>> Cc: Anup Patel > > > > >>>>>>>>>> Cc: Palmer Dabbelt > > > > >>>>>>>>>> Cc: Rob Herring > > > > >>>>>>>>>> --- > > > > >>>>>>>>>> Documentation/devicetree/bindings/riscv/cpus.yaml | 10 > > ++++++++++ > > > > >>>>>>>>>> 1 file changed, 10 insertions(+) > > > > >>>>>>>>>> > > > > >>>>>>>>>> diff --git > > a/Documentation/devicetree/bindings/riscv/cpus.yaml > > b/Documentation/devicetree/bindings/riscv/cpus.yaml > > > > >>>>>>>>>> index aa5fb64d57eb..9ff9cbdd8a85 100644 > > > > >>>>>>>>>> --- a/Documentation/devicetree/bindings/riscv/cpus.yaml > > > > >>>>>>>>>> +++ b/Documentation/devicetree/bindings/riscv/cpus.yaml > > > > >>>>>>>>>> @@ -63,6 +63,16 @@ properties: > > > > >>>>>>>>>> - riscv,sv48 > > > > >>>>>>>>>> - riscv,none > > > > >>>>>>>>>> > > > > >>>>>>>>>> + mmu: > > > > >>>>>>>>> > > > > >>>>>>>>> Shouldn't we keep the items be in alphabetic order, i.e. mmu > > before > > > > >>>>>>>>> mmu-type? > > > > >>>>>>>>> > > > > >>>>>>>>>> + description: > > > > >>>>>>>>>> + Describes the CPU's MMU Standard Extensions support. > > > > >>>>>>>>>> + These values originate from the RISC-V Privileged > > > > >>>>>>>>>> + Specification document, available from > > > > >>>>>>>>>> + https://riscv.org/specifications/ > > > > >>>>>>>>>> + $ref: '/schemas/types.yaml#/definitions/string' > > > > >>>>>>>>>> + enum: > > > > >>>>>>>>>> + - riscv,svpmbt > > > > >>>>>>>>> > > > > >>>>>>>>> The privileged specification has multiple MMU related > > extensions: > > > > >>>>>>>>> Svnapot, Svpbmt, Svinval. Shall they all be modeled in this > > enum? > > > > >>>>>>>> > > > > >>>>>>>> I remember in some earlier version some way back there was the > > > > >>>>>>>> suggestion of using a sub-node instead and then adding boolean > > > > >>>>>>>> properties for the supported extensions. > > > > >>>>>>>> > > > > >>>>>>>> Aka something like > > > > >>>>>>>> mmu { > > > > >>>>>>>> riscv,svpbmt; > > > > >>>>>>>> }; > > > > >>>>>>> > > > > >>>>>>> For the record, I'm talking about the mail from september > > > > >>>>>>> > > https://lore.kernel.org/linux-riscv/CAAeLtUChjjzG+P8yg45GLZMJy5UR2K5RRBoLFVZhtOaZ5pPtEA@mail.gmail.com/ > > > > >>>>>>> > > > > >>>>>>> So having a sub-node would make adding future extensions > > > > >>>>>>> way nicer. > > > > >>>>>> > > > > >>>>>> Svpbmt is just an ISA extension, and should be treated like any > > other. > > > > >>>>>> Let’s not invent two different ways of representing that in the > > device > > > > >>>>>> tree. > > > > >>>>> > > > > >>>>> Heinrich asked how the other extensions should be handled > > > > >>>>> (Svnapot, Svpbmt, Svinval), so what do you suggest to do with > > these? > > > > >>>> > > > > >>>> Whatever is done for Zb[abcs], Zk*, Zv*, Zicbo*, etc. There may > > not be > > > > >>>> a concrete plan for that yet, but that means you should speak > > with the > > > > >>>> people involved with such extensions and come up with something > > > > >>>> appropriate together. > > > > >>>> > > > > >>>> Jess > > > > >>>> > > > > >> > > > > >> > > > > >> _______________________________________________ > > > > >> linux-riscv mailing list > > > > >> linux-riscv@lists.infradead.org > > > > >> http://lists.infradead.org/mailman/listinfo/linux-riscv > > > > > > > > > > > > > > > > > > > > -- > > > > > Regards, > > > > > Atish > > > > > > > > > > _______________________________________________ > > > > > linux-riscv mailing list > > > > > linux-riscv@lists.infradead.org > > > > > http://lists.infradead.org/mailman/listinfo/linux-riscv > > > > > > > > > > > > > > > > > > > > > > > > > > > > >