summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Changes Integrated since 1.8.14RC2IPMITOOL_1_8_14Jim Mankovich2014-05-051-1/+3
|
* ID: 311 man page update for new sunoem commandsJim Mankovich2014-05-051-24/+107
|
* ID: 280 - man page cleanupJim Mankovich2014-04-291-3/+0
|
* Make comment about 'source offset' explicitZdenek Styblik2014-04-261-1/+1
|
* ID: 308 - "fru edit" no longer works for non-zero fieldsZdenek Styblik2014-04-251-9/+9
| | | | | | | | | | | | | | | | 1. Warnings about "FRU Area Length" based on uninitialized (malloc'd) memory contents (due to fru->max_read_size not being initialized, left at 0) and fru_data not being zeroed after malloc() in ipmi_fru_set_field_string(). 2. "fru edit" commands for any field index other than 0 would fail (with "Field not found !" error) due to a couple offset and length calculation errors (for all the supported "area" types) in ipmi_fru_set_field_string(). 3. "fru edit" commands would corrupt the FRU Inventory Area due to incorrect "source offset" value being specified in write_fru_area() call in impi_fru_set_field_string(). Commit for Rob Swindell
* ID: 310 - lanplus retry regressionZdenek Styblik2014-04-251-7/+3
| | | | | | | | | The lanplus driver for has a bug in ipmi_lanplus_open_session(). There is an extra and undesired check for a null response (timeout). As a result, it returns 1 to the caller. The result of this is an occasional assertion. Commit mends this. Commit for Pat Donlin
* Add ticket IDs, if we have oneZdenek Styblik2014-04-181-7/+9
|
* Add hpm2.h to list of distributed include filesIPMITOOL_1_8_14RC2Jim Mankovich2014-04-171-1/+1
|
* Update ChangeLog for 1.8.14RC2 changesJim Mankovich2014-04-171-0/+6
|
* Incorrect byteswap in SOL maximum payload codeAnton Blanchard2014-04-121-11/+0
| | | | | | | | | | | | | | | | | | | | max_inbound_payload_size, max_outbound_payload_size and port are assembled byte by byte using shifts. This works correctly in both little and big endian and doing a subsequent byte swap is wrong. To highlight this issue I dumped the values on a big endian machine: max_inbound_payload_size 51200 max_outbound_payload_size 51200 port 28418 And after this fix: max_inbound_payload_size 200 max_outbound_payload_size 200 port 623 Signed-off-by: Zdenek Styblik <stybla@turnovfree.net>
* Fix build error in HPM.2 codeAnton Blanchard2014-04-121-0/+1
| | | | | | lib/hpm2.c uses BSWAP_16 without including ipmitool/bswap.h. Signed-off-by: Zdenek Styblik <stybla@turnovfree.net>
* ID: 300 - new sunoem functionalityZdenek Styblik2014-04-092-296/+1887
| | | | | | | | | | | | Main changes include: * direct connection to ILOM command line interface * ability to add/delete ssh keys fot ILOM users * ability to set ILOM properties * ability to retrieve various logs from SP * removal of obsolete/non-functioning code and other misc changes Commit for Martin Hovorka of Oracle
* ID: 144 - Fix 'dcmi power set_limit action <value>'Zdenek Styblik2014-04-081-12/+139
| | | | | | | | | | | | From the ticket: (v1.8.12) "dcmi power set_limit action <value>" (where <value> is either "sel_logging" or "power_off") does not actually send the "Set Power Limit" command (so no change to the power limit policy is made). This was caused by all atoi(), strtol(), strtoul() calls being replaced. This patch should fix it. Commit for Holger Liebig
* ID:302 - HPM.2 long message supportZdenek Styblik2014-04-0815-52/+741
| | | | | | | | | | | | | | | This patch adds basic long message support for PICMG-based systems according to the HPM.2 specification. It also introduces APIs for setting inbound and outbound messages sizes per selected interface. This APIs are used in LAN and LAN+ interfaces to set autonomously detected inbound and outbound message sizes. The newly introduced APIs also replace the existing message size detection code in several ipmitool commands in order to leverage the advantages of long message support (HPM.1 upgrade, SDR acquring, FRU inventory read and write). The Kontron-specific long message support is moved under a OEM option. Commit for Dmitry Bazhenov
* Add new SEL entries for ipmi 2.0 rev 1.1Jordan Hargrave2014-04-051-0/+8
| | | | | Signed-off-by: Jordan Hargrave <jordan_hargrave@dell.com> Signed-off-by: Zdenek Styblik <stybla@turnovfree.net>
* Update ipmtool version to 1.8.14-cvsIPMITOOL_1_8_14RC1Jim Mankovich2014-03-281-1/+1
|
* Update For 1.8.14 ReleaseJim Mankovich2014-03-281-0/+32
|
* ID: 299 - openipmi plugin writes zero to wrong byteZdenek Styblik2014-03-271-1/+1
| | | | | | The intent is to zero the byte that no longer contains valid data (because the data was shifted one byte to the left). However, the wrong byte is being zeroed. One way this shows up is when displaying the descriptions with hpm compprop.
* ID: 301 - Add OS/Hypervisor installation status eventsZdenek Styblik2014-03-271-0/+4
| | | | | | | Add OS/Hypervisor installation status event types to SEL as noted in table-24 in section 42.2. Commit-for: Charles Rose
* ID: 298 - fix LANplus retryZdenek Styblik2014-03-231-38/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ``I had submitted a patch back on Nov 19, 2013 regarding a fix to lanplus retry. This had resolved a problem whereby a retry of a payload type of IPMI_PAYLOAD_TYPE_IPMI first removed the request from the queue before going back for a retry of the message. I have been able to determine why this fix works correctly. More importantly I have been able to resolve other retry problems in lanplus where assertion panics were hitting on certain retry operations. A new, replacement patch for resolving both of these types of retry bugs follows. The first bug,where the ipmi_lanplus_send_payload() is sending a payload type of IPMI_PAYLOAD_TYPE_IPMI is retryable, however I found in testing that it did not remove the previous request entry from the list of requests chain. If the original message had timed out, a second message sent, the second reply would not match up to the right entry on the list as the req command and sequence numbers are the same. By first removing the first request from the chain this resolves it. The consequence of not removing the stale entry was random errors. The second bug is when waiting for a message response times out during the ipmi_lanplus_send_payload types IPMI_PAYLOAD_TYPE(s) RCMP_OPEN_REQUEST, RAKP1, RAKP_3. In various testing where the message timed out on either of these three payload types, ipmitool would assertion panic upon retry as the session_state was wrong. The timeout could be due to the message never getting to the BMC, the BMC never acting/responding to the message, or the reply message packet dropped (it is UDP after all). If the BMC had acted on the message but the reply was not received, the BMC state would had advanced, and a retry of any of these three commands would error. It is not knowable at retry time if the BMC had acted on the message or not. The solution is upon message timeout failure, retry all three commands in the sequence. This has shown to be reliable and does not result in assertions or any unexpected BMC behaviors. Should the original message response eventually arrive very late, it is just discarded. The testing for these problems was elusive until we found a moderately slow BMC and had separate sessions direct a fusillade of nmap operations on the BMC, then run simple ipmitool commands. This caused sufficient loading of the network and BMC to cause lengthy delays and outright packet drops. The general approach on the second fix is to return a timeout error code back through ipmi_lanplus_open where the sequence can be retried.'' Patch-by: Pat Donlin
* ID: 295 - inform user if SOL session disconnectedZdenek Styblik2014-03-231-1/+1
| | | | | | | | Currently if we are connected to ipmitool sol session and if service processor goes down due to reset/reload, then user has no way to know that sol session has been disconnected. Commit aims to fix such case. Patch-by: Mamatha Inamdar
* ID: 297 - don't print-out SEL entry if ID not presentZdenek Styblik2014-03-231-1/+5
| | | | | | Don't print-out SEL log entry if SEL Record ID isn't present. Patch-by: Mamatha Inamdar
* ID:296 - Fix PSD size decodingZdenek Styblik2014-03-231-3/+11
| | | | | | | | Commit replaces size decoding code with code ported from "decode-dimms" Perl script found in i2c-tools 3.0.3. Former code has reported incorrect size of DDR2 modules. Patch-by: Alexander Amelkin
* Add options to chassis bootparam set bootflagJim Mankovich2014-03-112-17/+255
|
* ID: 293 - Use of uninitialized variable in ipmi_main()Zdenek Styblik2014-02-271-1/+2
| | | | | | | The variable addr is used uninitialized on line 918 of ipmi_main(). Commit ensures it is initialized to 0 before being used. Reported-by: NUXI
* Properly handle plugin non-zero target adddress with -t specification Bug 292Jim Mankovich2014-02-101-7/+9
|
* Move all files one level up in the file hierarcy, to avoid the useless ↵Petter Reinholdtsen2014-02-05171-0/+0
| | | | ipmitool directory.
* Add myself to AUTHORS to test git commit emails.Petter Reinholdtsen2014-01-311-0/+1
|
* Add link to commit archive to test git commit emails.Petter Reinholdtsen2014-01-311-0/+2
|
* Typo, missing semicolon.Petter Reinholdtsen2014-01-312-2/+2
|
* Use TIOCFLUSH if TCFLSH is missing to get the serial plugin building on Hurd.Petter Reinholdtsen2014-01-302-1/+14
|
* Disable imb and open plugins by default on Hurd. The platform lack the ↵Petter Reinholdtsen2014-01-301-0/+5
| | | | required kernel support.
* Change serial plugin to only try to disable the IUCLC serial line flag on ↵Petter Reinholdtsen2014-01-302-2/+13
| | | | platforms supporting it. Fixes build problem on Hurd and FreeBSD.
* Add missing break; in switch block (Coverity CID 1149010).Petter Reinholdtsen2014-01-151-0/+1
|
* Add missing format string placeholder (Coverity CID 1149038).Petter Reinholdtsen2014-01-151-1/+1
|
* Add missing format string placeholder (Coverity CID 1149037).Petter Reinholdtsen2014-01-151-1/+1
|
* Add missing format string placeholder (Coverity CID 1149036).Petter Reinholdtsen2014-01-151-1/+1
|
* Set pointer to NULL just after free() two other places too.Petter Reinholdtsen2014-01-111-0/+2
|
* Make sure to set free()ed pointer to NULL immediately, as specified byPetter Reinholdtsen2014-01-111-0/+1
| | | | <URL: http://sourceforge.net/p/ipmitool/wiki/coding_standards/ >.
* Make sure blockId buffer is never overflown (Coverity CID 1149058).Petter Reinholdtsen2014-01-111-1/+4
|
* Fix incorrect printf() arguments (Coverity CID 1149035).Petter Reinholdtsen2014-01-111-1/+1
|
* Make sure to release malloc()-ed data when fread() fail to read boardPetter Reinholdtsen2014-01-111-0/+1
| | | | type data (Coverity CID 1149052).
* Check return value from open() in ipmi_start_daemon() (Coverity CID CID ↵Petter Reinholdtsen2014-01-111-3/+5
| | | | 1148983).
* Make sure input_file is not used if it is NULL (Coverity CID 1149055).Petter Reinholdtsen2014-01-111-1/+2
|
* ID: 278 - Error in sol looptestZdenek Styblik2013-12-171-1/+1
| | | | Commit fixes use of uninitialized variable in SOL "looptest".
* ID: 290 - ipmi_sol.c needs a clean-upZdenek Styblik2013-12-171-30/+28
| | | | ipmi_sol_payload_access() - change formatting and simplify.
* ID: 85 - Supermicro memory ecc error displayZdenek Styblik2013-12-173-4/+196
| | | | | Commit adds memory ECC error display for Supermicro boards and adds a Supermicro oem sensor type for SEL.
* ID: 85 - Supermicro memory ECC error displayZdenek Styblik2013-12-172-0/+30
| | | | Commit adds ipmi_get_oem_id() function.
* ID: 290 - ipmi_sol.c needs a clean-upZdenek Styblik2013-12-081-113/+54
| | | | Clean up code formatting in ipmi_sol_main().
* ID: 286 - Open session retries hit assert in ipmi_lanplus_send_payloadZdenek Styblik2013-12-081-1/+2
| | | | | | | | | | If we have to retry an open session request, we hit an assert that assumes we can only be in LANPLUS_STATE_PRESESSION state. Add LANPLUS_STATE_OPEN_SESSION_SENT so we don't abort if we retry. Commit for Anton Blanchard
View Lorry Controller Status