On Thu, 09 Jun 2011 16:10:39 -0700, Venkateswararao Jujjuri
wrote:
> On 06/06/2011 10:16 AM, Aneesh Kumar K.V wrote:
> > Signed-off-by: Aneesh Kumar K.V
>
> Just one minor issue below; otherwise
>
> Reviewed-by: Venkateswararao Jujjuri "
>
> > ---
> > hw/9pfs/virtio-9p.c | 205
> > ++
Hi,
I've two scenarios at hand now: one involves a Windows 7 guest with
-soundhw hda, the other is my Musicpal (do you still have a firmware
image at hand?). Some Linux guest here with identical configuration like
Win7 does not show any problems in polling mode.
Noteworthy difference between
On Thu, 09 Jun 2011 17:27:00 -0700, Venkateswararao Jujjuri
wrote:
> On 06/06/2011 10:16 AM, Aneesh Kumar K.V wrote:
> > On interrupted syscall on client we can end up having fid
> > being acted upon by glib pool but still get a clunk request on that
>
> Couple of comments below.
>
> - JV
>
>
On Fri, Jun 10, 2011 at 09:20, Ice Frog wrote:
> Hi all,
>
> I will analyze OS kernel security using virtualization. I'm aware both
> QEMU and XEN can be good platforms to build my tool. Any one can give
> some suggestions?
precisely, can you tell us why do you need virtualization? to check
every
Tested-by: Alexandre Raymond
That's definitely a better patch ;-)
Thanks,
Alexandre
On Thu, Jun 9, 2011 at 3:03 PM, Andreas Färber wrote:
> Avoid compiler confusion as to which method signature to use for the
> handleEvent: selector on OSX >= 10.6 by making the variable type-safe
> as oppose
Hi all,
I will analyze OS kernel security using virtualization. I'm aware both
QEMU and XEN can be good platforms to build my tool. Any one can give
some suggestions?
Thanks,
On 06/06/2011 10:16 AM, Aneesh Kumar K.V wrote:
Signed-off-by: Aneesh Kumar K.V
Reviewed-by: Venkateswararao Jujjuri
---
hw/9pfs/codir.c |9 +
hw/9pfs/virtio-9p.c | 26 --
2 files changed, 33 insertions(+), 2 deletions(-)
diff --git a/hw/9pfs/cod
On 06/06/2011 10:16 AM, Aneesh Kumar K.V wrote:
we should use the local abstraction instead of
directly calling close.
Let us fold this also into our coroutine patches.
- JV
Signed-off-by: Aneesh Kumar K.V
---
hw/9pfs/virtio-9p.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(
On 06/06/2011 10:16 AM, Aneesh Kumar K.V wrote:
Signed-off-by: Aneesh Kumar K.V
Minor comments below; I think this looks good.
Reviewed-by : Venkateswararao Jujjuri
---
hw/9pfs/codir.c|4 +-
hw/9pfs/cofile.c | 19 -
hw/9pfs/virtio-9p-coth.h |4 +-
h
On 06/06/2011 10:16 AM, Aneesh Kumar K.V wrote:
On interrupted syscall on client we can end up having fid
being acted upon by glib pool but still get a clunk request on that
Couple of comments below.
- JV
Signed-off-by: Aneesh Kumar K.V
---
hw/9pfs/virtio-9p.c | 139 ++
On 06/06/2011 10:16 AM, Aneesh Kumar K.V wrote:
Signed-off-by: Aneesh Kumar K.V
Just one minor issue below; otherwise
Reviewed-by: Venkateswararao Jujjuri "
---
hw/9pfs/virtio-9p.c | 205 +++
hw/9pfs/virtio-9p.h |7 ++
2 files changed,
On 06/09/2011 06:04 PM, Michael Roth wrote:
On 06/09/2011 03:02 PM, Luiz Capitulino wrote:
On Thu, 09 Jun 2011 14:04:37 -0500
Anthony Liguori wrote:
On 06/09/2011 01:47 PM, Luiz Capitulino wrote:
I've started writing some tests with the glib test framework (used
by the qapi
patches) but am f
On 06/09/2011 03:02 PM, Luiz Capitulino wrote:
On Thu, 09 Jun 2011 14:04:37 -0500
Anthony Liguori wrote:
On 06/09/2011 01:47 PM, Luiz Capitulino wrote:
I've started writing some tests with the glib test framework (used by the qapi
patches) but am facing some issues that doesn't seem to exist
Older versions of libcurl don't have some of the features we try to
use, in particular curl_multi_setopt(). Check for this in the 'is
libcurl available?' configure test so we disable curl support if the
library is too old.
Signed-off-by: Peter Maydell
---
configure |2 +-
1 files changed, 1
When qemu gets compiled without support of vhost-net, any attempt
to use it fails with a very clear error message:
qemu-system-x86_64: -netdev ...,vhost=on: vhost-net requested but could not be
initialized
there's absolutely no reason given _why_ it coult not be
initialized, and even strace'ing
On 2011-06-09 22:00, Anthony Liguori wrote:
> On 06/09/2011 11:44 AM, Jan Kiszka wrote:
>> A first step towards getting rid of register_device_unmigratable
>> (ivshmem and lacking vmstate support in virtio are blocking this):
>>
>> Allow to register an unmigratable vmstate via qdev, i.e. tag a devi
On Thu, 09 Jun 2011 14:04:37 -0500
Anthony Liguori wrote:
> On 06/09/2011 01:47 PM, Luiz Capitulino wrote:
> >
> > I've started writing some tests with the glib test framework (used by the
> > qapi
> > patches) but am facing some issues that doesn't seem to exist with check
> > (our
> > current
On 06/09/2011 11:44 AM, Jan Kiszka wrote:
A first step towards getting rid of register_device_unmigratable
(ivshmem and lacking vmstate support in virtio are blocking this):
Allow to register an unmigratable vmstate via qdev, i.e. tag a device
declaratively.
I thought part of the problem with
On 2011-06-09 20:08, Eduardo Habkost wrote:
> On Thu, Jun 09, 2011 at 07:41:03PM +0200, Jan Kiszka wrote:
>
>>> I'm wondering if it wouldn't be simpler to keep the existing interface
>>> but just initialize CPUState->kvm_state earlier (today it is initialized
>>> only on kvm_init_vcpu(), although
On 06/09/2011 12:47 PM, Luiz Capitulino wrote:
On Fri, 3 Jun 2011 17:33:09 -0500
Michael Roth wrote:
Type of Visiter class that serves as the inverse of the input visiter:
it takes a series of native C types and uses their values to construct a
corresponding QObject. The command marshaling/di
Am 09.06.2011 um 20:47 schrieb Andreas Färber:
Am 07.06.2011 um 05:34 schrieb Alexandre Raymond:
Changes since v1: create a wrapper function named qemu_daemon() in
oslib-posix.c
instead of putting the OS specific workaround in qemu-nbd.c directly.
On OSX >= 10.5, daemon() is deprecated, res
Hi Alexandre,
Am 07.06.2011 um 01:39 schrieb Alexandre Raymond:
On Tue, May 31, 2011 at 4:07 PM, Andreas Färber > wrote:
Hello Alexandre,
Am 30.05.2011 um 00:36 schrieb Alexandre Raymond:
This fix forces a message to be sent to the handleEvent method
of the QemuCocoaView class instead of let
On 06/09/2011 01:47 PM, Luiz Capitulino wrote:
I've started writing some tests with the glib test framework (used by the qapi
patches) but am facing some issues that doesn't seem to exist with check (our
current framework).
Of course that it's possible that I'm missing something, in this case p
On 9 June 2011 18:44, Andreas Färber wrote:
> Am 09.06.2011 um 17:52 schrieb Marc-Antoine Perennou:
>> Manually including stddef.h or replacing NULL by 0 or (void*)0 makes it
>> work.
>
> Then please submit a new patch that explicit includes that header with a
> comment explaining why, so that it
Avoid compiler confusion as to which method signature to use for the
handleEvent: selector on OSX >= 10.6 by making the variable type-safe
as opposed to generic 'id' type.
Requires moving the variable definition to after the class definition.
8<
ui/cocoa.m: In function ‘cocoa_refresh’:
ui/
Am 07.06.2011 um 05:34 schrieb Alexandre Raymond:
Changes since v1: create a wrapper function named qemu_daemon() in
oslib-posix.c
instead of putting the OS specific workaround in qemu-nbd.c directly.
On OSX >= 10.5, daemon() is deprecated, resulting in the following
warning:
8<
qe
I've started writing some tests with the glib test framework (used by the qapi
patches) but am facing some issues that doesn't seem to exist with check (our
current framework).
Of course that it's possible that I'm missing something, in this case pointers
are welcome, but I must admit that my fir
Am 02.06.2011 um 20:59 schrieb Andreas Färber:
In 821601ea5b02a68ada479731a4d3d07a9876632a (Make VNC support
optional)
cocoa.o was moved from ui-obj-$(CONFIG_COCOA) to vnc-obj-$
(CONFIG_COCOA),
adding a dependency on $(CONFIG_VNC). That must've been unintentional.
Cc: Jes Sorensen
Cc: Antho
Aurelien Jarno writes:
> On Fri, Apr 29, 2011 at 05:59:19PM +0200, Marc-Antoine Perennou wrote:
>> pulse/simple.h does not include stdlib.h
>> We cannot use NULL since it may not be defined
>> Use 0 instead
>
> I am unable to reproduce this issue, even with gcc-4.6. Also please note
> that NULL i
Am 29.05.2011 um 21:58 schrieb Andreas Färber:
This fixes a missing prototype warning in vl.c and obsoletes
the prototype in cocoa.m. Adjust callers in cocoa.m to supply
third argument, which is currently only used on Linux/ppc.
The prototype is designed so that it could be shared with SDL
and
Jan Kiszka writes:
> A first step towards getting rid of register_device_unmigratable
> (ivshmem and lacking vmstate support in virtio are blocking this):
>
> Allow to register an unmigratable vmstate via qdev, i.e. tag a device
> declaratively.
Declarative is nicer than imperative. ACK.
On 06/09/2011 10:14 AM, Luiz Capitulino wrote:
On Fri, 3 Jun 2011 17:33:07 -0500
Michael Roth wrote:
Base definitions/includes for Visiter interface used by generated
visiter/marshalling code.
Signed-off-by: Michael Roth
---
qapi/qapi-visit-core.h | 187 +++
On 06/09/2011 11:41 AM, Michael Roth wrote:
On 06/09/2011 11:26 AM, Peter Maydell wrote:
On 3 June 2011 23:33, Michael Roth wrote:
A type of Visiter class
[randomly noted against this patch because this is where I happened
to notice it...]
Should be "Visitor" throughout (and in other patches
On Thu, Jun 09, 2011 at 07:41:03PM +0200, Jan Kiszka wrote:
> > I'm wondering if it wouldn't be simpler to keep the existing interface
> > but just initialize CPUState->kvm_state earlier (today it is initialized
> > only on kvm_init_vcpu(), although the kvm_state global is initialized
> > much ear
On Fri, 3 Jun 2011 17:33:09 -0500
Michael Roth wrote:
> Type of Visiter class that serves as the inverse of the input visiter:
> it takes a series of native C types and uses their values to construct a
> corresponding QObject. The command marshaling/dispatcher functions will
> use this to conver
Am 09.06.2011 um 17:52 schrieb Marc-Antoine Perennou:
On 4 June 2011 00:54, malc wrote:
On Sat, 4 Jun 2011, Aurelien Jarno wrote:
On Sat, Jun 04, 2011 at 01:57:23AM +0400, malc wrote:
On Fri, 3 Jun 2011, Aurelien Jarno wrote:
On Fri, Apr 29, 2011 at 05:59:19PM +0200, Marc-Antoine Perennou
On platforms that doesn't support makecontext use gthread
based coroutine implementation.
Signed-off-by: Aneesh Kumar K.V
---
NOTE: Tested on linux with force compliation of coroutine-gthread.c
Makefile.objs |5 ++
configure | 18 +
coroutine-gthread.c | 172
On 2011-06-09 19:33, Eduardo Habkost wrote:
> On Wed, Jun 08, 2011 at 04:11:05PM +0200, Jan Kiszka wrote:
>> kvm_arch_get_supported_cpuid checks for global cpuid restrictions, it
>> does not require any CPUState reference. Changing its interface allows
>> to call it before any VCPU is initialized.
On Wed, Jun 08, 2011 at 04:11:05PM +0200, Jan Kiszka wrote:
> kvm_arch_get_supported_cpuid checks for global cpuid restrictions, it
> does not require any CPUState reference. Changing its interface allows
> to call it before any VCPU is initialized.
I'm wondering if it wouldn't be simpler to keep
On 2011-06-09 18:09, malc wrote:
> On Thu, 9 Jun 2011, Jan Kiszka wrote:
>
>> On 2011-06-09 17:13, malc wrote:
>>> On Thu, 9 Jun 2011, Jan Kiszka wrote:
>>>
>
> [..snip..]
>
>>
>> Sound output over (at least) ALSA is not smoothly working in polled
>> mode. It's similar to [1], which is yet unsol
On 2011-06-09 18:40, Markus Armbruster wrote:
> Jan Kiszka writes:
>
>> On 2011-06-08 13:33, Peter Maydell wrote:
>>> At the moment you can't really implement one sysbus device by saying
>>> that it's composed of a set of other sysbus devices. This patch adds
>>> new functions sysbus_pass_mmio()
On 06/09/2011 10:55 AM, Luiz Capitulino wrote:
On Thu, 09 Jun 2011 10:41:53 -0500
Michael Roth wrote:
On 06/09/2011 10:30 AM, Luiz Capitulino wrote:
On Fri, 3 Jun 2011 17:33:08 -0500
Michael Roth wrote:
A type of Visiter class that is used to walk a qobject's
structure and assign each en
A first step towards getting rid of register_device_unmigratable
(ivshmem and lacking vmstate support in virtio are blocking this):
Allow to register an unmigratable vmstate via qdev, i.e. tag a device
declaratively.
Signed-off-by: Jan Kiszka
---
hw/hw.h |1 +
savevm.c |1 +
2 files ch
Am 09.06.2011 um 17:35 schrieb Markus Armbruster:
Andreas Färber writes:
Incorporate ISA VMState. Add "enabled" property.
Could you explain why you need to stick VMSTATE_ISA_DEVICE_V() into
vmstate_isa_serial, but not for the other devices?
I stuck it in all affected devices that have VMS
Jan Kiszka writes:
> On 2011-06-08 13:33, Peter Maydell wrote:
>> At the moment you can't really implement one sysbus device by saying
>> that it's composed of a set of other sysbus devices. This patch adds
>> new functions sysbus_pass_mmio() and sysbus_pass_one_irq() which
>> allow a sysbus devi
On 06/09/2011 11:26 AM, Peter Maydell wrote:
On 3 June 2011 23:33, Michael Roth wrote:
A type of Visiter class
[randomly noted against this patch because this is where I happened
to notice it...]
Should be "Visitor" throughout (and in other patches), please?
-- PMM
Stefan pointed that out
Andreas Färber writes:
> Hi,
>
> Am 09.06.2011 um 14:23 schrieb Gerd Hoffmann:
>
I get the feeling that doing all this in the pc87312 emulation is
easier as it needs to have this logic anyway for config register
writes and you can probably reuse the code for loadvm pre- and
po
On 3 June 2011 23:33, Michael Roth wrote:
> A type of Visiter class
[randomly noted against this patch because this is where I happened
to notice it...]
Should be "Visitor" throughout (and in other patches), please?
-- PMM
On Thu, 9 Jun 2011, Jan Kiszka wrote:
> On 2011-06-09 17:13, malc wrote:
> > On Thu, 9 Jun 2011, Jan Kiszka wrote:
> >
[..snip..]
>
> Sound output over (at least) ALSA is not smoothly working in polled
> mode. It's similar to [1], which is yet unsolved.
>
> Jan
>
> [1] http://thread.gmane.or
On Thu, 09 Jun 2011 10:41:53 -0500
Michael Roth wrote:
> On 06/09/2011 10:30 AM, Luiz Capitulino wrote:
> > On Fri, 3 Jun 2011 17:33:08 -0500
> > Michael Roth wrote:
> >
> >> A type of Visiter class that is used to walk a qobject's
> >> structure and assign each entry to the corresponding nativ
Hello Stefan,
Can you expand on this some more? I have similar concerns for Livebackup.
At the beginning of your paragraph, did you mean 'asynchronous I/O
emulation' instead of 'synchronous I/O emulation'?
Also, I don't understand the 'stack' construct that you refer to. When you
say 'push a
On 06/09/2011 10:30 AM, Luiz Capitulino wrote:
On Fri, 3 Jun 2011 17:33:08 -0500
Michael Roth wrote:
A type of Visiter class that is used to walk a qobject's
structure and assign each entry to the corresponding native C type.
Command marshaling function will use this to pull out QMP command
p
On 06/09/2011 10:05 AM, Luiz Capitulino wrote:
On Fri, 3 Jun 2011 17:33:03 -0500
Michael Roth wrote:
This is the code generator for qapi types. It will generation the
following files:
$(prefix)qapi-types.h - C types corresponding to types defined in
the schema y
On 4 June 2011 00:54, malc wrote:
> On Sat, 4 Jun 2011, Aurelien Jarno wrote:
>
>> On Sat, Jun 04, 2011 at 01:57:23AM +0400, malc wrote:
>> > On Fri, 3 Jun 2011, Aurelien Jarno wrote:
>> >
>> > > On Fri, Apr 29, 2011 at 05:59:19PM +0200, Marc-Antoine Perennou wrote:
>> > > > pulse/simple.h does no
On Thu, 9 Jun 2011, Jan Kiszka wrote:
> On 2011-06-09 09:08, Andr? Weidemann wrote:
> > Hi Jan,
> >
> > On 09.06.2011 08:53, Jan Kiszka wrote:
> >> On 2011-06-09 08:19, Andr? Weidemann wrote:
> >>> Hi Gerd,
> >>>
> >>> On 08.06.2011 14:49, Gerd Hoffmann wrote:
> Hi,
>
> > The sound
On Fri, 3 Jun 2011 17:33:08 -0500
Michael Roth wrote:
> A type of Visiter class that is used to walk a qobject's
> structure and assign each entry to the corresponding native C type.
> Command marshaling function will use this to pull out QMP command
> parameters recieved over the wire and pass
Andreas Färber writes:
> Incorporate ISA VMState. Add "enabled" property.
Could you explain why you need to stick VMSTATE_ISA_DEVICE_V() into
vmstate_isa_serial, but not for the other devices?
>
> Signed-off-by: Andreas Färber
> ---
> hw/serial.c | 30 +-
> 1 fil
Markus Armbruster writes:
> Andreas Färber writes:
>
>> Signed-off-by: Andreas Färber
>> ---
>> hw/isa-bus.c | 14 ++
>> hw/isa.h |1 +
>> 2 files changed, 15 insertions(+), 0 deletions(-)
>>
>> diff --git a/hw/isa-bus.c b/hw/isa-bus.c
>> index 1f64673..6ac3e61 100644
>>
On 2011-06-09 17:13, malc wrote:
> On Thu, 9 Jun 2011, Jan Kiszka wrote:
>
>> On 2011-06-09 09:08, Andr? Weidemann wrote:
>>> Hi Jan,
>>>
>>> On 09.06.2011 08:53, Jan Kiszka wrote:
On 2011-06-09 08:19, Andr? Weidemann wrote:
> Hi Gerd,
>
> On 08.06.2011 14:49, Gerd Hoffmann wrote:
On Fri, 3 Jun 2011 17:33:07 -0500
Michael Roth wrote:
> Base definitions/includes for Visiter interface used by generated
> visiter/marshalling code.
>
> Signed-off-by: Michael Roth
> ---
> qapi/qapi-visit-core.h | 187
>
> 1 files changed, 1
On Fri, 3 Jun 2011 17:33:03 -0500
Michael Roth wrote:
> This is the code generator for qapi types. It will generation the
> following files:
>
> $(prefix)qapi-types.h - C types corresponding to types defined in
> the schema you pass in
> $(prefix)qapi-types.c - Cle
Andreas Färber writes:
> Signed-off-by: Andreas Färber
> ---
> hw/isa-bus.c | 14 ++
> hw/isa.h |1 +
> 2 files changed, 15 insertions(+), 0 deletions(-)
>
> diff --git a/hw/isa-bus.c b/hw/isa-bus.c
> index d258932..1f64673 100644
> --- a/hw/isa-bus.c
> +++ b/hw/isa-bus.c
Am 09.06.2011 um 14:36 schrieb Andreas Färber:
Am 09.06.2011 um 00:55 schrieb Alexander Graf:
Qemu uses signalfd to figure out, if a signal occured without the
need
to actually receive the signal. Instead, it can read from the fd to
receive
its news.
Now, we obviously don't always have si
Andreas Färber writes:
> Signed-off-by: Andreas Färber
> ---
> hw/isa-bus.c | 14 ++
> hw/isa.h |1 +
> 2 files changed, 15 insertions(+), 0 deletions(-)
>
> diff --git a/hw/isa-bus.c b/hw/isa-bus.c
> index 1f64673..6ac3e61 100644
> --- a/hw/isa-bus.c
> +++ b/hw/isa-bus.c
On Thu, Jun 09, 2011 at 10:56:52AM +0900, Isaku Yamahata wrote:
> Ping?
I put this on my tree, it will get in with the next merge,
hope to send that next week.
--
MST
Andreas Färber writes:
> To allow enabling/disabling present ISA devices without hotplug,
> keep track of state and add a helper to avoid enabling twice.
> Since the properties to be configured are defined at device level,
> delegate the actual work to a callback function.
>
> If no callback is s
Andreas Färber writes:
> VMState supports the type bool but qdev instead supports bit, backed by
> uint32_t. Therefore let's add DEFINE_PROP_BOOL() and qdev_prop_set_bool().
>
> Since, e.g., enabled=on does not look nice, parse/print "yes" and "no".
The difference between bool and bit properties
On 9 June 2011 14:59, Markus Armbruster wrote:
> The fact that we duplicate qtree information in a separate list "busses"
> either means we've been too lazy to garbage collect busses, or we've
> failed to make working with the qtree as easy as it should be.
If you get rid of busses you need to up
On Thu, Jun 09, 2011 at 09:39:56AM +0200, Jan Kiszka wrote:
> From: Jan Kiszka
>
> This also cleans up an open-coded 64-bit message address readout.
>
> Signed-off-by: Jan Kiszka
Thanks, applied.
> ---
>
> Depends on the pci_regs.h update.
>
> hw/msix.c | 40 +++--
On 06/09/2011 04:07 PM, Kevin Wolf wrote:
I think you're right. We could enable it unconditionally (and change the
bm->status line from = to |=), but anyway it's redundant if the
subsections are present, so it wouldn't make a difference.
Yes, I was thinking the same.
Paolo
On Thu, Jun 09, 2011 at 09:18:30AM +0200, Jan Kiszka wrote:
> From: Jan Kiszka
>
> Pulls in latest version from kernel 3.0-rc2.
>
> Some changes around AER now require local defines as QEMU accesses the
> error source identification register via sub-words.
>
> CC: Isaku Yamahata
> Signed-off-b
Hi,
In that case it won't work (out-of-sync) and we shouldn't introduce
VMState for pc87312 at all imo. In theory we'd probably need a
pc87312-owned bus to put the devices on but then I don't see how to
reuse the existing isa devices.
Oh, that should actually work just fine and is maybe the
Hi,
Gave up after resending it one or two times, the forgot about it,
wasn't *that* important to me.
Still got a pointer to the patch?
Not in my git tree any more. Google found me this instead:
http://permalink.gmane.org/gmane.comp.emulators.qemu/74360
Looks like I'm not the only one wh
Hi,
Am 09.06.2011 um 14:23 schrieb Gerd Hoffmann:
I get the feeling that doing all this in the pc87312 emulation is
easier as it needs to have this logic anyway for config register
writes and you can probably reuse the code for loadvm pre- and
postprocessing.
Well, I wasn't looking for the ea
Am 09.06.2011 15:27, schrieb Paolo Bonzini:
> On 06/09/2011 03:15 PM, Kevin Wolf wrote:
>> +/* This function accesses bm->bus->error_status which is loaded only after
>> + * BMDMA itself. This is why the function is called from ide_pci_post_load
>> + * instead of being registered with VMState where
Gerd Hoffmann writes:
> Hi,
>
> Peter Maydell writes:
>
>> If you have a model with more than one USB bus, and you
>> create a USB device on the command line without specifying
>> which bus to plug it into, QEMU will choose a different bus
>> depending on whether you use the legacy "-usbdevic
Signed-off-by: Kevin Wolf
---
hw/ide/core.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/hw/ide/core.c b/hw/ide/core.c
index e5def8b..399b74c 100644
--- a/hw/ide/core.c
+++ b/hw/ide/core.c
@@ -1864,6 +1864,7 @@ const VMStateDescription vmstate_ide_atapi_gesn_state = {
On 06/09/2011 03:15 PM, Kevin Wolf wrote:
+/* This function accesses bm->bus->error_status which is loaded only after
+ * BMDMA itself. This is why the function is called from ide_pci_post_load
+ * instead of being registered with VMState where it would run too early. */
+static int ide_bmdma_pos
When a failed PIO request caused the VM to stop, we still need to transfer the
PIO state even though DRQ=0 at this point.
Signed-off-by: Kevin Wolf
---
hw/ide/core.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/hw/ide/core.c b/hw/ide/core.c
index da250ac..e5def8b 10
When adding the werror=stop mode, some flags were added to s->status
which are used to determine what kind of operation should be restarted
when the VM is continued.
Unfortunately, it turns out that s->status is in fact a device register
and as such is visible to the guest (some of the abused bits
On 06/09/2011 02:48 PM, Kevin Wolf wrote:
This means that subsections are silently ignored if the old version
didn't have any subsection for the device. Wasn't it a fundamental point
of the design of subsections that this would fail?
Unfortunately you cannot have it both ways because the migrat
Patch 1 describes the real problem that I was going to solve. While fixing
this, I stumbled across some more migration problems that are included in this
series. Some refactoring is still left to do after this, but I'd prefer to
keep this series with the hard stuff short.
This is the test case tha
Am 24.01.2011 10:30, schrieb Yoshiaki Tamura:
> Although it's rare to happen in live migration, when the head of a
> byte stream contains 0x05 which is the marker of subsection, the
> loader gets corrupted because vmstate_subsection_load() continues even
> the device doesn't require it. This patch
On 06/06/2011 07:49 AM, Gerd Hoffmann wrote:
Hi,
Resending pull request for the spice patch queue. Almost nothing
changed, the queue gained one additional fix from Alon and was rebased
to latest master.
Pulled. Thanks.
Regards,
Anthony Liguori
please pull,
Gerd
The following chan
On 06/01/2011 10:54 AM, Luiz Capitulino wrote:
Anthony,
The following patches have been sent to the list and look good to me. I've
also tested them.
The changes (since 578c7b2ca8ee9e97fa8693b1a83d517e8e3f962e) are available
in the following repository:
git://repo.or.cz/qemu/qmp-unstable.g
On 06/08/2011 03:24 AM, Stefan Hajnoczi wrote:
The following changes since commit a90d4690074526f54ad0851fce19fa6783f06803:
Add an isa device for SGA (2011-06-07 13:52:30 -0500)
are available in the git repository at:
git://repo.or.cz/qemu/stefanha.git trivial-patches
Pulled. Thanks.
On 06/01/2011 12:33 PM, Venkateswararao Jujjuri wrote:
The following changes since commit
578c7b2ca8ee9e97fa8693b1a83d517e8e3f962e:
Juha Riihim?ki (1):
audio: fix integer overflow expression
are available in the git repository at:
git://repo.or.cz/qemu/aliguori/jvrao.git for-anthony
Pulled.
On 06/08/2011 08:48 AM, Kevin Wolf wrote:
The following changes since commit a90d4690074526f54ad0851fce19fa6783f06803:
Add an isa device for SGA (2011-06-07 13:52:30 -0500)
are available in the git repository at:
git://repo.or.cz/qemu/kevin.git for-anthony
Pulled. Thanks.
Regards,
An
On 06/06/2011 07:49 AM, Gerd Hoffmann wrote:
Hi,
Resending pull request for the spice patch queue. Almost nothing
changed, the queue gained one additional fix from Alon and was rebased
to latest master.
Pulled. Thanks.
Regards,
Anthony Liguori
please pull,
Gerd
The following chan
Am 09.06.2011 um 00:55 schrieb Alexander Graf:
Qemu uses signalfd to figure out, if a signal occured without the need
to actually receive the signal. Instead, it can read from the fd to
receive
its news.
Now, we obviously don't always have signalfd around. Especially not on
non-Linux systems
Hi,
Btw is 1 correct here or should an initial version be 0?
I think for sub-structs it doesn't matter at all, only for top-level
vmstates.
I get the feeling that doing all this in the pc87312 emulation is
easier as it needs to have this logic anyway for config register
writes and you ca
Hi,
Am 09.06.2011 um 12:39 schrieb Gerd Hoffmann:
+const VMStateDescription vmstate_isa_device = {
+.name = "ISADevice",
+.version_id = 1,
+.minimum_version_id = 1,
+.fields = (VMStateField []) {
+VMSTATE_BOOL(enabled, ISADevice),
+VMSTATE_END_OF_LIST()
+}
+}
Added file tcg/optimize.c to hold TCG optimizations. Function tcg_optimize
is called from tcg_gen_code_common. It calls other functions performing
specific optimizations. Stub for constant folding was added.
Signed-off-by: Kirill Batuzov
---
Makefile.target |2 +-
tcg/optimize.c | 91
A timer that wakes up every millisecond puts a lot of stress on the
iothread. The large amount of IPIs causes very high context switch
activity, making emulation slow and the UI unusable. This is by the
way the same reason why the Windows timers were switched to dynticks.
Signed-off-by: Paolo Bo
This conveys the intention better, and scales to more than >1
threads contending the mutex with the iothread (as long as all
of them have a "quiescent point" like the TCG thread has).
Also, on Mac OS X the fair_mutex somehow didn't work as intended
and deadlocked.
Signed-off-by: Paolo Bonzini
Te
These are two old patches that I never submitted because I didn't really
think they were useful except as cleanups. Recently, however, Alex Graf
mentioned some problems that Mac OS X has with iothread, and they sounded
to me like they were related to these patches. And quite surprisingly,
both of
This series implements some basic machine-independent optimizations. They
simplify code and allow liveness analysis do it's work better.
Suppose we have following ARM code:
movwr12, #0xb6db
movtr12, #0xdb6d
In TCG before optimizations we'll have:
movi_i32 tmp8,$0xb6db
mov_i32 r12,t
On Thu, Jun 9, 2011 at 11:20 AM, Fam Zheng wrote:
> On Thu, Jun 9, 2011 at 3:58 PM, Stefan Hajnoczi wrote:
>> On Sat, Jun 04, 2011 at 08:40:22AM +0800, Fam Zheng wrote:
>>> fail:
>>> - qemu_free(s->l1_backup_table);
>>> - qemu_free(s->l1_table);
>>> - qemu_free(s->l2_cache);
>>> + i
Perform constant folding for NOT and EXT{8,16,32}{S,U} operations.
Signed-off-by: Kirill Batuzov
---
tcg/optimize.c | 83
1 files changed, 83 insertions(+), 0 deletions(-)
diff --git a/tcg/optimize.c b/tcg/optimize.c
index 653f399..2cdc
pcie_aer.c change looks good.
Thanks.
On Thu, Jun 09, 2011 at 09:18:30AM +0200, Jan Kiszka wrote:
> From: Jan Kiszka
>
> Pulls in latest version from kernel 3.0-rc2.
>
> Some changes around AER now require local defines as QEMU accesses the
> error source identification register via sub-words.
1 - 100 of 124 matches
Mail list logo