linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Andi Kleen <ak@suse.de>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Andi Kleen <ak@suse.de>, Christoph Lameter <clameter@sgi.com>,
	Mel Gorman <mel@skynet.ie>,
	Lee.Schermerhorn@hp.com, linux-kernel@vger.kernel.org,
	linux-mm@kvack.org
Subject: Re: [PATCH 3/4] Embed zone_id information within the zonelist->zones pointer
Date: Tue, 14 Aug 2007 02:14:41 +0200	[thread overview]
Message-ID: <20070814001441.GN3406@bingen.suse.de> (raw)
In-Reply-To: <20070814002223.2d8d42c5@the-village.bc.nu>

On Tue, Aug 14, 2007 at 12:22:23AM +0100, Alan Cox wrote:
> > The only tricky part were skbs in a few drivers, but luckily they are only
> > needed for bouncing which can be done without a skb too. For RX it adds
> > one copy, but we can live with that because they're only slow devices.
> 
> Usually found on slow hardware that can't cope with extra copies very
> well.

It's essentially only lance, meth, b44 and bcm43xx and lots of s390.

meth is only used on SGI O2s which are not that slow and unlikely
to work in tree anyways.

b44 and bcm43xx run in fast enough new systems to have no trouble
with copies.

s390 won't change.

That only leaves lance. If it runs in a system with <= 16MB 
of memory is fine. I checked with David if he would consider
adding a second destructor to the skb for this case and he 
said no. Which was an answer which was fine for m.e

So the only systems really affected are lance systems with >16MB.
I don't think we can stop Linux evolution for those sorry. They'll
just have to live with it.

-Andi

--
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:[~2007-08-14  0:14 UTC|newest]

Thread overview: 51+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-09 21:06 [PATCH 0/4] Use one zonelist per node instead of multiple zonelists v3 Mel Gorman
2007-08-09 21:06 ` [PATCH 1/4] Use zonelists instead of zones when direct reclaiming pages Mel Gorman
2007-08-09 21:19   ` Christoph Lameter
2007-08-09 21:06 ` [PATCH 2/4] Use one zonelist that is filtered instead of multiple zonelists Mel Gorman
2007-08-09 21:27   ` Christoph Lameter
2007-08-09 21:07 ` [PATCH 3/4] Embed zone_id information within the zonelist->zones pointer Mel Gorman
2007-08-09 21:37   ` Christoph Lameter
2007-08-09 23:33     ` Mel Gorman
2007-08-10  1:44       ` Christoph Lameter
2007-08-10 10:47         ` Mel Gorman
2007-08-10 17:37           ` Christoph Lameter
2007-08-10 18:13             ` Andi Kleen
2007-08-10 19:02               ` Christoph Lameter
2007-08-11  1:04                 ` Andi Kleen
2007-08-13 21:25                   ` Christoph Lameter
2007-08-13 22:50                     ` Andi Kleen
2007-08-13 22:00                       ` Christoph Lameter
2007-08-13 22:58                         ` Andi Kleen
2007-08-13 22:09                           ` Christoph Lameter
2007-08-13 23:08                             ` Andi Kleen
2007-08-13 22:22                               ` Christoph Lameter
2007-08-13 23:42                                 ` Andi Kleen
2007-08-13 22:52                                   ` Christoph Lameter
2007-08-13 23:55                                     ` Andi Kleen
2007-08-13 23:12                                       ` Christoph Lameter
2007-08-14  0:16                                         ` Andi Kleen
2007-08-13 23:25                                           ` Christoph Lameter
2007-08-14  0:25                                             ` Andi Kleen
2007-08-13 22:38                               ` Christoph Lameter
2007-08-13 23:43                                 ` Andi Kleen
2007-08-13 22:54                                   ` Christoph Lameter
2007-08-14  0:00                                     ` Andi Kleen
2007-08-13 23:16                                       ` Christoph Lameter
2007-08-14  0:16                                         ` Andi Kleen
2007-08-13 23:27                                           ` Christoph Lameter
2007-08-14  0:26                                             ` Andi Kleen
2007-08-14 19:56                                               ` Christoph Lameter
2007-08-13 23:22                                       ` Alan Cox
2007-08-14  0:14                                         ` Andi Kleen [this message]
2007-08-13 23:44                                           ` Alan Cox
2007-08-14 19:11                                           ` Andy Isaacson
2007-08-14 20:23                                             ` Andi Kleen
2007-08-14 19:43                                               ` Andy Isaacson
2007-08-14 21:05                                                 ` Andi Kleen
2007-08-15 11:37                                           ` Ralf Baechle
2007-08-15 12:59                                             ` Andi Kleen
2007-08-15 12:32                                               ` Ralf Baechle
2007-08-15 19:59                                               ` Christoph Lameter
2007-08-09 21:07 ` [PATCH 4/4] Apply MPOL_BIND policy to two highest zones when highest is ZONE_MOVABLE Mel Gorman
2007-08-09 21:37   ` Christoph Lameter
2007-08-09 21:19 ` [PATCH 0/4] Use one zonelist per node instead of multiple zonelists v3 Christoph Lameter

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=20070814001441.GN3406@bingen.suse.de \
    --to=ak@suse.de \
    --cc=Lee.Schermerhorn@hp.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=clameter@sgi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mel@skynet.ie \
    /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