From: David Miller <davem@davemloft.net>
To: ilpo.jarvinen@helsinki.fi
Cc: netdev@vger.kernel.org
Subject: Re: [RFC PATCH 09/10] [TCP]: Rewrite SACK block processing & sack_recv_cache use
Date: Mon, 19 Nov 2007 22:08:07 -0800 (PST) [thread overview]
Message-ID: <20071119.220807.216500794.davem@davemloft.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0711161306490.29353@kivilampi-30.cs.helsinki.fi>
From: "Ilpo_Järvinen" <ilpo.jarvinen@helsinki.fi>
Date: Fri, 16 Nov 2007 15:44:40 +0200 (EET)
> Besides the log(n) search for which patches already exists, only O(new
> information) scan start..end range would be necessary. Or had you
> something else in mind (I'm not that sure what the "ideas", in plural,
> were :-))? ...I haven't forgotten the RB-tree things either (and I'm
> well aware of the limited scope of things this patch addresses).
Since you brought it up, can you respin the RB-tree patches? Let's
just put them into net-2.6.25 and let them cook in there for a long
time.
If they prove to be too expensive in the no-loss case we can do
something like lazy tree creation or simply improving the
insert/delete implmentation.
I think it's important to put RB-tree in because a lot of the ideas
we are discussing have different implications if we go one way or
the other so we should commit to something now in order to focus
our efforts better.
next prev parent reply other threads:[~2007-11-20 6:08 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-11-15 13:37 [RFC PATCH net-2.6.25 0/10] [TCP]: Cleanups, tweaks & sacktag recode Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 01/10] [TCP]: Move !in_sack test earlier in sacktag & reorganize if()s Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 02/10] [TCP]: Extend reordering detection to cover CA_Loss partially Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 3/10] [TCP]: non-FACK SACK follows conservative SACK loss recovery Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 04/10] [TCP]: Convert highest_sack to sk_buff to allow direct access Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 05/10] [TCP]: Make lost retrans detection more self-contained Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 06/10] [TCP]: Prior_fackets can be replaced by highest_sack seq Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 07/10] [TCP]: Create tcp_sacktag_one() Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 08/10] [TCP]: Earlier SACK block verification & simplify access to them Ilpo Järvinen
2007-11-15 13:37 ` [RFC PATCH 09/10] [TCP]: Rewrite SACK block processing & sack_recv_cache use Ilpo Järvinen
2007-11-15 13:37 ` [PATCH 10/10] [TCP]: Track sacktag (DEVEL PATCH) Ilpo Järvinen
2007-11-16 4:00 ` [RFC PATCH 09/10] [TCP]: Rewrite SACK block processing & sack_recv_cache use David Miller
2007-11-16 13:44 ` Ilpo Järvinen
2007-11-20 6:08 ` David Miller [this message]
2007-11-20 9:22 ` Ilpo Järvinen
2007-11-16 3:50 ` [PATCH 08/10] [TCP]: Earlier SACK block verification & simplify access to them David Miller
2007-11-16 3:45 ` [PATCH 07/10] [TCP]: Create tcp_sacktag_one() David Miller
2007-11-16 3:44 ` [PATCH 06/10] [TCP]: Prior_fackets can be replaced by highest_sack seq David Miller
2007-11-16 3:43 ` [PATCH 05/10] [TCP]: Make lost retrans detection more self-contained David Miller
2007-11-16 3:42 ` [PATCH 04/10] [TCP]: Convert highest_sack to sk_buff to allow direct access David Miller
2007-11-16 3:41 ` [PATCH 3/10] [TCP]: non-FACK SACK follows conservative SACK loss recovery David Miller
2007-11-16 3:35 ` [PATCH 02/10] [TCP]: Extend reordering detection to cover CA_Loss partially David Miller
2007-11-16 3:33 ` [PATCH 01/10] [TCP]: Move !in_sack test earlier in sacktag & reorganize if()s David Miller
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=20071119.220807.216500794.davem@davemloft.net \
--to=davem@davemloft.net \
--cc=ilpo.jarvinen@helsinki.fi \
--cc=netdev@vger.kernel.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).