Thread (64 messages) 64 messages, 6 authors, 2024-10-25

Re: [PATCH net-next v3 3/8] rust: time: Change output of Ktime's sub operation to Delta

From: FUJITA Tomonori <fujita.tomonori@gmail.com>
Date: 2024-10-17 07:11:04
Also in: lkml, rust-for-linux

On Wed, 16 Oct 2024 10:25:11 +0200
Alice Ryhl [off-list ref] wrote:
On Wed, Oct 16, 2024 at 5:53 AM FUJITA Tomonori
[off-list ref] wrote:
quoted
Change the output type of Ktime's subtraction operation from Ktime to
Delta. Currently, the output is Ktime:

Ktime = Ktime - Ktime

It means that Ktime is used to represent timedelta. Delta is
introduced so use it. A typical example is calculating the elapsed
time:

Delta = current Ktime - past Ktime;

Signed-off-by: FUJITA Tomonori <fujita.tomonori@gmail.com>
So this means that you are repurposing Ktime as a replacement for
Instant rather than making both a Delta and Instant type? Okay. That
seems reasonable enough.
Yes.

Surely, we could create both Delta and Instant. What is Ktime used
for? Both can simply use bindings::ktime_t like the followings?

pub struct Instant {
    inner: bindings::ktime_t,
}

pub struct Delta {
    inner: bindings::ktime_t,
}
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help