On 08/20/13 13:43, arm...@redhat.com wrote:
> From: Markus Armbruster
>
> v2: Nominate for qemu-stable (Andreas)
>
> Markus Armbruster (2):
> tests: Fix schema parser test for in-tree build
> tests: Update .gitignore for test-int128 and test-bitops
>
> tests/.gitignore | 3 +++
> tests/Mak
- Original Message -
> On 08/20/2013 09:41 PM, Andrew Jones wrote:
> >> +
> >> +/* This is a workaround for a long standing bug in Linux'
> >> + * mbind implementation, which cuts off the last specified
> >> + * node. To stay compatible should this bug be fixed, we
> >> +
On 20.08.2013, at 12:38, Benjamin Herrenschmidt wrote:
> On Tue, 2013-08-20 at 12:14 +0200, Paolo Bonzini wrote:
>>> I suppose if RH is going to deploy 3.10 and we aren't going to backport
>>> the multitce patches then there *might* be a case for supporting that
>>> combo specifically... but it's
Export RAM_SAVE_xxx flags for localhost migration.
Signed-off-by: Lei Li
---
arch_init.c | 12
include/migration/migration.h | 14 ++
2 files changed, 14 insertions(+), 12 deletions(-)
diff --git a/arch_init.c b/arch_init.c
index 68a7ab7..1ea7c29 1
Hi,
This patch series tries to add a new approach to localhost migration
support in QEMU.
When doing localhost migration, the host memory will balloon up
during the period. This procedure might need to consume twice the
memory needed to run the guest for some time. So we want to add
a new li
Signed-off-by: Lei Li
---
include/migration/migration.h | 10 ++
migration.c |9 -
2 files changed, 10 insertions(+), 9 deletions(-)
diff --git a/include/migration/migration.h b/include/migration/migration.h
index 140e6b4..6a24e65 100644
--- a/include/migr
is_active is used to identify block migration, rename to is_block_active
to make it more clear.
Signed-off-by: Lei Li
---
block-migration.c |2 +-
include/migration/vmstate.h |2 +-
savevm.c| 16
3 files changed, 10 insertions(+), 10 delet
Add comments for qemu_file_get_error(), as its return value
is not very clear.
Signed-off-by: Lei Li
---
savevm.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/savevm.c b/savevm.c
index 6362275..1522d95 100644
--- a/savevm.c
+++ b/savevm.c
@@ -566,6 +566,13 @@ QE
Export qemu_save_device_state() for localhost migration.
Signed-off-by: Lei Li
---
include/sysemu/sysemu.h |1 +
savevm.c|7 ++-
2 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/include/sysemu/sysemu.h b/include/sysemu/sysemu.h
index d7a77b6..f1f15f4 100
Commit 1964a397063967acc5ce71a2a24ed26e74824ee1 refactors rate
limiting to QEMUFile, but set the return value for qemu_file_rate_limit
to 1 in the case of qemu_file_get_error. It is wrong and should be negative
compared to the original function buffered_rate_limit and the current logic
in ram_save_
Implementation of incoming part of localhost migration.
Signed-off-by: Lei Li
---
include/migration/migration.h |2 +
include/migration/qemu-file.h |8 ++--
migration-local.c | 99 +
3 files changed, 105 insertions(+), 4 deletions(-)
Export qemu_get_ram_block() for localhost migration to
get the RAMBlock of given ram address.
Signed-off-by: Lei Li
---
exec.c|7 ++-
include/exec/cpu-common.h |1 +
2 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/exec.c b/exec.c
index 3ca9381..45b0
Introduce migration capability localhost.
Signed-off-by: Lei Li
---
include/migration/migration.h |3 +++
migration.c | 12
qapi-schema.json |8 +++-
3 files changed, 22 insertions(+), 1 deletions(-)
diff --git a/include/migration/migrat
It should set negative error value if there has been an error.
Signed-off-by: Lei Li
---
savevm.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/savevm.c b/savevm.c
index 1522d95..f10e031 100644
--- a/savevm.c
+++ b/savevm.c
@@ -649,7 +649,7 @@ void ram_control_after_i
Factor out ram_save_blocks(), which will be used to send ram
blocks hooked by save_page in ram_save_setup stage in coming
patch.
Signed-off-by: Lei Li
---
arch_init.c | 25 +++--
1 files changed, 15 insertions(+), 10 deletions(-)
diff --git a/arch_init.c b/arch_init.c
inde
Integrate localhost migration thread to migration_thread. As the mechanism
of local migration is different than the current migration, and it does
not need to do iterate.
So the whole thread can split into two stages, it will send all the ram
pages in qemu_savevm_state_begin stage, and send the de
Signed-off-by: Lei Li
---
arch_init.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/arch_init.c b/arch_init.c
index cbbb4db..e6069eb 100644
--- a/arch_init.c
+++ b/arch_init.c
@@ -375,6 +375,11 @@ static void migration_bitmap_sync(void)
int64_t end_time;
i
Implementation of incoming part of localhost migration.
Signed-off-by: Lei Li
---
include/migration/migration.h |2 +
include/migration/qemu-file.h |8 ++--
migration-local.c | 99 +
3 files changed, 105 insertions(+), 4 deletions(-)
Signed-off-by: Lei Li
---
migration.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/migration.c b/migration.c
index 2471664..17cf2fd 100644
--- a/migration.c
+++ b/migration.c
@@ -81,6 +81,9 @@ void qemu_start_incoming_migration(const char *uri, Error
**errp)
Send all the ram blocks hooked by save_page, which will copy
ram page and MADV_DONTNEED the page just copied.
Signed-off-by: Lei Li
---
arch_init.c | 19 +--
1 files changed, 13 insertions(+), 6 deletions(-)
diff --git a/arch_init.c b/arch_init.c
index 434a4ca..cbbb4db 100644
As we have more and more capabilities, better to display
in lines.
Signed-off-by: Lei Li
---
hmp.c |5 ++---
1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/hmp.c b/hmp.c
index c45514b..3544219 100644
--- a/hmp.c
+++ b/hmp.c
@@ -226,13 +226,12 @@ void hmp_info_migrate_capabili
--On 21 August 2013 10:15:52 +0800 Liu Ping Fan wrote:
-void slirp_update_timeout(uint32_t *timeout)
+static void slirp_update_timeout(uint32_t *timeout)
{
-if (!QTAILQ_EMPTY(&slirp_instances)) {
-*timeout = MIN(1000, *timeout);
If you are putting things in macros, you might as
On 08/21/2013 03:15 PM, Andrew Jones wrote:
>
>
> - Original Message -
>> On 08/20/2013 09:41 PM, Andrew Jones wrote:
+
+/* This is a workaround for a long standing bug in Linux'
+ * mbind implementation, which cuts off the last specified
+ * node. To stay
On 08/20/13 21:16, Andreas Färber wrote:
> Am 20.08.2013 20:39, schrieb Luiz Capitulino:
>> On Tue, 20 Aug 2013 18:23:00 +0200
>> Andreas Färber wrote:
>>> Am 20.08.2013 18:10, schrieb Luiz Capitulino:
diff --git a/.gitignore b/.gitignore
index 0fe114d..a8e0f17 100644
--- a/.gitigno
On 08/20/13 03:09, Wanlong Gao wrote:
> On 08/20/2013 06:35 AM, Laszlo Ersek wrote:
>> v1->v2:
>
>
> Tested-by: Wanlong Gao
Thank you.
Laszlo
Implementation of outgoing part for localhost migration.
The integration of migration thread and corresponding
adjustment will be in coming patches.
Signed-off-by: Lei Li
---
include/migration/migration.h |2 +
migration-local.c | 85 +
2
On Wed, 2013-08-14 at 10:02 +0300, Ronen Hod wrote:
> How about adding a flag that tells QEMU whether to pause or reboot the guest
> after the panic?
> We cannot assume that we always have a management layer that takes care
> of this.
> One example is Microsoft's WHQL that deliberately generates a
Introduce read/write backend of QEMUFileLocal used by localhost
migration. The unix domain socket will be replaced by PIPE with
vmsplice mechanism.
Signed-off-by: Lei Li
---
Makefile.objs |1 +
migration-local.c | 211 +
2 files change
On Wed, Aug 21, 2013 at 3:36 PM, Alex Bligh wrote:
>
>
> --On 21 August 2013 10:15:52 +0800 Liu Ping Fan wrote:
>
>> -void slirp_update_timeout(uint32_t *timeout)
>> +static void slirp_update_timeout(uint32_t *timeout)
>> {
>> -if (!QTAILQ_EMPTY(&slirp_instances)) {
>> -*timeout = MI
Il 21/08/2013 00:53, Charlie Shepherd ha scritto:
> On 20/08/2013 21:48, Paolo Bonzini wrote:
>> Il 20/08/2013 20:34, Charlie Shepherd ha scritto:
>>> /* Return true if first block has been changed (ie. current version is
>>> @@ -146,40 +114,82 @@ static inline int is_bit_set(BlockDriverState
>>>
Il 21/08/2013 04:02, Asias He ha scritto:
> In block/gluster.c, we have
>
> gluster_finish_aiocb
> {
>if (retval != sizeof(acb)) {
> qemu_mutex_lock_iothread(); /* We are in gluster thread context */
> ...
> qemu_mutex_unlock_iothread();
>}
> }
>
> qemu tools, e.g. qemu-
On 08/20/13 10:21, Kevin Wolf wrote:
> Am 16.08.2013 um 16:15 hat Laszlo Ersek geschrieben:
>> Paolo asked me to write such a driver based on his textual specification
>> alone. The first patch captures his email in full, the rest re-quotes
>> parts that are being implemented.
>>
>> The tree compil
Il 21/08/2013 10:03, Marcel Apfelbaum ha scritto:
> On Wed, 2013-08-14 at 10:02 +0300, Ronen Hod wrote:
>> How about adding a flag that tells QEMU whether to pause or reboot the guest
>> after the panic?
>> We cannot assume that we always have a management layer that takes care
>> of this.
>> One e
In 4146b46c42e0989cb5842e04d88ab6ccb1713a48 (block: Produce zeros when
protocols reading beyond end of file), we break qemu-iotests ./check
-qcow2 022. This happens because qcow2 temporarily sets ->growable = 1
for vmstate accesses (which are stored beyond the end of regular image
data).
We introd
On Tue, Aug 20, 2013 at 05:59:58PM +0800, Wenchao Xia wrote:
> 于 2013-8-16 16:12, Wenchao Xia 写道:
> >于 2013-8-16 15:15, Wenchao Xia 写道:
> >>于 2013-8-16 0:32, Michael Roth 写道:
> >>>Quoting Michael Roth (2013-08-15 10:23:20)
> Quoting Wenchao Xia (2013-08-13 03:44:39)
> >于 2013-8-13 1:01, Mic
On 21.08.2013, at 08:11, Alexander Graf wrote:
>
> On 20.08.2013, at 12:38, Benjamin Herrenschmidt wrote:
>
>> On Tue, 2013-08-20 at 12:14 +0200, Paolo Bonzini wrote:
I suppose if RH is going to deploy 3.10 and we aren't going to backport
the multitce patches then there *might* be a c
Il 21/08/2013 07:31, Alexey Kardashevskiy ha scritto:
> memory_region_ref(section->mr);
> ===
>
> You want me to move memory_region_ref() earlier to add a reference even if
> two last checks fail?
No, of course not---I was looking at _del source code not _add.
Paolo
On Tue, 2013-08-13 at 11:57 +0200, Markus Armbruster wrote:
> This isn't patch review, just a couple of observations and questions.
>
> Current use of categories, please correct misunderstandings:
>
> * A device can have multiple categories. Most (all?) devices currently
> have exactly one.
Al
Il 21/08/2013 03:22, Wanlong Gao ha scritto:
> On 08/20/2013 09:43 PM, Paolo Bonzini wrote:
>> Il 20/08/2013 03:07, Wanlong Gao ha scritto:
>>> -numa node,nodeid=0,cpus=0, \
>>> -numa mem,size=1024M,policy=membind,host-nodes=0-1 \
>>> -numa node,nodeid=1,cpus=1 \
>>> -numa mem,size=1024M,policy
On Thu, Aug 15, 2013 at 09:34:21PM +0100, Alex Bligh wrote:
> @@ -150,13 +150,14 @@ aio_ctx_prepare(GSource *source, gint*timeout)
> {
> AioContext *ctx = (AioContext *) source;
> QEMUBH *bh;
> +int deadline;
>
> for (bh = ctx->first_bh; bh; bh = bh->next) {
> if
Il 21/08/2013 10:20, Laszlo Ersek ha scritto:
> On 08/20/13 10:21, Kevin Wolf wrote:
>> Am 16.08.2013 um 16:15 hat Laszlo Ersek geschrieben:
>>> Paolo asked me to write such a driver based on his textual specification
>>> alone. The first patch captures his email in full, the rest re-quotes
>>> par
cpu hotplug uses bit 2 of GPE instead of bit 1.
Signed-off-by: Hu Tao
---
src/q35-acpi-dsdt.dsl | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/q35-acpi-dsdt.dsl b/src/q35-acpi-dsdt.dsl
index c031d83..417ac48 100644
--- a/src/q35-acpi-dsdt.dsl
+++ b/src/q35-acpi-dsdt.
OSPM may disable the sci by clearing GPEx_BLK EN bit, in the case
we have to set sci level to 0 or guest will receive sci interrupts
endlessly.
Signed-off-by: Hu Tao
---
hw/acpi/ich9.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
index 3fb443d..8717c15 10
Signed-off-by: Hu Tao
---
hw/acpi/ich9.c | 91 --
include/hw/acpi/ich9.h | 11 ++
2 files changed, 100 insertions(+), 2 deletions(-)
diff --git a/hw/acpi/ich9.c b/hw/acpi/ich9.c
index 8717c15..146216a 100644
--- a/hw/acpi/ich9.c
+++ b/
On Wed, Aug 21, 2013 at 11:03:49AM +0200, Stefan Hajnoczi wrote:
> On Tue, Aug 20, 2013 at 03:57:40PM +0100, Daniel P. Berrange wrote:
> > On Tue, Aug 20, 2013 at 04:43:48PM +0200, Stefan Hajnoczi wrote:
> > > On Fri, Aug 16, 2013 at 06:52:32PM +0530, Deepak C Shetty wrote:
> > > > 2) qemu-img info
On Tue, Aug 20, 2013 at 12:12:21PM +0100, Peter Maydell wrote:
> On 20 August 2013 11:56, Paolo Bonzini wrote:
> > I guess "patches are welcome"---if they are of good quality, why not.
>
> Well, sort of. Given the fairly minimal level of support we currently
> have for OSX (ie Andreas and I test
On 08/21/2013 05:00 PM, Paolo Bonzini wrote:
> Il 21/08/2013 03:22, Wanlong Gao ha scritto:
>> On 08/20/2013 09:43 PM, Paolo Bonzini wrote:
>>> Il 20/08/2013 03:07, Wanlong Gao ha scritto:
-numa node,nodeid=0,cpus=0, \
-numa mem,size=1024M,policy=membind,host-nodes=0-1 \
-numa nod
On Tue, Aug 20, 2013 at 03:57:40PM +0100, Daniel P. Berrange wrote:
> On Tue, Aug 20, 2013 at 04:43:48PM +0200, Stefan Hajnoczi wrote:
> > On Fri, Aug 16, 2013 at 06:52:32PM +0530, Deepak C Shetty wrote:
> > > 2) qemu-img info /var/run/vdsm/017d-1278-4bfb-8129-62bded257399
> > > image: /var/run
Added: Igor Mammedov
On Wed, Aug 21, 2013 at 05:04:28PM +0800, Hu Tao wrote:
>
> Signed-off-by: Hu Tao
> ---
> hw/acpi/ich9.c | 91
> --
> include/hw/acpi/ich9.h | 11 ++
> 2 files changed, 100 insertions(+), 2 deletions(-)
Hi Igor,
On 21/08/2013 09:14, Paolo Bonzini wrote:
Il 21/08/2013 00:53, Charlie Shepherd ha scritto:
What if nb_sectors > 512 * 8?
For cow_co_is_allocated, you have the luxury of returning information
only for the fewer than nb_sectors. That is, you can set *num_same to a
smaller value than nb_sectors,
Il 21/08/2013 11:08, Stefan Hajnoczi ha scritto:
> On Tue, Aug 20, 2013 at 12:12:21PM +0100, Peter Maydell wrote:
>> On 20 August 2013 11:56, Paolo Bonzini wrote:
>>> I guess "patches are welcome"---if they are of good quality, why not.
>>
>> Well, sort of. Given the fairly minimal level of suppor
于 2013-8-20 22:04, Luiz Capitulino 写道:
On Tue, 30 Jul 2013 12:03:11 -0400
Luiz Capitulino wrote:
On Fri, 26 Jul 2013 11:20:29 +0800
Wenchao Xia wrote:
This series make auto completion and help functions works normal for sub
command, by using reentrant functions. In order to do that, global
Marcel Apfelbaum writes:
> On Tue, 2013-08-13 at 11:57 +0200, Markus Armbruster wrote:
>> This isn't patch review, just a couple of observations and questions.
>>
>> Current use of categories, please correct misunderstandings:
>>
>> * A device can have multiple categories. Most (all?) devices
Il 21/08/2013 11:08, Wanlong Gao ha scritto:
Also, do you still need a "-m" option if you use "-numa mem"?
>>>
>>> The "-m" options will be used to compute the memory size of each node
>>> if the memory size of each node is not set by "-numa mem" option. This
>>> is also be consistent with the
I made a couple of small patches while debugging VFIO on SPAPR
which uses IOMMU MemoryRegion 2^64 bytes long.
I was specifically fixing faults with the 2^64 bytes case while there
may be other proglems.
Alexey Kardashevskiy (2):
vfio: Fix debug output for int128 values
vfio: Fix 128 bit hand
Upcoming VFIO on SPAPR PPC64 support will initialize the IOMMU
memory region with UINT64_MAX (2^64 bytes) size so int128_get64()
will assert.
The patch takes care of this check. The existing type1 IOMMU code
is not expected to map all 64 bits of RAM so the patch does not
touch that part.
Signed-o
Memory regions can easily be 2^64 byte long and therefore overflow
for just a bit but that is enough for int128_get64() to assert.
This takes care of debug printing of huge section sizes.
Signed-off-by: Alexey Kardashevskiy
---
hw/misc/vfio.c | 6 --
1 file changed, 4 insertions(+), 2 delet
Alex Williamson writes:
> Signed-off-by: Alex Williamson
> ---
> docs/q35-chipset.cfg | 23 +++
> 1 file changed, 23 insertions(+)
>
> diff --git a/docs/q35-chipset.cfg b/docs/q35-chipset.cfg
> index 1b6efc0..e4ddb7d 100644
> --- a/docs/q35-chipset.cfg
> +++ b/docs/q35-chi
On 08/20/13 03:07, Wanlong Gao wrote:
> V7-V8:
> rebase to current master with Laszlo's V2 of OptsVisitor patch set
> fix an adding white space line error
My R-b's that you've kept from V7, for patches 01 and 02, stand.
Thanks
Laszlo
Am 21.08.2013 11:11, schrieb Paolo Bonzini:
> Still, I must say merging Xen has given us zero headaches. It's taken a
> while, but (at least from the QEMU project's POV) the wait has been
> worthwhile.
>
> I would hope that HAXN support would be almost as easy as adding
> haxn-all.c and haxn-stub.
于 2013-8-21 16:45, Stefan Hajnoczi 写道:
On Tue, Aug 20, 2013 at 05:59:58PM +0800, Wenchao Xia wrote:
于 2013-8-16 16:12, Wenchao Xia 写道:
于 2013-8-16 15:15, Wenchao Xia 写道:
于 2013-8-16 0:32, Michael Roth 写道:
Quoting Michael Roth (2013-08-15 10:23:20)
Quoting Wenchao Xia (2013-08-13 03:44:39)
于
On 08/21/2013 05:22 PM, Paolo Bonzini wrote:
> Il 21/08/2013 11:08, Wanlong Gao ha scritto:
> Also, do you still need a "-m" option if you use "-numa mem"?
The "-m" options will be used to compute the memory size of each node
if the memory size of each node is not set by "-numa m
On Tue, Aug 20, 2013 at 12:21:57PM -0600, Alex Williamson wrote:
> Signed-off-by: Alex Williamson
Thanks, applied.
> ---
> docs/q35-chipset.cfg | 23 +++
> 1 file changed, 23 insertions(+)
>
> diff --git a/docs/q35-chipset.cfg b/docs/q35-chipset.cfg
> index 1b6efc0..e4ddb
On 08/21/2013 05:33 PM, Laszlo Ersek wrote:
> On 08/20/13 03:07, Wanlong Gao wrote:
>
>> V7-V8:
>> rebase to current master with Laszlo's V2 of OptsVisitor patch set
>> fix an adding white space line error
>
> My R-b's that you've kept from V7, for patches 01 and 02, stand.
Yeah, thank y
On Wed, Aug 21, 2013 at 10:18:23AM +0200, Paolo Bonzini wrote:
> Il 21/08/2013 10:03, Marcel Apfelbaum ha scritto:
> > On Wed, 2013-08-14 at 10:02 +0300, Ronen Hod wrote:
> >> How about adding a flag that tells QEMU whether to pause or reboot the
> >> guest
> >> after the panic?
> >> We cannot ass
On Wed, Aug 21, 2013 at 12:42:37PM +0300, Michael S. Tsirkin wrote:
> On Wed, Aug 21, 2013 at 10:18:23AM +0200, Paolo Bonzini wrote:
> > Il 21/08/2013 10:03, Marcel Apfelbaum ha scritto:
> > > On Wed, 2013-08-14 at 10:02 +0300, Ronen Hod wrote:
> > >> How about adding a flag that tells QEMU whether
Il 21/08/2013 11:42, Michael S. Tsirkin ha scritto:
> On Wed, Aug 21, 2013 at 10:18:23AM +0200, Paolo Bonzini wrote:
>> Il 21/08/2013 10:03, Marcel Apfelbaum ha scritto:
>>> On Wed, 2013-08-14 at 10:02 +0300, Ronen Hod wrote:
How about adding a flag that tells QEMU whether to pause or reboot t
Il 21/08/2013 11:34, Wanlong Gao ha scritto:
> On 08/21/2013 05:22 PM, Paolo Bonzini wrote:
>> Il 21/08/2013 11:08, Wanlong Gao ha scritto:
>> Also, do you still need a "-m" option if you use "-numa mem"?
>
> The "-m" options will be used to compute the memory size of each node
> if
Il 21/08/2013 11:33, Stefan Weil ha scritto:
> Am 21.08.2013 11:11, schrieb Paolo Bonzini:
>> Still, I must say merging Xen has given us zero headaches. It's taken a
>> while, but (at least from the QEMU project's POV) the wait has been
>> worthwhile.
>>
>> I would hope that HAXN support would be
--On 21 August 2013 10:45:53 +0200 Stefan Hajnoczi
wrote:
Since AioContext provides a GSource, integrating with an application's
glib event loop should also be easy. The only hard part is timers,
since we use nanosecond timers - there we should just round up to
millisecond granularity to l
Stefan,
--On 21 August 2013 11:01:42 +0200 Stefan Hajnoczi
wrote:
I agree with Wenchao:
The docs explicitly say that .prepare() can set timeout to the maximum
timeout value that is required. It then explains that the "actual
timeout used" is the minimum of all timeout values - it's not our
Il 21/08/2013 11:28, Alexey Kardashevskiy ha scritto:
> Upcoming VFIO on SPAPR PPC64 support will initialize the IOMMU
> memory region with UINT64_MAX (2^64 bytes) size so int128_get64()
> will assert.
>
> The patch takes care of this check. The existing type1 IOMMU code
> is not expected to map a
Il 21/08/2013 11:11, Charlie Shepherd ha scritto:
>>>
>> For cow_co_is_allocated, you have the luxury of returning information
>> only for the fewer than nb_sectors. That is, you can set *num_same to a
>> smaller value than nb_sectors, even if sector_num + *num_same has the
>> same state as the [s
On Wed, Aug 21, 2013 at 11:59:36AM +0200, Paolo Bonzini wrote:
> Il 21/08/2013 11:42, Michael S. Tsirkin ha scritto:
> > On Wed, Aug 21, 2013 at 10:18:23AM +0200, Paolo Bonzini wrote:
> >> Il 21/08/2013 10:03, Marcel Apfelbaum ha scritto:
> >>> On Wed, 2013-08-14 at 10:02 +0300, Ronen Hod wrote:
>
On 21/08/2013 10:19, Paolo Bonzini wrote:
Il 21/08/2013 11:11, Charlie Shepherd ha scritto:
It still seems
worthwhile to me to be as efficient as possible, I guess that means
processing a sector's worth of metadata at a time?
Yes, that's what my patches do. My is_allocated and flushing strateg
Il 21/08/2013 12:16, Michael S. Tsirkin ha scritto:
> >
> > By the way, this means two things:
> >
> > - I am now sold on the idea that explicitly enabling of pvpanic is the
> > right thing to do;
> >
> > - on the other hand this is the proof that the change was not fully
> > understood, and rus
On 08/05/13 15:03, Paolo Bonzini wrote:
>
> [...]
>
> 1) BlockDriver is a struct in which these function members are
> interesting:
>
> .bdrv_reopen_prepare
> .bdrv_co_readv
> .bdrv_co_writev
> .bdrv_co_is_allocated
> .bdrv_co_write_zeroes
> .bdrv_co_discard
> .bdrv_getl
On 08/05/13 15:03, Paolo Bonzini wrote:
>
> [...]
>
> 3) These members are special
>
> .format_name is the string "raw"
> .bdrv_open raw_open should set bs->sg to bs->file->sg and return 0
> .bdrv_closeraw_close should do nothing
> .bdrv_proberaw_probe should just retu
On 08/05/13 15:03, Paolo Bonzini wrote:
>
> [...]
>
> 2) This is also a simple forwarder function:
>
> .bdrv_create
>
> but there is no BlockDriverState argument so the forwarded-to function
> does not have a bs->file argument either. The forwarded-to function is
> bdrv_create_file.
Signed-of
v1->v2:
- added comment to raw_probe() [Kevin]
- fixed raw_reopen_prepare() [Paolo & Kevin]
- keeping BSDL: according to the v1 discussion, LGPLv2+ could work too,
but "but most of the block layer is BSD" [Paolo] and I haven't been
clearly instructed to switch to LGPLv2+.
diff --git a/block/
On 08/05/13 15:03, Paolo Bonzini wrote:
>
>
> - Original Message -
>> From: "Laszlo Ersek"
>> To: "Paolo Bonzini"
>> Sent: Monday, August 5, 2013 2:43:46 PM
>> Subject: Re: [PATCH 1/2] raw: add license header
>>
>> On 08/02/13 00:27, Paolo Bonzini wrote:
>>> On 08/01/2013 10:13 AM, Christ
Il 21/08/2013 09:18, Lei Li ha scritto:
> It should set negative error value if there has been an error.
>
> Signed-off-by: Lei Li
> ---
> savevm.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/savevm.c b/savevm.c
> index 1522d95..f10e031 100644
> --- a/savevm.c
On 08/05/13 15:03, Paolo Bonzini wrote:
>
> [...]
>
> 4) There is another member, .create_options, which is an array of
> QEMUOptionParameter structs, terminated by an all-zero item. The only
> option you need is for the virtual disk size. You will find something
> to copy from in other block dri
"Incoming" function prototypes and "outgoing" function calls must match
reality. Implemented using the "struct BlockDriver" definition in
"include/block/block_int.h", and gcc errors & warnings.
v1->v2:
On 08/20/13 09:51, Kevin Wolf wrote:
> Am 18.08.2013 um 16:29 hat Paolo Bonzini geschrieben:
>>
On 08/05/13 15:03, Paolo Bonzini wrote:
>
> [...]
>
> 5) Formats are registered with bdrv_register (takes a BlockDriver*). You
> also need to pass the caller of bdrv_register to block_init.
Fill in the BlockDriver structure with the raw_*() functions that have
been added to "block/raw_bsd.c", in t
Il 21/08/2013 09:18, Lei Li ha scritto:
> Commit 1964a397063967acc5ce71a2a24ed26e74824ee1 refactors rate
> limiting to QEMUFile, but set the return value for qemu_file_rate_limit
> to 1 in the case of qemu_file_get_error. It is wrong and should be negative
> compared to the original function buffer
Il 21/08/2013 09:18, Lei Li ha scritto:
> Add comments for qemu_file_get_error(), as its return value
> is not very clear.
>
> Signed-off-by: Lei Li
> ---
> savevm.c |7 +++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/savevm.c b/savevm.c
> index 6362275..1522d95
Il 21/08/2013 12:31, Charlie Shepherd ha scritto:
> On 21/08/2013 10:19, Paolo Bonzini wrote:
>> Il 21/08/2013 11:11, Charlie Shepherd ha scritto:
>>> It still seems
>>> worthwhile to me to be as efficient as possible, I guess that means
>>> processing a sector's worth of metadata at a time?
>> Yes
On Wed, 2013-08-21 at 11:23 +0200, Markus Armbruster wrote:
> Marcel Apfelbaum writes:
>
> > On Tue, 2013-08-13 at 11:57 +0200, Markus Armbruster wrote:
> >> This isn't patch review, just a couple of observations and questions.
> >>
> >> Current use of categories, please correct misunderstanding
Il 21/08/2013 09:18, Lei Li ha scritto:
> Implementation of outgoing part for localhost migration.
> The integration of migration thread and corresponding
> adjustment will be in coming patches.
>
> Signed-off-by: Lei Li
> ---
> include/migration/migration.h |2 +
> migration-local.c
Hey guys,
I've put up a wiki page with a kvm PCI todo list,
mainly to avoid effort duplication, but also in the hope
to draw attention to what I think we should try addressing
in KVM:
http://www.linux-kvm.org/page/PCITodo
This page could cover all PCI related activity in KVM,
it is very incomplet
Il 21/08/2013 09:18, Lei Li ha scritto:
> Integrate localhost migration thread to migration_thread. As the mechanism
> of local migration is different than the current migration, and it does
> not need to do iterate.
>
> So the whole thread can split into two stages, it will send all the ram
> pag
Il 21/08/2013 09:18, Lei Li ha scritto:
> Send all the ram blocks hooked by save_page, which will copy
> ram page and MADV_DONTNEED the page just copied.
You should implement this entirely in the hook.
It will be a little less efficient because of the dirty bitmap overhead,
but you should aim at
Il 21/08/2013 09:18, Lei Li ha scritto:
> Export RAM_SAVE_xxx flags for localhost migration.
>
> Signed-off-by: Lei Li
> ---
> arch_init.c | 12
> include/migration/migration.h | 14 ++
> 2 files changed, 14 insertions(+), 12 deletions(-)
>
> diff
Il 21/08/2013 09:18, Lei Li ha scritto:
> Signed-off-by: Lei Li
> ---
> arch_init.c |5 +
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/arch_init.c b/arch_init.c
> index cbbb4db..e6069eb 100644
> --- a/arch_init.c
> +++ b/arch_init.c
> @@ -375,6 +375,11 @@ static vo
Il 21/08/2013 09:18, Lei Li ha scritto:
> Signed-off-by: Lei Li
> ---
> migration.c |3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/migration.c b/migration.c
> index 2471664..17cf2fd 100644
> --- a/migration.c
> +++ b/migration.c
> @@ -81,6 +81,9 @@ void qemu_star
Hi,
> +#define ICH9_PROC_BASE 0xaf00
> +#define ICH9_PROC_LEN 32
No, please don't. It makes it impossible to assign the 0xa000 -> 0xafff
I/O port window to a PCI bridge. Please lets stop occupy random io
ports above 0x1000 and burn I/O address space that way.
I'd suggest to place it at 0x0a0
Hi,
> Method(_L01) {
> +}
> +Method(_E02) {
> // CPU hotplug event
> \_SB.PRSC()
> }
> -Method(_L02) {
> -}
E02? Typo?
cheers,
Gerd
Il 21/08/2013 09:18, Lei Li ha scritto:
> Export qemu_save_device_state() for localhost migration.
>
> Signed-off-by: Lei Li
> ---
> include/sysemu/sysemu.h |1 +
> savevm.c|7 ++-
> 2 files changed, 7 insertions(+), 1 deletions(-)
>
> diff --git a/include/sysemu/sys
Il 21/08/2013 13:01, Gerd Hoffmann ha scritto:
> Hi,
>
>> +#define ICH9_PROC_BASE 0xaf00
>> +#define ICH9_PROC_LEN 32
>
> No, please don't. It makes it impossible to assign the 0xa000 -> 0xafff
> I/O port window to a PCI bridge. Please lets stop occupy random io
> ports above 0x1000 and burn
1 - 100 of 255 matches
Mail list logo