Hello all,
I noticed that QEMU(both 1.2.1 and git head) acts very slow when
accessing HD. I wonder if it is a fault in my build environment?
Spec:
Windows XP SP3
gcc-4.6.2
binutils-2.22.51.20120114
SDL-1.2.14
glib-2.28.8 (configure line: CFLAGS="-Os -I/mingw/include" LDFLAGS=-s
./configure --pref
Hello: I am Alan Jacobs in Green Bay Wisconsin USA ( north of Chicago). I have
a difficult emulation question regarding interrupt driven bidirectional
parallel port communication on a Celeron 220 Intel processor with 2GB RAM ( but
no hardware KVM extension I can find ), but with bios option to b
The lsi driver simply hangs at startup. Try the following to reproduce:
# ./x86_64-softmmu/qemu-system-x86_64 -monitor stdio -device lsi
The monitor is blocked, and VM shows black screen.
But it works perfectly when I use 'megasas' instead of 'lsi'.
And ideas?
Add OpenRisc fpu excepiton.
When an exception raised, CPU can save right PC.
Signed-off-by: Feng Gao
---
target-openrisc/exception.c| 31 +--
target-openrisc/exception.h|8 --
target-openrisc/exception_helper.c |2 +-
target-openrisc/fpu_helper.c
On 10/12/2012 12:49 AM, Mike Lovell wrote:
This makes a few changes to allow ifname to be specified when using
qemu-bridge-helper with both the bridge and tap network interfaces. It adds
the --ifname option to qemu-bridge-helper, removes the restriction that ifname
cannot be specified with helper
These functions are needed by libqblock, so add them into stubs.
Signed-off-by: Wenchao Xia
---
stubs/Makefile.objs|1 +
stubs/set-fd-handler.c |8
stubs/signalfd.c |7 +++
3 files changed, 16 insertions(+), 0 deletions(-)
create mode 100644 stubs/signalfd.c
Public API design header files: libqblock.h, libqblock-error.h.
Public type define header files: libqblock-types.h.
Private internal used header files: libqblock-internal, libqblock-aio.h.
For ABI some reserved bytes are used in structure defines.
v11:
Moved API design into this patch.
In this example, first it will create some qcow2 images, then try get
information including backing file relationship, then it will do sync IO on
the image.
v11:
Use only gtester option -k, to avoid silent fail for segment fault.
Little change according to API.
v12:
Little change accordin
Libqblock was placed in new directory ./libqblock, libtool will build
dynamic library there, source files of block layer remains in ./block.
So block related source code will generate 3 sets of binary, first is old
ones used in qemu, second and third are non PIC and PIC ones in ./libqblock.
GCC
Improve error reporting in execute_fsfreeze_hook() using
ga_wait_child().
This patch depends on patchset to improve error reporting by Luiz
Capitulino, especially on http://patchwork.ozlabs.org/patch/202208/ .
Signed-off-by: Tomoki Sekiyama
---
qga/commands-posix.c | 23 +-
Adds sample hook scripts for --fsfreeze-hook option of qemu-ga.
- fsfreeze-hook : execute scripts in fsfreeze-hook.d/
- fsfreeze-hook.d.sample/mysql-flush.sh : quiesce MySQL before snapshot
Signed-off-by: Tomoki Sekiyama
---
docs/qemu-guest-agent/fsfreeze-hook| 33 +
To use the online disk snapshot for online-backup, application-level
consistency of the snapshot image is required. However, currently the
guest agent can provide only filesystem-level consistency, and the
snapshot may contain dirty data, for example, incomplete transactions.
This patch provides th
Hi,
This is version 5 of the qemu-ga fsfreeze hook patchset.
*Changes from v4: ( http://patchwork.ozlabs.org/patch/200924/ )
1/3: Improve error messages when fsfreeze hook is not executable.
Make fsfreeze-hook disabled by default, and enabled only if
--fsfreeze-hook option is specifi
This patch contains implemention for APIs.
Important APIs:
1 QBlockContext. This structure was used to retrieve errors, every thread
must create one first.
2 QBlockImage. It stands for an block image object.
3 QBlockStaticInfo. It contains static information such as location, backing
file,
Currently root Makefile try clean tests/tcg, hard to extend.
This patch added command make check-clean, which clean all
generated files used in tests. With this command root Makefile
do not care tests clean method any more, it simply calls the
command to do it, so any more clean script could be a
This function is needed in other module, so export it. There is
already some patch on mail-list try export it, If that patch was applied,
pls ignore this one.
Signed-off-by: Wenchao Xia
---
block.c |2 +-
block.h |2 ++
2 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/blo
These patches introduce libqblock API, make subdir-libqblock and make
check-libqblock could build this library.
Functionalities:
1 create a new image.
2 sync access of an image.
3 basic image information retrieving such as backing file.
4 detect if a sector is allocated in an image.
Supported
Marcelo,
My latest QEMU patch ([PATCH V5] target-i386: Enabling IA32_TSC_ADJUST for QEMU
KVM guest VMs) seems to be OK with regards to reset. The tsc_adjust variable is
being zeroed by the memset() in x86_cpu_reset(). Later code seems to write
these values through kvm_put_msr() by way of cpu_s
> I need a Signed-off-by so why don't you just go ahead and send a v2 and
> I'll test that.
OK I sent a v2 which is this fixup + the original patch smashed
together.
Commit 0d8d7690850eb0cf2b2b60933cf47669a6b6f18f introduced
a regression in virtio-net performance because it looks
into the ring aggressively while we really only care
about a single packet worth of buffers.
Reported as bugzilla 1066055 in launchpad.
To fix, add parameters limiting lookahead, and
Hi Paolo,
Am 29.11.2012 16:23, schrieb Paolo Bonzini:
+qemu_vfree(acb->bounce);
This vfree is not needed, since the BH will run and do the free.
new patch v5 sent.
Greets,
Stefan
This one fixes a race which qemu had also in iscsi block driver
between cancellation and io completition.
qemu_rbd_aio_cancel was not synchronously waiting for the end of
the command.
To archieve this it introduces a new status flag which uses
-EINPROGRESS.
Changes since PATCHv4:
- removed unnec
Paolo Bonzini writes:
> Il 29/11/2012 20:24, Alexander Graf ha scritto:
>>> > It is in hw/multiboot.c:
>>> >
>>> >stl_p(bootinfo + MBI_MEM_LOWER, 640);
>> You want to remove that one then.
>
> I wasn't sure of what happens if the multiboot option ROM is old. Do we
> support that to any ex
On Thu, Nov 29, 2012 at 07:21:28PM +, Auld, Will wrote:
> Marcelo,
>
> The behavior on reset is to return the TSC_AJUST msr value to 0x0. I am
> currently initializing this emulated msr in kvm_arch_vcpu_init().
Will,
Reset is handled by QEMU. kvm_arch_vcpu_init is only called during vcpu
a
Il 29/11/2012 20:24, Alexander Graf ha scritto:
>> > It is in hw/multiboot.c:
>> >
>> >stl_p(bootinfo + MBI_MEM_LOWER, 640);
> You want to remove that one then.
I wasn't sure of what happens if the multiboot option ROM is old. Do we
support that to any extent?
> > Regarding the testcase,
On 29.11.2012, at 20:21, Paolo Bonzini wrote:
>
>
> - Messaggio originale -
>> Da: "Alexander Graf"
>> A: "Paolo Bonzini"
>> Cc: qemu-devel@nongnu.org
>> Inviato: Giovedì, 29 novembre 2012 19:51:07
>> Oggetto: Re: [PATCH 1.3?] multiboot: fix e801 memory map
>>
>>
>> On 29.11.2012, a
Marcelo,
The behavior on reset is to return the TSC_AJUST msr value to 0x0. I am
currently initializing this emulated msr in kvm_arch_vcpu_init().
>- Behaviour on reset: what is the behaviour on RESET?
I am testing the rebase now. I would like to get any needed changes for this
initialization
- Messaggio originale -
> Da: "Alexander Graf"
> A: "Paolo Bonzini"
> Cc: qemu-devel@nongnu.org
> Inviato: Giovedì, 29 novembre 2012 19:51:07
> Oggetto: Re: [PATCH 1.3?] multiboot: fix e801 memory map
>
>
> On 29.11.2012, at 18:11, Paolo Bonzini wrote:
>
> > The e801 memory sizes in
On Thu, Nov 29, 2012 at 07:30:29PM +0100, Igor Mammedov wrote:
> On Thu, 29 Nov 2012 13:47:37 -0200
> Eduardo Habkost wrote:
>
> > On Thu, Nov 29, 2012 at 03:56:09PM +0100, Igor Mammedov wrote:
> > > On Fri, 9 Nov 2012 16:48:07 +0100
> > > Eduardo Habkost wrote:
> > >
> > > >
> > > > On 22/10/
Hello Stefan
Yes, we tested this device with various guests (both Linux and Windows) and
it also passed WHQL tests for Windows Server 2008 R2 OS.
The issue is Anthony asked us to write a basic test for the device (a
packet transmission test) and we are trying to get some reference code or
document
On Thu, Nov 29, 2012 at 01:57:11PM +, Ian Campbell wrote:
> On Thu, 2012-11-29 at 13:21 +, Stefano Stabellini wrote:
> > > Also I think 4.2.1 need these patches to enable the basic Xen on Xen
> > > nested virtualization usage scenario.
> >
> > I agree.
>
> Nested virt was a tech preview i
"Michael S. Tsirkin" writes:
> On Thu, Nov 29, 2012 at 06:34:46PM +0530, Amit Shah wrote:
>> On (Wed) 28 Nov 2012 [23:53:08], Michael S. Tsirkin wrote:
>> > On Tue, Nov 27, 2012 at 06:25:04PM +0200, Michael S. Tsirkin wrote:
>> > > On Thu, Nov 01, 2012 at 06:07:21PM +0200, Michael S. Tsirkin wrot
Am 23.11.2012 08:02, schrieb Stefan Hajnoczi:
> On Thu, Nov 22, 2012 at 03:29:52PM +0100, Peter Lieven wrote:
>> is anyone aware of a problem with the linux network bridge that in very rare
>> circumstances stops
>> a bridge from sending pakets to a tap device?
>>
>> My problem occurs in conjuncti
On 29.11.2012, at 18:11, Paolo Bonzini wrote:
> The e801 memory sizes in the multiboot structures hard-code the available
> low memory to 640. However, the value should not include the size of the
> EBDA. Fill the value in the option ROM, getting the size of low memory
> from the BIOS.
The des
On 29.11.2012, at 18:11, Paolo Bonzini wrote:
> The e801 memory sizes in the multiboot structures hard-code the available
> low memory to 640. However, the value should not include the size of the
> EBDA. Fill the value in the option ROM, getting the size of low memory
> from the BIOS.
>
> Cc:
On Wed, Nov 28, 2012 at 02:20:23PM +0800, liguang wrote:
> 1.remove unused variable env
It's not unused. You are removing the line that sets env->cpu_model_str.
> 2.remove redundant error handling
>
> Signed-off-by: liguang
> ---
> target-i386/helper.c | 17 -
> 1 files chang
On Thu, 29 Nov 2012 13:47:37 -0200
Eduardo Habkost wrote:
> On Thu, Nov 29, 2012 at 03:56:09PM +0100, Igor Mammedov wrote:
> > On Fri, 9 Nov 2012 16:48:07 +0100
> > Eduardo Habkost wrote:
> >
> > >
> > > On 22/10/2012, at 17:03, Igor Mammedov wrote:
> > >
> > > > Signed-off-by: Igor Mammedov
On Fri, Nov 16, 2012 at 2:55 PM, Dmitry Fleytman wrote:
> We didn't succeed to find any guide or sample for the
> king of tests required (packets transmission).
> If someone can provide a pointer to the relevant
> information we'll be very grateful.
QEMU does not have a NI
On Thu, Nov 29, 2012 at 1:34 AM, liu ping fan wrote:
> On Thu, Nov 29, 2012 at 12:42 PM, Cam Macdonell wrote:
>> On Tue, Nov 27, 2012 at 7:53 PM, liu ping fan wrote:
>>> On Wed, Nov 28, 2012 at 5:48 AM, Cam Macdonell wrote:
On Sat, Nov 24, 2012 at 8:51 PM, Liu Ping Fan wrote:
> From:
Signed-off-by: Luiz Capitulino
---
o v2: use error_setg() in build_fs_mount_list()
qga/commands-posix.c | 30 --
1 file changed, 16 insertions(+), 14 deletions(-)
diff --git a/qga/commands-posix.c b/qga/commands-posix.c
index b3a3f26..ff14174 100644
--- a/qga/comman
Fix the code in the secondary CPU boot stubs so that it correctly
initialises the GIC rather than relying on bugs or implementation
dependent aspects of the QEMU GIC implementation:
* set the GIC_PMR.Priority field to all-ones, so that all
interrupts are passed through. The default of all-zeroe
The GIC spec states that only interrupts with higher priority
than the value in the GICC_PMR priority mask register are
passed through to the processor. We were incorrectly allowing
through interrupts with a priority equal to the specified
value: correct the comparison operation to match the spec.
The secondary CPU boot code we use on ARM had a couple of
places where it was accidentally relying on bugs or implementation
dependent behaviour of QEMU's on GIC implementation:
* we weren't initialising the GICC_PMR priority mask, which
in a correct v1 or v2 GIC is set to mask out all interrup
The e801 memory sizes in the multiboot structures hard-code the available
low memory to 640. However, the value should not include the size of the
EBDA. Fill the value in the option ROM, getting the size of low memory
from the BIOS.
Cc: Alexander Graf
Signed-off-by: Paolo Bonzini
---
pc-bios/
The e801 memory sizes in the multiboot structures hard-code the available
low memory to 640. However, the value should not include the size of the
EBDA. Fill the value in the option ROM, getting the size of low memory
from the BIOS.
Cc: Alexander Graf
Signed-off-by: Paolo Bonzini
---
A
The GIC architecture specification for v1 and v2 GICs (as found
on the Cortex-A9 and newer) states that the GICC_PMR reset value
is zero; this differs from the 0xf0 reset value used on 11MPCore.
The NVIC is different again in not having a CPU interface; since
we share the GIC code we must force the
On Fri, 2 Nov 2012 11:42:32 -0200
Luiz Capitulino wrote:
> On Fri, 02 Nov 2012 14:40:03 +0100
> Kevin Wolf wrote:
>
> > Am 02.11.2012 14:25, schrieb Luiz Capitulino:
> > > On Fri, 19 Oct 2012 11:27:59 -0300
> > > Luiz Capitulino wrote:
> > >
> > >> By adding error propagation to bdrv_img_crea
Ping?
P.S. Please note a typo in the cover letter: when I referred to KSM, I
meant "kernel samepage merging", not "kernel shared memory" :-)
On Tue, Nov 20, 2012 at 7:10 AM, Peter Feiner wrote:
> This patch makes the -mem-path filenames deterministic and allows some
> control
> over how QEMU m
On Thursday, November 29, 2012 01:56:41 PM Eduardo Otubo wrote:
> According to the bug 855162[0] - there's the need of adding new syscalls
> to the whitelist when using Qemu with Libvirt.
>
> [0] - https://bugzilla.redhat.com/show_bug.cgi?id=855162
>
> Reported-by: Paul Moore
> Signed-off-by: Ed
According to the bug 855162[0] - there's the need of adding new syscalls
to the whitelist when using Qemu with Libvirt.
[0] - https://bugzilla.redhat.com/show_bug.cgi?id=855162
Reported-by: Paul Moore
Signed-off-by: Eduardo Otubo
Signed-off-by: Corey Bryant
---
qemu-seccomp.c | 156 +
Debug option is available since QEMU-1.2; FP coprocessor and
coprocessor context is available since QEMU-1.3.
Signed-off-by: Max Filippov
---
qemu-tech.texi | 10 +-
1 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/qemu-tech.texi b/qemu-tech.texi
index d73dda8..8aefa74 10
- Messaggio originale -
> Da: "Michael S. Tsirkin"
> A: "Paolo Bonzini"
> Cc: qemu-devel@nongnu.org, "Kevin Wolf" , "Anthony Liguori"
> , "Blue Swirl"
> , k...@us.ibm.com, "Asias He" ,
> "Stefan Hajnoczi"
> Inviato: Giovedì, 29 novembre 2012 16:27:39
> Oggetto: Re: [PATCH v4 10/11] d
On Thu, Nov 29, 2012 at 03:56:09PM +0100, Igor Mammedov wrote:
> On Fri, 9 Nov 2012 16:48:07 +0100
> Eduardo Habkost wrote:
>
> >
> > On 22/10/2012, at 17:03, Igor Mammedov wrote:
> >
> > > Signed-off-by: Igor Mammedov
> > > ---
> > > target-i386/cpu.c | 9 +++--
> > > 1 file changed, 3 in
Am 29.11.2012 16:12, schrieb Anthony Liguori:
> We are currently checking for an exact type match. Use QOM dynamic_cast to
> check for a compatible type instead.
>
> Cc: Konrad Frederic
> Cc: Peter Maydell
> Signed-off-by: Anthony Liguori
Reviewed-by: Andreas Färber
Andreas
--
SUSE LINUX
On Thu, Nov 29, 2012 at 03:48:04PM +0100, Stefan Hajnoczi wrote:
> On Thu, Nov 29, 2012 at 04:09:28PM +0200, Michael S. Tsirkin wrote:
> > On Thu, Nov 29, 2012 at 10:18:59AM +0100, Stefan Hajnoczi wrote:
> > > Michael, Paolo: Are you happy with v4?
> >
> > Looks pretty clean by itself. I sent some
> > I don't understand. If memory in the VGA region returns true from
> > memory_region_is_ram(), why would there be a problem?
>
> If you change this memory but you don't update the display.
> Never happens with non buggy guests but we should catch and fail if
> it does.
Actually it _could_ hap
On Thu, Nov 29, 2012 at 10:21:31AM -0500, Paolo Bonzini wrote:
>
> > > +unsigned int num_reqs;
> > > +QemuMutex num_reqs_lock;
> >
> > OK the only reason this lock is needed is because
> > you want to drain outside the thread.
> > Won't it be better to queue process the drain request thro
> > @@ -574,6 +570,12 @@ static void
> > qemu_rbd_aio_cancel(BlockDriverAIOCB *blockacb)
> > {
> > RBDAIOCB *acb = (RBDAIOCB *) blockacb;
> > acb->cancelled = 1;
> > +
> > +while (acb->status == -EINPROGRESS) {
> > +qemu_aio_wait();
> > +}
> > +
>
> There should be a
- Messaggio originale -
> Da: "Stefan Priebe"
> A: qemu-devel@nongnu.org
> Cc: stefa...@gmail.com, "josh durgin" ,
> ceph-de...@vger.kernel.org, pbonz...@redhat.com,
> "Stefan Priebe"
> Inviato: Giovedì, 29 novembre 2012 15:28:35
> Oggetto: [PATCHv4] rbd block driver fix race between a
> > +unsigned int num_reqs;
> > +QemuMutex num_reqs_lock;
>
> OK the only reason this lock is needed is because
> you want to drain outside the thread.
> Won't it be better to queue process the drain request through
> the thread?
> You won't need any locks then.
Draining is processed in
On 29/11/2012 16:12, Anthony Liguori wrote:
We are currently checking for an exact type match. Use QOM dynamic_cast to
check for a compatible type instead.
Cc: Konrad Frederic
Cc: Peter Maydell
Signed-off-by: Anthony Liguori
---
v1 -> v2:
- also add cast to qbus_find_recursive (Peter)
- si
> > +/* Toggle guest->host notifies */
> > +void vring_set_notification(VirtIODevice *vdev, Vring *vring, bool
> > enable)
> > +{
> > +if (vdev->guest_features & (1 << VIRTIO_RING_F_EVENT_IDX)) {
> > +if (enable) {
> > +vring_avail_event(&vring->vr) = vring->vr.avail->idx;
We are currently checking for an exact type match. Use QOM dynamic_cast to
check for a compatible type instead.
Cc: Konrad Frederic
Cc: Peter Maydell
Signed-off-by: Anthony Liguori
---
v1 -> v2:
- also add cast to qbus_find_recursive (Peter)
- simplify by doing object_dynamic_cast instead of
On Fri, 9 Nov 2012 16:48:07 +0100
Eduardo Habkost wrote:
>
> On 22/10/2012, at 17:03, Igor Mammedov wrote:
>
> > Signed-off-by: Igor Mammedov
> > ---
> > target-i386/cpu.c | 9 +++--
> > 1 file changed, 3 insertions(+), 6 deletions(-)
> >
> > diff --git a/target-i386/cpu.c b/target-i386/c
On Thu, Nov 29, 2012 at 03:45:55PM +0100, Stefan Hajnoczi wrote:
> On Thu, Nov 29, 2012 at 03:12:35PM +0200, Michael S. Tsirkin wrote:
> > On Thu, Nov 22, 2012 at 04:16:52PM +0100, Stefan Hajnoczi wrote:
> > > The virtio-blk-data-plane feature is easy to integrate into
> > > hw/virtio-blk.c. The d
On Thu, Nov 29, 2012 at 04:09:28PM +0200, Michael S. Tsirkin wrote:
> On Thu, Nov 29, 2012 at 10:18:59AM +0100, Stefan Hajnoczi wrote:
> > Michael, Paolo: Are you happy with v4?
>
> Looks pretty clean by itself. I sent some comments but they can be
> addressed later. What worries me most is the c
On Thu, Nov 29, 2012 at 03:12:35PM +0200, Michael S. Tsirkin wrote:
> On Thu, Nov 22, 2012 at 04:16:52PM +0100, Stefan Hajnoczi wrote:
> > The virtio-blk-data-plane feature is easy to integrate into
> > hw/virtio-blk.c. The data plane can be started and stopped similar to
> > vhost-net.
> >
> > U
On Thu, Nov 29, 2012 at 03:26:56PM +0100, Stefan Hajnoczi wrote:
> On Thu, Nov 29, 2012 at 03:54:25PM +0200, Michael S. Tsirkin wrote:
> > On Thu, Nov 22, 2012 at 04:16:44PM +0100, Stefan Hajnoczi wrote:
> > > The data plane thread needs to map guest physical addresses to host
> > > pointers. Norm
Hi,
i hope i've done everything correctly. I've send a new v4 patch.
Am 29.11.2012 14:58, schrieb Stefan Hajnoczi:
On Thu, Nov 22, 2012 at 11:00:19AM +0100, Stefan Priebe wrote:
@@ -406,10 +401,11 @@ static void qemu_rbd_complete_aio(RADOSCB *rcb)
acb->ret = r;
}
This one fixes a race which qemu had also in iscsi block driver
between cancellation and io completition.
qemu_rbd_aio_cancel was not synchronously waiting for the end of
the command.
To archieve this it introduces a new status flag which uses
-EINPROGRESS.
Changes since PATCHv3:
- removed unnec
Hi,
can someone give me a short status of the topic Live Migration and VMs that use
TSC as clocksource?
Broken, yes / no? And if yes under what circumstances?
I have some TSC VMs that freeze occasionally after LiveMigration.
Thank you,
Peter
On 29/11/2012 14:55, Andreas Färber wrote:
Am 29.11.2012 14:47, schrieb Konrad Frederic:
On 29/11/2012 14:09, Peter Maydell wrote:
On 29 November 2012 12:37, Konrad Frederic
wrote:
On 26/11/2012 17:59, Anthony Liguori wrote:
virtio-pci-bus extends virtio-bus
- is constructed with a pointe
On Thu, Nov 29, 2012 at 03:54:25PM +0200, Michael S. Tsirkin wrote:
> On Thu, Nov 22, 2012 at 04:16:44PM +0100, Stefan Hajnoczi wrote:
> > The data plane thread needs to map guest physical addresses to host
> > pointers. Normally this is done with cpu_physical_memory_map() but the
> > function ass
On Wed, Nov 28, 2012 at 03:08:24AM +, Furukawa, Eiji wrote:
> About a source of qemu-1.2.0/main-loop.c
> The select handling of os_host_main_loop_wait function
> I do not seem to do Exit by interrupts such as SIGUSR1
> Will not it be necessary to make modifications?
>
> Before
> LineNumber:30
On Tue, Nov 27, 2012 at 09:16:24AM +0100, Paolo Bonzini wrote:
> g_assert_cmpint is not available on glib 2.12, which is the minimum
> version required to build QEMU (we only require 2.16 to run tests,
> since that is the first version including GTester). Do not use it
> in hardware models, use a
On Thu, Nov 29, 2012 at 06:34:46PM +0530, Amit Shah wrote:
> On (Wed) 28 Nov 2012 [23:53:08], Michael S. Tsirkin wrote:
> > On Tue, Nov 27, 2012 at 06:25:04PM +0200, Michael S. Tsirkin wrote:
> > > On Thu, Nov 01, 2012 at 06:07:21PM +0200, Michael S. Tsirkin wrote:
> > > > Commit 0d8d7690850eb0cf2b
On Thu, Nov 29, 2012 at 10:18:59AM +0100, Stefan Hajnoczi wrote:
> Michael, Paolo: Are you happy with v4?
Looks pretty clean by itself. I sent some comments but they can be
addressed later. What worries me most is the code duplication with
regular virtio.
I see two ways to reduce the maintainanc
On 29/11/2012 14:56, Peter Maydell wrote:
On 29 November 2012 13:49, Anthony Liguori wrote:
We are currently checking for an exact type match. Use QOM dynamic_cast to
check for a compatible type instead.
I think this only catches the case where a bus was explicitly
specified via bus=. For the
On Thu, Nov 22, 2012 at 04:16:51PM +0100, Stefan Hajnoczi wrote:
> virtio-blk-data-plane is a subset implementation of virtio-blk. It only
> handles read, write, and flush requests. It does this using a dedicated
> thread that executes an epoll(2)-based event loop and processes I/O
> using Linux
On Thu, Nov 22, 2012 at 11:00:19AM +0100, Stefan Priebe wrote:
> @@ -406,10 +401,11 @@ static void qemu_rbd_complete_aio(RADOSCB *rcb)
> acb->ret = r;
> }
> }
> +acb->status = 0;
> +
I suggest doing this in the BH. The qemu_aio_wait() loop in
qemu_rbd_aio_cancel() n
On Thu, 2012-11-29 at 13:21 +, Stefano Stabellini wrote:
> > Also I think 4.2.1 need these patches to enable the basic Xen on Xen
> > nested virtualization usage scenario.
>
> I agree.
Nested virt was a tech preview in 4.2.0, is it really worth
backporting ?
Ian.
On 29 November 2012 13:49, Anthony Liguori wrote:
> We are currently checking for an exact type match. Use QOM dynamic_cast to
> check for a compatible type instead.
I think this only catches the case where a bus was explicitly
specified via bus=. For the default case you also need to change
the
Am 29.11.2012 14:47, schrieb Konrad Frederic:
> On 29/11/2012 14:09, Peter Maydell wrote:
>> On 29 November 2012 12:37, Konrad Frederic
>> wrote:
>>> On 26/11/2012 17:59, Anthony Liguori wrote:
virtio-pci-bus extends virtio-bus
- is constructed with a pointer to a PCIDevice
-
On 29 November 2012 13:47, Konrad Frederic wrote:
> On 29/11/2012 14:09, Peter Maydell wrote:
>> I suspect that qbus_find_recursive should be doing an
>> object_class_dynamic_cast() to check that the bus is of a suitable
>> type, rather than the
>> (strcmp(object_get_typename(OBJECT(bus)), bu
Konrad Frederic writes:
> On 26/11/2012 17:59, Anthony Liguori wrote:
>> Peter Maydell writes:
>>
>>> On 26 November 2012 14:33, Anthony Liguori wrote:
VirtioBusInfo is not a great name. This is a proxy class that allows
for a device to implement the virtio bus interface.
T
On Thu, Nov 22, 2012 at 04:16:44PM +0100, Stefan Hajnoczi wrote:
> The data plane thread needs to map guest physical addresses to host
> pointers. Normally this is done with cpu_physical_memory_map() but the
> function assumes the global mutex is held. The data plane thread does
> not touch the g
We are currently checking for an exact type match. Use QOM dynamic_cast to
check for a compatible type instead.
Cc: Konrad Frederic
Signed-off-by: Anthony Liguori
---
hw/qdev-monitor.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/hw/qdev-monitor.c b/hw/qdev-monitor
On 29/11/2012 14:09, Peter Maydell wrote:
On 29 November 2012 12:37, Konrad Frederic wrote:
On 26/11/2012 17:59, Anthony Liguori wrote:
virtio-pci-bus extends virtio-bus
- is constructed with a pointer to a PCIDevice
- implements the methods necessary to be a virtio bus
I still have tro
Hi,
This is the usb patch queue, carrying three little fixes for 1.3.
please pull,
Gerd
The following changes since commit e9bff10f8db94912b1b0e6e2e3394cae02faf614:
event notifier: Fix setup for win32 (2012-11-28 13:33:01 -0600)
are available in the git repository at:
git://git.kraxel.
On Thu, Nov 22, 2012 at 04:16:45PM +0100, Stefan Hajnoczi wrote:
> The virtio-blk-data-plane cannot access memory using the usual QEMU
> functions since it executes outside the global mutex and the memory APIs
> are this time are not thread-safe.
>
> This patch introduces a virtqueue module based
From: Peter Crosthwaite
This was left as NULL on the initial merge due to debate on the mailing list on
how to handle DMA contexts for sysbus devices. Patch
9e11908f12f92e31ea94dc2a4c962c836cba9f2a was later merged to fix OHCI. This is
the,
equivalent fix for sysbus EHCI.
Signed-off-by: Peter C
On Thu, Nov 22, 2012 at 04:16:51PM +0100, Stefan Hajnoczi wrote:
> virtio-blk-data-plane is a subset implementation of virtio-blk. It only
> handles read, write, and flush requests. It does this using a dedicated
> thread that executes an epoll(2)-based event loop and processes I/O
> using Linux
On Tue, Nov 27, 2012 at 01:11:25PM -0700, Bruce Rogers wrote:
> The second parameter to qapi_dealloc_type_size should be a uint64_t *,
> not a size_t *. This was causing our 32 bit x86 build to fail, since
> warnings are treated as errors.
>
> Signed-off-by: Bruce Rogers
> ---
> qapi/qapi-deallo
On Thu, 29 Nov 2012, Xu, Dongxiao wrote:
> > -Original Message-
> > From: Stefano Stabellini [mailto:stefano.stabell...@eu.citrix.com]
> > Sent: Tuesday, September 18, 2012 6:24 PM
> > To: Xu, Dongxiao
> > Cc: Stefano Stabellini; xen-de...@lists.xensource.com; Ian Jackson;
> > qemu-devel@no
Hi,
Pretty short this time, with a single lonely bugfix.
please pull,
Gerd
The following changes since commit e9bff10f8db94912b1b0e6e2e3394cae02faf614:
event notifier: Fix setup for win32 (2012-11-28 13:33:01 -0600)
are available in the git repository at:
git://anongit.freedesktop.org/
From: Yonit Halperin
The devram memslot stays active when qxl enters UNDEFINED mode (i.e, no
primary surface). If migration has occurred while the device is in
UNDEFINED stae, the memslots have to be reloaded at the destination.
Fixes rhbz#874574
Signed-off-by: Yonit Halperin
Signed-off-by: Ge
Hotplugging them simply doesn't work, so tag them accordingly to
avoid users trying and then crashing qemu.
For xhci there is nothing fundamental which prevents hotplug from
working, we'll "only" need a exit() function which cleans up
everything properly. That isn't for 1.3 though.
For ehci+uhci
From: Stefan Hajnoczi
Report an error instead of segfaulting when attaching a USB device to a
machine with no USB busses:
$ qemu-system-arm -machine vexpress-a9 \
-sd Fedora-17-armhfp-vexpress-mmcblk0.img \
-kernel vmlinuz-3.4.2-3.fc17.armv7hl \
-initrd initramfs-3.4.2-3.fc17
From: Bruce Rogers
The second parameter to qapi_dealloc_type_size should be a uint64_t *,
not a size_t *. This was causing our 32 bit x86 build to fail, since
warnings are treated as errors.
Signed-off-by: Bruce Rogers
Reviewed-by: Michael Roth
Reviewed-by: Stefan Weil
Signed-off-by: Luiz Cap
This is an important fix as it fixes a 32-bit breakage.
The changes (since e9bff10f8db94912b1b0e6e2e3394cae02faf614) are available
in the following repository:
git://repo.or.cz/qemu/qmp-unstable.git queue/qmp
Bruce Rogers (1):
qapi: fix qapi_dealloc_type_size parameter type
qapi/qapi-dea
1 - 100 of 123 matches
Mail list logo
