[PATCH 5/8] domain: save/restore the state of dbus-daemon running
Michal Privoznik
mprivozn at redhat.com
Tue Feb 25 11:30:01 UTC 2020
On 2/24/20 4:58 PM, Marc-André Lureau wrote:
> Hi
>
> On Thu, Feb 20, 2020 at 10:04 AM Michal Privoznik <mprivozn at redhat.com> wrote:
>>
>> On 1/14/20 2:46 PM, marcandre.lureau at redhat.com wrote:
>>> From: Marc-André Lureau <marcandre.lureau at redhat.com>
>>>
>>> This avoids trying to start a dbus-daemon when its already running.
>>>
>>> Signed-off-by: Marc-André Lureau <marcandre.lureau at redhat.com>
>>> ---
>>> src/qemu/qemu_domain.c | 5 +++++
>>> 1 file changed, 5 insertions(+)
>>>
>>> diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
>>> index 7722a53c62..dda3cb781f 100644
>>> --- a/src/qemu/qemu_domain.c
>>> +++ b/src/qemu/qemu_domain.c
>>> @@ -2916,6 +2916,9 @@ qemuDomainObjPrivateXMLFormat(virBufferPtr buf,
>>> virDomainChrTypeToString(priv->monConfig->type));
>>> }
>>>
>>> + if (priv->dbusDaemonRunning)
>>> + virBufferAddLit(buf, "<dbusDaemon/>\n");
>>> +
>>> if (priv->namespaces) {
>>> ssize_t ns = -1;
>>>
>>> @@ -3697,6 +3700,8 @@ qemuDomainObjPrivateXMLParse(xmlXPathContextPtr ctxt,
>>> goto error;
>>> }
>>>
>>> + priv->dbusDaemonRunning = virXPathBoolean("boolean(./dbusDaemon)", ctxt) > 0;
>>> +
>>> if ((node = virXPathNode("./namespaces", ctxt))) {
>>> xmlNodePtr next;
>>>
>>>
>>
>> I'd push these a bit down - closer to PR daemon and slirp so that they
>> are grouped together.
>
> Well, as we introduce DBus bus for the VM, it would be a foundation
> for IPC/multi-process communication, not specific to slirp. So I'd
> leave it near the top.
>
My reasoning was that while in private data the order doesn't matter
really, but so far we keep internal flags towards the beginning and
helper daemons related flags towards the end. That is not to say that
slirp and dbus have something in common. But apparently, our
parser/formater are out of order anyway. Ideally, we should have the
same order for parsing/formatting as defined in the struct. But that
ship sailed long ago.
Michal
More information about the libvir-list
mailing list