[Qemu-devel] SPARC64: immediate segfault on startup with git mastervery

2012-03-17 Thread Mark Cave-Ayland
Hi Avi/Blue, I've just updated to git master and found that SPARC64 is broken again; a git bisect shows the following commit causes this: commit f3705d53296d78b14f5823472ae2add16a25a0a5 Author: Avi Kivity Date: Thu Mar 8 16:16:34 2012 +0200 memory: make phys_page_find() return an unad

[Qemu-devel] Build broken -- qemu-ga: add guest-network-get-interfaces command

2012-03-17 Thread Brad Smith
Michal, http://git.qemu.org/?p=qemu.git;a=commit;h=3424fc9f16a1e7d1c48eb6d605eb0ca63e199ec2 This broke the build. Un-break the tree. -- This message has been scanned for viruses and dangerous content by MailScanner, and is believed to be clean.

[Qemu-devel] [PATCH] PNG screendump alternative

2012-03-17 Thread Rafael
Hi, I had already started to work on supporting PNG dump output on screendump when got to know there was another effort to do the same (see Add PNG screendump thread from Daniel P. Berrange). Anyway, I'd like to share my implementation which is a little bit different. It does not use any resource

Re: [Qemu-devel] [PATCH] Fix build on FreeBSD

2012-03-17 Thread Nathan Whitehorn
On Mar 17, 2012, at 11:20 AM, Blue Swirl wrote: On Sat, Mar 3, 2012 at 16:34, Nathan Whitehorn wrote: Fix a missing header required to build on recent FreeBSD. Signed-off-by: Nathan Whitehorn --- os-posix.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/os-pos

Re: [Qemu-devel] [PATCH] alpha-user: Initialize FPCR with round-to-nearest.

2012-03-17 Thread Richard Henderson
Ping. On 03/11/2012 11:54 AM, Richard Henderson wrote: > Signed-off-by: Richard Henderson > --- > target-alpha/translate.c |3 ++- > 1 files changed, 2 insertions(+), 1 deletions(-) > > diff --git a/target-alpha/translate.c b/target-alpha/translate.c > index 1d2142b..fe78630 100644 > --- a/

Re: [Qemu-devel] [PATCH v5 12/43] target-alpha: Don't overuse CPUState

2012-03-17 Thread Richard Henderson
On 03/14/2012 02:42 PM, Andreas Färber wrote: > Scripted conversion: > sed -i "s/CPUState/CPUAlphaState/g" target-alpha/*.[hc] > sed -i "s/#define CPUAlphaState/#define CPUState/" target-alpha/cpu.h > > Signed-off-by: Andreas Färber > Acked-by: Anthony Liguori > --- > target-alpha/cpu.h

Re: [Qemu-devel] [PATCH 6/6] prep: use pc87312 Super I/O chip instead of collection of random ISA devices

2012-03-17 Thread Paolo Bonzini
Il 17/03/2012 15:39, Hervé Poussineau ha scritto: > +qdev_prop_set_chr(&isa->qdev, "parallel", parallel_hds[0]); > +qdev_prop_set_chr(&isa->qdev, "uart1", serial_hds[0]); > +qdev_prop_set_chr(&isa->qdev, "uart2", serial_hds[1]); Set these conditionally on *_hds[] being non-NULL, so tha

Re: [Qemu-devel] [PULL 0/4] arm-devs queue

2012-03-17 Thread Peter Maydell
On 17 March 2012 16:22, Blue Swirl wrote: > On Fri, Mar 16, 2012 at 18:12, Peter Maydell wrote: >> Hi; this is a pullreq for the arm-devs queue; nothing hugely exciting >> here unless you count the final part of the -dtb support. Please pull. > > Thanks, pulled. Thanks. I see that I somehow mana

Re: [Qemu-devel] [PATCH] configure: Fix creation of symbolic links for MinGW toolchain

2012-03-17 Thread Peter Maydell
On 17 March 2012 15:59, Stefan Weil wrote: > Am 17.03.2012 16:49, schrieb Peter Maydell: >> Where do we symlink directories? (I exclude the setting up >> of the linux headers because that will only happen on Linux >> hosts where we know we have working symlinks.) > pc-bios/keymaps Oh yes, missed

Re: [Qemu-devel] [PATCH] w64: Fix data type of next_tb and tcg_qemu_tb_exec

2012-03-17 Thread Blue Swirl
Thanks, applied. On Fri, Mar 16, 2012 at 22:50, Stefan Weil wrote: > next_tb is the numeric value of a tcg target (= QEMU host) address. > > Using tcg_target_ulong instead of unsigned long shows this and makes > the code portable for hosts with an unusual size of long (w64). > > The type cast '(l

Re: [Qemu-devel] [PATCH] softfloat: fix for C99

2012-03-17 Thread Blue Swirl
Thanks, applied. On Tue, Dec 27, 2011 at 15:11, Avi Kivity wrote: > C99 appears to consider compound literals as non-constants, and complains > when they are used in static initializers.  Switch to ordinary initializer > syntax. > > Signed-off-by: Avi Kivity > --- >  fpu/softfloat-specialize.h |

Re: [Qemu-devel] [PULL 0/3] target-arm queue

2012-03-17 Thread Blue Swirl
On Fri, Mar 16, 2012 at 18:21, Peter Maydell wrote: > Hi; this is a pullreq for my target-arm queue. Just three fairly > minor bug fixes this time. Please pull. Thanks, pulled. > Thanks > -- PMM > > The following changes since commit ae7d54d489540b49b7c13a7df7ddc220588a2ced: > >  target-lm32/mic

Re: [Qemu-devel] [PULL 0/4] arm-devs queue

2012-03-17 Thread Blue Swirl
On Fri, Mar 16, 2012 at 18:12, Peter Maydell wrote: > Hi; this is a pullreq for the arm-devs queue; nothing hugely exciting > here unless you count the final part of the -dtb support. Please pull. Thanks, pulled. > thanks > -- PMM > > The following changes since commit 7c51c1aa03a52b9fd75ed1ade2

Re: [Qemu-devel] [PULL] Malta patches

2012-03-17 Thread Blue Swirl
On Thu, Mar 15, 2012 at 20:16, Stefan Weil wrote: > Am 25.02.2012 15:43, schrieb Stefan Weil: >> >> Hi Aurelien, >> >> could you please pull some Malta patches which I had sent in January? >> >> http://patchwork.ozlabs.org/patch/138394/ >> http://patchwork.ozlabs.org/patch/138392/ >> http://patchw

Re: [Qemu-devel] [PULL 00/16] ppc patch queue 2012-03-15

2012-03-17 Thread Blue Swirl
On Thu, Mar 15, 2012 at 12:14, Alexander Graf wrote: > Hi Blue / Aurelien, > > This is my current patch queue for ppc. Please pull. Thanks, pulled. > Alex > > > The following changes since commit ae7d54d489540b49b7c13a7df7ddc220588a2ced: >  Andreas Färber (1): >        target-lm32/microblaze: Dr

Re: [Qemu-devel] [RESEND PATCH] vmstate: fix varrays with uint32_t indexes

2012-03-17 Thread Blue Swirl
Thanks, applied. On Tue, Mar 13, 2012 at 06:05, Amos Kong wrote: > VMSTATE_VARRAY_UINT32() is used in hw/ds1225y.c, and we checked > VMS_VARRAY_UINT32 bit of field->flags in vmstate_load_state(), > but we don't check this bit in vmstate_save_state(). > > Signed-off-by: Amos Kong > --- >  savevm.

Re: [Qemu-devel] [PATCH] Fix build on FreeBSD

2012-03-17 Thread Blue Swirl
On Sat, Mar 3, 2012 at 16:34, Nathan Whitehorn wrote: > Fix a missing header required to build on recent FreeBSD. > > Signed-off-by: Nathan Whitehorn > --- >  os-posix.c |    4 >  1 files changed, 4 insertions(+), 0 deletions(-) > > diff --git a/os-posix.c b/os-posix.c > index dbf3b24..83b14

Re: [Qemu-devel] [PATCH 1/1] Fix large memory chunks allocation with tcg_malloc.

2012-03-17 Thread Blue Swirl
Thanks, applied. On Fri, Mar 2, 2012 at 09:22, Kirill Batuzov wrote: > An attempt to allocate a large memory chunk after a small one resulted in > circular links in list of pools.  It caused the same memory being > allocated twice for different arrays. > > Now pools for large memory chunks are ke

Re: [Qemu-devel] [PATCH] configure: Fix creation of symbolic links for MinGW toolchain

2012-03-17 Thread Stefan Weil
Am 17.03.2012 16:49, schrieb Peter Maydell: On 17 March 2012 15:31, Stefan Weil wrote: Macro symlink is also used with directories. To remove them on w32 hosts, a recursive rm is needed. Where do we symlink directories? (I exclude the setting up of the linux headers because that will only happ

Re: [Qemu-devel] [PATCH] configure: Fix creation of symbolic links for MinGW toolchain

2012-03-17 Thread Peter Maydell
On 17 March 2012 15:31, Stefan Weil wrote: > Macro symlink is also used with directories. To remove them on w32 > hosts, a recursive rm is needed. Where do we symlink directories? (I exclude the setting up of the linux headers because that will only happen on Linux hosts where we know we have wor

Re: [Qemu-devel] [PATCH] QEMU: Add pflash support for versatile and integrator

2012-03-17 Thread Peter Maydell
On 17 March 2012 02:43, Marek Vasut wrote: > Signed-off-by: Marek Vasut This patch breaks previously working command lines -- you can't make providing the flash binary blob mandatory. > --- >  hw/integratorcp.c |   26 ++ >  hw/versatilepb.c  |   24 ++

[Qemu-devel] [PATCH] configure: Fix creation of symbolic links for MinGW toolchain

2012-03-17 Thread Stefan Weil
The MinGW toolchain on w32/w64 hosts does not create symbolic links, but implements 'ln -s' similar to 'cp -r'. In incremental out of tree builds, this resulted in files which were not updated when their counterparts in the QEMU source tree changed. Especially for Makefile* this happened very ofte

[Qemu-devel] [PATCH 5/6] prep: add pc87312 Super I/O emulation

2012-03-17 Thread Hervé Poussineau
This provides floppy and IDE controllers as well as serial and parallel ports. However, dynamic configuration of devices is not yet supported. Cc: Andreas Färber Signed-off-by: Hervé Poussineau --- Makefile.objs |1 + hw/pc87312.c | 425

[Qemu-devel] [PATCH 6/6] prep: use pc87312 Super I/O chip instead of collection of random ISA devices

2012-03-17 Thread Hervé Poussineau
We can't however replace the built-in IDE controller, as the one in pc87312 is only single-channel and can use only IRQ 14. Cc: Andreas Färber Signed-off-by: Hervé Poussineau --- default-configs/ppc-softmmu.mak |2 + hw/ppc_prep.c | 50 ++-

[Qemu-devel] [PATCH 4/6] fdc: Parametrize ISA base, IRQ and DMA

2012-03-17 Thread Hervé Poussineau
Keep the PC values as defaults but allow to override them for PReP. Signed-off-by: Hervé Poussineau Cc: Markus Armbruster Signed-off-by: Andreas Färber --- hw/fdc.c | 17 ++--- 1 files changed, 10 insertions(+), 7 deletions(-) diff --git a/hw/fdc.c b/hw/fdc.c index a0236b7..756d

Re: [Qemu-devel] Failed to set a breakpoint on start_kernel

2012-03-17 Thread Jan Kiszka
On 2012-03-17 15:16, Wei Yang wrote: >>> You can also try my patch : >>> >>> http://patchwork.ozlabs.org/patch/137543/ >> >> Unless there is a use case beyond this x86 band-aid, lets focus on >> getting gdb right. Reminds me that gdb folks asked me to file a bug >> about this - which I still need t

[Qemu-devel] [PATCH 3/6] isa: add isa_bus_from_device() method

2012-03-17 Thread Hervé Poussineau
Signed-off-by: Hervé Poussineau --- hw/isa.h |5 + 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/hw/isa.h b/hw/isa.h index 40373fb..f7bc4b5 100644 --- a/hw/isa.h +++ b/hw/isa.h @@ -76,6 +76,11 @@ void isa_register_portio_list(ISADevice *dev, uint16_t start,

[Qemu-devel] [PATCH 0/6] prep: some fixes and Super I/O emulation

2012-03-17 Thread Hervé Poussineau
Hi, First two patches repair some functionality broken since 2009! Debian install root floppy can now be started again. Patches 4 to 6 implement the pc87312 Super I/O chip. Some versions by me or by Andreas Färber have already been sent on mailing list. This patch has been tested on PReP emulatio

[Qemu-devel] [PATCH 2/6] prep: change default cpu to '7448'

2012-03-17 Thread Hervé Poussineau
In b37fc148bb81b5b022846156a6cba266e6c23962, PReP default cpu was changed from 'default' to '602'. However, '7448' is closer of 'default' than '602'. This repairs following command line, which is available in some tutorials: qemu-system-ppc -kernel zImage.prep -fda debian_install_root.bin -M prep

[Qemu-devel] [PATCH 1/6] i82378/i82374: do not create DMA controller twice

2012-03-17 Thread Hervé Poussineau
This fixes a crash in PReP emulation when using DMA controller to access floppy drive. Signed-off-by: Hervé Poussineau --- hw/i82374.c |5 - hw/i82378.c |5 +++-- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/hw/i82374.c b/hw/i82374.c index 67298a3..4a922c3 100644 -

Re: [Qemu-devel] Failed to set a breakpoint on start_kernel

2012-03-17 Thread Wei Yang
>> You can also try my patch : >> >> http://patchwork.ozlabs.org/patch/137543/ > > Unless there is a use case beyond this x86 band-aid, lets focus on > getting gdb right. Reminds me that gdb folks asked me to file a bug > about this - which I still need to do. :-/ > > Jan > Jan, I didn't try your

Re: [Qemu-devel] Failed to set a breakpoint on start_kernel

2012-03-17 Thread Wei Yang
2012/3/17 Jan Kiszka : > [ re-added qemu-devel to CC ] > > On 2012-03-17 13:10, Wei Yang wrote: >>> Two major issues with this procedure: >>> >>> 1. When using kvm, a soft breakpoint (as set by 'b') will inject a trap >>> instruction into the guest image - which is not yet loaded after the >>> bios

Re: [Qemu-devel] Failed to set a breakpoint on start_kernel

2012-03-17 Thread Jan Kiszka
On 2012-03-17 12:25, Laurent Vivier wrote: > Le samedi 17 mars 2012 à 09:53 +0100, Jan Kiszka a écrit : >> On 2012-03-16 03:43, Wei Yang wrote: >>> All >>> >>> I like qemu very much and know it could debug the kernel. >>> >>> I tried what I searched on web but couldn't stop at the break point. >>>

Re: [Qemu-devel] Failed to set a breakpoint on start_kernel

2012-03-17 Thread Jan Kiszka
[ re-added qemu-devel to CC ] On 2012-03-17 13:10, Wei Yang wrote: >> Two major issues with this procedure: >> >> 1. When using kvm, a soft breakpoint (as set by 'b') will inject a trap >> instruction into the guest image - which is not yet loaded after the >> bios ran. You need to use a hardware

Re: [Qemu-devel] [PATCH 3/3] build: check if libm is needed in configure

2012-03-17 Thread Blue Swirl
On Wed, Feb 22, 2012 at 04:55, Roger Pau Monne wrote: > Remove the hardcoded use of libm and instead rely on configure to > check for it. It is needed at least for qemu-ga and qemu-system. This would break linux-user build, linker can't find 'sin' etc. > > Signed-off-by: Roger Pau Monne > --- >

[Qemu-devel] [PATCH] target-mips: Remove unused inline function

2012-03-17 Thread Stefan Weil
Function set_HILO is not needed anywhere. Signed-off-by: Stefan Weil --- target-mips/op_helper.c |6 -- 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/target-mips/op_helper.c b/target-mips/op_helper.c index 3a20731..7b77d5a 100644 --- a/target-mips/op_helper.c +++ b/targe

Re: [Qemu-devel] Failed to set a breakpoint on start_kernel

2012-03-17 Thread Laurent Vivier
Le samedi 17 mars 2012 à 09:53 +0100, Jan Kiszka a écrit : > On 2012-03-16 03:43, Wei Yang wrote: > > All > > > > I like qemu very much and know it could debug the kernel. > > > > I tried what I searched on web but couldn't stop at the break point. > > Below is what I did. > > > > 1. Both host a

Re: [Qemu-devel] [PATCH] Initial support for Ilumos build and Illumos-kvm

2012-03-17 Thread Jan Kiszka
On 2012-03-16 14:46, Lee Essen wrote: > > On 16 Mar 2012, at 13:14, Jan Kiszka wrote: > >> On 2012-03-16 10:23, Lee Essen wrote: >>> +#ifdef __sun__ >>> +#include >>> +#else >>> #include >>> #include >>> +#endif >> >> As Paolo already said, this should somehow be centralised. > > Yep, fair po

Re: [Qemu-devel] Failed to set a breakpoint on start_kernel

2012-03-17 Thread Jan Kiszka
On 2012-03-16 03:43, Wei Yang wrote: > All > > I like qemu very much and know it could debug the kernel. > > I tried what I searched on web but couldn't stop at the break point. > Below is what I did. > > 1. Both host and guest installed the same OS, Fedora16 x86_64. > > 2. Compile the qemu wit

[Qemu-devel] [PATCH v2] Basic Illumos support

2012-03-17 Thread Lee Essen
(third email attempt, apologies if you get duplicates) This patch adds some basic constructs to better support Illumos/Solaris. I've kept away from kvm, configure etc. This just covers making sure the right libs are used, and the code is Solaris/Illumos compatible. In qemu-timer.c there are lots

Re: [Qemu-devel] VDI patches

2012-03-17 Thread Paolo Bonzini
Il 17/03/2012 08:10, Stefan Weil ha scritto: > Hi Kevin, hi Paolo, > > the build failure is caused by the vdi changes: some versions of gcc > detect uninitialized local variables (my gcc 4.4.5 does not). > > I don't think that it's really an error because the code is guarded > by the local variab

[Qemu-devel] buildbot failure in qemu on block_x86_64_debian_6_0

2012-03-17 Thread qemu
The Buildbot has detected a new failure on builder block_x86_64_debian_6_0 while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/block_x86_64_debian_6_0/builds/215 Buildbot URL: http://buildbot.b1-systems.de/qemu/ Buildslave for this Build: yuzuki Buil

[Qemu-devel] [Bug 957622] Re: kvm -kernel with grub multiboot kernel dumps core or exits

2012-03-17 Thread Scott Moser
I've a fix for this upstream at http://thread.gmane.org/gmane.comp.emulators.kvm.devel/88404 ** Changed in: qemu-kvm (Ubuntu) Status: New => In Progress ** Changed in: qemu Status: New => Confirmed ** Changed in: qemu-kvm (Ubuntu) Importance: Undecided => Medium -- You receive

[Qemu-devel] [Bug 957622] Re: kvm -kernel with grub multiboot kernel dumps core or exits

2012-03-17 Thread Scott Moser
I'm pretty sure this is a bug in the linked commit above, in that it does not account for this statement in the multiboot spec: `load_end_addr' Contains the physical address of the end of the data segment. (load_end_addr - load_addr) specifies how much data to load. This implies th

Re: [Qemu-devel] QEMU was not selected for Google Summer of Code this year

2012-03-17 Thread Chris Wright
* Natalia Portillo (clau...@claunia.com) wrote: > QEMU hosted on Haiku would be interesting. The fun of Haiku especially when it is hosting QEMU

Re: [Qemu-devel] VDI patches (was: buildbot failure in qemu on block_i386_debian_6_0)

2012-03-17 Thread Stefan Weil
Am 17.03.2012 04:11, schrieb q...@buildbot.b1-systems.de: The Buildbot has detected a new failure on builder block_i386_debian_6_0 while building qemu. Full details are available at: http://buildbot.b1-systems.de/qemu/builders/block_i386_debian_6_0/builds/215 Buildbot URL: http://buildbot.b1-sy