On 2/21/25 7:10 AM, Peter Krempa wrote:
On Fri, Feb 21, 2025 at 02:06:42 -0500, Laine Stump wrote:
> passt in vhost-user mode doesn't support using multiple queues.
>
> The path of the socket is auto-generated by libvirt for
> vhost-user/passt; it can't be set by the user
>
> The passt end of a vhost-user socket is always the server, and the
> qemu end is always a client.
>
> Signed-off-by: Laine Stump <laine(a)redhat.com>
> ---
> src/qemu/qemu_validate.c | 16 ++++++++++++++++
> 1 file changed, 16 insertions(+)
>
> diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c
> index 3e3e368da3..53affcdcff 100644
> --- a/src/qemu/qemu_validate.c
> +++ b/src/qemu/qemu_validate.c
> @@ -1824,6 +1824,22 @@ qemuValidateDomainDeviceDefNetwork(const virDomainNetDef
*net,
> net->backend.type == VIR_DOMAIN_NET_BACKEND_PASST) {
> if (qemuValidateDomainDefVhostUserRequireSharedMemory(def, "interface
type=\"vhostuser\" backend type=\"passt\"") < 0)
> return -1;
> +
> + if (net->driver.virtio.queues > 0) {
> + virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
> + _("interface type=\"vhostuser\" backend
type=\"passt\" does not support multiple queues"));
In our XML formatter we use single quotes for XML attribute value, in
order to not have to escape stuff.
Yeah, I'm trying to remember why it was that I decided to use double
quotes in the error messages. I do recall thinking about it and making a
conscious decision, but don't remember the reason. I'll think about it
for awhile again and hopefully I'll remember; it *might* have been that
there was existing precedent for using double quotes. If I can't recall
the reason then I'll change them all to single quotes before I push.