* Re: [Qemu-devel] [PATCH v10 0/7] trace: Generic event state description
[not found] ` <87fw0g4kic.fsf@fimbulvetr.bsc.es>
@ 2013-03-01 10:20 ` Stefan Hajnoczi
0 siblings, 0 replies; only message in thread
From: Stefan Hajnoczi @ 2013-03-01 10:20 UTC (permalink / raw)
To: Lluís Vilanova; +Cc: Blue Swirl, qemu-devel
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
^ permalink raw reply [flat|nested] only message in thread