Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Sipke Vriend <sipke.vriend@xilinx.com>,
	"Stanacar, StefanX" <stefanx.stanacar@intel.com>
Cc: "paul.eggleton@linux.intel.com" <paul.eggleton@linux.intel.com>,
	"openembedded-core@lists.openembedded.org"
	<openembedded-core@lists.openembedded.org>
Subject: Re: [RFC OE-core/meta/lib] BSP Specific Qemurunner
Date: Wed, 08 Jan 2014 13:12:41 +0000	[thread overview]
Message-ID: <1389186761.6899.99.camel@ted> (raw)
In-Reply-To: <00beaba3-2d15-4d36-936c-88fd8734ed8a@CO1EHSMHS012.ehs.local>

On Tue, 2014-01-07 at 22:59 +0000, Sipke Vriend wrote:
> Hi Richard,
> 
> >
> >-----Original Message-----
> >On Wednesday, 8 January 2014 12:00 AM, Richard Purdie wrote:
> >
> >On Tue, 2014-01-07 at 03:09 +0000, Sipke Vriend wrote:
> >> Hi,
> >> 
> >> This RFC is a proposal to allow BSP layers to setup qemu with their specific
> >> requirements for the testimage oe-core functionality.
> >> The suggested changes will be exercised by the 
> >> bitbake -c testimage <image> 
> >> command.
> >> Similarly to the oeqa test cases this proposal extends the meta/lib/oeqa 
> >> python modules to allow inclusion of python utility scripts in the BSP 
> >> layers.
> >> Any BSP layer wishing to supply their own qemu setup would need to create
> >> an appropriate meta-bsplayer/lib/oeqa/utils/<machine>starter.py
> >> The effect is that the lib/oeqa/utils/qemurunner will either allow the 
> >> bsp layer provided <machine>starter to spawn qemu or if not provided, 
> >> spawn qemu via runqemu as currently.
> >> An example bsp layer is available here:
> >> https://github.com/sipke/meta-xilinx/tree/sipke/qemurunner
> >> with all required additions in the meta-xilinx/lib directory.
> >> 
> >> This RFC is triggered by and indirectly related to
> >> Bugzilla report "runqemu shouldn't hard-code machine knowledge"
> >> https://bugzilla.yoctoproject.org/show_bug.cgi?id=4827
> >
> >Why would we do this rather than improve runqemu to be extendable from
> >BSP layers?
> >
> 
> Proposing as an additional way to launch qemu for oeqa testimage functionality,
> Improving runqemu can and probably should still happen.
>  
> To consider:
> * it keeps testimage functionality (for bsp layers specific things) in 
> the lib directly (similar to test cases) and as python.
> * testing (via testimage) may have a different requirement to that of
> running runqemu on the command line, so an alternate way to launch qemu 
> could be useful.
> * should this approach of extending the oeqa testimage functionality 
> into bsp layers be accepted, this could allow also for bsp specific 
> hardware setup for testimage functionality in bsp layers.
> 
> Primary aim is a solution which allows the bsp layer to control the 
> setup of qemu (and eventually hardware) for testimage functionality. This is 
> a proposal towards that goal.

I thought Stefan was already also working on something towards this
goal. I'd like to ensure we don't end up with two things doing the same
thing.

Stefan?

To be clear, I would like to see runqemu enhanced so BSP layers can
extend it, I think that would be useful for everyone. Once we've done
that, I'd like to revisit the qemu abstraction in testimage and figure
out what changes it needs. Some may be required, some may not if we fix
runqemu first, I'm unclear from these commits what those would be
though.

Cheers,

Richard



  reply	other threads:[~2014-01-08 13:13 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-07  3:09 [RFC OE-core/meta/lib] BSP Specific Qemurunner Sipke Vriend
2014-01-07 13:59 ` Richard Purdie
2014-01-07 22:59   ` Sipke Vriend
2014-01-08 13:12     ` Richard Purdie [this message]
2014-01-08 13:52       ` Paul Eggleton
2014-01-09  0:01         ` Sipke Vriend
2014-01-09 12:19           ` Stanacar, StefanX
2014-01-09 23:29             ` Sipke Vriend

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=1389186761.6899.99.camel@ted \
    --to=richard.purdie@linuxfoundation.org \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=paul.eggleton@linux.intel.com \
    --cc=sipke.vriend@xilinx.com \
    --cc=stefanx.stanacar@intel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox