On 10 May 2012 23:40, Andreas Färber wrote:
> Am 10.05.2012 23:41, schrieb Peter Maydell:
>> On 10 May 2012 01:14, Andreas Färber wrote:
>>> Allows us to use cpu_reset() in place of cpu_state_reset().
>>>
>>> Signed-off-by: Andreas Färber
>>> ---
>>> hw/arm_boot.c | 9 ++---
>>> 1 files
Legacy (non-pvops) gntdev drivers may require this to be done when the
number of grants intended to be used simultaneously exceeds a certain
driver specific default limit.
Signed-off-by: Jan Beulich
--- a/hw/xen_disk.c
+++ b/hw/xen_disk.c
@@ -536,6 +536,10 @@ static void blk_alloc(struct XenDevi
Whitelist kvm pv eoi feature. The feature is enabled
with -cpu kvm64. To disable: -cpu kvm64,-kvm_eoi.
Signed-off-by: Michael S. Tsirkin
---
Sending a copy to kernel list as this is needed
to test the pv eoi feature recently submitted.
target-i386/cpuid.c |2 +-
1 files changed, 1 insertio
Normally the pci_add_capability is called on devices to add new
capability. This is ok for emulated devices which capabilities list
is being built by QEMU.
In the case of VFIO the capability may already exist and adding new
capability into the beginning of the linked list may create a loop.
For e
Normally the pci_add_capability is called on devices to add new
capability. This is ok for emulated devices which capabilities list
is being built by QEMU.
In the case of VFIO the capability may already exist and adding new
capability into the beginning of the linked list may create a loop.
For e
On 05/10/12 22:08, Jan Kiszka wrote:
> Call msi_reset on device reset as still required by the core.
Note: msi on xhci is disabled by default (and also broken as far I know).
> +static void xhci_reset(void *opaque)
> +{
> +XHCIState *xhci = opaque;
> +
if (xhci->msi)
> +msi_reset(&xhci-
On Fri, May 11, 2012 at 01:46:17AM +0800, Jiang Liu wrote:
> On 05/11/2012 01:42 AM, Michael S. Tsirkin wrote:
> > On Fri, May 11, 2012 at 01:17:38AM +0800, Jiang Liu wrote:
> >> On 05/09/2012 03:24 PM, Amos Kong wrote:
> >>
> >>> ---
> >>> src/ssdt-pcihp.dsl | 17
> >>> src/ssdt-pcihp.hex | 88
List, Kevin
Please find a small patch to the iscsi driver to only set the fd handler for
read events IF we have i/o in flight and we are thus waiting for replies coming
back from the target.
There are situations where qemu_aio_set_fd_handler(fd, read_callback, ...
will invoke the read_callback
Signed-off-by: Ronnie Sahlberg
---
block/iscsi.c |4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
diff --git a/block/iscsi.c b/block/iscsi.c
index d37c4ee..989b5e9 100644
--- a/block/iscsi.c
+++ b/block/iscsi.c
@@ -105,7 +105,9 @@ iscsi_set_events(IscsiLun *iscsilun)
{
struct
On 11.05.2012, at 08:45, Alexey Kardashevskiy wrote:
> Normally the pci_add_capability is called on devices to add new
> capability. This is ok for emulated devices which capabilities list
> is being built by QEMU.
>
> In the case of VFIO the capability may already exist and adding new
> capabil
Am 06.05.2012 10:30, schrieb Stefan Weil:
> Am 05.05.2012 13:40, schrieb Andreas Färber:
>> There is no function cpu_mips_get_clock(), so drop it.
>>
>> Signed-off-by: Andreas Färber
>> Cc: Stefan Weil
>> ---
>> target-mips/cpu.h |1 -
>> 1 files changed, 0 insertions(+), 1 deletions(-)
>>
>
On 10 May 2012 01:14, Andreas Färber wrote:
> Also use cpu_reset() in place of cpu_state_reset().
>
> Signed-off-by: Andreas Färber
The pxa2xx stuff is probably going to clash with the cp15
rework, but I guess we'll sort that out when one or the
other of these series hits master.
Acked-by: Pete
On 10 May 2012 01:14, Andreas Färber wrote:
> Needed for armv7m_reset().
>
> Signed-off-by: Andreas Färber
Acked-by: Peter Maydell
I'd have preferred it if you hadn't made the style change
from "if (!foo)" to "if (foo == NULL)", incidentally, but I'm
not going to insist on a respin for it.
--
On 10 May 2012 01:14, Andreas Färber wrote:
> Allows us to use cpu_reset() in place of cpu_state_reset().
>
> Signed-off-by: Andreas Färber
Acked-by: Peter Maydell
-- PMM
On 10 May 2012 01:14, Andreas Färber wrote:
> Eliminates cpu_state_reset() usage.
>
> Signed-off-by: Andreas Färber
> ---
> linux-user/main.c | 2 +-
> linux-user/syscall.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/linux-user/main.c b/linux-user/main.c
Am 11.05.2012 13:16, schrieb Peter Maydell:
> On 10 May 2012 01:14, Andreas Färber wrote:
>> Also use cpu_reset() in place of cpu_state_reset().
>>
>> Signed-off-by: Andreas Färber
>
> The pxa2xx stuff is probably going to clash with the cp15
> rework, but I guess we'll sort that out when one or
On 10 May 2012 01:13, Andreas Färber wrote:
> Commit 3c30dd5a68e9fee6af67cfd0d14ed7520820f36a (target-arm: Move reset
> handling to arm_cpu_reset) QOM'ified CPU reset. Complete it by replacing
> cpu_state_reset() with cpu_reset().
>
> Signed-off-by: Andreas Färber
Acked-by: Peter Maydell
-- PM
Am 11.05.2012 13:22, schrieb Peter Maydell:
> On 10 May 2012 01:14, Andreas Färber wrote:
>> Eliminates cpu_state_reset() usage.
>>
>> Signed-off-by: Andreas Färber
>> ---
>> linux-user/main.c|2 +-
>> linux-user/syscall.c |2 +-
>> 2 files changed, 2 insertions(+), 2 deletions(-)
>>
Il 11/05/2012 12:22, Ronnie Sahlberg ha scritto:
> Signed-off-by: Ronnie Sahlberg
> ---
> block/iscsi.c |4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
> diff --git a/block/iscsi.c b/block/iscsi.c
> index d37c4ee..989b5e9 100644
> --- a/block/iscsi.c
> +++ b/block/iscsi.c
> @@
On 11.05.2012, at 13:26, Andreas Färber wrote:
> Am 11.05.2012 13:22, schrieb Peter Maydell:
>> On 10 May 2012 01:14, Andreas Färber wrote:
>>> Eliminates cpu_state_reset() usage.
>>>
>>> Signed-off-by: Andreas Färber
>>> ---
>>> linux-user/main.c|2 +-
>>> linux-user/syscall.c |2 +
Il 10/05/2012 22:58, Jan Kiszka ha scritto:
>>> >> Otherwise, non-string properties without a legacy counterpart are missed.
>>> >> Also fix error propagation in object_property_print itself, otherwise
>>> >> pointer properties are printed as "".
>>> >>
>>> >> Signed-off-by: Paolo Bonzini
>> >
>>
Am 10.05.2012 02:14, schrieb Andreas Färber:
> Needed for main_cpu_reset().
>
> Signed-off-by: Andreas Färber
Got to forget something:
Acked-by: Hervé Poussineau
Depends on the target-mips patch though, so need a review of that from
someone first.
Andreas
--
SUSE LINUX Products GmbH, Maxfe
Am 10.05.2012 02:14, schrieb Andreas Färber:
> Allows us to use cpu_reset() in place of cpu_state_reset().
>
> Signed-off-by: Andreas Färber
Forgot:
Acked-by: Hervé Poussineau
/-F
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendö
On 11 May 2012 12:28, Alexander Graf wrote:
>
> On 11.05.2012, at 13:26, Andreas Färber wrote:
>
>> Am 11.05.2012 13:22, schrieb Peter Maydell:
>>> On 10 May 2012 01:14, Andreas Färber wrote:
Eliminates cpu_state_reset() usage.
Signed-off-by: Andreas Färber
---
linux-use
Am 11.05.2012 13:28, schrieb Alexander Graf:
>
> On 11.05.2012, at 13:26, Andreas Färber wrote:
>
>> Am 11.05.2012 13:22, schrieb Peter Maydell:
>>> On 10 May 2012 01:14, Andreas Färber wrote:
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 20d2a74..539af3f 100644
-
Am 11.05.2012 13:28, schrieb Paolo Bonzini:
> Il 10/05/2012 22:58, Jan Kiszka ha scritto:
>> Otherwise, non-string properties without a legacy counterpart are missed.
>> Also fix error propagation in object_property_print itself, otherwise
>> pointer properties are printed as "".
>>
Am 10.05.2012 21:14, schrieb Igor Mammedov:
> - Original Message -
>> From: "Andreas Färber"
>> To: qemu-devel@nongnu.org
>> Cc: "Eduardo Habkost" , "Michael Roth"
>> , "Anthony Liguori"
>> , "Paolo Bonzini" ,
>> imamm...@redhat.com, "Andreas Färber"
>>
>> Sent: Thursday, May 10, 2012 1
Am 11.05.2012 13:24, schrieb Peter Maydell:
> On 10 May 2012 01:13, Andreas Färber wrote:
>> Commit 3c30dd5a68e9fee6af67cfd0d14ed7520820f36a (target-arm: Move reset
>> handling to arm_cpu_reset) QOM'ified CPU reset. Complete it by replacing
>> cpu_state_reset() with cpu_reset().
>>
>> Signed-off-b
On Fri, May 11, 2012 at 9:27 PM, Paolo Bonzini wrote:
> Il 11/05/2012 12:22, Ronnie Sahlberg ha scritto:
>> Signed-off-by: Ronnie Sahlberg
>> ---
>> block/iscsi.c | 4 +++-
>> 1 files changed, 3 insertions(+), 1 deletions(-)
>>
>> diff --git a/block/iscsi.c b/block/iscsi.c
>> index d37c4ee..9
Am 11.05.2012 13:16, schrieb Peter Maydell:
> On 10 May 2012 01:14, Andreas Färber wrote:
>> Also use cpu_reset() in place of cpu_state_reset().
>>
>> Signed-off-by: Andreas Färber
>
> The pxa2xx stuff is probably going to clash with the cp15
> rework, but I guess we'll sort that out when one or
[adding libvirt]
On 05/11/2012 12:46 AM, Amos Kong wrote:
> On 05/11/2012 02:12 AM, Michael Roth wrote:
>> On Thu, May 10, 2012 at 11:29:48AM -0600, Eric Blake wrote:
>>> On 05/10/2012 10:27 AM, Amos Kong wrote:
Those patches updated help functions in qemu-socket.c,
and used them in migr
11.05.2012 20:52, Alexander Graf написал:
>
> On 11.05.2012, at 08:45, Alexey Kardashevskiy wrote:
>
>> Normally the pci_add_capability is called on devices to add new
>> capability. This is ok for emulated devices which capabilities list
>> is being built by QEMU.
>>
>> In the case of VFIO the c
[adding libvirt, dropping kernel]
On 05/11/2012 01:43 AM, Michael S. Tsirkin wrote:
> Whitelist kvm pv eoi feature. The feature is enabled
> with -cpu kvm64. To disable: -cpu kvm64,-kvm_eoi.
>
> Signed-off-by: Michael S. Tsirkin
> ---
>
> Sending a copy to kernel list as this is needed
> to tes
Am 10.05.2012 23:38, schrieb Peter Maydell:
> On 10 May 2012 01:14, Andreas Färber wrote:
>> Fix tab indentations of comments, add braces, use cpu_reset().
>>
>> Signed-off-by: Andreas Färber
>> ---
>> hw/nseries.c |6 +++---
>> hw/omap.h |2 +-
>> hw/omap1.c| 20 +++--
On 2012-05-11 05:16, Gerd Hoffmann wrote:
> On 05/10/12 22:08, Jan Kiszka wrote:
>> Call msi_reset on device reset as still required by the core.
>
> Note: msi on xhci is disabled by default (and also broken as far I know).
OK, then we can likely skip this patch for 1.1/stable.
>
>> +static voi
Hi,
>> And can't we let the pci core handle it so we don't need ugly wrappers
>> like this?
>
> That's what patches later in the series do. But Michael was preferring
> this approach for 1.1 and the cleanup for 1.2.
Ah, ok, good. Yea, lets leave it alone for 1.1 and fix it properly in 1.2
ch
Am 11.05.2012 13:18, schrieb Peter Maydell:
> On 10 May 2012 01:14, Andreas Färber wrote:
>> Needed for armv7m_reset().
>>
>> Signed-off-by: Andreas Färber
>
> Acked-by: Peter Maydell
Thanks, applied to qom-next:
http://repo.or.cz/w/qemu/afaerber.git/shortlog/refs/heads/qom-next
> I'd have pr
I just noticed it didn't get into -rc1 either. So, this is definitely
not going to be in qemu-1.1, I guess?
On Wed, May 02, 2012 at 01:07:24PM -0300, Eduardo Habkost wrote:
> Changes v2 -> v3:
> - Actually change 'defconfig' type declaration to bool
> - Rebase against latest qemu.git (commit
Am 11.05.2012 13:19, schrieb Peter Maydell:
> On 10 May 2012 01:14, Andreas Färber wrote:
>> Allows us to use cpu_reset() in place of cpu_state_reset().
>>
>> Signed-off-by: Andreas Färber
>
> Acked-by: Peter Maydell
Thanks, applied to qom-next:
http://repo.or.cz/w/qemu/afaerber.git/shortlog/r
On 05/11/2012 08:34 AM, Eduardo Habkost wrote:
I just noticed it didn't get into -rc1 either. So, this is definitely
not going to be in qemu-1.1, I guess?
I've got this applied and am testing it right now for -rc2.
Regards,
Anthony Liguori
On Wed, May 02, 2012 at 01:07:24PM -0300, Eduard
Am 11.05.2012 09:17, schrieb Peter Maydell:
> On 10 May 2012 23:40, Andreas Färber wrote:
>> Am 10.05.2012 23:41, schrieb Peter Maydell:
>>> On 10 May 2012 01:14, Andreas Färber wrote:
Allows us to use cpu_reset() in place of cpu_state_reset().
Signed-off-by: Andreas Färber
-
On Fri, May 11, 2012 at 08:37:33AM -0500, Anthony Liguori wrote:
> On 05/11/2012 08:34 AM, Eduardo Habkost wrote:
> >
> >I just noticed it didn't get into -rc1 either. So, this is definitely
> >not going to be in qemu-1.1, I guess?
>
> I've got this applied and am testing it right now for -rc2.
A
On Fri, May 11, 2012 at 08:37:33 -0500, Anthony Liguori wrote:
> On 05/11/2012 08:34 AM, Eduardo Habkost wrote:
> >
> > I just noticed it didn't get into -rc1 either. So, this is definitely
> > not going to be in qemu-1.1, I guess?
>
> I've got this applied and am testing it right now for -rc2.
O
On 06.11.2011, at 14:54, Jan Kiszka wrote:
> On 2011-08-24 23:38, Alexander Graf wrote:
>> On LinuxCon I had a nice chat with Linus on what he thinks kvm-tool
>> would be doing and what he expects from it. Basically he wants a
>> small and simple tool he and other developers can run to try out an
On 05/11/2012 08:24 AM, Amos Kong wrote:
> On 05/11/2012 07:54 AM, Amos Kong wrote:
>> On 05/11/2012 02:55 AM, Michael S. Tsirkin wrote:
>>> On Fri, May 11, 2012 at 01:09:13AM +0800, Jiang Liu wrote:
On 05/10/2012 11:44 PM, Amos Kong wrote:
> diff --git a/drivers/pci/hotplug/acpiphp_g
Am 02.05.2012 13:30, schrieb Paolo Bonzini:
> Since most property types do not have a parse property now, this was
> broken. Fix it by looking at the setter instead.
>
> Signed-off-by: Paolo Bonzini
Reviewed-by: Anthony Liguori
Acked-by: Andreas Färber
I'm guessing Anthony has this one in hi
On 2012-05-11 10:42, Alexander Graf wrote:
>
> On 06.11.2011, at 14:54, Jan Kiszka wrote:
>
>> On 2011-08-24 23:38, Alexander Graf wrote:
>>> On LinuxCon I had a nice chat with Linus on what he thinks kvm-tool
>>> would be doing and what he expects from it. Basically he wants a
>>> small and simp
Am 03.04.2012 15:05, schrieb Paolo Bonzini:
> Il 03/04/2012 14:28, Jan Kiszka ha scritto:
if (object_property_get_type(OBJECT(dev), legacy_name, NULL)) {
value = object_property_get_str(OBJECT(dev), legacy_name,
&err);
>> This is not a criticism on this patch,
On 11.05.2012, at 14:47, Alexey Kardashevskiy wrote:
> 11.05.2012 20:52, Alexander Graf написал:
>>
>> On 11.05.2012, at 08:45, Alexey Kardashevskiy wrote:
>>
>>> Normally the pci_add_capability is called on devices to add new
>>> capability. This is ok for emulated devices which capabilities l
>>> On 11.05.12 at 09:19, "Jan Beulich" wrote:
> Legacy (non-pvops) gntdev drivers may require this to be done when the
> number of grants intended to be used simultaneously exceeds a certain
> driver specific default limit.
>
> Signed-off-by: Jan Beulich
>
> --- a/hw/xen_disk.c
> +++ b/hw/xen_
Am 02.05.2012 13:31, schrieb Paolo Bonzini:
> Otherwise, non-string properties without a legacy counterpart are missed.
> Also fix error propagation in object_property_print itself.
>
> Signed-off-by: Paolo Bonzini
> ---
> hw/qdev-monitor.c |2 +-
> qom/object.c |2 +-
> 2 files cha
Signed-off-by: Kevin Wolf
---
block/qcow2.c |5 -
1 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/block/qcow2.c b/block/qcow2.c
index 3bae2d8..655799c 100644
--- a/block/qcow2.c
+++ b/block/qcow2.c
@@ -300,7 +300,10 @@ static int qcow2_open(BlockDriverState *bs, int flags)
Properly register reset function via the device class.
Signed-off-by: Jan Kiszka
---
Broken out from the MSI series where I will no longer touch xhci.
hw/usb/hcd-xhci.c |9 -
1 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/hw/usb/hcd-xhci.c b/hw/usb/hcd-xhci.c
index
v3 is v2 - xhci changes as MSI is instable there anyway. So, only
patches 1 and 2 are for 1.1/stable now.
CC: Alexander Graf
CC: Gerd Hoffmann
CC: Isaku Yamahata
CC: qemu-sta...@nongnu.org
Jan Kiszka (8):
ahci: Fix reset of MSI function
intel-hda: Fix reset of MSI function
ahci: Clean up
Call msi_reset on device reset as still required by the core.
CC: Gerd Hoffmann
CC: qemu-sta...@nongnu.org
Signed-off-by: Jan Kiszka
---
hw/intel-hda.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/hw/intel-hda.c b/hw/intel-hda.c
index bb11af2..e38861e 100644
--- a/
Replace some open-coded msi/msix_present checks.
Signed-off-by: Jan Kiszka
---
hw/msi.c |2 +-
hw/msix.c | 13 -
2 files changed, 9 insertions(+), 6 deletions(-)
diff --git a/hw/msi.c b/hw/msi.c
index 556c7c4..5233204 100644
--- a/hw/msi.c
+++ b/hw/msi.c
@@ -175,7 +175,7 @@ v
Il 11/05/2012 16:20, Andreas Färber ha scritto:
> char *string = NULL;
>
>> >
>> > mo = string_output_visitor_new();
>> > -object_property_get(obj, string_output_get_visitor(mo), name, NULL);
>> > +object_property_get(obj, string_output_get_visitor(mo), name, errp);
> If we do error
On 05/11/2012 02:42 PM, Paolo Bonzini wrote:
> Il 11/05/2012 04:15, Amos Kong ha scritto:
>> Start VM with 8 multiple-function block devs, hot-removing
>> those block devs by 'device_del ...' would cause qemu abort.
>>
>> object_ref() is called in object_property_add_child(),
>> but we don't unref
The whole PCI slot should be removed once. Currently only one func
is cleaned in pci_unplug_device(), if you try to remove a single
func by monitor cmd.
Start VM with 8 multiple-function block devs, hot-removing
those block devs by 'device_del ...' would cause qemu abort.
| (qemu) device_del virt
On 05/11/12 16:36, Jan Kiszka wrote:
> Properly register reset function via the device class.
Patch added to usb patch queue.
thanks,
Gerd
Am 12.04.2012 18:03, schrieb Andreas Färber:
> Am 11.04.2012 23:30, schrieb Paolo Bonzini:
>> A utility function that will be used to implement hierarchical realization.
>>
>> Signed-off-by: Paolo Bonzini
Reviewed-by: Anthony Liguori
>> ---
>> include/qemu/object.h | 14 +-
>> qo
On 05/08/12 14:06, Gerd Hoffmann wrote:
> Most important here is to update our internal endpoint state so we know
> the endpoint isn't in halted state any more. Without this usb-host
> tries to clear halt again with the next data transfer submitted. Doing
> this twice is (a) not correct and (b) c
There is no point in pushing this burden to the devices, they may rather
forget to call them (like intel-hda, ahci, xhci did). Instead, reset
functions are now called from pci_device_reset and pci_bridge_reset.
They do nothing if MSI/MSI-X is not in use.
CC: Alexander Graf
CC: Gerd Hoffmann
CC:
On Fri, May 11, 2012 at 03:22:15AM +0200, Andreas Färber wrote:
> Am 27.04.2012 22:21, schrieb Michael Roth:
> > These patches apply on top of qemu.git master, and can also be obtained
> > from:
> > git://github.com/mdroth/qemu.git visitor-fixed-width-v5
> >
> > Some of these were being carried a
This is probaly more about QOM than about floppy. It's not a working
solution, yet. I'm posting it to give us something concrete to
discuss.
Based on Kevin's block branch, commit ad431215.
Markus Armbruster (2):
Un-inline fdctrl_init_isa()
Split fdd devices off the floppy controller
hw/fd
Signed-off-by: Markus Armbruster
---
hw/fdc.c | 20
hw/fdc.h | 24 ++--
hw/ide/piix.c |3 ++-
hw/isa.h |2 --
hw/pc_sysfw.c |1 +
qemu-common.h |1 +
6 files changed, 26 insertions(+), 25 deletions(-)
diff --git a/hw/fd
For historical reasons, and unlike other block devices, our floppy
devices isa-fdc, sysbus-fdc and SUNW,fdtwo integrate the controller
and the drive(s) in a single qdev. This makes them weird: we need
-global to set up floppy drives, unlike every other optional device.
This patch explores separat
Forgot to cc: Paolo and Mike, apologies!
Properly register reset functions via the device class.
CC: Alexander Graf
Signed-off-by: Jan Kiszka
---
hw/ide/ahci.c | 25 +++--
hw/ide/ahci.h |2 +-
hw/ide/ich.c | 10 --
3 files changed, 20 insertions(+), 17 deletions(-)
diff --git a/hw/ide/ahci.c b/hw/
Terminate msi/msix_write_config early if support is not enabled. This
allows to remove checks at the caller site if MSI is optional.
Signed-off-by: Jan Kiszka
---
hw/msi.c |3 ++-
hw/msix.c |2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/hw/msi.c b/hw/msi.c
index b
Call msi_reset on device reset as still required by the core.
CC: Alexander Graf
CC: qemu-sta...@nongnu.org
Signed-off-by: Jan Kiszka
---
hw/ide/ich.c | 12 ++--
1 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/hw/ide/ich.c b/hw/ide/ich.c
index 560ae37..242254e 100644
-
Signed-off-by: Jan Kiszka
---
hw/msi.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/hw/msi.c b/hw/msi.c
index 5d6ceb6..b2903fc 100644
--- a/hw/msi.c
+++ b/hw/msi.c
@@ -191,6 +191,10 @@ void msi_reset(PCIDevice *dev)
uint16_t flags;
bool msi64bit;
+if
Also this functions is better invoked by the core than by each and every
device. This allows to drop the config_write callbacks from ich and
intel-hda.
CC: Alexander Graf
CC: Gerd Hoffmann
CC: Isaku Yamahata
Signed-off-by: Jan Kiszka
---
hw/ide/ich.c|8
hw/intel-hda.c
On 05/11/2012 06:14 PM, Gleb Natapov wrote:
>> I'm not familiar with qemu:(
>> On native OS, admin could trigger PCI device hotplug operations through
>> /sys/bus/pci/slot/xx/power. Not sure whether that's needed for guest OS too.
>>
> Why is it needed on physical HW? May be it is needed in a VM f
On 7 May 2012 12:38, Alexander Graf wrote:
>
> On 07.05.2012, at 13:32, Alexander Graf wrote:
>
>>
>> On 07.05.2012, at 12:37, Peter Maydell wrote:
>>
>>> On 7 May 2012 10:30, Alexander Graf wrote:
@@ -587,6 +587,7 @@ abi_long target_mmap(abi_ulong start, abi_ulong len,
int prot,
On LinuxCon I had a nice chat with Linus on what he thinks kvm-tool
would be doing and what he expects from it. Basically he wants a
small and simple tool he and other developers can run to try out and
see if the kernel they just built actually works.
Fortunately, QEMU can do that today already! T
On Thu, 10 May 2012, Jan Beulich wrote:
> While for the "normal" case (called from blk_send_response_all())
> decrementing requests_finished is correct, doing so in the parse error
> case is wrong; requests_inflight needs to be decremented instead.
>
> Signed-off-by: Jan Beulich
>
> --- a/hw/xen
If we execute linux-user code that does the following:
* A = mmap()
* execute code in A
* munmap(A)
* B = mmap(), but mmap returns the same address as A
* execute code in B
we end up executing a stale cached tb that contains translated code
from A, while we want new code from B.
This p
On 11.05.2012, at 17:46, Peter Maydell wrote:
> On 7 May 2012 12:38, Alexander Graf wrote:
>>
>> On 07.05.2012, at 13:32, Alexander Graf wrote:
>>
>>>
>>> On 07.05.2012, at 12:37, Peter Maydell wrote:
>>>
On 7 May 2012 10:30, Alexander Graf wrote:
> @@ -587,6 +587,7 @@ abi_long tar
On Fri, 27 Apr 2012 15:21:18 -0500
Michael Roth wrote:
> JSON numbers can be interpreted as either integers or floating point
> values depending on their representation. As a result, QMP input visitor
> might visit a QInt when it was expecting a QFloat, so add handling to
> account for this.
>
>
On Thu, 10 May 2012, Jan Kiszka wrote:
> Push msi_supported enabling to the APIC implementations where we can
> encapsulate the decision more cleanly, hiding the details from the
> generic code.
>
> CC: Stefano Stabellini
> Signed-off-by: Jan Kiszka
This patch is missing the KVM part, but assum
On 11 May 2012 09:40, Alexander Graf wrote:
> If we execute linux-user code that does the following:
>
> * A = mmap()
> * execute code in A
> * munmap(A)
> * B = mmap(), but mmap returns the same address as A
> * execute code in B
>
> we end up executing a stale cached tb that contains transl
> Make the include paths for cpu-qom.h consistent to allow using LM32CPU
> in cpu.h.
>
> Turn cpu_init macro into a static inline function returning CPULM32State
> for backwards compatibility.
>
> Signed-off-by: Andreas Färber
Acked-by: Michael Walle
--
Michael
On Fri, 27 Apr 2012 15:21:20 -0500
Michael Roth wrote:
> Currently string-output-visitor formats floats as %g, which is nice in
> that trailing 0's are automatically truncated, but otherwise this causes
> some issues:
>
> - it 6 uses significant figures instead of 6 decimal places, which
>m
On Fri, May 11, 2012 at 01:22:33PM -0300, Luiz Capitulino wrote:
> On Fri, 27 Apr 2012 15:21:18 -0500
> Michael Roth wrote:
>
> > JSON numbers can be interpreted as either integers or floating point
> > values depending on their representation. As a result, QMP input visitor
> > might visit a QIn
On Fri, 11 May 2012, Jan Beulich wrote:
> >>> On 11.05.12 at 09:19, "Jan Beulich" wrote:
> > Legacy (non-pvops) gntdev drivers may require this to be done when the
> > number of grants intended to be used simultaneously exceeds a certain
> > driver specific default limit.
> >
> > Signed-off-by: J
Am 11.05.2012 19:04, schrieb Michael Roth:
> On Fri, May 11, 2012 at 01:22:33PM -0300, Luiz Capitulino wrote:
>> On Fri, 27 Apr 2012 15:21:18 -0500
>> Michael Roth wrote:
>>
>>> JSON numbers can be interpreted as either integers or floating point
>>> values depending on their representation. As a
On Fri, May 11, 2012 at 01:34:01PM -0300, Luiz Capitulino wrote:
> On Fri, 27 Apr 2012 15:21:20 -0500
> Michael Roth wrote:
>
> > Currently string-output-visitor formats floats as %g, which is nice in
> > that trailing 0's are automatically truncated, but otherwise this causes
> > some issues:
>
On Fri, May 11, 2012 at 07:16:18PM +0200, Andreas Färber wrote:
> Am 11.05.2012 19:04, schrieb Michael Roth:
> > On Fri, May 11, 2012 at 01:22:33PM -0300, Luiz Capitulino wrote:
> >> On Fri, 27 Apr 2012 15:21:18 -0500
> >> Michael Roth wrote:
> >>
> >>> JSON numbers can be interpreted as either in
JSON numbers can be interpreted as either integers or floating point
values depending on their representation. As a result, QMP input visitor
might visit a QInt when it was expecting a QFloat, so add handling to
account for this.
Signed-off-by: Michael Roth
---
qapi/qmp-input-visitor.c | 11 ++
Am 11.05.2012 19:32, schrieb Michael Roth:
> On Fri, May 11, 2012 at 01:34:01PM -0300, Luiz Capitulino wrote:
>> On Fri, 27 Apr 2012 15:21:20 -0500
>> Michael Roth wrote:
>>
>>> Currently string-output-visitor formats floats as %g, which is nice in
>>> that trailing 0's are automatically truncated
Am 11.05.2012 19:43, schrieb Michael Roth:
> JSON numbers can be interpreted as either integers or floating point
> values depending on their representation. As a result, QMP input visitor
> might visit a QInt when it was expecting a QFloat, so add handling to
> account for this.
>
> Signed-off-by
On Fri, May 11, 2012 at 11:44:02PM +0800, Jiang Liu wrote:
> On 05/11/2012 06:14 PM, Gleb Natapov wrote:
> >> I'm not familiar with qemu:(
> >> On native OS, admin could trigger PCI device hotplug operations through
> >> /sys/bus/pci/slot/xx/power. Not sure whether that's needed for guest OS
> >>
Signed-off-by: Kevin Wolf
---
block/qcow2-refcount.c | 27 +--
block/qcow2.c |6 +-
block/qcow2.h |3 ++-
3 files changed, 28 insertions(+), 8 deletions(-)
diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
index 443c021..6a9a672
On Fri, 11 May 2012 12:04:47 -0500
Michael Roth wrote:
> On Fri, May 11, 2012 at 01:22:33PM -0300, Luiz Capitulino wrote:
> > On Fri, 27 Apr 2012 15:21:18 -0500
> > Michael Roth wrote:
> >
> > > JSON numbers can be interpreted as either integers or floating point
> > > values depending on their
On Fri, May 11, 2012 at 12:37:49PM +0800, Amos Kong wrote:
> good: 3.3.0 guest kernel & qemu-kvm-rhel6
> guest panic: 3.3.0 guest kernel & qemu-upstream (contains fix [1])
>
> I didn't change anything of guest kernel,
> It seems a bug of qemu-upstream.
>
> [1] http://marc.info/?l=qemu-devel&m=13
The QED block driver already provides the functionality to not only
detect inconsistencies in images, but also fix them. However, this
functionality cannot be manually invoked with qemu-img, but the
check happens only automatically during bdrv_open().
This adds a -r switch to qemu-img check that a
A prerequisite for a "QED mode" in qcow2, which doesn't update the refcount
table except on clean shutdown, is that refcounts can be repaired when the
image is opened the next time after a crash.
This series adds a qemu-img check option that doesn't only check, but also
tries to fix the errors tha
When any inconsistencies have been fixed, print the statistics and run
another check to make sure everything is correct now.
Signed-off-by: Kevin Wolf
---
block.h |2 ++
block/qed-check.c |2 ++
qemu-img.c| 10 ++
3 files changed, 14 insertions(+), 0 deletions
On 2012-05-11 13:24, Stefano Stabellini wrote:
> On Thu, 10 May 2012, Jan Kiszka wrote:
>> Push msi_supported enabling to the APIC implementations where we can
>> encapsulate the decision more cleanly, hiding the details from the
>> generic code.
>>
>> CC: Stefano Stabellini
>> Signed-off-by: Jan
1 - 100 of 149 matches
Mail list logo
