Re: [PATCH 6/7] rust: workqueue: add Work::cancel_work_sync()

From: Danilo Krummrich

Date: Fri Aug 07 2026 - 07:59:09 EST


On Fri Aug 7, 2026 at 9:51 AM CEST, Onur Özkan wrote:
> Perhaps you could pull the exiting effort for `cancel_sync` support [1] which
> is also used by the tyr reset implementation [2]?

Please also see my reply to John [3].

I think for your implementation in Tyr you should just use ScopedWork, which
should simplify the implementation significantly.

cancel_sync() for Work could be useful regardless, but it seems to be an
orthogonal effort that requires a separate use-case, so I rather not include it
in this series.

> [1]: https://lore.kernel.org/all/20260617131731.145337-2-work@xxxxxxxxxxxxx
> [2]: https://lore.kernel.org/all/20260708114358.957305-1-work@xxxxxxxxxxxxx

[3] https://lore.kernel.org/all/DKIOSPRHO8V1.286KQFRH751TE@xxxxxxxxxx/