From: David Gibson da...@gibson.dropbear.id.au
Recent (host) kernels support emulating the PAPR defined XICS interrupt
controller system within KVM. This patch allows qemu to initialize and
configure the in-kernel XICS, and keep its state in sync with qemu's XICS
state as necessary.
This should
Public bug reported:
Environment:
Host OS (ia32/ia32e/IA64):ia32e
Guest OS (ia32/ia32e/IA64):ia32e
Guest OS Type (Linux/Windows):Linux
kvm.git Commit:205befd9a5c701b56f569434045821f413f08f6d
qemu-kvm uq/master Commit:ca916d3729564d0eb3c2374a96903f7e8aced8a7
Host Kernel
Hi,
qemu-kvm no longer exists as it has been merged into qemu.
You might want to have a try with the lastest upstream/master qemu tree, or
pre-release versions of it. :)
--
You received this bug notification because you are a member of qemu-
devel-ml, which is subscribed to QEMU.
On Fri, Aug 16, 2013 at 10:41:36AM +0200, Stefan Hajnoczi wrote:
On Mon, Aug 5, 2013 at 10:11 AM, Asias He as...@redhat.com wrote:
From: MORITA Kazutaka morita.kazut...@lab.ntt.co.jp
While Asias is debugging an issue creating qcow2 images on top of
non-file protocols. It boils down to
On Sun, Aug 18, 2013 at 10:54 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 14/08/2013 02:34, liu ping fan ha scritto:
On Tue, Aug 13, 2013 at 10:53 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 13/08/2013 07:43, Liu Ping Fan ha scritto:
After disabling the QemuClock, we should make sure
On Fri, Aug 16, 2013 at 09:09:38PM +1000, Alexey Kardashevskiy wrote:
spapr-pci config space accessors use find_dev() to find a PCI device.
However find_dev() only searched on a primary bus and did not do
recursive search through secondary buses so config space access was not
possible for
On 08/19/2013 01:22 AM, Paolo Bonzini wrote:
Il 16/08/2013 11:49, Alexey Kardashevskiy ha scritto:
With KVM, we could fall back to the qemu implementation
+ * when KVM doesn't support them, but that would be much slower
+ * than just using the KVM implementations of the single TCE
+
On Wed, Aug 07, 2013 at 06:51:32PM +1000, Alexey Kardashevskiy wrote:
On PPC64 systems MSI Messages are translated to system IRQ in a PCI
host bridge. This is already supported for emulated MSI/MSIX but
not for irqfd where the current QEMU allocates IRQ numbers from
irqchip and maps
On 08/19/2013 05:35 PM, Michael S. Tsirkin wrote:
On Wed, Aug 07, 2013 at 06:51:32PM +1000, Alexey Kardashevskiy wrote:
On PPC64 systems MSI Messages are translated to system IRQ in a PCI
host bridge. This is already supported for emulated MSI/MSIX but
not for irqfd where the current QEMU
On Mon, Aug 19, 2013 at 05:44:04PM +1000, Alexey Kardashevskiy wrote:
On 08/19/2013 05:35 PM, Michael S. Tsirkin wrote:
On Wed, Aug 07, 2013 at 06:51:32PM +1000, Alexey Kardashevskiy wrote:
On PPC64 systems MSI Messages are translated to system IRQ in a PCI
host bridge. This is already
Am 19.08.2013 um 09:30 schrieb Alexey Kardashevskiy a...@ozlabs.ru:
On 08/19/2013 01:22 AM, Paolo Bonzini wrote:
Il 16/08/2013 11:49, Alexey Kardashevskiy ha scritto:
With KVM, we could fall back to the qemu implementation
+ * when KVM doesn't support them, but that would be much slower
Bring it up to support discussion in
[PATCH 0/2 v4] kvm irqfd: support msimessage to irq translation in PHB
On 06/30/2013 12:28 AM, Anthony Liguori wrote:
On Sat, Jun 29, 2013 at 8:45 AM, Alexey Kardashevskiy a...@ozlabs.ru wrote:
On PPC64 systems MSI Messages are translated to system IRQ in a
On 08/19/2013 05:54 PM, Michael S. Tsirkin wrote:
On Mon, Aug 19, 2013 at 05:44:04PM +1000, Alexey Kardashevskiy wrote:
On 08/19/2013 05:35 PM, Michael S. Tsirkin wrote:
On Wed, Aug 07, 2013 at 06:51:32PM +1000, Alexey Kardashevskiy wrote:
On PPC64 systems MSI Messages are translated to system
Any comments?
Regards,
On 08/02/2013 02:48 PM, Fabien Chouteau wrote:
Signed-off-by: Fabien Chouteau chout...@adacore.com
---
monitor.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/monitor.c b/monitor.c
index 5dc0aa9..78e93af 100644
--- a/monitor.c
+++
Any comments?
Regards,
On 08/02/2013 02:48 PM, Fabien Chouteau wrote:
Part of M731-018.
Signed-off-by: Fabien Chouteau chout...@adacore.com
---
disas.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/disas.c b/disas.c
index 71007fb..3ffb3ae
On 09.08.2013 18:29, Daniel P. Berrange wrote:
On Fri, Aug 09, 2013 at 10:58:55AM -0500, Anthony Liguori wrote:
Michal Privoznik mpriv...@redhat.com writes:
[CC'ing qemu-devel list]
On 09.08.2013 15:17, Daniel P. Berrange wrote:
On Fri, Aug 09, 2013 at 07:13:58AM -0600, Eric Blake wrote:
On
On 19 August 2013 09:26, Fabien Chouteau chout...@adacore.com wrote:
Any comments?
Regards,
On 08/02/2013 02:48 PM, Fabien Chouteau wrote:
Signed-off-by: Fabien Chouteau chout...@adacore.com
---
monitor.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/monitor.c
On 2 August 2013 13:48, Fabien Chouteau chout...@adacore.com wrote:
Part of M731-018.
What is this a reference to?
Signed-off-by: Fabien Chouteau chout...@adacore.com
---
disas.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/disas.c b/disas.c
On 08/19/2013 06:01 PM, Alexander Graf wrote:
Am 19.08.2013 um 09:30 schrieb Alexey Kardashevskiy a...@ozlabs.ru:
On 08/19/2013 01:22 AM, Paolo Bonzini wrote:
Il 16/08/2013 11:49, Alexey Kardashevskiy ha scritto:
With KVM, we could fall back to the qemu implementation
+ * when KVM
On Mon, Aug 19, 2013 at 06:10:01PM +1000, Alexey Kardashevskiy wrote:
On 08/19/2013 05:54 PM, Michael S. Tsirkin wrote:
On Mon, Aug 19, 2013 at 05:44:04PM +1000, Alexey Kardashevskiy wrote:
On 08/19/2013 05:35 PM, Michael S. Tsirkin wrote:
On Wed, Aug 07, 2013 at 06:51:32PM +1000, Alexey
On 08/16/2013 11:15 PM, Alexander Graf wrote:
On 07.08.2013, at 10:08, Alexey Kardashevskiy wrote:
Currently only single TCE entry per requiest is supported (H_PUT_TCE).
However PAPR+ specification allows multiple entry requests such as
H_PUT_TCE_INDIRECT and H_STUFFF_TCE. Having less
On Mon, Aug 19, 2013 at 10:29:10AM +0200, Michal Privoznik wrote:
On 09.08.2013 18:29, Daniel P. Berrange wrote:
On Fri, Aug 09, 2013 at 10:58:55AM -0500, Anthony Liguori wrote:
Michal Privoznik mpriv...@redhat.com writes:
[CC'ing qemu-devel list]
On 09.08.2013 15:17, Daniel P. Berrange
Laszlo Ersek ler...@redhat.com writes:
On 08/16/13 13:13, arm...@redhat.com wrote:
static void pc_init_isa(QEMUMachineInitArgs *args)
{
-ram_addr_t ram_size = args-ram_size;
-const char *cpu_model = args-cpu_model;
-const char *kernel_filename = args-kernel_filename;
-
Laszlo Ersek ler...@redhat.com writes:
comments below
On 08/16/13 13:13, arm...@redhat.com wrote:
From: Markus Armbruster arm...@redhat.com
Don't explode QEMUMachineInitArgs before passing it to
sun4m_hw_init(), sun4uv_init().
Signed-off-by: Markus Armbruster arm...@redhat.com
---
Laszlo Ersek ler...@redhat.com writes:
comments below
On 08/16/13 13:13, arm...@redhat.com wrote:
From: Markus Armbruster arm...@redhat.com
Pass on the generic arguments unadulterated, and the machine-specific
ones as separate argument.
Signed-off-by: Markus Armbruster
Il 18/08/2013 20:23, Liu, Jinsong ha scritto:
From 1273f8b2e5464ec987facf9942fd3ccc0b69087e Mon Sep 17 00:00:00 2001
From: Liu Jinsong jinsong@intel.com
Date: Mon, 19 Aug 2013 09:33:30 +0800
Subject: [PATCH] qemu-kvm bugfix for IA32_FEATURE_CONTROL
This patch is to fix the bug
By the time that qemu 1.7 will be released, enough time will has passed
since qemu 1.1, which is the first version to understand version 3
images, that changing the default shouldn't hurt many people any more
and the benefits of using the new format outweigh the pain.
qemu-iotests already runs
Hi,all:
in func virtqueue_get_avail_bytes, when found a indirect desc, we need
loop over it.
/* loop over the indirect descriptor table */
indirect = 1;
max = vring_desc_len(desc_pa, i) / sizeof(VRingDesc);
num_bufs = i = 0;
Il 19/08/2013 04:18, Fam Zheng ha scritto:
On Sun, 08/18 17:19, Paolo Bonzini wrote:
Il 13/08/2013 03:21, Fam Zheng ha scritto:
VMDK3 header has the field l1dir_size, but vmdk_open_vmdk3 hardcoded the
value. This patch honors the header field.
And the L2 table size is 4096 according to VMDK
Il 19/08/2013 09:14, liu ping fan ha scritto:
But does timerlists need a lock, or does the BQL suffice? If it
doesn't, there is no need for events_list_lock either. Is
qemu_clock_enable called outside the BQL?
Currently, no such guarantee based on BQL. But if we enforce that an
backend
On 08/19/13 11:24, Markus Armbruster wrote:
Laszlo Ersek ler...@redhat.com writes:
Please always use
-O/path/to/order_file
when invoking git-format-patch.
The contents of order_file should be minimally
configure
Makefile*
*.json
*.h
*.c
It's much easier to review a
Andreas Färber afaer...@suse.de writes:
Am 16.08.2013 15:18, schrieb arm...@redhat.com:
From: Markus Armbruster arm...@redhat.com
I'd like to access QEMUMachine from a QEMUMachine init() method, which
is currently not possible. Instead of passing it as an argument, I
simply set
Il 19/08/2013 10:44, Alexey Kardashevskiy ha scritto:
It means that if you use the same QEMU version with the same command
line on a different kernel version, your guest looks different because
we generate the dtb differently.
Oh. Sorry for my ignorance again, I am not playing dump or
Am 13.08.2013 um 03:21 hat Fam Zheng geschrieben:
VMDK3 header has the field l1dir_size, but vmdk_open_vmdk3 hardcoded the
value. This patch honors the header field.
And the L2 table size is 4096 according to VMDK spec[1], instead of
1 9 (512).
[1]:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
Migration code assumes that each RAM block is a multiple of target page
size.
Isn't that a valid assumption, considering the TARGET_PAGE_ALIGN() macro
call in qemu_ram_alloc_from_ptr() [exec.c]?
We can fix this in a variety of ways, the simplest
Am 13.08.2013 um 03:21 hat Fam Zheng geschrieben:
This header check is common to VMDK3 and VMDK4, so move it into
vmdk_add_extent().
Signed-off-by: Fam Zheng f...@redhat.com
Aha, so this is the fix for patch 1. If you reorder the patches so that
this one comes first, you don't have broken
On 19.08.2013 11:06, Daniel P. Berrange wrote:
On Mon, Aug 19, 2013 at 10:29:10AM +0200, Michal Privoznik wrote:
On 09.08.2013 18:29, Daniel P. Berrange wrote:
On Fri, Aug 09, 2013 at 10:58:55AM -0500, Anthony Liguori wrote:
Michal Privoznik mpriv...@redhat.com writes:
[CC'ing qemu-devel
Il 10/08/2013 13:21, Alex Bligh ha scritto:
Currently we use a separate timer list group (main_loop_tlg)
for the main loop. This patch replaces this with a dummy AioContext
used just for timers in the main loop.
Signed-off-by: Alex Bligh a...@alex.org.uk
I guess for now this complicates
On 19 August 2013 10:59, Laszlo Ersek ler...@redhat.com wrote:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
Migration code assumes that each RAM block is a multiple of target page
size.
Isn't that a valid assumption, considering the TARGET_PAGE_ALIGN() macro
call in qemu_ram_alloc_from_ptr()
usb-bt-dongle device can't be disabled as there's dependency in vl.c file. This
patch add preprocesor condition to be able to disable it.
Signed-off-by: Miroslav Rezanina mreza...@redhat.com
---
hw/usb/Makefile.objs | 1 -
vl.c | 18 ++
2 files changed, 14
This series add support for VMFS and VMFSSPARSE extents, these types are found
in description file from ESX hosts.
- VMFS is in monolithiFlat format (raw), but hosted in ESX.
- VMFSSPARSE is the format we call vmdk3 with magic bytes COWD. This patch
fix the opening of vmdk3 and rename it to
VMDK3 header has the field l1dir_size, but vmdk_open_vmdk3 hardcoded the
value. This patch honors the header field.
And the L2 table size is 4096 according to VMDK spec[1], instead of
1 9 (512).
[1]:
http://www.vmware.com/support/developer/vddk/vmdk_50_technote.pdf?src=vmdk
Signed-off-by: Fam
VMware ESX hosts use a variant of the VMDK3 format, identified by the
vmfsSparse create type ad the VMFSSPARSE extent type.
It has 16 KB grain tables (L2) and a variable-size grain directory (L1).
In addition, the grain size is always 512, but that is not a problem
because it is included in the
This header check is common to VMDK3 and VMDK4, so move it into
vmdk_add_extent().
Signed-off-by: Fam Zheng f...@redhat.com
---
block/vmdk.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/block/vmdk.c b/block/vmdk.c
index 346bb5c..f8c0a4e 100644
---
From: Paolo Bonzini pbonz...@redhat.com
VMware ESX hosts also use different create and extent types for flat
files, respectively vmfs and VMFS. This is not documented, but it
can be found at http://kb.vmware.com/kb/10002511 (Recreating a missing
virtual machine disk (VMDK) descriptor file).
On Mon, Aug 19, 2013 at 11:59:25AM +0200, Laszlo Ersek wrote:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
Migration code assumes that each RAM block is a multiple of target page
size.
Isn't that a valid assumption, considering the TARGET_PAGE_ALIGN() macro
call in
On 08/19/13 12:21, Peter Maydell wrote:
On 19 August 2013 10:59, Laszlo Ersek ler...@redhat.com wrote:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
Migration code assumes that each RAM block is a multiple of target page
size.
Isn't that a valid assumption, considering the TARGET_PAGE_ALIGN()
On 08/13/13 00:43, Michael S. Tsirkin wrote:
ROM files that are put in FW CFG are copied to guest ram, by BIOS, but
they are not backed by RAM so they don't get migrated.
Can you please elaborate on this? Do you mean the 384 KB range between
640KB and 1MB that is covered by RAMBlock, but no
On Mon, Aug 19, 2013 at 01:06:07PM +0200, Laszlo Ersek wrote:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
ROM files that are put in FW CFG are copied to guest ram, by BIOS, but
they are not backed by RAM so they don't get migrated.
Can you please elaborate on this? Do you mean the 384 KB
I note some recent work on vmdk file support. Has anyone looked at
supporting vmdk streaming format as an /output/ file format (I
think we currently support it as an input format). This is what
you need to use to upload a VM to vCenter. Currently the route
is convert to raw then use:
Hi Anthony,
+if (resource == 4) {
This ought to be a #define. There's no else here, is that expected?
Should you return failure for a different resource?
Good point, I made it a define. We were returning H_P2 for a different
resource, but it was a bit of a twisted maze of return
On 08/19/13 13:06, Laszlo Ersek wrote:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
@@ -646,6 +669,7 @@ int rom_add_file(const char *file, const char *fw_dir,
if (rom-fw_file fw_cfg) {
const char *basename;
char fw_file_name[56];
+void *data;
On Mon, Aug 19, 2013 at 01:09:36PM +0200, Laszlo Ersek wrote:
On 08/19/13 12:21, Peter Maydell wrote:
On 19 August 2013 10:59, Laszlo Ersek ler...@redhat.com wrote:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
Migration code assumes that each RAM block is a multiple of target page
size.
On Mon, Aug 19, 2013 at 01:15:36PM +0200, Laszlo Ersek wrote:
On 08/19/13 13:06, Laszlo Ersek wrote:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
@@ -646,6 +669,7 @@ int rom_add_file(const char *file, const char *fw_dir,
if (rom-fw_file fw_cfg) {
const char *basename;
On Mon, 08/19 12:09, Alex Bligh wrote:
I note some recent work on vmdk file support. Has anyone looked at
supporting vmdk streaming format as an /output/ file format (I
think we currently support it as an input format). This is what
you need to use to upload a VM to vCenter. Currently the
Il 16/08/2013 13:45, Stefan Hajnoczi ha scritto:
+#define BUCKETS_COUNT 6
+
+typedef enum {
+THROTTLE_BPS_TOTAL = 0,
+THROTTLE_BPS_READ = 1,
+THROTTLE_BPS_WRITE = 2,
+THROTTLE_OPS_TOTAL = 3,
+THROTTLE_OPS_READ = 4,
+THROTTLE_OPS_WRITE = 5,
+}
On 08/19/13 13:18, Michael S. Tsirkin wrote:
On Mon, Aug 19, 2013 at 01:09:36PM +0200, Laszlo Ersek wrote:
On 08/19/13 12:21, Peter Maydell wrote:
On 19 August 2013 10:59, Laszlo Ersek ler...@redhat.com wrote:
On 08/13/13 00:43, Michael S. Tsirkin wrote:
Migration code assumes that each RAM
On 08/13/13 00:43, Michael S. Tsirkin wrote:
Changes from v2: address comments on v2 by Peter Maydell
- switch from global constant to function
- use memory_region_init_ram instead of _ram_ptr
- disable for 1.6
Changes from v1: address comments by Peter Maydell
- drop
On 19 Aug 2013, at 12:23, Fam Zheng wrote:
Yes, that would be something possible to do. Is it the only format to
upload a VM to vCenter now, shouldn't normal VMDK be supported as well?
Ian puts it better than I can:
This format is only useful if you are attempting to import virtual machines
On Sun, Aug 18, 2013 at 04:50:02PM +0300, Michael S. Tsirkin wrote:
Make 1.4 compat code call the 1.6 one, reducing
code duplication. Add comment explaining why we can't
make 1.4 call 1.5 as usual.
Signed-off-by: Michael S. Tsirkin m...@redhat.com
---
hw/i386/pc_piix.c | 4 ++--
Il 14/08/2013 14:32, Jan Kiszka ha scritto:
I still need to check more corner cases as timer dequeuing can now race
with the handler execution, ie. a dequeued timer can still see one more
handler run after timer_del returned. That's a property one can easily
take into account when writing
On Mon, Aug 19, 2013 at 02:36:14PM +0800, Asias He wrote:
On Fri, Aug 16, 2013 at 10:41:36AM +0200, Stefan Hajnoczi wrote:
On Mon, Aug 5, 2013 at 10:11 AM, Asias He as...@redhat.com wrote:
A simple but ugly way to fix this is for block.c to also have a
-zero_beyond_eof flag which enables
Il 12/08/2013 14:49, Stefan Hajnoczi ha scritto:
Introduce QEMUTimerList-active_timers_lock to protect the linked list
of active timers. This allows qemu_timer_mod_ns() to be called from any
thread.
Note that vm_clock is not thread-safe and its use of
qemu_clock_has_timers() works fine
Il 15/08/2013 14:00, Stefan Hajnoczi ha scritto:
The common case is that we only check the first timer in
-active_timers. Usually the first timer has not expired yet and we
return; the lock was taken once only.
I'm not sure it's worth complicating the case where we iterate multiple
times.
Hi,
This patch series implement support for dumping guest memory using qemu gdb
server.
Without this patch we would get.
(gdb) x/10 do_fork
0xc0098660 do_fork: Cannot access memory at address
0xc0098660
(gdb)
With this patch series we can now do
(gdb) target remote
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
Without this, a value of rb=0 and rs=0, results in replacing the 0th
index. This can be observed when using gdb remote debugging support.
(gdb) x/10i do_fork
0xc0085330 do_fork:Cannot access memory at address
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
With kvm enabled, we store the hash page table information in the hypervisor.
Use ioctl to read the htab contents. Without this we get the below error when
trying to read the guest address
(gdb) x/10 do_fork
0xc0098660 do_fork:
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
Use SLB_ESID_V instead of (1 27) in the code
Signed-off-by: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
---
target-ppc/mmu_helper.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/target-ppc/mmu_helper.c
Am 19.08.2013 12:48, schrieb Miroslav Rezanina:
usb-bt-dongle device can't be disabled as there's dependency in vl.c file.
This patch add preprocesor condition to be able to disable it.
Please limit to 76 chars per line (check `git log` output).
Signed-off-by: Miroslav Rezanina
Am 19.08.2013 14:29, schrieb Aneesh Kumar K.V:
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
Without this, a value of rb=0 and rs=0, results in replacing the 0th
... rs=0 results in ...
index. This can be observed when using gdb remote debugging support.
(gdb) x/10i do_fork
Am 19.08.2013 14:29, schrieb Aneesh Kumar K.V:
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
Use SLB_ESID_V instead of (1 27) in the code
Signed-off-by: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
Reviewed-by: Andreas Färber afaer...@suse.de
Andreas
--
SUSE LINUX Products
Am 19.08.2013 14:42, schrieb Andreas Färber:
Am 19.08.2013 14:29, schrieb Aneesh Kumar K.V:
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
Without this, a value of rb=0 and rs=0, results in replacing the 0th
... rs=0 results in ...
index. This can be observed when using gdb
Il 19/08/2013 12:48, Miroslav Rezanina ha scritto:
usb-bt-dongle device can't be disabled as there's dependency in vl.c file.
This patch add preprocesor condition to be able to disable it.
Signed-off-by: Miroslav Rezanina mreza...@redhat.com
---
hw/usb/Makefile.objs | 1 -
vl.c
Am 19.08.2013 14:29, schrieb Aneesh Kumar K.V:
From: Aneesh Kumar K.V aneesh.ku...@linux.vnet.ibm.com
With kvm enabled, we store the hash page table information in the hypervisor.
Use ioctl to read the htab contents. Without this we get the below error when
trying to read the guest address
On Mon, Aug 19, 2013 at 11:25:33AM +0200, Kevin Wolf wrote:
By the time that qemu 1.7 will be released, enough time will has passed
s/has/have/
since qemu 1.1, which is the first version to understand version 3
images, that changing the default shouldn't hurt many people any more
and the
On Fri, Aug 16, 2013 at 04:37:28PM +0100, Alex Bligh wrote:
Stefan,
On 16 Aug 2013, at 16:24, Stefan Hajnoczi wrote:
Please run qemu-iotests on this series and fix any failures. It is
failing with I/O thread has spun for 1000 iterations.
It does that without my patch. Below is the
Hi Aneesh,
Am 19.08.2013 14:29, schrieb Aneesh Kumar K.V:
This patch series implement support for dumping guest memory using qemu gdb
server.
I had a quick look through but will leave in-depth review to Alex or
Anthony.
Do you plan to implement dumping guest memory via QMP, too?
Cheers,
Il 15/08/2013 08:02, Alexey Kardashevskiy ha scritto:
On 08/13/2013 08:07 AM, Alex Williamson wrote:
+static void vfio_listener_region_add(MemoryListener *listener,
+ MemoryRegionSection *section)
+{
+VFIOContainer *container = container_of(listener,
Il 13/08/2013 00:07, Alex Williamson ha scritto:
+if (pci_iommu_as(pdev) != address_space_memory) {
+error_report(vfio: DMA address space must be system memory);
+return -ENXIO;
-EFAULT? It's a bad address of sorts.
Accessing it would SIGSEGV, so it is not really
Il 13/08/2013 16:54, Jan Kiszka ha scritto:
Using an AioContext lock for timers is somewhat complicated for lock
ordering, because context A could try to modify a timer from context B,
at the same time when context B is modifying a timer from context A.
This would cause a deadlock.
That's
On Fri, 16 Aug 2013 14:21:50 +0100
Peter Maydell peter.mayd...@linaro.org wrote:
On 16 August 2013 08:59, Erik Rull erik.r...@rdsoftware.de wrote:
Hi all,
when using the released qemu-1.6.0.tar.bz2, I get the following error
message:
File /home/erik/qemu-1.6.0/scripts/qapi.py, line
On 08/19/13 14:31, Andreas Färber wrote:
Am 19.08.2013 12:48, schrieb Miroslav Rezanina:
usb-bt-dongle device can't be disabled as there's dependency in vl.c file.
This patch add preprocesor condition to be able to disable it.
Please limit to 76 chars per line (check `git log` output).
On Sat, Aug 17, 2013 at 10:52:34AM +0530, Bharata B Rao wrote:
From: Stefan Hajnoczi stefa...@redhat.com
Since .io_flush() is no longer called we do not need
qemu_gluster_aio_flush_cb() anymore. It turns out that qemu_aio_count
is unused now and can be dropped.
Signed-off-by: Stefan
On Fri, Aug 16, 2013 at 05:47:06PM +0200, Stefan Hajnoczi wrote:
fatal: Needed a single revision
The following changes since commit f202039811d8746b0586d2fd5f61de6c8cf68056:
Open up 1.7 development branch (2013-08-15 15:41:13 -0500)
are available in the git repository at:
On 2013-08-19 15:21, Paolo Bonzini wrote:
Il 13/08/2013 16:54, Jan Kiszka ha scritto:
Using an AioContext lock for timers is somewhat complicated for lock
ordering, because context A could try to modify a timer from context B,
at the same time when context B is modifying a timer from context
Am 19.08.2013 15:30, schrieb Laszlo Ersek:
On 08/19/13 14:31, Andreas Färber wrote:
Am 19.08.2013 12:48, schrieb Miroslav Rezanina:
usb-bt-dongle device can't be disabled as there's dependency in vl.c file.
This patch add preprocesor condition to be able to disable it.
Please limit to 76
Am 16.08.2013 15:21, schrieb Peter Maydell:
On 16 August 2013 08:59, Erik Rull erik.r...@rdsoftware.de wrote:
Hi all,
when using the released qemu-1.6.0.tar.bz2, I get the following error
message:
File /home/erik/qemu-1.6.0/scripts/qapi.py, line 164
except QAPISchemaError as e:
My IBM email address will be unaccessible after August 23rd, 2013.
Signed-off-by: Anthony Liguori anth...@codemonkey.ws
---
.mailmap| 2 +-
MAINTAINERS | 14 +++---
2 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/.mailmap b/.mailmap
index 9797802..7b91a95 100644
---
Am 19.08.2013 07:55, schrieb Alexey Kardashevskiy:
The upcoming support of in-kernel XICS will redefine migration callbacks
for both ICS and ICP so classes and callback pointers are added.
This adds a cpu_setup callback to the XICS device class (as XICS-KVM
will do it different) and
On 19 Aug 2013, at 14:21, Paolo Bonzini wrote:
I like Stefan's patches to make the timer list thread-safe, especially
if we can optimize it (with RCU?) to make the read side lockless.
We will have to be careful on the write side. I'm trying to
work out just how slow this would with Paolo's
Il 19/08/2013 15:41, Andreas Färber ha scritto:
By my reading of `git grep CONFIG_USB_BLUETOOTH` it isn't disappearing,
check default-configs/usb.mak. All targets that include usb.mak will
have CONFIG_USB_BLUETOOTH.
It's only used in the build system and with this patch in vl.c, so
assuming
Hi,
I've decided to take an opportunity outside of IBM which means that my
IBM email address will no longer work after this friday (August 23rd).
Please start CC'ing my anth...@codemonkey.ws address on all patches and
pull requests.
I'm excited about my new opportunity but I am going to wait
On 08/19/2013 10:33 AM, Peter Maydell wrote:
On 2 August 2013 13:48, Fabien Chouteau chout...@adacore.com wrote:
Part of M731-018.
What is this a reference to?
I'm sorry this is not supposed to be here. Do you have comments before I
resend the patch?
Regards,
--
Fabien Chouteau
Il 19/08/2013 15:40, Jan Kiszka ha scritto:
On 2013-08-19 15:21, Paolo Bonzini wrote:
Il 13/08/2013 16:54, Jan Kiszka ha scritto:
Using an AioContext lock for timers is somewhat complicated for lock
ordering, because context A could try to modify a timer from context B,
at the same time when
Am 02.08.2013 14:48, schrieb Fabien Chouteau:
Part of M731-018.
Signed-off-by: Fabien Chouteau chout...@adacore.com
---
disas.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/disas.c b/disas.c
index 71007fb..3ffb3ae 100644
--- a/disas.c
+++
Since .io_flush() is no longer called we do not need
qemu_gluster_aio_flush_cb() anymore. It turns out that qemu_aio_count
is unused now and can be dropped.
Thanks to Bharata B Rao bhar...@linux.vnet.ibm.com for catching a
build failure with CONFIG_GLUSTERFS_DISCARD, which has been fixed.
.io_flush() is no longer called so drop curl_aio_flush(). The acb[]
array that the function checks is still used in other parts of
block/curl.c. Therefore we cannot remove acb[], it is needed.
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
---
block/curl.c | 22 +++---
1
.io_flush() is no longer called so drop qemu_rbd_aio_flush_cb().
qemu_aio_count is unused now so drop it too.
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
---
block/rbd.c | 14 +-
1 file changed, 1 insertion(+), 13 deletions(-)
diff --git a/block/rbd.c b/block/rbd.c
index
Check exit conditions before entering blocking aio_poll(). This is
mainly for consistency since it's unlikely that we are stopping in the
first event loop iteration.
Signed-off-by: Stefan Hajnoczi stefa...@redhat.com
---
hw/block/dataplane/virtio-blk.c | 4 ++--
1 file changed, 2 insertions(+),
aio_poll(ctx, true) will soon block when fd handlers have been set.
Previously aio_poll() would return early if all .io_flush() returned
false. This means we need to check the equivalent of the .io_flush()
condition *before* calling aio_poll(ctx, true) to avoid deadlock.
Reviewed-by: Paolo
1 - 100 of 190 matches
Mail list logo