linux-kernel - Re: [PATCH net-next v2 0/6] rust: Add IO polling

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20241013.101505.2305788717444047197.fujita.tomonori@gmail.com>
Date: Sun, 13 Oct 2024 10:15:05 +0900 (JST)
From: FUJITA Tomonori <fujita.tomonori@...il.com>
To: boqun.feng@...il.com
Cc: fujita.tomonori@...il.com, netdev@...r.kernel.org,
 rust-for-linux@...r.kernel.org, andrew@...n.ch, hkallweit1@...il.com,
 tmgross@...ch.edu, ojeda@...nel.org, alex.gaynor@...il.com,
 gary@...yguo.net, bjorn3_gh@...tonmail.com, benno.lossin@...ton.me,
 a.hindborg@...sung.com, aliceryhl@...gle.com, anna-maria@...utronix.de,
 frederic@...nel.org, tglx@...utronix.de, arnd@...db.de,
 linux-kernel@...r.kernel.org, jstultz@...gle.com, sboyd@...nel.org
Subject: Re: [PATCH net-next v2 0/6] rust: Add IO polling

On Sat, 12 Oct 2024 08:29:06 -0700
Boqun Feng <boqun.feng@...il.com> wrote:

> While, we are at it, I want to suggest that we also add
> rust/kernel/time{.rs, /} into the "F:" entries of TIME subsystem like:
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b77f4495dcf4..09e46a214333 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -23376,6 +23376,8 @@ F:      kernel/time/timeconv.c
>  F:     kernel/time/timecounter.c
>  F:     kernel/time/timekeeping*
>  F:     kernel/time/time_test.c
> +F:     rust/kernel/time.rs
> +F:     rust/kernel/time/
>  F:     tools/testing/selftests/timers/
> 
>  TIPC NETWORK LAYER
> 
> This will help future contributers copy the correct people while
> submission. Could you maybe add a patch of this in your series if this
> sounds reasonable to you? Thanks!

Agreed that it's better to have Rust time abstractions in
MAINTAINERS. You add it into the time entry but there are two options
in the file; time and timer?

TIMEKEEPING, CLOCKSOURCE CORE, NTP, ALARMTIMER
M:      John Stultz <jstultz@...gle.com>
M:      Thomas Gleixner <tglx@...utronix.de>
R:      Stephen Boyd <sboyd@...nel.org>

HIGH-RESOLUTION TIMERS, TIMER WHEEL, CLOCKEVENTS
M:      Anna-Maria Behnsen <anna-maria@...utronix.de>
M:      Frederic Weisbecker <frederic@...nel.org>
M:      Thomas Gleixner <tglx@...utronix.de>

The current Rust abstractions which play mainly with ktimer.h. it's 
not time, timer stuff, I think.

As planned, we'll move *.rs files from rust/kernel in the future,
how we handle time and timer abstractions?