[xen master] xenalyze: Reduce warnings about leaving a vcpu in INIT

[patchbot@xxxxxxx]

commit 0ce2ee7a16f2886c32b3f070bba3172f4a577aa4
Author:     George Dunlap <george.dunlap@xxxxxxxxx>
AuthorDate: Mon Oct 9 11:19:57 2023 +0100
Commit:     George Dunlap <george.dunlap@xxxxxxxxx>
CommitDate: Mon Oct 16 15:01:52 2023 +0100

    xenalyze: Reduce warnings about leaving a vcpu in INIT
    
    We warn when we see data for a vcpu moving into a non-RUNNING state,
    just so that people know why we're ignoring it.  On full traces, this
    happens only once.  However, if the trace was limited to a subset of
    pcpus, then this will happen every time the domain in question is
    woken on that pcpu.
    
    Add a 'delayed_init' flag to the vcpu struct to indicate when a vcpu
    has experienced a delayed init.  Print a warning message once when
    entering the state, and once when leaving it.
    
    Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxx>
    Release-acked-by: Henry Wang <Henry.Wang@xxxxxxx>
    Acked-by: Anthony PERARD <anthony.perard@xxxxxxxxxx>
---
 tools/xentrace/xenalyze.c | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/tools/xentrace/xenalyze.c b/tools/xentrace/xenalyze.c
index 4b6db59d87..ce6a85d50b 100644
--- a/tools/xentrace/xenalyze.c
+++ b/tools/xentrace/xenalyze.c
@@ -1625,7 +1625,7 @@ struct vlapic_struct {
 struct vcpu_data {
     int vid;
     struct domain_data *d; /* up-pointer */
-    unsigned activated:1;
+    unsigned activated:1, delayed_init:1;
 
     int guest_paging_levels;
 
@@ -6979,10 +6979,17 @@ void vcpu_start(struct pcpu_info *p, struct vcpu_data 
*v,
      * bring a vcpu out of INIT until it's seen to be actually
      * running somewhere. */
     if ( new_runstate != RUNSTATE_RUNNING ) {
-        fprintf(warn, "First schedule for d%dv%d doesn't take us into a 
running state; leaving INIT\n",
-                v->d->did, v->vid);
+        if ( !v->delayed_init ) {
+            fprintf(warn, "First schedule for d%dv%d doesn't take us into a 
running state; leaving in INIT\n",
+                    v->d->did, v->vid);
+            v->delayed_init = 1;
+        }
 
         return;
+    } else if ( v->delayed_init ) {
+        fprintf(warn, "d%dv%d RUNSTATE_RUNNING detected, leaving INIT",
+                v->d->did, v->vid);
+        v->delayed_init = 0;
     }
 
     tsc = ri_tsc;
--
generated by git-patchbot for /home/xen/git/xen.git#master