perfbook.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH -perfboof] memorder: Fix word choice (data dependencies --> address dependencies)
@ 2023-09-18  1:00 Akira Yokosawa
  2023-09-18  5:50 ` Paul E. McKenney
  0 siblings, 1 reply; 2+ messages in thread
From: Akira Yokosawa @ 2023-09-18  1:00 UTC (permalink / raw)
  To: Paul E. McKenney; +Cc: perfbook, Akira Yokosawa

There remain obsolete word choices of "data dependencies" in section
Alpha.

It is load-to-load address dependencies that other archs respect.

Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
---
Hi Paul,

I caught them while attempting to add index markers to
address/data/control dependency.

        Thanks, Akira
--
 memorder/memorder.tex | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/memorder/memorder.tex b/memorder/memorder.tex
index 1a4edf8bbb6d..86bb8f41b5f8 100644
--- a/memorder/memorder.tex
+++ b/memorder/memorder.tex
@@ -5403,7 +5403,7 @@ One could place an \co{smp_rmb()} primitive
 between the pointer fetch and dereference in order to force Alpha
 to order the pointer fetch with the later dependent load.
 However, this imposes unneeded overhead on systems (such as \ARM,
-Itanium, PPC, and SPARC) that respect data dependencies on the read side.
+Itanium, PPC, and SPARC) that respect address dependencies on the read side.
 A \co{smp_read_barrier_depends()} primitive was therefore added to the
 Linux kernel to eliminate overhead on these systems, but was removed
 in v5.9 of the Linux kernel in favor of augmenting Alpha's definition
@@ -5428,7 +5428,7 @@ Upon receipt of such an IPI, a CPU would execute a memory-barrier
 instruction, implementing a system-wide memory barrier similar to that
 provided by the Linux kernel's \co{sys_membarrier()} system call.
 Additional logic is required to avoid deadlocks.
-Of course, CPUs that respect data dependencies would define such a barrier
+Of course, CPUs that respect address dependencies would define such a barrier
 to simply be \co{smp_store_release()}.
 However, this approach was deemed by the Linux community
 to impose excessive overhead~\cite{McKenney01f}, and to their point would

base-commit: 8224a12917dd115ead6fce0f785b19a4c7d50f18
-- 
2.25.1


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH -perfboof] memorder: Fix word choice (data dependencies --> address dependencies)
  2023-09-18  1:00 [PATCH -perfboof] memorder: Fix word choice (data dependencies --> address dependencies) Akira Yokosawa
@ 2023-09-18  5:50 ` Paul E. McKenney
  0 siblings, 0 replies; 2+ messages in thread
From: Paul E. McKenney @ 2023-09-18  5:50 UTC (permalink / raw)
  To: Akira Yokosawa; +Cc: perfbook

On Mon, Sep 18, 2023 at 10:00:18AM +0900, Akira Yokosawa wrote:
> There remain obsolete word choices of "data dependencies" in section
> Alpha.
> 
> It is load-to-load address dependencies that other archs respect.
> 
> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> ---
> Hi Paul,
> 
> I caught them while attempting to add index markers to
> address/data/control dependency.

Good catch, queued and pushed, thank you!!!

							Thanx, Paul

>         Thanks, Akira
> --
>  memorder/memorder.tex | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/memorder/memorder.tex b/memorder/memorder.tex
> index 1a4edf8bbb6d..86bb8f41b5f8 100644
> --- a/memorder/memorder.tex
> +++ b/memorder/memorder.tex
> @@ -5403,7 +5403,7 @@ One could place an \co{smp_rmb()} primitive
>  between the pointer fetch and dereference in order to force Alpha
>  to order the pointer fetch with the later dependent load.
>  However, this imposes unneeded overhead on systems (such as \ARM,
> -Itanium, PPC, and SPARC) that respect data dependencies on the read side.
> +Itanium, PPC, and SPARC) that respect address dependencies on the read side.
>  A \co{smp_read_barrier_depends()} primitive was therefore added to the
>  Linux kernel to eliminate overhead on these systems, but was removed
>  in v5.9 of the Linux kernel in favor of augmenting Alpha's definition
> @@ -5428,7 +5428,7 @@ Upon receipt of such an IPI, a CPU would execute a memory-barrier
>  instruction, implementing a system-wide memory barrier similar to that
>  provided by the Linux kernel's \co{sys_membarrier()} system call.
>  Additional logic is required to avoid deadlocks.
> -Of course, CPUs that respect data dependencies would define such a barrier
> +Of course, CPUs that respect address dependencies would define such a barrier
>  to simply be \co{smp_store_release()}.
>  However, this approach was deemed by the Linux community
>  to impose excessive overhead~\cite{McKenney01f}, and to their point would
> 
> base-commit: 8224a12917dd115ead6fce0f785b19a4c7d50f18
> -- 
> 2.25.1
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-09-18  5:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-18  1:00 [PATCH -perfboof] memorder: Fix word choice (data dependencies --> address dependencies) Akira Yokosawa
2023-09-18  5:50 ` Paul E. McKenney

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).