mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Changli Gao <xiaosuo@gmail.com>
To: Amerigo Wang <xiyou.wangcong@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: PROPOSAL: extend pipe() to support NULL argument.
Date: Fri, 3 Jul 2009 12:08:58 +0800	[thread overview]
Message-ID: <412e6f7f0907022108p7c533ed2wd16fceb0f282ed62@mail.gmail.com> (raw)
In-Reply-To: <20090703030008.GD5880@cr0.nay.redhat.com>

On Fri, Jul 3, 2009 at 11:00 AM, Amerigo Wang<xiyou.wangcong@gmail.com> wrote:
>
> When I said 'pipe', I meant *anonymous* pipe, definitely!
> There is nothing related with *named* pipe here.
>
Pipe means *anonymous* pipe? Who defined that? What the difference
between an anonymou pipe and a named pipe? Two ends or one ends? I
don't think so. Having explicit name is the right difference between
them.

And when I said 'kernel buffer handler', I means it is not a
traditional pipe, and I just extend and reuse the pipe API to create a
kernel buffer handler for splicing use.

> You are going to a *wrong* direction.
>
What is the right direction you think? Keep the pipe() API unchanged
and keep two fds refering to the same kernel buffer? If you want to
keep the API unchanged just for compatiblity with POSIX, I agree with
you, and maybe a new system call is needed.

>
> If you repeat your (pipe,splice,splice,close,close) sequence for
> thousands times, it is still the same, nothing saves...
>
You means that?

while (<condition>) {
 pipe/splice/splice/close/close?
}

You don't know my meaning. As a proxy server, there maybe lots of
connections to maintain, and these connections will keep open for a
long time. If the data received can be sent in a relay cycle, the
kernel buffer can be reused. If not, the kernel buffer must be
reserved. When there are lots of these kinds of connections, lots of
kernel buffers must be reserved. At this time, whether two fds per
kernel buffer or one fds per kernel buffer matters.


-- 
Regards,
Changli Gao(xiaosuo@gmail.com)

  reply	other threads:[~2009-07-03  4:09 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-02  7:36 Changli Gao
2009-07-02  9:13 ` Amerigo Wang
2009-07-02  9:18   ` Changli Gao
2009-07-02  9:46     ` Amerigo Wang
2009-07-02 10:04       ` Changli Gao
2009-07-02 10:21         ` Amerigo Wang
2009-07-03  0:43           ` Changli Gao
2009-07-03  1:55             ` Amerigo Wang
2009-07-03  2:10               ` Changli Gao
2009-07-03  3:00                 ` Amerigo Wang
2009-07-03  4:08                   ` Changli Gao [this message]
2009-07-03  5:19                     ` Amerigo Wang
2009-07-03  5:42                       ` Changli Gao
2009-07-03  7:15                         ` Amerigo Wang
2009-07-03  7:40                           ` Changli Gao
2009-07-03  8:16                             ` Amerigo Wang
2009-07-03  8:27                               ` Changli Gao
2009-07-03  9:42                                 ` Amerigo Wang
2009-07-03  9:59                                   ` Changli Gao
2009-07-03 10:23                                     ` Amerigo Wang
2009-07-03 10:28                                     ` Albert ARIBAUD
2009-07-06  1:12                                       ` Changli Gao
2009-07-06  6:15                                         ` Albert ARIBAUD
2009-07-06  6:23                                           ` Changli Gao
2009-07-06  6:30                                             ` Albert ARIBAUD
2009-07-07  4:43                                       ` Denys Vlasenko
2009-07-07  7:45                                         ` Changli Gao
2009-07-07 20:37                                           ` David Newall
2009-07-07 21:02                                             ` Zan Lynx
2009-07-08  4:41                                               ` Changli Gao
2009-07-08  4:43                                                 ` David Newall
2009-07-02 10:42     ` Alan Cox
2009-07-03  0:45       ` Changli Gao

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=412e6f7f0907022108p7c533ed2wd16fceb0f282ed62@mail.gmail.com \
    --to=xiaosuo@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=torvalds@linux-foundation.org \
    --cc=xiyou.wangcong@gmail.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®