From: Ralf Baechle <ralf@linux-mips.org>
To: Keith M Wesolowski <wesolows@foobazco.org>
Cc: Ladislav Michl <ladis@linux-mips.org>, linux-mips@linux-mips.org
Subject: Re: [PATCH] Highmem detection for Indigo2
Date: Fri, 9 May 2003 21:08:23 +0200 [thread overview]
Message-ID: <20030509190823.GA29398@linux-mips.org> (raw)
In-Reply-To: <20030508061117.GA30191@foobazco.org>
On Wed, May 07, 2003 at 11:11:17PM -0700, Keith M Wesolowski wrote:
> > Following patch builds whole RAM map based of MC's memory configuration
> > registers, does some samity checks adds high system memory (if any) to
> > bootmem.
>
> > +static void init_bootmem(void)
> ...
> > + init_bootmem();
>
> This is a pretty unfortunate choice of names for this function. See
> mm/bootmem.c.
>
> Other than that, your patch works fine for me; my Indy has 192MB
> memory and it's detected properly. I do get an oops in do_be from
> xdm, but I get that without the patch also.
>
> Determined physical RAM map:
> memory: 00001000 @ 00000000 (reserved)
> memory: 00001000 @ 00001000 (reserved)
> memory: 001e1000 @ 08002000 (reserved)
> memory: 0055d000 @ 081e3000 (usable)
> memory: 000c0000 @ 08740000 (ROM data)
> memory: 0b800000 @ 08800000 (usable)
>
> I need to do the same kind of thing for ip32 as the ARC memory
> detection has the same shortcoming on that platform. No sense having
> a machine support 1GB memory and only looking for 256MB of it,
> especially in a 64-bit kernel. ARC[S] really does seem to be useless.
That's what I'm saying since '94. ARC was a commitee approach of the
ACE consortium which soon died. The firmware part of the ARC standard
was also published as the Microsoft Portable Bootloader Standard but
Today every ARC implementation has some major deviations from the
standard rendering the term standard into nothing but a cynic demonation ...
Ralf
prev parent reply other threads:[~2003-05-09 19:08 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-04-28 7:16 [PATCH] Highmem detection for Indigo2 Ladislav Michl
2003-05-08 6:11 ` Keith M Wesolowski
2003-05-08 7:32 ` Ladislav Michl
2003-05-08 8:50 ` Guido Guenther
2003-05-08 8:58 ` Guido Guenther
2003-05-08 16:40 ` xdm oopses Keith M Wesolowski
2003-05-08 17:00 ` Guido Guenther
2003-05-09 19:08 ` Ralf Baechle [this message]
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=20030509190823.GA29398@linux-mips.org \
--to=ralf@linux-mips.org \
--cc=ladis@linux-mips.org \
--cc=linux-mips@linux-mips.org \
--cc=wesolows@foobazco.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox