summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* ipmiwhitelist: Change .H -> .hppPatrick Williams2016-09-063-2/+2
| | | | | Change-Id: I1b4d00211913d1be61f33dfb02227ffc8f859584 Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* Add bootstrap.shPatrick Williams2016-09-063-7/+23
| | | | | Change-Id: Id01b9ad969f96b0f79623ad9cf2a06f6714a07b1 Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* Minor reorder and simplification of autotools filesPatrick Williams2016-09-022-22/+24
| | | | | Change-Id: I55db0cf7f6041890564f557c98286c8b4bde80cb Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* Convert build process to autotoolsMatthew Barth2016-09-024-85/+99
| | | | | | | | | Replaced the use of manual Makefile(s) with the use of autotools to automatically verify and generate the necessary build files. Follow the steps outlined within the README file to build the package. Change-Id: Ia3e1e1be7b18fb90382fd56f555866120ae5ece9 Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
* Implement Network Settings OverrideRatan Gupta2016-09-023-20/+463
| | | | | | | | | | | | | | | | | | | | | | We want to implement the network settings override functionality. This is an OEM boot parameter selector, using parameter ID 0x61. The first byte of the parameter data is the same as parameter ID 5 (ie, the valid flag in the top bit). Persistency doesn't matter here, as petitboot will not clear out this setting. The rest of the parameter data is a host network settings description. For other BMC firmware, the BMC just treats this as opaque (the BMC will just return the same data (including that data1 byte), in response to a Get System Boot Option). For OpenBMC, we may want the BMC to know how to construct that data; the spec for that is at: http://ozlabs.org/~sam/ipmi-network-format Closes openbmc/openbmc#267 Change-Id: I2af8776718deda97acf90d10504783085ccf7323 Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>
* Hard-coded service name replaced with call to ObjectMapperSergey Solomin2016-08-257-28/+107
| | | | | | | | | | | | | In the openbmc code, the dbus service name is hardcoded. The ObjectMapper provides a way to find out the service name by issuing a GetObject method call to the ObjectMapper. That functionality is implemented in mapper.c The output of the call to the mapper is a service name. Resolves openbmc/openbmc#319. Change-Id: I724a31ede6bdf3ae4b361e730857b1f2696a8966 Signed-off-by: Sergey Solomin <sergey.solomin@us.ibm.com>
* Correct whitelist generated source file extensionMatthew Barth2016-08-101-1/+1
| | | | | Change-Id: I9480ef48c410708cf2b5fb97c4ac93ef486c6cd2 Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
* Ensure Get-Device-ID conforms to IPMI specNan Li2016-08-101-13/+59
| | | | | | | | | | | | | | * Refactor response from constant array to explicitly commented fields. * Update comments for fields. * Refer to FSP implementation where appropriate. Note: Some of the fields present here should be filled in from MRW or runtime settings in the future. Resolves openbmc/openbmc#435. Change-Id: I733274e237cf28d5048c44d527a8fd67a0392168 Signed-off-by: Nan Li <william.bjlinan@hotmail.com>
* Switch C++ source files to .cpp extensionMatthew Barth2016-08-0814-1/+1
| | | | | Change-Id: I495186b4afd58ec1e5d9d5c6472394f96541d17d Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
* Whitelist IPMI commands based on Restricted modeTom Joseph2016-08-086-4/+159
| | | | | | | | | | | | | | Whitelisting of IPMI commands is done to ensure that in restricted mode only whitelisted commands are executed. Commands that are not whitelisted is restricted and insufficient privilege is returned as the completion code. When the server is deployed it would be set to restricted mode. In this scenario certain IPMI commands need to be restricted which would not be added to the whitelist. Change-Id: I90b8124e34263c4ffc5bcf06a28a7e88231aaf40 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Use mapper calls to lookup objectsBrad Bishop2016-07-273-94/+28
| | | | | | | | | This is a reaction to a managers.System API update. The update removes the bus name field from the 'GetObject' method responses. Change-Id: I51fb1d1bad20317a429e5b06712475929b6fb4fb Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Add support to load versioned librariesAdriana Kobylak2016-07-102-3/+11
| | | | | | | | | | | | Linux libraries, including the ipmi plugins, can be versioned, with suffix .so.* Add support in ipmid to find and load versioned libraries. There seems to not be an issue if ipmid loads the same library twice (a non-versioned and versioned) as with the cases where a symlink .so points to a .so.* library. Change-Id: I96d73e90dfec01c1a830dab4e6c20797b6130528 Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
* Merge pull request #95 from tomjoseph83/issue80Patrick Williams2016-06-302-2/+4
|\ | | | | Fix two issues in Issue #80
| * Fix two issues in Issue #80tomjose2016-06-302-2/+4
|/ | | | | | | | Corrected the comment for the Get Channel Info command to reflect the logic in the code. For issue two, corrected the condition for returning the unspecifed error. Fixes openbmc/phosphor-host-ipmid#80
* Merge pull request #94 from anoo1/bcdPatrick Williams2016-06-221-1/+3
|\ | | | | Fix minor fw revision to be BCD encoded
| * Fix minor fw revision to be BCD encodedAdriana Kobylak2016-06-221-1/+3
|/ | | | | | | | | The IPMI 2.0 spec requires the minor version revision to be BCD encoded, but the current implementation uses a binary encoding. Fixes openbmc/phosphor-host-ipmid#89 Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
* Merge pull request #93 from bradbishop/mapper-renamePatrick Williams2016-06-152-6/+6
|\ | | | | Update ObjectMapper service/interface/path
| * Update ObjectMapper service/interface/pathBrad Bishop2016-06-142-6/+6
|/ | | | | | | The well known service name, interface and path for the mapper all changed. This patch reacts to that. Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Merge pull request #42 from shenki/travisPatrick Williams2016-06-062-0/+29
|\ | | | | travis: Add Travis CI building
| * travis: Add Travis CI buildingJoel Stanley2015-11-242-0/+29
| | | | | | | | | | | | Builds on Ubuntu 15.10 under docker. Signed-off-by: Joel Stanley <joel@jms.id.au>
* | Merge pull request #88 from williamli80/issue77Patrick Williams2016-06-061-0/+4
|\ \ | | | | | | Returning correct value from handle_ipmid_command in ipmid.C
| * | Returning correct value from handle_ipmid_command in ipmid.CNan Li2016-05-231-0/+4
| | | | | | | | | | | | | | | | | | Instead of -1, we return 0xFF(Unspecified error) to the bus. Signed-off-by: Nan Li <bjlinan@cn.ibm.com>
* | | Merge pull request #87 from williamli80/story76_2Patrick Williams2016-06-061-4/+31
|\ \ \ | | | | | | | | Cleanup phosphor-host-ipmid storageaddsel.C
| * | | Cleanup phosphor-host-ipmid storageaddsel.CNan Li2016-05-181-4/+31
| |/ / | | | | | | | | | | | | | | | 1.Check returns and handle errors for file IO. Signed-off-by: Nan Li <bjlinan@cn.ibm.com>
* | | Merge pull request #83 from ratagupt/146621Patrick Williams2016-06-061-14/+58
|\ \ \ | | | | | | | | Adding Boot Policy
| * | | Adding Boot Policyratagupt2016-04-191-14/+58
| | | | | | | | | | | | | | | | Review Comment added for 146621
* | | | Merge pull request #82 from williamli80/code0308Patrick Williams2016-05-243-8/+20
|\ \ \ \ | | | | | | | | | | Modify eSEL to follow IPMI spec:
| * | | | Handle multiple attempts at ipmi reservation idsNan Li2016-04-203-8/+20
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Keep Reservation ID follow IPMI Spec. 2. Make Reservation ID increase sequentially. 3. Make Reservation ID available in shared librarys. Signed-off-by: Nan Li <bjlinan@cn.ibm.com>
* | | | Merge pull request #86 from williamli80/issues75Patrick Williams2016-05-242-0/+15
|\ \ \ \ | |_|/ / |/| | | Handle errors finding openbmc_path #75
| * | | Handle errors finding openbmc_path #75Nan Li2016-05-172-0/+15
|/ / / | | | | | | | | | | | | | | | 1.Check return value properly. Signed-off-by: Nan Li <bjlinan@cn.ibm.com>
* | | Merge pull request #85 from causten/deviceidnkskjames2016-05-031-13/+14
|\ \ \ | | | | | | | | ipmid crashing with simplified VERSION_ID
| * | | ipmid crashing with simplified VERSION_IDChris Austen2016-05-021-13/+14
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | if the VERSION_ID is v0.1 instead of v0.1-1 then the code will crash. The fix is to always check for a NULL token each time. Change ipmitool output to reflect hex value VERSION_ID is equal to v0.6-36-gece9f78-dirty I was forcing the '36' to be shown as 36. Turns out that the ipmitool prepends the "Aux" section with a '0x' indicating the value is in hex. So Rather then it show up as 0x36 I made it actually be 0x24. root@barreleye:~# ./ipmitool -I dbus mc info IPMI GROUP EXTENTIONS IPMI GROUP EXTENTIONS Device ID : 0 Device Revision : 0 Firmware Revision : 0.06 IPMI Version : 2.0 Manufacturer ID : 42817 Manufacturer Name : Unknown (0xA741) Product ID : 16451 (0x4043) Product Name : Unknown (0x4043) Device Available : yes Provides Device SDRs : no Additional Device Support : Sensor Device SEL Device FRU Inventory Device Aux Firmware Rev Info : 0x00 0x24 <---- is decimal 36 0x01 0x00 root@barreleye:~#
* | | Merge pull request #84 from causten/deviceidnkskjames2016-04-291-3/+88
|\ \ \ | |/ / |/| | Adjust ipmi device id to accept dbus versioning
| * | Adjust ipmi device id to accept dbus versioningChris Austen2016-04-181-3/+88
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before this the ipmi device id would return ipmitool -Idbus mc info ... Firmware Revision : 0.06 Now it will track the version that you get from the version property of /org/openbmc/inventory/system/chassis/motherboard/bmc i.e. version = v0.6-19-gf363f61-dirty ^ ^ ^^ ^ | | |----------|-- additional details | |---------------- Minor |------------------ Major Now... ipmitool -Idbus mc info ... Firmware Revision : 0.06 ... Aux Firmware Rev Info : 0x00 0x19 0x00 0x01 If there is no -dirty (or anything really after the last '-') will force the aux details byte 4 to 0. A 0 means the FW image was created without any changes in the yocto build.
* | Merge pull request #74 from tomjoseph83/ipmi_lan_1nkskjames2016-03-082-2/+1
|\ \ | | | | | | ipmi daemon return code modification
| * | ipmi daemon return code modificationtomjose2016-03-082-2/+1
|/ /
* | Merge pull request #71 from tomjoseph83/ipmi_lannkskjames2016-03-071-143/+168
|\ \ | | | | | | Refactoring of IPMI Transport Handler
| * | Refactoring of IPMI Transport Handlertomjose2016-03-071-143/+168
|/ /
* | Merge pull request #70 from anoo1/bootcountnkskjames2016-03-052-6/+6
|\ \ | | | | | | Read/Write Boot Count Sensor as int
| * | Read/Write Boot Count Sensor as intAdriana Kobylak2016-03-012-7/+7
| | | | | | | | | | | | | | | Change the dbus request to read/write the boot count sensor from byte to int. Passing a type of byte causes the dbus interfaces to fail with ENXIO (no such device or address).
* | | Merge pull request #69 from vishwabmc/masternkskjames2016-03-056-224/+177
|\ \ \ | | | | | | | | Fix memory leak in good paths in ipmid
| * | | memory leak found during multiple reboots. it was found that in many cases thevishwa2016-03-036-224/+177
| |/ / | | | | | | | | | | | | | | | reply was not unref'ed --signed off by vishwanath@in.ibm.com---
* | | Merge pull request #67 from ratagupt/DuplicateRegsiterationnkskjames2016-03-031-20/+1
|\ \ \ | |/ / |/| | Fixing the Duplicate registration for NetFn [0x6], Cmd:[0xFF]
| * | Removed the handler function of the wild card commandRatan Gupta2016-02-291-16/+0
| | | | | | | | | | | | enter the commit message for your changes. Lines starting
| * | Fixing the Duplicate registration for NetFn [0x6], Cmd:[0xFF]Ratan Gupta2016-02-251-4/+1
| | |
* | | Merge pull request #66 from causten/sysinfonkskjames2016-02-292-14/+5
|\ \ \ | |/ / |/| | Enable debug data section of event logs
| * | Enable debug data section of event logsChris Austen2016-02-202-14/+5
|/ / | | | | | | | | | | | | | | until now event logs have 3 bytes of debug data. This was due to a bug in our debug / testing interface. Now that the problem has been resolved we can start seeing the full ESEL over the rest interface. This will lead to larger files in the writable filesystem.
* | Merge pull request #64 from anoo1/getipnkskjames2016-02-101-39/+36
|\ \ | | | | | | IPMI Get IP Support
| * | IPMI Get IP SupportAdriana Kobylak2016-02-101-39/+36
|/ / | | | | | | | | | | Fix parsing of Get IP address. Dbus method returns additional information such as family and gateway, but this IPMI option just requires the IP to be returned.
* | Merge pull request #63 from anoo1/getmacnkskjames2016-02-102-55/+83
|\ \ | | | | | | Get MAC address
OpenPOWER on IntegriCloud