Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Dor Laor
On 01/06/2010 05:16 PM, Anthony Liguori wrote: On 01/06/2010 08:48 AM, Dor Laor wrote: On 01/06/2010 04:32 PM, Avi Kivity wrote: On 01/06/2010 04:22 PM, Michael S. Tsirkin wrote: We can probably default -enable-kvm to -cpu host, as long as we explain very carefully that if users wish to preser

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Avi Kivity
On 01/07/2010 10:03 AM, Dor Laor wrote: We can debate about the exact name/model to represent the Nehalem family, I don't have an issue with that and actually Intel and Amd should define it. AMD and Intel already defined their names (in cat /proc/cpuinfo). They don't define families, the w

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Daniel P. Berrange
On Thu, Jan 07, 2010 at 10:03:28AM +0200, Dor Laor wrote: > On 01/06/2010 05:16 PM, Anthony Liguori wrote: > >On 01/06/2010 08:48 AM, Dor Laor wrote: > >>On 01/06/2010 04:32 PM, Avi Kivity wrote: > >>>On 01/06/2010 04:22 PM, Michael S. Tsirkin wrote: > >We can probably default -enable-kvm to -c

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Dor Laor
On 01/07/2010 10:18 AM, Avi Kivity wrote: On 01/07/2010 10:03 AM, Dor Laor wrote: We can debate about the exact name/model to represent the Nehalem family, I don't have an issue with that and actually Intel and Amd should define it. AMD and Intel already defined their names (in cat /proc/cpui

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Dor Laor
On 01/07/2010 10:24 AM, Daniel P. Berrange wrote: On Thu, Jan 07, 2010 at 10:03:28AM +0200, Dor Laor wrote: On 01/06/2010 05:16 PM, Anthony Liguori wrote: On 01/06/2010 08:48 AM, Dor Laor wrote: On 01/06/2010 04:32 PM, Avi Kivity wrote: On 01/06/2010 04:22 PM, Michael S. Tsirkin wrote: We ca

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Avi Kivity
On 01/07/2010 11:11 AM, Dor Laor wrote: On 01/07/2010 10:18 AM, Avi Kivity wrote: On 01/07/2010 10:03 AM, Dor Laor wrote: We can debate about the exact name/model to represent the Nehalem family, I don't have an issue with that and actually Intel and Amd should define it. AMD and Intel alrea

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Dor Laor
On 01/07/2010 11:24 AM, Avi Kivity wrote: On 01/07/2010 11:11 AM, Dor Laor wrote: On 01/07/2010 10:18 AM, Avi Kivity wrote: On 01/07/2010 10:03 AM, Dor Laor wrote: We can debate about the exact name/model to represent the Nehalem family, I don't have an issue with that and actually Intel and

Re: [Qemu-devel] [PATCH 06/17] eepro100: symbolic names for pci registers

2010-01-07 Thread Stefan Weil
Michael S. Tsirkin schrieb: > No functional changes. I verified that the generated binary > does not change in meaningful ways. Survived light usage > with linux guest. > > Signed-off-by: Michael S. Tsirkin > --- > hw/eepro100.c | 49 - > 1 files chan

[Qemu-devel] [PATCH] eepro100: Fix initial value for PCI_STATUS

2010-01-07 Thread Stefan Weil
The numerical value was wrong (0x2800 instead of 0x0280) which indeed did not make sense. Signed-off-by: Stefan Weil --- hw/eepro100.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/hw/eepro100.c b/hw/eepro100.c index 336ca49..a21c984 100644 --- a/hw/eepro100.c +++ b

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Anthony Liguori
On 01/07/2010 03:40 AM, Dor Laor wrote: There's no simple solution except to restrict features to what was available on the first processors. What's not simple about the above 4 options? What's a better alternative (that insures users understand it and use it and guest msi and even skype appli

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Dor Laor
On 01/07/2010 01:39 PM, Anthony Liguori wrote: On 01/07/2010 03:40 AM, Dor Laor wrote: There's no simple solution except to restrict features to what was available on the first processors. What's not simple about the above 4 options? What's a better alternative (that insures users understand i

Re: [Qemu-devel] [PATCH 3/6] target-alpha: Reduce internal processor registers for user-mode.

2010-01-07 Thread Tristan Gingold
On Jan 6, 2010, at 6:04 PM, Andreas Färber wrote: > > Am 06.01.2010 um 17:29 schrieb Richard Henderson: > >> since (1) ROMs other than the few supported by MILO are probably not >> redistributable > > Tristan's trick here was to provide a way for the user to extract a > non-distributable ROM

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Richard W.M. Jones
On Sun, Dec 13, 2009 at 08:24:54AM +, Blue Swirl wrote: > I guess e1c09175bc00dd8dfb2ad1b26e1858dcdc109b59 or > 998bbd74b9d813b14a3a3b5009a5d5a48c7dce51 broke -serial stdio for all > targets: > qemu -serial stdio -monitor stdio > chardev: opening backend "stdio" failed > qemu: could not open se

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Avi Kivity
On 01/07/2010 11:40 AM, Dor Laor wrote: There's no such thing as Nehalem. Intel were ok with it. Again, you can name is corei7 or xeon34234234234, I don't care, the principle remains the same. There are several processors belonging to the Nehalem family and each have different features.

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Avi Kivity
On 01/07/2010 01:44 PM, Dor Laor wrote: So if you had a 2.6.18 kernel and a 2.6.33 kernel, it may be necessary to say: (2.6.33) qemu -cpu Nehalem,-syscall (2.6.18) qemu -cpu Nehalem Or let qemu do it automatically for you. qemu on 2.6.33 doesn't know that you're running qemu on 2.6.18 on a

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Richard W.M. Jones
On Thu, Jan 07, 2010 at 11:55:05AM +, Richard W.M. Jones wrote: > chardev: opening backend "stdio" failed > qemu: could not open serial device 'stdio': Invalid argument Or sometimes: chardev: opening backend "stdio" failed qemu: could not open serial device 'stdio': Success (!) So what

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Dor Laor
On 01/07/2010 01:59 PM, Avi Kivity wrote: On 01/07/2010 11:40 AM, Dor Laor wrote: There's no such thing as Nehalem. Intel were ok with it. Again, you can name is corei7 or xeon34234234234, I don't care, the principle remains the same. There are several processors belonging to the Nehalem f

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Dor Laor
On 01/07/2010 02:00 PM, Avi Kivity wrote: On 01/07/2010 01:44 PM, Dor Laor wrote: So if you had a 2.6.18 kernel and a 2.6.33 kernel, it may be necessary to say: (2.6.33) qemu -cpu Nehalem,-syscall (2.6.18) qemu -cpu Nehalem Or let qemu do it automatically for you. qemu on 2.6.33 doesn't kn

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Gerd Hoffmann
Hi, So what seems to be happening here is there is an implicit monitor being set up which grabs stdio. Because: #define STDIO_MAX_CLIENTS 1 my own -serial stdio option subsequently fails. This is a regression over previous behaviour. I didn't specify a monitor device, because I don't w

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Anthony Liguori
On 01/07/2010 06:20 AM, Dor Laor wrote: On 01/07/2010 02:00 PM, Avi Kivity wrote: On 01/07/2010 01:44 PM, Dor Laor wrote: So if you had a 2.6.18 kernel and a 2.6.33 kernel, it may be necessary to say: (2.6.33) qemu -cpu Nehalem,-syscall (2.6.18) qemu -cpu Nehalem Or let qemu do it automatic

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Anthony Liguori
On 01/07/2010 05:55 AM, Richard W.M. Jones wrote: On Sun, Dec 13, 2009 at 08:24:54AM +, Blue Swirl wrote: I guess e1c09175bc00dd8dfb2ad1b26e1858dcdc109b59 or 998bbd74b9d813b14a3a3b5009a5d5a48c7dce51 broke -serial stdio for all targets: qemu -serial stdio -monitor stdio chardev: opening b

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Anthony Liguori
On 01/07/2010 06:10 AM, Richard W.M. Jones wrote: On Thu, Jan 07, 2010 at 11:55:05AM +, Richard W.M. Jones wrote: chardev: opening backend "stdio" failed qemu: could not open serial device 'stdio': Invalid argument Or sometimes: chardev: opening backend "stdio" failed qemu:

[Qemu-devel] Re: [PATCH] eepro100: Fix initial value for PCI_STATUS

2010-01-07 Thread Michael S. Tsirkin
On Thu, Jan 07, 2010 at 12:15:25PM +0100, Stefan Weil wrote: > The numerical value was wrong (0x2800 instead of 0x0280) > which indeed did not make sense. Aha! conversion to symbolic names paying off. > Signed-off-by: Stefan Weil Applied, thanks. > --- > hw/eepro100.c |4 +--- > 1 files c

[Qemu-devel] Qemu's internal TFTP server breaks lock-step-iness of TFTP

2010-01-07 Thread Milan Plzik
According to RFC 1350 and RFC 2347, TFTP server should answer RRQ by either OACK or DATA packet. Qemu's internal TFTP server answers RRQ with additional options by sending both OACK and DATA packet, thus breaking the "lock-step" feature of the protocol, and also confuses client. Proposed solut

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Avi Kivity
On 01/07/2010 02:33 PM, Anthony Liguori wrote: There's another option. Make cpuid information part of live migration protocol, and then support something like -cpu Xeon-3550. We would remember the exact cpuid mask we present to the guest and then we could validate that we can obtain the sam

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Daniel P. Berrange
On Thu, Jan 07, 2010 at 06:34:13AM -0600, Anthony Liguori wrote: > On 01/07/2010 05:55 AM, Richard W.M. Jones wrote: > >On Sun, Dec 13, 2009 at 08:24:54AM +, Blue Swirl wrote: > > > >>I guess e1c09175bc00dd8dfb2ad1b26e1858dcdc109b59 or > >>998bbd74b9d813b14a3a3b5009a5d5a48c7dce51 broke -seri

[Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Gleb Natapov
Initialize KVM paravirt cpuid leaf and allow user to control guest visible PV features through -cpu flag. Signed-off-by: Gleb Natapov diff --git a/target-i386/cpu.h b/target-i386/cpu.h index f3834b3..216b00e 100644 --- a/target-i386/cpu.h +++ b/target-i386/cpu.h @@ -701,7 +701,8 @@ typedef struct

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Daniel P. Berrange
On Thu, Jan 07, 2010 at 02:40:34PM +0200, Avi Kivity wrote: > On 01/07/2010 02:33 PM, Anthony Liguori wrote: > > > >There's another option. > > > >Make cpuid information part of live migration protocol, and then > >support something like -cpu Xeon-3550. We would remember the exact > >cpuid mask

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Avi Kivity
On 01/07/2010 02:47 PM, Daniel P. Berrange wrote: With the introduction of the new -device spport, there's no need to replay hotplug events in order any more. Instead just use static PCI addresses when starting the guest, and the same addresses after migration. You could argue that QEMU should p

Re: [Qemu-devel] Qemu's internal TFTP server breaks lock-step-iness of TFTP

2010-01-07 Thread Milan Plzik
V Streda, 6. január 2010 o 16:43 -0600, Anthony Liguori napísal(a): > Thanks for the patch, I assume this fixes -boot n when using -bootp and > -tftp. Not sure what was the original issue, but at least for me it fixed booting pxegrub :) > Can you add an appropriate Signed-off-by and resubmit

Re: [Qemu-devel] [PATCH 06/17] eepro100: symbolic names for pci registers

2010-01-07 Thread Michael S. Tsirkin
On Thu, Jan 07, 2010 at 12:14:17PM +0100, Stefan Weil wrote: > Michael S. Tsirkin schrieb: > > No functional changes. I verified that the generated binary > > does not change in meaningful ways. Survived light usage > > with linux guest. > > > > Signed-off-by: Michael S. Tsirkin > > --- > > hw/eep

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Anthony Liguori
On 01/07/2010 06:40 AM, Avi Kivity wrote: On 01/07/2010 02:33 PM, Anthony Liguori wrote: There's another option. Make cpuid information part of live migration protocol, and then support something like -cpu Xeon-3550. We would remember the exact cpuid mask we present to the guest and then we

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Anthony Liguori
On 01/07/2010 06:44 AM, Daniel P. Berrange wrote: NB, QEMU 0.12 introduces a new flag '-nodefaults' that can be used to get rid of this imlied 'serial stdio', and all other implied devices. It is well worth using this new -nodefaults flag if you're managing qemu from an app to avoid these surpris

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Richard W.M. Jones
On Thu, Jan 07, 2010 at 01:32:09PM +0100, Gerd Hoffmann wrote: > Hi, > >> So what seems to be happening here is there is an implicit monitor >> being set up which grabs stdio. Because: >> >>#define STDIO_MAX_CLIENTS 1 >> >> my own -serial stdio option subsequently fails. This is a regressio

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Richard W.M. Jones
On Thu, Jan 07, 2010 at 12:44:17PM +, Daniel P. Berrange wrote: > On Thu, Jan 07, 2010 at 06:34:13AM -0600, Anthony Liguori wrote: > > On 01/07/2010 05:55 AM, Richard W.M. Jones wrote: > > >On Sun, Dec 13, 2009 at 08:24:54AM +, Blue Swirl wrote: > > > > > >>I guess e1c09175bc00dd8dfb2ad1

Re: [Qemu-devel] [PATCH 06/17] eepro100: symbolic names for pci registers

2010-01-07 Thread Anthony Liguori
On 01/07/2010 05:14 AM, Stefan Weil wrote: Michael S. Tsirkin schrieb: No functional changes. I verified that the generated binary does not change in meaningful ways. Survived light usage with linux guest. Signed-off-by: Michael S. Tsirkin --- hw/eepro100.c | 49

Re: [Qemu-devel] [PATCH 06/17] eepro100: symbolic names for pci registers

2010-01-07 Thread Anthony Liguori
On 01/07/2010 06:51 AM, Michael S. Tsirkin wrote: A simple solution that worked for me is to publish a git tree and send pull requests (in addition to individual patches). This guarantees that nothing is lost and makes it easy for Anthony to apply a set of changes in one go. git merge also has s

Re: [Qemu-devel] cpuid problem in upstream qemu with kvm

2010-01-07 Thread Dor Laor
On 01/07/2010 03:14 PM, Anthony Liguori wrote: On 01/07/2010 06:40 AM, Avi Kivity wrote: On 01/07/2010 02:33 PM, Anthony Liguori wrote: There's another option. Make cpuid information part of live migration protocol, and then support something like -cpu Xeon-3550. We would remember the exact c

Re: [Qemu-devel] [PATCH V4 00/18] support NEC PC-9821

2010-01-07 Thread TAKEDA, toshiya
Dear members, I made contact with Mr.Yui, the author of famous PC-9821 emulator Neko Project2 and I heared there are several legal problems to emulate PC-9821 with 32bit CPU for commercial use. To my regret, I withdraw the patches I posted and give up to merge them to tree. (I will continue to d

Re: [Qemu-devel] [PATCH 06/17] eepro100: symbolic names for pci registers

2010-01-07 Thread Stefan Weil
Anthony Liguori schrieb: > On 01/07/2010 05:14 AM, Stefan Weil wrote: >> Michael S. Tsirkin schrieb: >> >>> No functional changes. I verified that the generated binary >>> does not change in meaningful ways. Survived light usage >>> with linux guest. >>> >>> Signed-off-by: Michael S. Tsirkin >>>

[Qemu-devel] [PATCH 3/3] use pkg-config for libcurl when cross compiling

2010-01-07 Thread Paolo Bonzini
Signed-off-by: Paolo Bonzini --- configure | 10 -- 1 files changed, 8 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 9fc3173..92a0d21 100755 --- a/configure +++ b/configure @@ -1309,13 +1309,19 @@ fi ## # curl probe +

[Qemu-devel] [PATCH 1/3] use cross-prefix for pkg-config

2010-01-07 Thread Paolo Bonzini
Since pkg-config can give different output for different targets, it should be tried with the cross-compilation prefix first. (Note that the autoconf macro for pkg-config uses AC_PATH_TOOL, which also tries to add the prefix). Signed-off-by: Paolo Bonzini --- configure | 19 ++-

[Qemu-devel] [PATCH 0/3] Fixes for Linux->Win32 cross-compilation

2010-01-07 Thread Paolo Bonzini
With this series I could test cross-compiled Win32 QEMU under Wine. The VNC server fails with "inet_parse: ipv4 parse error (0)", so I had to use SDL; but this patch series is needed to find SDL when cross compiling. Paolo Bonzini (3): use cross-prefix for pkg-config use pkg-config for sdl wh

[Qemu-devel] [PATCH 2/3] use pkg-config for sdl when cross compiling

2010-01-07 Thread Paolo Bonzini
Together with the previous patch this enables using the prefixed pkg-config, thus picking up the correct flags for SDL. Since pkg-config has an awful lot of dependencies, I still use sdl-config when not cross-compiling since some people may only have the latter. Signed-off-by: Paolo Bonzini ---

[Qemu-devel] [RFC] API change for pci_set_word and related functions (was Re: [PATCH] eepro100: Fix initial value for PCI_STATUS)

2010-01-07 Thread Stefan Weil
Michael S. Tsirkin schrieb: > On Thu, Jan 07, 2010 at 12:15:25PM +0100, Stefan Weil wrote: > ... >> --- >> hw/eepro100.c | 4 +--- >> 1 files changed, 1 insertions(+), 3 deletions(-) >> >> diff --git a/hw/eepro100.c b/hw/eepro100.c >> index 336ca49..a21c984 100644 >> --- a/hw/eepro100.c >> +++ b/hw/

[Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Anthony Liguori
On 01/05/2010 11:08 AM, Adam Litke wrote: This patch has been discussed (and ACKed) in the past, but has not yet been committed due to the congestion surrounding the 0.12 release and other conflicting changes. I would like to rekindle the discussion so that I can make the necessary changes to ge

[Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Avi Kivity
On 01/07/2010 05:12 PM, Anthony Liguori wrote: 3) Make qemu request balloon stats regularly (maybe every 10 seconds) and display the latest stats with info balloon. This avoids the problem in #2 but it means that qemu determines the poll rate instead of a management tool. 4) Make info-ball

[Qemu-devel] Re: [PATCH 3/5] PCI: Convert pci_info() to QObject

2010-01-07 Thread Anthony Liguori
On 01/05/2010 02:34 PM, Luiz Capitulino wrote: The returned QObject is a QList of all buses. Each bus is represented by a QDict, which has a key with a QList of all PCI devices attached to it. Each device is represented by a QDict. IMPORTANT: support for printing PCI bridge information and its d

[Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Anthony Liguori
On 01/07/2010 09:18 AM, Avi Kivity wrote: On 01/07/2010 05:12 PM, Anthony Liguori wrote: 3) Make qemu request balloon stats regularly (maybe every 10 seconds) and display the latest stats with info balloon. This avoids the problem in #2 but it means that qemu determines the poll rate instead

Re: [Qemu-devel] [PATCH 0/3] Fixes for Linux->Win32 cross-compilation

2010-01-07 Thread Stefan Weil
Paolo Bonzini schrieb: > With this series I could test cross-compiled Win32 QEMU under Wine. > The VNC server fails with "inet_parse: ipv4 parse error (0)", so I had > to use SDL; but this patch series is needed to find SDL when cross > compiling. > > > Paolo Bonzini (3): > use cross-prefix for p

[Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Adam Litke
Thanks for the ideas and feedback. I will play around with #4 and see if I can hack up a prototype. On Thu, 2010-01-07 at 09:22 -0600, Anthony Liguori wrote: > On 01/07/2010 09:18 AM, Avi Kivity wrote: > > On 01/07/2010 05:12 PM, Anthony Liguori wrote: > >> > >> 3) Make qemu request balloon stats

Re: [Qemu-devel] Qemu's internal TFTP server breaks lock-step-iness of TFTP

2010-01-07 Thread H. Peter Anvin
On 01/06/2010 02:11 PM, Milan Plzik wrote: > Hello, > > according to RFC 1350 and RFC 2347, TFTP server should answer RRQ by > either OACK or DATA packet. Qemu's internal TFTP server answers RRQ with > additional options by sending both OACK and DATA packet, thus breaking > the "lock-step" fea

Re: [Qemu-devel] -serial stdio broken

2010-01-07 Thread Anthony Liguori
On 01/07/2010 07:25 AM, Richard W.M. Jones wrote: On Thu, Jan 07, 2010 at 01:32:09PM +0100, Gerd Hoffmann wrote: Hi, So what seems to be happening here is there is an implicit monitor being set up which grabs stdio. Because: #define STDIO_MAX_CLIENTS 1 my own -serial stdio

Re: [Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Anthony Liguori
On 01/07/2010 06:44 AM, Gleb Natapov wrote: Initialize KVM paravirt cpuid leaf and allow user to control guest visible PV features through -cpu flag. Signed-off-by: Gleb Natapov diff --git a/target-i386/cpu.h b/target-i386/cpu.h index f3834b3..216b00e 100644 --- a/target-i386/cpu.h +++ b/target-

[Qemu-devel] [PATCH] eepro100: Update ROM file support

2010-01-07 Thread Stefan Weil
Use new way to associate ROM files to devices. Currently, there is only a ROM file for i82559er included in QEMU, so the patch does not add .romfile for the other devices. When flexible mode is fixed in eepro100, adding more ROM files will be possible. It should be possible to create them from px

Re: [Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Adam Litke
On Thu, 2010-01-07 at 15:49 +, Daniel P. Berrange wrote: > On Thu, Jan 07, 2010 at 09:12:10AM -0600, Anthony Liguori wrote: > > On 01/05/2010 11:08 AM, Adam Litke wrote: > > >This patch has been discussed (and ACKed) in the past, but has not yet > > >been committed due to the congestion surroun

[Qemu-devel] [PATCH] pc-bios: Update README (SeaBIOS)

2010-01-07 Thread Stefan Weil
The PC BIOS no longer comes from Bochs. This patch updates the related entry. Signed-off-by: Stefan Weil --- pc-bios/README |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/pc-bios/README b/pc-bios/README index 1b7a666..f1c9b2f 100644 --- a/pc-bios/README +++ b/pc-bios/

Re: [Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Gleb Natapov
On Thu, Jan 07, 2010 at 10:09:16AM -0600, Anthony Liguori wrote: > On 01/07/2010 06:44 AM, Gleb Natapov wrote: > >Initialize KVM paravirt cpuid leaf and allow user to control guest > >visible PV features through -cpu flag. > > > >Signed-off-by: Gleb Natapov > >diff --git a/target-i386/cpu.h b/targe

Re: [Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Avi Kivity
On 01/07/2010 06:27 PM, Adam Litke wrote: I think 'info-balloon' should be synchronous and without side-effects. ie return the current stats that QEMU has. We could then add a separate 'refresh-balloon' command + async event notification when that completes. The tool that is requiring the stats

Re: [Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Daniel P. Berrange
On Thu, Jan 07, 2010 at 09:12:10AM -0600, Anthony Liguori wrote: > On 01/05/2010 11:08 AM, Adam Litke wrote: > >This patch has been discussed (and ACKed) in the past, but has not yet > >been committed due to the congestion surrounding the 0.12 release and > >other conflicting changes. I would like

Re: [Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Anthony Liguori
On 01/07/2010 10:15 AM, Gleb Natapov wrote: KVM_CPUID_SIGNATURE is unnecessary as a guard because it's always been present in kvm_para.h. Since we include kvm_para.h unconditionally, there's no way we wouldn't have KVM_CPUID_SIGNATURE. It is there in qemu-kvm and we suppose to bring know

Re: [Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Anthony Liguori
On 01/07/2010 10:39 AM, Avi Kivity wrote: On 01/07/2010 06:27 PM, Adam Litke wrote: I think 'info-balloon' should be synchronous and without side-effects. ie return the current stats that QEMU has. We could then add a separate 'refresh-balloon' command + async event notification when that com

Re: [Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Avi Kivity
On 01/07/2010 06:53 PM, Anthony Liguori wrote: I don't want to deviate unnecessarily from qemu-kvm so we can approach this in one of two ways. We can remove the guard in qemu-kvm and Avi can send me a pull request with this feature. I can also just pull in the patch as-is and we can fix thi

Re: [Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Gleb Natapov
On Thu, Jan 07, 2010 at 10:53:30AM -0600, Anthony Liguori wrote: > On 01/07/2010 10:15 AM, Gleb Natapov wrote: > >>KVM_CPUID_SIGNATURE is unnecessary as a guard because it's always > >>been present in kvm_para.h. Since we include kvm_para.h > >>unconditionally, there's no way we wouldn't have > >>

[Qemu-devel] [PATCH] [tftp] Handle TFTP ERROR from client

2010-01-07 Thread Thomas Horsten
If a PXE client only wants to find out the size of a file, it will open the file and then abort the transfer by sending a TFTP ERROR packet. The ERROR packet should cause qemu to terminate the session. If not, the sessions will soon run out and cause timeouts in the client. Also, if a TFTP sessio

Re: [Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Gleb Natapov
On Thu, Jan 07, 2010 at 06:56:31PM +0200, Avi Kivity wrote: > On 01/07/2010 06:53 PM, Anthony Liguori wrote: > > > >I don't want to deviate unnecessarily from qemu-kvm so we can > >approach this in one of two ways. We can remove the guard in > >qemu-kvm and Avi can send me a pull request with this

[Qemu-devel] [PATCHv2] Add KVM paravirt cpuid leaf

2010-01-07 Thread Gleb Natapov
Initialize KVM paravirt cpuid leaf and allow user to control guest visible PV features through -cpu flag. Signed-off-by: Gleb Natapov --- v1->v2 fix indentation remove unneeded ifdefs diff --git a/target-i386/cpu.h b/target-i386/cpu.h index f3834b3..fbe11b8 100644 --- a/target-i386/cpu.h +++

Re: [Qemu-devel] [PATCH 8/9] sparc64: interrupt trap handling

2010-01-07 Thread Igor Kovalenko
On Wed, Jan 6, 2010 at 8:00 PM, Blue Swirl wrote: > On Tue, Jan 5, 2010 at 11:19 PM, Igor V. Kovalenko > wrote: >> From: Igor V. Kovalenko >> >> cpu_check_irqs >> - handle SOFTINT register TICK and STICK timer bits >> - only check interrupt levels greater than PIL value >> - handle preemption by

Re: [Qemu-devel] [PATCH] pc-bios: Update README (SeaBIOS)

2010-01-07 Thread Gleb Natapov
On Thu, Jan 07, 2010 at 05:31:58PM +0100, Stefan Weil wrote: > The PC BIOS no longer comes from Bochs. > This patch updates the related entry. > > Signed-off-by: Stefan Weil > --- > pc-bios/README |3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/pc-bios/README b/p

Re: [Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Luiz Capitulino
On Thu, 07 Jan 2010 10:56:19 -0600 Anthony Liguori wrote: > On 01/07/2010 10:39 AM, Avi Kivity wrote: > > On 01/07/2010 06:27 PM, Adam Litke wrote: > >> > >>> I think 'info-balloon' should be synchronous and without side-effects. > >>> ie return the current stats that QEMU has. We could then add

[Qemu-devel] Re: [PATCH 3/5] PCI: Convert pci_info() to QObject

2010-01-07 Thread Luiz Capitulino
On Thu, 07 Jan 2010 09:19:28 -0600 Anthony Liguori wrote: > > -static void pci_info_device(PCIBus *bus, PCIDevice *d) > > +static void pci_for_each_device_under_bus(PCIBus *bus, > > + void (*fn)(PCIBus *b, PCIDevice > > *d)) > > { > > -Monitor *mon

Re: [Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Anthony Liguori
On 01/07/2010 11:08 AM, Gleb Natapov wrote: On Thu, Jan 07, 2010 at 06:56:31PM +0200, Avi Kivity wrote: On 01/07/2010 06:53 PM, Anthony Liguori wrote: I don't want to deviate unnecessarily from qemu-kvm so we can approach this in one of two ways. We can remove the guard in qemu-kvm a

Re: [Qemu-devel] [PATCH] Add KVM paravirt cpuid leaf

2010-01-07 Thread Gleb Natapov
On Thu, Jan 07, 2010 at 12:02:01PM -0600, Anthony Liguori wrote: > On 01/07/2010 11:08 AM, Gleb Natapov wrote: > >On Thu, Jan 07, 2010 at 06:56:31PM +0200, Avi Kivity wrote: > >>On 01/07/2010 06:53 PM, Anthony Liguori wrote: > >>>I don't want to deviate unnecessarily from qemu-kvm so we can > >>>ap

[Qemu-devel] Re: [PATCH v3] Drop --whole-archive and static libraries

2010-01-07 Thread Blue Swirl
Thanks, applied. On Wed, Jan 6, 2010 at 7:24 PM, Andreas Färber wrote: > From: Andreas Färber > > Juan has contributed a cool Makefile infrastructure that enables us to drop > static libraries completely: > > Move shared obj-y definitions to Makefile.objs, prefixed {common-,hw-,user-}, > and li

[Qemu-devel] Re: [PATCH v3] Drop --whole-archive and static libraries

2010-01-07 Thread Palle Lyckegaard
On Wed, 6 Jan 2010, Andreas F?rber wrote: v3: - Fix dependency modelling for tools - Remove comment on GENERATED_HEADERS obsoleted by this patch Hi, as the previous patches this v3 version solves the linking problems on Solaris hosts regards Palle

[Qemu-devel] [PATCH] pc-bios: Update README (SeaBIOS)

2010-01-07 Thread Stefan Weil
The PC BIOS no longer comes from Bochs. This patch updates the related entry. V2 - Modify SeaBIOS description and URL (Thanks to Gleb Natapov for the hint). Signed-off-by: Stefan Weil --- pc-bios/README |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/pc-bios/READ

[Qemu-devel] Re: [PATCH 0/3] Fixes for Linux->Win32 cross-compilation

2010-01-07 Thread Paolo Bonzini
Cross compilation works without patches when PATH is modified so it finds the correct sdl-config True, but anything beyond "./configure --cross-prefix=cpu-vendor-os-" (or --host for autoconf) is a bug in either your setup or your package or both. Paolo

Re: [Qemu-devel] Re: [RFD] virtio: Add memory statistics reporting to the balloon driver

2010-01-07 Thread Luiz Capitulino
On Thu, 7 Jan 2010 15:58:30 -0200 Luiz Capitulino wrote: > I like Daniel's idea too. In practice 'refresh-balloon' is going to > be Anthony's idea #1 for the QMP case, which seems the right way to > do it with QMP. Hm, something that has just occurred to me: it's easy to have async messages in

Re: [Qemu-devel] Re: Missing BIOS32 in QEMU 0.12.1

2010-01-07 Thread Anthony Liguori
On 01/04/2010 08:28 PM, Kevin O'Connor wrote: On Sun, Jan 03, 2010 at 10:37:58PM +0100, Aurelien Jarno wrote: Hi, I have just discovered that contrary to QEMU 0.11.x, version 0.12.1 does not support BIOS32. This is due to to the switch from pc-bios to SeaBIOS. This prevents booting certain

[Qemu-devel] [PATCH 0/6] tcg conditional set, round 5

2010-01-07 Thread Richard Henderson
Changes since round 4: * Document double-word support opcodes in a new section of the README, as requested by Aurelien. * Shorten the branch-to-next in the EQ case of i386 brcond2, as pointed out by Laurent. r~ Richard Henderson (6): tcg: Document double-word support opcodes. tcg

[Qemu-devel] [PATCH 2/6] tcg: Generic support for conditional set

2010-01-07 Thread Richard Henderson
Defines setcond_{i32,i64} and setcond2_i32 for 64-on-32-bit. Signed-off-by: Richard Henderson --- tcg/README| 13 + tcg/tcg-op.h | 47 +++ tcg/tcg-opc.h |3 +++ tcg/tcg.c | 21 +++-- 4 files changed, 78 in

[Qemu-devel] [PATCH 3/6] tcg: Add tcg_invert_cond.

2010-01-07 Thread Richard Henderson
It is very handy to have a reliable mapping of a condition to its inverse. Signed-off-by: Richard Henderson --- tcg/tcg.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/tcg/tcg.h b/tcg/tcg.h index 121b17c..b218abe 100644 --- a/tcg/tcg.h +++ b/tcg/tcg.h @@ -205,6 +20

[Qemu-devel] [PATCH 5/6] tcg-i386: Implement setcond.

2010-01-07 Thread Richard Henderson
Signed-off-by: Richard Henderson --- tcg/i386/tcg-target.c | 73 +++-- 1 files changed, 70 insertions(+), 3 deletions(-) diff --git a/tcg/i386/tcg-target.c b/tcg/i386/tcg-target.c index e1691b3..3914570 100644 --- a/tcg/i386/tcg-target.c +++ b/tcg/i3

[Qemu-devel] [PATCH 4/6] tcg-i386: Implement small forward branches.

2010-01-07 Thread Richard Henderson
There are places, like brcond2, where we know that the destination of a forward branch will be within 127 bytes. Add the R_386_PC8 relocation type to support this. Add a flag to tcg_out_jxx and tcg_out_brcond* to enable it. Set the flag in the brcond2 label_next branches; pass along the input fl

[Qemu-devel] [PATCH 6/6] tcg-x86_64: Implement setcond.

2010-01-07 Thread Richard Henderson
Signed-off-by: Richard Henderson --- tcg/x86_64/tcg-target.c | 38 +- 1 files changed, 33 insertions(+), 5 deletions(-) diff --git a/tcg/x86_64/tcg-target.c b/tcg/x86_64/tcg-target.c index 2339091..33dc452 100644 --- a/tcg/x86_64/tcg-target.c +++ b/tcg/x86_6

[Qemu-devel] [PATCH 1/6] tcg: Document double-word support opcodes.

2010-01-07 Thread Richard Henderson
The internal opcodes brcond2, add2, sub2, mulu2 were undocumented. Place these in a new section that clearly indicates that they are not to be emitted by translators. Signed-off-by: Richard Henderson --- tcg/README | 23 +++ 1 files changed, 23 insertions(+), 0 deletions(-)

Re: [Qemu-devel] Re: [PATCH 0/3] Fixes for Linux->Win32 cross-compilation

2010-01-07 Thread Anthony Liguori
On 01/07/2010 12:28 PM, Paolo Bonzini wrote: Cross compilation works without patches when PATH is modified so it finds the correct sdl-config True, but anything beyond "./configure --cross-prefix=cpu-vendor-os-" (or --host for autoconf) is a bug in either your setup or your package or both.

Re: [Qemu-devel] [PATCH v2] Added 'access' option to -drive flag

2010-01-07 Thread Anthony Liguori
On 12/24/2009 09:02 AM, Naphtali Sprei wrote: The new option is: access=[rw|ro|auto] rw: open the drive's file with Read and Write permission, don't continue if failed ro: open the file only with Read permission auto: open the file with Read and Write permission, if failed, try only Read permis

Re: [Qemu-devel] [PATCH] write option roms in pc-bios/

2010-01-07 Thread Anthony Liguori
On 12/24/2009 08:27 AM, Paolo Bonzini wrote: The gdt address calculation in linuxboot.bin is broken in two ways: first it loads %cs into %eax, but that instruction leaves the high bits of %eax undefined and we did not clear them. Secondly, we completely ignore the incorrect %eax, and use the unde

Re: [Qemu-devel] [PATCH 6/9] sparc64: clear exception_index with -1 value

2010-01-07 Thread Blue Swirl
On Wed, Jan 6, 2010 at 11:57 PM, Igor Kovalenko wrote: > On Thu, Jan 7, 2010 at 2:29 AM, Artyom Tarasenko > wrote: >> What's the effect of the patch? Don't we need it for sparc32 too? The >> code looks similar. >> >> 2010/1/6 Blue Swirl : >>> Thanks, applied. >>> >>> On Tue, Jan 5, 2010 at 11:19

Re: [Qemu-devel] [PATCH 3/6] target-alpha: Reduce internal processor registers for user-mode.

2010-01-07 Thread Andreas Färber
Am 07.01.2010 um 12:54 schrieb Tristan Gingold: On Jan 6, 2010, at 6:04 PM, Andreas Färber wrote: Am 06.01.2010 um 17:29 schrieb Richard Henderson: since (1) ROMs other than the few supported by MILO are probably not redistributable Tristan's trick here was to provide a way for the use

[Qemu-devel] [PATCH 0/9] sparc64: interrupts and tick timers v1

2010-01-07 Thread Igor V. Kovalenko
The following series is a cleanup over previous one. v0 -> v1: post-review changes - dropped patch "clear exception_index with -1 val" (applied) - new patch "change_pstate should have 32bit argument" - cleanups for coding style and hexadecimal output convention - wrpil is no-op for CONFIG_USER_ONL

[Qemu-devel] [PATCH 1/9] sparc64: change_pstate should have 32bit argument

2010-01-07 Thread Igor V. Kovalenko
From: Igor V. Kovalenko - pstate is 32bit variable, no need to pass 64bit value around Signed-off-by: Igor V. Kovalenko --- target-sparc/op_helper.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/target-sparc/op_helper.c b/target-sparc/op_helper.c index c63de07..

[Qemu-devel] [PATCH 2/9] sparc64: trace pstate and global register set changes

2010-01-07 Thread Igor V. Kovalenko
From: Igor V. Kovalenko Signed-off-by: Igor V. Kovalenko --- target-sparc/op_helper.c | 20 1 files changed, 20 insertions(+), 0 deletions(-) diff --git a/target-sparc/op_helper.c b/target-sparc/op_helper.c index fd3286d..1d3adef 100644 --- a/target-sparc/op_helper.c +++

[Qemu-devel] [PATCH 4/9] sparc64: use helper_wrpil to check pending irq on write

2010-01-07 Thread Igor V. Kovalenko
From: Igor V. Kovalenko Signed-off-by: Igor V. Kovalenko --- target-sparc/helper.h|1 + target-sparc/op_helper.c | 14 ++ target-sparc/translate.c |5 + 3 files changed, 16 insertions(+), 4 deletions(-) diff --git a/target-sparc/helper.h b/target-sparc/helper.h in

[Qemu-devel] [PATCH 3/9] sparc64: add PIL to cpu state dump

2010-01-07 Thread Igor V. Kovalenko
From: Igor V. Kovalenko Signed-off-by: Igor V. Kovalenko --- target-sparc/helper.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/target-sparc/helper.c b/target-sparc/helper.c index a06923a..e801474 100644 --- a/target-sparc/helper.c +++ b/target-sparc/helper.c @@ -145

[Qemu-devel] [PATCH 5/9] sparc64: check for pending irq when pil, pstate or softint is changed

2010-01-07 Thread Igor V. Kovalenko
From: Igor V. Kovalenko Signed-off-by: Igor V. Kovalenko --- target-sparc/op_helper.c | 41 ++--- 1 files changed, 38 insertions(+), 3 deletions(-) diff --git a/target-sparc/op_helper.c b/target-sparc/op_helper.c index 3d7f64c..381e6c4 100644 --- a/target-

[Qemu-devel] [PATCH 6/9] sparc64: add macros to deal with softint and timer interrupt

2010-01-07 Thread Igor V. Kovalenko
From: Igor V. Kovalenko Signed-off-by: Igor V. Kovalenko --- hw/sun4u.c |1 - target-sparc/cpu.h |4 2 files changed, 4 insertions(+), 1 deletions(-) diff --git a/hw/sun4u.c b/hw/sun4u.c index 9d46f08..029e3ed 100644 --- a/hw/sun4u.c +++ b/hw/sun4u.c @@ -73,7 +73,6 @@ #

[Qemu-devel] [PATCH 7/9] sparc64: move cpu_interrupts_enabled to cpu.h

2010-01-07 Thread Igor V. Kovalenko
From: Igor V. Kovalenko - to be used by cpu_check_irqs Signed-off-by: Igor V. Kovalenko --- target-sparc/cpu.h | 13 + target-sparc/exec.h | 13 - 2 files changed, 13 insertions(+), 13 deletions(-) diff --git a/target-sparc/cpu.h b/target-sparc/cpu.h index 0f0e38c

[Qemu-devel] [PATCH 8/9] sparc64: interrupt trap handling

2010-01-07 Thread Igor V. Kovalenko
From: Igor V. Kovalenko cpu_check_irqs - handle SOFTINT register TICK and STICK timer bits - only check interrupt levels greater than PIL value - handle preemption by higher level traps cpu_exec - handle CPU_INTERRUPT_HARD only if interrupts are enabled - PIL 15 is not special level on sparcv9

  1   2   >