Re: [PATCH -perfbook] memorder: fix a line number reference error

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

On Fri, 14 Oct 2022 13:55:43 +0800, Zhouyi Zhou wrote:
> In section 15.2.3, "Listing 15.10 shows how to make this work
> reliably on pre-v4.15 Linux kernels running on DEC Alpha, by
> replacing line 19' READ_ONCE() with lockless_dereference()"
> 
> line 19 should be line 18.
Close!

The reference should be to line 18 of Listing 15.9, where the to-be-
replaced READ_ONCE() resides.

I'll submit a patch soon with your Reported-by.

        Thanks, Akira

> 
> Signed-off-by: Zhouyi Zhou <zhouzhouyi@xxxxxxxxx>
> ---
> Hi Paul
> I like Chapter 15 very much as well as other chapters,
> I used to read one paragraph a day when I am travelling ;-)
> 
> I think there is a small line number reference error in chapter
> 15.2.3, ignore me if it is not a error.
> 
> Many Thanks
> Zhouyi
> --
>  memorder/memorder.tex | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/memorder/memorder.tex b/memorder/memorder.tex
> index c350758d..1839e276 100644
> --- a/memorder/memorder.tex
> +++ b/memorder/memorder.tex
> @@ -1279,7 +1279,7 @@ exists (1:r2=x0 /\ 1:r3=1)
>  % \path{C-MP+o-wmb-o+ld-addr-o.litmus} available at commit bc4b1c3f3b35
>  % ("styleguide: Loosen restriction on comment in litmus test")
>  shows how to make this work reliably on pre-v4.15 Linux kernels running on
> -DEC Alpha, by replacing \clnref{read}'s \co{READ_ONCE()} with
> +DEC Alpha, by replacing \clnref{deref}'s \co{READ_ONCE()} with
>  \apikh{lockless_dereference()},\footnote{
>  	Note that \co{lockless_dereference()} is not needed on v4.15 and
>  	later, and therefore is not available in these later Linux kernels.



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux