[Qemu-devel] More build breakage: signrom.sh vs kvmvapic.bin

2012-03-31 Thread Brad Smith
Here is another source of build breakage that popped up over a month ago. The patch below fixes the build but I'm wondering why the file in question is being built on anything but Linux. It seems to be used for KVM support and thus has no relevance on any other OS. ASoptionrom/kvmvapic.o

[Qemu-devel] [PATCH 1/5] tests: fix out-of-tree building for lm32 target

2012-03-31 Thread Michael Walle
Signed-off-by: Michael Walle --- configure |4 +++- tests/tcg/lm32/Makefile | 13 - 2 files changed, 11 insertions(+), 6 deletions(-) diff --git a/configure b/configure index b51a749..4ef5ec6 100755 --- a/configure +++ b/configure @@ -3876,7 +3876,8 @@ echo "QEMU_

[Qemu-devel] [PULL 0/5] lm32 fixes and additions

2012-03-31 Thread Michael Walle
Hi Anthony or Blue, Please pull the following lm32 fixes and additions. The following changes since commit b7c8e15a146a7b20021b524f41d6b8072ee093b5: Merge branch 'arm-devs.for-upstream' of git://git.linaro.org/people/pmaydell/qemu-arm (2012-03-31 12:10:07 +) are available in the git repo

[Qemu-devel] No more win2k virtio-net drivers in latest Fedora

2012-03-31 Thread Reeted
Hello all, I have noticed that the virtio-net drivers for win2k exist in the 3 years' old: http://sourceforge.net/projects/kvm/files/kvm-driver-disc/20081229/NETKVM-20081229.iso/download but not in the newest: http://alt.fedoraproject.org/pub/alt/virtio-win/latest/images/bin/virtio-win-0.1-22

Re: [Qemu-devel] Bug(?): Can't unset vnc password (v1.0)

2012-03-31 Thread Reeted
On 03/30/12 16:00, Daniel P. Berrange wrote: On Fri, Mar 30, 2012 at 03:57:15PM +0200, Reeted wrote: In qemu-kvm 1.0 (from ubuntu Precise) I don't seem to be able to UNSET the vnc password once it has been set via qemu monitor. I can set it to "" which is empty password, but a VNC client connect

[Qemu-devel] [PATCH 5/5] milkymist-vgafb: add missing register

2012-03-31 Thread Michael Walle
This bug existed since the first commit. Fortunately, the affected registers have no functionality in qemu. This will only prevent the following warning: milkymist_vgafb: write access to unknown register 0x0034 Signed-off-by: Michael Walle --- hw/milkymist-vgafb.c |5 - 1 files cha

[Qemu-devel] [PATCH 3/5] milkymist-sysctl: support for new core version

2012-03-31 Thread Michael Walle
The new version introduces the following new registers: - SoC clock frequency: read-only of system clock used on the SoC - debug scratchpad: 8 bit scratchpad register - debug write lock: write once register, without any function on QEMU Signed-off-by: Michael Walle --- hw/milkymist-sysctl.c |

[Qemu-devel] [PATCH 2/5] target-lm32: init tcg only if available

2012-03-31 Thread Michael Walle
Once qtest support for target-lm32 arrives, tcg may be disabled. Signed-off-by: Michael Walle --- target-lm32/helper.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-lm32/helper.c b/target-lm32/helper.c index 5db8f8d..78076e4 100644 --- a/target-lm32/helper.c ++

[Qemu-devel] [PATCH 4/5] target-lm32: add simple disassembler

2012-03-31 Thread Michael Walle
Because binutils disassembler is based on libopcode, this is a rewrite from scratch. Signed-off-by: Michael Walle --- Makefile.objs |1 + configure |4 + dis-asm.h |3 + disas.c |6 + lm32-dis.c| 351 + 5

[Qemu-devel] [Bug 962880] Re: having a tr_TR.UTF-8 locale creates problems during compile

2012-03-31 Thread Stefan Weil
This was fixed in git master with commit bb55b712e8dc4d4eb515144d5c26798fea178cba. ** Changed in: qemu Status: Confirmed => Fix Committed -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/962880

[Qemu-devel] [PATCH] make qemu_event_handle static

2012-03-31 Thread Frediano Ziglio
Signed-off-by: Frediano Ziglio --- main-loop.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/main-loop.c b/main-loop.c index db23de0..fc738d1 100644 --- a/main-loop.c +++ b/main-loop.c @@ -164,7 +164,7 @@ static int qemu_signal_init(void) #else /* _WIN32 */ -HAND

[Qemu-devel] [PATCH 12/20] ppc: split exception helpers

2012-03-31 Thread Blue Swirl
Signed-off-by: Blue Swirl --- Makefile.target |5 +- target-ppc/excp_helper.c | 216 ++ target-ppc/op_helper.c | 193 - 3 files changed, 220 insertions(+), 194 deletions(-) create mode 100644 tar

[Qemu-devel] [PATCH 18/20] ppc: avoid AREG0 for misc helpers

2012-03-31 Thread Blue Swirl
Add an explicit CPUPPCState parameter instead of relying on AREG0. Signed-off-by: Blue Swirl --- Makefile.target |3 --- target-ppc/helper.h | 18 +- target-ppc/misc_helper.c| 19 +-- target-ppc/translate.c |2 +- target-pp

[Qemu-devel] [PATCH 15/20] ppc: split off timebase helpers

2012-03-31 Thread Blue Swirl
Move decrementer and timebase helpers to a dedicated file. Signed-off-by: Blue Swirl --- Makefile.target |5 +- target-ppc/op_helper.c | 135 --- target-ppc/timebase_helper.c | 160 ++ 3 files change

[Qemu-devel] [PATCH 19/20] ppc: move misc helpers from helper.c to misc_helper.c

2012-03-31 Thread Blue Swirl
Signed-off-by: Blue Swirl --- target-ppc/helper.c |9 - target-ppc/misc_helper.c |9 + 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/target-ppc/helper.c b/target-ppc/helper.c index 6b3a0cd..b38b5a8 100644 --- a/target-ppc/helper.c +++ b/target-ppc/hel

Re: [Qemu-devel] [PATCH] qemu/configure: fix CFLAGS handling for i386

2012-03-31 Thread Olaf Hering
On Sat, Mar 31, Peter Maydell wrote: > On 31 March 2012 16:41, Andreas Färber wrote: > > This is the only usage of += outside Makefile fragments, so I wonder if > > its use may have been by accident. Is it safe in a POSIX context? > > Or should we better use CFLAGS="$CFLAGS -march=486"? > > For Q

Re: [Qemu-devel] [PATCH] qemu/configure: fix CFLAGS handling for i386

2012-03-31 Thread Peter Maydell
On 31 March 2012 16:41, Andreas Färber wrote: > This is the only usage of += outside Makefile fragments, so I wonder if > its use may have been by accident. Is it safe in a POSIX context? > Or should we better use CFLAGS="$CFLAGS -march=486"? > For QEMU_CFLAGS we use the pattern QEMU_CFLAGS="-opti

Re: [Qemu-devel] [PATCH] qemu/configure: fix CFLAGS handling for i386

2012-03-31 Thread Olaf Hering
On Sat, Mar 31, Andreas Färber wrote: > This is the only usage of += outside Makefile fragments, so I wonder if > its use may have been by accident. Is it safe in a POSIX context? > Or should we better use CFLAGS="$CFLAGS -march=486"? Now that I look at the shebang, configure is a sh script and +

[Qemu-devel] [PATCH 17/20] ppc: split off misc helpers

2012-03-31 Thread Blue Swirl
Signed-off-by: Blue Swirl --- Makefile.target |5 ++- target-ppc/misc_helper.c | 116 ++ target-ppc/op_helper.c | 94 - 3 files changed, 120 insertions(+), 95 deletions(-) create mode 100644 target-

Re: [Qemu-devel] [PATCH] qemu/configure: fix CFLAGS handling for i386

2012-03-31 Thread Andreas Färber
Hi Olaf, Am 30.03.2012 17:24, schrieb Olaf Hering: > > configure will generate incorrect CFLAGS which will lead to compile > errors due to unknown gcc options, IFF CFLAGS was already in the > environment during configure invocation. > > Add a space before the -march=i486 gcc option. > > This pa

[Qemu-devel] [PATCH 16/20] ppc: avoid AREG0 for timebase helpers

2012-03-31 Thread Blue Swirl
Add an explicit CPUPPCState parameter instead of relying on AREG0. Signed-off-by: Blue Swirl --- Makefile.target |3 -- target-ppc/helper.h | 42 target-ppc/timebase_helper.c | 43 - ta

[Qemu-devel] [PATCH 13/20] ppc: avoid AREG0 for exception helpers

2012-03-31 Thread Blue Swirl
Add an explicit CPUPPCState parameter instead of relying on AREG0. Signed-off-by: Blue Swirl --- Makefile.target |3 -- target-ppc/excp_helper.c | 59 - target-ppc/fpu_helper.c |6 ++-- target-ppc/helper.h | 28 +++---

[Qemu-devel] [PATCH 11/20] ppc: cleanup MMU merge

2012-03-31 Thread Blue Swirl
Remove useless wrappers. In some cases 'int' parameters are changed to uint32_t. Make internal functions static. Signed-off-by: Blue Swirl --- target-ppc/cpu.h| 22 target-ppc/mmu_helper.c | 122 ++- 2 files changed, 36 insertions(

[Qemu-devel] [PATCH 09/20] ppc: avoid a warning with the next patch

2012-03-31 Thread Blue Swirl
When the code is moved together by the next patch, compiler detects a possible uninitialized variable use. Avoid the warning by initializing the variables. Signed-off-by: Blue Swirl --- target-ppc/mmu_helper.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/target-ppc

[Qemu-devel] [PATCH 00/20] PPC: AREG0 conversion

2012-03-31 Thread Blue Swirl
Split PPC op helpers, convert to AREG0 free operation. Survives simple tests. FPU and SPE as well as integer and vector ops could be split further, each to their own file. This can be done also later. It could be possible to move some code from hw/ppc.c to various helpers but that can also be don

[Qemu-devel] [PATCH] block/curl: Replace usleep by g_usleep

2012-03-31 Thread Stefan Weil
The function usleep is not available for all supported platforms: at least some versions of MinGW don't support it. usleep was also declared obsolete by POSIX.1-2001. The function g_usleep is part of glib2.0, so it is available for all supported platforms. Using nanosleep would also be possible

[Qemu-devel] [PATCH 3/3] usb-redir: Not finding an async urb id is not an error

2012-03-31 Thread Hans de Goede
We clear our pending async urb list on device disconnect and we may still receive "packet complete" packets from our peer after this, which will then refer to packet ids no longer in our list. Signed-off-by: Hans de Goede --- hw/usb/redirect.c |2 +- 1 file changed, 1 insertion(+), 1 deletio

[Qemu-devel] [PATCH 2/3] usb-redir: Reset device address and speed on disconnect

2012-03-31 Thread Hans de Goede
Without this disconnected devices look like the last redirected device in the monitor in "info usb". Signed-off-by: Hans de Goede --- hw/usb/redirect.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c index 9cea7c1..11d3b2b 100644 --- a/hw/usb/redire

[Qemu-devel] [PATCH 1/3] usb-redir: An interface count of 0 is a valid value

2012-03-31 Thread Hans de Goede
An interface-count of 0 happens when a device is in unconfigured state when it gets redirected. So we should not use 0 to detect not having received interface info from our peer. Signed-off-by: Hans de Goede --- hw/usb/redirect.c |5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) dif

[Qemu-devel] [PATCH] qtest: Add missing GCC_FMT_ATTR

2012-03-31 Thread Stefan Weil
gcc reports an error when the code is compiled with -Wmissing-format-attribute. Signed-off-by: Stefan Weil --- qtest.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/qtest.c b/qtest.c index cd7186c..53e2ed6 100644 --- a/qtest.c +++ b/qtest.c @@ -156,7 +156,8 @@ static

[Qemu-devel] [PATCH v3] Replace bdrv_* to bdrv_aio_* functions in DMA mode in fdc.c.

2012-03-31 Thread Li Zhi Hui
Replace bdrv_* to bdrv_aio_* functions in DMA mode in fdc.c. Signed-off-by: Li Zhi Hui --- hw/dma.c | 36 + hw/fdc.c | 260 +- hw/isa.h |1 + 3 files changed, 212 insertions(+), 85 deletions(-) diff --git a/hw/dma.c b/hw

[Qemu-devel] [PATCH v2] Replace bdrv_* to bdrv_aio_* functions in pio mode in fdc.c.

2012-03-31 Thread Li Zhi Hui
Replace bdrv_* to bdrv_aio_* functions in pio mode in fdc.c. Signed-off-by: Li Zhi Hui --- hw/fdc.c | 123 +- 1 files changed, 89 insertions(+), 34 deletions(-) diff --git a/hw/fdc.c b/hw/fdc.c index a0236b7..0d178d2 100644 --- a/hw/f

Re: [Qemu-devel] [PATCH v2] Fix conversion from lower to upper case with Turkish locale

2012-03-31 Thread Blue Swirl
Thanks, applied. On Tue, Mar 27, 2012 at 17:23, Stefan Weil wrote: > Some locale settings let make fail or create wrong results > because tr '[:lower:]' '[:upper:]' which is used to convert > from lower to upper case depends on the locale. > > With locale tr_TR.UTF-8, lower case 'i' is not conver

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

2012-03-31 Thread Blue Swirl
On Fri, Mar 30, 2012 at 10:39, Peter Maydell wrote: > Usual fortnightly pullreq for arm-devs: this just has Paolo's rtc > related changes. Please pull. Thanks, pulled. > > thanks > -- PMM > > The following changes since commit f638f0d3ae214d995cdd94578195700cda24597b: > >  qemu tcg: Remove one e

Re: [Qemu-devel] [PULL] QOM CPUState for unicore32

2012-03-31 Thread Blue Swirl
On Fri, Mar 30, 2012 at 10:26, Andreas Färber wrote: > Hello Anthony or Blue, > > Please pull the unicore32 QOM CPU conversion. Thanks, pulled. > Due to my alphabetical ordering in Makefile.target, it is independent > of Peter's upcoming ARM pull. > > Cc: Anthony Liguori > Cc: Blue Swirl > Cc:

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

2012-03-31 Thread Blue Swirl
On Fri, Mar 30, 2012 at 10:26, Peter Maydell wrote: > Hi; this is a pullreq for my target-arm queue; please pull. Thanks, pulled. > The following changes since commit f638f0d3ae214d995cdd94578195700cda24597b: > >  qemu tcg: Remove one entry of INDEX_op_ld_i64 from ppc_op_defs (2012-03-29 > 11:1

Re: [Qemu-devel] [PATCH] buildfix: check for old pod2man versions

2012-03-31 Thread Blue Swirl
Thanks, applied. On Thu, Mar 29, 2012 at 08:55, Gerd Hoffmann wrote: > Older pod2man don't have a --utf8 switch, check for this in conffigure > and use it only when present.  Fixes build on RHEL-5. > > Signed-off-by: Gerd Hoffmann > --- >  Makefile  |    1 - >  configure |    8 >  2 fil

Re: [Qemu-devel] [PATCH v3 2/2] block: disable I/O throttling on sync api

2012-03-31 Thread Zhi Yong Wu
On Fri, Mar 30, 2012 at 6:29 PM, Stefan Hajnoczi wrote: > On Fri, Mar 30, 2012 at 9:50 AM,   wrote: >> +     * When all sync I/O drivers are converted to async I/O, it will be >> restored >> +     * to the original state. > > The problem here is that guess_disk_lchs() uses bdrv_read(), not > whet

Re: [Qemu-devel] [PATCH v3 2/2] block: disable I/O throttling on sync api

2012-03-31 Thread Zhi Yong Wu
On Fri, Mar 30, 2012 at 6:29 PM, Stefan Hajnoczi wrote: > On Fri, Mar 30, 2012 at 9:50 AM,   wrote: >> +     * When all sync I/O drivers are converted to async I/O, it will be >> restored >> +     * to the original state. > > The problem here is that guess_disk_lchs() uses bdrv_read(), not > whet

[Qemu-devel] [PULL] slirp: cleanups and a performance boost

2012-03-31 Thread Jan Kiszka
The following changes since commit 1658dd32408c75e850b55ab0ff70d5cc402674d5: sparc: pass page aligned addresses to tlb_set_page (2012-03-27 19:46:42 +) are available in the git repository at: git://git.kiszka.org/qemu.git queues/slirp Jan Kiszka (2): w32/slirp: Undefine error const

[Qemu-devel] [PATCH] tests: remove .SECONDARY special target

2012-03-31 Thread Paolo Bonzini
The special target should not be needed anymore, and caused (perhaps due to a Make bug) a failure with "make -j2". In any case, the main makefile is a better place for such special targets rather than an included makefile. Reported-by: Luiz Capitulino Signed-off-by: Paolo Bonzini --- An

Re: [Qemu-devel] [PATCH] kvm: deassign irqs in reset path

2012-03-31 Thread Jan Kiszka
On 2012-03-30 21:18, Jason Baron wrote: > diff --git a/hw/device-assignment.c b/hw/device-assignment.c > index 89823f1..31aed17 100644 > --- a/hw/device-assignment.c > +++ b/hw/device-assignment.c > @@ -1609,10 +1609,32 @@ static void reset_assigned_device(DeviceState *dev) > { > PCIDevice *p

Re: [Qemu-devel] [PATCH] Better support for dma_addr_t variables

2012-03-31 Thread David Gibson
On Fri, Mar 30, 2012 at 11:34:25AM +0200, Andreas Färber wrote: > Am 30.03.2012 11:32, schrieb Andreas Färber: > > Am 27.03.2012 04:43, schrieb David Gibson: > >> A while back, we introduced the dma_addr_t type, which is supposed to > >> be used for bus visible memory addresses. At present, this i

Re: [Qemu-devel] [PATCH] Better support for dma_addr_t variables

2012-03-31 Thread David Gibson
On Fri, Mar 30, 2012 at 11:32:45AM +0200, Andreas Färber wrote: > Am 27.03.2012 04:43, schrieb David Gibson: > > A while back, we introduced the dma_addr_t type, which is supposed to > > be used for bus visible memory addresses. At present, this is an > > alias for target_phys_addr_t, but this wil