From: Hagen Paul Pfeifer <hagen@jauu.net>
To: Satoru Moriya <satoru.moriya@hds.com>
Cc: <netdev@vger.kernel.org>, <davem@davemloft.net>,
<nhorman@tuxdriver.com>, <tgraf@infradead.org>,
Stephen Hemminger <stephen.hemminger@vyatta.com>,
<eric.dumazet@gmail.com>, Seiji Aguchi <seiji.aguchi@hds.com>
Subject: RE: [PATCH v2 0/2] Tracepoint for tcp retransmission
Date: Fri, 03 Feb 2012 21:55:51 +0100 [thread overview]
Message-ID: <6e2d275588a21cad8c267699c5231c44@localhost> (raw)
In-Reply-To: <65795E11DBF1E645A09CEC7EAEE94B9CB8D3EA3E@USINDEVS02.corp.hds.com>
On Fri, 3 Feb 2012 15:43:30 -0500, Satoru Moriya wrote:
> Actually, we've already used systemtap in our flight recorder.
> But we believe that tcp retransmission is one of the fundamental
> function in tcp stack and so kernel itself should provide the
> instruments from which we can get enough information without
> tools which is not included in kernel.
Mhh, that's no real reason to add tracepoints all over. There where some
lengthy debates on lkml about inflationary use of tracepoints. Especially
this case where systemtap can provide the same information (btw: for
tracepoints you may also need some userspace tools). Maybe you should start
to improve systemtap. ;) But hey, it is David who decides if he is fine
with your patch.
Hagen
prev parent reply other threads:[~2012-02-03 20:55 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-20 18:07 [PATCH v2 0/2] Tracepoint for tcp retransmission Satoru Moriya
2012-01-20 18:08 ` [PATCH v2 1/2] tcp: refactor tcp_retransmit_skb() for a single return point Satoru Moriya
2012-01-20 18:09 ` [PATCH v2 2/2] tcp: add tracepoint for tcp retransmission Satoru Moriya
2012-01-20 18:50 ` [PATCH v2 0/2] Tracepoint " David Miller
2012-02-03 21:47 ` Satoru Moriya
2012-02-04 4:40 ` Yuchung Cheng
2012-02-06 18:32 ` Satoru Moriya
2012-02-04 14:28 ` Neil Horman
2012-02-04 15:58 ` Hagen Paul Pfeifer
2012-02-04 20:09 ` Neil Horman
2012-02-05 12:53 ` Frank Ch. Eigler
2012-02-05 19:17 ` Neil Horman
2012-02-05 20:04 ` Hagen Paul Pfeifer
2012-02-05 21:48 ` David Miller
2012-02-06 1:32 ` Neil Horman
2012-02-06 15:20 ` Frank Ch. Eigler
2012-02-06 15:24 ` Eric Dumazet
2012-02-06 15:38 ` Neil Horman
2012-02-06 15:53 ` Neil Horman
2012-02-06 16:18 ` Steven Rostedt
2012-02-06 17:02 ` Eric Dumazet
2012-02-06 17:18 ` Steven Rostedt
2012-02-06 16:21 ` Frank Ch. Eigler
2012-02-06 18:21 ` Satoru Moriya
2012-01-25 13:27 ` Hagen Paul Pfeifer
2012-01-25 14:44 ` Eric Dumazet
2012-01-26 18:51 ` David Miller
2012-02-03 20:31 ` Satoru Moriya
2012-02-03 20:43 ` Satoru Moriya
2012-02-03 20:55 ` Hagen Paul Pfeifer [this message]
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=6e2d275588a21cad8c267699c5231c44@localhost \
--to=hagen@jauu.net \
--cc=davem@davemloft.net \
--cc=eric.dumazet@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=nhorman@tuxdriver.com \
--cc=satoru.moriya@hds.com \
--cc=seiji.aguchi@hds.com \
--cc=stephen.hemminger@vyatta.com \
--cc=tgraf@infradead.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).