[Qemu-devel] [Bug 1037606] Re: vmwgfx does not work with kvm vmware vga

2014-03-15 Thread Andrew Engelbrecht
the bug on the kernel bug tracker is marked as "resolved obsolete", not "confirmed". -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU. https://bugs.launchpad.net/bugs/1037606 Title: vmwgfx does not work with kvm vmware vga Status in

[Qemu-devel] [Bug 1037606] Re: vmwgfx does not work with kvm vmware vga

2014-03-15 Thread Andrew Engelbrecht
i am also having this problem on amd64 debian sid, using: qemu1.7.0+dfsg-5 on the guest system, i'm also running sid: linux-image-amd64 3.13+56 xserver-xorg-video-vmware 1:13.0.1-3+b1 i added vmwgfx to /etc/modules, trying to follow th

[Qemu-devel] [Bug 788734] Re: vmwgfx does not work in Linux guest in kvm

2014-03-15 Thread Andrew Engelbrecht
*** This bug is a duplicate of bug 1037606 *** https://bugs.launchpad.net/bugs/1037606 i am also having this problem on amd64 debian sid. using qemu verion 1.7.0+dfsg-5. on the guest system, i'm also running sid. i have version 1:13.0.1-3+b1 of xserver-xorg-video-vmware, and kernel version 3.

Re: [Qemu-devel] [PATCH] target-alpha: fix subl and s8subl indentation

2014-03-15 Thread Peter Maydell
On 15 March 2014 18:33, Paolo Bonzini wrote: > Two missing braces, one close and one open, fabulously let the code > compile. > > Signed-off-by: Paolo Bonzini > --- > target-alpha/translate.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/target-alpha/translate.c b/ta

Re: [Qemu-devel] [PATCH] qcow2: Fix fail path in realloc_refcount_block()

2014-03-15 Thread Benoît Canet
The Saturday 15 Mar 2014 à 21:55:54 (+0100), Max Reitz wrote : > If qcow2_alloc_clusters() fails, new_offset and ret will both be > negative after the fail label, thus passing the first if condition and > subsequently resulting in a call of qcow2_free_clusters() with an > invalid (negative) offset

Re: [Qemu-devel] [PATCH] qcow2: Fix fail path in realloc_refcount_block()

2014-03-15 Thread Benoît Canet
The Saturday 15 Mar 2014 à 21:55:54 (+0100), Max Reitz wrote : > If qcow2_alloc_clusters() fails, new_offset and ret will both be > negative after the fail label, thus passing the first if condition and > subsequently resulting in a call of qcow2_free_clusters() with an > invalid (negative) offset

[Qemu-devel] Multiple pci buses

2014-03-15 Thread BALATON Zoltan
Hello, I'm trying to change hw/ppc/mac_newworld.c and hw/pci-host/uninorth.c to bring the mac99 machine type closer to what's seen in these dumps: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=604134 http://nandra.segv.jp/NetBSD/G4.dump-device-tree.txt http://raveland.org/ports/eeprom.txt

[Qemu-devel] [PATCH] qcow2: Fix fail path in realloc_refcount_block()

2014-03-15 Thread Max Reitz
If qcow2_alloc_clusters() fails, new_offset and ret will both be negative after the fail label, thus passing the first if condition and subsequently resulting in a call of qcow2_free_clusters() with an invalid (negative) offset parameter. Fix this by checking for new_offset being positive instead.

Re: [Qemu-devel] [PATCH] target-alpha: fix subl and s8subl indentation

2014-03-15 Thread Richard Henderson
On 03/15/2014 11:33 AM, Paolo Bonzini wrote: > Two missing braces, one close and one open, fabulously let the code > compile. > > Signed-off-by: Paolo Bonzini Wow. That's ... amazing. Reviewed-by: Richard Henderson r~

Re: [Qemu-devel] [PATCH v4 0/2] convert -m to QemuOpts

2014-03-15 Thread Igor Mammedov
On Thu, 06 Mar 2014 20:32:39 +0100 Andreas Färber wrote: > mjt, [...] > Since I believe you have a track record of reviewing and queuing such > QemuOpts conversions, can you review and take this for 2.0 please? ping > > Regards, > Andreas > > -- > SUSE LINUX Products GmbH, Maxfeldstr. 5, 9040

Re: [Qemu-devel] [PATCH V3 3/3] qemu-iotests: Add 088 new test for drive-mirror-replace.

2014-03-15 Thread Max Reitz
On 14.03.2014 16:18, Benoît Canet wrote: Tests for drive-mirror-replace whose purpose is to enable quorum file mirroring and replacement after failure. Signed-off-by: Benoit Canet --- tests/qemu-iotests/041| 34 +-- tests/qemu-iotests/088| 219 +++

Re: [Qemu-devel] [PATCH V3 2/3] block: Add drive-mirror-replace command to repair quorum files.

2014-03-15 Thread Max Reitz
On 14.03.2014 16:18, Benoît Canet wrote: When a quorum file is totally destroyed (broken NAS or SAN) the user can start a drive-mirror job on the quorum block backend and then replace the broken quorum file with drive-mirror-replace given it has a node-name. Signed-off-by: Benoit Canet --- bl

Re: [Qemu-devel] [PATCH] target-arm: Add ARM_CP_IO notation to PMCR reginfo

2014-03-15 Thread Laurent Desnogues
On Sat, Mar 15, 2014 at 7:31 PM, Peter Maydell wrote: > Now that the PMCR writefn makes timer accesses, its reginfo needs > the ARM_CP_IO flag, so that icount mode works correctly. (Fixes > the bug accidentally introduced in commit 7c2cb42b). > > Reported-by: Laurent Desnogues > Signed-off-by: Pe

[Qemu-devel] [PATCH] target-alpha: fix subl and s8subl indentation

2014-03-15 Thread Paolo Bonzini
Two missing braces, one close and one open, fabulously let the code compile. Signed-off-by: Paolo Bonzini --- target-alpha/translate.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/target-alpha/translate.c b/target-alpha/translate.c index a9ef1a7..e7e319b 100644 --- a/tar

[Qemu-devel] [PATCH] target-arm: Add ARM_CP_IO notation to PMCR reginfo

2014-03-15 Thread Peter Maydell
Now that the PMCR writefn makes timer accesses, its reginfo needs the ARM_CP_IO flag, so that icount mode works correctly. (Fixes the bug accidentally introduced in commit 7c2cb42b). Reported-by: Laurent Desnogues Signed-off-by: Peter Maydell --- target-arm/helper.c | 1 + 1 file changed, 1 ins

Re: [Qemu-devel] [PULL 00/14] tcg/aarch64 improvements, part 2

2014-03-15 Thread Peter Maydell
On 14 March 2014 20:09, Richard Henderson wrote: > Recently reviewed. > > It'd be nice if this makes -rc1, since it significantly improves code > generation for the host. But it's no big deal if it's too late. > > > r~ > > > The following changes since commit b19fc63cadb2815d5bcb1ec25b22849e455cb

Re: [Qemu-devel] [PULL for-2.0 0/6] Block patches

2014-03-15 Thread Peter Maydell
On 14 March 2014 16:14, Stefan Hajnoczi wrote: > Bug fixes for the next QEMU 2.0 release candidate. > > The following changes since commit 5d92c74f8a1728a202ba9457872ab0f27ff15e81: > > Update version for v2.0-rc0 (2014-03-13 20:08:15 -0700) > > are available in the git repository at: > > g...@

[Qemu-devel] [PATCH v3 6/7] allwinner-emac: set autonegotiation complete bit on link up

2014-03-15 Thread Beniamino Galvani
Signed-off-by: Beniamino Galvani --- hw/net/allwinner_emac.c |4 ++-- include/hw/net/allwinner_emac.h |1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/hw/net/allwinner_emac.c b/hw/net/allwinner_emac.c index 469f2f0..91931ac 100644 --- a/hw/net/allwinner_emac.c

[Qemu-devel] [PATCH v3 7/7] allwinner-emac: update irq status after writes to interrupt registers

2014-03-15 Thread Beniamino Galvani
The irq line status must be updated after writes to the INT_CTL and INT_STA registers. Signed-off-by: Beniamino Galvani Reviewed-by: Peter Crosthwaite --- hw/net/allwinner_emac.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/net/allwinner_emac.c b/hw/net/allwinner_emac.c index 919

[Qemu-devel] [PATCH v3 5/7] allwinner-a10-pit: implement prescaler and source selection

2014-03-15 Thread Beniamino Galvani
This implements the prescaler and source fields of the timer control register as described in the A10 user manual. Signed-off-by: Beniamino Galvani --- hw/timer/allwinner-a10-pit.c | 30 +- include/hw/timer/allwinner-a10-pit.h |8 2 files change

[Qemu-devel] [PATCH v3 2/7] allwinner-a10-pic: fix behaviour of pending register

2014-03-15 Thread Beniamino Galvani
The pending register is read-only and the value returned upon a read reflects the state of irq input pins (interrupts are level triggered). This patch implements such behaviour. Signed-off-by: Beniamino Galvani --- hw/intc/allwinner-a10-pic.c |8 +++- 1 file changed, 7 insertions(+), 1 d

[Qemu-devel] [PATCH v3 3/7] allwinner-a10-pit: avoid generation of spurious interrupts

2014-03-15 Thread Beniamino Galvani
The model was generating interrupts for all enabled timers after the expiration of one of them. Avoid this by passing explicitly the timer index to the callback function. Signed-off-by: Beniamino Galvani --- hw/timer/allwinner-a10-pit.c | 25 ++--- include/hw/timer/

[Qemu-devel] [PATCH v3 1/7] allwinner-a10-pic: set vector address when an interrupt is pending

2014-03-15 Thread Beniamino Galvani
This patch implements proper updating of the vector register which should hold, according to the A10 user manual, the vector address for the interrupt currently active on the CPU IRQ input. Interrupt priority is not implemented at the moment and thus the first pending interrupt is returned. Signe

[Qemu-devel] [PATCH v3 4/7] allwinner-a10-pit: use level triggered interrupts

2014-03-15 Thread Beniamino Galvani
Convert the interrupt generation logic to the use of level triggered interrupts. Signed-off-by: Beniamino Galvani --- hw/timer/allwinner-a10-pit.c | 13 - 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/hw/timer/allwinner-a10-pit.c b/hw/timer/allwinner-a10-pit.c index

[Qemu-devel] [PATCH v3 0/7] Allwinner A10 fixes

2014-03-15 Thread Beniamino Galvani
This series introduces some fixes and missing features found while trying to run mainline Linux kernel on emulated Allwinner A10. Most of the changes concern interrupt handling, but there are also improvements in the timer and the ethernet MAC. With this applied I'm able to boot Linux 3.14-rc2 us

[Qemu-devel] [PULL 09/13] tests: Fix 'make test' for i686 hosts (build regression)

2014-03-15 Thread Michael Tokarev
From: Stefan Weil 'make test' is broken at least since commit baacf04799ace72a9c735dd9306a1ceaf305e7cf. Several source files were moved to util/, and some of them there split, so add the missing prefix and new files to fix the compiler and linker errors. There remain more issues, but these chang

[Qemu-devel] [PULL 12/13] scripts/make-release: Don't distribute .git directories

2014-03-15 Thread Michael Tokarev
From: Cole Robinson [crobinso@localhost qemu-2.0.0-rc0]$ find . -name .git ./dtc/.git ./pixman/.git This is already done for the rom submodules. https://bugs.launchpad.net/qemu/+bug/1224414 Signed-off-by: Cole Robinson Signed-off-by: Michael Tokarev --- scripts/make-release |2 +- 1 file

[Qemu-devel] [PULL 10/13] audio: Add 'static' attributes to several variables

2014-03-15 Thread Michael Tokarev
From: Stefan Weil Signed-off-by: Stefan Weil Signed-off-by: Michael Tokarev --- hw/audio/fmopl.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hw/audio/fmopl.c b/hw/audio/fmopl.c index f0a0234..290a224 100644 --- a/hw/audio/fmopl.c +++ b/hw/audio/fmopl.c @@ -223,

[Qemu-devel] [PULL 11/13] configure: Don't use __int128_t for clang versions before 3.2

2014-03-15 Thread Michael Tokarev
From: Stefan Weil Those versions don't fully support __int128_t. Cc: qemu-sta...@nongnu.org Signed-off-by: Stefan Weil Signed-off-by: Michael Tokarev --- configure |5 + 1 file changed, 5 insertions(+) diff --git a/configure b/configure index 8c2838e..2bc6b77 100755 --- a/configure +

[Qemu-devel] [PULL 06/13] hw/timer/grlib_gptimer: Avoid integer overflows

2014-03-15 Thread Michael Tokarev
From: Sebastian Huber The GPTIMER uses 32-bit registers. Use a 64-bit operation to get the ptimer count, otherwise we end up with a count of 0 for GPTIMER counter values of 0x. Use the GPTIMER counter value for tracing to avoid an overflow of the 32-bit value passed to trace_grlib_gptim

[Qemu-devel] [PULL 07/13] Add qga/qapi-generated to .gitignore

2014-03-15 Thread Michael Tokarev
From: "Gabriel L. Somlo" The folder "qga/qapi-generated" shows up after building QEMU, and gets in the way during e.g. "git add ."; Add it to .gitignore to keep it from accidentally ending up in the wrong place. Signed-off-by: Gabriel Somlo Signed-off-by: Michael Tokarev --- .gitignore |1

[Qemu-devel] [PULL 08/13] misc: Fix typos in comments

2014-03-15 Thread Michael Tokarev
From: Stefan Weil Codespell found and fixed these new typos: * doesnt -> doesn't * funtion -> function * perfomance -> performance * remaing -> remaining A coding style issue (line too long) was fixed manually. Signed-off-by: Stefan Weil Reviewed-by: Andreas Färber Signed-off-by: Michael Tok

[Qemu-devel] [PULL 13/13] FSL eTSEC: Fix typo in rx ring

2014-03-15 Thread Michael Tokarev
From: Fabien Chouteau Signed-off-by: Fabien Chouteau Signed-off-by: Michael Tokarev --- hw/net/fsl_etsec/rings.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/net/fsl_etsec/rings.c b/hw/net/fsl_etsec/rings.c index 26b71f6..e36cfbe 100644 --- a/hw/net/fsl_etsec/r

[Qemu-devel] [PULL 03/13] .travis.yml: re-enable lttng user space trace test

2014-03-15 Thread Michael Tokarev
From: Alex Bennée This build was disabled while the lttng tracing was broken. Stefan has recently submitted a pull request with it re-enabled. Signed-off-by: Alex Bennée Reviewed-by: Stefan Hajnoczi Signed-off-by: Michael Tokarev --- .travis.yml |9 - 1 file changed, 4 insertions

[Qemu-devel] [PULL 01/13] sasl: Avoid 'Could not find keytab file' in syslog

2014-03-15 Thread Michael Tokarev
From: Laszlo Ersek The "keytab" specification in "qemu.sasl" only makes sense if "gssapi" is selected in "mech_list". Even if the latter is not done (ie. "gssapi" is not selected), the cyrus-sasl library tries to open the specified keytab file, although nothing has a use for it outside the gssapi

[Qemu-devel] [PULL 02/13] .travis.yml: add a new build target with non-core devlibs

2014-03-15 Thread Michael Tokarev
From: Alex Bennée The current builds don't include all the features which are auto-detected and then disabled when the appropriate test packages don't exist. I've added another target that enables all known additional packages for increased coverage. I didn't add it to the core package list to re

[Qemu-devel] [PULL 04/13] .travis.yml: trivial whitespace fixup

2014-03-15 Thread Michael Tokarev
From: Alex Bennée Purely cosmetic but satisfies my OCD. Signed-off-by: Alex Bennée Reviewed-by: Stefan Hajnoczi Signed-off-by: Michael Tokarev --- .travis.yml | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/.travis.yml b/.travis.yml

[Qemu-devel] [PULL 00/13] Trivial patches for 2014-03-15

2014-03-15 Thread Michael Tokarev
Here's another pull request for the trivial-patches queue. This time it does not include Peter's shifts-to-sign-bit patchset, but I'd hope to get it included in the next few days. All the changes appears to be fine for 2.0 (including the shifts-to-sign-bit series actually). Please consider applyi

[Qemu-devel] [PULL 05/13] .travis.yml: add IRC notifications for build failures

2014-03-15 Thread Michael Tokarev
From: Alex Bennée I'm trying to avoid spamming the IRC channel (not overly likely as builds take a while). So failure will always be reported but if the build continues to work then the IRC notifications will be quiet. Note any GitHub based repository with Travis enabled will use this notificati

[Qemu-devel] [PATCH v2] convert fprintf() calls to error_setg() in block/qed.c:bdrv_qed_create()

2014-03-15 Thread Aakriti Gupta
This patch converts fprintf() calls to error_setg() in block/qed.c:bdrv_qed_create() (error_setg() is part of error reporting API in include/qapi/error.h) Signed-off-by: Aakriti Gupta --- block/qed.c |8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/block/qed.c b/bl