From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=42223 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Q7XZ4-0003fj-ES for qemu-devel@nongnu.org; Wed, 06 Apr 2011 14:34:51 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Q7XZ2-0001LP-Jm for qemu-devel@nongnu.org; Wed, 06 Apr 2011 14:34:49 -0400 Received: from mailout-de.gmx.net ([213.165.64.23]:34077) by eggs.gnu.org with smtp (Exim 4.71) (envelope-from ) id 1Q7XZ2-0001Ko-70 for qemu-devel@nongnu.org; Wed, 06 Apr 2011 14:34:48 -0400 From: =?utf-8?b?TGx1w61z?= Date: Wed, 06 Apr 2011 20:33:41 +0200 Message-ID: <20110406183341.22854.93929.stgit@ginnungagap.bsc.es> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Subject: [Qemu-devel] [RFC][PATCH v2 00/11] trace-state: make the behaviour of "disable" consistent across all backends List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: stefanha@gmail.com, chouteau@adacore.com 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 "simple" backend now has a "-trace events=" argument to let the user select which events must be enabled from the very beginning. NOTES: * Parsing of -trace arguments is not done in the OS-specific frontends. Signed-off-by: Lluís Vilanova --- Changes in v2: * Documentation fixes. * Seggregate whitespace/indentation changes. * Minor code beautifications. * Make all -trace suboptions explicit. * Fix minor comments from Stefan. * Minor trace-events format fixes. * Integrate changes from Fabien. * Rebase on qemu.git/master (c8f930c0eeb696d638f4d4bf654e955fa44ff40f). Lluís Vilanova (11): minor whitespace/indentation fixes docs/tracing.txt: minor documentation fixes trace: [ust] fix generation of 'trace.c' on events without args trace: [trace-events] fix print formats in some events trace: [simple] minor code fixes on conditional compilation trace: generalize the "property" concept in the trace-events file trace-state: always use the "nop" backend on events with the "disable" keyword trace-state: [simple] disable all trace points by default trace-state: [simple] add "-trace events" argument to control initial state trace-state: [stderr] add support for dynamically enabling/disabling events trace: enable all events Makefile.objs | 8 + configure | 33 ++-- docs/tracing.txt | 50 +++-- hmp-commands.hx | 38 ++-- monitor.c | 24 ++- qemu-config.c | 8 + qemu-options.hx | 26 ++- scripts/tracetool | 121 +++++++------ simpletrace.h | 7 - stderrtrace.c | 24 +++ stderrtrace.h | 14 ++ trace-events | 491 ++++++++++++++++++++++++++--------------------------- vl.c | 104 ++++++++--- 13 files changed, 530 insertions(+), 418 deletions(-) create mode 100644 stderrtrace.c create mode 100644 stderrtrace.h