public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Andi Kleen <ak@suse.de>
Cc: Andy Whitcroft <apw@shadowen.org>, Andrew Morton <akpm@osdl.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] x86 NUMA panic compile error
Date: Mon, 15 May 2006 20:14:13 +0200	[thread overview]
Message-ID: <20060515181413.GA18652@elte.hu> (raw)
In-Reply-To: <200605152001.16813.ak@suse.de>


* Andi Kleen <ak@suse.de> wrote:

> > i still strongly oppose the original Andi hack... numerous reasons were 
> > given not to apply it (it's nice to simulate/trigger rarer features on 
> > mainstream hardware too, and this ability to boot NUMA on my flat x86 
> > testbox found at least one other NUMA bug already). Furthermore, the 
> > crash i reported was fixed by the NUMA patchset. Andrew, please drop:
> 
> The problem is that it's not regularly used on a wide range of boxes 
> so it will eventually break again. We had this cycle several times 
> already.

so it will find new bugs. Since you wrote that patch the ability to try 
numa on 'flat' x86 hardware has found two bugs, one of which (the zone 
alignment thing) could very well affect non-obsolete hardware too.

Andi, triggering bugs as widely as possible is the _whole point_ of QA 
and diversity of testing! Do you disable the compilation of the floppy 
driver just because it's obsolete hardware? Will you remove NUMA 
emulation from x86_64 just because it regularly broke in the past?

Fact is, the more hardware a given feature can be tried on, the better 
tested that feature becomes. I find it quite extreme that this point has 
to even be argued...

	Ingo

  reply	other threads:[~2006-05-15 18:14 UTC|newest]

Thread overview: 94+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-05-15  7:56 2.6.17-rc4-mm1 Andrew Morton
2006-05-15 10:09 ` 2.6.17-rc4-mm1 Eric Dumazet
2006-05-15 11:03   ` 2.6.17-rc4-mm1 Andrew Morton
2006-05-15 11:42     ` 2.6.17-rc4-mm1 Pekka Enberg
2006-05-15 13:28     ` 2.6.17-rc4-mm1 Eric Dumazet
2006-05-15 13:12 ` 2.6.17-rc4-mm1 Reuben Farrelly
2006-05-15 22:09   ` 2.6.17-rc4-mm1 Neil Brown
2006-05-15 14:08 ` [PATCH] x86 NUMA panic compile error Andy Whitcroft
2006-05-15 17:53   ` Ingo Molnar
2006-05-15 18:01     ` Andi Kleen
2006-05-15 18:14       ` Ingo Molnar [this message]
2006-05-15 18:08     ` Andrew Morton
2006-05-15 18:08       ` Andy Whitcroft
2006-05-15 18:24         ` Andrew Morton
2006-05-15 18:24           ` Andi Kleen
2006-05-15 19:45           ` Martin Bligh
2006-05-15 18:13       ` Andi Kleen
2006-05-15 18:31         ` Ingo Molnar
2006-05-15 18:34         ` Andrew Morton
2006-05-15 18:37           ` Andi Kleen
2006-05-15 18:49             ` Ingo Molnar
2006-05-15 19:05             ` Dave Hansen
2006-05-15 19:11               ` Andi Kleen
2006-05-15 19:26                 ` Ingo Molnar
2006-05-15 19:38                   ` Andi Kleen
2006-05-16  7:06                     ` Ingo Molnar
2006-05-16  9:22                       ` Andi Kleen
2006-05-15 19:39                   ` Andrew Morton
2006-05-15 19:47                     ` Andi Kleen
2006-05-15 19:59                       ` Andrew Morton
2006-05-15 20:02                         ` Andi Kleen
2006-05-15 23:06                 ` H. Peter Anvin
2006-05-17  0:39                   ` Dave Jones
2006-05-17  1:21                     ` H. Peter Anvin
2006-05-15 18:43           ` Ingo Molnar
2006-05-15 18:28       ` Ingo Molnar
2006-05-15 18:52         ` Andrew Morton
2006-05-15 18:56           ` Andi Kleen
2006-05-15 18:59             ` Ingo Molnar
2006-05-15 18:56           ` Ingo Molnar
2006-05-15 19:06             ` Ingo Molnar
2006-05-15 16:40 ` 2.6.17-rc4-mm1 Michal Piotrowski
2006-05-15 17:04   ` 2.6.17-rc4-mm1 Andrew Morton
2006-05-15 17:30     ` 2.6.17-rc4-mm1 Takashi Iwai
2006-05-15 17:56       ` 2.6.17-rc4-mm1 Takashi Iwai
2006-05-15 18:11         ` 2.6.17-rc4-mm1 Andrew Morton
2006-05-16  9:04           ` 2.6.17-rc4-mm1 Takashi Iwai
2006-05-15 18:50         ` 2.6.17-rc4-mm1 Michal Piotrowski
2006-05-15 16:49 ` 2.6.17-rc4-mm1 Alexey Dobriyan
2006-05-15 17:01   ` 2.6.17-rc4-mm1 Andrew Morton
2006-05-15 18:01     ` 2.6.17-rc4-mm1 Alexey Dobriyan
2006-05-15 19:29     ` 2.6.17-rc4-mm1 Michael Halcrow
2006-05-15 17:48 ` 2.6.17-rc4-mm1 Michal Piotrowski
2006-05-15 18:00   ` 2.6.17-rc4-mm1 Andrew Morton
2006-05-15 18:05 ` 2.6.17-rc4-mm1 Jesper Juhl
2006-05-15 18:37 ` 2.6.17-rc4-mm1 Michal Piotrowski
2006-05-15 18:53   ` 2.6.17-rc4-mm1 Andrew Morton
2006-05-15 19:10     ` 2.6.17-rc4-mm1 Michal Piotrowski
2006-05-15 19:26       ` 2.6.17-rc4-mm1 Andrew Morton
2006-05-15 20:17         ` 2.6.17-rc4-mm1 Michal Piotrowski
2006-05-16  8:39           ` 2.6.17-rc4-mm1 Jean Delvare
2006-05-16 12:55             ` 2.6.17-rc4-mm1 Jean Delvare
2006-05-16 14:48               ` 2.6.17-rc4-mm1 Jean Delvare
2006-05-16 15:18                 ` 2.6.17-rc4-mm1 Stephen Hemminger
2006-05-16 15:23                 ` 2.6.17-rc4-mm1 Jean Delvare
2006-05-16 16:08                   ` 2.6.17-rc4-mm1 Greg KH
2006-05-16 16:27                     ` 2.6.17-rc4-mm1 Stephen Hemminger
2006-05-16 16:49                       ` 2.6.17-rc4-mm1 Greg KH
2006-05-15 19:28 ` 2.6.17-rc4-mm1 Ingo Molnar
     [not found] ` <6bffcb0e0605151003x5d3518b9o70dae3b3349c4f9f@mail.gmail.com>
2006-05-15 22:24   ` 2.6.17-rc4-mm1 David Woodhouse
2006-05-16 10:25 ` 2.6.17-rc4-mm1: no help text for MTD_NAND_CS553 Adrian Bunk
2006-05-16 12:13   ` David Woodhouse
2006-05-16 11:46 ` [-mm patch] drivers/mtd/devices/docprobe.c: correct #if's Adrian Bunk
2006-05-16 12:14   ` David Woodhouse
2006-05-16 11:48 ` [-mm patch] make dvb/b2c2/flexcop-fe-tuner.c:alps_tdee4_stv0297_tuner_set_params() static Adrian Bunk
2006-05-16 12:15 ` [-mm patch] drivers/media/video/pwc/: make code static Adrian Bunk
2006-05-16 12:32 ` [-mm patch] make variables static after klibc merge Adrian Bunk
2006-05-16 12:37 ` [-mm patch] make drivers/mtd/nand/cs553x_nand.c:cs553x_init() static Adrian Bunk
2006-05-16 13:04   ` David Woodhouse
2006-05-16 12:39 ` [-mm patch] fs/nfs/: make code static Adrian Bunk
2006-05-16 13:24 ` [-mm patch] arch/i386/oprofile/: make functions static Adrian Bunk
2006-05-16 18:16   ` Andi Kleen
2006-05-16 19:07   ` Andi Kleen
2006-05-16 15:26 ` [-mm patch] fs/ocfs2/dlm/: cleanups Adrian Bunk
2006-05-17  1:36   ` Mark Fasheh
2006-05-16 15:30 ` [-mm patch] drivers/net/s2io.c: make bus_speed[] static Adrian Bunk
2006-05-16 15:36   ` Andreas Mohr
2006-05-16 16:42     ` Adrian Bunk
2006-05-24  5:28   ` Jeff Garzik
2006-05-16 16:12 ` 2.6.17-rc4-mm1: please drop add-raw-driver-kconfig-entry-for-s390.patch Adrian Bunk
2006-05-16 16:37   ` Olaf Hering
2006-05-16 17:44     ` [2.6 patch] the overdue removal of the obsolete raw driver Adrian Bunk
2006-05-16 17:50       ` Olaf Hering
2006-05-17 13:12   ` 2.6.17-rc4-mm1: please drop add-raw-driver-kconfig-entry-for-s390.patch Alan Cox

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=20060515181413.GA18652@elte.hu \
    --to=mingo@elte.hu \
    --cc=ak@suse.de \
    --cc=akpm@osdl.org \
    --cc=apw@shadowen.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox