qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PULL 09/19] libqos/qgraph: add qos_node_create_driver_named()


From: Paolo Bonzini
Subject: Re: [PULL 09/19] libqos/qgraph: add qos_node_create_driver_named()
Date: Thu, 18 Feb 2021 10:14:12 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.7.0

On 18/02/21 10:10, Christian Schoenebeck wrote:
On Montag, 15. Februar 2021 15:06:41 CET Christian Schoenebeck wrote:
On Montag, 15. Februar 2021 14:16:16 CET Paolo Bonzini wrote:
From: qemu_oss--- via <qemu-devel@nongnu.org>

So far the qos subsystem of the qtest framework had the limitation
that only one instance of the same official QEMU (QMP) driver name
could be created for qtests. That's because a) the created qos
node names must always be unique, b) the node name must match the
official QEMU driver name being instantiated and c) all nodes are
in a global space shared by all tests.

This patch removes this limitation by introducing a new function
qos_node_create_driver_named() which allows test case authors to
specify a node name being different from the actual associated
QEMU driver name. It fills the new 'qemu_name' field of
QOSGraphNode for that purpose.

Adjust build_driver_cmd_line() and qos_graph_node_set_availability()
to correctly deal with either accessing node name vs. node's
qemu_name correctly.

Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
Message-Id:
<3be962ff38f3396f8040deaa5ffdab525c4e0b16.1611704181.git.qemu_oss@crudebyt
e
.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>

Just a side note: The odd "From:" line was because of a temporary issue with
the mailman version running GNU lists, which caused mailman to rewrite
certain sender addresses. The problem with mailman had been fixed in the
meantime.

I personally don't care about it, but just that you know that this did not
happen by purpose or something.

Paolo, do you want me to resend these patches as v2 for fixing the author
rewrite issue?

The patches already in and with the right author.

Paolo




reply via email to

[Prev in Thread] Current Thread [Next in Thread]