summaryrefslogtreecommitdiffstats
path: root/chassishandler.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Remove useless wildcard handlersVernon Mauery2020-02-061-16/+0
| | | | | | | | | The default action is to return Command Not Found, which is exactly what the wildcard handlers are doing. This just removes that extra code that is not needed. Change-Id: I0a25153395858efcdf56384ececaabc7eaf4e1af Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* chassishandler:Get Chassis Capabilities to new APIanil kumar appana2020-01-141-79/+75
| | | | | | | | | | | | | | | Rewrite "get chassis capabilities" command to use newly introduced IPMI provider API. Tested: verified using ipmitool raw command. Command: ipmitool raw 0x00 0x00 // get chassis capabilities command Response: 06 20 20 20 20 20 Change-Id: I16f7efec58f438f1f04392b246b27ab665a65ed5 Signed-off-by: anil kumar appana <anil.kumarx.appana@intel.com> Signed-off-by: jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>
* chassis: remove startup dependenciesJames Feist2019-12-041-11/+29
| | | | | | | | | | | Currently, on start if services aren't avaialble ipmid throws and crashes. This changes the lookup of settings to when the command is actually called. Tested: ipmi chassis status still works Change-Id: I2106270d3f6f6bdd02ad67f7dface3bf04f98a3d Signed-off-by: James Feist <james.feist@linux.intel.com>
* Implement NMI command in Chassis Control command.Kuiying Wang2019-11-131-0/+36
| | | | | | | | | | | | | | Set NMI (Diagnostic Interrupt) signal source to Chassis command. Please refer ipmi spec 1.3 table 1 for details. Set Enabled property to inform NMI source handling to trigger a NMI_OUT BSOD. Tested: NMI is actived(host is hang) due to NMI command ipmitool chassis power diag Change-Id: I8066aa19ba1f5a81897857de2a5fe551876101b3 Signed-off-by: Kuiying Wang <kuiying.wang@intel.com>
* Add compile option to disable safe mode option in boot flagsYong Li2019-11-061-0/+4
| | | | | | | | | | | | | | | | | | Some platforms doe not support safe mode in set/get system boot options commands, add an compile option to disable it. Using the below command: ./configure --disable-boot-flag-safe-mode-support Tested: After adding this compile option, below command fails: // set safe mode in set system boot options command ipmitool raw 0 8 5 0x80 0xc Unable to send RAW command (channel=0x0 netfn=0x0 lun=0x0 cmd=0x8 rsp=0xcc): Invalid data field in request Signed-off-by: Yong Li <yong.b.li@linux.intel.com> Change-Id: Ie479a0c68a8e9fc79f99c9d4cc10cdd1e6ceb20e
* chassishandler: move set chassis cap to new apianil kumar appana2019-07-251-53/+54
| | | | | | | | | | | | | | | Rewrite set chassis capabilities to use new ipmi provider api Tested: verified ipmitool raw get chassis capabilities command ipmitool raw 0 5 00 20 20 20 20 20 // set chassis cap command ipmitool raw 0 0 // get chassis cap command 00 14 14 14 14 14 // updated response in hex Signed-off-by: anil kumar appana <anil.kumarx.appana@intel.com> Change-Id: Idcc7c12250d4465ff838c15ba0d511701688a3a2
* Use xyz.openbmc_project.State.Chassis for IPMI chassis statusJason M. Bills2019-06-241-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of directly using pgood on dbus, this change uses the xyz.openbmc_project.State.Chassis "CurrentPowerState" property for the IPMI chassis status command. This will allow us to remove pgood from dbus. Tested: Ran IPMI chassis commands and confirmed that they behave as expected: ipmitool power status Chassis Power is on ipmitool power off Chassis Power Control: Down/Off ipmitool power status Chassis Power is off ipmitool power on Chassis Power Control: Up/On ipmitool power status Chassis Power is on Change-Id: I7836c16b76c3b309f176186f3e2453082e4cd1af Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
* Fix power restore policy input parameter checkJason M. Bills2019-06-241-9/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes the power restore policy command to return invalid parameter for any unsupported policy option (greater than 0x03). Tested: ipmitool raw 0 6 0 07 ipmitool chassis status Power Restore Policy : always-off ipmitool raw 0 6 1 07 ipmitool chassis status Power Restore Policy : previous ipmitool raw 0 6 2 07 ipmitool chassis status Power Restore Policy : always-on ipmitool raw 0 6 3 07 ipmitool chassis status Power Restore Policy : always-on ipmitool raw 0 6 4 Unable to send RAW command (channel=0x0 netfn=0x0 lun=0x0 cmd=0x6 rsp=0xcc): Invalid data field in request Change-Id: Ie78e00753f6dfd8b7bccce38c239a8103b6cc3fd Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
* Chassishandler: Add chassis identify state support for chassis status commandYong Li2019-05-241-1/+6
| | | | | | | | | | | | | | | | | | | | Return the present chassis identify state in get chassis status command: Off Temporary on Indefinite on Reserved Tested: Set chassis identify in different states: ipmitool raw 0 4 10 0 ipmitool raw 0 4 0 1 ipmitool raw 0 4 0 0 Then query the chassis status and check on the third byte ipmitool raw 0 1 Signed-off-by: Yong Li <yong.b.li@linux.intel.com> Change-Id: I66690b74823a6b819c4fda25d95bc8690b98181a
* chassishandler: add AC failed bit support for chassis statusYong Li2019-05-231-1/+33
| | | | | | | | | | | | | PFail property is provided by power control service, true means the booting is from AC power loss. Tested: Remove the AC cable and reconnect it to power on the system ipmitool chassis status will return: Last Power Event : ac-failed Change-Id: I3743f15da259491052ee8fc869205438e79046f8 Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
* Add support for removable media/FloppyJia, chunhui2019-05-231-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | 1. fix issue for handling unsupported parameter 2. add support for floppy/USB boot There is dbus interface change in: https://gerrit.openbmc-project.xyz/c/openbmc/phosphor-dbus-interfaces/+/21177 Test: $ ipmitool -H <ip> -P 0penBmc -I lanplus raw 0x0 0x8 0x5 0x80 0x14 0x00 0x00 0x00 $ ipmitool -H <ip> -P 0penBmc -I lanplus chassis bootparam get 5 ... - Boot Device Selector : Force Boot from CD/DVD ... $ipmitool -H <ip> -P 0penBmc -I lanplus raw 0x0 0x8 0x5 0x80 0x3c 0x00 0x00 0x00 $ipmitool -H <ip> -P 0penBmc -I lanplus chassis bootparam get 5 ... - Boot Device Selector : Force Boot from Floppy/primary removable media ... Change-Id: I915aaa3f684d02d0dfc4b65afb3c1db139c41781 Signed-off-by: Jia, chunhui <chunhui.jia@linux.intel.com>
* Get Chassis Status should not bail if button interfaces are not presentVernon Mauery2019-05-161-8/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The recent rewrite of Get Chassis Status also added support for reading the front panel enables status instead of returning a hard-coded false. But the implementation also errors out if the interface is not present This makes the interfaces optional, returning false if they are not presnt and reading them correctly if they are. Tested-by: running ipmitool chassis status with and without the xyz.openbmc_project.Chassis.Buttons service running: # ipmitool chassis status System Power : on Power Overload : false Power Interlock : inactive Main Power Fault : false Power Control Fault : false Power Restore Policy : previous Last Power Event : Chassis Intrusion : inactive Front-Panel Lockout : inactive Drive Fault : false Cooling/Fan Fault : false Sleep Button Disable : not allowed Diag Button Disable : not allowed Reset Button Disable : allowed Power Button Disable : allowed Sleep Button Disabled: false Diag Button Disabled : false Reset Button Disabled: false Power Button Disabled: false # systemctl stop xyz.openbmc_project.Chassis.Buttons@0.service # ipmitool chassis status System Power : on Power Overload : false Power Interlock : inactive Main Power Fault : false Power Control Fault : false Power Restore Policy : previous Last Power Event : Chassis Intrusion : inactive Front-Panel Lockout : inactive Drive Fault : false Cooling/Fan Fault : false Front Panel Control : none Change-Id: If845194b6f052ba84f8b062ac4259ec66f706bb5 Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* rewrite Get Chassis Status commandVernon Mauery2019-05-141-142/+198
| | | | | | | | | | | | | | The Set Front Panel Enables command touches one of the same interfaces that this command does, so rewrite this command to reflect the changes that set front panel enables causes. Tested-by: manually run get chassis status command via ipmitool: # ipmitool raw 0 1 01 00 40 30 # ipmitool chassis status Change-Id: Ifbb13a6f344c58b2140933447995d3854cca0c0d Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* ipmid: Update set chassis power restore policyVernon Mauery2019-05-141-45/+28
| | | | | | | | | | | | | | Use the new ipmi api, including the yield context mechanism. Modify to use the new provider API. One by one, change calls to remove any legacy API constructs. Tested-by: ipmitool chassis policy list ipmitool chassis policy previous ipmitool chassis policy always-on ipmitool chassis policy always-off Change-Id: I02a955c3946b940daaa3fe90c2a220b0eb719bad Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* remove usage of sdbusplus::message::variantVernon Mauery2019-05-141-7/+5
| | | | | | | | | | | sdbusplus has had its alias of std::variant in place for long enough. This changes all ipmid references to use std::variant directly instead of the sdbusplus alias. Tested-by: building and running ipmid Change-Id: Id5b4136d4589aa598815edd3ef4202e64a7698e2 Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* chassishandler: move chassis control to new APIanil kumar appana2019-05-071-22/+15
| | | | | | | | | | | | | | Rewritten "Chassis control" command to the use newly introduced IPMI provider API. Tested: Verified ipmitool raw chassis control command ipmitool raw 0 2 0/1/2/3/4/5 and for invalid value got invalid field in request response. Change-Id: I0213157640e34c693b53999f69d7d47ced950e0d Signed-off-by: anil kumar appana <anil.kumarx.appana@intel.com> Signed-off-by: Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com>
* chassishandler: move get POH counter to new APIanil kumar appana2019-05-031-29/+17
| | | | | | | | | | | | | | Rewritten "Get POH counter" command to use the newly introduced IPMI provider API. Tested: verified using ipmitool chassis poh ipmitool chassis poh POH Counter : 1 days, 6 hours // Response Change-Id: Ife250269da19e511783f1a0cf1298a366f3c2728 Signed-off-by: anil kumar appana <anil.kumarx.appana@intel.com> Signed-off-by: Richard Marian Thomaiyar <richard.marian.thomaiyar@linux.intel.com>
* move variant to std namespaceVernon Mauery2019-04-181-25/+22
| | | | | | | | | | | sdbusplus::message::variant_ns has been std for a while now. This moves ipmid away from sdbusplus::message::variant_ns to directly use std::variant. Tested-by: built, compiles, and runs the same as before. Change-Id: I8caa945f31c926c2721319f001b9d7f83fd3f1b7 Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* ipmid: update chassis identify commandVernon Mauery2019-04-091-26/+16
| | | | | | | | | | | | Modify to use the new provider API. One by one, change calls to remove any legacy API constructs. Tested-by: ipmitool chassis identify ipmitool chassis identify 0 ipmitool chassis identify force Change-Id: I91bc21fdaef74eb32f297e0fa54edbc2e1bf4c2b Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* Only include ipmid/api.hpp for the new APIVernon Mauery2019-04-081-1/+1
| | | | | | | | | | | | | | After some feedback from users of the new IPMI API, they wanted to see two things: 1) don't require ipmid/api.hpp and ipmid/registration.hpp to be able to write new handlers 2) only require including ipmid/api.hpp (instead of ipmid/api.h) So now, by simply including ipmid/api.hpp instead of ipmid/api.h (deprecated), handlers incorporating the new IPMI API can be written. Change-Id: I446dcce70cff03d4ecc28c658292d052485f77fc Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* remove legacy ipmid.cpp and ipmid.hppVernon Mauery2019-04-011-1/+0
| | | | | | | These two files have now been replaced with newer APIs and can be removed. Change-Id: If60227b24188c6796211ca43379169f9a95c9cb3 Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* Remove experimental filesystem spliceVernon Mauery2019-03-281-13/+1
| | | | | | | | | | In order to be compatible with older versions of GCC that did not have the experimental filesystem moved over to the standard paths, the code would splice the experimental code into the std namespace. This is no longer necessary with yocto 2.6 and the latest versions of GCC. Change-Id: I2db13c52a91456318795819f2d45c3386b4c56d2 Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* move types.hpp to ipmid/types.hpp for exportVernon Mauery2019-03-251-1/+1
| | | | | | | | | | types.hpp is required by utility.hpp, which is exported, so it needs to be exported as well. This moves it to the include/libipmid directory, changes the Makefile to export it, and changes all the files that include it so it can be found in the right place. Change-Id: I30ec365446e4de466c266ec4faa327478460ec05 Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* Move util.cpp/util.hpp to libipmidVernon Mauery2019-03-251-1/+1
| | | | | | | | | | | | | | | | These are functions that are used widely by ipmid providers, so it makes sense to put them in libipmi.so (the library that all providers must link against). Tested-by: use nm to inspect the binaries to see that the symbols are in the expected library. arm-openbmc-linux-gnueabi-nm libipmid.so.0.0.0 \ | grep getDbusObject 0001063c T _ZN4ipmi13getDbusObjectERN9sdbusplus.... Change-Id: I1221f807f2711c5301c5574623564ea1ae48a437 Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* Extra Bytes in response data for chassis Identify commandJia, Chunhui2019-03-041-0/+1
| | | | | | | | | | From IPMI spec, response should be complete code only. Tested: ipmitool raw 0x00 0x04 0x02 0x00 Change-Id: I48a9f71e01c89ee1125d515c50e3e2655510d0ce Signed-off-by: Jia, Chunhui <chunhui.jia@linux.intel.com>
* Set init_priority attribute for global variables in shared libLei YU2019-02-121-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | The code gets unspecified initializatio order for the global/static variables in a shared library. If unluck, a global/static variable may be initialized in *constructor* function, and then initialized by the default contructor. For exmaple, if `std::unique_ptr<xxx> var{nullptr};` is initialized in constructor function, below init order may occur: 1. It is initialized in constructor; 2. Then it is initialized as nullptr; And eventually when the code is to use the variable, we got nullptr. We met such issues before on openbmc/openbmc#1581, and the technical details could be found at [this SO question][1] and [this gcc email][2] The solution is to specify the init_priority attribute, to make the global/staic variables inititalize earlier than the contructors. [1]: https://stackoverflow.com/questions/43941159/global-static-variables-initialization-issue-with-attribute-constructor-i [2]: https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00863.html Change-Id: I901a6a5cddec12aec9512fe58b16735fa2ad90d7 Signed-off-by: Lei YU <mine260309@gmail.com>
* Create libipmid and libipmid-hostWilliam A. Kennington III2019-02-071-1/+1
| | | | | | | | | | | | | | | | | | | This starts a transition to common ipmid libraries that providers can link against. It will allow for a cleaner separation between common ipmid functionality and daemon type specific code. This is needed so we can resolve all of the symbols in the providers at link time instead of discovering bad linkage by building and running a full ipmi daemon. In future commits libraries will be packaged for libipmid and libipmid-host which provide all of the symbols used by the current set of ipmid providers. This is the first step, it just separates and renames the headers. Legacy symlinks are still kept around for compatability. It also adds stub libraries so that external users can start linking as intended. Change-Id: I6bbd7a146362012d26812a7b039d1c4075862cbd Signed-off-by: William A. Kennington III <wak@google.com>
* Changes to align with IPMI specYong Li2019-02-051-4/+5
| | | | | | | | | | | | IPMI Spec Section 19.2: Unless otherwise noted, Reserved bits and fields in commands(request messages) and responses shall be written as '0'. Return "out of range" error for unsupported power policy and also return the supported policies on success. Change-Id: I51d63b86ef0d3076438649abc49c63267c57cbd5 Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
* Remove unused legacy code referencing org.openbmc.settings.HostTom Joseph2018-12-201-134/+0
| | | | | | | Resolves openbmc/phosphor-host-ipmid#115 Change-Id: Iab7f878b8b3011caa95f532f99062af9cbc1c33c Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Implement set chassis capabilities IPMI commandsYong Li2018-12-111-26/+183
| | | | | | | | | | | | Create a new dbus interface for the chassis capabilities, and store the chassis capabilities using phosphor settings manager Tested By: ipmitool -H $BMCIP -U root -P 0penBmc -I lanplus raw 0 5 0 2 4 6 8 0xa ipmitool -H $BMCIP -U root -P 0penBmc -I lanplus raw 0 0 Change-Id: Id01555358adfec3814fd03b3e6d422916c861039 Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
* Convert variant usage to std interfaceWilliam A. Kennington III2018-10-181-16/+20
| | | | | | | | | | | | | This is just a refactoring to use the c++17 std::variant interfaces instead of the mapbox::variant specific ones. We should be able to use mapbox::variant and std::variant interchangeably now. Tested: Built against sdbusplus with mapbox::variant and sbusplus using std::variant. Both variant compile and test out. Change-Id: I6fbaad3d12dd34968db6a10f3d74a65e07d0f0cc Signed-off-by: William A. Kennington III <wak@google.com>
* Use the common timer classVernon Mauery2018-10-181-12/+10
| | | | | | | | | The common timer class from sdbusplus offers all the timer goodness that we currently use. The unit test is also no longer needed (and has been added to sdbusplus's version of the timer.hpp implementation). Change-Id: I278817489433a29ca739f70fdacd8bb897797d66 Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>
* add std namespace prefix to common methodsPatrick Venture2018-09-251-28/+30
| | | | | | | | Standard clib methods should be explicitly grabbed from the standard namespace. Change-Id: I8bcfcb260a99180d535ec8102c1a24c318cbc209 Signed-off-by: Patrick Venture <venture@google.com>
* move filesystem inclusion to bottom of listPatrick Venture2018-09-251-15/+14
| | | | | Change-Id: I4aa3c984992236bb5b4a5c62c3d42a33c12d55de Signed-off-by: Patrick Venture <venture@google.com>
* replaced c headers with cpp where applicablePatrick Venture2018-09-251-4/+0
| | | | | Change-Id: I23a70eb540ccde5d2aba467426769feffb07b516 Signed-off-by: Patrick Venture <venture@google.com>
* rename headers to match stylePatrick Venture2018-09-251-4/+5
| | | | | | | | | | | | | | | | | | Moving headers from ".h" to ".hpp" Reworked the header inclusion a bit so that host-ipmid is treated as a library, and local headers aren't. renamed apphandler.h => apphandler.hpp renamed chassishandler.h => chassishandler.hpp renamed globalhandler.h => globalhandler.hpp renamed sensorhandler.h => sensorhandler.hpp renamed storageaddsel.h => storageaddsel.hpp renamed storagehandler.h => storagehandler.hpp renamed systemintfcmds.h => systemintfcmds.hpp Change-Id: I9d4ce3dd57e2e996800f9020a10cc10cdf2c3914 Signed-off-by: Patrick Venture <venture@google.com>
* Implement chassis set power restore policy commandYong Li2018-09-251-0/+91
| | | | | | | | | | | | | | | | | | Implement the IPMI set power restore policy command 0x06. This command can be used to configure the power restore policy. This configuration parameter is kept in nonvolatile storage. The power restore policy determines how the system or chassis behaves when AC power returns after an AC power loss Tested: Run the below command to check the current Power Restore Policy: ipmitool -H <IP> -P 0penBmc -I lanplus chassis status Run the below command to change it: ipmitool -H <IP> -P 0penBmc -I lanplus chassis policy always-off Change-Id: I224912890f9a9e8b4dc98f840cd6f223c9f7dfe5 Signed-off-by: Yong Li <yong.b.li@linux.intel.com>
* add .clang-formatPatrick Venture2018-09-071-365/+365
| | | | | Change-Id: I7c2a527b4751a560703a61fcbe9638b150546af5 Signed-off-by: Patrick Venture <venture@google.com>
* Handle the case to turn off the Chassis Identify LEDTom Joseph2018-08-081-0/+6
| | | | | | | Resolves openbmc/openbmc#3327 Change-Id: Ib0cbfccdd91b6cdd2267a48b312d2736a4ce8fbc Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Standardize use of filesystemVernon Mauery2018-08-081-1/+11
| | | | | | | | | If <filesystem> exists, use that, if it doesn't, try <experimental/filesystem>. Either way, access it from std via a namespace splice. Change-Id: I5b35ee9f74b88e188cb3ca3fed9abaec64933697 Signed-off-by: Vernon Mauery <vernon.mauery@intel.com>
* Fix D-Bus error from callback method invoked from async threadMarri Devender Rao2018-05-231-74/+105
| | | | | | | | | | | | | Noticed D-bus method invoked as part of the callback method invoked from async thread returns error Switching to use sd_event loop timer for callback after timer expiry Resolves openbmc/openbmc#3130 Change-Id: Ibe87a6b3aa179cc887593c7dea635c11d9ea844c Signed-off-by: Marri Devender Rao <devenrao@in.ibm.com>
* Implement Get POH Counter commandNagaraju Goruganti2018-05-211-0/+71
| | | | | | | | | | It returns a counter value proportional to the system operating power-on hours. Partially Resolves openbmc/openbmc#2979 Change-Id: Ib0cd43fe5cbc055ae84991577d766bedae58d775 Signed-off-by: Nagaraju Goruganti <ngorugan@in.ibm.com>
* Fix get boot options returning unexpected resultsMarri Devender Rao2018-05-081-0/+16
| | | | | | | | | | | | | | | | | | | | | | | Get boot options is returning wrong value when boot device is set to cdrom followed by setting boot device to bios. In the IPMI spec the boot device selector has options for both "boot source" (default/disk/crdom/network) and "boot mode" (default/bios/safe/diag). Since this isn't a bit field, you can't override for eg both the source and mode at the same time. The solution is if a set boot device is mapping to a boot source, then reset the boot mode D-Bus property to default, or vice versa. This way the ipmid code can determine which property is not at the default value Resolves openbmc/openbmc#3067 Resolves openbmc/openbmc#3068 Change-Id: I072c82a161d56f118510fe1f9e5c75cb5c610295 Signed-off-by: Marri Devender Rao <devenrao@in.ibm.com>
* Add set boot source and boot mode methods for reuseMarri Devender Rao2018-05-081-39/+62
| | | | | | | | | setBootMode and setBootSource methods are introduced by refactoring code from ipmi_chassis_set_sys_boot_options for code reuse. Change-Id: I30b641239686b0411a0995468d098f5ad3d61754 Signed-off-by: Marri Devender Rao <devenrao@in.ibm.com>
* Reducing IPMI logging footprintAditya Saripalli2018-04-181-40/+39
| | | | | | | | | | | | -host-ipmid is very noisy in terms of journal logging. A small step towards cleaning that up. -Also converted printfs to phosphor-logging. Partially Resolves openbmc/openbmc#2507 Change-Id: I749c19c18d1cabf6f0216830c8cb0a08ee43d6de Signed-off-by: Aditya Saripalli <aditya0124@gmail.com> Signed-off-by: Nagaraju Goruganti <ngorugan@in.ibm.com>
* Revert "Revert "Add chassis identify ipmi command.""Tom Joseph2018-04-041-0/+115
| | | | | | | | | | This reverts commit 82feb17d0c244ed04b82b0fbed3c0a9456986a86. The fix for armv5 futures is provided in yocto 2.4 Resolves openbmc/openbmc#3010 Change-Id: Ifc7e224f6f1fa115e0299642e8866b82d6e4f38c
* Revert "Add chassis identify ipmi command."Joel Stanley2018-02-281-115/+0
| | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 5bfbde351aabe2096de99ee959a8a2e50edcf84c. See https://github.com/openbmc/openbmc/issues/2740 for details. This commit causes ipmid to fail to load. From github user artemsen: > ipmid[995]: ERROR opening [/usr/lib/host-ipmid/libapphandler.so.0]: > /usr/lib/host-ipmid/libapphandler.so.0: undefined symbol: _ZTINSt13__future_base12_Result_baseE As reported by Lei: > This issue prevent all P8 systems (with ast2400) booting with latest > OpenBMC, because ipmid fails to load libapphandler.so and cause watchdog > will not be kicked and always timeout on power on. As suggested by James Feist: > We probably should just revert the changes until we update to yocto > 2.4 which has the newer version of GCC with the fix for armv5 futures > as the backport seemed to fail. Change-Id: I0059cc3f939c1e3e5f88531fcb14ae3faa451387 Tested: Booted a ast2400 machine and checked journal Signed-off-by: Joel Stanley <joel@jms.id.au>
* Add chassis identify ipmi command.James Feist2017-12-131-0/+115
| | | | | | | | | Add command with support for chassis identify. Resolves openbmc/openbmc#2621 Change-Id: Ie656b71eba68141117838363456c6e075225e3e6 Signed-off-by: James Feist <james.feist@linux.intel.com>
* Spelling fixesGunnar Mills2017-10-311-1/+1
| | | | | | | | | Spelling errors found using github.com/lucasdemarchi/codespell A tool to fix common misspellings. This tool is licensed under GNU General Public License, version 2. Change-Id: I78e9d126f19fd9fb2e3f2fa9fe8c2404dc505f82 Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
* boot policy: Support boot parameter #4 (boot info)Tom Joseph2017-10-041-2/+9
| | | | | | | Resolves openbmc/openbmc#2356 Change-Id: If775a01a97f29a60575c03f26d32bba0315af920 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
OpenPOWER on IntegriCloud