v2:
* Fix some coding style
* New patch to separate i440fx_init
* Just add prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough
* Based on patch #2 to regenerate
* Unify prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough like patch #3
* Test: boot with a preinstalled ubuntu 14.04
We'd like to split i440fx_init and then we can share something
with other stuff.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/pci-host/piix.c | 91 --
1 file changed, 75 insertions(+), 16 deletions(-)
v2:
* New patch to separate
We'd like to split pc_init1 and then we can share something
with other stuff.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/i386/pc_piix.c | 117 +++---
1 file changed, 93 insertions(+), 24 deletions(-)
v2:
* Fix some coding style
diff
Implement a pci host bridge specific to passthrough. Actually
this just inherits the standard one.
This is based on http://patchwork.ozlabs.org/patch/363810/.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/pci-host/piix.c | 42 ++
1 file changed,
Now we can introduce a new machine, xenigd, specific to IGD
passthrough. This can avoid involving other common codes.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/i386/pc_piix.c | 105 ++
1 file changed, 105 insertions(+)
v2:
*
This is almost same as an original i440fx_init but just
work with that xen igd host bridge to passthrough.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/pci-host/piix.c | 32
include/hw/i386/pc.h | 11 +++
2 files changed, 43 insertions(+)
Am 17.07.2014 13:03, schrieb Pavel Dovgalyuk:
This patch adds interrupt fields to VMState for correct saving the CPU state.
Signed-off-by: Pavel Dovgalyuk pavel.dovga...@ispras.ru
---
target-arm/machine.c |5 -
target-i386/machine.c |5 -
2 files changed, 8 insertions(+),
Hi,
Am 31.07.2014 07:41, schrieb Pavel Dovgaluk:
This patch adds subsection with exception_index field to the VMState for
correct saving the CPU state.
Without this patch simulator could miss the pending exception in the saved
virtual machine state.
Signed-off-by: Pavel Dovgalyuk
On Tue, 06/17 10:45, Fam Zheng wrote:
On Thu, 06/05 16:47, Fam Zheng wrote:
There is a change in qemu-io sub-commands aio_read and aio_write, which
makes the aio requests accounted and the statistics reflected in blockstats.
Note that IO throttling implementation allows overcommiting of
The Thursday 31 Jul 2014 à 11:55:28 (+0800), Ming Lei wrote :
On Thu, Jul 31, 2014 at 7:37 AM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 30/07/2014 19:15, Ming Lei ha scritto:
On Wed, Jul 30, 2014 at 9:45 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 30/07/2014 13:39, Ming Lei ha
The Wednesday 30 Jul 2014 à 09:53:30 (+0100), Stefan Hajnoczi wrote :
When request A is a strict superset of request B:
multiwrite_merge() merges them as follows:
AA
The tail of request A should have been included:
AAAA
This patch fixes data loss
We have
- wait (wait bit in PSW)
- disabled wait (wait bit and interrupt fencing in PSW)
- STOPPED (not related to PSW, state change usually handled via service
processor or hypervisor)
I think we have to differentiate between KVM/TCG. On KVM we always do in
kernel halt and qemu
The Wednesday 30 Jul 2014 à 09:53:31 (+0100), Stefan Hajnoczi wrote :
This test case covers the basic bdrv_aio_multiwrite() scenarios:
1. Single request
2. Sequential requests (AABB)
3. Superset overlapping requests (AABBAA)
4. Subset overlapping requests (BBAABB)
5. Head overlapping
The Saturday 26 Jul 2014 à 21:22:05 (+0200), Max Reitz wrote :
Depending on the changed options and the image format,
bdrv_amend_options() may take a significant amount of time. In these
cases, a way to be informed about the operation's status is desirable.
Since the operation is rather
The Saturday 26 Jul 2014 à 21:22:06 (+0200), Max Reitz wrote :
Now that bdrv_amend_options() supports a status callback, use it to
display a progress report.
Signed-off-by: Max Reitz mre...@redhat.com
---
qemu-img.c | 26 +++---
1 file changed, 23 insertions(+), 3
The Saturday 26 Jul 2014 à 21:22:07 (+0200), Max Reitz wrote :
As soon as options is set in img_amend(), it needs to be freed before
the function returns. This leak is rather insignifcant, as qemu-img will
exit subsequently anyway, but there's no point in not fixing it.
Signed-off-by: Max
The Saturday 26 Jul 2014 à 21:22:08 (+0200), Max Reitz wrote :
The only really time-consuming operation potentially performed by
qcow2_amend_options() is zero cluster expansion when downgrading qcow2
images from compat=1.1 to compat=0.10, so report status of that
operation and that operation
The Thursday 31 Jul 2014 à 09:51:05 (+0200), Benoît Canet wrote :
The Saturday 26 Jul 2014 à 21:22:05 (+0200), Max Reitz wrote :
Depending on the changed options and the image format,
bdrv_amend_options() may take a significant amount of time. In these
cases, a way to be informed about
The Saturday 26 Jul 2014 à 21:22:09 (+0200), Max Reitz wrote :
Reading the refcount of a cluster is an operation which can be useful in
all of the qcow2 code, so make that function globally available.
Signed-off-by: Max Reitz mre...@redhat.com
---
block/qcow2-refcount.c | 23
The Saturday 26 Jul 2014 à 21:22:10 (+0200), Max Reitz wrote :
Currently, we have a bitmap for keeping track of which clusters have
been created during the zero cluster expansion process. This was
necessary because we need to properly increase the refcount for shared
L2 tables.
However,
The Saturday 26 Jul 2014 à 21:22:11 (+0200), Max Reitz wrote :
Add some tests for progress output to 061.
Signed-off-by: Max Reitz mre...@redhat.com
---
tests/qemu-iotests/061 | 27 +++
tests/qemu-iotests/061.out | 32
Il 31/07/2014 05:47, Ming Lei ha scritto:
On Wed, Jul 30, 2014 at 11:25 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 30/07/2014 17:12, Michael S. Tsirkin ha scritto:
Dataplane must not be a change to the guest ABI. If you implement this
feature you have to implement it for both dataplane
On Thu, Jul 31, 2014 at 02:31:34PM +0800, Tiejun Chen wrote:
v2:
* Fix some coding style
* New patch to separate i440fx_init
* Just add prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough
* Based on patch #2 to regenerate
* Unify prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough like
On Thu, Jul 31, 2014 at 7:37 AM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 30/07/2014 19:15, Ming Lei ha scritto:
On Wed, Jul 30, 2014 at 9:45 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 30/07/2014 13:39, Ming Lei ha scritto:
This patch introduces several APIs for supporting bypass qemu
Il 31/07/2014 10:59, Ming Lei ha scritto:
No guesses please. Actually that's also my guess, but since you are
submitting the patch you must do better and show profiles where stack
switching disappears after the patches.
Follows the below hardware events reported by 'perf stat' when running
Il 31/07/2014 05:22, Ming Lei ha scritto:
The problem is that g_slice here is not using the slab-style allocator
because the object is larger than roughly 500 bytes. One solution would
be to make virtqueue_pop/vring_pop allocate a VirtQueueElement of the
right size (and
On Thu, Jul 31, 2014 at 02:31:36PM +0800, Tiejun Chen wrote:
We'd like to split i440fx_init and then we can share something
with other stuff.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
I think this is too much work for very little benefit.
Just pass const char *type to i440fx_init.
--
On 2014/7/31 17:10, Michael S. Tsirkin wrote:
On Thu, Jul 31, 2014 at 02:31:36PM +0800, Tiejun Chen wrote:
We'd like to split i440fx_init and then we can share something
with other stuff.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
I think this is too much work for very little benefit.
On Wed, Jul 30, 2014 at 12:52:41PM -0600, Chris Friesen wrote:
Hi,
I'm working on a native user of virtio-serial (ie, not going via the
qemu guest agent).
The information at http://www.linux-kvm.org/page/Virtio-serial_API;
does a good job of describing the guest side of things, but has
Current object_child_foreach() uses QTAILQ_FOREACH() to walk
through children and that makes children removal from the callback
impossible.
This makes object_child_foreach() use QTAILQ_FOREACH_SAFE().
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
---
This went to Andreas's qom-next tree,
This implements DDW for VFIO. Host kernel support is required for this.
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
---
hw/ppc/spapr_pci_vfio.c | 75 +
1 file changed, 75 insertions(+)
diff --git a/hw/ppc/spapr_pci_vfio.c
At the moment sPAPR PHB supports only a single 32bit window
which is normally 1..2GB which is not enough for high performance devices.
PAPR spec enables creating an additional window(s) to support 64bit
DMA and bigger page sizes.
This patchset adds DDW support for pseries. The host kernel
This adds support for Dynamic DMA Windows (DDW) option defined by
the SPAPR specification which allows to have additional DMA window(s)
which can support page sizes other than 4K.
The existing implementation of DDW in the guest tries to create one huge
DMA window with 64K or 16MB pages and map
This implements DDW for emulated PHB.
This advertises DDW in device tree.
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
---
The DDW has not been tested as QEMU does not implement any 64bit DMA capable
device and existing linux guests do not use DDW for 32bit DMA.
---
hw/ppc/spapr_pci.c
The existing KVM_CREATE_SPAPR_TCE ioctl only support 4G windows max.
We are going to add huge DMA windows support so this will create small
window and unexpectedly fail later.
This disables KVM_CREATE_SPAPR_TCE for windows bigger that 4GB. Since
those windows are normally mapped at the boot time,
This adds a new ddw option for a machine to control presense
of the Dynamic DMA window (DDW) feature.
This option will be used by pseries to decide whether to put
DDW RTAS tokens to PHB device tree nodes or not.
This is not a PHB property because there is no way to change
the emulated PHB
This makes find_phb()/find_dev() public and changed its names
to spapr_pci_find_phb()/spapr_pci_find_dev() as they are going to
be used from other parts of QEMU such as VFIO DDW (dynamic DMA window)
or VFIO PCI error injection or VFIO EEH handling - in all these
cases there are RTAS calls which
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
---
linux-headers/linux/vfio.h | 37 -
1 file changed, 36 insertions(+), 1 deletion(-)
diff --git a/linux-headers/linux/vfio.h b/linux-headers/linux/vfio.h
index 26c218e..f0aa97d 100644
---
This enables DDW RTAS-related ioctls in VFIO.
Signed-off-by: Alexey Kardashevskiy a...@ozlabs.ru
---
hw/misc/vfio.c | 4
1 file changed, 4 insertions(+)
diff --git a/hw/misc/vfio.c b/hw/misc/vfio.c
index 0b9eba0..e7b4d6e 100644
--- a/hw/misc/vfio.c
+++ b/hw/misc/vfio.c
@@ -4437,6 +4437,10
Il 31/07/2014 04:07, Ming Lei ha scritto:
On Wed, Jul 30, 2014 at 9:51 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 30/07/2014 13:39, Ming Lei ha scritto:
diff --git a/include/hw/virtio/virtio.h b/include/hw/virtio/virtio.h
index a60104c..943e72f 100644
--- a/include/hw/virtio/virtio.h
Il 31/07/2014 07:35, Peter Crosthwaite ha scritto:
Obsoleted by automatic object_property_add arrayification.
Signed-off-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
memory.c | 30 +-
1 file changed, 5 insertions(+), 25 deletions(-)
diff --git
On Wed, Jul 30, 2014 at 08:59:09PM +0300, Chrysostomos Nanakos wrote:
Introduce new enum BlockdevOptionsArchipelago.
@volume: #Name of the Archipelago volume image
@mport: #'mport' is the port number on which mapperd is
listening. This is
Il 31/07/2014 07:34, Peter Crosthwaite ha scritto:
If [*] is given as the last part of a QOM property name, treat that
as an array property. The added property is given the first available
name, replacing the * with a decimal number counting from 0.
First add with name foo[*] will be foo[0].
At the moment spapr_tce_find_by_liobn() is used by H_PUT_TCE/...
handlers to find an IOMMU by LIOBN.
We are going to implement Dynamic DMA windows (DDW), new code
will go to a new file and we will use spapr_tce_find_by_liobn()
there too so let's make it public.
Signed-off-by: Alexey
On Thu, Jul 31, 2014 at 3:37 PM, Benoît Canet benoit.ca...@irqsave.net wrote:
The Thursday 31 Jul 2014 à 11:55:28 (+0800), Ming Lei wrote :
On Thu, Jul 31, 2014 at 7:37 AM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 30/07/2014 19:15, Ming Lei ha scritto:
On Wed, Jul 30, 2014 at 9:45 PM,
On Wed, Jul 30, 2014 at 12:27:08PM +0200, Paolo Bonzini wrote:
The memory regions should be destroyed in the unrealize function;
since these NICs are not even qdev-ified, they cannot be unplugged
and they do not have to do anything to destroy their memory regions.
Cc: stefa...@redhat.com
From: Gonglei arei.gong...@huawei.com
Introduce del_boot_device_path() to clean up fw_cfg content when
hot-unplugging a device that refers to a bootindex.
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Chenliang chenlian...@huawei.com
---
include/sysemu/sysemu.h | 1 +
vl.c
From: Gonglei arei.gong...@huawei.com
We must assure that the changed bootindex can take effect
when guest is rebooted. So we introduce fw_cfg_machine_reset(),
which change the fw_cfg file's bootindex data using the new
global fw_boot_order list.
Signed-off-by: Chenliang chenlian...@huawei.com
From: Gonglei arei.gong...@huawei.com
When we want to change one device's bootindex, we
should lookup the device and change the bootindex.
it is simply that remove it from the global boot list,
then re-add it, sorted by new bootindex.
If the new bootindex has already used by another device
just
From: Gonglei arei.gong...@huawei.com
Add HMP set-bootindex wrapper to allow setting
devcie's bootindex via monitor.
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Chenliang chenlian...@huawei.com
---
hmp-commands.hx | 15 +++
hmp.c | 13 +
hmp.h
From: Gonglei arei.gong...@huawei.com
Adds query-bootindex QMP command.
Example QMP command:
- { execute: query-bootindex}
- {
return:[
{
id:ide0-0-0,
bootindex:1,
suffix:/disk@0
},
{
id:nic1,
From: Gonglei arei.gong...@huawei.com
Sometimes, we want to modify boot order of a guest, but no need to
shutdown it. We can call dynamic changing bootindex of a guest, which
can be assured taking effect just after the guest rebooting.
For example, in P2V scene, we boot a guest and then attach a
From: Gonglei arei.gong...@huawei.com
Device should be removed from global boot list when
it is hot-unplugged.
Signed-off-by: Chenliang chenlian...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/block/virtio-blk.c| 1 +
hw/i386/kvm/pci-assign.c | 1 +
hw/misc/vfio.c
From: Gonglei arei.gong...@huawei.com
Add HMP info-bootindex command to getting
devcie's bootindex via monitor.
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by: Chenliang chenlian...@huawei.com
---
hmp-commands.hx | 2 ++
hmp.c | 20
hmp.h
From: Gonglei arei.gong...@huawei.com
Adds set-bootindex id=xx,bootindex=xx,suffix=xx QMP command.
Example QMP command:
- { execute: set-bootindex,
arguments: { id: ide0-0-1, bootindex: 1, suffix: /disk@0}}
- { return: {} }
Signed-off-by: Gonglei arei.gong...@huawei.com
Signed-off-by:
On Thu, Jul 31, 2014 at 05:26:41PM +0800, Chen, Tiejun wrote:
On 2014/7/31 17:10, Michael S. Tsirkin wrote:
On Thu, Jul 31, 2014 at 02:31:36PM +0800, Tiejun Chen wrote:
We'd like to split i440fx_init and then we can share something
with other stuff.
Signed-off-by: Tiejun Chen
On Tue, Jul 29, 2014 at 9:50 AM, zhou link freb...@gmail.com wrote:
here hit an assertion:
qemu-system-x86_64 -monitor stdio -vga qxl -spice
port=,disable-ticketing
(/home/brook/local/bin/qemu-system-x86_64:27280): Spice-Warning **:
reds.c:3295:spice_server_init: [07-29 23:41:47]ct: Jul
On Thu, Jul 31, 2014 at 5:15 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Il 31/07/2014 10:59, Ming Lei ha scritto:
No guesses please. Actually that's also my guess, but since you are
submitting the patch you must do better and show profiles where stack
switching disappears after the
On 2014/7/31 17:53, Michael S. Tsirkin wrote:
On Thu, Jul 31, 2014 at 05:26:41PM +0800, Chen, Tiejun wrote:
On 2014/7/31 17:10, Michael S. Tsirkin wrote:
On Thu, Jul 31, 2014 at 02:31:36PM +0800, Tiejun Chen wrote:
We'd like to split i440fx_init and then we can share something
with other
On Wed, Jul 30, 2014 at 06:28:28PM -0400, John Snow wrote:
+#define bitany(X, MASK) ((X) (MASK))
+#define bitset(X, MASK) (bitany((X), (MASK)) == (MASK))
This is subjective but macros like this should be avoided. This macro does not
encapsulate anything substantial. It forces the reader to
On Wed, Jul 30, 2014 at 06:28:25PM -0400, John Snow wrote:
This set collects two patches by Marc Marí already on the mailing list,
but goes further by adding a simple memory allocator that allows us to
track and debug freed memory, and optionally keep track of any leaks.
v2: use QTAILQ as
Il 31/07/2014 11:47, Ming Lei ha scritto:
Block mirroring of a device for example is done using coroutines.
As block mirroring can be done on a raw device you need coroutines.
If block layer knows the mirroring is in-progress, it still can enable
coroutine by ignoring bypass coroutine, or
On Mon, Jul 07, 2014 at 02:17:44PM -0400, John Snow wrote:
@@ -489,6 +490,72 @@ static void test_flush(void)
ide_test_quit();
}
+static void prepare_blkdebug_script(const char *debug_path, const char
*event)
+{
+FILE *debug_file = fopen(debug_path, w);
Please avoid shadowing
Can you dump *env before and after the call to kvm_arch_get_registers?
Yes, but it seems they are equal - I used memcmp() to compare them. Is
there any other side effect that cpu_synchronize_all_states() may have?
I think I found it.
The reason for hang is, because when second call to
On Wed, Jul 30, 2014 at 8:27 PM, Paolo Bonzini pbonz...@redhat.com wrote:
This ensures that the children's unparent callback will still
have a usable parent.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Reviewed-by: Peter Crosthwaite peter.crosthwa...@xilinx.com
---
qom/object.c | 2
On Mon, Jul 07, 2014 at 02:17:46PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
Drop the unused return value and make the callback optional.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/ahci.c | 6 --
On Mon, Jul 07, 2014 at 02:17:45PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
This ensures that operations are completed after a reset
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/core.c | 4 +++-
1 file
On Mon, Jul 07, 2014 at 02:17:48PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
Drop the unused return value.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/ahci.c | 4 +---
hw/ide/internal.h | 2 +-
2 files
On Mon, Jul 07, 2014 at 02:17:49PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
Drop the unused return value and make the callback optional.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/ahci.c | 4 +---
On Mon, Jul 07, 2014 at 02:17:47PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
Drop the unused return value and make the callback optional.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/ahci.c | 6 --
On Mon, Jul 07, 2014 at 02:17:50PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
Make it optional and prepare for the next patches.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/atapi.c| 6 ++
On Mon, Jul 07, 2014 at 02:17:51PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
Similar to the case removed in commit 69c38b8 (ide/core: Remove explicit
setting of BM_STATUS_INT, 2011-05-19), the only remaining use of
add_status(..., BM_STATUS_INT) is for short PRDs. The
On Mon, Jul 07, 2014 at 02:17:53PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
They are not used by AHCI, and should not be even available there.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/internal.h | 11
On Mon, Jul 07, 2014 at 02:17:52PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
It is now called only after the set_inactive callback. Put the two together.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/ahci.c
On Wed, Jul 30, 2014 at 8:27 PM, Paolo Bonzini pbonz...@redhat.com wrote:
It is never used, since ISA device are not hot-unpluggable.
Is it not good design practice though for the uninit to be correctly
implemented regardless of whether there is current-day usage? This
seems like the kind of
On Wed, Jul 30, 2014 at 8:27 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Instead, add a boolean variable to indicate the presence of the region.
Patch is good. Can you add a sentence on why you are doing this
though? Is it just the save on the repeated malloc and free (which is
sane in its own
Il 31/07/2014 14:00, Peter Crosthwaite ha scritto:
On Wed, Jul 30, 2014 at 8:27 PM, Paolo Bonzini pbonz...@redhat.com wrote:
It is never used, since ISA device are not hot-unpluggable.
Is it not good design practice though for the uninit to be correctly
implemented regardless of whether
On Thu, Jul 31, 2014 at 7:46 PM, Stefan Hajnoczi stefa...@redhat.com wrote:
On Wed, Jul 30, 2014 at 12:27:08PM +0200, Paolo Bonzini wrote:
The memory regions should be destroyed in the unrealize function;
since these NICs are not even qdev-ified, they cannot be unplugged
and they do not have
On Mon, Jul 07, 2014 at 02:17:54PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/core.c | 20 ++--
hw/ide/internal.h | 12 ++--
hw/ide/pci.c
Il 31/07/2014 14:01, Peter Crosthwaite ha scritto:
On Wed, Jul 30, 2014 at 8:27 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Instead, add a boolean variable to indicate the presence of the region.
Patch is good. Can you add a sentence on why you are doing this
though? Is it just the save on
On 2014/7/31 18:12, Chen, Tiejun wrote:
On 2014/7/31 17:53, Michael S. Tsirkin wrote:
On Thu, Jul 31, 2014 at 05:26:41PM +0800, Chen, Tiejun wrote:
On 2014/7/31 17:10, Michael S. Tsirkin wrote:
On Thu, Jul 31, 2014 at 02:31:36PM +0800, Tiejun Chen wrote:
We'd like to split i440fx_init and
Implement a pci host bridge specific to passthrough. Actually
this just inherits the standard one.
This is based on http://patchwork.ozlabs.org/patch/363810/.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/pci-host/piix.c | 41 +
On Mon, Jul 07, 2014 at 02:17:55PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
These are defined twice, just use one set consistently.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
hw/ide/ahci.c | 6 +++---
Xen wants to supply a different pci and host devices,
inheriting i440fx devices. Make types configurable.
Signed-off-by: Michael S. Tsirkin m...@redhat.com
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/i386/pc_piix.c| 4 +++-
hw/pci-host/piix.c | 9 -
include/hw/i386/pc.h
v3:
* Drop patch #4
* Add one patch #1 from Michael
* Rebase
v2:
* Fix some coding style
* New patch to separate i440fx_init
* Just add prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough
* Based on patch #2 to regenerate
* Unify prefix with XEN_IGD_PASSTHROUGH/xen_igd_passthrough like patch #3
We'd like to split pc_init1 and then we can share something
with other stuff.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/i386/pc_piix.c | 117 +++---
1 file changed, 93 insertions(+), 24 deletions(-)
v3:
* Rebase
v2:
* Fix some
Now we can introduce a new machine, xenigd, specific to IGD
passthrough. This can avoid involving other common codes.
Signed-off-by: Tiejun Chen tiejun.c...@intel.com
---
hw/i386/pc_piix.c | 106 ++
1 file changed, 106 insertions(+)
v3:
*
On Mon, Jul 07, 2014 at 02:17:56PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
This will provide a hook for sending the result of the command via the
FIS receive area.
Signed-off-by: Paolo Bonzini pbonz...@redhat.com
Signed-off-by: John Snow js...@redhat.com
---
On Mon, Jul 07, 2014 at 02:17:57PM -0400, John Snow wrote:
From: Paolo Bonzini pbonz...@redhat.com
AHCI has code to fill in the D2H FIS trigger the IRQ all over the place.
Centralize this in a single cmd_done callback by generalizing the existing
async_cmd_done callback.
Signed-off-by:
From: Gonglei arei.gong...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/usb/dev-audio.c | 2 +-
hw/usb/dev-mtp.c | 4 ++--
hw/usb/hcd-ehci.c | 2 +-
3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/hw/usb/dev-audio.c b/hw/usb/dev-audio.c
index
From: Gonglei arei.gong...@huawei.com
Gonglei (7):
usb: a trivial code change for more idiomatic writing style
audio: a trivial code change for more idiomatic writing style
isa-bus: a trivial code change for more idiomatic writing style
a trivial code change for more idiomatic writing
From: Gonglei arei.gong...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
ui/spice-core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/ui/spice-core.c b/ui/spice-core.c
index 7bb91e6..ff54dac 100644
--- a/ui/spice-core.c
+++ b/ui/spice-core.c
@@ -677,7
From: Gonglei arei.gong...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
vl.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/vl.c b/vl.c
index fe451aa..69f0f20 100644
--- a/vl.c
+++ b/vl.c
@@ -1136,7 +1136,7 @@ static int drive_init_func(QemuOpts *opts,
From: Gonglei arei.gong...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/net/vmxnet3.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/hw/net/vmxnet3.c b/hw/net/vmxnet3.c
index 77bea6f..34c5aff 100644
--- a/hw/net/vmxnet3.c
+++
From: Gonglei arei.gong...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
qdev-monitor.c | 2 +-
qemu-char.c | 2 +-
util/qemu-sockets.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/qdev-monitor.c b/qdev-monitor.c
index f87f3d8..3e30d38
On Mon, Jul 07, 2014 at 02:17:58PM -0400, John Snow wrote:
+static void ahci_write_fis_pio(AHCIDevice *ad, uint16_t len)
+{
+AHCIPortRegs *pr = ad-port_regs;
+uint8_t *pio_fis, *cmd_fis;
+uint64_t tbl_addr;
+dma_addr_t cmd_len = 0x80;
+
+if (!ad-res_fis || !(pr-cmd
From: Gonglei arei.gong...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/isa/isa-bus.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/isa/isa-bus.c b/hw/isa/isa-bus.c
index b28981b..851f953 100644
--- a/hw/isa/isa-bus.c
+++ b/hw/isa/isa-bus.c
@@ -50,7
On Wed, Jul 30, 2014 at 8:27 PM, Paolo Bonzini pbonz...@redhat.com wrote:
Of the two functions portio_list_del and portio_list_destroy,
the latter is just freeing a memory area. However, portio_list_del
is the logical equivalent of memory_region_del_subregion so
destruction of memory regions
From: Gonglei arei.gong...@huawei.com
Signed-off-by: Gonglei arei.gong...@huawei.com
---
hw/audio/gus.c | 2 +-
hw/audio/hda-codec.c | 3 ++-
hw/audio/sb16.c | 6 +++---
3 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/hw/audio/gus.c b/hw/audio/gus.c
index
On Mon, Jul 07, 2014 at 02:17:59PM -0400, John Snow wrote:
Currently, the ioapic device can not be found in a qtest environment
when requesting irq_interrupt_in ioapic via the qtest socket.
By mirroring how the ioapic is added in i44ofx (hw/i440/pc_piix.c),
as a child of q35, the device is
1 - 100 of 283 matches
Mail list logo