mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: kuznet@ms2.inr.ac.ru
To: andrea@suse.de (Andrea Arcangeli)
Cc: mingo@elte.hu, torvalds@transmeta.com, raj@cup.hp.com,
	linux-kernel@vger.kernel.org, davem@redhat.com
Subject: Re: [Fwd: [Fwd: Is sendfile all that sexy? (fwd)]]
Date: Fri, 19 Jan 2001 21:18:04 +0300 (MSK)	[thread overview]
Message-ID: <200101191818.VAA24360@ms2.inr.ac.ru> (raw)
In-Reply-To: <20010119162552.D3447@athlon.random> from "Andrea Arcangeli" at Jan 19, 1 04:25:52 pm

Hello!

> The "uncork" won't push the last skb on the wire if there is not acknowledged
> data in the write_queue and the payload of the last skb in the write_queue
> isn't large MSS. This because the `uncork' will only re-evaluate the
> write_queue in function of the _nagle_ algorithm, quite correctly because the
> "uncork" will move frok "cork" to "nagle" (not from "cork" to "nodelay").

At least for your own implementation of SIOCPUSH has "1" among
arguments of push_pending_frames, so that this does not happen. 8)8)

The second thing, which makes argument above wrong is that
both classic Nagle and linux-2.4 Nagle (extended by Minshall),
do not have this problem. Your argument applies to buggy flavor
of nagling specific to 2.2.


However, SIOCPUSH really affects latency badly in some curcumstances.
Actually, Ingo already learned this from bad experience with TUX. 8)
Namely, when push cannot push anything due to congestion window
or receiver window, http/1.0 style synchronous connections suffer.
The lesson is that when reply is better to be disabled.

Alexey
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

  reply	other threads:[~2001-01-19 18:34 UTC|newest]

Thread overview: 84+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <3A65E825.FFEB194@cup.hp.com>
2001-01-17 19:27 ` Linus Torvalds
2001-01-17 20:03   ` Rick Jones
2001-01-17 20:38     ` dean gaudet
2001-01-17 20:57       ` Rick Jones
2001-01-18 13:06         ` Ingo Molnar
2001-01-18 14:57           ` Andi Kleen
2001-01-18 18:24           ` Rick Jones
2001-01-19  2:46         ` dean gaudet
2001-01-19  3:03           ` dean gaudet
2001-01-19 19:07           ` Rick Jones
2001-01-19 20:03             ` kuznet
2001-01-19 21:20               ` Rick Jones
2001-01-20 18:03                 ` kuznet
2001-01-22 18:44                   ` Rick Jones
2001-01-20 14:56         ` Kai Henningsen
2001-01-23  7:20           ` dean gaudet
2001-01-20 23:09         ` Lincoln Dale
2001-01-20 23:27           ` James Sutherland
2001-01-21  0:02             ` Chris Wedgwood
2001-01-18 12:56       ` Ingo Molnar
2001-01-25 17:58         ` Jamie Lokier
2001-01-17 21:22     ` Linus Torvalds
2001-01-17 22:17       ` Rick Jones
2001-01-17 22:53         ` Linus Torvalds
2001-01-17 22:44           ` Jonathan Walther
2001-01-18 13:18           ` Ingo Molnar
2001-01-18 16:49             ` Linus Torvalds
2001-01-18 17:32               ` Ingo Molnar
2001-01-18 17:49               ` Zach Brown
2001-01-19  3:16                 ` dean gaudet
2001-01-19  3:56                   ` David Ford
2001-01-18 18:29               ` Rick Jones
2001-01-18 18:50                 ` Linus Torvalds
2001-01-18 19:38               ` Andrea Arcangeli
2001-01-18 19:43                 ` Ingo Molnar
2001-01-18 19:52                   ` Linus Torvalds
2001-01-18 20:11                     ` kuznet
2001-01-18 20:33                     ` Ingo Molnar
2001-01-18 21:14                     ` Andrea Arcangeli
2001-01-18 20:24                   ` Andrea Arcangeli
2001-01-18 20:37                     ` kuznet
2001-01-18 21:04                       ` Andrea Arcangeli
2001-01-19  0:27                         ` Olivier Galibert
2001-01-19  0:59                           ` Rick Jones
2001-01-19 17:52                         ` kuznet
2001-01-19 20:54                           ` Andrea Arcangeli
2001-01-18 20:44                     ` Ingo Molnar
2001-01-18 21:54                       ` Andrea Arcangeli
2001-01-18 21:57                         ` Ingo Molnar
2001-01-18 22:16                           ` Andrea Arcangeli
2001-01-18 22:18                             ` Ingo Molnar
2001-01-19 15:25                               ` Andrea Arcangeli
2001-01-19 18:18                                 ` kuznet [this message]
2001-01-19 21:13                                   ` Andrea Arcangeli
2001-01-20 17:28                                     ` kuznet
2001-01-20 18:14                                       ` Abramo Bagnara
2001-01-20 18:23                                       ` Andrea Arcangeli
2001-01-20 19:05                                         ` kuznet
2001-01-20 19:30                                           ` Andrea Arcangeli
2001-01-20 19:39                                             ` Linus Torvalds
2001-01-20 20:22                                               ` kuznet
2001-01-20 21:20                                                 ` Andrea Arcangeli
2001-01-20 20:56                                               ` Andrea Arcangeli
2001-01-21 18:37                                                 ` kuznet
2001-01-20 19:39                                             ` kuznet
2001-01-20 21:05                                               ` Andrea Arcangeli
2001-01-20 21:31                                               ` Guus Sliepen
2001-01-18 22:20                             ` Ingo Molnar
2001-01-18 19:45                 ` Linus Torvalds
2001-01-18 19:59                 ` kuznet
2001-01-18 20:44                   ` Andrea Arcangeli
2001-01-19  3:25                 ` dean gaudet
2001-01-19  3:35                   ` dean gaudet
2001-01-18  9:34         ` Andi Kleen
2001-01-18 18:20           ` Rick Jones
2001-01-18 19:45             ` Andi Kleen
2001-01-18 20:30             ` kuznet
2001-01-18 20:50               ` Ingo Molnar
2001-01-18 22:49               ` Rick Jones
2001-01-18 13:29         ` Ingo Molnar
2001-01-18 16:51           ` Linus Torvalds
2001-01-18 17:04             ` Ingo Molnar
2001-01-17 21:51 Dan Kegel
2001-01-24  0:19 Cacophonix

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=200101191818.VAA24360@ms2.inr.ac.ru \
    --to=kuznet@ms2.inr.ac.ru \
    --cc=andrea@suse.de \
    --cc=davem@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=raj@cup.hp.com \
    --cc=torvalds@transmeta.com \
    /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

all inboxes | Powered by JetHome®