Re: [PATCH v1 0/2] RISC-V: enable rust

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 778 bytes --]

On Sat, Jan 27, 2024 at 02:46:38PM +0100, Miguel Ojeda wrote:
> On Fri, Jan 26, 2024 at 11:01 PM Conor Dooley <conor@kernel.org> wrote:
> >
> > Is that even needed? We already have ARCH_SUPPORTS_CFI_CLANG and AFAIU
> > rust supports it if clang does, so a second option is superfluous?
> 
> From a quick look, I don't see it enabled in any RISC-V built-in
> target in `rustc` yet.
> 
> It may also still be the case that KCFI needs some tweaks for, say,
> RISC-V, before the flag actually works, i.e. we couldn't just test the
> flag in that case -- Ramon: how likely is it that RISC-V would work if
> KCFI works for aarch64 and x86_64?

Well, there's been no reply here. I'll do sa you suggested and add a 
depends on !CFI_CLANG to RUST.

Cheers,
Conor.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]