Re: [Qemu-devel] [PATCH v10 0/7] trace: Generic event state description

[Stefan Hajnoczi <stefanha@gmail.com>]

On Thu, Feb 28, 2013 at 02:48:27PM +0100, Lluís Vilanova wrote:
> Stefan Hajnoczi writes:
> 
> > On Fri, Jan 25, 2013 at 05:33:46PM +0100, Lluís Vilanova wrote:
> >> Provides a generic event state description structure (TraceEvent) and a more
> >> detailed event control and query interface.
> >> 
> >> This is achieved by creating a new "non-public" tracing backend (i.e., not
> >> selectable by the user at configure time) that will generate the appropriate
> >> event description information.
> >> 
> >> Signed-off-by: Lluís Vilanova <vilanova@ac.upc.edu>
> >> ---
> 
> > Hi Lluís,
> > I hit an issue testing the different backends with this series applied.
> 
> > The simple trace writeout thread hangs when the 'quit' monitor command
> > is issued.  Do you see the same issue?
> 
> > $ ./configure --target-list=x86_64-softmmu --enable-trace-backend=simple
> > $ make -j4
> > $ cat my-events 
> > bdrv_open_common
> > $ x86_64-softmmu/qemu-system-x86_64 -trace events=my-events
> > (qemu) quit
> 
> I've rebased the series in master (which forced me to fix a tiny line in a
> makefile, due recent changes), and everything's working fine.
> 
> I can resend the rebased series, if you want.

Okay, sounds good.  I'll retest your rebased series.

Stefan