[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 06/11] virtio-s390: switch to bus specific qu
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH V2 06/11] virtio-s390: switch to bus specific queue limit |
Date: |
Fri, 27 Feb 2015 10:49:54 +0100 |
On Fri, 27 Feb 2015 06:42:57 +0008
Jason Wang <address@hidden> wrote:
> On Thu, Feb 26, 2015 at 9:05 PM, Cornelia Huck
> <address@hidden> wrote:
> > On Thu, 26 Feb 2015 15:04:41 +0800
> > Jason Wang <address@hidden> wrote:
> >
> >> typedef struct AdapterRoutes {
> >> AdapterInfo adapter;
> >> int num_routes;
> >> - int gsi[VIRTIO_PCI_QUEUE_MAX];
> >> + int gsi[VIRTIO_S390_QUEUE_MAX];
> >
> > Adapter routes are only applicable for the ccw transport, not for the
> > old s390 transport.
>
> Sure, will fix this.
>
> >
> >
> > (I'm also wondering whether this should be the generic limit instead.)
>
> As you pointed out in V1, there will be more issues if we just increase
> the generic limit. So I switch to use per transport limit. Since the
> limit was not changed for both s390 and ccw, it should be ok.
I'm just wondering how many gsis we want to support for adapter routes.
They were introduced for virtio-ccw, but recently s390 pci has started
to use them as well, so a virtio limit seems silly here. I'll switch
them to some kind of generic limit instead, I think.
>
> >> } AdapterRoutes;
> >>
> >> #define TYPE_S390_FLIC_COMMON "s390-flic"
> >
> >
>
- [Qemu-devel] [PATCH V2 05/11] virtio-serial-bus: switch to bus specific queue limit, (continued)
[Qemu-devel] [PATCH V2 07/11] virtio-mmio: switch to bus specific queue limit, Jason Wang, 2015/02/26
[Qemu-devel] [PATCH V2 06/11] virtio-s390: switch to bus specific queue limit, Jason Wang, 2015/02/26
[Qemu-devel] [PATCH V2 09/11] virtio-pci: increase the maximum number of virtqueues to 513, Jason Wang, 2015/02/26
[Qemu-devel] [PATCH V2 08/11] virtio-pci: switch to use bus specific queue limit, Jason Wang, 2015/02/26
[Qemu-devel] [PATCH V2 10/11] pci: remove hard-coded bar size in msix_init_exclusive_bar(), Jason Wang, 2015/02/26
[Qemu-devel] [PATCH V2 11/11] virtio-pci: introduce auto_msix_bar_size property, Jason Wang, 2015/02/26