[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <875xpvhlgm.ffs@tglx>
Date: Sun, 13 Oct 2024 21:06:01 +0200
From: Thomas Gleixner <tglx@...utronix.de>
To: Dirk Behme <dirk.behme@...il.com>, Lyude Paul <lyude@...hat.com>,
rust-for-linux@...r.kernel.org
Cc: Danilo Krummrich <dakr@...hat.com>, airlied@...hat.com, Ingo Molnar
<mingo@...hat.com>, Will Deacon <will@...nel.org>, Waiman Long
<longman@...hat.com>, Peter Zijlstra <peterz@...radead.org>,
linux-kernel@...r.kernel.org, Miguel Ojeda <ojeda@...nel.org>, Alex Gaynor
<alex.gaynor@...il.com>, Wedson Almeida Filho <wedsonaf@...il.com>, Boqun
Feng <boqun.feng@...il.com>, Gary Guo <gary@...yguo.net>, Björn Roy Baron
<bjorn3_gh@...tonmail.com>, Benno Lossin <benno.lossin@...ton.me>, Andreas
Hindborg <a.hindborg@...sung.com>, Alice Ryhl <aliceryhl@...gle.com>,
Trevor Gross <tmgross@...ch.edu>
Subject: Re: [PATCH v6 0/3] rust: Add irq abstraction, SpinLockIrq
On Sat, Oct 12 2024 at 07:29, Dirk Behme wrote:
> 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@kernel.org/
>
> which results in the impression that this patch series needs to update
> `CondVar::wait` to support waiting with irq disabled.
What means waiting with interrupts disabled?
Spinning? Why would you want to do that in the first place?
There are not a lot of use cases to do so, except for core code.
Thanks,
tglx
Powered by blists - more mailing lists