Re: [PATCH v4] locking/memory-barriers.txt: Improve documentation for writel() example

From: Arnd Bergmann
Date: Mon Oct 17 2022 - 16:55:33 EST


On Mon, Oct 10, 2022, at 12:13 PM, Parav Pandit wrote:
> The cited commit describes that when using writel(), explcit wmb()
> is not needed. wmb() is an expensive barrier. writel() uses the needed
> platform specific barrier instead of expensive wmb().
>
> Hence update the example to be more accurate that matches the current
> implementation.
>
> commit 5846581e3563 ("locking/memory-barriers.txt: Fix broken DMA vs.
> MMIO ordering example")
>
> Signed-off-by: Parav Pandit <parav@xxxxxxxxxx>

I have no objections, though I still don't see a real need to change
the wording here.

Acked-by: Arnd Bergmann <arnd@xxxxxxxx>