Merge branch 'remove-rtnl-lock-protection-of-cvq'
authorPaolo Abeni <pabeni@redhat.com>
Tue, 7 May 2024 09:42:03 +0000 (11:42 +0200)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 7 May 2024 09:42:03 +0000 (11:42 +0200)
Daniel Jurgens says:

====================
Remove RTNL lock protection of CVQ

Currently the buffer used for control VQ commands is protected by the
RTNL lock. Previously this wasn't a major concern because the control VQ
was only used during device setup and user interaction. With the recent
addition of dynamic interrupt moderation the control VQ may be used
frequently during normal operation.

This series removes the RNTL lock dependency by introducing a mutex
to protect the control buffer and writing SGs to the control VQ.

v6:
- Rebased over new stats code.
- Added comment to cvq_lock, init the mutex unconditionally,
  and replaced some duplicate code with a goto.
- Fixed minor grammer errors, checkpatch warnings, and clarified
  a comment.
v5:
- Changed cvq_lock to a mutex.
- Changed dim_lock to mutex, because it's held taking
  the cvq_lock.
- Use spin/mutex_lock/unlock vs guard macros.
v4:
- Protect dim_enabled with same lock as well intr_coal.
- Rename intr_coal_lock to dim_lock.
- Remove some scoped_guard where the error path doesn't
  have to be in the lock.
v3:
- Changed type of _offloads to __virtio16 to fix static
  analysis warning.
- Moved a misplaced hunk to the correct patch.
v2:
- New patch to only process the provided queue in
  virtnet_dim_work
- New patch to lock per queue rx coalescing structure.
====================

Link: https://lore.kernel.org/r/20240503202445.1415560-1-danielj@nvidia.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>

Trivial merge