Re: [PATCH 1/3] rust: completion: implement initial abstraction
From: Danilo Krummrich
Date: Thu Jun 12 2025 - 06:41:39 EST
On Thu, Jun 12, 2025 at 10:15:55AM +0200, Benno Lossin wrote:
> On Tue Jun 3, 2025 at 10:48 PM CEST, Danilo Krummrich wrote:
> > + /// Signal all tasks waiting on this completion.
> > + ///
> > + /// This method wakes up all tasks waiting on this completion; after this operation the
> > + /// completion is permanently done.
> > + pub fn complete_all(&self) {
> > + // SAFETY: `self.as_raw()` is a pointer to a valid `struct completion`.
> > + unsafe { bindings::complete_all(self.as_raw()) };
> > + }
> > +
> > + /// Wait for completion of a task.
> > + ///
> > + /// This method waits for the completion of a task; it is not interruptible and there is no
> > + /// timeout.
>
> Another thing that we should document is weather this function returns
> immediately when `complete_all` was already called in the past.
The details are all documented in [1], which is also linked in the module
documentation of this file.
[1] https://docs.kernel.org/scheduler/completion.html