[PATCH ipmi-fru-parser] Using new getFRUOjbect() API to fix 'not found in lookup' error message
OpenBMC Patches
openbmc-patches at stwcx.xyz
Mon Apr 11 19:50:12 AEST 2016
The ipmi-fru-parser calls the dbus method
SystemManager::getObjectFromId('FRU_STR', fru_area_name) 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
For FRUs mananged by BMC (eeprom), ipmi-fru-parser sets the
'present' and 'fault' status of a fru area.
ipmi-fru-parser needs to know whether a fru should be present, based on
whether getObjectFromId() returns valid dbus object path.
So it is expected that getObjectFromId() may be called with
a 'fru_area_name' not defined in Skeleton's ID_LOOKUP table.
In another pull request for skeleton, a new method:
getFRUObject('fru_area_name') is added specially for ipmi-fru-parser.
See PR: https://github.com/openbmc/skeleton/pull/60.
This new method does not report any error message when 'fru_area_name'
not defined in skeleton.
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/14)
<!-- Reviewable:end -->
https://github.com/openbmc/ipmi-fru-parser/pull/14
Yi Li (1):
Using new getFRUOjbect() API to fix 'not found in lookup' error
message
writefrudata.C | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
--
2.7.1
More information about the openbmc
mailing list