From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Fri, 08 May 2015 16:51:31 +0200 (CEST) Received: from mailapp01.imgtec.com ([195.59.15.196]:59735 "EHLO imgpgp01.kl.imgtec.org" rhost-flags-OK-OK-OK-FAIL) by eddie.linux-mips.org with ESMTP id S27026566AbbEHOv3e-t2w (ORCPT ); Fri, 8 May 2015 16:51:29 +0200 Received: from imgpgp01.kl.imgtec.org (imgpgp01.kl.imgtec.org [127.0.0.1]) by imgpgp01.kl.imgtec.org (PGP Universal) with ESMTP id 6447441F8E1F; Fri, 8 May 2015 15:51:26 +0100 (BST) Received: from mailapp01.imgtec.com ([10.100.180.242]) by imgpgp01.kl.imgtec.org (PGP Universal service); Fri, 08 May 2015 15:51:26 +0100 X-PGP-Universal: processed; by imgpgp01.kl.imgtec.org on Fri, 08 May 2015 15:51:26 +0100 Received: from KLMAIL01.kl.imgtec.org (unknown [192.168.5.35]) by Websense Email Security Gateway with ESMTPS id 91CA2BA4FD130; Fri, 8 May 2015 15:51:23 +0100 (IST) Received: from LEMAIL01.le.imgtec.org (192.168.152.62) by KLMAIL01.kl.imgtec.org (192.168.5.35) with Microsoft SMTP Server (TLS) id 14.3.195.1; Fri, 8 May 2015 15:51:26 +0100 Received: from [192.168.154.110] (192.168.154.110) by LEMAIL01.le.imgtec.org (192.168.152.62) with Microsoft SMTP Server (TLS) id 14.3.210.2; Fri, 8 May 2015 15:51:25 +0100 Message-ID: <554CCD6D.9010004@imgtec.com> Date: Fri, 8 May 2015 15:51:25 +0100 From: James Hogan User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Nicholas Krause , CC: , , , , Subject: Re: [PATCH] mips:Remove unneeded duplicate declaration of cpu_callin_map in smp.h References: <1431094355-28145-1-git-send-email-xerofoify@gmail.com> In-Reply-To: <1431094355-28145-1-git-send-email-xerofoify@gmail.com> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="9iVb4aFBWr9IQMm9c8pC720DtJVvCU8NI" X-Originating-IP: [192.168.154.110] X-ESG-ENCRYPT-TAG: b93fcccb Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 47284 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: james.hogan@imgtec.com Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips --9iVb4aFBWr9IQMm9c8pC720DtJVvCU8NI Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 08/05/15 15:12, Nicholas Krause wrote: > This removes the unneeded duplicate declaration of cpu_callin_map > in smp.h due to use already declaring it in the file,smp.c that No, it isn't declared in arch/mips/kernel/smp.c, its *defined* there (no "extern"). It's referenced by: arch/mips/cavium-octeon/smp.c arch/mips/kernel/process.c arch/mips/kernel/smp-bmips.c arch/mips/kernel/smp-cps.c arch/mips/loongson/loongson-3/smp.c as well as arch/mips/kernel/smp.c, which is why the declaration is needed in a header. If you're attempting to fix the build errors in this area, please see: http://patchwork.linux-mips.org/patch/9970/ Cheers James > already uses it internally for functions required this variable > for their various internal work. >=20 > Signed-off-by: Nicholas Krause > --- > arch/mips/include/asm/smp.h | 2 -- > 1 file changed, 2 deletions(-) >=20 > diff --git a/arch/mips/include/asm/smp.h b/arch/mips/include/asm/smp.h > index bb02fac..7752011 100644 > --- a/arch/mips/include/asm/smp.h > +++ b/arch/mips/include/asm/smp.h > @@ -45,8 +45,6 @@ extern int __cpu_logical_map[NR_CPUS]; > #define SMP_DUMP 0x8 > #define SMP_ASK_C0COUNT 0x10 > =20 > -extern volatile cpumask_t cpu_callin_map; > - > /* Mask of CPUs which are currently definitely operating coherently */= > extern cpumask_t cpu_coherent_mask; > =20 >=20 --9iVb4aFBWr9IQMm9c8pC720DtJVvCU8NI Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBAgAGBQJVTM1tAAoJEGwLaZPeOHZ69fAP/00c7JAQpUQgzzskxtwFyo/2 wYGB4ABIZzCEz7wfHEzvnNoHCrgOE26o5tNXh+C07SwnFZguSfBqdEeSS/5Q3a1L pbQa5gaV3NAYyOy2CppzX+15IWN/AIWt8i0uEHOHMt5D69rimCE7mTscsBB1vZY7 cIOv9GpWeYEZomJ3ajhSgI6txSih+f0mu3vs356ZSpSNYFjgrkOl4Dd7mESwMQji FH6D9zni0Pd7o+j5yj4FSgkLGGorLV48F6VgFns6i6zhk/r/ZuUTNfdO81m6uOvz cxnCg78pNRY1XxQRn15APpHKC59rp9uKUio3V3AToww/Fbmml6i9WrDpHsOR4FUF RYMQU5Y43fBdBpZ368nJ+kA54f23zaa/caobFPRrHpHKY+GSV0/ePlA56iOMEoGf StAAoEn1+w5zbz99nJiU4eCkIWO9mvmwHEaeZJxaOpLJiR7S0bvTbwdVzqm965uL PNwNL0XhljyAnwEEWdegcnL0SvjpPtxKl9NWZmYCoLrEs3zmTtwJT3Cr530P3gSK mh9sQ+k/K3tLahjS9D5/anmAb5N2G7xt8SehDFoTRl5Yo+rpTGEQf1Ln7uHZx/Ll 4EI6bmI6MDh97rh78ZoOxFeIrsIDZmLlcJgl9LBA9GNyDIHYMa5woG/pfi3ZlclS JQKZtkv48rb3cb+HzJBV =crdc -----END PGP SIGNATURE----- --9iVb4aFBWr9IQMm9c8pC720DtJVvCU8NI-- From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailapp01.imgtec.com ([195.59.15.196]:59735 "EHLO imgpgp01.kl.imgtec.org" rhost-flags-OK-OK-OK-FAIL) by eddie.linux-mips.org with ESMTP id S27026566AbbEHOv3e-t2w (ORCPT ); Fri, 8 May 2015 16:51:29 +0200 Message-ID: <554CCD6D.9010004@imgtec.com> Date: Fri, 8 May 2015 15:51:25 +0100 From: James Hogan MIME-Version: 1.0 Subject: Re: [PATCH] mips:Remove unneeded duplicate declaration of cpu_callin_map in smp.h References: <1431094355-28145-1-git-send-email-xerofoify@gmail.com> In-Reply-To: <1431094355-28145-1-git-send-email-xerofoify@gmail.com> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="9iVb4aFBWr9IQMm9c8pC720DtJVvCU8NI" Return-Path: Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: Nicholas Krause , ralf@linux-mips.org Cc: chenhc@lemote.com, andreas.herrmann@caviumnetworks.com, rusty@rustcorp.com.au, linux-mips@linux-mips.org, linux-kernel@vger.kernel.org Message-ID: <20150508145125.3n5-ESe76qwYhl_GukP9LTlnJwrZ9ZJ9jSXuSqDfOqA@z> --9iVb4aFBWr9IQMm9c8pC720DtJVvCU8NI Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 08/05/15 15:12, Nicholas Krause wrote: > This removes the unneeded duplicate declaration of cpu_callin_map > in smp.h due to use already declaring it in the file,smp.c that No, it isn't declared in arch/mips/kernel/smp.c, its *defined* there (no "extern"). It's referenced by: arch/mips/cavium-octeon/smp.c arch/mips/kernel/process.c arch/mips/kernel/smp-bmips.c arch/mips/kernel/smp-cps.c arch/mips/loongson/loongson-3/smp.c as well as arch/mips/kernel/smp.c, which is why the declaration is needed in a header. If you're attempting to fix the build errors in this area, please see: http://patchwork.linux-mips.org/patch/9970/ Cheers James > already uses it internally for functions required this variable > for their various internal work. >=20 > Signed-off-by: Nicholas Krause > --- > arch/mips/include/asm/smp.h | 2 -- > 1 file changed, 2 deletions(-) >=20 > diff --git a/arch/mips/include/asm/smp.h b/arch/mips/include/asm/smp.h > index bb02fac..7752011 100644 > --- a/arch/mips/include/asm/smp.h > +++ b/arch/mips/include/asm/smp.h > @@ -45,8 +45,6 @@ extern int __cpu_logical_map[NR_CPUS]; > #define SMP_DUMP 0x8 > #define SMP_ASK_C0COUNT 0x10 > =20 > -extern volatile cpumask_t cpu_callin_map; > - > /* Mask of CPUs which are currently definitely operating coherently */= > extern cpumask_t cpu_coherent_mask; > =20 >=20 --9iVb4aFBWr9IQMm9c8pC720DtJVvCU8NI Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBAgAGBQJVTM1tAAoJEGwLaZPeOHZ69fAP/00c7JAQpUQgzzskxtwFyo/2 wYGB4ABIZzCEz7wfHEzvnNoHCrgOE26o5tNXh+C07SwnFZguSfBqdEeSS/5Q3a1L pbQa5gaV3NAYyOy2CppzX+15IWN/AIWt8i0uEHOHMt5D69rimCE7mTscsBB1vZY7 cIOv9GpWeYEZomJ3ajhSgI6txSih+f0mu3vs356ZSpSNYFjgrkOl4Dd7mESwMQji FH6D9zni0Pd7o+j5yj4FSgkLGGorLV48F6VgFns6i6zhk/r/ZuUTNfdO81m6uOvz cxnCg78pNRY1XxQRn15APpHKC59rp9uKUio3V3AToww/Fbmml6i9WrDpHsOR4FUF RYMQU5Y43fBdBpZ368nJ+kA54f23zaa/caobFPRrHpHKY+GSV0/ePlA56iOMEoGf StAAoEn1+w5zbz99nJiU4eCkIWO9mvmwHEaeZJxaOpLJiR7S0bvTbwdVzqm965uL PNwNL0XhljyAnwEEWdegcnL0SvjpPtxKl9NWZmYCoLrEs3zmTtwJT3Cr530P3gSK mh9sQ+k/K3tLahjS9D5/anmAb5N2G7xt8SehDFoTRl5Yo+rpTGEQf1Ln7uHZx/Ll 4EI6bmI6MDh97rh78ZoOxFeIrsIDZmLlcJgl9LBA9GNyDIHYMa5woG/pfi3ZlclS JQKZtkv48rb3cb+HzJBV =crdc -----END PGP SIGNATURE----- --9iVb4aFBWr9IQMm9c8pC720DtJVvCU8NI--