[tor-bugs] #32910 [Core Tor/Tor]: trace: Add tracepoints and userspace tracer support

Tor Bug Tracker & Wiki blackhole at torproject.org
Thu Feb 13 17:42:56 UTC 2020


#32910: trace: Add tracepoints and userspace tracer support
--------------------------+------------------------------------
 Reporter:  dgoulet       |          Owner:  dgoulet
     Type:  enhancement   |         Status:  needs_revision
 Priority:  Medium        |      Milestone:  Tor: 0.4.4.x-final
Component:  Core Tor/Tor  |        Version:
 Severity:  Normal        |     Resolution:
 Keywords:  tracing       |  Actual Points:
Parent ID:                |         Points:  3
 Reviewer:  nickm         |        Sponsor:
--------------------------+------------------------------------

Comment (by dgoulet):

 So CI is only failing because of an issue with `make check-spaces`:

 {{{
      GUARD:./src/core/or/trace_probes_circuit.h:19: Header guard macro
 mismatch.
 }}}

 The reason for this is because of the unusual guard macro that LTTng-UST
 requires:

 {{{
 #if !defined(TOR_TRACE_PROBES_CIRCUIT_H) ||
 defined(TRACEPOINT_HEADER_MULTI_READ)
 #define TOR_TRACE_PROBES_CIRCUIT_H
 }}}

 I think we need to tell `checkSpace.pl` to allow the `#if !defined(` style
 and not only the `#ifndef`.

 This specific macro style is required because LTTng-UST, for internal
 probes creation, re-include multiple time the header file and thus
 overrides it with `TRACEPOINT_HEADER_MULTI_READ`.

 I've tried to patch `./scripts/maint/checkSpace.pl` to accept it but I'm
 unable to make it work... my perl skills are very poor. Nickm, advice?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/32910#comment:8>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online


More information about the tor-bugs mailing list