From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ingo Molnar Subject: Re: linux-next: tip-core build failure Date: Mon, 15 Sep 2008 10:11:29 +0200 Message-ID: <20080915081129.GD29585@elte.hu> References: <20080913111423.981554c3.sfr@canb.auug.org.au> <200809131144.36507.rui.p.m.sousa@gmail.com> <20080914124320.GG16097@elte.hu> <20080915040100.5a6f634f.sfr@canb.auug.org.au> <20080914183629.GA28010@elte.hu> <20080915050234.3519203a.sfr@canb.auug.org.au> <20080914191709.GA3246@elte.hu> <20080915054508.1a7f4a24.sfr@canb.auug.org.au> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mx2.mail.elte.hu ([157.181.151.9]:50883 "EHLO mx2.mail.elte.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750754AbYIOILl (ORCPT ); Mon, 15 Sep 2008 04:11:41 -0400 Content-Disposition: inline In-Reply-To: <20080915054508.1a7f4a24.sfr@canb.auug.org.au> Sender: linux-next-owner@vger.kernel.org List-ID: To: Stephen Rothwell Cc: Rui Sousa , linux-next@vger.kernel.org * Stephen Rothwell wrote: > Hi Ingo, > > On Sun, 14 Sep 2008 21:17:09 +0200 Ingo Molnar wrote: > > > > * Stephen Rothwell wrote: > > > > > On Sun, 14 Sep 2008 20:36:29 +0200 Ingo Molnar wrote: > > > > > > > > [ ... just used by 90%+ of our active testers/developers ;-) ... ] > > > > > > An irrelevant argument in this case. > > > > why is the actual usage distribution of Linux irrelevant? We make > > Notice the "in this case"? In this case the code being changed > clearly could affect other architectures, so checking that x86 is OK > is clearly not enough (and that does not mean building it for 20+ > other architectures, just using grep to see the consequences of the > change and maybe discussing it with the affected architecture > maintainers or on linux-arch). again, the side effects were not realized. Pretty much _any_ patch that is not strictly restricted to a single architecture 'could' affect other architectures. Any change to a common .h or .c file could do that - and 90% of Linux's source code is in common files. Hence your suggestion that we should have found this breakage makes no sense in practice as it's not reliably testable. Yes, for things like sparseirq support we pretty much expected cross-arch fallout so we checked a ton of architectures. For other patches bugs can slip through, and that's OK as we really dont want to dog down pretty much every patch with the requirement to cross-build. Ingo