diff options
| author | Anup Patel <apatel@ventanamicro.com> | 2023-11-21 14:49:42 +0530 |
|---|---|---|
| committer | Anup Patel <anup@brainfault.org> | 2023-11-26 18:45:06 +0530 |
| commit | f520256d03da84e7ff8bddd084ef873192f46304 (patch) | |
| tree | b7903d467c55604303472f67190ffe8ab8a87b7f /lib/utils | |
| parent | 791704cd09c019decf438f6e4f56423cc95cb66a (diff) | |
| download | opensbi-f520256d03da84e7ff8bddd084ef873192f46304.tar.gz opensbi-f520256d03da84e7ff8bddd084ef873192f46304.tar.bz2 opensbi-f520256d03da84e7ff8bddd084ef873192f46304.zip | |
lib: sbi: Allow relaxed MMIO writes in device ipi_send() callback
Currently, we have a smp_wmb() between atomic_raw_set_bit() and
ipi_send() device callback whereas the MMIO writes done by the
device ipi_send() callback will also include a barrier.
We can avoid unnecessary/redundant barriers described above by
allowing relaxed MMIO writes in device ipi_send() callback. To
achieve this, we simply use wmb() instead of smp_wmb() before
calling device ipi_send().
Signed-off-by: Anup Patel <apatel@ventanamicro.com>
Reported-by: Bo Gan <ganboing@gmail.com>
Diffstat (limited to 'lib/utils')
| -rw-r--r-- | lib/utils/ipi/aclint_mswi.c | 2 | ||||
| -rw-r--r-- | lib/utils/irqchip/imsic.c | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/lib/utils/ipi/aclint_mswi.c b/lib/utils/ipi/aclint_mswi.c index a3bfb4a9..bfd6a45b 100644 --- a/lib/utils/ipi/aclint_mswi.c +++ b/lib/utils/ipi/aclint_mswi.c @@ -41,7 +41,7 @@ static void mswi_ipi_send(u32 hart_index) /* Set ACLINT IPI */ msip = (void *)mswi->addr; - writel(1, &msip[sbi_hartindex_to_hartid(hart_index) - + writel_relaxed(1, &msip[sbi_hartindex_to_hartid(hart_index) - mswi->first_hartid]); } diff --git a/lib/utils/irqchip/imsic.c b/lib/utils/irqchip/imsic.c index 78f5895a..36ef66cf 100644 --- a/lib/utils/irqchip/imsic.c +++ b/lib/utils/irqchip/imsic.c @@ -186,8 +186,8 @@ static void imsic_ipi_send(u32 hart_index) } if (regs->size && (reloff < regs->size)) - writel(IMSIC_IPI_ID, - (void *)(regs->addr + reloff + IMSIC_MMIO_PAGE_LE)); + writel_relaxed(IMSIC_IPI_ID, + (void *)(regs->addr + reloff + IMSIC_MMIO_PAGE_LE)); } static struct sbi_ipi_device imsic_ipi_device = { |