The following exception is threw:
Python Exception name 'long' is not defined:
Error occurred in Python command: name 'long' is not defined
In python3, long is rename to int
Signed-off-by: Yang Wei
---
scripts/qemugdb/mtree.py | 17 -
1 file changed, 12 insertions(+), 5 deletio
On Tue, Dec 01, 2015 at 11:45:15AM -0500, John Snow wrote:
> On 12/01/2015 07:16 AM, Amit Shah wrote:
> > Ping.
> >
> > I haven't received any responses so far, please let me know if you're
> > planning on doing this.
> >
> > Thanks,
> >
>
> I did amend the changelog on the wiki for my part, bu
NVDIMM devices is defined in ACPI 6.0 9.20 NVDIMM Devices
There is a root device under \_SB and specified NVDIMM devices are under the
root device. Each NVDIMM device has _ADR which returns its handle used to
associate MEMDEV structure in NFIT
Currently, we do not support any function on _DSM, th
Add NVDIMM maintainer
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Xiao Guangrong
---
MAINTAINERS | 7 +++
1 file changed, 7 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index bb1f3e4..7e82340 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -940,6 +940,13 @@ M: Jiri Pirko
S: Maintai
This patchset can be found at:
https://github.com/xiaogr/qemu.git nvdimm-v9
It is based on pci branch on Michael's tree and the top commit is:
commit 0c73277af7 (vhost-user-test: fix crash with glib < 2.36).
Changelog in v9:
- the changes address Michael's comments:
1) move the control pa
NFIT is defined in ACPI 6.0: 5.2.25 NVDIMM Firmware Interface Table (NFIT)
Currently, we only support PMEM mode. Each device has 3 structures:
- SPA structure, defines the PMEM region info
- MEM DEV structure, it has the @handle which is used to associate specified
ACPI NVDIMM device we will i
Introduce "nvdimm" device which is based on pc-dimm device type
Currently, nothing is specific for nvdimm but hotplug is disabled
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Xiao Guangrong
---
default-configs/i386-softmmu.mak | 1 +
default-configs/x86_64-softmmu.mak | 1 +
hw/acpi/memory_
Let build_header() support specified OEM table id so that we can build
multiple SSDT later
If the oem table id is not specified (aka, NULL), we use the default id
instead as the previous behavior
Reviewed-by: Stefan Hajnoczi
Signed-off-by: Xiao Guangrong
---
hw/acpi/aml-build.c | 15 ++
On 12/01/2015 09:32 PM, Victor Kaplansky wrote:
> Fix QEMU crash when -netdev type=vhost-user,queues=n is passed
> with zero number of queues.
>
> Signed-off-by: Victor Kaplansky
> ---
> net/vhost-user.c | 5 +
> 1 file changed, 5 insertions(+)
>
> diff --git a/net/vhost-user.c b/net/vhost-
On 12/02/2015 09:00 AM, Wen Congyang wrote:
> On 12/01/2015 06:40 PM, Dr. David Alan Gilbert wrote:
>> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>>> Block replication is a very important feature which is used for
>>> continuous checkpoints(for example: COLO).
>>>
>>> You can get the detailed inf
On Wed, Dec 02, 2015 at 08:50:48AM +0800, Fam Zheng wrote:
> On Tue, 12/01 21:28, Peter Xu wrote:
> > /* if there is a dump in background, we should wait until the dump
> > * finished */
> > if (dump_in_progress()) {
> > error_setg(errp, "There is a dump in process, please
2015-12-02 8:23 GMT+03:00 Hitoshi Mitake :
> Seems that your patch violates the coding style of qemu. You can check the
> style with scripts/checkpatch.pl.
>
>
> Comment outed code isn't good. You should remove it (in addition, it
> wouldn't be required).
Thanks!
--
Vasiliy Tolstov,
e-mail: v.t
On 12/01/2015 10:21 PM, Cornelia Huck wrote:
> On Tue, 1 Dec 2015 13:10:40 +0100
> Cornelia Huck wrote:
>
>> On Tue, 1 Dec 2015 11:11:08 +0100
>> Cornelia Huck wrote:
>>
>>> Some of our test folks tried to run a recent-ish qemu (nearly 2.5)
>>> combined with an old host kernel (and a virtio-1 c
On Tue, Dec 01, 2015 at 12:06:43PM +1100, David Gibson wrote:
> On Fri, Nov 20, 2015 at 06:24:36PM +0530, Bharata B Rao wrote:
> > Start supporting CPU hotplug from pseries-2.5 onwards. Add CPU
> > DRC (Dynamic Resource Connector) device tree entries.
> >
> > Signed-off-by: Bharata B Rao
> > ---
On Tue, Dec 01, 2015 at 12:34:06PM +1100, David Gibson wrote:
> On Fri, Nov 20, 2015 at 06:24:38PM +0530, Bharata B Rao wrote:
> > Support hot removal of CPU for sPAPR guests by sending the hot unplug
> > notification to the guest via EPOW interrupt. Release the vCPU object
> > after CPU hot unplug
On Tue, Dec 01, 2015 at 12:01:36PM +1100, David Gibson wrote:
> On Fri, Nov 20, 2015 at 06:24:35PM +0530, Bharata B Rao wrote:
> > XICS is setup for each CPU during initialization. Provide a routine
> > to undo the same when CPU is unplugged.
> >
> > This allows reboot of a VM that has undergone C
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
block.c | 83 +++
include/block/block.h | 4 +++
2 files changed, 87 insertions(+)
diff --git a/block.c b/block.c
index 213bee8..09ee7f1 100644
---
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Eric Blake
---
qapi/block-core.json | 21 ++---
1 file changed, 18 insertions(+), 3 deletions(-)
diff --git a/qapi/block-core.json b/qapi/block-core.json
index 2c6bd3f..acc9f8d 100644
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
block/Makefile.objs | 1 +
block/replication.c | 549
2 files changed, 550 insertions(+)
create mode 100644 block/replication.c
diff --git a/block/Makefil
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Cc: Luiz Capitulino
Cc: Michael Roth
Reviewed-by: Paolo Bonzini
---
block.c | 43 +++
include/block/block.h | 5 +
include/block/block_int.h | 14
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
docs/block-replication.txt | 227 +
1 file changed, 227 insertions(+)
create mode 100644 docs/block-replication.txt
diff --git a/docs/block-replication.txt b/docs/bl
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Alberto Garcia
---
block/quorum.c | 78 ++
1 file changed, 78 insertions(+)
diff --git a/block/quorum.c b/block/quorum.c
index b7df14b..6fa54f3
When opening BDS, we need to create backup jobs for
image-fleecing.
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Stefan Hajnoczi
Reviewed-by: Jeff Cody
---
block/Makefile.objs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/
We need to access the parent BDS to get the root BDS.
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
block.c | 1 +
include/block/block_int.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/block.c b/block.c
index eaf479a..0a0468f 10
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
Reviewed-by: Jeff Cody
---
block/backup.c | 14 ++
blockjob.c | 11 +++
include/block/blockjob.h | 12
3 files changed, 37 insertions(+)
diff --git a/block/
Block replication is a very important feature which is used for
continuous checkpoints(for example: COLO).
You can get the detailed information about block replication from here:
http://wiki.qemu.org/Features/BlockReplication
Usage:
Please refer to docs/block-replication.txt
This patch series is
Signed-off-by: Wen Congyang
Signed-off-by: zhanghailiang
Signed-off-by: Gonglei
---
block.c | 18 ++
1 file changed, 18 insertions(+)
diff --git a/block.c b/block.c
index bfc2be8..eaf479a 100644
--- a/block.c
+++ b/block.c
@@ -1275,6 +1275,24 @@ void bdrv_set_backing_hd(BlockDr
On Tue, Dec 01, 2015 at 11:55:58AM +1100, David Gibson wrote:
> On Fri, Nov 20, 2015 at 06:24:33PM +0530, Bharata B Rao wrote:
> > From: Gu Zheng
> >
> > In order to deal well with the kvm vcpus (which can not be removed without
> > any
> > protection), we do not close KVM vcpu fd, just record a
On Wed, 2015-12-02 at 13:24 +1100, Alexey Kardashevskiy wrote:
> > But on the whole I agree with you, since the LPC is part of the P8
> > chip, I think it makes sense to include it even with -nodefaults.
>
> POWER8 chips all have 8 threads per core but we do not always assume -smt
> ...,threads=8
It should be fairly obvious that qapi base classes need to
form an acyclic graph, since QMP cannot specify the same
key more than once, while base classes are included as flat
members alongside other members added by the child. But the
old check_member_clash() parser function was not prepared to
c
We document that members of enums and objects should be
'lower-case', although we were not enforcing it. We have to
whitelist a few pre-existing entities that violate the norms.
Add three new tests to expose the new error message, each of
which first uses the whitelisted name 'UuidInfo' to prove t
Pending prerequisites:
+ Markus' "typedefs: Put them back into alphabetical order"
https://lists.gnu.org/archive/html/qemu-devel/2015-11/msg04417.html
which is now queued in qemu-trivial:
https://lists.gnu.org/archive/html/qemu-devel/2015-11/msg06297.html
+ Markus' qapi-next branch
http://repo.or.c
On Wed, Dec 2, 2015 at 1:12 AM, Vasiliy Tolstov wrote:
> Signed-off-by: Vasiliy Tolstov
> ---
> block/sheepdog.c | 59
> ++--
> 1 file changed, 57 insertions(+), 2 deletions(-)
>
Seems that your patch violates the coding style of qemu. You ca
The QMP input visitor allows integral values to be assigned by
promotion to a QTYPE_QFLOAT. However, when parsing an alternate,
we did not take this into account, such that an alternate that
accepts 'number' and some other type, but not 'int', would reject
integral values.
With this patch, we now
With the recent commit 'qapi: Detect collisions in C member
names', we have two different locations for detecting clashes -
one at parse time, and another at QAPISchema*.check() time.
Remove all of the ad hoc parser checks, and delete associated
code (for example, the global check_member_clash() me
Rather than using just an array of strings, make enum.values be
an array of the new QAPISchemaMember type, and add a helper
member_names() method to get back at the original list of names.
Likewise, creating an enum requires wrapping strings, via a new
QAPISchema._make_enum_members() method. The b
Previously, working with alternates required two lookup arrays
and some indirection: for type Foo, we created Foo_qtypes[]
which maps each qtype to a value of the generated FooKind enum,
then look up that value in FooKind_lookup[] like we do for other
union types.
This has a couple of subtle bugs.
The QObject hierarchy is small enough, and unlikely to grow further
(since we only use it to map to JSON and already cover all JSON
types), that we can simplify things by not tracking a separate
vtable, but just inline the code element of the vtable QType
directly into QObject (renamed to type), an
We want to share some clash detection code between enum values
and object type members. To assist with that, split off part
of QAPISchemaObjectTypeMember into a new base class
QAPISchemaMember that tracks name, owner, and common clash
detection code; while the former keeps the additional fields
fo
Every non-implicit entity is associated with an 'info'
dictionary, but it is not easy to reverse-engineer the name of
the top-most entity associated with that 'info'.
For example, inside QAPISchemaMember.check_clash(), the value
of self.owner may be the implicit ':obj-block_passwd-arg', while
we w
None of the visitor callbacks would set an error when testing
if an optional field was present; make this part of the interface
contract by eliminating the errp argument.
The resulting generated code has a nice diff:
|-visit_optional(v, &has_fdset_id, "fdset-id", &err);
|-if (err) {
|-
Previously, the generated code in qapi-types.c initialized all
enum lookup tables first, prior to any other definitions. But
there are no topological sorting requirements that mandate this
layout, so we can drop the QAPISchemaGenTypeVisitor._fwdefn
field and just generate all definitions in visita
What's more meta than using qapi to define qapi? :)
Convert QType into a full-fledged[*] builtin qapi enum type, so
that a subsequent patch can then use it as the discriminator
type of qapi alternate types. Fortunately, the judicious use of
'prefix' in the qapi definition avoids churn to the spel
For less code, reflect the determined boolean value of an optional
visit back to the caller instead of making the caller read the
boolean after the fact.
The resulting generated code has the following diff:
|-visit_optional(v, &has_fdset_id, "fdset-id");
|-if (has_fdset_id) {
|+if (vi
Now that alternates no longer use an implicit tag, we can
inline _make_implicit_tag() into its one caller,
_def_union_type().
No change to generated code.
Suggested-by: Markus Armbruster
Signed-off-by: Eric Blake
---
v14: no change
v13: commit message touchup
v12: new patch
---
scripts/qapi.p
The name QType matches our CODING_STYLE conventions for type names
in CamelCase. It also matches the fact that we are already naming
all the enum members with a prefix of QTYPE, not QTYPE_CODE. And
doing the rename will also make it easier for the next patch to use
QAPI for providing the enum, wh
On Tue, 2015-12-01 at 11:59 -0500, Paolo Bonzini wrote:
> > What do you think about virtio-nvme+vhost-nvme?
>
> What would be the advantage over virtio-blk? Multiqueue is not supported
> by QEMU but it's already supported by Linux (commit 6a27b656fc).
I expect performance would be better.
Seems
On 12/01/2015 05:38 PM, Denis V. Lunev wrote:
> On 12/01/2015 06:31 AM, Jason Wang wrote:
>>
>> On 11/30/2015 02:22 PM, Denis V. Lunev wrote:
>>> On 11/30/2015 08:58 AM, Jason Wang wrote:
On 11/27/2015 07:42 PM, Denis V. Lunev wrote:
> On 11/27/2015 09:50 AM, Denis V. Lunev wrote:
>>
On Mon, Nov 30, 2015 at 06:30:52PM +1100, Alexey Kardashevskiy wrote:
> On 11/20/2015 11:54 PM, Bharata B Rao wrote:
> >From: Gu Zheng
> >
> >In order to deal well with the kvm vcpus (which can not be removed without
> >any
> >protection), we do not close KVM vcpu fd, just record and mark it as s
On Tue, Dec 01, 2015 at 11:44:03AM +1100, David Gibson wrote:
> On Fri, Nov 20, 2015 at 06:24:31PM +0530, Bharata B Rao wrote:
> > CPUState *cpu gets added to the cpus list during cpu_exec_init(). It
> > should be removed from cpu_exec_exit().
> >
> > cpu_exec_init() is called from generic CPU::in
On Tue, Dec 01, 2015 at 11:37:03AM +1100, David Gibson wrote:
> On Fri, Nov 20, 2015 at 06:24:30PM +0530, Bharata B Rao wrote:
> > Prevent guests from booting with CPU topologies that have partially
> > filled CPU cores or can result in partially filled CPU cores after
> > CPU hotplug like
> >
> >
On Wed, Dec 02, 2015 at 08:46:49AM +0800, Fam Zheng wrote:
> On Tue, 12/01 21:28, Peter Xu wrote:
> > +if (*errp) {
> > +s->status = DUMP_STATUS_FAILED;
> > +} else {
> > +s->status = DUMP_STATUS_COMPLETED;
> > +}
> > +
>
> To detect error, it's better to use local_err
On Wed, Dec 02, 2015 at 08:37:36AM +0800, Fam Zheng wrote:
> On Tue, 12/01 21:28, Peter Xu wrote:
> > It might be a little bit confusing to do dump_cleanup() in these two
> > functions and error prone. A better way is to do dump_cleanup()
>
> I would say "It might be a little bit confusing and err
On Tue, Dec 01, 2015 at 11:49:31AM -0800, Sukadev Bhattiprolu wrote:
> David Gibson [da...@gibson.dropbear.id.au] wrote:
> | > @@ -240,6 +241,36 @@ static void rtas_ibm_get_system_parameter(PowerPCCPU
> *cpu,
> | > target_ulong ret = RTAS_OUT_SUCCESS;
> | >
> | > switch (parameter) {
>
ae31fb5 "vhost-user-test: wrap server in TestServer struct" cleaned up
the handling of the test server in vhost-user-test. Unfortunately it
introduced a subtle use-after-free if a race goes the wrong way.
When the server structure is freed inside test_server_free() the GThread
started earlier is
On Tue, Dec 01, 2015 at 04:09:48PM +0100, Paolo Bonzini wrote:
>
>
> On 01/12/2015 03:18, Peter Xu wrote:
> > I think (2) is better in term of lines of codes (and also
> > clear). However I may need to keep the QMP interface (to keep the
> > has_detach parameter in qmp_dump_guest_memory), so I'd
On 12/01/2015 05:43 PM, David Gibson wrote:
On Tue, Nov 17, 2015 at 11:40:04AM +1100, Benjamin Herrenschmidt wrote:
On Tue, 2015-11-17 at 11:32 +1100, Alexey Kardashevskiy wrote:
On 11/11/2015 11:27 AM, Benjamin Herrenschmidt wrote:
This adds a model of the POWER8 LPC controller. It is then us
The field is not used for anything.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_q35.c | 1 -
include/hw/pci-host/q35.h | 1 -
2 files changed, 2 deletions(-)
diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.c
index f124e59..ad7a51e 100644
--- a/hw/i386/pc_q35.c
+++ b/hw/i386/pc_q35.c
@
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 6 +++---
hw/i386/pc.c | 8
include/hw/i386/pc.h | 2 +-
3 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 9f2129d..efc4151 100644
--- a/hw/i386/acpi-build.c
+
With this, we can eliminate struct PcGuestInfo completely.
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 25 +++--
hw/i386/pc.c | 23 ++-
include/hw/i386/pc.h | 15 +--
3 files changed, 26 insertions(+), 37 deletions(-)
dif
Remove the fields: legacy_acpi_table_size, has_acpi_build,
has_reserved_memory, and rsdp_in_ram from PcGuestInfo, and let
the ACPI code use the PCMachineClass fields directly.
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 11 ++-
hw/i386/pc.c | 6 +++---
hw/i386/pc_p
Some PCMachineState and PCMachineClass fields will be used by
build_srat().
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index b1548e7..8841798 100644
--- a/hw/i386/ac
The code can use the PCMachineClass.pci_enabled field directly.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc.c | 2 +-
hw/i386/pc_piix.c| 1 -
hw/i386/pc_q35.c | 1 -
include/hw/i386/pc.h | 1 -
4 files changed, 1 insertion(+), 4 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386
PCMachineState will be used in some of the steps of ACPI table
building.
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 85a5c53..ca11c88 100644
--- a/hw/i386/acp
We don't need the FWCfgState return value and the PcGuestInfo
parameter.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc.c | 5 +
hw/i386/pc_piix.c| 2 +-
include/hw/i386/pc.h | 3 +--
3 files changed, 3 insertions(+), 7 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 7 ---
hw/i386/pc.c | 2 +-
include/hw/i386/pc.h | 2 +-
3 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 7771be7..cdbe5b9 100644
--- a/hw/i386/acpi-build.c
+++ b/
Instead of allocating a new struct just for PcGuestInfo and the
mchine_done Notifier, place them inside PCMachineState.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc.c | 27 ++-
include/hw/i386/pc.h | 2 ++
2 files changed, 12 insertions(+), 17 deletions(-)
diff
We can get the PcGuestInfo struct directly from PCMachineState.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc.c | 4 ++--
hw/i386/pc_piix.c| 2 +-
hw/i386/pc_q35.c | 2 +-
include/hw/i386/pc.h | 3 +--
4 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/hw/i386/pc.c b/
Some PCMachineState and PCMachineClass fields will be used by
acpi_build().
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index ca11c88..b1548e7 100644
--- a/hw/i386/a
The ACPI code can use the PCMachineState fields directly.
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 10 +-
hw/i386/pc.c | 2 --
include/hw/i386/pc.h | 1 -
3 files changed, 5 insertions(+), 8 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/hw/i386/acpi-build.c b/hw/i386/acpi-build.c
index 74f0922..85a5c53 100644
--- a/hw/i386/acpi-build.c
+++ b/hw/i386/acpi-build.c
@@ -295,7 +295,7 @@ static void acpi_alig
This moves all data from PcGuestInfo to either PCMachineState or
PCMachineClass.
This series depends on other two series:
* [PATCH v3 0/6] pc: Initialization and compat function cleanup
* [PATCH V3 0/3] hw/pcie: Multi-root support for Q35
For reference, there's a git tree containing this series
Lots of PcGuestInfo fields are duplicates of PCMachineClass or
PCMachineState fields. Pass PCMachineState as argument to
acpi_setup(), so we can simply let the ACPI code use those fields
directly.
Signed-off-by: Eduardo Habkost
---
hw/i386/acpi-build.c | 3 ++-
hw/i386/acpi-build.h | 2 +-
hw/i3
The struct will be used inside PCMachineState.
Signed-off-by: Eduardo Habkost
---
include/hw/i386/pc.h | 31 ---
1 file changed, 16 insertions(+), 15 deletions(-)
diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h
index 9811229..a74bded 100644
--- a/include/hw/
On Tue, 12/01 21:28, Peter Xu wrote:
> Here, total_size is the size in bytes to be dumped (raw data, which
> means before compression), while written_size are bytes handled (raw
> size too).
>
> Signed-off-by: Peter Xu
> ---
> dump.c| 32
> includ
On Tue, 12/01 21:28, Peter Xu wrote:
> One new QMP event DUMP_COMPLETED is added. When a dump finishes, one
> DUMP_COMPLETED event will occur to notify the user.
>
> Signed-off-by: Peter Xu
> ---
> docs/qmp-events.txt | 16
> dump.c | 11 +--
> qapi-schema.j
On 12/01/2015 07:58 PM, Hailiang Zhang wrote:
> On 2015/12/1 18:40, Dr. David Alan Gilbert wrote:
>> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>>> Block replication is a very important feature which is used for
>>> continuous checkpoints(for example: COLO).
>>>
>>> You can get the detailed infor
On 12/01/2015 06:40 PM, Dr. David Alan Gilbert wrote:
> * Wen Congyang (we...@cn.fujitsu.com) wrote:
>> Block replication is a very important feature which is used for
>> continuous checkpoints(for example: COLO).
>>
>> You can get the detailed information about block replication from here:
>> http
On Tue, 12/01 21:28, Peter Xu wrote:
> Signed-off-by: Peter Xu
> ---
> dump.c | 6 +-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/dump.c b/dump.c
> index e46749d..65d1f7e 100644
> --- a/dump.c
> +++ b/dump.c
> @@ -1626,13 +1626,17 @@ void qmp_dump_guest_memory(bool pagi
On Tue, 12/01 21:28, Peter Xu wrote:
> No functional change. Cleanup only.
>
> Signed-off-by: Peter Xu
> ---
> dump.c| 35 ++-
> include/sysemu/dump.h | 3 +++
> 2 files changed, 25 insertions(+), 13 deletions(-)
>
> diff --git a/dump.c b/dump.c
On Tue, 12/01 21:28, Peter Xu wrote:
> Instead of malloc/free each time for DumpState, make it
> static. Added DumpStatus to show status for dump.
>
> This is to be used for detached dump.
>
> Signed-off-by: Peter Xu
> ---
> dump.c| 20 +---
> include/sysemu/dump
On Tue, 12/01 21:28, Peter Xu wrote:
> It might be a little bit confusing to do dump_cleanup() in these two
> functions and error prone. A better way is to do dump_cleanup()
I would say "It might be a little bit confusing and error prone to do
dump_cleanup() in ..."
Other than that,
Reviewed-by:
On 1 December 2015 at 23:06, Richard Henderson wrote:
> On 12/01/2015 08:40 AM, Aurelien Jarno wrote:
>> That said the problem reported is likely fixed by this commit that went
>> just after it has been reported:
>
>
> It does seem likely, but I don't see how we can know that the out size is
> 110
Split it into an abstract test class and an implementation class.
The split is primarily to facilitate more flexible setUp variations
for other kinds of tests without having to rewrite or shuffle around
all of these helpers.
See the following two patches for more of the "why."
Signed-off-by: Joh
Code motion only, in preparation for adjusting
the setUp procedure for this test.
Signed-off-by: John Snow
---
tests/qemu-iotests/124 | 117 +
1 file changed, 60 insertions(+), 57 deletions(-)
diff --git a/tests/qemu-iotests/124 b/tests/qemu-iotes
Don't create two interfaces to the same drive in the recently moved
failure test.
Signed-off-by: John Snow
---
tests/qemu-iotests/124 | 26 ++
1 file changed, 14 insertions(+), 12 deletions(-)
diff --git a/tests/qemu-iotests/124 b/tests/qemu-iotests/124
index 2a0119d..7d
Kevin caught me being Naughty, and because I don't want Santa to be
mad at me, I have corrected my ways.
Split iotest 124 into two classes so that the iotest that requires
a blkdebug filter from the get-go can forego the standard setUp
routine and just do it correct the first time.
Does this warr
On 12/01/2015 08:40 AM, Aurelien Jarno wrote:
On 2015-12-01 17:34, Aurelien Jarno wrote:
On 2015-12-01 16:28, Peter Maydell wrote:
On 1 December 2015 at 16:19, Richard Henderson wrote:
If there are a lot of guest memory ops in the TB, the amount of
code generated by tcg_out_tb_finalize could
This way the compat flags can be initialized in the machine_options()
function. This will help us to eventually eliminate the pc_compat_*()
functions.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc.c | 8 +
hw/i386/pc_piix.c| 84 +--
This way, these settings can be simply set on the corresponding
machine_options() function, instead of requiring code in
pc_compat_*() functions.
Signed-off-by: Eduardo Habkost
---
hw/core/loader.c| 10 +-
hw/core/machine.c | 1 +
hw/i386/pc_piix.c | 8
hw/i386/pc_q35.
The property is read-only and not used for anything.
Cc: Igor Mammedov
Signed-off-by: Eduardo Habkost
---
hw/i386/pc.c | 11 ---
include/hw/i386/pc.h | 1 -
2 files changed, 12 deletions(-)
diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index 72c25ec..1b22ad2 100644
--- a/hw/i386/pc
enforce_aligned_dimm never changes after the machine is
initialized, so it can be simply set in PCMachineClass like all
the other compat fields.
Cc: Igor Mammedov
Signed-off-by: Eduardo Habkost
---
hw/i386/pc.c | 12 +++-
hw/i386/pc_piix.c| 4 +---
hw/i386/pc_q35.c | 4
This moves most compat data that are globals or PCMachineState
fields to PCMachineClass.
It looks like v2 (sent in August) has slipped through the cracks.
Rebased to latest qemu.git, and added 2 additional patches.
Eduardo Habkost (6):
pc: Move compat boolean globals to PCMachineClass
pc: Mov
This way we don't need code in pc_compat_*() functions to set the legacy
acpi_data_size value.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc.c | 17 ++---
hw/i386/pc_piix.c| 2 +-
hw/i386/pc_q35.c | 2 +-
include/hw/i386/pc.h | 6 --
4 files changed, 12 inserti
This way we can set legacy_acpi_table_size on the machine_options()
functions, instead of requirng code in pc_compat_*() functions.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_piix.c| 40 +++-
include/hw/i386/pc.h | 1 +
2 files changed, 20 insertions(+
If there are a lot of guest memory ops in the TB, the amount of
code generated by tcg_out_tb_finalize could be well more than 1k.
In the short term, increase the reservation larger than any TB
seen in practice.
Reported-by: Aurelien Jarno
Reviewed-by: Aurelien Jarno
Tested-by: Aurelien Jarno
Si
Thomas Huth, on Tue 01 Dec 2015 22:28:10 +0100, wrote:
> Are you still planning to rebase the patches again to current master and
> repost them, so we could maybe get this in for QEMU 2.6 ?
If we can get this reviewed, yes, sure!
> If you put me on CC:, I can try to review the series if you like
om/rth7680/qemu.git tags/pull-tcg-20151201
for you to fetch changes up to b17a6d3390f87620735f7efb03bb1c96682ff449:
tcg: Increase the highwater reservation (2015-12-01 14:36:32 -0800)
Last
On 11/27/2015 02:03 AM, Markus Armbruster wrote:
> Eric Blake writes:
>
>> We document that members of enums and objects should be
>> 'lower-case', although we were not enforcing it. We have to
>> whitelist a few pre-existing entities that violate the norms.
>> Add three new tests to expose the
On Tue, Dec 01, 2015 at 10:53:07PM +0200, Marcel Apfelbaum wrote:
> On 12/01/2015 08:20 PM, Eduardo Habkost wrote:
> >On Thu, Nov 26, 2015 at 06:00:28PM +0200, Marcel Apfelbaum wrote:
> >>Add bus property to PC machines and use it when looking
> >>for primary PCI root bus (bus 0).
> >>
> >>Signed-o
1 - 100 of 297 matches
Mail list logo
