Re: [Qemu-devel] [PATCH v2 3/3] trace: enable all events by default

[Lluís]

Blue Swirl writes:

> On Wed, Nov 10, 2010 at 7:57 PM, Lluís <address@hidden> wrote:
>> Blue Swirl writes:
>> 
>>> On Wed, Nov 10, 2010 at 5:59 PM, Lluís <address@hidden> wrote:
>>>> So, my patch is just a matter of having all events available _only_ when
>>>> you use a backend other than "nop".
>> 
>>> Then the default for simpletrace (and dtrace?) should also be disabled
>>> initial state. If you have 1000 tracepoints automatically enabled at
>>> start (with simpletrace, not nop), disabling all of them if you are
>>> only interested in a few of them is difficult.
>> 
>> That's how it is. The patch sets always ".state=0" in "simple", and
>> "dtrace" just does not do anything unless you plug something into the
>> probe (similarly with "ust").

> Oh, I missed .state=0 part, it was also not mentioned in the
> description. Then these changes should be OK.

My fault. If these are to be merged I can cook up a new series with a
more verbose description.

Lluis

-- 
 "And it's much the same thing with knowledge, for whenever you learn
 something new, the whole world becomes that much richer."
 -- The Princess of Pure Reason, as told by Norton Juster in The Phantom
 Tollbooth