On 09.09.19 09:53, Jan Beulich wrote:
On 09.09.2019 09:35, Juergen Gross wrote:
--- a/xen/drivers/char/console.c
+++ b/xen/drivers/char/console.c
@@ -1173,6 +1173,7 @@ static char        *debugtrace_buf; /* Debug-trace buffer 
*/
  static unsigned int debugtrace_prd; /* Producer index     */
  static unsigned int debugtrace_kilobytes = 128, debugtrace_bytes;
  static unsigned int debugtrace_used;
+static bool debugtrace_buf_empty;

Wouldn't it be more logical for this to start out as "true"?

In the end it doesn't matter, as last_buf[] will be empty initially.
But I can change it to let its semantics match the reality.


Juergen

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to