From: Julien Grall <julien.grall@arm.com>
Date: Wed, 21 Aug 2019 21:19:17 +0000 (+0100)
Subject: xen/console: debugtrace: Compute the buffer length is O(1) rather O(n)
X-Git-Tag: archive/raspbian/4.14.0+80-gd101b417b7-1+rpi1^2~63^2~1710
X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=a1dc65aaf470fa01f394e01ab6747d135961a0ae;p=xen.git

xen/console: debugtrace: Compute the buffer length is O(1) rather O(n)

This was meant to be part of commit e0bf98394e "xen/console: Fix build
when CONFIG_DEBUG_TRACE" but not addressed before been committed.

Signed-off-by: Julien Grall <julien.grall@arm.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
---

diff --git a/xen/drivers/char/console.c b/xen/drivers/char/console.c
index 75e493466e..d67e1993f2 100644
--- a/xen/drivers/char/console.c
+++ b/xen/drivers/char/console.c
@@ -1187,7 +1187,7 @@ static void debugtrace_dump_worker(void)
     ASSERT(debugtrace_buf[debugtrace_bytes - 1] == 0);
     if ( debugtrace_buf[debugtrace_prd] != '\0' )
         sercon_puts(&debugtrace_buf[debugtrace_prd],
-                    strlen(&debugtrace_buf[debugtrace_prd]));
+                    debugtrace_bytes - debugtrace_prd - 1);
 
     /* Print youngest portion of the ring. */
     debugtrace_buf[debugtrace_prd] = '\0';