From: "Radim Krčmář" <rkrcmar@ventanamicro.com>
To: "Atish Patra" <atish.patra@linux.dev>,
"Clément Léger" <cleger@rivosinc.com>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Anup Patel" <anup@brainfault.org>,
"Atish Patra" <atishp@atishpatra.org>,
"Shuah Khan" <shuah@kernel.org>,
"Jonathan Corbet" <corbet@lwn.net>,
linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-doc@vger.kernel.org, kvm@vger.kernel.org,
kvm-riscv@lists.infradead.org, linux-kselftest@vger.kernel.org
Cc: "Samuel Holland" <samuel.holland@sifive.com>,
"Andrew Jones" <ajones@ventanamicro.com>,
"Deepak Gupta" <debug@rivosinc.com>,
"Charlie Jenkins" <charlie@rivosinc.com>,
"linux-riscv" <linux-riscv-bounces@lists.infradead.org>
Subject: Re: [PATCH v8 13/14] RISC-V: KVM: add support for FWFT SBI extension
Date: Mon, 26 May 2025 10:58:46 +0200 [thread overview]
Message-ID: <DA5YVKW682V3.2DODRY4EDL3IW@ventanamicro.com> (raw)
In-Reply-To: <5dd587b3-8c04-41d1-b677-5b07266cfec5@linux.dev>
2025-05-23T11:02:11-07:00, Atish Patra <atish.patra@linux.dev>:
> On 5/23/25 9:27 AM, Radim KrÄmáŠwrote:
>> 2025-05-23T17:29:49+02:00, Clément Léger <cleger@rivosinc.com>:
>>> Is this something blocking though ? We'd like to merge FWFT once SBI 3.0
>>> is ratified so that would be nice not delaying it too much. I'll take a
>>> look at it to see if it isn't too long to implement.
>>
>> Not blocking, but I would at least default FWFT to disabled, because
>> current userspace cannot handle [14/14]. (Well... save/restore was
>> probably broken even before, but let's try to not make it worse. :])
>>
>
> User space can not enable or disable misaligned access delegation as
> there is no interface for now rightly pointed by you.
I mean setting default_disabled=true and just disabling FWFT for the
guest unless userspace explicitly enables the incomplete extension.
We would blame the user for wanting mutually exclusive features.
> I guess supporting
> that would be quicker than fixing the broader guest save/restore
> anyways. Isn't it ?
Yes. The save/restore for FWFT is simple (if we disregard the
discussions), but definitely more than a single line.
--
kvm-riscv mailing list
kvm-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kvm-riscv
WARNING: multiple messages have this Message-ID (diff)
From: "Radim Krčmář" <rkrcmar@ventanamicro.com>
To: "Atish Patra" <atish.patra@linux.dev>,
"Clément Léger" <cleger@rivosinc.com>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Anup Patel" <anup@brainfault.org>,
"Atish Patra" <atishp@atishpatra.org>,
"Shuah Khan" <shuah@kernel.org>,
"Jonathan Corbet" <corbet@lwn.net>,
linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-doc@vger.kernel.org, kvm@vger.kernel.org,
kvm-riscv@lists.infradead.org, linux-kselftest@vger.kernel.org
Cc: "Samuel Holland" <samuel.holland@sifive.com>,
"Andrew Jones" <ajones@ventanamicro.com>,
"Deepak Gupta" <debug@rivosinc.com>,
"Charlie Jenkins" <charlie@rivosinc.com>,
"linux-riscv" <linux-riscv-bounces@lists.infradead.org>
Subject: Re: [PATCH v8 13/14] RISC-V: KVM: add support for FWFT SBI extension
Date: Mon, 26 May 2025 10:58:46 +0200 [thread overview]
Message-ID: <DA5YVKW682V3.2DODRY4EDL3IW@ventanamicro.com> (raw)
In-Reply-To: <5dd587b3-8c04-41d1-b677-5b07266cfec5@linux.dev>
2025-05-23T11:02:11-07:00, Atish Patra <atish.patra@linux.dev>:
> On 5/23/25 9:27 AM, Radim KrÄmáŠwrote:
>> 2025-05-23T17:29:49+02:00, Clément Léger <cleger@rivosinc.com>:
>>> Is this something blocking though ? We'd like to merge FWFT once SBI 3.0
>>> is ratified so that would be nice not delaying it too much. I'll take a
>>> look at it to see if it isn't too long to implement.
>>
>> Not blocking, but I would at least default FWFT to disabled, because
>> current userspace cannot handle [14/14]. (Well... save/restore was
>> probably broken even before, but let's try to not make it worse. :])
>>
>
> User space can not enable or disable misaligned access delegation as
> there is no interface for now rightly pointed by you.
I mean setting default_disabled=true and just disabling FWFT for the
guest unless userspace explicitly enables the incomplete extension.
We would blame the user for wanting mutually exclusive features.
> I guess supporting
> that would be quicker than fixing the broader guest save/restore
> anyways. Isn't it ?
Yes. The save/restore for FWFT is simple (if we disregard the
discussions), but definitely more than a single line.
WARNING: multiple messages have this Message-ID (diff)
From: "Radim Krčmář" <rkrcmar@ventanamicro.com>
To: "Atish Patra" <atish.patra@linux.dev>,
"Clément Léger" <cleger@rivosinc.com>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Anup Patel" <anup@brainfault.org>,
"Atish Patra" <atishp@atishpatra.org>,
"Shuah Khan" <shuah@kernel.org>,
"Jonathan Corbet" <corbet@lwn.net>,
linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-doc@vger.kernel.org, kvm@vger.kernel.org,
kvm-riscv@lists.infradead.org, linux-kselftest@vger.kernel.org
Cc: "Samuel Holland" <samuel.holland@sifive.com>,
"Andrew Jones" <ajones@ventanamicro.com>,
"Deepak Gupta" <debug@rivosinc.com>,
"Charlie Jenkins" <charlie@rivosinc.com>,
"linux-riscv" <linux-riscv-bounces@lists.infradead.org>
Subject: Re: [PATCH v8 13/14] RISC-V: KVM: add support for FWFT SBI extension
Date: Mon, 26 May 2025 10:58:46 +0200 [thread overview]
Message-ID: <DA5YVKW682V3.2DODRY4EDL3IW@ventanamicro.com> (raw)
In-Reply-To: <5dd587b3-8c04-41d1-b677-5b07266cfec5@linux.dev>
2025-05-23T11:02:11-07:00, Atish Patra <atish.patra@linux.dev>:
> On 5/23/25 9:27 AM, Radim KrÄmáŠwrote:
>> 2025-05-23T17:29:49+02:00, Clément Léger <cleger@rivosinc.com>:
>>> Is this something blocking though ? We'd like to merge FWFT once SBI 3.0
>>> is ratified so that would be nice not delaying it too much. I'll take a
>>> look at it to see if it isn't too long to implement.
>>
>> Not blocking, but I would at least default FWFT to disabled, because
>> current userspace cannot handle [14/14]. (Well... save/restore was
>> probably broken even before, but let's try to not make it worse. :])
>>
>
> User space can not enable or disable misaligned access delegation as
> there is no interface for now rightly pointed by you.
I mean setting default_disabled=true and just disabling FWFT for the
guest unless userspace explicitly enables the incomplete extension.
We would blame the user for wanting mutually exclusive features.
> I guess supporting
> that would be quicker than fixing the broader guest save/restore
> anyways. Isn't it ?
Yes. The save/restore for FWFT is simple (if we disregard the
discussions), but definitely more than a single line.
_______________________________________________
linux-riscv mailing list
linux-riscv@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-riscv
next prev parent reply other threads:[~2025-05-26 8:58 UTC|newest]
Thread overview: 123+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-23 10:19 [PATCH v8 00/14] riscv: add SBI FWFT misaligned exception delegation support Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` [PATCH v8 01/14] riscv: sbi: add Firmware Feature (FWFT) SBI extensions definitions Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` [PATCH v8 02/14] riscv: sbi: remove useless parenthesis Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` [PATCH v8 03/14] riscv: sbi: add new SBI error mappings Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` [PATCH v8 04/14] riscv: sbi: add FWFT extension interface Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` [PATCH v8 05/14] riscv: sbi: add SBI FWFT extension calls Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` [PATCH v8 06/14] riscv: misaligned: request misaligned exception from SBI Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-12-25 10:14 ` Vivian Wang
2025-12-25 10:14 ` Vivian Wang
2025-12-25 10:14 ` Vivian Wang
2025-12-28 13:46 ` Clément Léger
2025-12-28 13:46 ` Clément Léger
2025-12-28 13:46 ` Clément Léger
2025-05-23 10:19 ` [PATCH v8 07/14] riscv: misaligned: use on_each_cpu() for scalar misaligned access probing Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 18:37 ` Charlie Jenkins
2025-05-23 18:37 ` Charlie Jenkins
2025-05-23 18:37 ` Charlie Jenkins
2025-05-23 10:19 ` [PATCH v8 08/14] riscv: misaligned: declare misaligned_access_speed under CONFIG_RISCV_MISALIGNED Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 18:36 ` Charlie Jenkins
2025-05-23 18:36 ` Charlie Jenkins
2025-05-23 18:36 ` Charlie Jenkins
2025-05-29 12:43 ` Andrew Jones
2025-05-29 12:43 ` Andrew Jones
2025-05-29 12:43 ` Andrew Jones
2025-05-23 10:19 ` [PATCH v8 09/14] riscv: misaligned: move emulated access uniformity check in a function Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 18:30 ` Charlie Jenkins
2025-05-23 18:30 ` Charlie Jenkins
2025-05-23 18:30 ` Charlie Jenkins
2025-05-23 19:21 ` Clément Léger
2025-05-23 19:21 ` Clément Léger
2025-05-23 19:21 ` Clément Léger
2025-05-26 8:41 ` Andrew Jones
2025-05-26 8:41 ` Andrew Jones
2025-05-26 8:41 ` Andrew Jones
2025-05-26 9:38 ` Clément Léger
2025-05-26 9:38 ` Clément Léger
2025-05-26 9:38 ` Clément Léger
2025-05-23 10:19 ` [PATCH v8 10/14] riscv: misaligned: add a function to check misalign trap delegability Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 18:39 ` Charlie Jenkins
2025-05-23 18:39 ` Charlie Jenkins
2025-05-23 18:39 ` Charlie Jenkins
2025-05-23 10:19 ` [PATCH v8 11/14] RISC-V: KVM: add SBI extension init()/deinit() functions Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-06-12 13:24 ` Anup Patel
2025-06-12 13:24 ` Anup Patel
2025-06-12 13:24 ` Anup Patel
2025-05-23 10:19 ` [PATCH v8 12/14] RISC-V: KVM: add SBI extension reset callback Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-06-12 13:24 ` Anup Patel
2025-06-12 13:24 ` Anup Patel
2025-06-12 13:24 ` Anup Patel
2025-05-23 10:19 ` [PATCH v8 13/14] RISC-V: KVM: add support for FWFT SBI extension Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 13:05 ` Radim Krčmář
2025-05-23 13:05 ` Radim Krčmář
2025-05-23 13:05 ` Radim Krčmář
2025-05-23 15:29 ` Clément Léger
2025-05-23 15:29 ` Clément Léger
2025-05-23 15:29 ` Clément Léger
2025-05-23 16:27 ` Radim Krčmář
2025-05-23 16:27 ` Radim Krčmář
2025-05-23 16:27 ` Radim Krčmář
2025-05-23 18:02 ` Atish Patra
2025-05-23 18:02 ` Atish Patra
2025-05-23 18:02 ` Atish Patra
2025-05-23 19:23 ` Clément Léger
2025-05-23 19:23 ` Clément Léger
2025-05-23 19:23 ` Clément Léger
2025-05-26 8:58 ` Radim Krčmář [this message]
2025-05-26 8:58 ` Radim Krčmář
2025-05-26 8:58 ` Radim Krčmář
2025-06-12 13:25 ` Anup Patel
2025-06-12 13:25 ` Anup Patel
2025-06-12 13:25 ` Anup Patel
2025-05-23 10:19 ` [PATCH v8 14/14] RISC-V: KVM: add support for SBI_FWFT_MISALIGNED_DELEG Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 10:19 ` Clément Léger
2025-05-23 13:08 ` Radim Krčmář
2025-05-23 13:08 ` Radim Krčmář
2025-05-23 13:08 ` Radim Krčmář
2025-06-12 13:26 ` Anup Patel
2025-06-12 13:26 ` Anup Patel
2025-06-12 13:26 ` Anup Patel
2025-06-04 18:02 ` [PATCH v8 00/14] riscv: add SBI FWFT misaligned exception delegation support Palmer Dabbelt
2025-06-04 18:02 ` Palmer Dabbelt
2025-06-04 18:02 ` Palmer Dabbelt
2025-06-04 19:32 ` Charlie Jenkins
2025-06-04 19:32 ` Charlie Jenkins
2025-06-04 19:32 ` Charlie Jenkins
2025-06-05 7:12 ` Alexandre Ghiti
2025-06-05 7:12 ` Alexandre Ghiti
2025-06-05 7:12 ` Alexandre Ghiti
2025-06-05 1:30 ` patchwork-bot+linux-riscv
2025-06-05 1:30 ` patchwork-bot+linux-riscv
2025-06-05 1:30 ` patchwork-bot+linux-riscv
2025-08-10 21:12 ` patchwork-bot+linux-riscv
2025-08-10 21:12 ` patchwork-bot+linux-riscv
2025-08-10 21:12 ` patchwork-bot+linux-riscv
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DA5YVKW682V3.2DODRY4EDL3IW@ventanamicro.com \
--to=rkrcmar@ventanamicro.com \
--cc=ajones@ventanamicro.com \
--cc=anup@brainfault.org \
--cc=atish.patra@linux.dev \
--cc=atishp@atishpatra.org \
--cc=charlie@rivosinc.com \
--cc=cleger@rivosinc.com \
--cc=corbet@lwn.net \
--cc=debug@rivosinc.com \
--cc=kvm-riscv@lists.infradead.org \
--cc=kvm@vger.kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=linux-riscv-bounces@lists.infradead.org \
--cc=linux-riscv@lists.infradead.org \
--cc=palmer@dabbelt.com \
--cc=paul.walmsley@sifive.com \
--cc=samuel.holland@sifive.com \
--cc=shuah@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.