summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* 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
| * | Address review comments for IPMI transport functionsAdriana Kobylak2016-02-092-26/+42
| | | | | | | | | | | | | | | | | | | | | Call get bus interface instead of using extern. Define the IPMI request parameters. Use snprintf. Initialize dbus variables at the beginning of the function.
| * | Get MAC addressAdriana Kobylak2016-02-091-29/+41
| | | | | | | | | | | | | | | Support for Get MAC address command. Fix parsing logic to get correct data.
* | | Merge pull request #62 from causten/groupextnkskjames2016-02-105-1/+80
|\ \ \ | |/ / |/| | Add minimal support for ipmitool
| * | Add minimal support for ipmitoolChris Austen2016-02-055-1/+80
|/ / | | | | | | | | | | Adding just a couple of ipmi commands that ipmitool always calls. I'm doing this because in order to run the ipmitool lan 1 <parms> instead of just the raw command the get channel info was needed.
* | Merge pull request #60 from anoo1/macnkskjames2016-02-031-0/+32
|\ \ | | | | | | Add Set MAC address
| * | Add Set MAC addressAdriana Kobylak2016-02-031-0/+32
|/ / | | | | | | Add Set MAC address parameter command
* | Merge pull request #56 from williamli80/cold-resetnkskjames2016-02-013-0/+184
|\ \ | | | | | | Add ipmi cold-reset command, which call a dbus method.
| * | Add ipmi coldReset command, which call a dbus method, belongs to NETFUN_APP.William2016-02-013-0/+184
| | | | | | | | | | | | | | | | | | | | | | | | 1. A method dbus_warm_reset() to talk with the dbus method 'warmRest'; 2. Also get service name by ipmid_get_sd_bus_connection() instead of object_mapper_get_connection(); 3. Register the ipmi command; 4. Add related .o to the Makefile; 5. Add wildcard function.
* | | Merge pull request #59 from hramasub/dbus-getset-lannkskjames2016-02-011-42/+191
|\ \ \ | |/ / |/| | Using systemd dbus API to set get LAN parameters.
| * | Using systemd dbus API to set get LAN parameters.Hariharasubramanian R2016-01-201-42/+191
| | |
OpenPOWER on IntegriCloud