Hey Dan, Finally got a chance to look through your patch. For the most part it's fine. I'll tweak a few formatting things to be consistent to other parts of FreeIPMI, and I think I'll do "first" and "last" instead of "0" or "-1" for the first/last SEL entry. But I'm a tad confused on this in your printf at the end:
data_len == text_len ? "" : " ... (text incomplete)"); Is there some subtlety of the get-sel-long-text OEM command that isn't clear to me (b/c I don't own one and haven't seen it in action)?? Al On Sun, 2010-08-15 at 20:35 -0700, Dan Lukes wrote: > Hi. > > The Fujitsu's iRMC has implemented proprietary function that return text > representation of SEL. It's important especially for OEM-Reserved > events. For example, the ipmi-sel show something like this: > > > ipmi-sel --interpret-oem-data --display=1008 > > ID | Date | Time | Name | Type > > | Event > > 1008 | Dec-16-2009 | 15:40:29 | iRMC | OEM Reserved > > | Event Offset = 00h ; OEM Event Data2 code = 83h ; OEM Event Data3 code = > > 4Dh > or > > ipmi-sel --interpret-oem-data --display=1032 > > ID | Date | Time | Name | Type > > | Event > > 1032 | Dec-16-2009 | 15:41:42 | iRMC | System Firmware Progress > > | System Firmware Error ; Unspecified ; OEM Event Data3 code = 5Fh > > utilising the OEM function we can get something like this: > > > ipmi-oem -- fujitsu get-sel-entry-long-text 1008 > > 1008 | Dec-16-2009 | 15:40:29 | INFORMATIONAL: Online firmware flash EEPROM > > 2 Version 3.77 > or > > ipmi-oem -- fujitsu get-sel-entry-long-text 1032 > > 1032 | Dec-16-2009 | 15:41:42 | Server Management Configuration NVRam bad, > > default configuration is used > > The implementation si based on (not so good) documentation: > http://*manuals.ts.fujitsu.com/file/4390/irmc-s2-ug-en.pdf > > The patch that add the support for OEM/fujitsu/get-sel-entry-long-text > into ipmi-oem is attached. It's tested on iRMC with firmware 3.77 > > In the fact, it would be better to use the function as part of > interpret-oem-data within ipmi-sel, but it require better knowledge of > current implementation of ipmi-sel than I have. Someone with better > skills may implement it (using the code from patch). Then I can help > with testing, if necessary. > > Hope it help, but you can freely ignore this message if not interested. > > Dan > -- Albert Chu [email protected] Computer Scientist High Performance Systems Division Lawrence Livermore National Laboratory _______________________________________________ Freeipmi-devel mailing list [email protected] http://lists.gnu.org/mailman/listinfo/freeipmi-devel
