All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH ipmi-fru-parser v2] Using new getFRUOjbect() API to fix 'not found in lookup' error message
@ 2016-04-15  8:10 OpenBMC Patches
  2016-04-15  8:10 ` [PATCH ipmi-fru-parser v2] Using new getFRUArea() " OpenBMC Patches
  0 siblings, 1 reply; 2+ messages in thread
From: OpenBMC Patches @ 2016-04-15  8:10 UTC (permalink / raw)
  To: openbmc

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@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 getFRUArea() API to fix 'not found in lookup' error message

 writefrudata.C | 91 ++++++++++++++++++++++++++++++++++++++++++++++++++++++----
 1 file changed, 86 insertions(+), 5 deletions(-)

-- 
2.7.1

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-04-15  8:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-15  8:10 [PATCH ipmi-fru-parser v2] Using new getFRUOjbect() API to fix 'not found in lookup' error message OpenBMC Patches
2016-04-15  8:10 ` [PATCH ipmi-fru-parser v2] Using new getFRUArea() " OpenBMC Patches

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.