Re: [PATCH RFC tools/lkmm 09/12] tools/memory-model: Add required herd7 version to README file

From: Alan Stern
Date: Wed Feb 21 2018 - 10:11:00 EST


On Tue, 20 Feb 2018, Paul E. McKenney wrote:

> LKMM and the herd7 tool are co-evolving, and out-of-date herd7 tools
> produce inaccurate results, often with no obvious error messages. This
> commit therefore adds the required herd7 version to the LKMM README file.
>
> Longer term, it would be good if .cat files could specify the required
> version in a manner allowing herd7 to produce clear diagnostics.
>
> Suggested-by: Akira Yokosawa <akiyks@xxxxxxxxx>
> Signed-off-by: Paul E. McKenney <paulmck@xxxxxxxxxxxxxxxxxx>
> ---
> tools/memory-model/README | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tools/memory-model/README b/tools/memory-model/README
> index 91414a49fac5..ea950c566ffd 100644
> --- a/tools/memory-model/README
> +++ b/tools/memory-model/README
> @@ -20,7 +20,8 @@ that litmus test to be exercised within the Linux kernel.
> REQUIREMENTS
> ============
>
> -The "herd7" and "klitmus7" tools must be downloaded separately:
> +Version 7.48 of the "herd7" and "klitmus7" tools must be downloaded
> +separately:
>
> https://github.com/herd/herdtools7

The text immediately below this mentions the Docker image and gentoo
package. Aren't they both seriously out of date at this point? In
which case, shouldn't we remove them from the README?

Alan