From: Wenchao Xia xiaw...@linux.vnet.ibm.com
This patch change all info call back function to take
additional QDict * parameter, which allow those command
take parameter. Now it is set to NULL at default case.
Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com
Signed-off-by: Luiz Capitulino
From: Amos Kong ak...@redhat.com
We will send vq command to set mac address in virtnet_set_mac_address()
a little fix of coding style
Signed-off-by: Amos Kong ak...@redhat.com
---
drivers/net/virtio_net.c | 89
1 file changed, 44 insertions(+),
Signed-off-by: Igor Mammedov imamm...@redhat.com
Reviewed-by: Eduardo Habkost ehabk...@redhat.com
---
target-i386/cpu.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/target-i386/cpu.c b/target-i386/cpu.c
index 333745b..ce914da 100644
--- a/target-i386/cpu.c
+++
It's rebase of remnants of [PATCH qom-cpu 00/17] x86 CPU cleanup, part 3 [1]
taking in account comments of reviewers.
This series is several cleanups, moved out from CPU properties series,
since they do not really depend on CPU properties re-factoring and could
simplify CPU subclasses work as
commit 8935499831312 makes cpuid return to guest host's vendor value
instead of built-in one by default if kvm_enabled() == true and allows
to override this behavior if 'vendor' is specified on -cpu command line.
But every time guest calls cpuid to get 'vendor' value, host's value is
read again
Setting tsc-frequency from x86_def_t is NOP because default tsc_khz
in x86_def_t is 0 and CPUX86State.tsc_khz is also initialized to 0
by default. So there is not need to set ovewrite tsc_khz with default
0 because field was already initialized to 0.
custom tsc-frequency setting is not affected
Move custom features parsing after built-in cpu_model defaults are set
and set custom features directly on CPU instance. That allows to make
clear distinction between built-in cpu model defaults that eventually
should go into clas_init() and extra property setting which is done
after defaults are
Juan Quintela quint...@redhat.com writes:
This is the intersect of the paolo me patches for migration thread,
Changes from 2 days ago:
- spelling check from Eric
- put the commit that patch refered from (Eric)
- drop the buffered rename at paolo request.
Please, pull.
If these were just
From: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: Juan Quintela quint...@redhat.com
Reviewed-by: Reviewed-by: Eric Blake ebl...@redhat.com
---
migration.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/migration.c
From: Wenchao Xia xiaw...@linux.vnet.ibm.com
Now info command takes a table of sub info commands,
and changed do_info() to do_info_help() to do help funtion
only.
Note that now info unknown-topic returns error instead
of list of info topics.
Signed-off-by: Wenchao Xia
From: Wenchao Xia xiaw...@linux.vnet.ibm.com
Because mon_cmds may use info_cmds, so adjust the declare sequence
of them.
Signed-off-by: Wenchao Xia xiaw...@linux.vnet.ibm.com
Signed-off-by: Luiz Capitulino lcapitul...@redhat.com
---
monitor.c | 12 ++--
1 file changed, 6
From: Wenchao Xia xiaw...@linux.vnet.ibm.com
This patch make parsing of hmp command aware of that it may
have sub command. Also discard simple encapsulation function
monitor_find_command(). For case @command , space after
@command is filtered out.
Signed-off-by: Wenchao Xia
On Thu, Jan 17, 2013 at 04:16:31PM +0100, Igor Mammedov wrote:
Vendor property setter takes string as vendor value but cpudefs
use uint32_t vendor[123] fields to define vendor value. It makes it
difficult to unify and use property setter for values from cpudefs.
Simplify code by using vendor
Hi Michael,
To trigger this I just:
1. Boot the VM, command-line:
# ./qemu -drive file=disks/test.img,if=virtio,cache=none,aio=native \
-enable-kvm -m 1G -monitor stdio \
-netdev type=tap,vhost=on,id=guest0,script=qemu-ifup-switch \
-device
On Thu, Jan 17, 2013 at 04:16:32PM +0100, Igor Mammedov wrote:
commit 8935499831312 makes cpuid return to guest host's vendor value
instead of built-in one by default if kvm_enabled() == true and allows
to override this behavior if 'vendor' is specified on -cpu command line.
But every time
Vendor property setter takes string as vendor value but cpudefs
use uint32_t vendor[123] fields to define vendor value. It makes it
difficult to unify and use property setter for values from cpudefs.
Simplify code by using vendor property setter, vendor[123] fields
are converted into vendor[13]
From: Amos Kong ak...@redhat.com
Currently we write MAC address to pci config space byte by byte,
this means that we have an intermediate step where mac is wrong.
This patch introduced a new control command to set MAC address,
it's atomic.
VIRTIO_NET_F_CTRL_MAC_ADDR is a new feature bit for
This is the intersect of the paolo me patches for migration thread,
Changes from 2 days ago:
- spelling check from Eric
- put the commit that patch refered from (Eric)
- drop the buffered rename at paolo request.
Please, pull.
Thanks, Juan.
The following changes since commit
On 01/17/2013 05:15 AM, Benoît Canet wrote:
+# @running: True if deduplication is running
Internally QCOW2 deduplication state switch between STOPPED, STARTING, RUNNING
and STOPPING.
Should the running field be a status field reflecting all theses states in a
string for
On 01/17/2013 04:11 AM, Benoît Canet wrote:
Le Wednesday 16 Jan 2013 à 13:10:12 (-0700), Eric Blake a écrit :
On 01/16/2013 09:25 AM, Benoît Canet wrote:
---
block/qcow2-dedup.c | 13 +
block/qcow2.h |1 +
2 files changed, 14 insertions(+)
diff --git
On Thu, Jan 17, 2013 at 01:28:48PM -0200, Luiz Capitulino wrote:
Hi Michael,
To trigger this I just:
Yes you need
virtio-pci: fix irqfd cleanup argument order
--
MST
The viostor virtio-blk driver for Windows does not use the
VIRTIO_CONFIG_S_DRIVER bit. It only sets the VIRTIO_CONFIG_S_DRIVER_OK
bit.
The viostor driver refreshes the virtio-pci status byte sometimes while
the guest is running. We misinterpret 0x4 (VIRTIO_CONFIG_S_DRIVER_OK)
as an indication
Valid quorum resources look like
quorum:threshold/total:path/to/image_1: ... :path/to/image_total
':' is used as a separator
'\' is the escaping character for filename containing ':'
'\' escape itself
',' must be escaped with ','
On the command line for quorum files img:test.raw, img2,raw
and
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 38 +-
1 file changed, 37 insertions(+), 1 deletion(-)
diff --git a/block/quorum.c b/block/quorum.c
index cb95eab..98052eb 100644
--- a/block/quorum.c
+++ b/block/quorum.c
@@ -192,15 +192,24
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 113
1 file changed, 113 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
index c7ecffd..cb95eab 100644
--- a/block/quorum.c
+++ b/block/quorum.c
@@ -171,6
Check that every bs file return the same length.
If not return -EIO to disable the quorum and
avoid length discrepancy.
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 20
1 file changed, 20 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/blkverify.c | 108 +
include/qemu-common.h |2 +
iov.c | 103 ++
3 files changed, 107 insertions(+), 106 deletions(-)
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 32
1 file changed, 32 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
index bd478ab..4bea906 100644
--- a/block/quorum.c
+++ b/block/quorum.c
@@ -621,6 +621,37 @@ static void
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 278 +++-
1 file changed, 277 insertions(+), 1 deletion(-)
diff --git a/block/quorum.c b/block/quorum.c
index 98052eb..2b4 100644
--- a/block/quorum.c
+++ b/block/quorum.c
On Thu, Jan 17, 2013 at 04:46:54PM +0100, Stefan Hajnoczi wrote:
The viostor virtio-blk driver for Windows does not use the
VIRTIO_CONFIG_S_DRIVER bit. It only sets the VIRTIO_CONFIG_S_DRIVER_OK
bit.
The viostor driver refreshes the virtio-pci status byte sometimes while
the guest is
On Thu, Jan 17, 2013 at 2:21 AM, Stefan Hajnoczi stefa...@gmail.com wrote:
On Thu, Jan 17, 2013 at 07:07:11AM +0100, Luigi Rizzo wrote:
The comment at the beginning of net/queue.c says that packets that
cannot be sent by qemu_net_queue_send() should not be enqueued
unless a callback is
From: Amos Kong ak...@redhat.com
Currenly mac is programmed byte by byte. This means that we
have an intermediate step where mac is wrong.
Second patch introduced a new vq control command to set mac
address, it's atomic.
V2: check return of sending command, delay eth_mac_addr()
V3: restore
Hello I am resuming the work on the quorum patchset.
This patchset create a block driver implementing a quorum using total qemu disk
images. Writes are mirrored on the $total files.
For the reading part the $total files are read at the same time and a vote is
done to determine if a qiov version
On Thu, 17 Jan 2013 17:50:09 +0200
Michael S. Tsirkin m...@redhat.com wrote:
On Thu, Jan 17, 2013 at 01:28:48PM -0200, Luiz Capitulino wrote:
Hi Michael,
To trigger this I just:
Yes you need
virtio-pci: fix irqfd cleanup argument order
Oh, ok, so it's already known. I assume the fix
On Tue, Jan 15, 2013 at 06:57:37PM +0200, Michael S. Tsirkin wrote:
create timestamp by rule without sideeffects.
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
trace/Makefile.objs | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
index 261e9b6..bd478ab 100644
--- a/block/quorum.c
+++ b/block/quorum.c
@@ -611,6 +611,16 @@ static int64_t
Remove an unnecessary mutual inclusion loop between qemu-pixman.h and
console.h, since the former was only including the latter for
'PixelFormat*', which can be provided by typedefs.h. This requires a
minor adjustment to qemu-pixman.c, which was relying on the implicit
inclusion of console.h for
Makes a vote to select error if any.
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
index 4bea906..dba2918 100644
--- a/block/quorum.c
+++ b/block/quorum.c
@@
On Thu, 17 Jan 2013 15:05:46 +
Peter Maydell peter.mayd...@linaro.org wrote:
On 17 January 2013 14:23, Cornelia Huck cornelia.h...@de.ibm.com wrote:
Base is kvm-next as of 2013/01/16.
--- a/linux-headers/asm-powerpc/kvm_para.h
+++ b/linux-headers/asm-powerpc/kvm_para.h
@@ -78,7
On Fri, Sep 07, 2012 at 06:48:14AM +, Nicholas A. Bellinger wrote:
From: Nicholas Bellinger n...@linux-iscsi.org
Hello Anthony Co,
This is the fourth installment to add host virtualized target support for
the mainline tcm_vhost fabric driver using Linux v3.6-rc into QEMU 1.3.0-rc.
On Thu, Jan 17, 2013 at 08:31:50AM +0100, Andreas Färber wrote:
Add a documentation section Methods and discuss among others how to
handle overriding virtual methods.
Clarify DeviceClass::realize documentation and refer to the above.
Signed-off-by: Andreas Färber afaer...@suse.de
---
On 3 December 2012 22:08, Michael Roth mdr...@linux.vnet.ibm.com wrote:
+assert(tlb_offset ~0xf == 0);
This doesn't compile if configured with --enable-debug:
In file included from /home/petmay01/git/qemu/tcg/tcg.c:174:0:
/home/petmay01/git/qemu/tcg/arm/tcg-target.c: In function
On 17.01.2013, at 17:20, Cornelia Huck wrote:
On Thu, 17 Jan 2013 15:05:46 +
Peter Maydell peter.mayd...@linaro.org wrote:
On 17 January 2013 14:23, Cornelia Huck cornelia.h...@de.ibm.com wrote:
Base is kvm-next as of 2013/01/16.
--- a/linux-headers/asm-powerpc/kvm_para.h
+++
On Thu, Jan 17, 2013 at 05:59:17PM +0200, Michael S. Tsirkin wrote:
On Thu, Jan 17, 2013 at 04:46:54PM +0100, Stefan Hajnoczi wrote:
The viostor virtio-blk driver for Windows does not use the
VIRTIO_CONFIG_S_DRIVER bit. It only sets the VIRTIO_CONFIG_S_DRIVER_OK
bit.
The viostor
On 11/01/13 20:58, Stefan Weil wrote:
Am 11.01.2013 14:40, schrieb Wolfgang Mauerer:
For slow targets and fast hosts, the emulation may be faster
than the actual hardware, which can be undesirable for various
reasons. Add a run-time option to slow down the emulation
by sleeping in the CPU
For slow targets and fast hosts, the emulation may be faster
than the actual hardware, which can be undesirable for various
reasons. Add a run-time option to slow down the emulation
by sleeping in the CPU emulation.
Signed-off-by: Wolfgang Mauerer wolfgang.maue...@siemens.com
---
cpus.c
On Thu, Jan 17, 2013 at 04:16:33PM +0100, Igor Mammedov wrote:
Move custom features parsing after built-in cpu_model defaults are set
and set custom features directly on CPU instance. That allows to make
clear distinction between built-in cpu model defaults that eventually
should go into
Same as Bug 1096712 I reported
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1091115
Title:
qemu-1.3.0 crashes when installing windows xp
Status in QEMU:
New
Bug description:
These are the
On 01/16/13 19:28, Eduardo Habkost wrote:
+static void test_parse_uint_negative(void)
+{
+unsigned long long i = 999;
+char f = 'X';
+char *endptr = f;
+const char *str = \t -321;
+int r;
+
+r = parse_uint(str, i, endptr, 0);
+
+g_assert_cmpint(r, ==,
On Wed, Jan 09, 2013 at 04:53:40PM -0200, Eduardo Habkost wrote:
This series uses a much simpler approach than the previous versions:
- The APIC ID calculation code is now inside cpu.c
- It doesn't require touching the PC CPU creation code at all
- It simply uses a static variable to
On Thu, Jan 17, 2013 at 07:29:17PM +0100, Laszlo Ersek wrote:
On 01/16/13 19:28, Eduardo Habkost wrote:
+static void test_parse_uint_negative(void)
+{
+unsigned long long i = 999;
+char f = 'X';
+char *endptr = f;
+const char *str = \t -321;
+int r;
+
+
Eduardo Habkost ehabk...@redhat.com writes:
On Thu, Jan 17, 2013 at 08:31:50AM +0100, Andreas Färber wrote:
+ * Alternatively, object_class_by_name() can be used to obtain the class and
+ * its non-overridden methods for a specific type. This would correspond to
+ * |[ MyClass::method(...) ]|
There are lots of duplicate parsing code using strto*() in QEMU, and
most of that code is broken in one way or another. Even the visitors
code have duplicate integer parsing code[1]. This introduces functions
to help parsing unsigned int values: parse_uint() and parse_uint_full().
Parsing
** Summary changed:
- qemu-1.3.0 crashes when installing windows xp
+ windowsXP install in qemu-system-i386 1.3.0 ends with a BSOD 0x7E in acpi.sys
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
On 01/17/13 20:06, Eduardo Habkost wrote:
Changes v4:
- Return -EINVAL in case a minus sign is found
- Make endptr point to beginning of string in case -EINVAL
is returned (like the strtoull() behavior)
Reviewed-by: Laszlo Ersek ler...@redhat.com
It is most likely the seabios bits missing in 1.3.0, namely this change:
http://git.qemu.org/?p=seabios.git;a=commitdiff;h=f64a472a481784231fbf8541825501df411b11d1
You may try this bios file:
http://git.qemu.org/?p=qemu.git;a=blob;f=pc-
bios/bios.bin;h=3910875311ceaed814f902e9e4e7e29cdf340fc6
On Tue, 15 Jan 2013 17:45:10 +0100
Paolo Bonzini pbonz...@redhat.com wrote:
Hi, I've been somewhat busy with other things during the early 1.3 cycle
so I have no idea what changes went in.
I created a stub at http://wiki.qemu.org/ChangeLog/1.4, please fill it
in. You're CCed if you sent
*** This bug is a duplicate of bug 1091115 ***
https://bugs.launchpad.net/bugs/1091115
** This bug has been marked a duplicate of bug 1091115
qemu-1.3.0 crashes when installing windows xp
--
You received this bug notification because you are a member of qemu-
devel-ml, which is
On 01/17/2013 12:06 PM, Eduardo Habkost wrote:
There are lots of duplicate parsing code using strto*() in QEMU, and
most of that code is broken in one way or another. Even the visitors
code have duplicate integer parsing code[1]. This introduces functions
to help parsing unsigned int values:
Vadim Evard v.e.ev...@gmail.com writes:
Signed-off-by: Vadim Evard v.e.ev...@gmail.com
Please post this as a top-level patch. (no in-reply-to)
Regards,
Anthony Liguori
---
configure |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure b/configure
index
Silence a (legitimate) complaint about missing parentheses:
tcg/arm/tcg-target.c: In function ‘tcg_out_qemu_ld’:
tcg/arm/tcg-target.c:1148:5: error: suggest parentheses around
comparison in operand of ‘’ [-Werror=parentheses]
tcg/arm/tcg-target.c: In function ‘tcg_out_qemu_st’:
On Mon, Jan 14, 2013 at 9:09 AM, Stefan Hajnoczi stefa...@gmail.com wrote:
On Sat, Jan 12, 2013 at 10:46:18AM +, Blue Swirl wrote:
On Thu, Jan 10, 2013 at 12:43 PM, Stefan Hajnoczi stefa...@gmail.com wrote:
On Wed, Jan 09, 2013 at 08:43:51PM +, Blue Swirl wrote:
On Tue, Jan 8, 2013
Hello Wolfgang,
please see comment below.
Am 17.01.2013 18:10, schrieb Wolfgang Mauerer:
For slow targets and fast hosts, the emulation may be faster
than the actual hardware, which can be undesirable for various
reasons. Add a run-time option to slow down the emulation
by sleeping in the CPU
On Mon, Jan 14, 2013 at 9:13 AM, Stefan Hajnoczi stefa...@gmail.com wrote:
On Sat, Jan 12, 2013 at 12:00:45PM +, Blue Swirl wrote:
On Fri, Jan 11, 2013 at 7:27 AM, 马磊 aware@gmail.com wrote:
On Fri, Jan 11, 2013 at 2:28 PM, Wanlong Gao gaowanl...@cn.fujitsu.com
wrote:
On
On Mon, Jan 14, 2013 at 7:01 PM, Markus Armbruster arm...@redhat.com wrote:
[Thread hijack, dropping cc: qemu-trivial, qemu-ppc]
Andreas Färber afaer...@suse.de writes:
Am 14.01.2013 13:19, schrieb Markus Armbruster:
Andreas Färber afaer...@suse.de writes:
[...]
diff --git a/qom/object.c
On Tue, Jan 15, 2013 at 8:49 AM, Paolo Bonzini pbonz...@redhat.com wrote:
I had missed the introduction of the gcov-files-* variables.
Cc: Blue Swirl blauwir...@gmail.com
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
---
Blue, can you look at introducing a common variable for the
These patches improve the error report if the file format was
specified explicitly (example: -drive file=myfile,format=qcow2)
and the given format does not match the real format.
They fix those bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=556482
https://bugs.launchpad.net/qemu/+bug/1090600
On Wed, Jan 16, 2013 at 6:04 PM, Stefan Weil s...@weilnetz.de wrote:
MinGW has no strtok_r, so we need a declaration in sysemu/os-win32.h.
We must also fix the include statements in util/envlist.c to include
that file.
We currently don't need an implementation of strtok_r because the
code is
The signature is a 32 bit value and needs up to 8 hex digits for printing.
Signed-off-by: Stefan Weil s...@weilnetz.de
---
block/vdi.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/block/vdi.c b/block/vdi.c
index 021abaa..0e1ed61 100644
--- a/block/vdi.c
+++
vdi_open did not check for a bad signature.
This check was only in vdi_probe.
Signed-off-by: Stefan Weil s...@weilnetz.de
---
block/vdi.c |6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/block/vdi.c b/block/vdi.c
index 8b768bf..257a592 100644
--- a/block/vdi.c
+++
vdi_open returned -1 in case of any error, but it should return an
error code (negative value of errno or -EMEDIUMTYPE).
Signed-off-by: Stefan Weil s...@weilnetz.de
---
block/vdi.c | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/block/vdi.c b/block/vdi.c
On Wed, Jan 16, 2013 at 4:06 PM, Markus Armbruster arm...@redhat.com wrote:
Eric Blake ebl...@redhat.com writes:
On 01/16/2013 07:36 AM, Markus Armbruster wrote:
Signed-off-by: Markus Armbruster arm...@redhat.com
---
qemu-log.c | 13 +
1 file changed, 9 insertions(+), 4
Am 17.01.2013 21:45, schrieb Blue Swirl:
On Wed, Jan 16, 2013 at 6:04 PM, Stefan Weils...@weilnetz.de wrote:
MinGW has no strtok_r, so we need a declaration in sysemu/os-win32.h.
We must also fix the include statements in util/envlist.c to include
that file.
We currently don't need an
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
---
Cc: k...@vger.kernel.org
Cc: Michael S. Tsirkin m...@redhat.com
Cc: Gleb Natapov g...@redhat.com
Cc: Marcelo Tosatti mtosa...@redhat.com
---
include/sysemu/kvm.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/sysemu/kvm.h
I am hoping to get this bug fixed in 1.4. I didn't get much feedback on the RFC
I sent last week, though.
Igor argued that APIC ID should be set by the board and not by the CPU itself,
but I am not doing that because:
- I want to keep the bug fix simple and isolated as we are past soft freeze
-
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
---
Changes v1 - v2:
- Support 32-bit APIC IDs (in case x2APIC is going to be used)
- Coding style changes
- Use TARGET_I386_TOPOLOGY_H instead of __QEMU_X86_TOPOLOGY_H__
- Rename topo_make_apic_id() to topo_apicid_for_cpu()
- Rename
The block drivers need a special error code for wrong format.
From the available error codes EMEDIUMTYPE fits best.
It is not available on all platforms, so a definition in
qemu-common.h and a specific error report are needed.
Signed-off-by: Stefan Weil s...@weilnetz.de
---
blockdev.c
On Thu, Jan 17, 2013 at 7:06 PM, Eduardo Habkost ehabk...@redhat.com wrote:
There are lots of duplicate parsing code using strto*() in QEMU, and
most of that code is broken in one way or another. Even the visitors
code have duplicate integer parsing code[1]. This introduces functions
to help
This function will be used by both the CPU initialization code and the
fw_cfg table initialization code.
Later this function will be updated to generate APIC IDs according to
the CPU topology.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
---
target-i386/cpu.c | 17 -
Currently, the pc-1.4 machine init function enables PV EOI and then
calls the pc-1.2 machine init function. The problem with this approach
is that now we can't enable any additional compatibility code inside the
pc-1.2 init function because it would end up enabling the compatibility
behavior on
The code that calculates the APIC ID will use smp_cores/smp_threads, so
just define them as 1 on *-user to avoid #ifdefs in the code.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
---
include/sysemu/cpus.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/include/sysemu/cpus.h
On Thu, Jan 17, 2013 at 8:54 PM, Stefan Weil s...@weilnetz.de wrote:
Am 17.01.2013 21:45, schrieb Blue Swirl:
On Wed, Jan 16, 2013 at 6:04 PM, Stefan Weils...@weilnetz.de wrote:
MinGW has no strtok_r, so we need a declaration in sysemu/os-win32.h.
We must also fix the include statements in
This improves error reports for bochs, cow, qcow, qcow2, qed and vmdk
when a file with the wrong format is selected.
Signed-off-by: Stefan Weil s...@weilnetz.de
---
block/bochs.c |2 +-
block/cow.c |2 +-
block/qcow.c |2 +-
block/qcow2.c |2 +-
block/qed.c |2 +-
PC will not use max_cpus for that field, so move it outside the common
code so it can use a different value on PC.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
---
hw/fw_cfg.c | 1 -
hw/pc.c | 2 +-
hw/ppc_newworld.c | 1 +
hw/ppc_oldworld.c | 1 +
hw/sun4m.c| 3 +++
To make unit tests that depend on target-specific files, use
check-unit-arch-y and test-obj-arch-y.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
---
tests/Makefile | 14 +-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/tests/Makefile b/tests/Makefile
index
On 01/17/2013 01:45 PM, Stefan Weil wrote:
These patches improve the error report if the file format was
specified explicitly (example: -drive file=myfile,format=qcow2)
and the given format does not match the real format.
They fix those bugs:
This keeps compatibility on machine-types pc-1.2 and older, and prints a
warning in case the requested configuration won't get the correct
topology.
I couldn't think of a better way to warn about broken topology when in
compat mode other than using error_report(). The warning message will be
This changes FW_CFG_MAX_CPUS and FW_CFG_NUMA to use apic_id_for_cpu(),
so the NUMA table can be based on the APIC IDs, instead of CPU index
(SeaBIOS knows nothing about CPU indexes, just APIC IDs).
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
---
Changes v2:
- Get PC object as argument
-
Hi MST Co,
On Thu, 2013-01-17 at 18:43 +0200, Michael S. Tsirkin wrote:
On Fri, Sep 07, 2012 at 06:48:14AM +, Nicholas A. Bellinger wrote:
From: Nicholas Bellinger n...@linux-iscsi.org
Hello Anthony Co,
This is the fourth installment to add host virtualized target support for
This will allow each architecture to define how the VCPU ID is set on
the KVM_CREATE_VCPU ioctl call.
Signed-off-by: Eduardo Habkost ehabk...@redhat.com
---
Cc: k...@vger.kernel.org
Cc: Michael S. Tsirkin m...@redhat.com
Cc: Gleb Natapov g...@redhat.com
Cc: Marcelo Tosatti mtosa...@redhat.com
On Thu, Jan 17, 2013 at 11:28:46AM +0100, Stefan Hajnoczi wrote:
On Tue, Jan 15, 2013 at 06:57:34PM +0200, Michael S. Tsirkin wrote:
Using a global pattern makes it easier to clean out
old generated files.
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
rules.mak | 5 +
The CPU ID in KVM is supposed to be the APIC ID, so change the
KVM_CREATE_VCPU call to match it. The current behavior didn't break
anything yet because today the APIC ID is assumed to be equal to the CPU
index, but this won't be true in the future.
Signed-off-by: Eduardo Habkost
This is a cleanup that tries to solve two small issues:
- We don't need a separate kvm_pv_eoi_features variable just to keep a
constant calculated at compile-time, and this style would require
adding a separate variable (that's declared twice because of the
CONFIG_KVM ifdef) for each
On 01/17/2013 12:40 AM, Wenchao Xia wrote:
This patch add function bdrv_query_snapshot_infolist(), which will
return snapshot info of an image in qmp object format. The implementation
code are mostly copied from qemu-img.c with modification to fit more
for qmp based block layer API.
To
On 01/17/2013 08:51 AM, Benoît Canet wrote:
Valid quorum resources look like
quorum:threshold/total:path/to/image_1: ... :path/to/image_total
':' is used as a separator
'\' is the escaping character for filename containing ':'
'\' escape itself
',' must be escaped with ','
On the
On 01/17/2013 08:51 AM, Benoît Canet wrote:
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 278
+++-
1 file changed, 277 insertions(+), 1 deletion(-)
diff --git a/block/quorum.c b/block/quorum.c
index
Am 17.01.2013 22:28, schrieb Eric Blake:
On 01/17/2013 01:45 PM, Stefan Weil wrote:
These patches improve the error report if the file format was
specified explicitly (example: -drive file=myfile,format=qcow2)
and the given format does not match the real format.
They fix those bugs:
On Thu, Jan 17, 2013 at 05:06:50PM -0200, Eduardo Habkost wrote:
There are lots of duplicate parsing code using strto*() in QEMU, and
most of that code is broken in one way or another. Even the visitors
code have duplicate integer parsing code[1]. This introduces functions
to help parsing
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/quorum.c | 22 ++
1 file changed, 22 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
index ce094a1..1f12438 100644
--- a/block/quorum.c
+++ b/block/quorum.c
@@ -15,6 +15,13 @@
#include block/block_int.h
Signed-off-by: Benoit Canet ben...@irqsave.net
---
block/Makefile.objs |1 +
block/quorum.c | 45 +
2 files changed, 46 insertions(+)
create mode 100644 block/quorum.c
diff --git a/block/Makefile.objs b/block/Makefile.objs
index
101 - 200 of 232 matches
Mail list logo