From: Tom Rini <trini@konsulko.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/1] test/py: provide example scripts for integrating qemu
Date: Mon, 18 Sep 2017 13:48:45 -0400 [thread overview]
Message-ID: <20170918174845.GF17137@bill-the-cat> (raw)
In-Reply-To: <328bd5f5-eb12-7532-dcf8-9b17724a8cdd@wwwdotorg.org>
On Mon, Sep 18, 2017 at 11:33:01AM -0600, Stephen Warren wrote:
> On 09/17/2017 01:32 PM, Heinrich Schuchardt wrote:
> >The necessary parameters for running Python tests on qemu are
> >tediouus to find.
> >
> >The patch adds examples for u-boot-test-console and
> >u-boot-test-reset.
>
> This README doesn't contain examples for other cases. I'm not sure
> whether we should add them for qemu either. Instead, does it make
> sense to augment my repo, which already contains a variety of
> examples, and is mentioned in the README?
>
> https://github.com/swarren/uboot-test-hooks
>
> Tom Rini already added some qemu examples to that repo, although
> they use much simpler command-lines than in the examples in this
> patch. Perhaps that's because they fire off a new qemu instance
> every time the virtual system is reset, whereas yours seem to create
> one qemu instance and then reset it multiple times. If so, your
> scripts are more how I imagined it would work; perhaps you could
> update Tom's scripts?
I was thinking about this too. Heinrich's example is along the lines of
"adapt QEMU to provide what test.py wants" while the uboot-test-hooks
QEMU parts are more along the lines of "plumb QEMU into the frameworks
for real HW too". What I like about the latter is that I leverage
information from my real HW too (tftp stuff, etc). Why I think we
should also apply Heinrich's patch, or maybe a v2 with a few wording
tweaks, is that it's less overhead and easier for people to setup if
they just want QEMU available for testing. Say testing while on the
road at some conference.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170918/206dbad9/attachment.sig>
next prev parent reply other threads:[~2017-09-18 17:48 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-17 19:32 [U-Boot] [PATCH 1/1] test/py: provide example scripts for integrating qemu Heinrich Schuchardt
2017-09-18 17:33 ` Stephen Warren
2017-09-18 17:48 ` Tom Rini [this message]
2017-09-18 18:13 ` Heinrich Schuchardt
2017-09-18 18:27 ` Stephen Warren
2017-09-18 19:55 ` Heinrich Schuchardt
2017-09-18 21:28 ` Stephen Warren
2017-09-18 21:37 ` Heinrich Schuchardt
2017-09-19 16:18 ` Stephen Warren
2017-09-19 20:15 ` Heinrich Schuchardt
2017-09-20 15:52 ` Stephen Warren
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=20170918174845.GF17137@bill-the-cat \
--to=trini@konsulko.com \
--cc=u-boot@lists.denx.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox