From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from gate.crashing.org (gate.crashing.org [63.228.1.57]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 40hbmp0JZpzF2C8 for ; Fri, 11 May 2018 00:52:41 +1000 (AEST) Date: Thu, 10 May 2018 09:52:35 -0500 From: Segher Boessenkool To: Nicholas Piggin Cc: linux-kbuild@vger.kernel.org, Masahiro Yamada , linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH v2 1/5] powerpc/kbuild: set default generic machine type for 32-bit compile Message-ID: <20180510145235.GP17342@gate.crashing.org> References: <20180510051659.15356-1-npiggin@gmail.com> <20180510051659.15356-2-npiggin@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20180510051659.15356-2-npiggin@gmail.com> List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, May 10, 2018 at 03:16:55PM +1000, Nicholas Piggin wrote: > The 64-bit toolchain uses the wrong ISA variant for compiling 32-bit > kernels, eve with -m32. Set -mcpu=powerpc which is the generic 32-bit > powerpc machine type and scheduling model. 32-bit platforms and CPUs > can override this with -mcpu= options that come later on the command > line. So it turns out your compiler was configured with --with-cpu=power8; it's not that GCC defaults are bad, but who built your compiler overrode the defaults. If the problem only happens in .S files, it might be better to put .machine directives in those. But your patch should work too, yes. Segher