[Qemu-devel] [PATCH] hw/xen_disk: Remove unused local variable

2011-05-08 Thread Stefan Weil
cppcheck report: hw/xen_disk.c:309: style: Variable 'len' is assigned a value that is never used Cc: Kevin Wolf Signed-off-by: Stefan Weil --- hw/xen_disk.c |4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) diff --git a/hw/xen_disk.c b/hw/xen_disk.c index 558bf8a..c3b8d4c 100644 -

[Qemu-devel] [PATCH] target-mips: Fix warning caused by unused local variable

2011-05-08 Thread Stefan Weil
cppcheck report: target-mips/helper.c:276: style: Variable 'access_type' is assigned a value that is never used 'access_type' is only used for system emulation, so the patch simply fixes the conditional compilation. There remains an issue with the return value 'ret' in user mode emulation. The a

Re: [Qemu-devel] [PATCH] usb-linux: Add missing break statement

2011-05-08 Thread Hans de Goede
Thanks, good catch. Acked-by: Hans de Goede On 05/07/2011 10:10 PM, Stefan Weil wrote: cppcheck report: usb-linux.c:661: warning: Redundant assignment of "len" in switch Please check whether adding a break statement is the correct solution for this warning. Cc: Hans de Goede Cc: Gerd Hoffman

Re: [Qemu-devel] Binary translation (of code)

2011-05-08 Thread Mulyadi Santosa
Hi... On Sat, May 7, 2011 at 18:26, Tarmo Pikaro wrote: > Hi ! > > I've analyzed quite a lot of emulators source code, including qemu as well. > > But I somehow would like to completely get rid of emulators, and simply > convert > application from one cpu/board to another cpu/board. > > On wiki

Re: [Qemu-devel] [RFC] darwin: work around sigfd

2011-05-08 Thread Andreas Färber
Am 05.05.2011 um 15:15 schrieb Alexander Graf: On 05.05.2011, at 14:56, Paolo Bonzini wrote: On 05/05/2011 11:36 AM, Alexander Graf wrote: When running qemu-system on Darwin, the vcpu processes guest code, but I don't get to see anything on the cocoa screen. Out of curiosity, does it work

Re: [Qemu-devel] [PATCH 2/7] PPC: Make MPC8544DS emulation work w/o KVM

2011-05-08 Thread Blue Swirl
On Sun, May 8, 2011 at 2:00 AM, Alexander Graf wrote: > The MPC8544DS board emulation was only used with KVM so far, so some > parts of the code didn't provide proper values for non-KVM execution. > > This patch makes the machine work without KVM enabled. To actually use > this, you also need prop

Re: [Qemu-devel] [RFC] darwin: work around sigfd

2011-05-08 Thread Andreas Färber
Alex, Am 08.05.2011 um 11:15 schrieb Andreas Färber: Am 05.05.2011 um 15:15 schrieb Alexander Graf: On 05.05.2011, at 14:56, Paolo Bonzini wrote: On 05/05/2011 11:36 AM, Alexander Graf wrote: When running qemu-system on Darwin, the vcpu processes guest code, but I don't get to see anythi

Re: [Qemu-devel] Binary translation (of code)

2011-05-08 Thread Blue Swirl
On Sat, May 7, 2011 at 2:26 PM, Tarmo Pikaro wrote: > Hi ! > > I've analyzed quite a lot of emulators source code, including qemu as well. > > But I somehow would like to completely get rid of emulators, and simply > convert > application from one cpu/board to another cpu/board. > > On wiki pages

Re: [Qemu-devel] [PATCH 4/7] target-arm: Refactor int-float conversions

2011-05-08 Thread Blue Swirl
On Fri, May 6, 2011 at 7:38 PM, Paul Brook wrote: >> On Fri, May 6, 2011 at 5:09 PM, Paul Brook wrote: >> >> The Neon versions of int-float conversions need their own helper >> >> routines because they must use the "standard FPSCR" rather than the >> >> default one. Refactor the helper functions

Re: [Qemu-devel] [Qemu-trivial] [PATCH 4/4] fix compilation when reconfiguring without dtrace backend

2011-05-08 Thread Stefan Hajnoczi
On Mon, May 2, 2011 at 8:54 AM, Paolo Bonzini wrote: > If QEMU is configured with the dtrace backend, then built, then > reconfigured without, the build fails. > > The culprit is the trace-dtrace.h dependency that many files will > have.  Due to this dependency, make will attempt to rebuild > trac

[Qemu-devel] [PATCH v2 2/2] ppc64: Don't try to build sPAPR RTAS on Darwin

2011-05-08 Thread Andreas Färber
The Darwin assembler fails to build it. Cc: David Gibson Cc: Alexander Graf Signed-off-by: Andreas Färber --- configure |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure b/configure index 9244260..b6276f3 100755 --- a/configure +++ b/configure @@ -2540,7 +2540

[Qemu-devel] [PATCH v2 1/2] ppc64: Fix out-of-tree builds

2011-05-08 Thread Andreas Färber
On ppc64 host, recursion into pc-bios/spapr-rtas/ fails for out-of-tree builds. Add missing dir and symlink. Cc: David Gibson Cc: Alexander Graf Signed-off-by: Andreas Färber --- v2: * Don't discard previous $DIRS, it would break a fresh build. Spotted by David Gibson. configure |2

Re: [Qemu-devel] [PATCH 01/18] Fix typos in comments (dependancy -> dependency)

2011-05-08 Thread Stefan Hajnoczi
On Thu, Apr 28, 2011 at 05:20:25PM +0200, Stefan Weil wrote: > Signed-off-by: Stefan Weil > --- > Changelog |2 +- > Makefile.objs |2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) Thanks, applied the series to the trivial-patches branch: http://repo.or.cz/w/qemu/stefanha.git

Re: [Qemu-devel] [PATCH] Fix typos in comments (chek -> check)

2011-05-08 Thread Stefan Hajnoczi
On Sat, Apr 30, 2011 at 10:49:26PM +0200, Stefan Weil wrote: > Signed-off-by: Stefan Weil > --- > exec.c|2 +- > target-ppc/STATUS |2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) Thanks, applied to the trivial-patches branch: http://repo.or.cz/w/qemu/stefanha.git/sh

Re: [Qemu-devel] [Qemu-trivial] [PATCH 3/4] lsi53c895a: Rename 'sense' to 'status'

2011-05-08 Thread Stefan Hajnoczi
On Mon, May 02, 2011 at 09:54:05AM +0200, Paolo Bonzini wrote: > From: Hannes Reinecke > > The 'sense' field in the HBA status structure is misnamed, as it > actually carries the SCSI status. Rename it. > > Signed-off-by: Hannes Reinecke > Reviewed-by: Stefan Hajnoczi > Signed-off-by: Paolo Bo

Re: [Qemu-devel] [PATCH] Fix spelling in comments (intruction -> instruction)

2011-05-08 Thread Stefan Hajnoczi
On Sat, May 07, 2011 at 10:20:03PM +0200, Stefan Weil wrote: > Signed-off-by: Stefan Weil > --- > bsd-user/main.c |2 +- > linux-user/main.c |2 +- > tests/test-i386.c |2 +- > 3 files changed, 3 insertions(+), 3 deletions(-) Thanks, applied to the trivial-patches branch: http://re

Re: [Qemu-devel] [PATCH] linux-user: Replace deprecated function

2011-05-08 Thread Stefan Hajnoczi
On Sat, May 07, 2011 at 10:32:25PM +0200, Stefan Weil wrote: > Function bzero is deprecated, so replace it by function memset. > > Signed-off-by: Stefan Weil > --- > linux-user/syscall.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) Thanks, applied to the trivial-patches branch:

Re: [Qemu-devel] [PATCH] ac97: Remove unused local variables

2011-05-08 Thread Stefan Hajnoczi
On Sun, May 08, 2011 at 08:58:11AM +0200, Stefan Weil wrote: > cppcheck report: > hw/ac97.c:1004: style: > Variable 'written' is assigned a value that is never used > hw/ac97.c:1072: style: > Variable 'written' is assigned a value that is never used > > Signed-off-by: Stefan Weil > --- > hw/ac

Re: [Qemu-devel] [Qemu-trivial] [PATCH 1/4] libcacard: add correct subdirectory dependencies

2011-05-08 Thread Stefan Hajnoczi
On Mon, May 02, 2011 at 09:54:03AM +0200, Paolo Bonzini wrote: > Signed-off-by: Paolo Bonzini > Cc: qemu-triv...@nongnu.org > --- > Makefile |2 ++ > Makefile.objs |7 +++ > libcacard/Makefile |9 + > 3 files changed, 6 insertions(+), 12 deletions(-) Thanks

[Qemu-devel] [PATCH] Clean up virtio-9p error handling code

2011-05-08 Thread Sassan Panahinejad
In a lot of cases, the handling of errors was quite ugly. This patch moves reading of errno to immediately after the system calls and passes it up through the system more cleanly. Also, in the case of the xattr functions (and possibly others), completely the wrong error was being returned. --- f

[Qemu-devel] Binary translation (of code)

2011-05-08 Thread Tarmo Pikaro
> In general, this is not possible. Consider for example self-modifying > or otherwise dynamically created code, or just code that examines > itself. > In some specific trivial cases it could work, and QEMU could be made > to abort if translation would be needed. I think self-modifying code is k

Re: [Qemu-devel] [PATCH 2/7] PPC: Make MPC8544DS emulation work w/o KVM

2011-05-08 Thread Alexander Graf
Am 08.05.2011 um 12:08 schrieb Blue Swirl : > On Sun, May 8, 2011 at 2:00 AM, Alexander Graf wrote: >> The MPC8544DS board emulation was only used with KVM so far, so some >> parts of the code didn't provide proper values for non-KVM execution. >> >> This patch makes the machine work without KV

Re: [Qemu-devel] Binary translation (of code)

2011-05-08 Thread Blue Swirl
On Sun, May 8, 2011 at 4:05 PM, Tarmo Pikaro wrote: > >> In general, this is not possible. Consider for example self-modifying >> or otherwise dynamically created code, or just code that examines >> itself. > >> In some specific trivial cases it could work, and QEMU could be made >> to abort if tr

Re: [Qemu-devel] Binary translation (of code)

2011-05-08 Thread Stefan Hajnoczi
On Sun, May 8, 2011 at 2:46 PM, Blue Swirl wrote: > On Sun, May 8, 2011 at 4:05 PM, Tarmo Pikaro wrote: >> >>> In general, this is not possible. Consider for example self-modifying >>> or otherwise dynamically created code, or just code that examines >>> itself. >> >>> In some specific trivial ca

Re: [Qemu-devel] Allow ARMv7M to be started without a kernel

2011-05-08 Thread Andreas Färber
Am 06.05.2011 um 14:48 schrieb Alexander Graf: On 06.05.2011, at 01:50, Rob Landley wrote: On 05/05/2011 06:26 PM, Alexander Graf wrote: As an aside: I think QEMU should have an option which is "just load a plain ELF or raw binary, with no funny Linux-kernel-specific behaviour" rather than ov

Re: [Qemu-devel] implementing ARM926EJ-S support

2011-05-08 Thread Andreas Färber
Hi, Am 03.05.2011 um 14:56 schrieb Alessandro: What SoC are you planning to model? I assume you have one in mind since you were specific about wanting the ARM926 rather than a more recent ARM core... I was originally thinking to TI-DM365: as you can see, this SoC includes several _inusual_

[Qemu-devel] [Bug 723871] Re: qemu-kvm-0.14.0 Aborts with -vga qxl

2011-05-08 Thread Boris Derzhavets
Build spice-gtk-0.6 on Ubuntu 11.04 ( --with-pyton) # cd /root/spice-gtk-0.6 # ./configure --prefix=/usr --build=x86_64-linux-gnu \ --mandir=/usr/share/man --infodir=/usr/share/info --datadir=/usr/share \ --sysconfdir=/etc --localstatedir=/var --libexecdir=/usr/bin \ --with-python . . . con

[Qemu-devel] [ARM] Re: How to test support of a new board?

2011-05-08 Thread Andreas Färber
Hi, Am 19.04.2011 um 17:42 schrieb Li Xin: I want to make a new support of arm board "TH68" of THOMSON TELECOM in QEMU. This board have a microprocessor called "PNX8019". So I made some simulations of the functional blocs. But I don't know how to test the code in QEMU. Is there anyon

Re: [Qemu-devel] [PATCH] USB: Fix build breakage in musb

2011-05-08 Thread Andreas Färber
Am 08.05.2011 um 00:59 schrieb Alexander Graf: Today's git failed building for me. It looks like a mismatching function prototype definition. This patch fixes compilation of arm-softmmu for me. Signed-off-by: Alexander Graf Yup, prototype was incorrectly introduced in 13a9a0d3e253e27274

Re: [Qemu-devel] [PATCH] USB: Fix build breakage in musb

2011-05-08 Thread Stefan Weil
Am 08.05.2011 00:59, schrieb Alexander Graf: Today's git failed building for me. It looks like a mismatching function prototype definition. This patch fixes compilation of arm-softmmu for me. Signed-off-by: Alexander Graf --- hw/usb-musb.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-

[Qemu-devel] [PATCH 07/24] Fix typo in comment (embeded -> embedded)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- target-ppc/cpu.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-ppc/cpu.h b/target-ppc/cpu.h index 04b1259..e438b17 100644 --- a/target-ppc/cpu.h +++ b/target-ppc/cpu.h @@ -225,7 +

[Qemu-devel] [PATCH 04/24] Fix typos in comments (accessable -> accessible, priveleged -> privileged)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/sh7750_regs.h |6 +++--- target-cris/cpu.h |2 +- tests/test-mmap.c |4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/hw/sh7750_regs.h b/hw/sh7750_regs.h index 5a23a2c..6ec13ab

[Qemu-devel] [PATCH 08/24] Fix typo in comment (consistant -> consistent)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- libcacard/vcard_emul_nss.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libcacard/vcard_emul_nss.c b/libcacard/vcard_emul_nss.c index baada52..f3db657 100644 --- a/libcacard/vcard_emul_

[Qemu-devel] [PATCH 20/24] ac97: Remove unused local variables

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil cppcheck report: hw/ac97.c:1004: style: Variable 'written' is assigned a value that is never used hw/ac97.c:1072: style: Variable 'written' is assigned a value that is never used Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/ac97.c |6 +- 1 files

[Qemu-devel] [PATCH 03/24] Fix typos in comments (accross -> across)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- darwin-user/syscall.c |2 +- target-microblaze/helper.c |2 +- tcg/tcg.h |4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/darwin-user/syscall.c b/darwin-user

[Qemu-devel] [PATCH 05/24] Fix typo in comment (colum -> column)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/ssd0303.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/ssd0303.c b/hw/ssd0303.c index 108c068..b39e259 100644 --- a/hw/ssd0303.c +++ b/hw/ssd0303.c @@ -93,7 +93,7 @@ static int ss

[Qemu-devel] [PATCH 14/24] Fix typos in comments (neccessary -> necessary)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- bsd-user/qemu.h|2 +- linux-user/qemu.h |2 +- target-arm/translate.c |4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/bsd-user/qemu.h b/bsd-user/qemu.h index e343894

[Qemu-devel] [PATCH 06/24] Fix typo in comment (auxilliary -> auxiliary)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/pci_regs.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/pci_regs.h b/hw/pci_regs.h index dd0bed4..5a5ab89 100644 --- a/hw/pci_regs.h +++ b/hw/pci_regs.h @@ -223,7 +223,7 @@ #defi

[Qemu-devel] [PATCH 09/24] Fix typo in comment (dieing -> dying)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- linux-user/signal.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/linux-user/signal.c b/linux-user/signal.c index ce033e9..6fe086b 100644 --- a/linux-user/signal.c +++ b/linux-user/signa

[Qemu-devel] [PATCH 13/24] Fix typos in comments (instanciation -> instantiation)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/ppc4xx_devs.c|2 +- target-ppc/translate_init.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/ppc4xx_devs.c b/hw/ppc4xx_devs.c index 7f9ed17..68bdfaa 100644 --- a/h

[Qemu-devel] [PATCH 15/24] Fix typos in comments and code (occured -> occurred and related)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil The code changed here is an unused data type name (evt_flush_occurred). Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- block.c|2 +- block/qcow2-refcount.c |2 +- cpu-all.h |2 +- cpu-exec.c |2 +- hw/bt

[Qemu-devel] [PATCH 19/24] Fix typos in comment (threshhold -> threshold, mapp -> map)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/xilinx_axidma.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/xilinx_axidma.c b/hw/xilinx_axidma.c index e32534f..571a5b0 100644 --- a/hw/xilinx_axidma.c +++ b/hw/xilinx_axidma.c

[Qemu-devel] [PATCH 11/24] Fix typos in comments (existance -> existence)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- libcacard/vscard_common.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libcacard/vscard_common.h b/libcacard/vscard_common.h index bebd52d..609ae98 100644 --- a/libcacard/vscard_common.

[Qemu-devel] [PATCH 10/24] Fix typos in comments (imediately -> immediately)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/lan9118.c |2 +- hw/syborg_serial.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/lan9118.c b/hw/lan9118.c index 2dc8d18..4c42fe9 100644 --- a/hw/lan9118.c +++ b/hw/lan91

[Qemu-devel] [PATCH 17/24] Fix typo in comment (responsiblity -> responsibility)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hw/pcie_aer.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/pcie_aer.c b/hw/pcie_aer.c index 0c4e8a5..f08d3c7 100644 --- a/hw/pcie_aer.c +++ b/hw/pcie_aer.c @@ -612,7 +612,7 @@ static

[Qemu-devel] [PATCH 02/24] Fix typos in comments (dependancy -> dependency)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- Changelog |2 +- Makefile.objs |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Changelog b/Changelog index 152feaa..1c41e14 100644 --- a/Changelog +++ b/Changelog @@ -525,7 +525,7

[Qemu-devel] [PATCH 22/24] linux-user: Replace deprecated function

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Function bzero is deprecated, so replace it by function memset. Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- linux-user/syscall.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 279c

[Qemu-devel] [PATCH 18/24] Fix typo in comment (truely -> truly)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- linux-user/mmap.c |2 +- target-cris/translate_v10.c |2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/linux-user/mmap.c b/linux-user/mmap.c index 0cf22f8..994c02b 100644 --- a

[Qemu-devel] [PULL] Trivial patches for April 26 to May 8 2011

2011-05-08 Thread Stefan Hajnoczi
The following changes since commit 3964f535c35c08470ac69bd553282af500bc8bb0: Merge remote-tracking branch 'mst/for_anthony' into staging (2011-05-05 13:05:32 -0500) are available in the git repository at: git://repo.or.cz/qemu/stefanha.git trivial-patches Hannes Reinecke (1): lsi53c8

[Qemu-devel] [PATCH 21/24] Fix spelling in comments (intruction -> instruction)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- bsd-user/main.c |2 +- linux-user/main.c |2 +- tests/test-i386.c |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/bsd-user/main.c b/bsd-user/main.c index 6b12f8b..0c3fca1 100644

[Qemu-devel] [PATCH 12/24] Fix typos in comments (interupt -> interrupt)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- cpu-exec.c |2 +- hw/mst_fpga.c|2 +- hw/pl031.c |2 +- hw/pl061.c |4 ++-- target-mips/translate_init.c |2 +- 5 files chan

[Qemu-devel] [PATCH 16/24] Fix typo in comment (relevent -> relevant)

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- hppa-dis.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hppa-dis.c b/hppa-dis.c index 49f99c8..a5760a95 100644 --- a/hppa-dis.c +++ b/hppa-dis.c @@ -1645,7 +1645,7 @@ static const char

[Qemu-devel] [PATCH 23/24] libcacard: add correct subdirectory dependencies

2011-05-08 Thread Stefan Hajnoczi
From: Paolo Bonzini Signed-off-by: Paolo Bonzini Cc: qemu-triv...@nongnu.org Signed-off-by: Stefan Hajnoczi --- Makefile |2 ++ Makefile.objs |7 +++ libcacard/Makefile |9 + 3 files changed, 6 insertions(+), 12 deletions(-) diff --git a/Makefile b/Makef

[Qemu-devel] [PATCH 24/24] lsi53c895a: Rename 'sense' to 'status'

2011-05-08 Thread Stefan Hajnoczi
From: Hannes Reinecke The 'sense' field in the HBA status structure is misnamed, as it actually carries the SCSI status. Rename it. Signed-off-by: Hannes Reinecke Reviewed-by: Stefan Hajnoczi Signed-off-by: Paolo Bonzini Cc: qemu-triv...@nongnu.org Signed-off-by: Stefan Hajnoczi --- hw/lsi5

[Qemu-devel] [PATCH 01/24] Fix typo in code and comments

2011-05-08 Thread Stefan Hajnoczi
From: Stefan Weil Replace writeable -> writable Signed-off-by: Stefan Weil Signed-off-by: Stefan Hajnoczi --- block.c |2 +- block/sheepdog.c |4 +- hw/eepro100.c|2 +- hw/eeprom93xx.c | 10 hw/msi.c

Re: [Qemu-devel] [PATCH 1/2] ppc64: Fix out-of-tree builds

2011-05-08 Thread Andreas Färber
Am 03.05.2011 um 12:18 schrieb Alexander Graf: On 01.05.2011, at 18:31, Andreas Färber wrote: Someone on Linux should check if a similar patch is necessary for roms/SLOF/: Mind to just send a patch I can try out with some instruction on how to trigger this? :) Well, using a build direct

Re: [Qemu-devel] [PATCH] USB: Fix build breakage in musb

2011-05-08 Thread Stefan Weil
Am 08.05.2011 17:54, schrieb Stefan Weil: Am 08.05.2011 00:59, schrieb Alexander Graf: Today's git failed building for me. It looks like a mismatching function prototype definition. This patch fixes compilation of arm-softmmu for me. Signed-off-by: Alexander Graf --- hw/usb-musb.c | 2 +- 1 fi

Re: [Qemu-devel] [PATCH 0/8] cpu interrupt cleanup, v2

2011-05-08 Thread Richard Henderson
Ping? On 05/04/2011 01:34 PM, Richard Henderson wrote: > v1->v2 > Rebase vs HEAD (d2d979c) > Cleanup whitespace errors. > > > r~ > > Richard Henderson (8): > irq: Introduce CPU_INTERRUPT_TGT_* defines. > irq: Introduce and use CPU_INTERRUPT_SSTEP_MASK. > target-mips: Do not check CPU_

Re: [Qemu-devel] [PATCH] virtio-9p: Remove statement without effect (fix warning from cppcheck)

2011-05-08 Thread Venkateswararao Jujjuri
On 05/07/2011 01:23 PM, Stefan Weil wrote: cppcheck report: virtio-9p.c:197: warning: Redundant assignment of "flags" to itself Cc: Venkateswararao Jujjuri Signed-off-by: Stefan Weil Reviewed-by: Venkateswararao Jujjuri BTW, New threading model will change these functions completely. But in

Re: [Qemu-devel] [PATCH] virtio-9p: Fix a memory leak

2011-05-08 Thread Venkateswararao Jujjuri
On 05/07/2011 01:27 PM, Pedro Scarapicchia Junior wrote: Hi Stefan, Thanks for the comment. I believe that it is possible to release the memory at v9fs_walk. However v9fs_walk_complete() is called from two another functions: v9fs_walk_post_newfid_lstat() and v9fs_walk_post_oldfid_lstat().

[Qemu-devel] [PATCH] ahci: Unbreak bar registration (was: Re: [PATCH 05/10] ich/ahci: convert to pci_register_bar_simple())

2011-05-08 Thread Jan Kiszka
On 2011-04-04 17:28, Avi Kivity wrote: > Signed-off-by: Avi Kivity > --- > hw/ide/ahci.c |9 - > hw/ide/ahci.h |3 --- > hw/ide/ich.c |3 +-- > 3 files changed, 1 insertions(+), 14 deletions(-) > > diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c > index 98bdf70..c6e0c77 100644 >

Re: [Qemu-devel] Allow ARMv7M to be started without a kernel

2011-05-08 Thread Rob Landley
On 05/08/2011 09:10 AM, Andreas Färber wrote: > Am 06.05.2011 um 14:48 schrieb Alexander Graf: > >> On 06.05.2011, at 01:50, Rob Landley wrote: >> >>> On 05/05/2011 06:26 PM, Alexander Graf wrote: > As an aside: I think QEMU should have an option which is "just load > a plain ELF or raw bi

Re: [Qemu-devel] [PATCH 0/8] cpu interrupt cleanup, v2

2011-05-08 Thread Blue Swirl
Thanks, applied. On Sun, May 8, 2011 at 7:50 PM, Richard Henderson wrote: > Ping? > > On 05/04/2011 01:34 PM, Richard Henderson wrote: >> v1->v2 >>   Rebase vs HEAD (d2d979c) >>   Cleanup whitespace errors. >> >> >> r~ >> >> Richard Henderson (8): >>   irq: Introduce CPU_INTERRUPT_TGT_* defines.

Re: [Qemu-devel] [PATCH] Clean up virtio-9p error handling code

2011-05-08 Thread Venkateswararao Jujjuri
On 05/08/2011 05:36 AM, Sassan Panahinejad wrote: In a lot of cases, the handling of errors was quite ugly. This patch moves reading of errno to immediately after the system calls and passes it up through the system more cleanly. Also, in the case of the xattr functions (and possibly others), co

Re: [Qemu-devel] Binary translation (of code)

2011-05-08 Thread Tarmo Pikaro
Hi ! > > I think self-modifying code is kinda rare case - it's made typically for > > protection againt hackers , and typically on pc side. Nintendo roms probably > > don't use this kind of > > protection. > > It doesn't have to be protection, for example GCC generates > trampolines to stack when

[Qemu-devel] ahci: crash after duplicate bh registration

2011-05-08 Thread Jan Kiszka
Hi Alex, I've seen crashes caused by ahci_check_cmd_bh unregistering a NULL bh. It looks like ahci_dma_set_inactive can a called while there is already a bh hanging around. Patch below cures the issue, but I have no clue if such an invocation order is valid at all. Jan --- diff --git a/hw/ide/a

Re: [Qemu-devel] Binary translation (of code)

2011-05-08 Thread Lluís
Tarmo Pikaro writes: > Hi ! >> > I think self-modifying code is kinda rare case - it's made typically for >> > protection againt hackers , and typically on pc side. Nintendo roms >> > probably >> > don't use this kind of >> > protection. >> >> It doesn't have to be protection, for example GCC ge

[Qemu-devel] [Bug 723871] Re: qemu-kvm-0.14.0 Aborts with -vga qxl

2011-05-08 Thread Boris Derzhavets
In spice session activated via virt-manager guest may run only in "VGA" mode. Setting video to "QXL" causes mouse to disappear in VM's window. -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/723871 Ti

[Qemu-devel] [PATCH] virtio: guard against negative vq notifies

2011-05-08 Thread Stefan Hajnoczi
The virtio_queue_notify() function checks that the virtqueue number is less than the maximum number of virtqueues. A signed comparison is used but the virtqueue number could be negative if a buggy or malicious guest is run. This results in memory accesses outside of the virtqueue array. It is ri

Re: [Qemu-devel] Supporting emulation of IOMMUs

2011-05-08 Thread David Gibson
On Thu, Apr 28, 2011 at 02:57:14PM -0700, Richard Henderson wrote: > On 04/21/2011 02:39 AM, Alexander Graf wrote: > > How exactly is this going to be used? Also, in the end I think that > > most devices should just go through a PCI specific interface that > > then calls the DMA helpers: > > > > p

Re: [Qemu-devel] Supporting emulation of IOMMUs

2011-05-08 Thread David Gibson
On Thu, Apr 21, 2011 at 11:39:22AM +0200, Alexander Graf wrote: > On 21.04.2011, at 09:03, David Gibson wrote: [snip] > > @@ -934,6 +939,8 @@ echo " --enable-docsenable documentation > > build" > > echo " --disable-docs disable documentation build" > > echo " --disable-vho

Re: [Qemu-devel] Supporting emulation of IOMMUs

2011-05-08 Thread David Gibson
On Fri, Apr 29, 2011 at 07:27:38AM -0700, Richard Henderson wrote: > On 04/28/2011 02:57 PM, Richard Henderson wrote: > > I've had a read through the patches posted in January. It all does > > seem relatively sane. At least, I can readily see how I would apply > > these interfaces to my Alpha por

Re: [Qemu-devel] [PATCH] qemu-kvm: Add CPUID support for VIA CPU

2011-05-08 Thread BrillyWu
> > I have submit a patch into upstream's KVM for supporting > these features before, and the patch has been applied in kvm-git. > > As far as I can see, nothing has been applied to any tree, neither > qemu.git nor qemu-kvm.git. Sorry, I have submit to kvm.git, not qemu.git or qemu-kvm.git. > >

[Qemu-devel] [PATCH uq/master] kvm: Add CPUID support for VIA CPU

2011-05-08 Thread BrillyWu
When KVM is running on VIA CPU with host cpu's model, the feautures of VIA CPU will be passed into kvm guest by calling the CPUID instruction for Centaur. Signed-off-by: BrillyWu Signed-off-by: KaryJin --- target-i386/cpu.h |3 +++ target-i386/cpuid.c | 46 ++

Re: [Qemu-devel] [PATCH v2] Import Linux headers for KVM and vhost

2011-05-08 Thread Rusty Russell
On Wed, 04 May 2011 10:55:26 +0200, Jan Kiszka wrote: > But one point of all this is to make life easier for those poor people > who have check a qemu source package before redistribution for included > licenses. The clearer the terms are expressed, the easier these > "unproductive" processes beco

[Qemu-devel] SeaBIOS image is lacking CONFIG_AHCI

2011-05-08 Thread Jan Kiszka
Hi Anthony, please rebuild SeaBIOS after enabling AHCI. QEMU's current version is still not able to boot from such controllers. Thanks, Jan signature.asc Description: OpenPGP digital signature

Re: [Qemu-devel] [PATCH uq/master] kvm: Add CPUID support for VIA CPU

2011-05-08 Thread Jan Kiszka
On 2011-05-09 07:42, brill...@viatech.com.cn wrote: > When KVM is running on VIA CPU with host cpu's model, the > feautures of VIA CPU will be passed into kvm guest by calling > the CPUID instruction for Centaur. > > Signed-off-by: BrillyWu > Signed-off-by: KaryJin > --- > target-i386/cpu.h |

Re: [Qemu-devel] [Qemu-trivial] [PATCH 4/4] fix compilation when reconfiguring without dtrace backend

2011-05-08 Thread Paolo Bonzini
On 05/08/2011 01:15 PM, Stefan Hajnoczi wrote: I think the real problem is that dependencies need to be regenerated after ./configure? There are three ways to do that, the trivial but wrong one, and the correct but overzealous one: - delete .d files. This forces regeneration of dependencies