Re: Bug in herd7 [Was: Re: Litmus test for question from Al Viro]
From: Alan Stern
Date: Wed Oct 07 2020 - 22:25:39 EST
On Wed, Oct 07, 2020 at 03:38:51PM -0700, Paul E. McKenney wrote:
> On Wed, Oct 07, 2020 at 03:40:50PM -0400, Alan Stern wrote:
> > On Wed, Oct 07, 2020 at 10:50:40AM -0700, Paul E. McKenney wrote:
> > > And here is the updated version.
> > >
> > > Thanx, Paul
> > >
> > > ------------------------------------------------------------------------
> > >
> > > commit b7cd60d4b41ad56b32b36b978488f509c4f7e228
> > > Author: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
> > > Date: Tue Oct 6 09:38:37 2020 -0700
> > >
> > > manual/kernel: Add LB+mb+data litmus test
> >
> > Let's change this to:
> >
> > manual/kernel: Add LB data dependency test with no intermediate variable
> >
> > Without that extra qualification, people reading just the title would
> > wonder why we need a simple LB litmus test in the archive.
> I might get this right sooner or later. You never know.
>
> Like this?
>
> Thanx, Paul
Paul, I think you must need new reading glasses. You completely missed
the text above.
Alan
> ------------------------------------------------------------------------
>
> commit 5b6a4ff2c8ad25fc77f4151e71e6cbd8f3268d7b
> Author: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
> Date: Tue Oct 6 09:38:37 2020 -0700
>
> manual/kernel: Add LB+mb+data litmus test
>
> Test whether herd7 can detect a data dependency when there is no
> intermediate local variable, as in WRITE_ONCE(*x, READ_ONCE(*y)).
> Commit 0f3f8188a326 in herdtools fixed an oversight which caused such
> dependencies to be missed.
>
> Signed-off-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Paul E. McKenney <paulmck@xxxxxxxxxx>