d/numam-dpdk
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

net/virtio: replace full barrier with thread fence

Browse Source 
Replace the smp barriers with atomic thread fence for synchronization
between different threads, if there are no load/store operations.

Signed-off-by: Joyce Kong <joyce.kong@arm.com>
Reviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>
Reviewed-by: Maxime Coquelin <maxime.coquelin@redhat.com>
This commit is contained in:
Joyce Kong 2020-12-21 22:23:21 +08:00 committed by Ferruh Yigit
parent e51a474ced
commit 240a9941d4
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
drivers/net/virtio/virtqueue.h
View File

@ -26,7 +26,7 @@ struct rte_mbuf;
/* /*
* Per virtio_ring.h in Linux. * Per virtio_ring.h in Linux.
* For virtio_pci on SMP, we don't need to order with respect to MMIO * For virtio_pci on SMP, we don't need to order with respect to MMIO
* accesses through relaxed memory I/O windows, so smp_mb() et al are * accesses through relaxed memory I/O windows, so thread_fence is
* sufficient. * sufficient.
* *
* For using virtio to talk to real devices (eg. vDPA) we do need real * For using virtio to talk to real devices (eg. vDPA) we do need real
@ -36,7 +36,7 @@ static inline void
virtio_mb(uint8_t weak_barriers) virtio_mb(uint8_t weak_barriers)
{ {
if (weak_barriers) if (weak_barriers)
rte_smp_mb(); rte_atomic_thread_fence(__ATOMIC_SEQ_CST);
else else
rte_mb(); rte_mb();
} }
@ -45,7 +45,7 @@ static inline void
virtio_rmb(uint8_t weak_barriers) virtio_rmb(uint8_t weak_barriers)
{ {
if (weak_barriers) if (weak_barriers)
rte_smp_rmb(); rte_atomic_thread_fence(__ATOMIC_ACQUIRE);
else else
rte_io_rmb(); rte_io_rmb();
} }
@ -54,7 +54,7 @@ static inline void
virtio_wmb(uint8_t weak_barriers) virtio_wmb(uint8_t weak_barriers)
{ {
if (weak_barriers) if (weak_barriers)
rte_smp_wmb(); rte_atomic_thread_fence(__ATOMIC_RELEASE);
else else
rte_io_wmb(); rte_io_wmb();
} }