diff options
author | Jason M. Bills <jason.m.bills@linux.intel.com> | 2019-12-06 00:29:56 +0300 |
---|---|---|
committer | Jason M. Bills <jason.m.bills@linux.intel.com> | 2019-12-07 00:29:02 +0300 |
commit | 243c130a919c7037b5edd3a8097317340796ce85 (patch) | |
tree | aeb0bc9b3b1f7dc804106a605f0177f8ee0e7dc5 /meta-openbmc-mods/meta-common/recipes-phosphor/ipmi/phosphor-ipmi-net | |
parent | ccd6cab15aedd223a42dfdb110ab2d71d8a87141 (diff) | |
download | openbmc-243c130a919c7037b5edd3a8097317340796ce85.tar.xz |
Update to internal 2019-12-05
Signed-off-by: Jason M. Bills <jason.m.bills@linux.intel.com>
Diffstat (limited to 'meta-openbmc-mods/meta-common/recipes-phosphor/ipmi/phosphor-ipmi-net')
-rw-r--r-- | meta-openbmc-mods/meta-common/recipes-phosphor/ipmi/phosphor-ipmi-net/0012-Do-not-stop-session-in-deactivate-payload.patch | 48 |
1 files changed, 48 insertions, 0 deletions
diff --git a/meta-openbmc-mods/meta-common/recipes-phosphor/ipmi/phosphor-ipmi-net/0012-Do-not-stop-session-in-deactivate-payload.patch b/meta-openbmc-mods/meta-common/recipes-phosphor/ipmi/phosphor-ipmi-net/0012-Do-not-stop-session-in-deactivate-payload.patch new file mode 100644 index 000000000..6430a6928 --- /dev/null +++ b/meta-openbmc-mods/meta-common/recipes-phosphor/ipmi/phosphor-ipmi-net/0012-Do-not-stop-session-in-deactivate-payload.patch @@ -0,0 +1,48 @@ +From cf8c0e1bf18334fe4a8f76c1e9b34ccfdc82f6f9 Mon Sep 17 00:00:00 2001 +From: Yong Li <yong.b.li@linux.intel.com> +Date: Fri, 15 Nov 2019 18:34:04 +0800 +Subject: [PATCH] Do not stop session in deactivate payload + +IPMI spec defines that: +The Deactivate Payload command does not cause the session to be terminated; + +Also during SOL looptest, there is only one time startSession call, +but multiple stopSessions calls, +This causes the looptest will fail if there is any new sessions comes in, +needs to remove the stopSession call. + +Tested: +Start the loop test in a terminal: +ipmitool -H $BMCIP -Uroot -P 0penBmc -I lanplus sol looptest 500 200 + +Then start a new session in another terminal: +ipmitool -H $BMCIP -Uroot -P 0penBmc -I lanplus raw 6 1 + +The looptest still works + +Signed-off-by: Yong Li <yong.b.li@linux.intel.com> +--- + command/payload_cmds.cpp | 7 ------- + 1 file changed, 7 deletions(-) + +diff --git a/command/payload_cmds.cpp b/command/payload_cmds.cpp +index f558781..363b843 100644 +--- a/command/payload_cmds.cpp ++++ b/command/payload_cmds.cpp +@@ -176,13 +176,6 @@ std::vector<uint8_t> deactivatePayload(const std::vector<uint8_t>& inPayload, + */ + return outPayload; + } +- +- auto check = +- std::get<session::Manager&>(singletonPool).stopSession(sessionID); +- if (!check) +- { +- response->completionCode = IPMI_CC_UNSPECIFIED_ERROR; +- } + } + catch (std::exception& e) + { +-- +2.7.4 + |