From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757271Ab0E3AnO (ORCPT ); Sat, 29 May 2010 20:43:14 -0400 Received: from chilli.pcug.org.au ([203.10.76.44]:33182 "EHLO smtps.tip.net.au" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756906Ab0E3AnM (ORCPT ); Sat, 29 May 2010 20:43:12 -0400 Date: Sun, 30 May 2010 10:43:03 +1000 From: Stephen Rothwell To: Linus Torvalds Cc: Andrew Morton , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Jack Steiner , Lee Schermerhorn Subject: Re: linux-next: build failure in Linus' tree Message-Id: <20100530104303.15864d35.sfr@canb.auug.org.au> In-Reply-To: References: <20100528105719.a356ba4f.sfr@canb.auug.org.au> X-Mailer: Sylpheed 3.0.2 (GTK+ 2.20.1; i486-pc-linux-gnu) Mime-Version: 1.0 Content-Type: multipart/signed; protocol="application/pgp-signature"; micalg="PGP-SHA1"; boundary="Signature=_Sun__30_May_2010_10_43_03_+1000__MvVWbMZ85qqb=j_" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --Signature=_Sun__30_May_2010_10_43_03_+1000__MvVWbMZ85qqb=j_ Content-Type: text/plain; charset=US-ASCII Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Linus, On Thu, 27 May 2010 18:09:17 -0700 (PDT) Linus Torvalds wrote: > > On Fri, 28 May 2010, Stephen Rothwell wrote: > >=20 > > Caused by commit 0ac0c0d0f837c499afd02a802f9cf52d3027fa3b ("cpusets: > > randomize node rotor used in cpuset_mem_spread_node()"). > >=20 > > This commit assumes that __node_random() exists if (MAX_NUMNODES > 1) a= nd > > uses it if CONFIG_CPUSETS is set, but only creates it for x86 ... there > > is at least one other architecture where those conditions are true. >=20 > Yeah, looking at that, it seems totally idiotic. >=20 > Why is that "__node_random()" in x86 code at all? There is absolutely=20 > nothing x86 about it that I can tell. And now I have an ia64 merge that=20 > just duplicates that moronic function. So, is it reasonable for me to ask you to revert commit 0ac0c0d0f837c499afd02a802f9cf52d3027fa3b ("cpusets: randomize node rotor used in cpuset_mem_spread_node()")? Reverting it won't break ia64 (since their fix was to just add code that would then be unreferenced). --=20 Cheers, Stephen Rothwell sfr@canb.auug.org.au http://www.canb.auug.org.au/~sfr/ --Signature=_Sun__30_May_2010_10_43_03_+1000__MvVWbMZ85qqb=j_ Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iEYEARECAAYFAkwBtJcACgkQjjKRsyhoI8wN1QCfXy6phQG05R6wsflQPUV37afj +BkAoIW8l469CvRK4RksvXk8JXYQS/Zl =Px2r -----END PGP SIGNATURE----- --Signature=_Sun__30_May_2010_10_43_03_+1000__MvVWbMZ85qqb=j_--