All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nikos Nikoleris <nikos.nikoleris@arm.com>
To: Andrew Jones <drjones@redhat.com>
Cc: kvm@vger.kernel.org, pbonzini@redhat.com,
	alexandru.elisei@arm.com, Jade Alglave <Jade.Alglave@arm.com>,
	maranget <luc.maranget@inria.fr>
Subject: Re: [kvm-unit-tests PATCH 0/3] Add support for external tests and litmus7 documentation
Date: Wed, 14 Apr 2021 09:50:23 +0100	[thread overview]
Message-ID: <40bfbd59-22c0-918b-d021-820933a15fe4@arm.com> (raw)
In-Reply-To: <20210414084216.khko7c7tk2tnu6bw@kamzik.brq.redhat.com>

On 14/04/2021 09:42, Andrew Jones wrote:
> On Tue, Apr 13, 2021 at 05:52:37PM +0100, Nikos Nikoleris wrote:
>> On 24/03/2021 17:13, Nikos Nikoleris wrote:
>>> This set of patches makes small changes to the build system to allow
>>> easy integration of tests not included in the repository. To this end,
>>> it adds a parameter to the configuration script `--ext-dir=DIR` which
>>> will instruct the build system to include the Makefile in
>>> DIR/Makefile. The external Makefile can then add extra tests,
>>> link object files and modify/extend flags.
>>>
>>> In addition, to demonstrate how we can use this functionality, a
>>> README file explains how to use litmus7 to generate the C code for
>>> litmus tests and link with kvm-unit-tests to produce flat files.
>>>
>>> Note that currently, litmus7 produces its own independent Makefile as
>>> an intermediate step. Once this set of changes is committed, litmus7
>>> will be modifed to make use hook to specify external tests and
>>> leverage the build system to build the external tests
>>> (https://github.com/relokin/herdtools7/commit/8f23eb39d25931c2c34f4effa096df58547a3bb4).
>>>
>>
>> Just wanted to add that if anyone's interested in trying out this series
>> with litmus7 I am very happy to help. Any feedback on this series or the way
>> we use kvm-unit-tests would be very welcome!
> 
> Hi Nikos,
> 
> It's on my TODO to play with this. I just haven't had a chance yet. I'm
> particularly slow right now because I'm in the process of handling a
> switch of my email server from one type to another, requiring rewrites
> of filters, new mail synchronization methods, and, in general, lots of
> pain... Hopefully by the end of this week all will be done. Then, I can
> start ignoring emails on purpose again, instead of due to the fact that
> I can't find them :-)
> 

No problem at all. If you run into problems with litmus7 please let me know!

Thanks,

Nikos

> Thanks,
> drew
> 
>>
>> Thanks,
>>
>> Nikos
>>
>>> Nikos Nikoleris (3):
>>>     arm/arm64: Avoid wildcard in the arm_clean recipe of the Makefile
>>>     arm/arm64: Add a way to specify an external directory with tests
>>>     README: Add a guide of how to run tests with litmus7
>>>
>>>    configure           |   7 +++
>>>    arm/Makefile.common |  11 +++-
>>>    README.litmus7.md   | 125 ++++++++++++++++++++++++++++++++++++++++++++
>>>    3 files changed, 141 insertions(+), 2 deletions(-)
>>>    create mode 100644 README.litmus7.md
>>>
>>
> 

  reply	other threads:[~2021-04-14  8:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-03-24 17:13 [kvm-unit-tests PATCH 0/3] Add support for external tests and litmus7 documentation Nikos Nikoleris
2021-03-24 17:14 ` [kvm-unit-tests PATCH 1/3] arm/arm64: Avoid wildcard in the arm_clean recipe of the Makefile Nikos Nikoleris
2021-03-24 17:14 ` [kvm-unit-tests PATCH 2/3] arm/arm64: Add a way to specify an external directory with tests Nikos Nikoleris
2021-03-24 17:14 ` [kvm-unit-tests PATCH 3/3] README: Add a guide of how to run tests with litmus7 Nikos Nikoleris
2021-03-24 18:27   ` Nikos Nikoleris
2021-04-13 16:52 ` [kvm-unit-tests PATCH 0/3] Add support for external tests and litmus7 documentation Nikos Nikoleris
2021-04-14  8:42   ` Andrew Jones
2021-04-14  8:50     ` Nikos Nikoleris [this message]
2021-06-29 13:32     ` Nikos Nikoleris
2021-06-29 16:13       ` Andrew Jones
2021-06-29 16:49         ` Nikos Nikoleris
2021-06-30 12:23           ` Andrew Jones
2021-06-30 14:19             ` Nikos Nikoleris
2021-06-30 16:03               ` Andrew Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=40bfbd59-22c0-918b-d021-820933a15fe4@arm.com \
    --to=nikos.nikoleris@arm.com \
    --cc=Jade.Alglave@arm.com \
    --cc=alexandru.elisei@arm.com \
    --cc=drjones@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=luc.maranget@inria.fr \
    --cc=pbonzini@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.