qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: "Ákos Kovács" <akoskovacs@gmx.com>,
	"QEMU Developers" <qemu-devel@nongnu.org>,
	"Richard Henderson" <rth@twiddle.net>
Subject: Re: [Qemu-devel] [PATCH 06/47] hw/alpha/Makefile.objs: Build objects depending on CLIPPER
Date: Tue, 27 Aug 2013 08:59:41 +0200	[thread overview]
Message-ID: <521C4E5D.3010103@redhat.com> (raw)
In-Reply-To: <CAFEAcA94icWtS4tT9cRMpNWoW3jh8fZzyRNKU0QFqvo2SPSDzw@mail.gmail.com>

Il 27/08/2013 01:17, Peter Maydell ha scritto:
> On 26 August 2013 23:33, Paolo Bonzini <pbonzini@redhat.com> wrote:
>> Il 26/08/2013 19:30, Richard Henderson ha scritto:
>>> This isn't the kernel, where non-pagable code size is a concern, so I don't see
>>> how full configuration of machine emulations and devices is helpful.  I'd be
>>> more inclined to go the other way, where all qemu-system-cpu images always
>>> build in all devices (compiled once of course).
>>
>> This is useful for different usecases.  One is QEMU that is bundled into
>> development platform such as the Android emulator.  Making it easier to
>> build limited versions of QEMU is one small step towards encouraging
>> working in-tree instead of having out-of-tree patches which quickly
>> become forks.
> 
> I simply don't believe that this is anything remotely approaching a
> major reason why the Android emulator is out of tree, or that merging
> this patchset would have any visible effect in moving the Android emulator
> into the tree. Anybody from Google is of course welcome to contradict me
> on this point.

No, it's not anything remotely approaching a major reason.  But still,
modifying default-configs/ is one of the out-of-tree patches that any
external emulator has to include.

>> The second is in distros that only want to distribute the subset of
>> features that are going to be supported (aka RHEL).  This includes both
>> devices (all of PCI, ISA, USB) and boards (-M isapc is removed nowadays,
>> perhaps one day goldfish or similar will be available too; for ARM and
>> PPC we surely would want to compile out almost all the boards).
> 
> Hrm. Yeah, I can see the security argument for wanting a very
> stripped down build for the KVM/production use.
> 
>> The third is that in the future some of the devices could be compiled as
>> modules, too.  This would help the "other" set of distros, those that
>> include everything.  QEMU now has an insane set of dependencies, and
>> having modules for e.g. SPICE or RBD or Gluster would help making them
>> optional.
> 
> I thought the plan was to address that by having a module system,
> not by having a huge config system. You still build everything all
> at once, you just split the binaries/shared objects you ship as a
> distro into multiple packages.

Sure.  But the above stripped-down build might just as well be
monolithic, so you need to configure what is a module and what is not.

>> Note that the Kconfig project is about giving end users _less_ config
>> options.
> 
> From my perspective it seems to be giving users more options,
> because at the moment there are none -- you just compile QEMU
> and you get everything. Nobody should (IMHO) be editing
> default-configs/ (despite the slightly confusing name).

At least RHEL is doing so (and RHEL originally motivated the first big
Makefile reorganization by Juan around 4 years ago, and the introduction
of default-configs).  Even though this is a summer of code project, my
proposal was driven by an actual need (and curiosity of course).

Paolo

  reply	other threads:[~2013-08-27  7:00 UTC|newest]

Thread overview: 91+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-25 22:58 [Qemu-devel] [RFC PATCH 00/47] Describing patchset Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 01/47] rules.mak: New logical functions Ákos Kovács
2013-09-13 13:43   ` Peter Maydell
2013-09-13 14:55     ` Paolo Bonzini
2013-09-13 15:02       ` Peter Maydell
2013-09-13 15:12         ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 02/47] Makefile.target: CONFIG_NO_* variables removed Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 03/47] default-configs/: CONFIG_GDBSTUB_XML removed Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 04/47] scripts/kconfig: kconfig-frontends submodule added Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 05/47] Makefile: Clone kconfig git submodule in Makefile Ákos Kovács
2013-08-26  2:33   ` Andreas Färber
2013-08-25 22:58 ` [Qemu-devel] [PATCH 06/47] hw/alpha/Makefile.objs: Build objects depending on CLIPPER Ákos Kovács
2013-08-26 14:41   ` Richard Henderson
2013-08-26 16:59     ` Paolo Bonzini
2013-08-26 17:30       ` Richard Henderson
2013-08-26 18:44         ` Lluís Vilanova
2013-08-26 19:47           ` Peter Maydell
2013-08-26 22:44             ` Paolo Bonzini
2013-08-26 22:33         ` Paolo Bonzini
2013-08-26 22:49           ` Richard Henderson
2013-08-26 23:17           ` Peter Maydell
2013-08-27  6:59             ` Paolo Bonzini [this message]
2013-08-25 22:58 ` [Qemu-devel] [PATCH 07/47] hw/arm/Makefile.objs: CONFIG_* created for each board Ákos Kovács
2013-08-25 23:57   ` Max Filippov
2013-08-26 10:42   ` Peter Maydell
2013-08-25 22:58 ` [Qemu-devel] [PATCH 09/47] hw/m68k/Makefile.objs: Conditionally build boards Ákos Kovács
2013-08-26  0:09   ` Max Filippov
2013-08-25 22:58 ` [Qemu-devel] [PATCH 10/47] hw/microblaze/Makefile.objs: Create configs for petalogix boards Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 11/47] hw/mips/Makefile.objs: Create CONFIG_* for mips boards Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 12/47] hw/ppc/Makefile.objs: Build all boards conditinally Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 13/47] hw/sh4/Makefile.objs: Build sh4 boards conditionally Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 14/47] hw/sparc/Makefile.objs: CONFIG_* for sun4m and leon3 created Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 15/47] hw/lm32/Makefile.objs: Conditionally build lm32 and milkmyst Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 16/47] hw/xtensa/Makefile.objs: Build xtensa_sim and xtensa_lx60 conditionally Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 17/47] hw/9pfs/Kconfig: Add 9pfs Kconfig Ákos Kovács
2013-08-26 10:39   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 18/47] hw/arm/Kconfig: Add ARM Kconfig Ákos Kovács
2013-08-26 10:38   ` Peter Maydell
2013-08-26 11:09   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 19/47] hw/audio/Kconfig: Add audio Kconfig Ákos Kovács
2013-08-26 10:41   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 20/47] hw/block/Kconfig: Add Kconfig file Ákos Kovács
2013-08-26 10:43   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 21/47] hw/char/Kconfig: " Ákos Kovács
2013-08-26 10:43   ` Paolo Bonzini
2013-08-26 17:15     ` Andreas Färber
2013-08-26 22:40       ` Paolo Bonzini
2013-09-13 14:00         ` Andreas Färber
2013-09-13 14:49           ` Paolo Bonzini
2013-09-15 10:43             ` Alberto Garcia
2013-08-25 22:58 ` [Qemu-devel] [PATCH 22/47] hw/core/Kconfig: " Ákos Kovács
2013-08-26 10:45   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 23/47] hw/cpu/Kconfig: " Ákos Kovács
2013-08-26 17:03   ` Andreas Färber
2013-08-25 22:58 ` [Qemu-devel] [PATCH 24/47] hw/display/Kconfig: " Ákos Kovács
2013-08-26 10:49   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 25/47] hw/dma/Kconfig: " Ákos Kovács
2013-08-26 10:49   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 26/47] hw/gpio/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 27/47] hw/i2c/Kconfig: " Ákos Kovács
2013-08-26 10:50   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 28/47] hw/ide/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 29/47] hw/input/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 30/47] hw/intc/Kconfig: " Ákos Kovács
2013-08-26 10:53   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 31/47] hw/isa/Kconfig: " Ákos Kovács
2013-08-26 11:03   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 32/47] hw/misc/Kconfig: " Ákos Kovács
2013-08-26 11:00   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 33/47] hw/net/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 34/47] hw/nvram/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 35/47] hw/pci/Kconfig: " Ákos Kovács
2013-08-26 11:01   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 36/47] hw/pci-bridge/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 37/47] hw/pci-host/Kconfig: " Ákos Kovács
2013-08-26 11:02   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 38/47] hw/scsi/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 39/47] hw/sd/Kconfig: " Ákos Kovács
2013-08-26 11:05   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 40/47] hw/ssi/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 41/47] hw/timer/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 42/47] hw/tpm/Kconfig: " Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 43/47] hw/usb/Kconfig: " Ákos Kovács
2013-08-26 11:06   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 44/47] hw/watchdog/Kconfig: " Ákos Kovács
2013-08-26 11:06   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 45/47] hw/Kconfig: Add the main Kconfig for hw/ Ákos Kovács
2013-08-25 22:58 ` [Qemu-devel] [PATCH 46/47] configure: Generate Kconfig.targets with --target-list Ákos Kovács
2013-08-26 11:14   ` Paolo Bonzini
2013-08-25 22:58 ` [Qemu-devel] [PATCH 47/47] Kconfig: Main kconfig file added Ákos Kovács
2013-08-26  7:35 ` [Qemu-devel] [RFC PATCH 00/47] Describing patchset Peter Maydell

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=521C4E5D.3010103@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=akoskovacs@gmx.com \
    --cc=peter.maydell@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=rth@twiddle.net \
    /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;
as well as URLs for NNTP newsgroup(s).