summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* apphandler: channel: split out commandsPatrick Venture2017-10-265-415/+490
| | | | | | | | | | | | | | 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-254-169/+208
| | | | | | | | | | 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>
* Add IBM OEM NetFn(0x3A)Tom Joseph2017-10-251-1/+2
| | | | | Change-Id: If6c4b2c031981f0dba70799dc8cb42840def2dd2 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Update Journal Variable Names in utils.cppGunnar Mills2017-10-191-8/+8
| | | | | | | | | Journal entry variable names should be uppercase. https://www.freedesktop.org/software/systemd/man/sd_journal_print.html Also fixed indentation. Change-Id: I02d30971123c2bb526f19e00b534e06fbcd9ee59 Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
* Update Journal Variable NamesGunnar Mills2017-10-193-6/+9
| | | | | | | | | Journal entry variable names should be uppercase. https://www.freedesktop.org/software/systemd/man/sd_journal_print.html Also, fixed indentation. Change-Id: I2c838332ab7811ccc9a2c99f5b1f558c48f25f76 Signed-off-by: Gunnar Mills <gmills@us.ibm.com>
* storagehandler: Use new phosphor-timemanager interfaceLei YU2017-10-191-89/+84
| | | | | | | | Update SEL-Time handlers to use sdbusplus and new phosphor-timemanager interface. Change-Id: Ib931a6bedc14037fc1a0a7c25698f67272520bb5 Signed-off-by: Lei YU <mine260309@gmail.com>
* storagehandler: sort include filesLei YU2017-10-171-11/+14
| | | | | Change-Id: Iaf49ed787a2e077af27f28d22507456e8efa2b41 Signed-off-by: Lei YU <mine260309@gmail.com>
* Add DCMI Get/Set Management Controller Id StringVladislav Vovchenko2017-10-162-1/+187
| | | | | | | Resolves openbmc/openbmc#1855 Change-Id: I878c7bcb1ea8b46cd3e932b1cbb2290fe612f652 Signed-off-by: Vladislav Vovchenko <vlad.vovchenko93@gmail.com> Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>
* Don't set/get zeroconfig address if other IP address is presentNagaraju Goruganti2017-10-114-14/+82
| | | | | | | Resolves openbmc/openbmc#2265 Change-Id: I65b54358f13231c78b3c6488a59b94bc9323484c Signed-off-by: Nagaraju Goruganti <ngorugan@in.ibm.com>
* IPMI changes to mark non present as non functionalDhruvaraj Subhashchandran2017-10-114-33/+103
| | | | | | | | | When marking a unit as functional, both functional state and presence need to be checked to avoid marking non-present units as functional. Change-Id: If7b710c39f1c2590b82378ebdb7014dc924599ff Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
* IPMI changes to skip updating non present coresDhruvaraj Subhashchandran2017-10-114-6/+65
| | | | | | | | | Cores which are not present need not to be updated to inventory. This change is for skipping several updates to inventory based on skipOn value. Change-Id: I29e005a715ccae1df6eeaf35561a20896ecde0ac Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
* Revert "Remove legacy reading of PS derating"Dhruvaraj Subhashchandran2017-10-041-0/+1
| | | | | | | This reverts commit 1bf3b749e9217f03fbc55bbb45802df9e715e3b0. Change-Id: Ib364199f4d676c2003450cad4b4d3d6077356fc1 Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
* boot policy: Support boot parameter #4 (boot info)Tom Joseph2017-10-042-2/+10
| | | | | | | Resolves openbmc/openbmc#2356 Change-Id: If775a01a97f29a60575c03f26d32bba0315af920 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Handle std::runtime_error exception in get sensor readingTom Joseph2017-10-031-0/+9
| | | | | | | Resolves openbmc/openbmc#2374 Change-Id: I596b5f0662b2506c8af4143ffa6438320d7bf4a2 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* boot policy: Correct the logic of setting the boot policyTom Joseph2017-09-281-53/+34
| | | | | | | Resolves openbmc/openbmc#2355 Change-Id: Ic7b0efb35887c987a1626e5d88f8a9513d22004f Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Use static path for inventory managerDhruvaraj Subhashchandran2017-09-261-5/+2
| | | | | | | | | | | During boot after sending the inventory details host is asking for the a sensor value, that request is getting blocked because the ipmid on BMC is busy with processing the inventory data. Adding an optimization in processing inventory, which will reduce the wait. Change-Id: I490c98220e0244fb6c766432aa1ea8cf25ae1502 Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
* Remove legacy reading of PS deratingDhruvaraj Subhashchandran2017-09-251-1/+0
| | | | | | | Resolves openbmc/openbmc#2154 Change-Id: Ia5e150951fd077eddb4ee5a11d361eceee404e44 Signed-off-by: Dhruvaraj Subhashchandran <dhruvaraj@in.ibm.com>
* typofix: s/impelmented/implemented/gPatrick Venture2017-09-211-2/+2
| | | | | | | Just noticed in my systemd journal. Change-Id: I0f958750cd5936079e41e819956d079baec26875 Signed-off-by: Patrick Venture <venture@google.com>
* Move error msg for trying wildcard into debug onlyPatrick Venture2017-09-201-0/+3
| | | | | | | | | | My logs are effectively full of this message, hiding out other useful messages because we run a lot of OEM traffic. Also, until the daemon fails to find the command via wildcard, it's not really an error worth reporting in most cases. Change-Id: Id6674f3e9a2f237de6bd37d6c9f89931a99d0951 Signed-off-by: Patrick Venture <venture@google.com>
* sensor: Pass the dbus property type as a template parameterTom Joseph2017-09-141-64/+7
| | | | | | | | | | | | readingAssertion and reading type of sensors, take the dbus property type as template parameters. Instead of generating code to handle that, the function parameter is updated with the template parameter. Resolves openbmc/openbmc#2220 Change-Id: Ib9e15727ba96e3a53c46f17bffc9ef68dbf73194 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Correct the offset field in reading calculationTom Joseph2017-09-141-2/+2
| | | | | Change-Id: If85dd3d6705ccc86a8e3406ef5f90fcc6e2967d1 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Use getFunc for TPM and RebootAttempt sensorTom Joseph2017-09-141-71/+0
| | | | | Change-Id: I8d7581e2c14344425683870daf395fc895d1034c Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Use getFunc to handle the get sensor readingTom Joseph2017-09-141-3/+22
| | | | | Change-Id: If0f3185e7ce1d300be20b58ee5004a58e35c222d Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Handle the response of find_openbmc_pathTom Joseph2017-09-141-9/+12
| | | | | | | | | | | The sensor path of the inventory items does not include the inventory root, so would fail in the case of find_openbmc_path. If the return code of find_openbmc_path indicates failure, then it is logged and allowed to continue. The call to find_openbmc_path would be eventually removed. Change-Id: I7513bf4ab9b897ff5b64cfb77fd504e6ef4e227a Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Modify mako template to generate sensor getter funcTom Joseph2017-09-142-1/+29
| | | | | Change-Id: If198d9a9fb1145bbaeda528e3f80356527b80059 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Map dbus info to sensor's reading fieldTom Joseph2017-09-142-0/+47
| | | | | Change-Id: I3e35bf78260b1fa29e992b00279f2dd166cd2fe1 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Add function to support sensor with readingAssertion typeTom Joseph2017-09-142-0/+48
| | | | | | | | readingAssertion is special type where the entire assert bitfield serves as the value or reading. Change-Id: Iaddbe846e04d2a53cff69d71670a96ccc66636a8 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Add function to map dbus info to reading fieldTom Joseph2017-09-143-0/+57
| | | | | | | | | For sensor's with reading type eventData2, the eventdata2 field is mapped to the reading field in the get sensor reading command response. Change-Id: I9ad85ddb48d6c273a22e476e29ea9bbb34c13e24 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensor: Add function to map dbus info to assertion statusTom Joseph2017-09-145-22/+195
| | | | | Change-Id: I30aae9abd7905ae3299856d798d41e10859fed7f Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* sensorhandler: support setting analog sensorsEmily Shaffer2017-09-135-62/+196
| | | | | | | | | | | | | | | | | | Refactor YAML format to denote mutability of sensors. Sensors which expect different formats for reads and writes should present two entries in the sensor YAML, one with the read interface and one with the write interface. Sensors which share a format for both reads and writes may present only one entry in the YAML with both readable and writable enums specified. If a sensor receives a write which has an interface of Sensor.Value, the "Set" message is sent via DBus to the path provided in the YAML. The previous codepath is maintained. Change-Id: I292f95b6fe936de759fd65ce72c842a1bfe66448 Signed-off-by: Emily Shaffer <emilyshaffer@google.com> Signed-off-by: Patrick Venture <venture@google.com>
* Modify mako script to generate code for empty interfacesMarri Devender Rao2017-09-132-5/+12
| | | | | | | | Allow generating empty interfaces in the FRU map, which are used to determine the type of the inventory object. Change-Id: I6dc97514b0bdf7ef9d3a63ee6790a7933b3e7cd9 Signed-off-by: Marri Devender Rao <devenrao@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>
* Refactor reading and parsing of inventory dataMarri Devender Rao2017-09-081-24/+24
| | | | | | | | Refactored to use 'GetAll' dbus method call over multiple 'Get' dbus method calls to read the properties of a inventory object. Change-Id: I0ddc2150830961666fafa661cf29042f66137977 Signed-off-by: Marri Devender Rao <devenrao@in.ibm.com>
* boot settings: use the new 'one time' settingDeepak Kodihalli2017-09-083-117/+247
| | | | | | | Resolves openbmc/openbmc#2008. Change-Id: Id79d1f61a3e627fef28b071e2a415e94de14634f Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
* settings: handle multiple objects/interfacesDeepak Kodihalli2017-09-084-14/+22
| | | | | | | | | Handle the fact that a settings object can now implement multiple interfaces, and also the fact that multiple settings objects can implement the same interface. Change-Id: Icf55b08fe2d355a10f7007489dfddb128e05d90e Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.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>
* GetLan: Support for get/set of ipsrc parameterRatan Gupta2017-09-085-59/+265
| | | | | Change-Id: Id9c52bb0963c5924f80f9e273b53ed5556b16a2c Signed-off-by: Ratan Gupta <ratagupt@in.ibm.com>
* Support Power Supply derating sensorTom Joseph2017-09-071-5/+4
| | | | | Change-Id: I893d09a4ecb88507386e8b89015c4b53b18951c3 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Remove extra header inclusionPatrick Venture2017-09-061-1/+0
| | | | | Change-Id: I4f77b3731a52d8a8271e827d017ef2e8b7923fe6 Signed-off-by: Patrick Venture <venture@google.com>
* sensorhandler: fix a couple bitbash bugs affecting linearizationEmily Shaffer2017-08-301-4/+6
| | | | | Change-Id: I2c50480eb8e74199efacf8db5f4d5a1d4f1fcdd8 Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
* Increase host shutdown timeout to 45 minutesAndrew Geissler2017-08-231-2/+2
| | | | | | | | | | | | | A fully loaded system can cause the OS to take up to 45 minutes to properly shut down Removed "quiesce" from the description of the timeout since that is used elsewhere for error paths Resolves openbmc/openbmc#2162 Change-Id: I3800ca8e3bb063be6e5d396fac066014dcce7577 Signed-off-by: Andrew Geissler <andrewg@us.ibm.com>
* tpm: Support TPM Sensor type in Get sensor reading commandTom Joseph2017-08-232-3/+70
| | | | | Change-Id: Icf4a10d8f6e2b95dae3d7dcf789711c5a3a46a39 Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>
* Implement Get channel access commandTom Joseph2017-08-162-0/+71
| | | | | | | | | | | | | | | | | | | 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>
* sensorhandler: correctly report data length in Reserve SDREmily Shaffer2017-08-161-0/+1
| | | | | Change-Id: I30a2641dcb548a3ff5141f7688942a748c458070 Signed-off-by: Emily Shaffer <emilyshaffer@google.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>
* Refactor set sensor handling codeDeepak Kodihalli2017-08-156-273/+213
| | | | | | | | | | | | | | | | | | | | | | | | | | A summary of the changes: - Do not generate per sensor type code to update d-bus objects corresponding to sensors. Function to update d-bus objects based on standard sensor event types, such as assertion, event data, are now generic functions - the need not be generated per sensor or per sensor type. - There's a special case where the assertion is treated as a reading (i.e read the entire assertion field as-is). In this case, code needs to be generated per sensor because the type of the mapped d-bus property can vary. In this case have a generic template function, and generate minimal code like so: inline ipmi_ret_t readingAssertion(const SetSensorReadingReq& cmdData, const Info& sensorInfo) { // Corresponding d-bus property is uint32_t return set::readingAssertion<uint32_t>(cmdData, sensorInfo); } - Make sensor-example.yaml succinct. - Make the code in writesensor.mako.cpp more pythonic. Change-Id: I84415ca6e3f756bbb51a90e290539eb086a7f78b Signed-off-by: Deepak Kodihalli <dkodihal@in.ibm.com>
* Support IPMI Chassis Soft Power Off commandVishwanatha Subbanna2017-08-141-11/+24
| | | | | | | Fixes openbmc/openbmc#1947 Change-Id: I099bd308f5e0ee92b7addf9727afd641d45730ea Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>
* Make host command manager accessible by OpenPower OEMVishwanatha Subbanna2017-08-149-12/+21
| | | | | | | | | Since OpenPower Host IPMI OEM also needs to use the Host Command manager, need to place the files in appropriate directories and make needed changes. Change-Id: I7536a0ff1e53a844ec05f73c4e0605cbf4341465 Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>
* Use Host Command Manager in host interface implementationVishwanatha Subbanna2017-08-1410-186/+162
| | | | | Change-Id: Icefce510a3a0022bf0288fa99518459b732a2e04 Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>
* Define the Manager for handling host bound commandsVishwanatha Subbanna2017-08-144-7/+292
| | | | | | | | | | Since some of the host bound commands are valid only for OpenPower systems, a manager is needed, which can handle commands from OpenBmc common code and also OpenPower implementations. Change-Id: Icf6566e701921ecea4c8cff1c16e498385303396 Signed-off-by: Vishwanatha Subbanna <vishwa@linux.vnet.ibm.com>
OpenPOWER on IntegriCloud