From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756964AbcBCPqj (ORCPT ); Wed, 3 Feb 2016 10:46:39 -0500 Received: from mailapp01.imgtec.com ([195.59.15.196]:36700 "EHLO mailapp01.imgtec.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755425AbcBCPqi (ORCPT ); Wed, 3 Feb 2016 10:46:38 -0500 Date: Wed, 3 Feb 2016 15:46:34 +0000 From: Paul Burton To: James Hogan CC: , Ralf Baechle , "Matt Redfearn" , , "Thomas Gleixner" , Markos Chandras Subject: Re: [PATCH 06/15] MIPS: CM: Fix mips_cm_max_vp_width for UP kernels Message-ID: <20160203154634.GC30470@NP-P-BURTON> References: <1454469335-14778-1-git-send-email-paul.burton@imgtec.com> <1454469335-14778-7-git-send-email-paul.burton@imgtec.com> <20160203145858.GH5464@jhogan-linux.le.imgtec.org> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline In-Reply-To: <20160203145858.GH5464@jhogan-linux.le.imgtec.org> User-Agent: Mutt/1.5.23 (2014-03-12) X-Originating-IP: [10.100.200.164] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 03, 2016 at 02:58:59PM +0000, James Hogan wrote: > On Wed, Feb 03, 2016 at 03:15:26AM +0000, Paul Burton wrote: > > Fix mips_cm_max_vp_width for UP kernels where it previously referenced > > smp_num_siblings, which is not declared for UP kernels. This led to > > build errors such as the following: > > > > drivers/built-in.o: In function `$L446': > > irq-mips-gic.c:(.text+0x1994): undefined reference to `smp_num_siblings' > > drivers/built-in.o:irq-mips-gic.c:(.text+0x199c): more undefined references to `smp_num_siblings' follow > > > > On UP kernels simply return 1, leaving the reference to smp_num_siblings > > in place only for SMP kernels. > > > > Signed-off-by: Paul Burton > > Need tagging for stable v4.3+? It happens that there were no uses of mips_cm_max_vp_width in UP kernels, but some are added later in this series (the next patch for instance). So I don't see a need to backport to stable branches. Sorry that could have been clearer. > I do wonder if this should be handled in the header files though... As in you don't think it should be handled in headers? It seems like the logical place to do it to me... Or do you mean smp_num_siblings should be defined as 1 for UP kernels? I did consider that approach, but thought this possibly more semantically correct since smp isn't in use at all so neither is smp_num_siblings. Thanks, Paul > Cheers > James > > > --- > > > > arch/mips/include/asm/mips-cm.h | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/arch/mips/include/asm/mips-cm.h b/arch/mips/include/asm/mips-cm.h > > index 1395bbc..3fdb6c9 100644 > > --- a/arch/mips/include/asm/mips-cm.h > > +++ b/arch/mips/include/asm/mips-cm.h > > @@ -462,7 +462,10 @@ static inline unsigned int mips_cm_max_vp_width(void) > > if (mips_cm_revision() >= CM_REV_CM3) > > return read_gcr_sys_config2() & CM_GCR_SYS_CONFIG2_MAXVPW_MSK; > > > > - return smp_num_siblings; > > + if (config_enabled(CONFIG_SMP)) > > + return smp_num_siblings; > > + > > + return 1; > > } > > > > /** > > -- > > 2.7.0 > > > >