The keyboard input issue looks the same as #1675549, and that's on
Linux/SDL. So not specific to this fix or Cocoa.
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
https://bugs.launchpad.net/bugs/1675108
Title:
Cocoa UI always cras
Tested on 10.12.3, it doesn't crash immediately (like before) but
crashes reliably once I send some keyboard input to qemu:
$ i386-softmmu/qemu-system-i386
**
ERROR:/Users/pip/no_backup/qemu/translate-common.c:34:tcg_handle_interrupt:
assertion failed: (qemu_mutex_iothread_locked())
Abort trap:
Kevin Wolf writes:
> Am 23.03.2017 um 11:55 hat Markus Armbruster geschrieben:
>> We have two list-values options:
>>
>> * "server" is a list of InetSocketAddress. We use members "host" and
>> "port", and silently ignore the rest.
>>
>> * "auth-supported" is a list of RbdAuthMethod. We use
Eric Blake writes:
> On 03/23/2017 02:18 PM, Eric Blake wrote:
>
> -if (port) {
> -/* check for ipv6 */
> -if (strchr(host, ':')) {
> -strbuf = g_strdup_printf("[%s]:%s", host, port);
> -} else {
>
On 03/24/2017 12:20 AM, Kevin Wolf wrote:
> Am 23.03.2017 um 16:35 hat Denis V. Lunev geschrieben:
>> On 03/23/2017 06:04 PM, Kevin Wolf wrote:
>>> Am 23.03.2017 um 15:17 hat Eric Blake geschrieben:
On 03/23/2017 08:28 AM, Denis V. Lunev wrote:
> ZSDT compression algorithm consumes 3-5 tim
On 23/03/17 19:22, Stefano Stabellini wrote:
> On Thu, 23 Mar 2017, Paolo Bonzini wrote:
>> On 23/03/2017 14:55, Juergen Gross wrote:
>>> On 23/03/17 14:00, Greg Kurz wrote:
On Mon, 20 Mar 2017 11:19:05 -0700
Stefano Stabellini wrote:
> Do not use the ring.h header installed on
On Thu, Mar 23, 2017 at 02:04:55PM +0400, Marc-André Lureau wrote:
> Running postcopy-test with ASAN produces the following error:
>
> QTEST_QEMU_BINARY=ppc64-softmmu/qemu-system-ppc64 tests/postcopy-test
> ...
> =
> ==23641==ERROR:
On Thu, Mar 23, 2017 at 04:56:30PM -0500, Eric Blake wrote:
> On 03/23/2017 04:43 PM, Eric Blake wrote:
>
> > We'd still have to allow libvirt's use of
> > ":key=...:auth_supported=cephx\\;none" on the command line, but from the
> > QMP side, we would support exactly one auth method, and libvirt w
On Thu, Mar 23, 2017 at 19:08:11 +, Peter Maydell wrote:
> Hi; I thought I'd have a look at the guest_mem_before trace event,
> but if I enable it (by deleting "disable" from the line in trace-events)
> QEMU doesn't compile:
>
> CC arm-softmmu/tcg/tcg-op.o
> In file included from
> /hom
Hi Juan,
First of all, I like the refactor patchset about RAMState, it makes
things clean, great!
On 2017/3/24 5:01, Juan Quintela wrote:
> Hi
>
> This series split the migration and unsent bitmaps by ramblock. This
> makes it easier to synchronize in small bits. This is on top of the
> RAMS
On Mar 23, 2017, at 7:35 AM, qemu-devel-requ...@nongnu.org wrote:
> Message: 15
> Date: Thu, 23 Mar 2017 11:13:02 +
> From: Alex Benn?e
> To: Peter Maydell
> Cc: Bug 1675108 <1675...@bugs.launchpad.net>, QEMU Developers
> , Gerd Hoffmann
> Subject: Re: [Qemu-devel] [Bug 1675108] [NEW
On 03/23/2017 09:02 PM, Peter Maydell wrote:
On 23 March 2017 at 10:33, Paolo Bonzini wrote:
I own a MIPS Creator ci20 board (donated by Imagination Technologies).
I cannot give it a public IP address, but I can try and use it to do
builds every now and then.
I think our best bet is probably
On 2017/3/24 4:45, Juan Quintela wrote:
> We change the meaning of start to be the offset from the beggining of
> the block.
>
> Signed-off-by: Juan Quintela
> ---
> include/exec/ram_addr.h | 2 ++
> migration/ram.c | 8
> 2 files changed, 6 insertions(+), 4 deletions(-)
>
> d
From: Wang guang
The channel socket was initialized manually,
but forgot to set QIO_CHANNEL_FEATURE_SHUTDOWN.
Thus, the colo_process_incoming_thread would hang at recvmsg.
This patch just call qio_channel_socket_new to get channel,
Which set QIO_CHANNEL_FEATURE_SHUTDOWN already.
Signed-off-by:
* Sebastian Ott [2017-03-23 13:02:46 +0100]:
> On Fri, 17 Mar 2017, Dong Jia Shi wrote:
> > Export the common I/O interfaces those are needed by an I/O
> > subchannel driver to actually talk to the subchannel.
> >
> > Reviewed-by: Pierre Morel
> > Signed-off-by: Dong Jia Shi
> > Cc: Sebastian
On Thu, 23 Mar 2017, Paul Durrant wrote:
> Commit 090fa1c8 "add support for unplugging NVMe disks..." extended the
> existing disk unplug flag to cover NVMe disks as well as IDE and SCSI.
>
> The recent thread on the xen-devel mailing list [1] has highlighted that
> this is not desirable behaviour
> Perhaps all trace.o files should be put into their own .a instead of
> being added directly to the linker line:
>
> COMMON_LDADDS = $(trace-obj-y) libqemuutil.a libqemustub.a
>
> I think the linker would only pull in .o files containing symbols that are
> actually referenced by the program.
Hi
On 03/23/2017 04:43 PM, Eric Blake wrote:
> We'd still have to allow libvirt's use of
> ":key=...:auth_supported=cephx\\;none" on the command line, but from the
> QMP side, we would support exactly one auth method, and libvirt will be
> updated to match when it starts targetting blockdev-add inste
On 03/23/2017 03:59 PM, Eric Blake wrote:
> On 03/23/2017 01:10 PM, Eric Blake wrote:
>
>>> # @RbdAuthMethod:
>>> #
>>> # An enumeration of rados auth_supported types
>>> #
>>> # Since: 2.9
>>> ##
>>> -{ 'struct': 'RbdAuthMethod',
>>> - 'data': { 'auth': 'RbdAuthSupport' } }
>>> +{ 'enum':
fw_cfg devices are supposed to be created by the q35 board code,
and not manually using -device. Remove them from the whitelist.
Suggested-by: Laszlo Ersek
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_q35.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35.
An ioapic device is always created by the q35 machine, so
"-device ioapic" and "-device kvm-ioapic" never worked before.
Remove it from the sysbus whitelist.
Signed-off-by: Eduardo Habkost
---
hw/i386/pc_q35.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/hw/i386/pc_q35.c b/hw/i386/pc_q35
Replace the existing has_dynamic_sysbus flag, that makes the
machine accept every single sysbus device type on the
command-line, with a short whitelist.
Most of the machines already have an internal whitelist
implemented using foreach_dynamic_sysbus_device(), so we just
encode the existing behavio
Am 23.03.2017 um 16:35 hat Denis V. Lunev geschrieben:
> On 03/23/2017 06:04 PM, Kevin Wolf wrote:
> > Am 23.03.2017 um 15:17 hat Eric Blake geschrieben:
> >> On 03/23/2017 08:28 AM, Denis V. Lunev wrote:
> >>> ZSDT compression algorithm consumes 3-5 times less CPU power with a
> >> s/ZSDT/ZSTD/
>
Summary
---
This series replaces the existing has_dynamic_sysbus flag (that
makes the machine accept every single sysbus device type on the
command-line) with a short whitelist.
This will be helpful when implementing the new query-device-slots
command, because each machine type will include o
The xen-backend devices created by the Xen code are not supposed
to be treated as dynamic sysbus devices. This is an attempt to
change that and see what happens, but I couldn't test it because
I don't have a Xen host set up.
If this patch breaks anything, this means we have a bug in
foreach_dynami
On Tue, Nov 22, 2016 at 07:10:57AM +0100, Juergen Gross wrote:
> In order to have an easy way to add a new qdev with a specific id
> carve out the needed functionality from qdev_device_add() into a new
> function qdev_set_id().
>
> Signed-off-by: Juergen Gross
> Reviewed-by: Stefano Stabellini
Hi
This series split the migration and unsent bitmaps by ramblock. This
makes it easier to synchronize in small bits. This is on top of the
RAMState and not-hotplug series.
Why?
reason 1:
People have complained that by the time that we detect that a page is
sent, it has already been marked di
On 03/23/2017 01:10 PM, Eric Blake wrote:
>> # @RbdAuthMethod:
>> #
>> # An enumeration of rados auth_supported types
>> #
>> # Since: 2.9
>> ##
>> -{ 'struct': 'RbdAuthMethod',
>> - 'data': { 'auth': 'RbdAuthSupport' } }
>> +{ 'enum': 'RbdAuthMethod',
>> + 'data': [ 'cephx', 'none' ] }
>
Hi,
I am reviewing the existing code that sets has_dynamic_sysbus,
and this patch confused me. I have a question below:
On Tue, Nov 22, 2016 at 07:10:58AM +0100, Juergen Gross wrote:
> Create a qdev plugged to the xen-sysbus for each new backend device.
> This device can be used as a parent for a
Am 23.03.2017 um 11:55 hat Markus Armbruster geschrieben:
> BlockdevOptionsRbd member auth-supported is a list of struct
> RbdAuthMethod, whose only member is enum RbdAuthSupport. There is no
> reason for wrapping the enum in a struct. Delete the wrapper, and
> rename the enum.
>
> Signed-off-by
Hi
This series disable hotplug/unplug during migration. Thank to Markus
for explaining where I had to put the checks. Why? Because during
migration we will fail if there are changes. For instance, in
postcopy, if we add a memory region, we would failing. Same for other
devices if they are not
Hello Stephen, or anyone else affected,
Accepted qemu into xenial-proposed. The package will build now and be
available at https://launchpad.net/ubuntu/+source/qemu/1:2.5+dfsg-
5ubuntu10.10 in a few hours, and then in the -proposed repository.
Please help us by testing this new package. See
http
On 03/24/2017 03:58 AM, Pranith Kumar wrote:
Sending again since I messed by pbonzini's email.
This fixes the bug: 'user-to-root privesc inside VM via bad translation
caching' reported by Jann Horn here:
https://bugs.chromium.org/p/project-zero/issues/detail?id=1122
CC: Richard Henderson
CC: P
Signed-off-by: Juan Quintela
---
migration/ram.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 1be9a6b..4d62788 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -1449,7 +1449,7 @@ void free_xbzrle_decoded_buf(void)
Am 23.03.2017 um 11:55 hat Markus Armbruster geschrieben:
> The conversion from QDict to QemuOpts is pointless. Simply get the
> stuff straight from the QDict.
>
> Signed-off-by: Markus Armbruster
Reviewed-by: Kevin Wolf
Both the ram bitmap and the unsent bitmap are split by RAMBlock.
Signed-off-by: Juan Quintela
---
include/exec/ram_addr.h | 13 +++-
migration/ram.c | 201 ++--
2 files changed, 85 insertions(+), 129 deletions(-)
diff --git a/include/exec/ram
We have disabled memory hotplug, so we don't need to handle
migration_bitamp there.
Signed-off-by: Juan Quintela
---
exec.c | 1 -
include/exec/ram_addr.h | 2 --
migration/ram.c | 34 --
3 files changed, 37 deletions(-)
diff --git a/ex
Only user don't have a MigrationState handly.
Signed-off-by: Juan Quintela
---
include/migration/migration.h | 2 +-
migration/migration.c | 8 +++-
2 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/include/migration/migration.h b/include/migration/migration.h
index 39a8
We are moving everything to work on pages, not addresses.
Signed-off-by: Juan Quintela
---
migration/ram.c | 50 +-
1 file changed, 25 insertions(+), 25 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 57b776b..ef3b428 100644
---
This removes the needto pass also the absolute offset.
Signed-off-by: Juan Quintela
---
migration/ram.c| 56 ++
migration/trace-events | 2 +-
2 files changed, 26 insertions(+), 32 deletions(-)
diff --git a/migration/ram.c b/migration/ram
Until we have reviewed what can/can't be hotplug during migration,
disable it. We can enable it later for the things that we know that
work. For instance, memory hotplug during postcopy don't work
currently.
Signed-off-by: Juan Quintela
---
hw/core/qdev.c | 5 +
qdev-monitor.c | 7 ++-
Signed-off-by: Juan Quintela
---
migration/ram.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index b1a031e..57b776b 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -171,8 +171,8 @@ struct RAMState {
RAMBlock *last
On 03/23/2017 02:18 PM, Eric Blake wrote:
-if (port) {
-/* check for ipv6 */
-if (strchr(host, ':')) {
-strbuf = g_strdup_printf("[%s]:%s", host, port);
-} else {
-strb
From: Chao Fan
The number of dirty pages outputed in 'pages' in the command
'info migrate', so add page-size to calculate the number of dirty
pages in bytes.
Signed-off-by: Chao Fan
Signed-off-by: Li Zhijian
Signed-off-by: Juan Quintela
---
hmp.c | 3 +++
migration/migration.
We use an unsigned long for the page number. Notice that our bitmaps
already got that for the index, so we have that limit.
Signed-off-by: Juan Quintela
---
migration/ram.c | 76 ++---
1 file changed, 34 insertions(+), 42 deletions(-)
diff --
We were setting it far away of when we changed it. Now everything is
done inside save_page_header. Once there, reorganize code to pass
RAMState.
Signed-off-by: Juan Quintela
---
migration/ram.c | 36 +++-
1 file changed, 15 insertions(+), 21 deletions(-)
diff -
Am 23.03.2017 um 11:55 hat Markus Armbruster geschrieben:
> runtime_opts is used for three different purposes:
>
> * qemu_rbd_open() uses it to accept options it recognizes, such as
> "pool" and "image". Other .bdrv_open() methods do it similarly.
>
> * qemu_rbd_open() accepts additional list-
We need to call for the migrate_get_current() in more that half of the
uses, so call that inside.
Signed-off-by: Juan Quintela
---
include/migration/migration.h | 2 +-
migration/migration.c | 6 --
migration/ram.c | 22 ++
migration/savevm.c
Signed-off-by: Juan Quintela
---
include/migration/migration.h | 3 +--
migration/postcopy-ram.c | 6 ++
migration/ram.c | 9 +++--
migration/savevm.c| 3 +--
4 files changed, 7 insertions(+), 14 deletions(-)
diff --git a/include/migration/migration.h b/inc
We change the meaning of start to be the offset from the beggining of
the block.
Signed-off-by: Juan Quintela
---
include/exec/ram_addr.h | 2 ++
migration/ram.c | 8
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/include/exec/ram_addr.h b/include/exec/ram_addr.h
We always use it as pages anyways.
Signed-off-by: Juan Quintela
---
exec.c | 6 +++---
include/exec/ram_addr.h | 2 +-
migration/ram.c | 11 +--
3 files changed, 9 insertions(+), 10 deletions(-)
diff --git a/exec.c b/exec.c
index 9a4c385..2cae288 100644
--- a/
It was used as a size in all cases except one.
Signed-off-by: Juan Quintela
---
exec.c | 4 ++--
include/sysemu/sysemu.h | 2 +-
migration/migration.c| 4 ++--
migration/postcopy-ram.c | 8
migration/savevm.c | 8
5 files changed, 13 insertions(+),
We receive the file from save_live operations and we don't use it
until 3 or 4 levels of calls down.
Signed-off-by: Juan Quintela
---
migration/ram.c | 84 +
1 file changed, 37 insertions(+), 47 deletions(-)
diff --git a/migration/ram.c b/
Rename it to preffer_xbzrle that is a more descriptive name.
Signed-off-by: Juan Quintela
---
migration/ram.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 6a39704..591cf89 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@
Remove it from callers and callees.
Signed-off-by: Juan Quintela
---
migration/ram.c | 27 ++-
1 file changed, 10 insertions(+), 17 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index cb5f06f..064b2c0 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -7
It was on MigrationState when it is only used inside ram.c for
postcopy. Problem is that we need to access it without being able to
pass it RAMState directly.
Signed-off-by: Juan Quintela
---
include/migration/migration.h | 2 --
migration/migration.c | 1 -
migration/ram.c
Signed-off-by: Juan Quintela
---
migration/ram.c | 17 ++---
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index c0d6841..7667e73 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -165,6 +165,8 @@ struct RAMSrcPageRequest {
/*
It can be recalculated from dirty_pages_rate.
Signed-off-by: Juan Quintela
Reviewed-by: Juan Quintela
---
include/migration/migration.h | 1 -
migration/migration.c | 6 +++---
migration/ram.c | 1 -
3 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/include/mi
Treat it like the rest of ram stats counters. Export its value the
same way. As an added bonus, no more MigrationState used in
migration_bitmap_sync();
Signed-off-by: Juan Quintela
Reviewed-by: Juan Quintela
---
include/migration/migration.h | 2 +-
migration/migration.c | 7 +++
Once there, rename the type to be shorter.
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 86 +++--
1 file changed, 47 insertions(+), 39 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index
Signed-off-by: Juan Quintela
---
include/migration/migration.h | 6 ++
migration/migration.c | 5 ++---
migration/ram.c | 13 +
3 files changed, 13 insertions(+), 11 deletions(-)
diff --git a/include/migration/migration.h b/include/migration/migration.h
in
Signed-off-by: Juan Quintela
---
migration/ram.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index cc19406..c398ff9 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -178,6 +178,8 @@ struct RAMState {
uint64_t xbzrle_pages
Just unfold it. Move ram_bytes_remaining() with the rest of exported
functions.
Signed-off-by: Juan Quintela
---
migration/ram.c | 19 +++
1 file changed, 7 insertions(+), 12 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 3ae00e2..dd5a453 100644
--- a/migrati
Signed-off-by: Juan Quintela
---
migration/ram.c | 15 ---
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index a890179..ae2b89f 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -184,6 +184,8 @@ struct RAMState {
uint64_t xbzr
This is a ram field that was inside MigrationState. Move it to
RAMState and make it the same that the other ram stats.
Signed-off-by: Juan Quintela
---
include/migration/migration.h | 2 +-
migration/migration.c | 3 +--
migration/ram.c | 6 +-
3 files changed, 7 inser
Signed-off-by: Juan Quintela
---
migration/ram.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 721fd66..b4e647a 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -174,6 +174,8 @@ struct RAMState {
uint64_t iterations;
And then init only things that are not zero by default.
Signed-off-by: Juan Quintela
---
migration/ram.c | 25 +++--
1 file changed, 3 insertions(+), 22 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index c6ba92c..a890179 100644
--- a/migration/ram.c
+++ b/migr
Once there, remove the now unused AccountingInfo struct and var.
Signed-off-by: Juan Quintela
---
migration/ram.c | 21 +
1 file changed, 5 insertions(+), 16 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index c398ff9..3292eb0 100644
--- a/migration/ram.c
+++ b
This are the last postcopy fields still at MigrationState. Once there
Move MigrationSrcPageRequest to ram.c and remove MigrationState
parameters where appropiate.
Signed-off-by: Juan Quintela
---
include/migration/migration.h | 17 +---
migration/migration.c | 5 +---
migration
Signed-off-by: Juan Quintela
---
migration/ram.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 690ca8f..721fd66 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -172,6 +172,8 @@ struct RAMState {
uint64_t norm_pages;
Signed-off-by: Juan Quintela
---
migration/ram.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 9fa3bd7..690ca8f 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -170,6 +170,8 @@ struct RAMState {
uint64_t zero_pa
Signed-off-by: Juan Quintela
---
migration/ram.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index b4e647a..cc19406 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -176,6 +176,8 @@ struct RAMState {
uint64_t xbzrle_
Somewhere it was passed by reference, just use it from RAMState.
Signed-off-by: Juan Quintela
Reviewed-by: Juan Quintela
---
migration/ram.c | 75 +
1 file changed, 27 insertions(+), 48 deletions(-)
diff --git a/migration/ram.c b/migratio
Its value can be calculated by other exported.
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
include/migration/migration.h | 1 -
migration/migration.c | 3 ++-
migration/ram.c | 5 -
3 files changed, 2 insertions(+), 7 deletions(-)
diff --git a
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 26 ++
1 file changed, 14 insertions(+), 12 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 57f5858..2c36729 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -16
Signed-off-by: Juan Quintela
---
migration/ram.c | 35 +--
1 file changed, 17 insertions(+), 18 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 090084b..872ea23 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -197,6 +197,8 @@ struct RAMStat
For compatibility, we need to still send a value, but just specify it
and comment the fact.
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
include/migration/migration.h | 2 --
migration/migration.c | 3 ++-
migration/ram.c | 11 ---
3 files
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 5e53b47..748d047 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -159,6 +15
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
include/migration/migration.h | 1 -
migration/ram.c | 5 -
2 files changed, 6 deletions(-)
diff --git a/include/migration/migration.h b/include/migration/migration.h
index 5720c88..3e6bb68 100644
--- a/inclu
Signed-off-by: Juan Quintela
---
migration/ram.c | 32 ++--
1 file changed, 18 insertions(+), 14 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 3292eb0..c6ba92c 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -182,6 +182,8 @@ struct RAMState {
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
Reviewed-by: Philippe Mathieu-Daudé
---
migration/ram.c | 13 ++---
1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 826ba6d..d8428c1 100644
--- a/migration/ram.c
+++ b/
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 5881805..5e53b47 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -157,6 +157,8 @@ s
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 23 ---
1 file changed, 12 insertions(+), 11 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 1d5bf22..f811e81 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -45,8
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
Reviewed-by: Philippe Mathieu-Daudé
---
migration/ram.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 748d047..826ba6d 100644
--- a/migration/ram.c
+++ b/migra
Once there rename it to its actual meaning, zero_pages.
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 29 ++---
1 file changed, 18 insertions(+), 11 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index d8428c1..0da13
We need to add a parameter to several functions to make this work.
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 21 +++--
1 file changed, 11 insertions(+), 10 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index a6e90d7..1d
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
---
migration/ram.c | 20 +++-
1 file changed, 11 insertions(+), 9 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index f811e81..5881805 100644
--- a/migration/ram.c
+++ b/migration/ram.c
@@ -154,6 +15
Added doc comments for existing functions comment and rewrite them in
a common style.
Signed-off-by: Juan Quintela
---
migration/ram.c | 348
1 file changed, 227 insertions(+), 121 deletions(-)
diff --git a/migration/ram.c b/migration/ram
So all places are consisten on the nambing of a block name parameter.
Signed-off-by: Juan Quintela
---
migration/ram.c | 17 -
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/migration/ram.c b/migration/ram.c
index 76f1fc4..21047c5 100644
--- a/migration/ram.c
+++ b
We create a struct where to put all the ram state
Start with the following fields:
last_seen_block, last_sent_block, last_offset, last_version and
ram_bulk_stage are globals that are really related together.
Signed-off-by: Juan Quintela
Reviewed-by: Dr. David Alan Gilbert
--
Fix typo and war
Hi
Continuation of previous series, all review comments addressed. New things:
- Consolidate all function comments in the same style (yes, docs)
- Be much more careful with maintaining comments correct
- Move all postcopy fields to RAMState
- Move QEMUFile to RAMState
- rename qemu_target_page_bit
Am 23.03.2017 um 11:55 hat Markus Armbruster geschrieben:
> We have two list-values options:
>
> * "server" is a list of InetSocketAddress. We use members "host" and
> "port", and silently ignore the rest.
>
> * "auth-supported" is a list of RbdAuthMethod. We use its only member
> "auth".
>
Public bug reported:
Hi,
today i try to test qemu 2.9rc 1 with qemu-system-i386 if i set display as sdl
and i push a key on keyboard qemu exit with an error
translate-common.c:34:tcg_handle_interrupt: assertion failed:
(qemu_mutex_iothread_locked())
This issue was not present on qemu 2.8.0
Te
On Thu, Mar 23, 2017 at 04:43:01PM +0100, Thomas Huth wrote:
> On 22.03.2017 20:13, Eduardo Habkost wrote:
> > On Wed, Mar 22, 2017 at 01:00:49PM -0300, Eduardo Habkost wrote:
> >> This series adds scripts/device-crashtest.py, that can be used to
> >> crash-test -device with multiple machine/accel/
Quoting Peter Maydell (2017-03-23 10:13:13)
> Hi; I find that qga-test fails on Linux/sparc host:
>
> test-qga.c:286:test_qga_get_fsinfo: assertion failed
> ret: GenericError opendir("/sys/devices/virtual/block/md0/md0p1/slaves")
>
> On this box /sys/devices/virtual/block/md0/md0p1/ does indeed
>
On Thu, Mar 23, 2017 at 12:42:41PM +, Peter Maydell wrote:
> The REG_PC define in disas/microblaze.c clashes with a define in
> the Linux SPARC system headers:
>
> /home/pm215/qemu/disas/microblaze.c:162:0: error: "REG_PC" redefined [-Werror]
> #define REG_PC 32 /* PC */
>
> In file include
On 03/23/2017 01:27 PM, Markus Armbruster wrote:
>>> -
>>> -num_entries = qdict_array_entries(options, prefix);
>>> +for (i = 0;; i++) {
>>> +sprintf(keybuf, "auth-supported.%d.auth", i);
>>
>> By my count, and including a trailing NUL, this is 21 bytes + the
>> maximum size of a f
On Fri, Mar 24, 2017 at 12:40 AM, Kevin Wolf wrote:
> Am 11.03.2017 um 12:54 hat Ashijeet Acharya geschrieben:
>> The vmdk driver in block/vmdk.c used to allocate cluster by cluster
>> which slowed down its I/O performance.
>>
>> Make vmdk driver allocate multiple clusters at once to reduce the
>>
Am 11.03.2017 um 12:54 hat Ashijeet Acharya geschrieben:
> The vmdk driver in block/vmdk.c used to allocate cluster by cluster
> which slowed down its I/O performance.
>
> Make vmdk driver allocate multiple clusters at once to reduce the
> overhead costs of multiple separate cluster allocation cal
Hi; I thought I'd have a look at the guest_mem_before trace event,
but if I enable it (by deleting "disable" from the line in trace-events)
QEMU doesn't compile:
CC arm-softmmu/tcg/tcg-op.o
In file included from
/home/petmay01/linaro/qemu-from-laptop/qemu/include/trace-tcg.h:4:0,
1 - 100 of 288 matches
Mail list logo
