From 331ddf251ce94e6d14f354b44565545bb95c1ed7 Mon Sep 17 00:00:00 2001 From: Andrew Jeffery Date: Mon, 8 Apr 2019 16:51:30 +0930 Subject: mboxd: Depend on xyz.openbmc_project.Control.Host This is one of the bus names claimed by ipmid, and according to bea22141ca95 ("Ensure ipmid doesn't start until on dbus") it's the last to be claimed. Depend on this to ensure our required IPMI services are available. I've observed a reasonably reliable difference in behaviour between depending on the canonical service (phosphor-ipmi-host.service) vs the alias (xyz.openbmc_project.Control.Host.service) in that setAttention()s from mboxd state changes were propagated as expected in the latter case but not in the former. (From meta-phosphor rev: 3b0c37e3e0bf2a6f6e9cca7f4f1e49f4c79cc1e1) Change-Id: Ibcda190e28aa169923f80f48ab98a2a2bd890e4a Signed-off-by: Andrew Jeffery Signed-off-by: Brad Bishop --- meta-phosphor/recipes-phosphor/mboxd/mboxd/mboxd.service | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/meta-phosphor/recipes-phosphor/mboxd/mboxd/mboxd.service b/meta-phosphor/recipes-phosphor/mboxd/mboxd/mboxd.service index b13f72c05..abc97e522 100644 --- a/meta-phosphor/recipes-phosphor/mboxd/mboxd/mboxd.service +++ b/meta-phosphor/recipes-phosphor/mboxd/mboxd/mboxd.service @@ -4,8 +4,8 @@ BindsTo=dev-aspeed\x2dlpc\x2dctrl.device After=dev-aspeed\x2dlpc\x2dctrl.device BindsTo=dev-aspeed\x2dmbox.device After=dev-aspeed\x2dmbox.device -Requires=phosphor-ipmi-host.service -After=phosphor-ipmi-host.service +Requires=xyz.openbmc_project.Control.Host.service +After=xyz.openbmc_project.Control.Host.service [Service] Restart=always -- cgit v1.2.3