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

2012-05-14 Thread Anthony Liguori

On 05/02/2012 11:07 AM, Eduardo Habkost wrote:

Changes v2 -  v3:
  - Actually change 'defconfig' type declaration to bool
  - Rebase against latest qemu.git (commit 
563987d0a799f90b58a575b190a57546c335191b)

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-develm=133278877315665


Applied (as discussed prior to -rc0).  Thanks.

Regards,

Anthony Liguori




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 (v2)
   implement -no-user-config command-line option (v3)
   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






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

2012-05-11 Thread Eduardo Habkost

I just noticed it didn't get into -rc1 either. So, this is definitely
not going to be in qemu-1.1, I guess?


On Wed, May 02, 2012 at 01:07:24PM -0300, Eduardo Habkost wrote:
 Changes v2 - v3:
  - Actually change 'defconfig' type declaration to bool
  - Rebase against latest qemu.git (commit 
 563987d0a799f90b58a575b190a57546c335191b)
 
 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-develm=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 (v2)
   implement -no-user-config command-line option (v3)
   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 v3 0/6] -no-user-config option, move CPU models to /usr/share

2012-05-11 Thread Anthony Liguori

On 05/11/2012 08:34 AM, Eduardo Habkost wrote:


I just noticed it didn't get into -rc1 either. So, this is definitely
not going to be in qemu-1.1, I guess?


I've got this applied and am testing it right now for -rc2.

Regards,

Anthony Liguori




On Wed, May 02, 2012 at 01:07:24PM -0300, Eduardo Habkost wrote:

Changes v2 -  v3:
  - Actually change 'defconfig' type declaration to bool
  - Rebase against latest qemu.git (commit 
563987d0a799f90b58a575b190a57546c335191b)

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-develm=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 (v2)
   implement -no-user-config command-line option (v3)
   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








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

2012-05-11 Thread Eduardo Habkost
On Fri, May 11, 2012 at 08:37:33AM -0500, Anthony Liguori wrote:
 On 05/11/2012 08:34 AM, Eduardo Habkost wrote:
 
 I just noticed it didn't get into -rc1 either. So, this is definitely
 not going to be in qemu-1.1, I guess?
 
 I've got this applied and am testing it right now for -rc2.

Awesome. Thanks!  :-)

-- 
Eduardo

 
 Regards,
 
 Anthony Liguori
 
 
 
 On Wed, May 02, 2012 at 01:07:24PM -0300, Eduardo Habkost wrote:
 Changes v2 -  v3:
   - Actually change 'defconfig' type declaration to bool
   - Rebase against latest qemu.git (commit 
  563987d0a799f90b58a575b190a57546c335191b)
 
 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-develm=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 (v2)
implement -no-user-config command-line option (v3)
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
 
 
 



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

2012-05-11 Thread Jiri Denemark
On Fri, May 11, 2012 at 08:37:33 -0500, Anthony Liguori wrote:
 On 05/11/2012 08:34 AM, Eduardo Habkost wrote:
 
  I just noticed it didn't get into -rc1 either. So, this is definitely
  not going to be in qemu-1.1, I guess?
 
 I've got this applied and am testing it right now for -rc2.

Oh, having this in rc2 would be awesome. I already tested this patch set (at
the time it was submitted) with my patches for libvirt which make use of it
and it all worked very nicely and I was finally able to use all the new fancy
CPUs :-) If it helps, you can count this as

Tested-by: Jiri Denemark jdene...@redhat.com

Jirka