summaryrefslogtreecommitdiffstats
path: root/apphandler.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Spelling fixesGunnar Mills2017-10-311-2/+2
| | | | | | | | | 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>
* apphandler: channel: split out commandsPatrick Venture2017-10-261-395/+2
| | | | | | | | | | | | | | I split out the channel commands into their own subordinate file from the apphandler.cpp. There are three commands in apphandler for the channel information. There are other commands in the transporthandler that correlate. Moving this is step one in a larger task to enable multiple channels. Change-Id: I2e434a1da942325fc420079415edb39622f4e346 Signed-off-by: Patrick Venture <venture@google.com>
* watchdog: split out commandsPatrick Venture2017-10-251-169/+1
| | | | | | | | | | I split out the watchdog command handlers into their own subordinate file from the apphandler. This reduces the size of the contents of the apphandler file, and logically cuts out code into its own files. Change-Id: Idc70acad6c93c12456cc7092786715643c9c1ffb Signed-off-by: Patrick Venture <venture@google.com>
* Don't set/get zeroconfig address if other IP address is presentNagaraju Goruganti2017-10-111-2/+5
| | | | | | | Resolves openbmc/openbmc#2265 Change-Id: I65b54358f13231c78b3c6488a59b94bc9323484c Signed-off-by: Nagaraju Goruganti <ngorugan@in.ibm.com>
* Independently set the watchdog Interval when setPatrick Venture2017-09-121-6/+6
| | | | | | | | When we receive a set watchdog command, this will now set the Interval value directly. Change-Id: I34b82507cbad95f2ad56932d03a48a80b2dfc11d Signed-off-by: Patrick Venture <venture@google.com>
* SetLan: Make changes in set channel access to support ipsrc paramRatan Gupta2017-09-081-88/+215
| | | | | | | Resolves openbmc/openbmc#2138 Change-Id: I4b4b8e915bcfea6ed5f913ede5f714290483e4f9 Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>
* Implement Get channel access commandTom Joseph2017-08-161-0/+50
| | | | | | | | | | | | | | | | | | | The get channel access command is implemented to support the set channel access command. The set channel access command is used to apply the network settings. Before the set channel access is issued ipmitool issues get channel access command to get all the fields and then copy into the buffer for set channel access. This would enable the following ipmitool command. ipmitool lan set 1 access on There is no backend support for these settings and values are hardcoded in the command implementation. Resolves openbmc/openbmc#1565 Change-Id: I6c7b2ca7971155b90914ab5b68ab3038bfaba185 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Handle an action byte value of 0 in the set watchdog timer commandPatrick Venture2017-08-151-0/+10
| | | | | | | | | | | | | | | | | There are multiple options for the action field of the command. This change only addresses one, where the caller wants there to be no action taken when the timer expires. Because the implementation of the watchdog doesn't support multiple actions in precisely that arrangement, it makes sense to simply disable the timer. The default behaviour on x86 of the ipmi_watchdog driver is to first set the watchdog to 10s and an action of "none" and then set it to 10s with an action of "hard reset." This change is necessary to stop the watchdog from resetting the host when it receives the initial setting with the action of "none." Change-Id: I1a541824d2bd0d7a0c33ec5759c7c959cc2ae5ee Signed-off-by: Patrick Venture <venture@google.com>
* Add support for vlan(set lan command)Ratan Gupta2017-08-131-5/+42
| | | | | | | Resolves openbmc/openbmc#1899 Change-Id: I3c436045676bc96e5d91fd9420509bc991549a13 Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>
* Move LAN handling commands to the new xyz interfacesRatan Gupta2017-08-131-88/+133
| | | | | Change-Id: I4631c79c171883e5f0d531cfb5957a09496e4b3a Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>
* apphandler: Fix version string leak in ipmi_app_get_device_idXo Wang2017-07-201-0/+1
| | | | | Signed-off-by: Xo Wang <xow@google.com> Change-Id: If64a9897f29e5d8dd267afdd2507b3165114bd98
* Update watchdog command handler to use new watchdog daemonVishwanatha Subbanna2017-06-151-33/+67
| | | | | | | | | | Below IPMI commands in NetFn:0x06 are implemented and they need to use the services from new refactored watchdog daemon Command:0x22 and 0x24. Change-Id: I2f7b89aa8b35d08033ecab30b59440c023520390 Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>
* Add privilege level for each IPMI commandTom2017-01-171-10/+29
| | | | | Change-Id: I07f160c6ea83dc91f46e21f263c4cf994aff081d Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Create library for System Interface Commands.Tom2017-01-171-98/+0
| | | | | | | | | | There are IPMI commands like Read Event Message Buffer, Set BMC Global Enables and Get Message Flags which is to be executed from host interface only. So creating a separate library for these commands which would be used only by phosphor-host-ipmid. Change-Id: I14cb4019b9ab19c84e463e5a7820010f57f1c5eb Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* ipmid: Fix Get BT Interface Capabilities responseAdriana Kobylak2016-12-131-1/+3
| | | | | | | | | | Per the IPMI 2.0 spec, the input and output buffer size must be maximum buffer size minus one byte to allocate space for the length byte. Change-Id: Ie1768b9c6844275f7897fbf8056ce2a19b1c11c3 Fixes: openbmc/openbmc#847 Signed-off-by: Adriana Kobylak <anoo@us.ibm.com>
* Add transaction support to Set Lan Cmd and Set Channel Access CmdNan Li2016-11-231-0/+121
| | | | | | | | | | | | | * Ipmid will cache single setting to memory by Set Lan cmd * Add Set Channel Access cmd to ipmid * Set Channel Access cmd is used to apply the previous cached settings TODO: parse the request data of Set Channel Access cmd if needed. Resolves openbmc/openbmc#444 Change-Id: Ia7ac57a551731920ffe69dad5c69c995521fed02 Signed-off-by: Nan Li <william.bjlinan@hotmail.com>
* Add IPMI Get Self Test Results supportNan Li2016-11-221-0/+42
| | | | | | | | | | | As discussed in design meeting, before the whole ipmi stack is implemented, we don't plan to support "self test". So just respond with self test function not implemented in this controller. Resolves openbmc/openbmc#436 Change-Id: Icd72e07674ce857f98ac036df2b7d6f6b93f1914 Signed-off-by: Nan Li <william.bjlinan@hotmail.com>
* Print failing object path after failed bus lookupBrad Bishop2016-10-061-4/+8
| | | | | Change-Id: I599d8297b4d2fd45831443685ea5b7d53f1515b6 Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Ensure all tools are building with -Wall and -WerrorMatthew Barth2016-09-211-2/+0
| | | | | | | | Corrected resulting compiler errors from correctly adding -Wall & -Werror within configure.ac. Change-Id: Ia7aaafa83103322ad4b50171d62144624dd71e42 Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
* Modify respond of un-supported IPMI commandNan Li2016-09-091-1/+1
| | | | | | | | | | | A number of callbacks are registered for 'WILDCARD' types, which currently do nothing except respond with CC_OK(00h). Change complete codes to CC_INVALID(C1h). Resolves openbmc/openbmc#469. Change-Id: I91ea5ee1a23284a4159ff1c1867342e89c6fa444 Signed-off-by: Nan Li <william.bjlinan@hotmail.com>
* Allow out-of-tree buildsPatrick Williams2016-09-061-1/+1
| | | | | Change-Id: If4b97e65f63c3abbf23140dd97b9febe00b3be0d Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* ipmid: Change .H -> .hppPatrick Williams2016-09-061-4/+4
| | | | | Change-Id: Ia1ed05a533736105909717b86f16274b30d36401 Signed-off-by: Patrick Williams <patrick@stwcx.xyz>
* Hard-coded service name replaced with call to ObjectMapperSergey Solomin2016-08-251-6/+33
| | | | | | | | | | | | | 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>
* 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-081-0/+539
Change-Id: I495186b4afd58ec1e5d9d5c6472394f96541d17d Signed-off-by: Matthew Barth <msbarth@us.ibm.com>
OpenPOWER on IntegriCloud