[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[win-pv-devel] [PATCH] Fix logging in free builds



Since commit 868cd40f "Work around bug in VS2017 SDV" logging in free builds
has been broken. This is because, unlike for checked builds, the logging
code only emitted messages beginning with 'XEN' (upper case). The effect
of commit 868cd40f was to change the log message prefix to 'xen' (lower
case) and so they were no longer emitted.

This patch simply changes the logging code to check for the new lower case
prefix.

Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
---
 src/xen/log.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/xen/log.c b/src/xen/log.c
index c10315e..9b3a5e8 100644
--- a/src/xen/log.c
+++ b/src/xen/log.c
@@ -507,12 +507,12 @@ LogDebugPrint(
     // If this is not a debug build then apply an aggressive
     // filter to reduce the noise.
 #if !DBG
-    if (Ansi->Length < sizeof ("XEN"))
+    if (Ansi->Length < sizeof ("xen"))
         return;
 
-    if (Ansi->Buffer[0] != 'X' ||
-        Ansi->Buffer[1] != 'E' ||
-        Ansi->Buffer[2] != 'N')
+    if (Ansi->Buffer[0] != 'x' ||
+        Ansi->Buffer[1] != 'e' ||
+        Ansi->Buffer[2] != 'n')
         return;
 #endif
 
-- 
2.5.3


_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/win-pv-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.