[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 5/5] vhost-user-blk: default num_queues to -smp N
From: |
Raphael Norwitz |
Subject: |
Re: [PATCH v5 5/5] vhost-user-blk: default num_queues to -smp N |
Date: |
Fri, 10 Jul 2020 11:48:42 -0700 |
On Fri, Jul 10, 2020 at 5:53 AM Stefan Hajnoczi <stefanha@redhat.com> wrote:
>
> On Thu, Jul 09, 2020 at 11:02:24AM -0700, Raphael Norwitz wrote:
> > On Mon, Jul 6, 2020 at 7:00 AM Stefan Hajnoczi <stefanha@redhat.com> wrote:
> > > diff --git a/hw/block/vhost-user-blk.c b/hw/block/vhost-user-blk.c
> > > index a00b854736..39aec42dae 100644
> > > --- a/hw/block/vhost-user-blk.c
> > > +++ b/hw/block/vhost-user-blk.c
> > > @@ -420,6 +420,9 @@ static void vhost_user_blk_device_realize(DeviceState
> > > *dev, Error **errp)
> > > return;
> > > }
> > >
> > > + if (s->num_queues == VHOST_USER_BLK_AUTO_NUM_QUEUES) {
> > > + s->num_queues = 1;
> > > + }
> >
> > What is this check for? Is it just a backstop to ensure that
> > num_queues is set to 1 if vhost-user-blk-pci doesn't update it?
>
> For the non-PCI VIRTIO transports that do not handle num_queues ==
> VHOST_USER_BLK_AUTO_NUM_QUEUES themselves.
>
Got it. Looks good then.
Reviewed-by: Raphael Norwitz <raphael.norwitz@nutanix.com>
> Stefan
- Re: [PATCH v5 3/5] virtio-scsi: default num_queues to -smp N, (continued)
[PATCH v5 1/5] virtio-pci: add virtio_pci_optimal_num_queues() helper, Stefan Hajnoczi, 2020/07/06
[PATCH v5 4/5] virtio-blk: default num_queues to -smp N, Stefan Hajnoczi, 2020/07/06
[PATCH v5 5/5] vhost-user-blk: default num_queues to -smp N, Stefan Hajnoczi, 2020/07/06
Re: [PATCH v5 0/5] virtio-pci: enable blk and scsi multi-queue by default, Michael S. Tsirkin, 2020/07/08