commit f536c9066b398a3cf374db570d9d41db4ed14eef
author Tom Joseph <tomjoseph@in.ibm.com> Mon Sep 25 18:08:15 2017 +0530
committer Brad Bishop <bradleyb@fuzziesquirrel.com> Wed Oct 04 02:18:05 2017 +0000
tree 7dd6b81537c4d24597176275cfa06bb1589fa394
parent 8202432fd5fe630c28e023aaccc632624f5933cc

boot policy: Support boot parameter #4 (boot info)

Resolves openbmc/openbmc#2356

Change-Id: If775a01a97f29a60575c03f26d32bba0315af920
Signed-off-by: Tom Joseph <tomjoseph@in.ibm.com>

chassishandler.cpp

diff --git a/chassishandler.cpp b/chassishandler.cpp
index 5c0c8c1..b9d4290 100644
--- a/chassishandler.cpp
+++ b/chassishandler.cpp
@@ -1334,8 +1334,15 @@
             fprintf(stderr, "setHostNetworkData failed for set_sys_boot_options.\n");
             rc = IPMI_CC_UNSPECIFIED_ERROR;
         }
-    }
-    else {
+    } else if (reqptr->parameter ==
+             static_cast<uint8_t>(BootOptionParameter::BOOT_INFO)) {
+        // Handle parameter #4 and return command completed normally
+        // (IPMI_CC_OK). There is no implementation in OpenBMC for this
+        // parameter. This is added to support the ipmitool command `chassis
+        // bootdev` which sends set on parameter #4, before setting the boot
+        // flags.
+        rc = IPMI_CC_OK;
+    } else {
         fprintf(stderr, "Unsupported parameter 0x%x\n", reqptr->parameter);
         rc = IPMI_CC_PARM_NOT_SUPPORTED;
     }