Re: [Qemu-devel] [PATCH qemu v2 0/6] -no-user-config option, move CPU models to /usr/share

Re: [Qemu-devel] [PATCH qemu v2 0/6] -no-user-config option, move CPU models to /usr/share

[Eduardo Habkost]

Anthony, isn't this going to get in for 1.1? I was expecting it to be
applied before the freeze.


On Tue, Apr 24, 2012 at 05:32:55PM -0300, Eduardo Habkost wrote:
> Changes v1 -> v2:
>  - Move qemu_read_default_config_files() prototype to qemu-config.h
>  - Make defconfig and userconfig variable bool
>  - Coding style change
> 
> Patches 1 to 4 just move some code around, patch 5 just adds the new option
> without adding any new config file. Patch 6 finally creates a /usr/share/qemu
> /cpus-x86_64.conf file, with the CPU models we currently have on Qemu.
> 
> Reference to previous discussion:
>  - http://marc.info/?l=qemu-devel&m=133278877315665
> 
> 
> Eduardo Habkost (6):
>   move code to read default config files to a separate function (v2)
>   eliminate arch_config_name variable
>   move list of default config files to an array
>   vl.c: change 'defconfig' variable to bool
>   implement -no-user-config command-line option (v2)
>   move CPU definitions to /usr/share/qemu/cpus-x86_64.conf (v2)
> 
>  Makefile                             |   12 +++-
>  arch_init.c                          |   32 ++++++++-
>  arch_init.h                          |    2 -
>  qemu-config.h                        |    4 +
>  qemu-options.hx                      |   16 ++++-
>  sysconfigs/target/cpus-x86_64.conf   |  128 ++++++++++++++++++++++++++++++++++
>  sysconfigs/target/target-x86_64.conf |  128 ----------------------------------
>  vl.c                                 |   18 ++---
>  8 files changed, 193 insertions(+), 147 deletions(-)
>  create mode 100644 sysconfigs/target/cpus-x86_64.conf
> 
> -- 
> 1.7.3.2
> 

-- 
Eduardo

Re: [Qemu-devel] [PATCH qemu v2 0/6] -no-user-config option, move CPU models to /usr/share

[Andreas Färber]

Am 02.05.2012 15:50, schrieb Eduardo Habkost:
> 
> Anthony, isn't this going to get in for 1.1? I was expecting it to be
> applied before the freeze.

You wrote you would respin it with s/int/bool/. :)

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

Re: [Qemu-devel] [PATCH qemu v2 0/6] -no-user-config option, move CPU models to /usr/share

[Eduardo Habkost]

On Wed, May 02, 2012 at 04:01:55PM +0200, Andreas Färber wrote:
> Am 02.05.2012 15:50, schrieb Eduardo Habkost:
> > 
> > Anthony, isn't this going to get in for 1.1? I was expecting it to be
> > applied before the freeze.
> 
> You wrote you would respin it with s/int/bool/. :)

I sent an additional patch instead of respinning (see patch 7/6 sent as
reply to this series).

-- 
Eduardo