From: Jack Steiner <steiner@sgi.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: tglx@linutronix.de, linux-kernel@vger.kernel.org,
Mike Travis <travis@sgi.com>,
the arch/x86 maintainers <x86@kernel.org>
Subject: Re: [PATCH] - Increase MAX_APICS for very large x86_64 configs
Date: Thu, 19 Jun 2008 10:02:09 -0500 [thread overview]
Message-ID: <20080619150209.GA9295@sgi.com> (raw)
In-Reply-To: <20080619103242.GA13213@elte.hu>
On Thu, Jun 19, 2008 at 12:32:42PM +0200, Ingo Molnar wrote:
>
> * Ingo Molnar <mingo@elte.hu> wrote:
>
> > > The patch has no effect on any 32-bit kernel. It adds ~4k to the
> > > size of 64-bit kernels but only if NR_CPUS > 255.
> > >
> > > This is a repost of an earlier patch (4/16) that was implicated in a
> > > random-qa failure. I have not been able to reproduce the failure
> > > using the same boot args & config. Best guess is that some other
> > > patch caused random data corruption & this patch was a victim.
> >
> > i have just re-tested the previously failing config on tip/master:
> >
> > http://redhat.com/~mingo/misc/config-Thu_Apr_17_10_17_14_CEST_2008.bad
> > http://people.redhat.com/mingo/tip.git/README
> >
> > and the good news is that it does not crash on bootup anymore. Perhaps
> > one of Mike's fixes/improvements did the trick?
>
> unfortunately it didnt last long, -tip randconfig auto-testing found
> that your patch causes a bootup crash:
>
I built a "git-checkout -b tip-latest tip/auto-latest" tree but get compile errors.
Is this the right tree??
Have you seen this before:
In file included from arch/x86/mm/srat_64.c:13:
include/linux/acpi.h:97:1: warning: "NR_NODE_MEMBLKS" redefined
In file included from include/acpi/platform/aclinux.h:59,
from include/acpi/platform/acenv.h:140,
from include/acpi/acpi.h:54,
from include/linux/acpi.h:39,
from arch/x86/mm/srat_64.c:13:
include/asm/acpi.h:165:1: warning: this is the location of the previous definition
arch/x86/mm/srat_64.c:101: error: expected identifier or '(' before 'do'
arch/x86/mm/srat_64.c:101: error: expected identifier or '(' before 'while'
arch/x86/mm/srat_64.c:108: error: expected identifier or '(' before 'do'
arch/x86/mm/srat_64.c:108: error: expected identifier or '(' before 'while'
arch/x86/mm/srat_64.c:211: error: expected identifier or '(' before 'do'
arch/x86/mm/srat_64.c:211: error: expected identifier or '(' before 'while'
...
--- jack
next prev parent reply other threads:[~2008-06-19 15:02 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-06-16 17:09 [PATCH] - Increase MAX_APICS for very large x86_64 configs Jack Steiner
2008-06-19 10:10 ` Ingo Molnar
2008-06-19 10:32 ` Ingo Molnar
2008-06-19 15:02 ` Jack Steiner [this message]
2008-06-19 15:17 ` Ingo Molnar
2008-06-19 15:19 ` Ingo Molnar
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=20080619150209.GA9295@sgi.com \
--to=steiner@sgi.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=tglx@linutronix.de \
--cc=travis@sgi.com \
--cc=x86@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox