On 11/18/13 12:53, Michael S. Tsirkin wrote:
From: Igor Mammedov imamm...@redhat.com
The BIOS that we ship in 1.7 does not use pci info
from host and so far isn't going to use it.
Taking in account problems it caused see 9604f70fdf and
to avoid future incompatibility issues, it's safest to
On Tue, Nov 26, 2013 at 5:22 PM, liguang lig.f...@cn.fujitsu.com wrote:
Signed-off-by: liguang lig.f...@cn.fujitsu.com
---
default-configs/arm-softmmu.mak |2 +
hw/timer/Makefile.objs |1 +
hw/timer/sunxi-pit.c| 295
+++
we currently do not check if a sector is allocated during convert.
This means if a sector is unallocated that we allocate a bounce
buffer of zeroes, find out its zero later and do not write it
in the best case. In the worst case this can lead to reading
blocks from a raw device (like iSCSI)
Signed-off-by: Peter Lieven p...@kamp.de
---
include/block/block_int.h |3 +++
1 file changed, 3 insertions(+)
diff --git a/include/block/block_int.h b/include/block/block_int.h
index 95140b6..6499516 100644
--- a/include/block/block_int.h
+++ b/include/block/block_int.h
@@ -242,6 +242,9 @@
Reviewed-by: Eric Blake ebl...@redhat.com
Signed-off-by: Peter Lieven p...@kamp.de
---
qemu-img.c |1 -
1 file changed, 1 deletion(-)
diff --git a/qemu-img.c b/qemu-img.c
index efb744c..e2d1a0a 100644
--- a/qemu-img.c
+++ b/qemu-img.c
@@ -105,7 +105,6 @@ static void help(void)
this patch aims to set bdi-cluster_size to the internal page size
of the iscsi target so that enabled callers can align requests
properly.
Reviewed-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: Peter Lieven p...@kamp.de
---
block/iscsi.c |7 +++
1 file changed, 7 insertions(+)
Signed-off-by: Peter Lieven p...@kamp.de
---
block/iscsi.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/block/iscsi.c b/block/iscsi.c
index 75d6b87..1109d8c 100644
--- a/block/iscsi.c
+++ b/block/iscsi.c
@@ -1457,6 +1457,9 @@ static int iscsi_open(BlockDriverState *bs, QDict
Suggested-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: Peter Lieven p...@kamp.de
---
qemu-img.c|4 ++--
qemu-img.texi |2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index cc7540f..6f1179b 100644
--- a/qemu-img.c
+++ b/qemu-img.c
this series adds some optimizations for qemu-img during convert which
have been developed recently:
- skipping input based on get_block_status
- variable I/O buffer size
- align write requests to cluster_size
- show progress in sectors or percent
v1-v2:
- introduce opt_transfer_length in
since the convert process is basically a sync operation it might
be benificial in some case to change the hardcoded I/O buffer
size to a greater value.
This patch increases the I/O buffer size if the output
driver advertises an optimal transfer length or discard alignment
that is greater than the
Signed-off-by: Peter Lieven p...@kamp.de
---
qemu-img-cmds.hx |4 ++--
qemu-img.c | 31 ---
qemu-img.texi|4 +++-
3 files changed, 33 insertions(+), 6 deletions(-)
diff --git a/qemu-img-cmds.hx b/qemu-img-cmds.hx
index da1d965..6c8183b 100644
---
this patch shortens requests to end at an aligned sector so that
the next request starts aligned.
Signed-off-by: Peter Lieven p...@kamp.de
---
qemu-img.c | 30 --
1 file changed, 20 insertions(+), 10 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index
Peter Crosthwaite wrote:
On Tue, Nov 26, 2013 at 5:22 PM, liguanglig.f...@cn.fujitsu.com wrote:
Signed-off-by: liguanglig.f...@cn.fujitsu.com
---
default-configs/arm-softmmu.mak |2 +
hw/timer/Makefile.objs |1 +
hw/timer/sunxi-pit.c| 295
Il 25/11/2013 20:28, Max Reitz ha scritto:
This series fixes the drive-mirror blockjob in case of none sync mode
to always use the old (current) image file as the backing file of the
newly created mirrored file (in case of absolute-paths mode).
It is rather important to get this into 1.7, as
On Tue, Nov 26, 2013 at 5:22 PM, liguang lig.f...@cn.fujitsu.com wrote:
Signed-off-by: liguang lig.f...@cn.fujitsu.com
---
default-configs/arm-softmmu.mak |1 +
hw/intc/Makefile.objs |1 +
hw/intc/sunxi-pic.c | 238
+++
Il 25/11/2013 22:05, Michael S. Tsirkin ha scritto:
Signed-off-by: Marcelo Tosatti mtosa...@redhat.com
[Reorganize code, keep same logic. - Paolo]
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
BTW how about a unit-test for this?
Can be something along the lines of the acpi tests:
run
On Tue, Nov 26, 2013 at 09:12:50AM +0100, Laszlo Ersek wrote:
On 11/18/13 12:53, Michael S. Tsirkin wrote:
From: Igor Mammedov imamm...@redhat.com
The BIOS that we ship in 1.7 does not use pci info
from host and so far isn't going to use it.
Taking in account problems it caused see
Peter Crosthwaite wrote:
On Tue, Nov 26, 2013 at 5:22 PM, liguanglig.f...@cn.fujitsu.com wrote:
Signed-off-by: liguanglig.f...@cn.fujitsu.com
---
default-configs/arm-softmmu.mak |1 +
hw/intc/Makefile.objs |1 +
hw/intc/sunxi-pic.c | 238
On Tue, Nov 26, 2013 at 6:59 PM, Li Guang lig.f...@cn.fujitsu.com wrote:
Peter Crosthwaite wrote:
On Tue, Nov 26, 2013 at 5:22 PM, liguanglig.f...@cn.fujitsu.com wrote:
Signed-off-by: liguanglig.f...@cn.fujitsu.com
---
default-configs/arm-softmmu.mak |2 +
hw/timer/Makefile.objs
Peter Crosthwaite wrote:
On Tue, Nov 26, 2013 at 6:59 PM, Li Guanglig.f...@cn.fujitsu.com wrote:
Peter Crosthwaite wrote:
On Tue, Nov 26, 2013 at 5:22 PM, liguanglig.f...@cn.fujitsu.com wrote:
Signed-off-by: liguanglig.f...@cn.fujitsu.com
---
Il 26/11/2013 03:07, Fam Zheng ha scritto:
+void bdrv_op_unblock(BlockDriverState *bs, BlockOpType op, Error
*reason)
What about making BlockOpType a bitmask, and having bdrv_op_{,un}block
take multiple ORed BlockOpTypes?
bdrv_op_{,un}block_all then are not necessary, you only need a
On Tue, Nov 26, 2013 at 5:22 PM, liguang lig.f...@cn.fujitsu.com wrote:
Signed-off-by: liguang lig.f...@cn.fujitsu.com
---
hw/arm/Makefile.objs |1 +
hw/arm/sunxi-soc.c | 98
++
2 files changed, 99 insertions(+), 0 deletions(-)
On Tue, Nov 26, 2013 at 7:19 PM, Li Guang lig.f...@cn.fujitsu.com wrote:
Peter Crosthwaite wrote:
On Tue, Nov 26, 2013 at 6:59 PM, Li Guanglig.f...@cn.fujitsu.com wrote:
Peter Crosthwaite wrote:
On Tue, Nov 26, 2013 at 5:22 PM, liguanglig.f...@cn.fujitsu.com
wrote:
Signed-off-by:
Il 26/11/2013 09:56, Peter Lieven ha scritto:
Reviewed-by: Eric Blake ebl...@redhat.com
Signed-off-by: Peter Lieven p...@kamp.de
---
qemu-img.c |1 -
1 file changed, 1 deletion(-)
diff --git a/qemu-img.c b/qemu-img.c
index efb744c..e2d1a0a 100644
--- a/qemu-img.c
+++ b/qemu-img.c
Il 26/11/2013 09:56, Peter Lieven ha scritto:
Signed-off-by: Peter Lieven p...@kamp.de
---
block/iscsi.c |3 +++
1 file changed, 3 insertions(+)
diff --git a/block/iscsi.c b/block/iscsi.c
index 75d6b87..1109d8c 100644
--- a/block/iscsi.c
+++ b/block/iscsi.c
@@ -1457,6 +1457,9 @@
Il 26/11/2013 09:56, Peter Lieven ha scritto:
Signed-off-by: Peter Lieven p...@kamp.de
---
include/block/block_int.h |3 +++
1 file changed, 3 insertions(+)
diff --git a/include/block/block_int.h b/include/block/block_int.h
index 95140b6..6499516 100644
---
Il 26/11/2013 09:56, Peter Lieven ha scritto:
since the convert process is basically a sync operation it might
be benificial in some case to change the hardcoded I/O buffer
size to a greater value.
This patch increases the I/O buffer size if the output
driver advertises an optimal transfer
Il 26/11/2013 09:56, Peter Lieven ha scritto:
this patch shortens requests to end at an aligned sector so that
the next request starts aligned.
Signed-off-by: Peter Lieven p...@kamp.de
---
qemu-img.c | 30 --
1 file changed, 20 insertions(+), 10 deletions(-)
Il 26/11/2013 09:56, Peter Lieven ha scritto:
Suggested-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: Peter Lieven p...@kamp.de
---
qemu-img.c|4 ++--
qemu-img.texi |2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/qemu-img.c b/qemu-img.c
index
On Thu, Nov 14, 2013 at 07:40:31AM -0600, Corey Minyard wrote:
On 11/14/2013 07:38 AM, Michael S. Tsirkin wrote:
On Thu, Nov 14, 2013 at 07:28:00AM -0600, Corey Minyard wrote:
On 11/14/2013 01:30 AM, Michael S. Tsirkin wrote:
On Tue, Nov 12, 2013 at 10:33:13AM -0600, Corey Minyard wrote:
On Thu, Nov 21, 2013 at 05:02:07PM -0500, Gabriel L. Somlo wrote:
Added qemu-devel, since that is where this stuff belongs now. Everyone
else, sorry for the dupe...
On Thu, Nov 21, 2013 at 07:14:27PM +0100, Paolo Bonzini wrote:
Can you remind us about your DSDT modifications? It should be
Il 26/11/2013 09:56, Peter Lieven ha scritto:
we currently do not check if a sector is allocated during convert.
This means if a sector is unallocated that we allocate a bounce
buffer of zeroes, find out its zero later and do not write it
in the best case. In the worst case this can lead to
I think the right way to do this would be to add the functionality to
qemu-progress.c (i.e. pass a number of sectors and let it choose between
printing % or sectors).
I don't like the qemu-progress.c API anyway, so a rewrite would be
welcome. :)
Paolo
Il 26/11/2013 09:56, Peter Lieven ha
On Fri, Nov 22, 2013 at 10:00:20AM +0100, Paolo Bonzini wrote:
Il 21/11/2013 23:02, Gabriel L. Somlo ha scritto:
On Thu, Nov 21, 2013 at 07:14:27PM +0100, Paolo Bonzini wrote:
Can you remind us about your DSDT modifications? It should be possible
to patch the HPET and applesmc bits
Il 26/11/2013 11:19, Fam Zheng ha scritto:
So that you can add the same Error to multiple items of the array with a
single bdrv_op_block call (by ORing them into the second parameter).
What data type to contain this? I'm not sure 64 is enough in long term...
I would just use an uint64_t, I
On Tue, Nov 26, 2013 at 6:25 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 26/11/2013 11:19, Fam Zheng ha scritto:
So that you can add the same Error to multiple items of the array with a
single bdrv_op_block call (by ORing them into the second parameter).
What data type to contain this?
On Tue, Nov 26, 2013 at 5:22 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 26/11/2013 03:07, Fam Zheng ha scritto:
+void bdrv_op_unblock(BlockDriverState *bs, BlockOpType op, Error
*reason)
What about making BlockOpType a bitmask, and having bdrv_op_{,un}block
take multiple ORed
Il 26/11/2013 11:31, Fam Zheng ha scritto:
I would just use an uint64_t, I think you have ~20 operations now.
There is still quite some breathing room.
OK. Then, can we still use QAPI enum? Because I think being possible
to query blockers makes sense, that way the user doesn't have to
Strictly speaking, this is only required for has_zero_init() == false,
but it's easy enough to just do a cluster-aligned write that is padded
with zeros after the header.
This fixes that after 'qemu-img create' header extensions are attempted
to be parsed that are really just random leftover
Hi,
I think it's down to other qemu bugs (such as _CRS not covering
all of PCI memory), we shall just fix them.
i.e. the attached patch should fixup things.
cheers,
Gerd
From a81b8d66e24fd298ce7654d424a378337e6cf132 Mon Sep 17 00:00:00 2001
From: Gerd Hoffmann kra...@redhat.com
Date:
Am 26.11.2013 um 04:06 hat Fam Zheng geschrieben:
On 2013年11月25日 19:23, Kevin Wolf wrote:
Am 22.11.2013 um 06:24 hat Fam Zheng geschrieben:
Drive backup is a read only operation on source bs. We want to allow
this specific case to enable image-fleecing. Note that when
image-fleecing job
On Tue, Nov 26, 2013 at 6:41 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 26/11/2013 11:31, Fam Zheng ha scritto:
I would just use an uint64_t, I think you have ~20 operations now.
There is still quite some breathing room.
OK. Then, can we still use QAPI enum? Because I think being
On 11/25/2013 05:48 PM, Paolo Bonzini wrote:
Il 25/11/2013 08:29, Lei Li ha scritto:
In this case, if the migration would fail just because the misconfiguration
of device state on destination, in the meantime the outgoing migration has
no aware of this failure, I think it should add such
Il 26/11/2013 12:07, Lei Li ha scritto:
Basically, -incoming without -S is a broken option because of the
missing handshake at the end of migration. With -S something else
(either a human or a program) can check that everything went well and
choose whether to restart the source or the
Il 21/11/2013 10:11, Lei Li ha scritto:
This patch implements save_page callback for the outside
of page flipping. It will write the address of the page
on the Unix socket and flip the page data on pipe by
vmsplice(). Every page address would have a header flag
RAM_SAVE_FLAG_HOOK, which will
On 11/25/2013 03:22 PM, Sebastian Huber wrote:
The env-pc is not necessarily up-to-date in the helper function. Use
the program counter of the disassembly context instead.
Looks good. Thanks Sebastian.
Reviewed-by: Fabien Chouteau chout...@adacore.com
Signed-off-by: Sebastian Huber
Il 21/11/2013 10:11, Lei Li ha scritto:
+static int qemu_local_ram_load(QEMUFile *f, void *opaque,
+ uint64_t flags)
+{
+QEMUFileLocal *s = opaque;
+ram_addr_t addr;
+struct iovec iov;
+ssize_t ret = -EINVAL;
+
+/*
+ * PIPE file
Il 21/11/2013 10:11, Lei Li ha scritto:
+struct cmsghdr *cmptr;
+char req[1] = { 0x01 };
About this, see my reply to patch 8.
+if (pipefd 0) {
+msg.msg_control = NULL;
+msg.msg_controllen = 0;
+/* Negative status means error */
+req[0] =
On 11/21/2013 05:11 PM, Lei Li wrote:
The control message for exchange of pipe file descriptor should
be received by recvmsg, and it might be eaten to stream file by
qemu_recv() when receiving by two callbacks. So this patch adds
unix_msgfd_lookup() to callback get_buffer as the only one
Il 21/11/2013 10:11, Lei Li ha scritto:
+/*
+ * recvmsg is called here to catch the control message for
+ * the exchange of PIPE file descriptor until it is received.
+ */
+len = recvmsg(s-sockfd, msg, 0);
+if (len !=
Il 21/11/2013 10:11, Lei Li ha scritto:
Signed-off-by: Lei Li li...@linux.vnet.ibm.com
---
migration.c | 10 +++---
1 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/migration.c b/migration.c
index 4ac466b..0f98ac1 100644
--- a/migration.c
+++ b/migration.c
@@ -579,10
On 09/27/2013 09:42 PM, Richard Henderson wrote:
On 09/26/2013 05:48 PM, Alexander Graf wrote:
+if (setflags) {
+tcg_dst = cpu_reg(dest);
+} else {
+tcg_dst = cpu_reg_sp(dest);
+}
Never sp for logicals.
This should be ok in my view, the manual explicitly shows
On 11/26/2013 07:32 PM, Paolo Bonzini wrote:
Il 21/11/2013 10:11, Lei Li ha scritto:
Signed-off-by: Lei Li li...@linux.vnet.ibm.com
---
migration.c | 10 +++---
1 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/migration.c b/migration.c
index 4ac466b..0f98ac1 100644
---
On Tue, Nov 26, 2013 at 12:56 PM, Claudio Fontana
claudio.font...@linaro.org wrote:
On 09/27/2013 09:42 PM, Richard Henderson wrote:
On 09/26/2013 05:48 PM, Alexander Graf wrote:
+if (setflags) {
+tcg_dst = cpu_reg(dest);
+} else {
+tcg_dst = cpu_reg_sp(dest);
+}
On 11/26/2013 07:22 PM, Paolo Bonzini wrote:
Il 21/11/2013 10:11, Lei Li ha scritto:
This patch implements save_page callback for the outside
of page flipping. It will write the address of the page
on the Unix socket and flip the page data on pipe by
vmsplice(). Every page address would have a
On 11/26/2013 07:25 PM, Paolo Bonzini wrote:
Il 21/11/2013 10:11, Lei Li ha scritto:
+static int qemu_local_ram_load(QEMUFile *f, void *opaque,
+ uint64_t flags)
+{
+QEMUFileLocal *s = opaque;
+ram_addr_t addr;
+struct iovec iov;
+ssize_t ret =
On 2013年11月26日 18:48, Kevin Wolf wrote:
Strictly speaking, this is only required for has_zero_init() == false,
but it's easy enough to just do a cluster-aligned write that is padded
with zeros after the header.
This fixes that after 'qemu-img create' header extensions are attempted
to be parsed
ping
http://patchwork.ozlabs.org/patch/287211/
http://patchwork.ozlabs.org/patch/287213/
Can we get these two changes in 1.7?
It has been reviewed and waiting for someone who has commits rights for two
months now.
Regards,
Petar
From: Petar Jovanovic
On 26.11.2013 11:04, Paolo Bonzini wrote:
I think the right way to do this would be to add the functionality to
qemu-progress.c (i.e. pass a number of sectors and let it choose between
printing % or sectors).
I was thinking about the same, but is this not beyond the scope of this patch?
;-)
I
Il 21/11/2013 10:11, Lei Li ha scritto:
{ RUN_STATE_DEBUG, RUN_STATE_SUSPENDED },
DEBUG - MEMORY_STALE is missing.
Paolo
{ RUN_STATE_RUNNING, RUN_STATE_SUSPENDED },
{ RUN_STATE_SUSPENDED, RUN_STATE_RUNNING },
{ RUN_STATE_SUSPENDED, RUN_STATE_FINISH_MIGRATE },
+{
Laszlo Ersek ler...@redhat.com writes:
On 11/25/13 16:22, Markus Armbruster wrote:
Laszlo Ersek ler...@redhat.com writes:
On 11/22/13 13:21, Markus Armbruster wrote:
Laszlo Ersek ler...@redhat.com writes:
This patch allows the user to usefully specify
-drive
Il 26/11/2013 13:23, Peter Lieven ha scritto:
I think the right way to do this would be to add the functionality to
qemu-progress.c (i.e. pass a number of sectors and let it choose between
printing % or sectors).
I was thinking about the same, but is this not beyond the scope of this
patch?
Hi all,
When guest set irq smp_affinity, VMEXIT occurs, then the vcpu thread will IOCTL
return to QEMU from hypervisor, then vcpu thread ask the hypervisor to update
the irq routing table,
in kvm_set_irq_routing, synchronize_rcu is called, current vcpu thread is
blocked for so much time to
Il 26/11/2013 13:40, Zhanghaoyu (A) ha scritto:
When guest set irq smp_affinity, VMEXIT occurs, then the vcpu thread will
IOCTL return to QEMU from hypervisor, then vcpu thread ask the hypervisor to
update the irq routing table,
in kvm_set_irq_routing, synchronize_rcu is called, current vcpu
On Tue, Nov 26, 2013 at 12:40:36PM +, Zhanghaoyu (A) wrote:
Hi all,
When guest set irq smp_affinity, VMEXIT occurs, then the vcpu thread will
IOCTL return to QEMU from hypervisor, then vcpu thread ask the hypervisor to
update the irq routing table,
Why vcpu thread ask the hypervisor to
On Tue, Nov 26, 2013 at 02:48:10PM +0200, Gleb Natapov wrote:
On Tue, Nov 26, 2013 at 12:40:36PM +, Zhanghaoyu (A) wrote:
Hi all,
When guest set irq smp_affinity, VMEXIT occurs, then the vcpu thread will
IOCTL return to QEMU from hypervisor, then vcpu thread ask the hypervisor
to
Laszlo Ersek ler...@redhat.com writes:
On 11/22/13 13:21, Markus Armbruster wrote:
Laszlo Ersek ler...@redhat.com writes:
This patch allows the user to usefully specify
-drive file=img_1,if=pflash,format=raw,readonly \
-drive file=img_2,if=pflash,format=raw
on the command line. The
Il 26/11/2013 13:03, Lei Li ha scritto:
+if (pending_size pending_size = max_size
+!runstate_needs_reset()) {
qemu_savevm_state_iterate(s-file);
I'm not sure why you need this.
The adjustment here is to avoid the iteration stage for page
On Tue, Nov 26, 2013 at 01:47:03PM +0100, Paolo Bonzini wrote:
Il 26/11/2013 13:40, Zhanghaoyu (A) ha scritto:
When guest set irq smp_affinity, VMEXIT occurs, then the vcpu thread will
IOCTL return to QEMU from hypervisor, then vcpu thread ask the hypervisor
to update the irq routing
Since it's a production machine, I had to try a remedy first: I changed one
network card (pub, of course) to e1000 and now it's up from 62 hours (maybe
a record!)
Here is the output for the other card (virtio): I guess parameters should
have been the same for the affected interface too (they are
Laszlo Ersek ler...@redhat.com writes:
On 11/25/13 16:32, Markus Armbruster wrote:
Laszlo Ersek ler...@redhat.com writes:
This patch allows the user to usefully specify
-drive file=img_1,if=pflash,format=raw,readonly \
-drive file=img_2,if=pflash,format=raw
on the command line. The
Il 26/11/2013 13:56, Gleb Natapov ha scritto:
I don't think a workqueue is even needed. You just need to use call_rcu
to free old after releasing kvm-irq_lock.
What do you think?
It should be rate limited somehow. Since it guest triggarable guest may cause
host to allocate a lot of
On Thu, 21 Nov 2013 11:12:43 +0100
Markus Armbruster arm...@redhat.com wrote:
Igor Mammedov imamm...@redhat.com writes:
Along with conversion extend -m option to support following parameters:
Please split this into two patches: first conversion to QemuOpts, then
extension.
mem -
On Tue, Nov 26, 2013 at 2:47 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 26/11/2013 13:40, Zhanghaoyu (A) ha scritto:
When guest set irq smp_affinity, VMEXIT occurs, then the vcpu thread
will IOCTL return to QEMU from hypervisor, then vcpu thread ask the
hypervisor to update the irq routing
Hello,
Is there any news on emulating sparc64 machine with openbios-sparc64 on a
x86_64 host machine? I followed some posts regarding this from 2010 however
didn't end-up with a conclusion.
Regards,
Mahmood
On 11/26/13 13:53, Markus Armbruster wrote:
Thus, we grab *all* if=pflash drives for this purpose.
Your stated use case wants just two.
Hmm. Are we sure we'll never want to map an if=pflash device somewhere
else?
No, I'm not sure.
Thanks
Laszlo
On 11/26/13 13:36, Markus Armbruster wrote:
Your stated purpose for multiple -pflash:
This accommodates the following use case: suppose that OVMF is split in
two parts, a writeable host file for non-volatile variable storage, and a
read-only part for bootstrap and decompressible
On 11/26/13 14:11, Markus Armbruster wrote:
Laszlo Ersek ler...@redhat.com writes:
On 11/25/13 16:32, Markus Armbruster wrote:
Laszlo Ersek ler...@redhat.com writes:
This patch allows the user to usefully specify
-drive file=img_1,if=pflash,format=raw,readonly \
-drive
On 26.11.2013 11:02, Paolo Bonzini wrote:
Il 26/11/2013 09:56, Peter Lieven ha scritto:
we currently do not check if a sector is allocated during convert.
This means if a sector is unallocated that we allocate a bounce
buffer of zeroes, find out its zero later and do not write it
in the best
Il 25/11/2013 20:45, Laszlo Ersek ha scritto:
From looking at hw/block/pflash_cfi01.c, it seems that the guest can
interrogate the flash device about its size (nb_blocs is stored in
cfi_table starting at 0x2D, and cfi_table can be queried by command 0x98
in pflash_read()). So, if the guest
We have the following comment about the default limit rate:
/* Set a default rate limit of 2^47 bytes per minute or roughly 2TB/s. If
* you have an entropy source capable of generating more entropy than this
* and you can pass it through via virtio-rng, then hats off to you. Until
* then,
Il 26/11/2013 14:18, Avi Kivity ha scritto:
I don't think a workqueue is even needed. You just need to use call_rcu
to free old after releasing kvm-irq_lock.
What do you think?
Can this cause an interrupt to be delivered to the wrong (old) vcpu?
No, this would be exactly the same code
On 11/26/13 14:41, Paolo Bonzini wrote:
Il 25/11/2013 20:45, Laszlo Ersek ha scritto:
From looking at hw/block/pflash_cfi01.c, it seems that the guest can
interrogate the flash device about its size (nb_blocs is stored in
cfi_table starting at 0x2D, and cfi_table can be queried by command 0x98
On 11/26/2013 08:54 PM, Paolo Bonzini wrote:
Il 26/11/2013 13:03, Lei Li ha scritto:
+if (pending_size pending_size = max_size
+!runstate_needs_reset()) {
qemu_savevm_state_iterate(s-file);
I'm not sure why you need this.
The adjustment here is
Il 26/11/2013 14:43, Amos Kong ha scritto:
/* Set a default rate limit of 2^47 bytes per minute or roughly 2TB/s. If
* you have an entropy source capable of generating more entropy than this
* and you can pass it through via virtio-rng, then hats off to you. Until
* then, this is
On Tue, Nov 26, 2013 at 12:00:51PM +0100, Gerd Hoffmann wrote:
Hi,
I think it's down to other qemu bugs (such as _CRS not covering
all of PCI memory), we shall just fix them.
i.e. the attached patch should fixup things.
cheers,
Gerd
Yes, I think it's a start. Q35 is a bit
On 11/26/2013 07:31 PM, Paolo Bonzini wrote:
Il 21/11/2013 10:11, Lei Li ha scritto:
+/*
+ * recvmsg is called here to catch the control message for
+ * the exchange of PIPE file descriptor until it is received.
+ */
+len =
On 11/26/2013 08:28 PM, Paolo Bonzini wrote:
Il 21/11/2013 10:11, Lei Li ha scritto:
{ RUN_STATE_DEBUG, RUN_STATE_SUSPENDED },
DEBUG - MEMORY_STALE is missing.
Good catch, I will add it, thanks. :)
Paolo
{ RUN_STATE_RUNNING, RUN_STATE_SUSPENDED },
{
The LEON3 processor has support for the CASA instruction which is
normally only available for SPARC V9 processors. Binutils 2.24
and GCC 4.9 will support this instruction for LEON3. GCC uses it to
generate C11 atomic operations.
---
target-sparc/cpu.c |3 +-
target-sparc/cpu.h
Il 26/11/2013 14:53, Laszlo Ersek ha scritto:
IIUC in the case of OVMF the guest just knows that there are multiple
devices backing the range. Is that right?
No, the guest (the flash driver) is unaware of that. It could know if it
wanted to, but it doesn't care. It cares about a base
On 11/26/13 10:10, Michael S. Tsirkin wrote:
seabios manages to enumerate PCI with information exported from qemu
so why can't OVMF?
SeaBIOS and qemu duplicate logic (code) between each other.
src/fw/pciinit.c grabs RamSize over fw_cfg, and i440fx_mem_addr_setup()
sets pcimem_start to one of
Il 26/11/2013 14:53, Lei Li ha scritto:
1) ram_save_setup stage, it will send all the bytes in this stages
to destination, and send_pipefd by ram_control_before_iterate
at the end of it.
ram_save_setup runs doesn't send anything from guest RAM. It sends the
lengths of the various
On 11/26/13 15:04, Michael S. Tsirkin wrote:
On Tue, Nov 26, 2013 at 12:00:51PM +0100, Gerd Hoffmann wrote:
Hi,
I think it's down to other qemu bugs (such as _CRS not covering
all of PCI memory), we shall just fix them.
i.e. the attached patch should fixup things.
cheers,
Gerd
On Tue, Nov 26, 2013 at 02:58:26PM +0100, Paolo Bonzini wrote:
Il 26/11/2013 14:43, Amos Kong ha scritto:
/* Set a default rate limit of 2^47 bytes per minute or roughly 2TB/s. If
* you have an entropy source capable of generating more entropy than this
* and you can pass it through via
[adding libvirt]
On 11/26/2013 06:58 AM, Paolo Bonzini wrote:
Il 26/11/2013 14:43, Amos Kong ha scritto:
/* Set a default rate limit of 2^47 bytes per minute or roughly 2TB/s. If
* you have an entropy source capable of generating more entropy than this
* and you can pass it through via
Juan Quintela quint...@redhat.com wrote:
Hi
Please, send any topic that you are interested in covering.
As there are no topics for agenda, call is cancelled.
Later, Juan.
Thanks, Juan.
Call details:
10:00 AM to 11:00 AM EDT
Every two weeks
If you need phone number details,
On Tue, Nov 26, 2013 at 3:47 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 26/11/2013 14:18, Avi Kivity ha scritto:
I don't think a workqueue is even needed. You just need to use call_rcu
to free old after releasing kvm-irq_lock.
What do you think?
Can this cause an interrupt to
On 11/25/2013 10:04 PM, Xu, Quan wrote:
Thanks Bryant, this problem has been solved by following
http://www.mail-archive.com/qemu-devel@nongnu.org/msg200808.html;.
But there is another problem when run configure with ./configure --target-list=x86_64-softmmu --enable-tpm. The
value
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
Il 26/11/2013 15:32, Eric Blake ha scritto:
Hmm. Libvirt is already converting a user's rate of bytes/period
into the qemu parameters, defaulting to 1 second as its default
period. Am I correct that as long as libvirt specified both rate
AND
Il 26/11/2013 15:36, Avi Kivity ha scritto:
No, this would be exactly the same code that is running now:
mutex_lock(kvm-irq_lock);
old = kvm-irq_routing;
kvm_irq_routing_update(kvm, new);
mutex_unlock(kvm-irq_lock);
1 - 100 of 223 matches
Mail list logo