All of lore.kernel.org
 help / color / mirror / Atom feed
From: Matt Mackall <mpm@selenic.com>
To: Nick Warne <nick@linicks.net>
Cc: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 14/15] misc: Configurable number of supported IDE interfaces
Date: Fri, 11 Nov 2005 09:11:40 -0800	[thread overview]
Message-ID: <20051111171140.GV11462@waste.org> (raw)
In-Reply-To: <200511111533.13474.nick@linicks.net>

On Fri, Nov 11, 2005 at 03:33:13PM +0000, Nick Warne wrote:
> Bartlomiej Zolnierkiewicz wrote:
> 
> > You are duplicating functionality of CONFIG_IDE_MAX_HWIFS,
> > please find a way to use it for EMBEDDED.
> > 
> > Also please cc: linux-ide on IDE related patches.
> > 
> > On 11/11/05, Matt Mackall <mpm@selenic.com> wrote:
> >> Configurable number of supported IDE interfaces
> >>
> >> This overrides the default limit (which may be set per arch with
> >> CONFIG_IDE_MAX_HWIFS). This is the result of setting interfaces to 1:
> 
> This is very similar to my unaccepted patch a few months ago:
> 
> http://lkml.org/lkml/2005/6/25/69

And it's practically identical to the one that's been in my -tiny tree
for two years.

I agree with Alan's complaint that this should in fact be done with
dynamic allocation. However, when I last set out to do that, it
appeared to be an _extremely_ invasive change so I quickly abandoned
it.

Meanwhile, this is currently the largest static allocation in a
typical kernel. Something has to be done.

-- 
Mathematics is the supreme nostalgia of our time.

  reply	other threads:[~2005-11-11 17:12 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-11-11 15:33 [PATCH 14/15] misc: Configurable number of supported IDE interfaces Nick Warne
2005-11-11 17:11 ` Matt Mackall [this message]
  -- strict thread matches above, loose matches on Subject: below --
2005-11-11  8:35 [PATCH 13/15] misc: Configure ELF core dump support Matt Mackall
2005-11-11  8:35 ` [PATCH 14/15] misc: Configurable number of supported IDE interfaces Matt Mackall
2005-11-11 10:14   ` Bartlomiej Zolnierkiewicz
2005-11-11 17:18     ` Matt Mackall
2005-11-11 17:34       ` Roman Zippel
2005-11-11 17:37         ` Matt Mackall
2005-11-11 17:47           ` Matt Mackall
2005-11-11 17:49           ` Roman Zippel

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=20051111171140.GV11462@waste.org \
    --to=mpm@selenic.com \
    --cc=bzolnier@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nick@linicks.net \
    /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.