From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Paul E. McKenney" Subject: Re: [PATCH RFC LKMM 7/7] EXP tools/memory-model: Add .cfg and .cat files for s390 Date: Sat, 1 Sep 2018 10:08:54 -0700 Message-ID: <20180901170854.GM4225@linux.vnet.ibm.com> References: <20180829211018.GA19646@linux.vnet.ibm.com> <20180829211053.20531-7-paulmck@linux.vnet.ibm.com> <20180831160630.GF30626@arm.com> Reply-To: paulmck@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20180831160630.GF30626@arm.com> Sender: linux-kernel-owner@vger.kernel.org To: Will Deacon Cc: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, mingo@kernel.org, stern@rowland.harvard.edu, parri.andrea@gmail.com, peterz@infradead.org, boqun.feng@gmail.com, npiggin@gmail.com, dhowells@redhat.com, j.alglave@ucl.ac.uk, luc.maranget@inria.fr, akiyks@gmail.com List-Id: linux-arch.vger.kernel.org On Fri, Aug 31, 2018 at 05:06:30PM +0100, Will Deacon wrote: > Hi Paul, > > On Wed, Aug 29, 2018 at 02:10:53PM -0700, Paul E. McKenney wrote: > > This commit adds s390.cat and s390.cfg files to allow users to check > > litmus tests for s390-specific code. Note that this change only enables > > herd7 checking of C-language litmus tests. Larger changes are required > > to enable the litmus7 and klitmus7 tools to check litmus tests on real > > hardare. > > > > Suggested-by: Martin Schwidefsky > > Suggested-by: Christian Borntraeger > > Signed-off-by: Paul E. McKenney > > [ paulmck: Add fixes suggested by Alan Stern. ] > > --- > > tools/memory-model/s390.cat | 18 ++++++++++++++++++ > > tools/memory-model/s390.cfg | 21 +++++++++++++++++++++ > > 2 files changed, 39 insertions(+) > > create mode 100644 tools/memory-model/s390.cat > > create mode 100644 tools/memory-model/s390.cfg > > As I said before, I'd *much* prefer this to be part of the upstream > herdtools7 repository. It's not really anything to do with the Linux > kernel, so I don't think it belongs in the source tree. Agreed. As the cover letter says, "Add .cfg and .cat files for s390, which is a not-for-mainline placeholder." Thanx, Paul