public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Andrea Arcangeli <andrea@suse.de>
To: kuznet@ms2.inr.ac.ru
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: Sat, 20 Jan 2001 19:23:20 +0100	[thread overview]
Message-ID: <20010120192320.J8717@athlon.random> (raw)
In-Reply-To: <20010119221327.C8717@athlon.random> <200101201728.UAA04351@ms2.inr.ac.ru>
In-Reply-To: <200101201728.UAA04351@ms2.inr.ac.ru>; from kuznet@ms2.inr.ac.ru on Sat, Jan 20, 2001 at 08:28:04PM +0300

On Sat, Jan 20, 2001 at 08:28:04PM +0300, kuznet@ms2.inr.ac.ru wrote:
> Hello!
> 
> > My argument applies to 2.4. The uncork _won't_ push on the wire the last
> > not mss-sized fragment until it's the last one in the write queue even once
> > cwnd and receiver window allows that. I think 
> 
> Look at the code again. You misread it.

This is possible indeed, but in a few minutes re-read I still understand the
same thing as yesterday: if there's only 1 packet in the write queue pointed by
the send_head the nonagle will remains zero and the tcp_nagle_check will get
nonagle zero as well and it will compute the nagle default algorithm, this last
packet will be delayed normally in function of the nagle algortith.  I can
think of a:
 
        CORK
        write(1)
        UNCORK
 
and the 1 byte of payload will stay there until there will be nothing in flight
and all previously sent data will be acknowledged from the receiver. It seems
even more clear that we can delay the push in the wire of the last packet with
the uncorking if the receiver window or the congestion window forbid us to xmit
such packet immediatly in the uncork call. I admit I didn't looked at all into
the tcp_minshall_check due lack of time (that in turn means I don't know the
semantics of snd_sml), maybe it makes the difference but then I don't see how.
The rest seems quite obvious, packets_out indicates that we sent packets and
that those packets aren't acknowledged yet.  I would have written a testcase to
try in practice what happens but I'm busy, I apologise for bothering if I'm
totally wrong.  However if you have the time you would make me a favour if you
could tell _where_ my understanding of the code is wrong and what exactly I'm
missing.  Thx!

Andrea
-
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/

  parent reply	other threads:[~2001-01-20 18:28 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 ` [Fwd: [Fwd: Is sendfile all that sexy? (fwd)]] 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
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 [this message]
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
  -- strict thread matches above, loose matches on Subject: below --
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=20010120192320.J8717@athlon.random \
    --to=andrea@suse.de \
    --cc=davem@redhat.com \
    --cc=kuznet@ms2.inr.ac.ru \
    --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