[PATCH ipmi-fru-parser v2] Fix 'dbus.String not found in lookup' error message
OpenBMC Patches
openbmc-patches at stwcx.xyz
Thu Apr 7 20:30:12 AEST 2016
Previously the ipmi-fru-parser will call dbus method getObjectFromId()
for each fru areas to get the dbus object path of fru area,
even if the fru area does not exist.
Also ipmi_validate_fru_area() may be invoked multipul times before
full fru data is parsed. This will generate duplicated error
messages from SystemManager:, e.g:
"ERROR SystemManager: dbus.String(u'INTERNAL_13') not found in lookup".
In this case, "INTERNAL" area does not exists for FRU 13.
This patch tries to reduce the messages. For FRUs managed by
hostboot, ipmi-fru-parser only asks for dbus object path when
updating the fru data to BMC inventory.
For FRUs mananged by BMC (eeprom), ipmi-fru-parser also asks for dbus
object path when setting the fru's 'present' and 'fault' status.
ipmi-fru-parser needs to call getObjectFromId() to know whether a
fru should be present, in order to decide the 'present' and 'fault'
status. So this message will still be triggered when SystemManager
does not define object path for a fru area.
This patch also sets 'present' and 'fault' properly when there is error
processing a BMC managed FRU.
Signed-off-by: Yi Li <adamliyi at msn.com>
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/openbmc/ipmi-fru-parser/13)
<!-- Reviewable:end -->
https://github.com/openbmc/ipmi-fru-parser/pull/13
Yi Li (1):
Fix 'dbus.String not found in lookup' error message
writefrudata.C | 85 +++++++++++++++++++++++++++++++++++-----------------------
1 file changed, 51 insertions(+), 34 deletions(-)
--
2.7.1
More information about the openbmc
mailing list