From mboxrd@z Thu Jan 1 00:00:00 1970 From: maxime.coquelin@st.com (Maxime Coquelin) Date: Wed, 10 Jun 2015 10:03:15 +0200 Subject: [PATCH v2 0/3] Add code to release secondary cores from holding pen. In-Reply-To: <7hd214sx9b.fsf@deeprootsystems.com> References: <1433856824-30689-1-git-send-email-peter.griffin@linaro.org> <7hd214sx9b.fsf@deeprootsystems.com> Message-ID: <5577EF43.6020600@st.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Kevin, On 06/09/2015 05:43 PM, Kevin Hilman wrote: > Peter Griffin writes: > >> Hi Maxime, Patrice, Srini, Kevin, Olof & Arnd, >> >> This patchset adds in the necessary code to manage the holding pen for STi >> platforms. >> >> Due to all the STi upstream devs using JTAG to boot the STi boards, there >> is currently a SMP bug when booting upstream kernels via u-boot where >> only the primary core is brought up. >> >> This hasn't been noticed until now because when booting via JTAG the >> stlinux_arm_boot script sets the PC of the secondary cores directly. >> >> With these patches applied booting via u-boot now works correctly. >> >> [ 0.045456] CPU: Testing write buffer coherency: ok >> [ 0.045597] CPU0: thread -1, cpu 0, socket 0, mpidr 80000000 >> [ 0.045734] Setting up static identity map for 0x40209000 - 0x40209098 >> [ 0.065047] CPU1: thread -1, cpu 1, socket 0, mpidr 80000001 >> [ 0.065081] Brought up 2 CPUs >> [ 0.065089] SMP: Total of 2 processors activated (5983.43 BogoMIPS). >> [ 0.065092] CPU: All CPU(s) started in SVC mode. >> >> Kevin / Olof / Arnd - Can you put this fix into the next ARM SoC fixes pull >> request (if there is one)? > At this stage of the cycle (we're already at -rc7) we limit things to > strictly regression fixes, and seems this has been broken for some time, > so I think this will have to wait for v4.2. > > After addresing the comments, Maxime should queue this up for arm-soc so > we can get it in early in the v4.2-rc cycle. Ok, I will take it for v4.2-rc cycle. Thanks, Maxime From mboxrd@z Thu Jan 1 00:00:00 1970 From: Maxime Coquelin Subject: Re: [PATCH v2 0/3] Add code to release secondary cores from holding pen. Date: Wed, 10 Jun 2015 10:03:15 +0200 Message-ID: <5577EF43.6020600@st.com> References: <1433856824-30689-1-git-send-email-peter.griffin@linaro.org> <7hd214sx9b.fsf@deeprootsystems.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <7hd214sx9b.fsf-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Kevin Hilman , Peter Griffin Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, srinivas.kandagatla-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, patrice.chotard-qxv4g6HH51o@public.gmane.org, arnd-r2nGTMty4D4@public.gmane.org, olof-nZhT3qVonbNeoWH0uzbU5w@public.gmane.org, lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Hi Kevin, On 06/09/2015 05:43 PM, Kevin Hilman wrote: > Peter Griffin writes: > >> Hi Maxime, Patrice, Srini, Kevin, Olof & Arnd, >> >> This patchset adds in the necessary code to manage the holding pen for STi >> platforms. >> >> Due to all the STi upstream devs using JTAG to boot the STi boards, there >> is currently a SMP bug when booting upstream kernels via u-boot where >> only the primary core is brought up. >> >> This hasn't been noticed until now because when booting via JTAG the >> stlinux_arm_boot script sets the PC of the secondary cores directly. >> >> With these patches applied booting via u-boot now works correctly. >> >> [ 0.045456] CPU: Testing write buffer coherency: ok >> [ 0.045597] CPU0: thread -1, cpu 0, socket 0, mpidr 80000000 >> [ 0.045734] Setting up static identity map for 0x40209000 - 0x40209098 >> [ 0.065047] CPU1: thread -1, cpu 1, socket 0, mpidr 80000001 >> [ 0.065081] Brought up 2 CPUs >> [ 0.065089] SMP: Total of 2 processors activated (5983.43 BogoMIPS). >> [ 0.065092] CPU: All CPU(s) started in SVC mode. >> >> Kevin / Olof / Arnd - Can you put this fix into the next ARM SoC fixes pull >> request (if there is one)? > At this stage of the cycle (we're already at -rc7) we limit things to > strictly regression fixes, and seems this has been broken for some time, > so I think this will have to wait for v4.2. > > After addresing the comments, Maxime should queue this up for arm-soc so > we can get it in early in the v4.2-rc cycle. Ok, I will take it for v4.2-rc cycle. Thanks, Maxime -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964828AbbFJIFN (ORCPT ); Wed, 10 Jun 2015 04:05:13 -0400 Received: from mx07-00178001.pphosted.com ([62.209.51.94]:40012 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753984AbbFJIE7 (ORCPT ); Wed, 10 Jun 2015 04:04:59 -0400 Message-ID: <5577EF43.6020600@st.com> Date: Wed, 10 Jun 2015 10:03:15 +0200 From: Maxime Coquelin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 MIME-Version: 1.0 To: Kevin Hilman , Peter Griffin CC: , , , , , , , Subject: Re: [PATCH v2 0/3] Add code to release secondary cores from holding pen. References: <1433856824-30689-1-git-send-email-peter.griffin@linaro.org> <7hd214sx9b.fsf@deeprootsystems.com> In-Reply-To: <7hd214sx9b.fsf@deeprootsystems.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.201.23.80] X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:5.14.151,1.0.33,0.0.0000 definitions=2015-06-10_02:2015-06-09,2015-06-10,1970-01-01 signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Kevin, On 06/09/2015 05:43 PM, Kevin Hilman wrote: > Peter Griffin writes: > >> Hi Maxime, Patrice, Srini, Kevin, Olof & Arnd, >> >> This patchset adds in the necessary code to manage the holding pen for STi >> platforms. >> >> Due to all the STi upstream devs using JTAG to boot the STi boards, there >> is currently a SMP bug when booting upstream kernels via u-boot where >> only the primary core is brought up. >> >> This hasn't been noticed until now because when booting via JTAG the >> stlinux_arm_boot script sets the PC of the secondary cores directly. >> >> With these patches applied booting via u-boot now works correctly. >> >> [ 0.045456] CPU: Testing write buffer coherency: ok >> [ 0.045597] CPU0: thread -1, cpu 0, socket 0, mpidr 80000000 >> [ 0.045734] Setting up static identity map for 0x40209000 - 0x40209098 >> [ 0.065047] CPU1: thread -1, cpu 1, socket 0, mpidr 80000001 >> [ 0.065081] Brought up 2 CPUs >> [ 0.065089] SMP: Total of 2 processors activated (5983.43 BogoMIPS). >> [ 0.065092] CPU: All CPU(s) started in SVC mode. >> >> Kevin / Olof / Arnd - Can you put this fix into the next ARM SoC fixes pull >> request (if there is one)? > At this stage of the cycle (we're already at -rc7) we limit things to > strictly regression fixes, and seems this has been broken for some time, > so I think this will have to wait for v4.2. > > After addresing the comments, Maxime should queue this up for arm-soc so > we can get it in early in the v4.2-rc cycle. Ok, I will take it for v4.2-rc cycle. Thanks, Maxime