tools: add newlines to xenstored WRL_LOG
authorOlaf Hering <olaf@aepfle.de>
Mon, 3 May 2021 15:47:12 +0000 (17:47 +0200)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Mon, 10 May 2021 13:51:52 +0000 (14:51 +0100)
According to syslog(3) the fmt string does not need a newline.
The mini-os implementation of syslog requires the trailing newline.
Other calls to syslog do include the newline already, add it also to WRL_LOG.

Signed-off-by: Olaf Hering <olaf@aepfle.de>
Reviewed-by: Juergen Gross <jgross@suse.com>
tools/xenstore/xenstored_domain.c

index 3d4d0649a2435808902f901806901e42912b1f2e..2d333b3ff68b49968942a6bce5e1c022ab5d8dcc 100644 (file)
@@ -1132,10 +1132,10 @@ void wrl_apply_debit_actual(struct domain *domain)
        if (domain->wrl_credit < 0) {
                if (!domain->wrl_delay_logged) {
                        domain->wrl_delay_logged = true;
-                       WRL_LOG(now, "domain %ld is affected",
+                       WRL_LOG(now, "domain %ld is affected\n",
                                (long)domain->domid);
                } else if (!wrl_log_last_warning) {
-                       WRL_LOG(now, "rate limiting restarts");
+                       WRL_LOG(now, "rate limiting restarts\n");
                }
                wrl_log_last_warning = now.sec;
        }
@@ -1145,7 +1145,7 @@ void wrl_log_periodic(struct wrl_timestampt now)
 {
        if (wrl_log_last_warning &&
            (now.sec - wrl_log_last_warning) > WRL_LOGEVERY) {
-               WRL_LOG(now, "not in force recently");
+               WRL_LOG(now, "not in force recently\n");
                wrl_log_last_warning = 0;
        }
 }