From:   Anup Patel <anup.patel@wdc.com>
To:     Paolo Bonzini <pbonzini@redhat.com>, Shuah Khan <shuah@kernel.org>,
        Atish Patra <atishp@atishpatra.org>
Cc:     Palmer Dabbelt <palmer@dabbelt.com>,
        Paul Walmsley <paul.walmsley@sifive.com>,
        Albert Ou <aou@eecs.berkeley.edu>,
        Alistair Francis <Alistair.Francis@wdc.com>,
        Anup Patel <anup@brainfault.org>, kvm@vger.kernel.org,
        kvm-riscv@lists.infradead.org, linux-riscv@lists.infradead.org,
        linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org,
        Anup Patel <anup.patel@wdc.com>,
        Atish Patra <atishp@rivosinc.com>
Subject: [PATCH v4 1/4] RISC-V: KVM: Forward SBI experimental and vendor extensions
Date:   Fri, 24 Dec 2021 13:06:01 +0530
Message-Id: <20211224073604.1085464-2-anup.patel@wdc.com>
In-Reply-To: <20211224073604.1085464-1-anup.patel@wdc.com>
References: <20211224073604.1085464-1-anup.patel@wdc.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
WDCIPOUTBOUND: EOP-TRUE
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?B5oHMc9NFJIvBsm+0MFGegJfzo1i3GdvV4IvCxAYImvEM9ELGi5sFKIxki09?=
 =?us-ascii?Q?VAUYmb5+iY8gnEOvfLnLaFSmGNkvrRLybiHgdWeNIDIg4gu5NqDtn6tzQ0Vm?=
 =?us-ascii?Q?r6kJU5IwTOUvbNd/Ivwcrspv2DQgPpSi5k01EHIvMt3eJN/Anv67/LsZWvPg?=
 =?us-ascii?Q?fAASa+F5axcaiT8MglY32upAeIJNjoF01KMErMjsI5baD+AgPUIOYzZ6oBRB?=
 =?us-ascii?Q?vXy2f8MxSjpA1PA5C8Y+ZflIXjjUK0KsjPyBmFNHwbE7ilyYSg50A0B3O7Xy?=
 =?us-ascii?Q?KS7jdMYHrmVf7o2O7cCQAXMymyoZUmlwM22X13HWfL1q29Tg0HNou0Bx1Rfc?=
 =?us-ascii?Q?6M7TwzftnLwDmaap5lq3aGRZWYoigxpVcBTh2arUclhiecTiZanhzEdoIpKh?=
 =?us-ascii?Q?0MxG2SmshkDGyG95LdSU7Smpll4FbYZVVVNSMzh0qdRjHzZ2XiGfgI8wg9Qu?=
 =?us-ascii?Q?/t+xbhxIbMpesDhq7dvG1+z8/1fsS8TAXnihgVz1pLTbP2rrDiqsU4pTkMXs?=
 =?us-ascii?Q?SVLZXPRUjWdHnD5ZJ+d4RhkL7NjUjmrJ9Bu5iCDj6/cJHDXviRh2pzjNZgI2?=
 =?us-ascii?Q?9W36dQOA8Qu2ItDRKCZH9vUWvtM05gzjnLW/BB/WQRfiOQ+WkewEXIFHYvYJ?=
 =?us-ascii?Q?r6OnhkqB5f+iEmqUrgCOJNAt8qIKGd6BHLNs8/h3tOclFcwFbGQPZiFw/4X5?=
 =?us-ascii?Q?0q4K45N0oy17N8qNGePaL2Bsa2tw35fdWIIbbyQmGMpnnU8PZwJZVm5knCPb?=
 =?us-ascii?Q?gu4hSVRjdPeOwssmKf3E4bYx3VALybRiRFvzYgMCbjZQWWrZOp07ftW7IjrQ?=
 =?us-ascii?Q?w9zxhEwtSKsUL0bzdKvXJDYQ5zvUyqMO1vAgt44TN7cmwxL3lMM+qrvr3wO+?=
 =?us-ascii?Q?HQNKZK1VJvkp+JSV1+N1Uks/9ImBeF63/YsfwLZGZx9/6vJLaJJ5Q5h+0EV4?=
 =?us-ascii?Q?FB5+Q4hoMZTDIr2ZVh9Y7O11D3leNBFSVO3kmTuNHknFO9PYfN3jjMDXbzqi?=
 =?us-ascii?Q?sAvESeG5r+TyGKYG9N3cWwQStL0kkzXKigcZex+IMjIV+niCXhYTDRyr4GCk?=
 =?us-ascii?Q?LpFp8SIjFocEGOZR6tAEVnMYppLFKu/MXdyQAJPeZ69LnLlQMK84wjCoYy+3?=
 =?us-ascii?Q?fh2G8RyYyjawyOIR8qiOS3liofUu4UvNI2lpyT7m+l3y3PEO4adeUJ1naClT?=
 =?us-ascii?Q?dolPnt3ysc4eEsumOeHfxCH/nAy+kxQnO0rduSajIoeLkQOZJStOwjHNtebN?=
 =?us-ascii?Q?Qljy9lc9+3xGoFNX+3JqLOWmESBscWmRwBWBkUDVU5MTASNjNooPrtQs+6oi?=
 =?us-ascii?Q?wU7V7MSW5OjlQMB+rVywKSqvqd/ih8K1S48JHs3w6hj3dk6POsJypyfq8O+h?=
 =?us-ascii?Q?JXFig9ckrb91LJZdar3YiTW7heAbrXWGysggl6gfn9PEbTImGlRTnvcbVoYW?=
 =?us-ascii?Q?dRVqGBjoFxpf2Fe4MeXJZKim7J456mFMh4jA7X+XH4H8PFV6Vgjg+S/Z1ybI?=
 =?us-ascii?Q?YoUfCS5qkMXSdrtQHhBG1F3wlkuu7ddfBaVdKcC6xwNyXygEAlsCmQBJvBAw?=
 =?us-ascii?Q?mg2/KXsWrvlKJRXOG86qc49h5BkBbowqY9zh+/BouTu1sSGRXddbNCZfJ1LK?=
 =?us-ascii?Q?uMB0rgTjs+VoJxVv8jRBQ78=3D?=
X-OriginatorOrg: wdc.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f93057ca-5c10-42c0-0820-08d9c6b01f69
X-MS-Exchange-CrossTenant-AuthSource: CO6PR04MB7812.namprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Dec 2021 07:36:39.6749
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: b61c8803-16f3-4c35-9b17-6f65f441df86
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 9VSpHcRa0hE5j4esGyX9xxRlmVVJXW9Zt4lnvsP8Vat4HwQSdVIextqBJsYW6w+6wqjLtxGmgU0Ti/FtQsnTYQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR04MB7859
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

The SBI experimental extension space is for temporary (or experimental)
stuff whereas SBI vendor extension space is for hardware vendor specific
stuff. Both these SBI extension spaces won't be standardized by the SBI
specification so let's blindly forward such SBI calls to the userspace.

Signed-off-by: Anup Patel <anup.patel@wdc.com>
Reviewed-and-tested-by: Atish Patra <atishp@rivosinc.com>
---
 arch/riscv/kvm/vcpu_sbi.c      |  4 ++++
 arch/riscv/kvm/vcpu_sbi_base.c | 27 +++++++++++++++++++++++++++
 2 files changed, 31 insertions(+)

diff --git a/arch/riscv/kvm/vcpu_sbi.c b/arch/riscv/kvm/vcpu_sbi.c
index f62d25bc9733..78aa3db76225 100644
--- a/arch/riscv/kvm/vcpu_sbi.c
+++ b/arch/riscv/kvm/vcpu_sbi.c
@@ -46,6 +46,8 @@ extern const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_time;
 extern const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_ipi;
 extern const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_rfence;
 extern const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_hsm;
+extern const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_experimental;
+extern const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_vendor;
 
 static const struct kvm_vcpu_sbi_extension *sbi_ext[] = {
 	&vcpu_sbi_ext_v01,
@@ -54,6 +56,8 @@ static const struct kvm_vcpu_sbi_extension *sbi_ext[] = {
 	&vcpu_sbi_ext_ipi,
 	&vcpu_sbi_ext_rfence,
 	&vcpu_sbi_ext_hsm,
+	&vcpu_sbi_ext_experimental,
+	&vcpu_sbi_ext_vendor,
 };
 
 void kvm_riscv_vcpu_sbi_forward(struct kvm_vcpu *vcpu, struct kvm_run *run)
diff --git a/arch/riscv/kvm/vcpu_sbi_base.c b/arch/riscv/kvm/vcpu_sbi_base.c
index d1ec08fe7289..4ecf377f483b 100644
--- a/arch/riscv/kvm/vcpu_sbi_base.c
+++ b/arch/riscv/kvm/vcpu_sbi_base.c
@@ -70,3 +70,30 @@ const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_base = {
 	.extid_end = SBI_EXT_BASE,
 	.handler = kvm_sbi_ext_base_handler,
 };
+
+static int kvm_sbi_ext_forward_handler(struct kvm_vcpu *vcpu,
+					struct kvm_run *run,
+					unsigned long *out_val,
+					struct kvm_cpu_trap *utrap,
+					bool *exit)
+{
+	/*
+	 * Both SBI experimental and vendor extensions are
+	 * unconditionally forwarded to userspace.
+	 */
+	kvm_riscv_vcpu_sbi_forward(vcpu, run);
+	*exit = true;
+	return 0;
+}
+
+const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_experimental = {
+	.extid_start = SBI_EXT_EXPERIMENTAL_START,
+	.extid_end = SBI_EXT_EXPERIMENTAL_END,
+	.handler = kvm_sbi_ext_forward_handler,
+};
+
+const struct kvm_vcpu_sbi_extension vcpu_sbi_ext_vendor = {
+	.extid_start = SBI_EXT_VENDOR_START,
+	.extid_end = SBI_EXT_VENDOR_END,
+	.handler = kvm_sbi_ext_forward_handler,
+};
-- 
2.25.1