So, there is exactly one caller of main_loop_wait() in the tree that passes it 'true' as an argument. That caller is in xen_init_display() in hw/dispaly/xenfb.c. The function was added in 2009 with the comment "FIXME/TODO: Kill this. Temporary needed while DisplayState reorganization is in flight."
I'd like to think that we've now completed whatever reorg that was, 8 years on, so can we now get rid of this function? It definitely seems very dubious to have a display init function with a busy loop and a call into main_loop_wait()... thanks -- PMM