From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:54172) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SDJgG-0004iA-C0 for qemu-devel@nongnu.org; Thu, 29 Mar 2012 14:02:41 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SDJgA-0005nz-Ii for qemu-devel@nongnu.org; Thu, 29 Mar 2012 14:02:39 -0400 Received: from mail-wg0-f53.google.com ([74.125.82.53]:50925) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SDJgA-0005ne-9a for qemu-devel@nongnu.org; Thu, 29 Mar 2012 14:02:34 -0400 Received: by wgbfm10 with SMTP id fm10so1785476wgb.10 for ; Thu, 29 Mar 2012 11:02:32 -0700 (PDT) Date: Thu, 29 Mar 2012 18:16:31 +0100 From: Stefan Hajnoczi Message-ID: <20120329171631.GA20821@stefanha-thinkpad.localdomain> References: <20120327194910.7241.47693.stgit@ginnungagap.bsc.es> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20120327194910.7241.47693.stgit@ginnungagap.bsc.es> Subject: Re: [Qemu-devel] [RFC PATCH v3 0/8] Rewrite tracetool using python modules List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?Llu=EDs?= Vilanova Cc: qemu-devel@nongnu.org On Tue, Mar 27, 2012 at 09:49:12PM +0200, Lluís Vilanova wrote: > A full rewrite of the tracetool script using per-format and per-backend modules, > so that it's easier to read and extend it in the future. > > Signed-off-by: Lluís Vilanova > --- > NOTE: This series applies in current master, ignoring the "Rewrite tracetool > using python" series. > > Changes in v3: > > * Some minor fixes according to comments from Alon Levy: > ** Fixed some rogue tabs. > ** Fixed definition of '_SCRIPT' in "tracetool.py". > ** Fixed some documentation. > ** Defensively treat results from 'try_import' in 'get_list'. > ** Fixed the no-arguments case in 'tracetool.backend.dtrace.d'. > ** Fixed the commandline checks for the "stap" format. > ** Added '__repr__' methods for both Event and Arguments objects. > > > Changes in v2: > > * Fixed a strange import error. > * Add a pointer to 'tracetool.out' in the format and backend documentation. > > Lluís Vilanova (8): > tracetool: Rewrite infrastructure as python modules > tracetool: Add module for the 'c' format > tracetool: Add module for the 'h' format > tracetool: Add support for the 'stderr' backend > tracetool: Add support for the 'simple' backend > tracetool: Add support for the 'ust' backend > tracetool: Add support for the 'dtrace' backend > tracetool: Add MAINTAINERS info > > > MAINTAINERS | 2 > Makefile.objs | 6 > Makefile.target | 13 - > configure | 4 > scripts/tracetool | 648 --------------------------------- > scripts/tracetool.py | 141 +++++++ > scripts/tracetool/__init__.py | 223 +++++++++++ > scripts/tracetool/backend/__init__.py | 119 ++++++ > scripts/tracetool/backend/dtrace.py | 104 +++++ > scripts/tracetool/backend/simple.py | 60 +++ > scripts/tracetool/backend/stderr.py | 61 +++ > scripts/tracetool/backend/ust.py | 102 +++++ > scripts/tracetool/format/__init__.py | 96 +++++ > scripts/tracetool/format/c.py | 20 + > scripts/tracetool/format/d.py | 20 + > scripts/tracetool/format/h.py | 45 ++ > scripts/tracetool/format/stap.py | 20 + > 17 files changed, 1025 insertions(+), 659 deletions(-) > delete mode 100755 scripts/tracetool > create mode 100755 scripts/tracetool.py > create mode 100644 scripts/tracetool/__init__.py > create mode 100644 scripts/tracetool/backend/__init__.py > create mode 100644 scripts/tracetool/backend/dtrace.py > create mode 100644 scripts/tracetool/backend/simple.py > create mode 100644 scripts/tracetool/backend/stderr.py > create mode 100644 scripts/tracetool/backend/ust.py > create mode 100644 scripts/tracetool/format/__init__.py > create mode 100644 scripts/tracetool/format/c.py > create mode 100644 scripts/tracetool/format/d.py > create mode 100644 scripts/tracetool/format/h.py > create mode 100644 scripts/tracetool/format/stap.py Only one point: please don't introduce PUBLIC yet. Let's add it when it's needed. At the moment nothing uses it. I have tested this series with all backends and looked at the diff between the old tracetool and tracetool.py. I'm happy with this series. We need to test it hard for 1.1 to make sure there are no hickups for tracing users. Stefan