On 10.09.2022 17:49, Juergen Gross wrote:
> In order to prepare a type-safe recursive spinlock structure, add
> explicitly non-recursive locking functions to be used for non-recursive
> locking of spinlocks, which are use recursively, too.
While I can see that something needs doing, a function name like
spin_unlock_nonrecursive_irqrestore() is really unwieldy, imo.
Just one minor further remark:
> @@ -64,7 +64,7 @@ void pcidevs_unlock(void)
>
> bool_t pcidevs_locked(void)
> {
> - return !!spin_is_locked(&_pcidevs_lock);
> + return !!spin_recursive_is_locked(&_pcidevs_lock);
> }
While touching this line, could you please also get rid of the !! ?
Jan
