tree 5a5c25c7e4c0fe975344beeebdb53c2c39c9c0c0
parent 07c35131cb7dca8f0c939417ca8eb379b0ea04c4
author Andrew Geissler <geissonator@yahoo.com> 1573597614 -0600
committer Andrew Geissler <geissonator@yahoo.com> 1573597614 -0600

dbus-name: do startup work before registering name

The following commit aggravated an existing issue within OpenBMC
software:

https://gerrit.openbmc-project.xyz/c/openbmc/meta-phosphor/+/26476

A busctl pcap of the failure shows mapper trying to introspect the
dbus monitor while the dbus monitor is trying to set itself up to
monitor its defined dbus events. This includes calls into mapper and
inventory manager.

This results in a timeout on the mapper side and the dbus monitor object
never being stored within mapper.

The above commit appears to aggravate this bug by allowing mapper to
start introspecting the dbus monitor sooner then it normally does.

Tested:
Verified with this fix that the issue, which was 100% recreatable prior
to this change, did not recreate with multiple BMC reboots

Signed-off-by: Andrew Geissler <geissonator@yahoo.com>
Change-Id: I6c9380af8b7331bbdbbee0ea3bd0b5ff17dc4187