> From: Feng Liu <fe...@nvidia.com> > Sent: Friday, March 10, 2023 12:34 AM > > - if (!is_power_of_2(num)) { > - dev_warn(&vp_dev->pci_dev->dev, "bad queue size %u", > num); > - return ERR_PTR(-EINVAL); > - } > - The check is still valid for split q. Maybe the right place for such a check is not the pci transport driver. But layer below where split vs packed q knowledge resides and hence, power of 2 check can be omitted for packed vq. _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization
RE: [PATCH v2 1/3] virtio_pci_modern: Allow non power of 2 sizes for virtqueues
Parav Pandit via Virtualization Fri, 10 Mar 2023 05:36:54 -0800
- [PATCH v2 0/3] virtio_ring: Clean up code ... Feng Liu via Virtualization
- [PATCH v2 3/3] virtio_ring: Use const... Feng Liu via Virtualization
- [PATCH v2 1/3] virtio_pci_modern: All... Feng Liu via Virtualization
- RE: [PATCH v2 1/3] virtio_pci_mod... Parav Pandit via Virtualization
- Re: [PATCH v2 1/3] virtio_pci... Feng Liu via Virtualization
- Re: [PATCH v2 1/3] virtio... Michael S. Tsirkin
- Re: [PATCH v2 1/3] v... Feng Liu via Virtualization
- Re: [PATCH v2 1/... Michael S. Tsirkin
- Re: [PATCH v... Feng Liu via Virtualization
- [PATCH v2 2/3] virtio_ring: Avoid usi... Feng Liu via Virtualization
- Re: [PATCH v2 0/3] virtio_ring: Clean... Michael S. Tsirkin
- Re: [PATCH v2 0/3] virtio_ring: C... Feng Liu via Virtualization
- Re: [PATCH v2 0/3] virtio_rin... Michael S. Tsirkin
- Re: [PATCH v2 0/3] virtio... Feng Liu via Virtualization