Re: [PATCH v3] rust: introduce sfile macro for succinct code tracing

["Benno Lossin" <lossin@kernel.org>]

On Fri Jun 13, 2025 at 10:06 PM CEST, Timur Tabi wrote:
> On Fri, 2025-06-13 at 14:32 -0500, Timur Tabi wrote:
>> On Tue, 2025-06-10 at 10:45 +0200, Benno Lossin wrote:
>> > > +
>> > > +        // [`.as_bytes()`] does not support non-ASCII filenames
>> > > +        assert!(FILE.is_ascii());
>> > 
>> > Also here.
>> 
>> I can't get this to work, either.  I've tried all sorts of things, none of them work:
>> 
>>         assert!(::str::is_ascii(FILE));
>>         assert!(::core::str::is_ascii(FILE));

This one should work (you still need to make the `assert` use a path
though).

>>         assert!(FILE.str::is_ascii());
>
> Never mind, I finally got it to work
>
> 	assert!(str::is_ascii(FILE));
>
> Is `str` not part of the `core` module in the kernel?  It seems there are slight differences in the
> module hierarchy in kernel vs. regular Rust.  Is there a document that shows this?

The kernel uses the usual `core` so `str` should exist. What's the error
that you get?

---
Cheers,
Benno