Page MenuHomeFreeBSD

jme: Convert driver to CTLFLAG_MPSAFE
ClosedPublic

Authored by seuros on Jan 9 2026, 3:40 PM.
Tags
None
Referenced Files
F167865179: D54618.id169495.diff
Tue, Aug 25, 2:31 AM
F167848763: D54618.diff
Mon, Aug 24, 11:54 PM
Unknown Object (File)
Tue, Aug 18, 9:27 AM
Unknown Object (File)
Sat, Aug 15, 9:49 PM
Unknown Object (File)
Sat, Aug 15, 4:46 PM
Unknown Object (File)
Fri, Aug 14, 7:37 PM
Unknown Object (File)
Fri, Aug 14, 12:49 PM
Unknown Object (File)
Fri, Aug 14, 9:42 AM
Subscribers
None

Details

Summary

Replace CTLFLAG_NEEDGIANT with CTLFLAG_MPSAFE for all interrupt
coalescing sysctls. The driver uses jme_mtx mutex for proper
synchronization and does not require Giant lock.

Sysctls converted:

  • dev.jme.X.tx_coal_to (TX coalescing timeout)
  • dev.jme.X.tx_coal_pkt (TX coalescing packet count)
  • dev.jme.X.rx_coal_to (RX coalescing timeout)
  • dev.jme.X.rx_coal_pkt (RX coalescing packet count)
  • dev.jme.X.process_limit (max RX events to process)

Diff Detail

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

Event Timeline

seuros requested review of this revision.Jan 9 2026, 3:40 PM
seuros created this revision.

speaker, clock: Replace Giant lock with fine-grained locking

Add sx lock to the speaker driver to protect spkr_active and spkr_inbuf,
removing the D_NEEDGIANT flag. Add mtx_lock_spin protection for timer2_state
in clock.c timer_spkr_acquire/release functions.

Sorry, still using arc incorrectly. Need i need to learn git-arc.

This revision is now accepted and ready to land.Jan 12 2026, 12:35 AM
This revision was automatically updated to reflect the committed changes.