From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43071) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cYCUY-0005jn-0q for qemu-devel@nongnu.org; Mon, 30 Jan 2017 08:59:35 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cYCUU-0003rR-7j for qemu-devel@nongnu.org; Mon, 30 Jan 2017 08:59:34 -0500 Received: from mail-wm0-x241.google.com ([2a00:1450:400c:c09::241]:34363) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1cYCUT-0003rI-VB for qemu-devel@nongnu.org; Mon, 30 Jan 2017 08:59:30 -0500 Received: by mail-wm0-x241.google.com with SMTP id c85so77392263wmi.1 for ; Mon, 30 Jan 2017 05:59:29 -0800 (PST) Date: Mon, 30 Jan 2017 13:59:26 +0000 From: Stefan Hajnoczi Message-ID: <20170130135926.GC2118@stefanha-x1.localdomain> References: <20170125161417.31949-1-berrange@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="HG+GLK89HZ1zG0kk" Content-Disposition: inline In-Reply-To: <20170125161417.31949-1-berrange@redhat.com> Subject: Re: [Qemu-devel] [PATCH v4 0/8] Switch all subdirs over to modular trace.h file List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Daniel P. Berrange" Cc: qemu-devel@nongnu.org, Paolo Bonzini , =?iso-8859-1?Q?Llu=EDs?= Vilanova , Stefan Hajnoczi --HG+GLK89HZ1zG0kk Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Jan 25, 2017 at 04:14:09PM +0000, Daniel P. Berrange wrote: > This is the final part of my trace events refactoring. >=20 > Previously we merged code that split trace-events up > into one file per sub-directory. We also merged code > to the code-generator that enables us to generate > and use multiple independant sets of trace events. >=20 > This is the final glue that makes use of these two > earlier changes, so that we actually generate a > separate trace.h & trace.c file per sub-directory. >=20 > The result is that when adding a new trace event to > a file, only files in that sub-directory get rebuilt. > Theoretically the build is faster too, since each > source file includes a much smaller trace.h that > only contains probes relevant to it. In reality this > speed benefit is probably marginal & lost in the noise. >=20 > Changed in v4: >=20 > - Fix commit message in first patch > - Re-add -I$(@D) to include path >=20 > Changed in v3: >=20 > - It no longer modifies the Makefile.objs rules in each > sub-directory. A bit of magic in the top level > Makefile.objs file lets us generate all the needed > rules from one place >=20 > - It sticks with bare "trace.h" includes instead of adding > sub-dir prefix "hw/block/trace.h". To achieve this we > had to rename the trace.h to trace-root.h in the top > level directory to avoid ambiguity in places where a > file needs to include the top-level instead of local > trace.h >=20 >=20 > Daniel P. Berrange (8): > make: move top level dir to end of include search path > trace: move hw/block/dataplane events to correct subdir > trace: move hw/xen events to correct subdir > trace: move hw/i386/xen events to correct subdir > trace: move setting of group name into Makefiles > trace: switch to modular code generation for sub-directories > trace: update docs to reflect new code generation approach > trace: improve error reporting when parsing simpletrace header >=20 > .gitignore | 22 +++-- > Makefile | 156 ++++++++++++++++++++++++= ++---- > Makefile.objs | 99 ++++++++++--------- > Makefile.target | 10 +- > aio-posix.c | 2 +- > balloon.c | 2 +- > block.c | 2 +- > blockdev-nbd.c | 1 - > blockdev.c | 2 +- > blockjob.c | 1 - > cpu-exec.c | 2 +- > dma-helpers.c | 2 +- > docs/tracing.txt | 55 ++++++++--- > exec.c | 2 +- > hw/block/dataplane/trace-events | 6 ++ > hw/block/trace-events | 5 - > hw/i386/trace-events | 7 -- > hw/i386/xen/trace-events | 6 ++ > hw/net/fsl_etsec/etsec.c | 1 - > hw/xen/trace-events | 13 +++ > include/exec/cpu_ldst_template.h | 2 +- > include/exec/cpu_ldst_useronly_template.h | 2 +- > include/hw/xen/xen_common.h | 2 +- > include/trace.h | 6 -- > ioport.c | 2 +- > kvm-all.c | 2 +- > memory.c | 2 +- > monitor.c | 2 +- > qom/cpu.c | 2 +- > rules.mak | 30 ++++-- > scripts/simpletrace.py | 10 +- > scripts/tracetool.py | 31 +++--- > scripts/tracetool/backend/dtrace.py | 7 +- > scripts/tracetool/backend/simple.py | 1 - > scripts/tracetool/backend/ust.py | 7 +- > scripts/tracetool/format/c.py | 7 +- > scripts/tracetool/format/tcg_h.py | 6 +- > scripts/tracetool/format/tcg_helper_c.py | 6 +- > scripts/tracetool/format/ust_events_c.py | 2 +- > scripts/tracetool/format/ust_events_h.py | 7 +- > spice-qemu-char.c | 2 +- > tests/Makefile.include | 2 +- > thread-pool.c | 2 +- > trace-events | 10 -- > trace/Makefile.objs | 93 ++---------------- > trace/control-target.c | 2 +- > trace/control.c | 2 +- > trace/ftrace.c | 2 +- > trace/simple.c | 1 - > translate-all.c | 2 +- > vl.c | 2 +- > xen-hvm.c | 2 +- > xen-mapcache.c | 2 +- > 53 files changed, 398 insertions(+), 258 deletions(-) > create mode 100644 hw/block/dataplane/trace-events > create mode 100644 hw/i386/xen/trace-events > create mode 100644 hw/xen/trace-events > delete mode 100644 include/trace.h Running through Travis-CI. Thanks, applied to my tracing tree: https://github.com/stefanha/qemu/commits/tracing Stefan --HG+GLK89HZ1zG0kk Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEcBAEBAgAGBQJYj0a+AAoJEJykq7OBq3PIYrEH/0wVZckPyqMyMQFFc7Fmp9Cp OCx1cZVGnM9tEFhbENLO9wdb4TJ8HvVwXeskj6MjTKqvtXH6pKjpePZs4UvQsE1i ArKXEQtMa/fokj3Oi/QVnGgOwwQvahBRQxHMJBK3WoIIccQZ1NO6vKfwRZwGU42p OHxKSy3oFWgSP6C9YtiBRYVf9+EjxNtprpUipPo1Coy3wHy9ARsK2W35JsZTDIYh LTiukzRUwEvbwup/BC7plJh8DTsWEVoCRF8qplK8Rv+8uVCG1reSA0GMCBxm8brA 9WCc1sPmX4DhnQJD2bIEGzOh41Iq47n1U0Qu7y+MOCWBsGnsehhsQv8jJgxyMuE= =r25g -----END PGP SIGNATURE----- --HG+GLK89HZ1zG0kk--