]> git.baikalelectronics.ru Git - kernel.git/commit
vdpa: mlx5: synchronize driver status with CVQ
authorJason Wang <jasowang@redhat.com>
Tue, 29 Mar 2022 04:21:08 +0000 (12:21 +0800)
committerMichael S. Tsirkin <mst@redhat.com>
Wed, 30 Mar 2022 08:18:14 +0000 (04:18 -0400)
commitb3d18d8141d577d1bd39286ca5bfa5107e6a9c0c
tree34f7baa0a47551ffe0912b35df814fe19930f918
parent4e4788bc4b23745a3e2b5795496cd38bd3c202be
vdpa: mlx5: synchronize driver status with CVQ

Currently, CVQ doesn't have any synchronization with the driver
status. Then CVQ emulation code run in the middle of:

1) device reset
2) device status changed
3) map updating

The will lead several unexpected issue like trying to execute CVQ
command after the driver has been teared down.

Fixing this by using reslock to synchronize CVQ emulation code with
the driver status changing:

- protect the whole device reset, status changing and set_map()
  updating with reslock
- protect the CVQ handler with the reslock and check
  VIRTIO_CONFIG_S_DRIVER_OK in the CVQ handler

This will guarantee that:

1) CVQ handler won't work if VIRTIO_CONFIG_S_DRIVER_OK is not set
2) CVQ handler will see a consistent state of the driver instead of
   the partial one when it is running in the middle of the
   teardown_driver() or setup_driver().

Cc: 463ccc085b882 ("vdpa/mlx5: Add support for control VQ and MAC setting")
Signed-off-by: Jason Wang <jasowang@redhat.com>
Link: https://lore.kernel.org/r/20220329042109.4029-2-jasowang@redhat.com
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Acked-by: Eli Cohen <elic@nvidia.com>
drivers/vdpa/mlx5/net/mlx5_vnet.c