[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V5 09/18] virtio-ccw: introduce ccw specific que
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH V5 09/18] virtio-ccw: introduce ccw specific queue limit |
Date: |
Thu, 2 Apr 2015 15:47:42 +0200 |
On Wed, 1 Apr 2015 16:15:03 +0800
Jason Wang <address@hidden> wrote:
> Instead of depending on marco, using a bus specific limit. Also make
> it clear that the number of gsis per I/O adapter is not directly
> depending on the number of virtio queues, but rather the other way
> around.
>
> Cc: Alexander Graf <address@hidden>
> Cc: Cornelia Huck <address@hidden>
> Cc: Christian Borntraeger <address@hidden>
> Cc: Richard Henderson <address@hidden>
> Signed-off-by: Jason Wang <address@hidden>
> ---
> hw/s390x/s390-virtio-ccw.c | 7 +++++--
> hw/s390x/virtio-ccw.c | 19 ++++++++++++-------
> include/hw/s390x/s390_flic.h | 4 +++-
> 3 files changed, 20 insertions(+), 10 deletions(-)
>
> diff --git a/hw/s390x/virtio-ccw.c b/hw/s390x/virtio-ccw.c
> index dfe6ded..935b880 100644
> --- a/hw/s390x/virtio-ccw.c
> +++ b/hw/s390x/virtio-ccw.c
> @@ -265,8 +267,9 @@ static int virtio_ccw_set_vqs(SubchDev *sch, uint64_t
> addr, uint32_t align,
> uint16_t index, uint16_t num)
> {
> VirtIODevice *vdev = virtio_ccw_get_vdev(sch);
> + int queue_max = virtio_get_queue_max(vdev);
>
> - if (index > VIRTIO_PCI_QUEUE_MAX) {
> + if (index > queue_max) {
trivial conflict on master (>= instead of >)
> return -EINVAL;
> }
>
In master, there's also a new instance of VIRTIO_PCI_QUEUE_MAX (in
virtio_ccw_cb()) which needs a similar treatment.
But on the whole, looks good.
- [Qemu-devel] [PATCH V5 02/18] pc: add 2.4 machine types, (continued)
- [Qemu-devel] [PATCH V5 02/18] pc: add 2.4 machine types, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 03/18] spapr: add machine type specific instance init function, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 04/18] ppc: spapr: add 2.4 machine type, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 06/18] monitor: check return value of qemu_find_net_clients_except(), Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 05/18] monitor: replace the magic number 255 with MAX_QUEUE_NUM, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 07/18] virtio-ccw: using VIRTIO_NO_VECTOR instead of 0 for invalid virtqueue, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 08/18] virtio: introduce bus specific queue limit, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 11/18] virtio-mmio: switch to bus specific queue limit, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 10/18] virtio-s390: switch to bus specific queue limit, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 09/18] virtio-ccw: introduce ccw specific queue limit, Jason Wang, 2015/04/01
- Re: [Qemu-devel] [PATCH V5 09/18] virtio-ccw: introduce ccw specific queue limit,
Cornelia Huck <=
- [Qemu-devel] [PATCH V5 12/18] virtio-pci: switch to use bus specific queue limit, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 13/18] virtio: introduce vector to virtqueues mapping, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 14/18] virtio: introduce virtio_queue_get_index(), Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 15/18] virtio-pci: speedup MSI-X masking and unmasking, Jason Wang, 2015/04/01
- [Qemu-devel] [PATCH V5 16/18] virtio-pci: increase the maximum number of virtqueues to 513, Jason Wang, 2015/04/01