Re: [PATCH v6 0/3] rust: Add irq abstraction, SpinLockIrq

[Dirk Behme]

Hi Lyude,

On 16.09.24 23:28, Lyude Paul wrote:
> This adds a simple interface for disabling and enabling CPUs, along with
> the ability to mark a function as expecting interrupts be disabled -
> along with adding bindings for spin_lock_irqsave/spin_lock_irqrestore().
>
> Current example usecase (very much WIP driver) in rvkms:
>
> https://gitlab.freedesktop.org/lyudess/linux/-/commits/rvkms-example-08012024
>
> specifically drivers/gpu/drm/rvkms/crtc.rs
>
> This series depends on
> https://lore.kernel.org/rust-for-linux/ZuKNszXSw-LbgW1e@boqun-archlinux/
>
> Lyude Paul (3):
>    rust: Introduce irq module
>    rust: sync: Introduce lock::Backend::Context
>    rust: sync: Add SpinLockIrq


To have it in this thread as well I just want to mention the discussion in

https://lore.kernel.org/rust-for-linux/87a5falmjy.fsf@xxxxxxxxxx/

which results in the impression that this patch series needs to update 
`CondVar::wait` to support waiting with irq disabled.

Best regards

Dirk


>   rust/helpers/helpers.c            |   1 +
>   rust/helpers/irq.c                |  22 +++++++
>   rust/kernel/irq.rs                |  96 +++++++++++++++++++++++++++
>   rust/kernel/lib.rs                |   1 +
>   rust/kernel/sync.rs               |   2 +-
>   rust/kernel/sync/lock.rs          |  17 ++++-
>   rust/kernel/sync/lock/mutex.rs    |   1 +
>   rust/kernel/sync/lock/spinlock.rs | 105 ++++++++++++++++++++++++++++++
>   8 files changed, 242 insertions(+), 3 deletions(-)
>   create mode 100644 rust/helpers/irq.c
>   create mode 100644 rust/kernel/irq.rs
>
>
> base-commit: a2f11547052001bd448ccec81dd1e68409078fbb
> prerequisite-patch-id: 926565461e47df321ce1bed92894cc1f265896ef