[Skiboot] [PATCH 7/9] ipmi-watchdog: Simplify our completion function

William A. Kennington III wak at google.com
Thu May 24 10:13:33 AEST 2018


This makes no functional changes, just refactors the completion function
to be used for reset only, since it does nothing but free the message
for set calls. This will be useful for future changes to reduce nesting
depth.

Signed-off-by: William A. Kennington III <wak at google.com>
---
 hw/ipmi/ipmi-watchdog.c | 27 +++++++++++++++------------
 1 file changed, 15 insertions(+), 12 deletions(-)

diff --git a/hw/ipmi/ipmi-watchdog.c b/hw/ipmi/ipmi-watchdog.c
index 6f2ac4abf..a43100ade 100644
--- a/hw/ipmi/ipmi-watchdog.c
+++ b/hw/ipmi/ipmi-watchdog.c
@@ -49,27 +49,18 @@ static struct timer wdt_timer;
 static bool wdt_stopped;
 static bool wdt_ticking;
 
-static void ipmi_wdt_complete(struct ipmi_msg *msg)
-{
-	if (msg->cmd == IPMI_CMD(IPMI_RESET_WDT) && wdt_ticking)
-		schedule_timer(&wdt_timer, msecs_to_tb(
-				       (WDT_TIMEOUT - WDT_MARGIN)*100));
-
-	ipmi_free_msg(msg);
-}
-
 static void set_wdt(uint8_t action, uint16_t count, uint8_t pretimeout,
 		bool dont_stop)
 {
 	struct ipmi_msg *ipmi_msg;
 
 	ipmi_msg = ipmi_mkmsg(IPMI_DEFAULT_INTERFACE, IPMI_SET_WDT,
-			      ipmi_wdt_complete, NULL, NULL, 6, 0);
+			      ipmi_free_msg, NULL, NULL, 6, 0);
 	if (!ipmi_msg) {
 		prerror("Unable to allocate set wdt message\n");
 		return;
 	}
-	ipmi_msg->error = ipmi_wdt_complete;
+	ipmi_msg->error = ipmi_free_msg;
 	ipmi_msg->data[0] = TIMER_USE_POST |
 		TIMER_USE_DONT_LOG |
 		(dont_stop ? TIMER_USE_DONT_STOP : 0);
@@ -81,12 +72,24 @@ static void set_wdt(uint8_t action, uint16_t count, uint8_t pretimeout,
 	ipmi_queue_msg(ipmi_msg);
 }
 
+static void reset_wdt_complete(struct ipmi_msg *msg)
+{
+	const uint64_t reset_delay_ms = (WDT_TIMEOUT - WDT_MARGIN) * 100;
+
+	/* If we are inside of skiboot we need to periodically restart the
+	 * timer. Reschedule a reset so it happens before the timeout. */
+	if (wdt_ticking)
+		schedule_timer(&wdt_timer, msecs_to_tb(reset_delay_ms));
+
+	ipmi_free_msg(msg);
+}
+
 static struct ipmi_msg *wdt_reset_mkmsg(void)
 {
 	struct ipmi_msg *ipmi_msg;
 
 	ipmi_msg = ipmi_mkmsg(IPMI_DEFAULT_INTERFACE, IPMI_RESET_WDT,
-			      ipmi_wdt_complete, NULL, NULL, 0, 0);
+			      reset_wdt_complete, NULL, NULL, 0, 0);
 	if (!ipmi_msg) {
 		prerror("Unable to allocate reset wdt message\n");
 		return NULL;
-- 
2.17.0.441.gb46fe60e1d-goog



More information about the Skiboot mailing list