netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Hemminger <stephen@networkplumber.org>
To: "Toke Høiland-Jørgensen" <toke@toke.dk>
Cc: Thorsten Glaser <t.glaser@tarent.de>,
	Jakub Kicinski <kuba@kernel.org>,
	netdev@vger.kernel.org, Haye.Haehne@telekom.de
Subject: Re: knob to disable locally-originating qdisc optimisation?
Date: Tue, 16 May 2023 16:01:11 -0700	[thread overview]
Message-ID: <20230516160111.56b2c345@hermes.local> (raw)
In-Reply-To: <87y1loapvt.fsf@toke.dk>

On Wed, 17 May 2023 00:11:02 +0200
Toke Høiland-Jørgensen <toke@toke.dk> wrote:

> Thorsten Glaser <t.glaser@tarent.de> writes:
> 
> > On Tue, 16 May 2023, Toke Høiland-Jørgensen wrote:
> >  
> >>Pushing stuff into a
> >>qdisc so it can be ECN-marked is also nonsensical for locally generated
> >>traffic; you don't need the ECN roundtrip, you can just directly tell
> >>the local TCP sender to slow down (which is exactly what TSQ does).  
> >
> > Yes, but the point of this exercise is to develop algorithms which
> > react to ECN marking; in production, the RAN BTS will do the marking
> > so the sender will not be at the place where congestion happens, so
> > adding that kind of insight is not needed.
> >
> > Some people have asked for the ability to make Linux behave as if
> > the sender was remote to ease the test setup (i.e. require one less
> > machine), nothing more.  
> 
> Well, if it's a custom qdisc you could just call skb_orphan() on the
> skbs when enqueueing them?
> 
> -Toke

If your qdisc was upstream, others could collaborate to resolve the issue.
As it stands, the kernel development process is intentionally hostile
to out of tree developer changes like this.

  reply	other threads:[~2023-05-16 23:01 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-26 12:54 knob to disable locally-originating qdisc optimisation? Thorsten Glaser
2023-04-27 20:21 ` Jakub Kicinski
2023-04-27 23:37   ` Stephen Hemminger
2023-05-16 16:38     ` Thorsten Glaser
2023-05-16 19:23       ` Toke Høiland-Jørgensen
2023-05-16 20:20         ` Thorsten Glaser
2023-05-16 22:11           ` Toke Høiland-Jørgensen
2023-05-16 23:01             ` Stephen Hemminger [this message]
2023-05-16 23:40               ` Thorsten Glaser
2023-05-16 23:44                 ` Stephen Hemminger
2023-05-16 23:47                   ` Thorsten Glaser
2023-05-17  3:02                     ` Stephen Hemminger
2023-05-17 15:53                       ` Thorsten Glaser
2023-05-16 23:41             ` Thorsten Glaser
2023-05-17 10:00               ` Toke Høiland-Jørgensen

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=20230516160111.56b2c345@hermes.local \
    --to=stephen@networkplumber.org \
    --cc=Haye.Haehne@telekom.de \
    --cc=kuba@kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=t.glaser@tarent.de \
    --cc=toke@toke.dk \
    /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).