[Qemu-devel] [Bug 1379340] Re: qemu-kvm guest panic for AMD smp trusty guests

2014-11-15 Thread new23d
I have more-or-less the same underlying hardware and tested with [1]. Same results. Keen to help in any possible way. [1] http://kernel.ubuntu.com/~kernel-ppa/mainline/v3.13.11.11-trusty /linux- image-3.13.11-0313-generic_3.13.11-0313.20141336_amd64.deb -- You received this bug notif

[Qemu-devel] [Bug 1379340] Re: qemu-kvm guest panic for AMD smp trusty guests

2014-11-15 Thread new23d
Update. Upon testing the 'daily' build as had been suggested, I found that [1] appears to work for me in a stably. I'll update this thread if I find it crashing in the near future. [1] http://kernel.ubuntu.com/~kernel-ppa/mainline/daily/current/linux- image-3.18.0-999-generic_3.18.0-999.2014111521

Re: [Qemu-devel] [PATCH v2 11/21] iotests: Prepare for refcount_width option

2014-11-15 Thread Eric Blake
On 11/14/2014 06:06 AM, Max Reitz wrote: > Some tests do not work well with certain refcount widths (i.e. you > cannot create internal snapshots with refcount_width=1), so make those > widths unsupported. > > Furthermore, add another filter to _filter_img_create in common.filter > which filters ou

Re: [Qemu-devel] [PATCH v2 10/21] qcow2: refcount_order parameter for qcow2_create2

2014-11-15 Thread Eric Blake
On 11/14/2014 06:06 AM, Max Reitz wrote: > Add a refcount_order parameter to qcow2_create2(), use that value for > the image header and for calculating the size required for > preallocation. > > For now, always pass 4. > > Signed-off-by: Max Reitz > --- > block/qcow2.c | 41

Re: [Qemu-devel] [PATCH v2 09/21] qcow2: Open images with refcount order != 4

2014-11-15 Thread Eric Blake
On 11/14/2014 06:06 AM, Max Reitz wrote: > No longer refuse to open images with a different refcount entry width > than 16 bits; only reject images with a refcount width larger than 64 > bits (which is prohibited by the specification). > > Signed-off-by: Max Reitz > --- > block/qcow2.c | 8 -

Re: [Qemu-devel] [PATCH v2 08/21] qcow2: More helpers for refcount modification

2014-11-15 Thread Eric Blake
On 11/14/2014 06:06 AM, Max Reitz wrote: > Add helper functions for getting and setting refcounts in a refcount > array for any possible refcount order, and choose the correct one during > refcount initialization. > > Signed-off-by: Max Reitz > --- > block/qcow2-refcount.c | 146 > +

Re: [Qemu-devel] [PATCH v2 07/21] qcow2: Helper function for refcount modification

2014-11-15 Thread Eric Blake
On 11/14/2014 06:06 AM, Max Reitz wrote: > Since refcounts do not always have to be a uint16_t, all refcount blocks > and arrays in memory should not have a specific type (thus they become > pointers to void) and for accessing them, two helper functions are used > (a getter and a setter). Those fun

Re: [Qemu-devel] [PATCH v2 06/21] qcow2: Helper for refcount array reallocation

2014-11-15 Thread Eric Blake
On 11/14/2014 06:05 AM, Max Reitz wrote: > Add a helper function for reallocating a refcount array, independently s/independently/independent/ > of the refcount order. The newly allocated space is zeroed and the > function handles failed reallocations gracefully. This patch is doing two things:

Re: [Qemu-devel] [PATCH v2 02/21] qcow2: Add refcount_width to format-specific info

2014-11-15 Thread Eric Blake
On 11/14/2014 06:05 AM, Max Reitz wrote: > Add the bit width of every refcount entry to the format-specific > information. > > In contrast to lazy_refcounts and the corrupt flag, this should be > always emitted, even for compat=0.10 although it does not support any > refcount width other than 16 b

Re: [Qemu-devel] [PATCH v2 21/21] iotests: Add test for different refcount widths

2014-11-15 Thread Eric Blake
On 11/14/2014 06:06 AM, Max Reitz wrote: > Add a test for conversion between different refcount widths and errors > specific to certain widths (i.e. snapshots with refcount_width=1). > > Signed-off-by: Max Reitz > --- > tests/qemu-iotests/112 | 252 >

[Qemu-devel] [PULL for-2.2] Update OpenBIOS images

2014-11-15 Thread Mark Cave-Ayland
Hi Peter, This update fixes a regression with the interrupt mapping for SPARC64 which broke virtio. Please pull. ATB, Mark. The following changes since commit 4e70f9271dabc58fbf14680843bfac510c193152: Merge remote-tracking branch 'remotes/sstabellini/xen-2014-11-14' into staging (2014-11

[Qemu-devel] [Bug 661696] Re: incomplete emulation of fstenv under TCG

2014-11-15 Thread Stefan Weil
** Changed in: qemu Status: New => 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/661696 Title: incomplete emulation of fstenv under TCG Status in QEMU: Confirmed Bug descriptio

[Qemu-devel] [Bug 1248376] Re: "fstenv" gets the wrong result in qemu 1.6.1

2014-11-15 Thread Stefan Weil
*** This bug is a duplicate of bug 661696 *** https://bugs.launchpad.net/bugs/661696 ** This bug has been marked a duplicate of bug 661696 incomplete emulation of fstenv under TCG -- You received this bug notification because you are a member of qemu- devel-ml, which is subscribed to QEMU

Re: [Qemu-devel] [PATCH] target-ppc: Altivec's mtvscr Decodes Wrong Register

2014-11-15 Thread Alexander Graf
On 14.11.14 21:01, Tom Musta wrote: > The Move to Vector Status and Control Register (mtvscr) instruction > uses VRB as the source register. Fix the code generator to correctly > decode the VRB field. That is, use "rB(ctx->opcode)" instead of > "rD(ctx->opcode)". > > Signed-off-by: Tom Musta

[Qemu-devel] [PATCH v4 2/3] target-i386: x87 exception pointers using TCG.

2014-11-15 Thread Jaume Marti Farriol
This modifies the helpers for instructions fstenv/fnstenv, fsave, fxsave, fldenv, frstor and fxrstor, to store to memory and restore from memory the x87 exception pointers. Also it add logic that efficiently stores the values of the exception pointers, to be used in the instructions mentioned above

[Qemu-devel] [PATCH v4 3/3] target-i386: x87 exception pointers using TCG.

2014-11-15 Thread Jaume Marti Farriol
This adds tests for the x87 exception pointers. Signed-off-by: jaume.mar...@gmail.com --- tests/tcg/test-i386.c | 75 ++- 1 file changed, 69 insertions(+), 6 deletions(-) diff --git a/tests/tcg/test-i386.c b/tests/tcg/test-i386.c index b05572b..a18

[Qemu-devel] [PATCH v4 1/3] target-i386: x87 exception pointers using TCG.

2014-11-15 Thread Jaume Marti Farriol
This adds new fields in the CPUX86State struct to store the x87 exception pointers. Also it adds a new enum type that encodes the operand size and the processor operating mode (protected and real mode). The patch also adds a new option (tcg-exception-pointers) in the configure script to enable or

[Qemu-devel] [PATCH v4 0/3] target-i386: x87 exception pointers using TCG.

2014-11-15 Thread Jaume Marti Farriol
I submit a patch to fix bugs 661696 and 1248376. This is the fourth version of this patch. As mentioned in a previous email, the patch implements, for TCG, the specifications provided in Intel and AMD programmer's manuals regarding the x87 exception pointers. That is, when executing instructions

[Qemu-devel] [PATCH 9/9] hcd-musb: fix dereference null return value

2014-11-15 Thread arei.gonglei
From: Gonglei Signed-off-by: Gonglei --- hw/usb/hcd-musb.c | 4 1 file changed, 4 insertions(+) diff --git a/hw/usb/hcd-musb.c b/hw/usb/hcd-musb.c index 66bc61a..f2cb73c 100644 --- a/hw/usb/hcd-musb.c +++ b/hw/usb/hcd-musb.c @@ -624,6 +624,10 @@ static void musb_packet(MUSBState *s, MUSBE

[Qemu-devel] [PATCH 0/9] Fix Coverity warning reports

2014-11-15 Thread arei.gonglei
From: Gonglei Those reports come from scan.coverity.com for Qemu, which Paolo told me. There are so many defects that I can't repair all of them. That's will be great if more and more people to join us. :) Cc: Paolo Bonzini Cc: zhanghailiang Cc: qemu-triv...@nongnu.org Gonglei (9): l2tpv3:

[Qemu-devel] [PATCH 2/9] mips_mipssim: fix use-after-free for filename

2014-11-15 Thread arei.gonglei
From: Gonglei May pass freed pointer filename as an argument to error_report. Signed-off-by: Gonglei --- hw/mips/mips_mipssim.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/mips/mips_mipssim.c b/hw/mips/mips_mipssim.c index 7ea0b9a..5d44c3f 100644 --- a/hw/mips/mips_m

[Qemu-devel] [PATCH 1/9] l2tpv3: fix fd leak

2014-11-15 Thread arei.gonglei
From: Gonglei In this false branch, fd will leak when it is zero. Change the testing condition. Signed-off-by: Gonglei --- net/l2tpv3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/net/l2tpv3.c b/net/l2tpv3.c index 528d95b..b2b0fc0 100644 --- a/net/l2tpv3.c +++ b/net/l2t

[Qemu-devel] [PATCH 8/9] shpc: fix dead code

2014-11-15 Thread arei.gonglei
From: Gonglei Signed-off-by: Gonglei --- hw/pci/shpc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hw/pci/shpc.c b/hw/pci/shpc.c index 65b2f51..9a39060 100644 --- a/hw/pci/shpc.c +++ b/hw/pci/shpc.c @@ -559,8 +559,9 @@ void shpc_device_hot_unplug_request_cb(HotplugHand

[Qemu-devel] [PATCH 5/9] nvme: remove superfluous check

2014-11-15 Thread arei.gonglei
From: Gonglei Operands don't affect result (CONSTANT_EXPRESSION_RESULT) ((n->bar.aqa >> AQA_ASQS_SHIFT) & AQA_ASQS_MASK) > 4095 is always false regardless of the values of its operands. This occurs as the logical second operand of '||'. Signed-off-by: Gonglei --- hw/block/nvme.c | 3 +-- 1 fil

[Qemu-devel] [PATCH 3/9] qga: fix false negative argument passing

2014-11-15 Thread arei.gonglei
From: Gonglei Function send_response(s, &qdict->base) returns a negative number when any faliures occured. But strerror()'s parameter cannot be negative. Let's change the testing condition and pass '-ret' to strerr(). Signed-off-by: Gonglei --- qga/main.c | 4 ++-- 1 file changed, 2 insertions

[Qemu-devel] [PATCH 6/9] acl: fix memory leak

2014-11-15 Thread arei.gonglei
From: Gonglei If 'i != index' for all acl->entries, variable entry leaks the storage it points to. Signed-off-by: Gonglei --- util/acl.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/util/acl.c b/util/acl.c index 938b7ae..571d686 100644 --- a/util/acl.c +++ b/u

[Qemu-devel] [PATCH 4/9] loader: fix NEGATIVE_RETURNS

2014-11-15 Thread arei.gonglei
From: Gonglei lseek will return -1 on error, g_malloc0(size) and read(,,size) paramenters cannot be negative. We should add a check for return value of lseek(). Signed-off-by: Gonglei --- hw/core/loader.c | 13 + 1 file changed, 13 insertions(+) diff --git a/hw/core/loader.c b/hw/

[Qemu-devel] [PATCH 7/9] qemu-char: fix MISSING_COMMA

2014-11-15 Thread arei.gonglei
From: Gonglei Signed-off-by: Gonglei --- qemu-char.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qemu-char.c b/qemu-char.c index bd0709b..4a76f0f 100644 --- a/qemu-char.c +++ b/qemu-char.c @@ -464,7 +464,7 @@ static const char * const mux_help[] = { "% hprint th