Page MenuHomeFreeBSD

LinuxKPI: Allow spin_lock_irqsave to be called within a critical section
ClosedPublic

Authored by wulf on Jan 13 2022, 11:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 15, 2:57 AM
Unknown Object (File)
Thu, Sep 18, 12:37 PM
Unknown Object (File)
Thu, Sep 18, 12:28 PM
Unknown Object (File)
Sep 15 2025, 2:47 AM
Unknown Object (File)
Sep 15 2025, 2:39 AM
Unknown Object (File)
Aug 15 2025, 12:52 AM
Unknown Object (File)
Aug 6 2025, 12:10 AM
Unknown Object (File)
Jul 7 2025, 7:04 AM

Details

Summary

with spinning on spin_trylock. dma-buf part of drm-kmod depends on this
property and absence of it support results in "mi_switch: switch in a
critical section" assertions [1][2].

[1] https://github.com/freebsd/drm-kmod/issues/116
[2] https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=261166

MFC after: 1 week

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

wulf requested review of this revision.Jan 13 2022, 11:48 PM
This revision is now accepted and ready to land.Jan 14 2022, 6:28 AM
sys/compat/linuxkpi/common/include/linux/spinlock.h
120

Is this a property of spin_lock_irqsave() only or should other spin_lock() functions be updated aswell?

sys/compat/linuxkpi/common/include/linux/spinlock.h
120

AFAIK no, they should not. Ordinary spin_lock() requires fully preemptible context at least on RT-kernels. See PREEMPT_RT caveats section of https://www.kernel.org/doc/html/latest/locking/locktypes.html

May be some other spin_lock_XXX functions e.g. spin_lock_irqsave_nested() can be updated too, but I did not dig in to this direction.

sys/compat/linuxkpi/common/include/linux/spinlock.h
120

Please updsate spin_lock_irqsave_nested() aswell and let's get moving :-)