From: nwatkins@ittc.ku.edu
To: mathieu.desnoyers@polymtl.ca
Cc: nwatkins@ittc.ku.edu, linux-kernel@vger.kernel.org
Subject: [patch 0/1] extending low-level markers
Date: Wed, 1 Aug 2007 16:57:24 -0500 [thread overview]
Message-ID: <20070801215723.GA10470@ittc.ku.edu> (raw)
Mathieu
I have been working with your Kernel Markers infrastructure now for some
time and have run into an extendability issue.
Essentially I am failing to find a way to extend the current
__trace_mark macro with site-specific context. That is, I would like the
ability to create different 'types' of instrumentation points by bulding
upon the __trace_mark macro. A consumer of this marker could examine
the type of marker, and attach an appropriate callback function /
private data.
I have included a patch which adds a flavor field to the __trace_mark
macro. This simplified example demonstrates the functionality I'm
looking for:
#define __trace_mark(flavor, name, format, args...)
<current macro plus the flavor field>
#define marker_flavor_XXX(name, format, args...)
__trace_mark(XXX, name, format, args)
Here a marker of type XXX is build upon the __trace_mark macro. When a
consumer of type XXX finds markers with the XXX flavor appropriate
registration can take place.
Unless I don't fully understand all the use cases of the markers, I
don't see any other way to do this except to encode the 'type'
information in the name of the marker, and require the consumer to parse
the string to determine the type. Restricting the names of the markers
in this way seems like a bad solution.
Any help and feedback is greatly appreciated.
- Noah
--
next reply other threads:[~2007-08-01 21:58 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-01 21:57 nwatkins [this message]
2007-08-02 16:44 ` [patch 0/1] extending low-level markers Mathieu Desnoyers
2007-08-02 18:32 ` Noah Watkins
2007-08-02 19:02 ` Mathieu Desnoyers
2007-08-02 19:31 ` Noah Watkins
2007-08-02 19:38 ` Mathieu Desnoyers
2007-08-02 19:58 ` Noah Watkins
2007-08-07 19:50 ` Frank Ch. Eigler
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20070801215723.GA10470@ittc.ku.edu \
--to=nwatkins@ittc.ku.edu \
--cc=linux-kernel@vger.kernel.org \
--cc=mathieu.desnoyers@polymtl.ca \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.