From: "Gérard Roudier" <groudier@free.fr>
To: "David S. Miller" <davem@redhat.com>
Cc: <anton@samba.org>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] small sym-2 fix
Date: Thu, 15 Nov 2001 19:07:14 +0100 (CET) [thread overview]
Message-ID: <20011115183811.F1902-100000@gerard> (raw)
In-Reply-To: <20011115.112606.62677098.davem@redhat.com>
On Thu, 15 Nov 2001, David S. Miller wrote:
> From: Gérard Roudier <groudier@free.fr>
> Date: Thu, 15 Nov 2001 17:27:38 +0100 (CET)
>
> The driver should not need more than 4096 bytes for a single allocation.
>
> If platform is 64-bit and PAGE_SIZE < 8K, yes it will.
> And ppc64 fits this criteria.
Btw, I didn't see any ppc64 stuff in linux kernel. As a result this
platform is still in the unsupported status. :-)
Indeed a 4K page on a 64 bit machine looks very suboptimal. Note that it
is also way too small for 32 bits machines with hundreds of megabytes of
memory. I am wondering about the reasons that made us keep with so small a
page size. As you know earlier BSDs used 2K on Vaxen that as you also know
had (have?) a physical PAGE_SIZE of 512 bytes.
About the sym-2 driver, it may well be some pointers that make it need
more than 4K allocation on 64 bit machines.
I will try to make it fit a page size max allocation even on such weird
configuration, since I do consider as high^H^H^H^Hslightly broken any
piece of software that requires more that 1 PAGE of physically contiguous
allocation. :-) :-) :-)
To be serious, the right fix is to have some logical page be some power of
two of the physical page when the physical page is too small. Can we hope
Linux-2.5 to allow this?
Gérard.
next prev parent reply other threads:[~2001-11-15 20:52 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-11-15 4:36 [PATCH] small sym-2 fix Anton Blanchard
2001-11-15 10:19 ` David S. Miller
2001-11-15 11:35 ` Anton Blanchard
2001-11-15 11:41 ` David S. Miller
2001-11-15 12:17 ` Anton Blanchard
2001-11-15 16:27 ` Gérard Roudier
2001-11-15 19:26 ` David S. Miller
2001-11-15 18:07 ` Gérard Roudier [this message]
2001-11-15 21:22 ` Hugh Dickins
2001-11-15 20:22 ` Gérard Roudier
2001-11-15 19:46 ` Gérard Roudier
2001-11-15 22:39 ` David S. Miller
2001-11-15 20:41 ` Gérard Roudier
2001-11-15 22:44 ` David S. Miller
2001-11-20 6:02 ` Anton Blanchard
2001-11-20 17:26 ` Gérard Roudier
2001-11-21 2:19 ` Anton Blanchard
2001-11-21 3:49 ` David S. Miller
2001-11-21 17:19 ` Gérard Roudier
2001-11-22 5:19 ` Paul Mackerras
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=20011115183811.F1902-100000@gerard \
--to=groudier@free.fr \
--cc=anton@samba.org \
--cc=davem@redhat.com \
--cc=linux-kernel@vger.kernel.org \
/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.