mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Fausto Richetti Blanco" <fausto.blanco@gmail.com>
To: linux-kernel@vger.kernel.org
Cc: miquels@cistron.nl, riel@redhat.com, jengelh@medozas.de,
	fausto.blanco@gmail.com
Subject: Re: Pipe buffers' limit of 16 * 4K
Date: Thu, 29 May 2008 12:46:56 -0300	[thread overview]
Message-ID: <d8d698ae0805290846w7cd2b14j99db2e1e2a9ec7ed@mail.gmail.com> (raw)
In-Reply-To: <alpine.LNX.1.10.0805291633000.19264@fbirervta.pbzchgretzou.qr>

On Thu, May 29, 2008 at 11:36 AM, Jan Engelhardt <jengelh@medozas.de> wrote:
>
> On Thursday 2008-05-29 15:19, Miquel van Smoorenburg wrote:
>>On Thu, 2008-05-29 at 10:00 -0300, Fausto Richetti Blanco wrote:
>>
>>Why not use a socketpair() instead of a pipe(). You can adjust the size
>>with setsockopt SO_SNDBUF/SO_RCVBUF (see man socket(7))
>
> Nah, if there's lots of POST requests, and a large buffer for
> each of it, you may end up running into allocation failures.

Well, I think it's an alternative.. A good one, indeed :)

However, I implemented it and run into the limit of /proc/sys/net/core/wmem_max

Do you guys think it's a big impact to change this to a higher value ?
It's meant to only affect the MAX window size, right ? Does it have
any other way of changing this limit (by process or by user, for
exemple) ?

The strange thing here is that setsockopt doesn't fail if I change the
size of the buffer to anything higher than
/proc/sys/net/core/wmem_max. It doesn't work either :)

The implementation with socketpair, adjusting
/proc/sys/net/core/wmem_max, seems good to me. However, I still think
dynamic buffers for pipes a good idea.

  reply	other threads:[~2008-05-29 15:47 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <d8d698ae0805231255i4d0da724p68cb2b9376134797@mail.gmail.com>
2008-05-24  0:19 ` Fausto Richetti Blanco
2008-05-24  0:56   ` Rik van Riel
     [not found]     ` <d8d698ae0805261043u125b653ve73d067cd92caa16@mail.gmail.com>
2008-05-28 21:22       ` Fausto Richetti Blanco
2008-05-28 22:05         ` Jan Engelhardt
2008-05-29 13:00           ` Fausto Richetti Blanco
2008-05-29 13:19             ` Miquel van Smoorenburg
2008-05-29 14:36               ` Jan Engelhardt
2008-05-29 15:46                 ` Fausto Richetti Blanco [this message]
2008-05-30 10:35                   ` Jens Axboe
2008-06-05  9:55                     ` Jan Engelhardt
2008-06-06  9:19                       ` Jens Axboe
2008-05-26 14:15   ` Jan Engelhardt

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=d8d698ae0805290846w7cd2b14j99db2e1e2a9ec7ed@mail.gmail.com \
    --to=fausto.blanco@gmail.com \
    --cc=jengelh@medozas.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=miquels@cistron.nl \
    --cc=riel@redhat.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®