On Thu, Oct 21, 2021 at 03:32:51PM +0800, Zhouyi Zhou wrote:
> In answer to Quick Quiz 15.33, update() here should be updater().
>
> Signed-off-by: Zhouyi Zhou <zhouzhouyi@xxxxxxxxx>
> ---
> Hi Paul,
>
> In answer to Quick Quiz 15.33, update() here should be updater().
> By the way, I tried to follow Akira's patch as an example to make a
> good looking patch ;-)
Good eyes, and the patch does look good, thank you! Queued and pushed.
Thanx, Paul
> Thanks Zhouyi
> --
> memorder/memorder.tex | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/memorder/memorder.tex b/memorder/memorder.tex
> index debfbcc3..eae93f9e 100644
> --- a/memorder/memorder.tex
> +++ b/memorder/memorder.tex
> @@ -2830,7 +2830,7 @@ instead of the expected value 144.
> not order later loads, only later stores.
> If you really need ordering, you could place an \co{smp_rmb()}
> between \clnref{equ,pc}.
> - Or better yet, have \co{update()}
> + Or better yet, have \co{updater()}
> allocate two structures instead of reusing the structure.
> For more information, see
> \cref{sec:memorder:Control-Dependency Calamities}.
> --
> 2.25.1
>
