All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eduardo Habkost <ehabkost@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: Thomas Huth <thuth@redhat.com>,
	Samuel Ortiz <sameo@linux.intel.com>,
	"Zhong, Yang" <yang.zhong@intel.com>,
	Peter Maydell <peter.maydell@linaro.org>,
	QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] QEMU and Kconfig
Date: Thu, 8 Nov 2018 16:42:42 -0200	[thread overview]
Message-ID: <20181108184242.GQ12503@habkost.net> (raw)
In-Reply-To: <dda55f75-58d3-111c-4f43-99a9f686981e@redhat.com>

On Thu, Nov 08, 2018 at 06:58:11PM +0100, Paolo Bonzini wrote:
> On 08/11/2018 18:14, Eduardo Habkost wrote:
> > Keeping in mind that I might be talking about extra challenges we
> > won't address right now (no cart before the horse), I have new
> > questions:
> > 
> > Why you say backends are not a target configuration and
> > accelerators are?  What's the definition of "target
> > configuration"?
> 
> Something that affects the way

?

> 
> > Are we explicitly restricting the scope of this work to
> > enabling/disabling device emulation code right now?  Why?  Why
> > wouldn't we use kconfig to enable/disable simple backends with no
> > host dependency like SLIRP?
> 
> I think it would be more confusing if some backends were to use kconfig
> and some wouldn't.  We could certainly add something like
> 
> config VHOST_NET
>     depends on HOST_LINUX
>     default y
> 
> config SPICE
>     depends on HAVE_SPICE_SERVER
>     default Y
> 
> etc. but I think we agree it's more of a long term idea.

Agreed.

> 
> > Don't we want to make backends configurable per binary, too?
> > e.g.: I would expect the default configuration for a NEMU-like
> > binary to disable many backends.
> 
> Sure, we could do that.  However, right now you cannot have multiple
> binaries for a single target, so you couldn't have one single build
> include both a "full-blown" and a "reduced" x86 target.  Therefore,
> including e.g. SLIRP in qemu-system-arm but not in qemu-system-x86_64
> does not seem too interesting to me.  It would be different if you could
> build qemu-system-arm, qemu-system-x86_64, qemu-system-x86_64-lite, etc.

Understood.  I assumed this would be one of the short-term goals.
We can work on that later, then.


> 
> Paolo
> 
> > 
> >> It would surely be possible for configure to call into minikconf to
> >> parse a configuration file and apply dependencies (do we actually have
> >> dependencies across configure options?) or something like that, but
> >> let's not put the cart before the horse...
> > 
> > Agreed.
> > 
> 

-- 
Eduardo

  reply	other threads:[~2018-11-08 18:43 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20180924092120.GA30163@caravaggio>
     [not found] ` <7c2b477e-bdac-c887-2510-82fe04acdcfe@redhat.com>
     [not found]   ` <87b22690-e39c-3b46-dcb4-f6abc3213142@redhat.com>
     [not found]     ` <CAFEAcA8yHb4MdtsJuXcswQp93DN5tSeN9dn8AWWL9se0DKvdcg@mail.gmail.com>
     [not found]       ` <41ceda53-467e-32a1-8fa6-13f0f9c08ad1@redhat.com>
     [not found]         ` <CAFEAcA9oCs+Y1fnY8dOR16n_Hn7PaHziL7K0HZK6=BAd_Gyecg@mail.gmail.com>
     [not found]           ` <e496287e-f8ba-3608-42e1-efb697cfc784@redhat.com>
2018-11-07 15:41             ` [Qemu-devel] QEMU and Kconfig Samuel Ortiz
2018-11-07 17:39               ` Paolo Bonzini
2018-11-07 19:24                 ` Eduardo Habkost
2018-11-07 19:30                   ` Thomas Huth
2018-11-08  9:55                     ` Paolo Bonzini
2018-11-08 10:14                       ` Thomas Huth
2018-11-08 10:53                         ` Philippe Mathieu-Daudé
2018-11-08 12:02                         ` Paolo Bonzini
2018-11-08 13:06                       ` Eduardo Habkost
2018-11-08 13:42                         ` Paolo Bonzini
2018-11-08 17:14                           ` Eduardo Habkost
2018-11-08 17:58                             ` Paolo Bonzini
2018-11-08 18:42                               ` Eduardo Habkost [this message]
2018-11-08 20:28                                 ` Paolo Bonzini
2018-11-08 21:00                                   ` Eduardo Habkost
2018-11-09 10:10                                     ` Paolo Bonzini
2018-11-09 19:16                                       ` Eduardo Habkost
2018-11-14 11:50                                         ` Paolo Bonzini
2018-11-08 11:06                   ` Samuel Ortiz
2018-12-13 11:50                     ` Yang Zhong
2018-12-13 13:27                       ` Paolo Bonzini
     [not found]         ` <20180926141518.GB9073@caravaggio>
     [not found]           ` <3743752b-2670-1d89-2088-1e67122f5dcd@redhat.com>
2018-11-08  8:46             ` Philippe Mathieu-Daudé
2018-11-08  9:56               ` Paolo Bonzini
2018-11-08 12:46                 ` Markus Armbruster

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=20181108184242.GQ12503@habkost.net \
    --to=ehabkost@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=sameo@linux.intel.com \
    --cc=thuth@redhat.com \
    --cc=yang.zhong@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 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.