Re: [Qemu-devel] [PATCH v5 00/10] trace-state: make the behaviour of disable consistent across all backends

2011-07-20 Thread Lluís
Stefan Hajnoczi writes: I did leave some comments, mainly #ifdef reduction. But let's merge the next version. I'm leaving on vacations until 13th august, so I'll fix it based on your comments when I come back. Hope that's not a problem. Bye, bye! :) Lluis -- And it's much the same thing

Re: [Qemu-devel] [PATCH v5 00/10] trace-state: make the behaviour of disable consistent across all backends

2011-07-15 Thread Stefan Hajnoczi
On Tue, Jun 28, 2011 at 06:52:55PM +0200, Lluís wrote: This patch defines the disable trace event state to always use the nop backend. As a side-effect, all events are now enabled (without disable) by default, as all backends (except stderr) have programmatic support for dynamically

Re: [Qemu-devel] [PATCH v5 00/10] trace-state: make the behaviour of disable consistent across all backends

2011-07-12 Thread Stefan Hajnoczi
On Tue, Jun 28, 2011 at 5:52 PM, Lluís xscr...@gmx.net wrote: This patch defines the disable trace event state to always use the nop backend. Apologies for not reviewing sooner. I have begun and will send feedback tomorrow. Stefan

[Qemu-devel] [PATCH v5 00/10] trace-state: make the behaviour of disable consistent across all backends

2011-06-28 Thread Lluís
This patch defines the disable trace event state to always use the nop backend. As a side-effect, all events are now enabled (without disable) by default, as all backends (except stderr) have programmatic support for dynamically (de)activating each trace event. In order to make this true, the