linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Benjamin LaHaise <bcrl@redhat.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: Rik van Riel <riel@conectiva.com.br>, linux-kernel@vger.kernel.org
Subject: Re: mempool design
Date: Sat, 15 Dec 2001 13:47:11 -0500	[thread overview]
Message-ID: <20011215134711.A30548@redhat.com> (raw)
In-Reply-To: <Pine.LNX.4.33.0112152020030.25153-100000@localhost.localdomain>
In-Reply-To: <Pine.LNX.4.33.0112152020030.25153-100000@localhost.localdomain>; from mingo@elte.hu on Sat, Dec 15, 2001 at 08:40:19PM +0100

On Sat, Dec 15, 2001 at 08:40:19PM +0100, Ingo Molnar wrote:
> With all respect, even if i had read it before, i'd have done mempool.c
> the same way as it is now. (but i'd obviously have Cc:-ed Ben on it during
> its development.) I'd like to sum up Ben's patch (Ben please correct me if
> i misrepresent your patch in any way):

You're making the assumption that an incomplete patch is useless and 
has no design pricipals behind it.  What I disagree with is the design 
of mempool, not the implementation.  The design for reservations is to 
use enforced accounting limits to achive the effect of seperate memory 
pools.  Mempool's design is to build seperate pools on top of existing 
pools of memory.  Can't you see the obvious duplication that implies?

The first implementation of the reservation patch is full of bogosities, 
I'm the first one to admit that.  But am I going to go off and write an 
entirely new patch that fixes everything and gets the design right to 
replace mempool?  Not with the current rate of patches being ignored.

		-ben

  reply	other threads:[~2001-12-15 18:47 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-12-15 19:40 mempool design Ingo Molnar
2001-12-15 18:47 ` Benjamin LaHaise [this message]
2001-12-15 22:18   ` Ingo Molnar
2001-12-17 15:04     ` Andrea Arcangeli
2001-12-17 15:38       ` Victor Yodaiken
2001-12-17 16:10         ` Andrea Arcangeli
2001-12-17 17:33         ` kernel panic Geoffrey
2001-12-18 16:55         ` mempool design Ingo Molnar
2001-12-18 16:06           ` Victor Yodaiken
2001-12-17 17:21       ` Ingo Molnar
2001-12-17 15:58         ` Andrea Arcangeli
2001-12-18  0:32           ` Rik van Riel
2001-12-18 15:21       ` Alan Cox
  -- strict thread matches above, loose matches on Subject: below --
2001-12-15  9:01 Ingo Molnar
2001-12-15 15:39 ` Rik van Riel

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=20011215134711.A30548@redhat.com \
    --to=bcrl@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=riel@conectiva.com.br \
    /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;
as well as URLs for NNTP newsgroup(s).