Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-06 Thread Hong Bo Li
On 7/6/2015 18:56, Michael S. Tsirkin wrote: On Mon, Jul 06, 2015 at 10:06:50AM +0800, Hong Bo Li wrote: On 7/5/2015 2:25, Michael S. Tsirkin wrote: On Fri, Jul 03, 2015 at 07:09:59PM +0800, Hong Bo Li wrote: But I would like to note that pci device drivers require driver handshake before

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-05 Thread Hong Bo Li
On 7/5/2015 2:25, Michael S. Tsirkin wrote: On Fri, Jul 03, 2015 at 07:09:59PM +0800, Hong Bo Li wrote: But I would like to note that pci device drivers require driver handshake before device goes away. IIUC s390 hotplug is immediate, which is a problem. Maybe doing the change will help make

[Qemu-devel] [PATCH v3 0/1] s390 pci infrastructure modelling

2015-07-03 Thread Hong Bo Li
I will make a cleanup patch later to change these names to a more suitable name. Comparing to v2, this patch does the following change: 1.use a link list to store all the zpci devices, and sort them by fid. 2.add a flag "token_valid" to mark whether the list zpci instructi

[Qemu-devel] [PATCH v3 1/1] s390 pci infrastructure modelling

2015-07-03 Thread Hong Bo Li
, and references the corresponding vfio pci device via device id. We create a s390 pci facility bus to hold all the zpci devices. Signed-off-by: Hong Bo Li --- hw/s390x/s390-pci-bus.c| 316 + hw/s390x/s390-pci-bus.h| 41 +- hw/s390x/s390-pci

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-03 Thread Hong Bo Li
On 7/2/2015 13:13, Michael S. Tsirkin wrote: On Thu, Jul 02, 2015 at 10:57:34AM +0800, Hong Bo Li wrote: On 7/1/2015 21:37, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 08:42:52PM +0800, Hong Bo Li wrote: On 7/1/2015 19:57, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:46:01PM

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/2/2015 13:13, Michael S. Tsirkin wrote: On Thu, Jul 02, 2015 at 10:57:34AM +0800, Hong Bo Li wrote: On 7/1/2015 21:37, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 08:42:52PM +0800, Hong Bo Li wrote: On 7/1/2015 19:57, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:46:01PM

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/1/2015 21:37, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 08:42:52PM +0800, Hong Bo Li wrote: On 7/1/2015 19:57, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:46:01PM +0800, Hong Bo Li wrote: On 7/1/2015 19:23, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:11:38PM

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/1/2015 19:57, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:46:01PM +0800, Hong Bo Li wrote: On 7/1/2015 19:23, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:11:38PM +0800, Hong Bo Li wrote: On 7/1/2015 18:36, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 06:04:24PM

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/1/2015 19:57, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:46:01PM +0800, Hong Bo Li wrote: On 7/1/2015 19:23, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:11:38PM +0800, Hong Bo Li wrote: On 7/1/2015 18:36, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 06:04:24PM

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/1/2015 19:23, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 07:11:38PM +0800, Hong Bo Li wrote: On 7/1/2015 18:36, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 06:04:24PM +0800, Hong Bo Li wrote: On 7/1/2015 17:22, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 05:13:11PM

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/1/2015 18:36, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 06:04:24PM +0800, Hong Bo Li wrote: On 7/1/2015 17:22, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 05:13:11PM +0800, Hong Bo Li wrote: On 7/1/2015 16:05, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 03:56:25PM

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/1/2015 17:22, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 05:13:11PM +0800, Hong Bo Li wrote: On 7/1/2015 16:05, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 03:56:25PM +0800, Hong Bo Li wrote: On 7/1/2015 14:22, Michael S. Tsirkin wrote: On Tue, Jun 30, 2015 at 02:16:59PM

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/1/2015 16:05, Michael S. Tsirkin wrote: On Wed, Jul 01, 2015 at 03:56:25PM +0800, Hong Bo Li wrote: On 7/1/2015 14:22, Michael S. Tsirkin wrote: On Tue, Jun 30, 2015 at 02:16:59PM +0800, Hong Bo Li wrote: On 6/29/2015 18:01, Michael S. Tsirkin wrote: On Mon, Jun 29, 2015 at 05:24

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-07-01 Thread Hong Bo Li
On 7/1/2015 14:22, Michael S. Tsirkin wrote: On Tue, Jun 30, 2015 at 02:16:59PM +0800, Hong Bo Li wrote: On 6/29/2015 18:01, Michael S. Tsirkin wrote: On Mon, Jun 29, 2015 at 05:24:53PM +0800, Hong Bo Li wrote: This patch introduce a new facility(and bus) to hold devices representing

Re: [Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-06-29 Thread Hong Bo Li
On 6/29/2015 18:01, Michael S. Tsirkin wrote: On Mon, Jun 29, 2015 at 05:24:53PM +0800, Hong Bo Li wrote: This patch introduce a new facility(and bus) to hold devices representing information actually provided by s390 firmware and I/O configuration. usage example: -device s390-pcihost -device

[Qemu-devel] [PATCH v2 1/1] KVM s390 pci infrastructure modelling

2015-06-29 Thread Hong Bo Li
, and references the corresponding vfio pci device via device id. We create a s390 pci facility bus to hold all the zpci devices. Signed-off-by: Hong Bo Li --- hw/s390x/s390-pci-bus.c| 314 + hw/s390x/s390-pci-bus.h| 48 ++- hw/s390x/s390

[Qemu-devel] [PATCH v2 0/1] s390 pci infrastructure modelling

2015-06-29 Thread Hong Bo Li
fid = 1 (0x1) uid = 2 (0x2) pci_id = "vpci1" dev: zpci, id "zpci2" fid = 6 (0x6) uid = 7 (0x7) pci_id = "vpci2" To make the review easier, I keep all of the old names, such as S390PCIBusDevice to name a zpci device.

Re: [Qemu-devel] [PATCH v1 1/1] S390 pci infrastructure modeling

2015-05-21 Thread Hong Bo Li
pci device via device id. We create a s390 pci facility bus to hold all the zpci devices. Signed-off-by: Hong Bo Li --- hw/s390x/s390-pci-bus.c| 317 + hw/s390x/s390-pci-bus.h| 48 ++- hw/s390x/s390-pci-inst.c | 4 +- hw/s390x

Re: [Qemu-devel] [PATCH v1 1/1] S390 pci infrastructure modeling

2015-05-19 Thread Hong Bo Li
On 5/12/2015 17:07, Cornelia Huck wrote: On Tue, 21 Apr 2015 13:29:01 +0800 Hong Bo Li wrote: Sorry, I sent the patch three times by accident. Please take this one as the formal patch. This is a followup to the discussion started by Frank Blaschka: Option 1: s390x/pci: Extend pci

Re: [Qemu-devel] [PATCH v1 1/1] S390 pci infrastructure modeling

2015-04-20 Thread Hong Bo Li
h is the major concerned point in previous discussion. The idea of option 3 is: for each vfio-pci device, we define a zpci device, and do all s390 specific action on that device. On 4/15/2015 16:04, Hong Bo Li wrote: This patch contains the actual interesting changes. usage example: -device

[Qemu-devel] [PATCH v1 1/1] S390 pci infrastructure modeling

2015-04-15 Thread Hong Bo Li
facility bus to hold all the zpci devices. Signed-off-by: Hong Bo Li --- hw/s390x/s390-pci-bus.c| 317 + hw/s390x/s390-pci-bus.h| 48 ++- hw/s390x/s390-pci-inst.c | 4 +- hw/s390x/s390-virtio-ccw.c | 4 +- 4 files changed, 285 insertions

[Qemu-devel] [PATCH v1 1/1] S390 pci infrastructure modeling

2015-04-15 Thread Hong Bo Li
facility bus to hold all the zpci devices. Signed-off-by: Hong Bo Li --- hw/s390x/s390-pci-bus.c| 317 + hw/s390x/s390-pci-bus.h| 48 ++- hw/s390x/s390-pci-inst.c | 4 +- hw/s390x/s390-virtio-ccw.c | 4 +- 4 files changed, 285 insertions

[Qemu-devel] [PATCH v1 0/1] s390 pci infrastucture modeling

2015-04-15 Thread Hong Bo Li
uid = 7 (0x7) pci_id = "vpci2" To make the review easier, I keep all of the old names, such as S390PCIBusDevice to name a zpci device. I will make a cleanup patch later to change these names to a more suitable name. Hong Bo Li (1): S390 pci infrastructure modeling hw/s390

[Qemu-devel] [PATCH v1 0/1] s390 pci infrastucture modeling

2015-04-14 Thread Hong Bo Li
uid = 7 (0x7) pci_id = "vpci2" To make the review easier, I keep all of the old names, such as S390PCIBusDevice to name a zpci device. I will make a cleanup patch later to change these names to a more suitable name. Hong Bo Li (1): s390 pci infrastructure modeling hw/s390

[Qemu-devel] [PATCH v1 1/1] s390 pci infrastructure modeling

2015-04-14 Thread Hong Bo Li
facility bus to hold all the zpci devices. Signed-off-by: Hong Bo Li --- hw/s390x/s390-pci-bus.c| 317 + hw/s390x/s390-pci-bus.h| 48 ++- hw/s390x/s390-pci-inst.c | 4 +- hw/s390x/s390-virtio-ccw.c | 4 +- 4 files changed, 285 insertions