All of lore.kernel.org
 help / color / mirror / Atom feed
From: Pavel Machek <pavel@suse.cz>
To: Sridhar Samudrala <sri@us.ibm.com>
Cc: Matthew Dobson <colpatch@us.ibm.com>, Paul Jackson <pj@sgi.com>,
	clameter@engr.sgi.com, linux-kernel@vger.kernel.org,
	andrea@suse.de, linux-mm@kvack.org
Subject: Let the flames begin... [was Re: [patch 3/9] mempool - Make mempools NUMA aware]
Date: Sun, 29 Jan 2006 00:10:58 +0100	[thread overview]
Message-ID: <20060128231058.GA1718@elf.ucw.cz> (raw)
In-Reply-To: <20060128225907.GA1612@elf.ucw.cz>

On So 28-01-06 23:59:07, Pavel Machek wrote:
> Hi!
> 
> > >If sending routines can work with constant ammount of memory, why use
> > >kmalloc at all? Anyway I thought we were talking receiving side
> > >earlier in the thread.
> > >
> > >Ouch and wait a moment. You claim that GFP_KERNEL allocations can't
> > >block/sleep? Of course they can, that's why they are GFP_KERNEL and
> > >not GFP_ATOMIC.
> > >  
> > I didn't meant GFP_KERNEL allocations cannot block/sleep? When in 
> > emergency, we
> > want even the GFP_KERNEL allocations that are made by critical sockets 
> > not to block/sleep.
> > So my original critical sockets patches changes the gfp flag passed to 
> > these allocation requests
> > to GFP_KERNEL|GFP_CRITICAL.

(I'd say Al Viro mode, but Al could take that personally)

IOW: You keep pushing complex and known-broken solution for problem
that does not exist.

(Now you should be angry enough, and please explain why I'm wrong in
easy to understand terms, so that even I will understand that we need
critical sockets for kernels in emergency).
								Pavel
-- 
Thanks, Sharp!

WARNING: multiple messages have this Message-ID (diff)
From: Pavel Machek <pavel@suse.cz>
To: Sridhar Samudrala <sri@us.ibm.com>
Cc: Matthew Dobson <colpatch@us.ibm.com>, Paul Jackson <pj@sgi.com>,
	clameter@engr.sgi.com, linux-kernel@vger.kernel.org,
	andrea@suse.de, linux-mm@kvack.org
Subject: Let the flames begin... [was Re: [patch 3/9] mempool - Make mempools NUMA aware]
Date: Sun, 29 Jan 2006 00:10:58 +0100	[thread overview]
Message-ID: <20060128231058.GA1718@elf.ucw.cz> (raw)
In-Reply-To: <20060128225907.GA1612@elf.ucw.cz>

On So 28-01-06 23:59:07, Pavel Machek wrote:
> Hi!
> 
> > >If sending routines can work with constant ammount of memory, why use
> > >kmalloc at all? Anyway I thought we were talking receiving side
> > >earlier in the thread.
> > >
> > >Ouch and wait a moment. You claim that GFP_KERNEL allocations can't
> > >block/sleep? Of course they can, that's why they are GFP_KERNEL and
> > >not GFP_ATOMIC.
> > >  
> > I didn't meant GFP_KERNEL allocations cannot block/sleep? When in 
> > emergency, we
> > want even the GFP_KERNEL allocations that are made by critical sockets 
> > not to block/sleep.
> > So my original critical sockets patches changes the gfp flag passed to 
> > these allocation requests
> > to GFP_KERNEL|GFP_CRITICAL.

(I'd say Al Viro mode, but Al could take that personally)

IOW: You keep pushing complex and known-broken solution for problem
that does not exist.

(Now you should be angry enough, and please explain why I'm wrong in
easy to understand terms, so that even I will understand that we need
critical sockets for kernels in emergency).
								Pavel
-- 
Thanks, Sharp!

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

  reply	other threads:[~2006-01-28 23:11 UTC|newest]

Thread overview: 86+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20060125161321.647368000@localhost.localdomain>
2006-01-25 19:39 ` [patch 1/9] mempool - Add page allocator Matthew Dobson
2006-01-25 19:39 ` [patch 2/9] mempool - Use common mempool " Matthew Dobson
2006-01-25 19:39   ` Matthew Dobson
2006-01-25 19:40 ` [patch 4/9] mempool - Update mempool page allocator user Matthew Dobson
2006-01-25 19:40 ` [patch 5/9] mempool - Update kmalloc mempool users Matthew Dobson
2006-01-25 19:40   ` Matthew Dobson
2006-01-25 19:40 ` [patch 6/9] mempool - Update kzalloc " Matthew Dobson
2006-01-25 19:40   ` Matthew Dobson
2006-01-26  7:30   ` Pekka Enberg
2006-01-26  7:30     ` Pekka Enberg
2006-01-26 22:03     ` Matthew Dobson
2006-01-26 22:03       ` Matthew Dobson
2006-01-25 19:40 ` [patch 8/9] slab - Add *_mempool slab variants Matthew Dobson
2006-01-25 19:40   ` Matthew Dobson
2006-01-26  7:41   ` Pekka Enberg
2006-01-26  7:41     ` Pekka Enberg
2006-01-26 22:40     ` Matthew Dobson
2006-01-26 22:40       ` Matthew Dobson
2006-01-27  7:09       ` Pekka J Enberg
2006-01-27  7:09         ` Pekka J Enberg
2006-01-27  7:10       ` Pekka J Enberg
2006-01-27  7:10         ` Pekka J Enberg
2006-01-25 19:40 ` [patch 9/9] slab - Implement single mempool backing for slab allocator Matthew Dobson
2006-01-25 19:40   ` Matthew Dobson
2006-01-26  8:11   ` Pekka Enberg
2006-01-26  8:11     ` Pekka Enberg
2006-01-26 22:48     ` Matthew Dobson
2006-01-26 22:48       ` Matthew Dobson
2006-01-27  7:22       ` Pekka J Enberg
2006-01-27  7:22         ` Pekka J Enberg
2006-01-25 23:51 ` [patch 1/9] mempool - Add page allocator Matthew Dobson
2006-01-25 23:51   ` Matthew Dobson
2006-01-25 23:51 ` [patch 3/9] mempool - Make mempools NUMA aware Matthew Dobson
2006-01-25 23:51   ` Matthew Dobson
2006-01-26 17:54   ` Christoph Lameter
2006-01-26 17:54     ` Christoph Lameter
2006-01-26 22:57     ` Matthew Dobson
2006-01-26 22:57       ` Matthew Dobson
2006-01-26 23:15       ` Christoph Lameter
2006-01-26 23:15         ` Christoph Lameter
2006-01-26 23:24         ` Matthew Dobson
2006-01-26 23:24           ` Matthew Dobson
2006-01-26 23:29           ` Christoph Lameter
2006-01-26 23:29             ` Christoph Lameter
2006-01-27  0:15             ` Matthew Dobson
2006-01-27  0:15               ` Matthew Dobson
2006-01-27  0:21               ` Christoph Lameter
2006-01-27  0:21                 ` Christoph Lameter
2006-01-27  0:34                 ` Matthew Dobson
2006-01-27  0:34                   ` Matthew Dobson
2006-01-27  0:39                   ` Christoph Lameter
2006-01-27  0:39                     ` Christoph Lameter
2006-01-27  0:44                     ` Matthew Dobson
2006-01-27  0:44                       ` Matthew Dobson
2006-01-27  0:57                       ` Christoph Lameter
2006-01-27  0:57                         ` Christoph Lameter
2006-01-27  1:07                         ` Andi Kleen
2006-01-27  1:07                           ` Andi Kleen
2006-01-27 10:51                   ` Paul Jackson
2006-01-27 10:51                     ` Paul Jackson
2006-01-28  1:00                     ` Matthew Dobson
2006-01-28  1:00                       ` Matthew Dobson
2006-01-28  5:08                       ` Paul Jackson
2006-01-28  5:08                         ` Paul Jackson
2006-01-28  8:16                       ` Pavel Machek
2006-01-28  8:16                         ` Pavel Machek
2006-01-28 16:14                         ` Sridhar Samudrala
2006-01-28 16:14                           ` Sridhar Samudrala
2006-01-28 16:41                           ` Pavel Machek
2006-01-28 16:41                             ` Pavel Machek
2006-01-28 16:53                             ` Sridhar Samudrala
2006-01-28 16:53                               ` Sridhar Samudrala
2006-01-28 22:59                               ` Pavel Machek
2006-01-28 22:59                                 ` Pavel Machek
2006-01-28 23:10                                 ` Pavel Machek [this message]
2006-01-28 23:10                                   ` Let the flames begin... [was Re: [patch 3/9] mempool - Make mempools NUMA aware] Pavel Machek
2006-01-27  0:23   ` [patch 3/9] mempool - Make mempools NUMA aware Benjamin LaHaise
2006-01-27  0:23     ` Benjamin LaHaise
2006-01-27  0:35     ` Matthew Dobson
2006-01-27  0:35       ` Matthew Dobson
2006-01-27  3:23       ` Benjamin LaHaise
2006-01-27  3:23         ` Benjamin LaHaise
2006-01-28  1:08         ` Matthew Dobson
2006-01-28  1:08           ` Matthew Dobson
2006-01-25 23:51 ` [patch 7/9] mempool - Update other mempool users Matthew Dobson
2006-01-25 23:51   ` Matthew Dobson

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=20060128231058.GA1718@elf.ucw.cz \
    --to=pavel@suse.cz \
    --cc=andrea@suse.de \
    --cc=clameter@engr.sgi.com \
    --cc=colpatch@us.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=pj@sgi.com \
    --cc=sri@us.ibm.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.