public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: torvalds@transmeta.com (Linus Torvalds)
To: linux-kernel@vger.kernel.org
Subject: Re: Is sendfile all that sexy?
Date: 14 Jan 2001 12:22:31 -0800	[thread overview]
Message-ID: <93t1q7$49c$1@penguin.transmeta.com> (raw)
In-Reply-To: <Pine.GSO.4.30.0101141237020.12354-100000@shell.cyberus.ca>

In article <Pine.GSO.4.30.0101141237020.12354-100000@shell.cyberus.ca>,
jamal  <hadi@cyberus.ca> wrote:
>
>Before getting excited i had the courage to give plain 2.4.0-pre3 a whirl
>and somethings bothered me.

Note that "sendfile(fd, file, len)" is never going to be faster than
"write(fd, userdata, len)". 

That's not the point of sendfile(). The point of sendfile() is to be
faster than the _combination_ of:

	addr = mmap(file, ...len...);
	write(fd, addr, len);

or

	read(file, userdata, len);
	write(fd, userdata, len);

and in your case you're not comparing sendfile() against this
combination.  You're just comparing sendfile() against a simple
"write()".

And no, I don't actually hink that sendfile() is all that hot. It was
_very_ easy to implement, and can be considered a 5-minute hack to give
a feature that fit very well in the MM architecture, and that the Apache
folks had already been using on other architectures.

The only obvious use for it is file serving, and as high-performance
file serving tends to end up as a kernel module in the end anyway (the
only hold-out is samba, and that's been discussed too), "sendfile()"
really is more a proof of concept than anything else.

Does anybody but apache actually use it?

			Linus

PS.  I still _like_ sendfile(), even if the above sounds negative.  It's
basically a "cool feature" that has zero negative impact on the design
of the system.  It uses the same "do_generic_file_read()" that is used
for normal "read()", and is also used by the loop device and by
in-kernel fileserving.  But it's not really "important". 

-
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-14 20:23 UTC|newest]

Thread overview: 130+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-01-14 18:29 Is sendfile all that sexy? jamal
2001-01-14 18:50 ` Ingo Molnar
2001-01-14 19:02   ` jamal
2001-01-14 19:09     ` Ingo Molnar
2001-01-14 19:18       ` jamal
2001-01-14 20:22 ` Linus Torvalds [this message]
2001-01-14 20:38   ` Ingo Molnar
2001-01-14 21:44     ` Linus Torvalds
2001-01-14 21:49       ` Ingo Molnar
2001-01-14 21:54     ` Gerhard Mack
2001-01-14 22:40       ` Linus Torvalds
2001-01-14 22:45         ` J Sloan
2001-01-15 20:15           ` H. Peter Anvin
2001-01-15  3:43         ` Michael Peddemors
2001-01-15 13:02       ` Florian Weimer
2001-01-15 13:45         ` Tristan Greaves
2001-01-15  1:14   ` Dan Hollis
2001-01-15 15:24   ` Jonathan Thackray
2001-01-15 15:36     ` Matti Aarnio
2001-01-15 20:17       ` H. Peter Anvin
2001-01-15 16:05     ` dean gaudet
2001-01-15 18:34       ` Jonathan Thackray
2001-01-15 18:46         ` Linus Torvalds
2001-01-15 20:47           ` [patch] sendpath() support, 2.4.0-test3/-ac9 Ingo Molnar
2001-01-16  4:51             ` dean gaudet
2001-01-16  4:59               ` Linus Torvalds
2001-01-16  9:48                 ` 'native files', 'object fingerprints' [was: sendpath()] Ingo Molnar
2000-01-01  2:02                   ` Pavel Machek
2001-01-16 11:13                   ` Andi Kleen
2001-01-16 11:26                     ` Ingo Molnar
2001-01-16 11:37                       ` Andi Kleen
2001-01-16 12:04                         ` O_ANY [was: Re: 'native files', 'object fingerprints' [was: sendpath()]] Ingo Molnar
2001-01-16 12:09                           ` Ingo Molnar
2001-01-16 12:13                           ` Peter Samuelson
2001-01-16 12:33                             ` Ingo Molnar
2001-01-16 14:40                               ` Felix von Leitner
2001-01-16 12:34                           ` Andi Kleen
2001-01-16 13:00                           ` Mitchell Blank Jr
2001-01-16 13:57                   ` 'native files', 'object fingerprints' [was: sendpath()] Jamie Lokier
2001-01-16 14:27                   ` Felix von Leitner
2001-01-16 17:47                   ` Linus Torvalds
2001-01-17  4:39                   ` dean gaudet
2001-01-16  9:19               ` [patch] sendpath() support, 2.4.0-test3/-ac9 Ingo Molnar
2001-01-17  0:03                 ` dean gaudet
2001-01-15 18:58         ` Is sendfile all that sexy? dean gaudet
2001-01-15 19:41     ` Ingo Molnar
2001-01-15 20:33       ` Albert D. Cahalan
2001-01-15 21:00         ` Linus Torvalds
2001-01-16 10:40         ` Felix von Leitner
2001-01-16 11:56           ` Peter Samuelson
2001-01-16 12:37           ` Ingo Molnar
2001-01-16 12:42           ` Ingo Molnar
2001-01-16 12:47             ` Felix von Leitner
2001-01-16 13:48               ` Jamie Lokier
2001-01-16 14:20                 ` Felix von Leitner
2001-01-16 15:05                   ` David L. Parsley
2001-01-16 15:05                     ` Jakub Jelinek
2001-01-16 15:46                       ` David L. Parsley
2001-01-18 14:00                         ` Laramie Leavitt
2001-01-17 19:27                     ` dean gaudet
2001-01-24  0:58   ` Sasi Peter
2001-01-24  8:44     ` James Sutherland
2001-01-25 10:20     ` Anton Blanchard
2001-01-25 10:58       ` Sasi Peter
2001-01-26  6:10         ` Anton Blanchard
2001-01-26 11:46           ` David S. Miller
2001-01-26 14:12             ` Anton Blanchard
2001-01-15 23:16 ` Pavel Machek
2001-01-16 13:47   ` jamal
2001-01-16 14:41     ` Pavel Machek
  -- strict thread matches above, loose matches on Subject: below --
2001-01-16 13:50 Andries.Brouwer
2001-01-17  6:56 ` Ton Hospel
2001-01-17  7:31   ` Steve VanDevender
2001-01-17  8:09     ` Ton Hospel
2001-01-17 15:02 Ben Mansell
2000-01-01  2:10 ` Pavel Machek
2001-01-17 19:32 ` Linus Torvalds
2001-01-18  2:34   ` Olivier Galibert
2001-01-21 21:22     ` LA Walsh
2001-01-18  8:23   ` Rogier Wolff
2001-01-18 10:01     ` Andreas Dilger
2001-01-18 11:04       ` Russell Leighton
2001-01-18 16:36         ` Larry McVoy
2001-01-19  1:53         ` Linus Torvalds
2001-01-18 16:24       ` Linus Torvalds
2001-01-18 18:46         ` Kai Henningsen
2001-01-18 18:58         ` Roman Zippel
2001-01-18 19:42           ` Linus Torvalds
2001-01-19  0:18             ` Roman Zippel
2001-01-19  1:14               ` Linus Torvalds
2001-01-19  6:57                 ` Alan Cox
2001-01-19 10:13                 ` Roman Zippel
2001-01-19 10:55                   ` Andre Hedrick
2001-01-19 20:18                   ` kuznet
2001-01-19 21:45                     ` Linus Torvalds
2001-01-20 18:53                       ` kuznet
2001-01-20 19:26                         ` Linus Torvalds
2001-01-20 21:20                           ` Roman Zippel
2001-01-21  0:25                             ` Linus Torvalds
2001-01-21  2:03                               ` Roman Zippel
2001-01-21 18:00                               ` kuznet
2001-01-21 23:21                           ` David Woodhouse
2001-01-20 15:36             ` Kai Henningsen
2001-01-20 21:01               ` Linus Torvalds
2001-01-20 21:10                 ` Mo McKinlay
2001-01-20 22:24                 ` Roman Zippel
2001-01-21  0:33                   ` Linus Torvalds
2001-01-21  1:29                     ` David Schwartz
2001-01-21  2:42                     ` Roman Zippel
2001-01-21  9:52                     ` James Sutherland
2001-01-21 10:02                       ` Ingo Molnar
2001-01-22  9:52                       ` Helge Hafting
2001-01-22 13:00                         ` James Sutherland
2001-01-23  9:01                           ` Helge Hafting
2001-01-23  9:37                             ` James Sutherland
2001-01-18 19:51           ` Rick Jones
2001-01-18 12:17     ` Peter Samuelson
2001-01-22 18:13   ` Val Henson
2001-01-22 18:27     ` David Lang
2001-01-22 19:37       ` Val Henson
2001-01-22 20:01         ` David Lang
2001-01-22 22:04           ` Ion Badulescu
2001-01-22 18:54     ` Linus Torvalds
     [not found] <Pine.LNX.4.10.10101190911130.10218-100000@penguin.transmeta.com>
2001-01-19 17:23 ` Rogier Wolff
2001-01-24 15:12 Sasi Peter
2001-01-24 15:29 ` James Sutherland
2001-01-25  1:11 ` Alan Cox
2001-01-25  9:06   ` James Sutherland
2001-01-25 10:42     ` bert hubert
2001-01-25 12:14       ` James Sutherland

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='93t1q7$49c$1@penguin.transmeta.com' \
    --to=torvalds@transmeta.com \
    --cc=linux-kernel@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