[PATCH v5 2/7] rust: hrtimer: Add HrTimerInstant

Lyude Paul posted 7 patches 3 months, 4 weeks ago
There is a newer version of this series
[PATCH v5 2/7] rust: hrtimer: Add HrTimerInstant
Posted by Lyude Paul 3 months, 4 weeks ago
Since we want to add HrTimer methods that can accept Instants, we will want
to make sure that for each method we are using the correct Clocksource for
the given HrTimer. This would get a bit overly-verbose, so add a simple
HrTimerInstant type-alias to handle this for us.

Signed-off-by: Lyude Paul <lyude@redhat.com>
---
 rust/kernel/time/hrtimer.rs | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/rust/kernel/time/hrtimer.rs b/rust/kernel/time/hrtimer.rs
index 294f257ef5c5a..c775d7abdf5ce 100644
--- a/rust/kernel/time/hrtimer.rs
+++ b/rust/kernel/time/hrtimer.rs
@@ -72,6 +72,11 @@
 use core::marker::PhantomData;
 use pin_init::PinInit;
 
+/// A type-alias to refer to the [`Instant<C>`] for a given `T` from [`HrTimer<T>`].
+///
+/// Where `C` is the [`ClockSource`] of the [`HrTimer`].
+pub type HrTimerInstant<T> = Instant<<<T as HasHrTimer<T>>::TimerMode as HrTimerMode>::Clock>;
+
 /// A timer backed by a C `struct hrtimer`.
 ///
 /// # Invariants
-- 
2.49.0
Re: [PATCH v5 2/7] rust: hrtimer: Add HrTimerInstant
Posted by Andreas Hindborg 3 months, 3 weeks ago
"Lyude Paul" <lyude@redhat.com> writes:

> Since we want to add HrTimer methods that can accept Instants, we will want
> to make sure that for each method we are using the correct Clocksource for
> the given HrTimer. This would get a bit overly-verbose, so add a simple
> HrTimerInstant type-alias to handle this for us.
>
> Signed-off-by: Lyude Paul <lyude@redhat.com>

Reviewed-by: Andreas Hindborg <a.hindborg@kernel.org>


Best regards,
Andreas Hindborg