Re: GEN2:Lager: Only 1 core works when turning off the SW8-PIN4
Hi Hiep, On Mon, Nov 7, 2016 at 11:37 AM, Hiep Cao Minh wrote: > On 11/07/2016 06:44 PM, Geert Uytterhoeven wrote: >> On Fri, Sep 30, 2016 at 12:05 PM, Hiep Cao Minh >> wrote: >> >> The issue you're seeing is due to a combination of commits >> 5f3bca0db8ac01a7 >> ("ARM: shmobile: apmu: Add APMU DT support via Enable method") and >> dc378795156d980c ("ARM: dts: r8a7790: Add APMU nodes"). >> >> When debug mode is enabled (SW8-4 = OFF), trying to boot secondary >> CPUs >> may >> lock up the system after a cold boot, depending of boot load version. >> Hence >> we explicitly prohibit that. BTW, this has been the case on Koelsch >> since >> commit >> 277efd30cfc72ec2 ("ARM: shmobile: Check r8a7791 MD21 at SMP boot"). >> >> Now, does series "[PATCH/RFT 0/4] ARM: shmobile: R-Car Gen2: Allow >> booting >> secondary CPU cores in debug mode" (included in all renesas-drivers >> releases >> during September) fix it for you? > > Thanks for your series patches. > I have had some problems of receiving post mail from Linux-Renesas > Mailing > List recently. > > Could you please forward these series patches to me? They're included in renesas-drivers. You can also get them from gitweb at https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/rcar-secondary-booting-in-debug-mode-v1 >>> >>> Thanks, I'll confirm these patches on my board. >> >> Have you tried this series on your Lager? If yes, can I add your >> Tested-by? > > Sorry for the late reply!. > I have totally forgotten my reporting test result to you. > I have tested this series on my Lager board. The bug has been fixed. Thanks, I will add your Tested-by. >> Which ES version of R-Car H1 do you have? If you don't know, you can >> tell me the result of "md.l 0xff44 1" in U-Boot. > > My board information: > LAGER SPI_LOADER V0.28 2014.09.29 > DEVICE S25FL512 > > > U-Boot 2016.01 (Jul 27 2016 - 15:27:42 +0900) > > CPU: Renesas Electronics R8A7790 rev 2.0 > - > => md.l 0xff44 1 > ff44: 4510 .E.. R-Car H2 ES2.0. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Re: GEN2:Lager: Only 1 core works when turning off the SW8-PIN4
Hi Geert On 11/07/2016 06:44 PM, Geert Uytterhoeven wrote: Hi Hiep, On Fri, Sep 30, 2016 at 12:05 PM, Hiep Cao Minh wrote: The issue you're seeing is due to a combination of commits 5f3bca0db8ac01a7 ("ARM: shmobile: apmu: Add APMU DT support via Enable method") and dc378795156d980c ("ARM: dts: r8a7790: Add APMU nodes"). When debug mode is enabled (SW8-4 = OFF), trying to boot secondary CPUs may lock up the system after a cold boot, depending of boot load version. Hence we explicitly prohibit that. BTW, this has been the case on Koelsch since commit 277efd30cfc72ec2 ("ARM: shmobile: Check r8a7791 MD21 at SMP boot"). Now, does series "[PATCH/RFT 0/4] ARM: shmobile: R-Car Gen2: Allow booting secondary CPU cores in debug mode" (included in all renesas-drivers releases during September) fix it for you? Thanks for your series patches. I have had some problems of receiving post mail from Linux-Renesas Mailing List recently. Could you please forward these series patches to me? They're included in renesas-drivers. You can also get them from gitweb at https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/rcar-secondary-booting-in-debug-mode-v1 Thanks, I'll confirm these patches on my board. Have you tried this series on your Lager? If yes, can I add your Tested-by? Sorry for the late reply!. I have totally forgotten my reporting test result to you. I have tested this series on my Lager board. The bug has been fixed. Which ES version of R-Car H1 do you have? If you don't know, you can tell me the result of "md.l 0xff44 1" in U-Boot. My board information: LAGER SPI_LOADER V0.28 2014.09.29 DEVICE S25FL512 U-Boot 2016.01 (Jul 27 2016 - 15:27:42 +0900) CPU: Renesas Electronics R8A7790 rev 2.0 - => md.l 0xff44 1 ff44: 4510 .E.. Thanks, Jinzai Solution Inc, Hiep. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Re: GEN2:Lager: Only 1 core works when turning off the SW8-PIN4
Hi Hiep, On Fri, Sep 30, 2016 at 12:05 PM, Hiep Cao Minh wrote: The issue you're seeing is due to a combination of commits 5f3bca0db8ac01a7 ("ARM: shmobile: apmu: Add APMU DT support via Enable method") and dc378795156d980c ("ARM: dts: r8a7790: Add APMU nodes"). When debug mode is enabled (SW8-4 = OFF), trying to boot secondary CPUs may lock up the system after a cold boot, depending of boot load version. Hence we explicitly prohibit that. BTW, this has been the case on Koelsch since commit 277efd30cfc72ec2 ("ARM: shmobile: Check r8a7791 MD21 at SMP boot"). Now, does series "[PATCH/RFT 0/4] ARM: shmobile: R-Car Gen2: Allow booting secondary CPU cores in debug mode" (included in all renesas-drivers releases during September) fix it for you? >>> >>> Thanks for your series patches. >>> I have had some problems of receiving post mail from Linux-Renesas >>> Mailing >>> List recently. >>> >>> Could you please forward these series patches to me? >> >> They're included in renesas-drivers. >> You can also get them from gitweb at >> >> https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/rcar-secondary-booting-in-debug-mode-v1 > > Thanks, I'll confirm these patches on my board. Have you tried this series on your Lager? If yes, can I add your Tested-by? Which ES version of R-Car H1 do you have? If you don't know, you can tell me the result of "md.l 0xff44 1" in U-Boot. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Re: GEN2:Lager: Only 1 core works when turning off the SW8-PIN4
Hi Geert, The issue you're seeing is due to a combination of commits 5f3bca0db8ac01a7 ("ARM: shmobile: apmu: Add APMU DT support via Enable method") and dc378795156d980c ("ARM: dts: r8a7790: Add APMU nodes"). When debug mode is enabled (SW8-4 = OFF), trying to boot secondary CPUs may lock up the system after a cold boot, depending of boot load version. Hence we explicitly prohibit that. BTW, this has been the case on Koelsch since commit 277efd30cfc72ec2 ("ARM: shmobile: Check r8a7791 MD21 at SMP boot"). Now, does series "[PATCH/RFT 0/4] ARM: shmobile: R-Car Gen2: Allow booting secondary CPU cores in debug mode" (included in all renesas-drivers releases during September) fix it for you? Thanks for your series patches. I have had some problems of receiving post mail from Linux-Renesas Mailing List recently. Could you please forward these series patches to me? They're included in renesas-drivers. You can also get them from gitweb at https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/rcar-secondary-booting-in-debug-mode-v1 Thanks, I'll confirm these patches on my board. Thank you. Hiep.
Re: GEN2:Lager: Only 1 core works when turning off the SW8-PIN4
Hi Hiep, On Fri, Sep 30, 2016 at 11:55 AM, Hiep Cao Minh wrote: > On 09/29/2016 10:40 PM, Geert Uytterhoeven wrote: >> On Thu, Sep 29, 2016 at 1:03 PM, Hiep Cao Minh >> wrote: >>> I'd like to report the issue of the CPU operation. >>> We tested and found it on Lager board at linux-v4.8-rcx. >>> >>> The test procedure is the following: >>> >>> Confirmation command: >>> >>> # cat /proc/interrupts >>> CPU0 >>> 19: 2509 GIC-0 27 Level arch_timer >>> 21: 0 GIC-0 36 Level e605.gpio >>> 22: 0 GIC-0 37 Level e6051000.gpio >>> 23: 0 GIC-0 38 Level e6052000.gpio >>> 24: 0 GIC-0 39 Level e6053000.gpio >>> 25: 0 GIC-0 40 Level e6054000.gpio >>> 26: 0 GIC-0 41 Level e6055000.gpio >>> 27: 23 GIC-0 101 Level e61f.thermal >>> …” >>> >>> This issue appears when we changed the SW8-PIN4 of Lager board. >>> >>> SW8-PIN4: ON >>> >>> + At linux-v4.7: OK (4 cores work together normally). >>> + At linux-v4.8-rc2: OK (4 cores work together normally). >>> >>> SW8-PIN4: OFF >>> >>> + At linux-v4.7: -> OK(4 cores work together normally). >>> + At linux-v4.8-rc2: -> NG(Only 1 core works). >> >> And the kernel prints "Unable to boot CPU%u when MD21 is set", right? >> >>> We tried to find out the issued patch and we realize that it happens from >>> the following patch: >>> " 043248c Merge tag 'armsoc-dt' of >>> git://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc" >> >> The issue you're seeing is due to a combination of commits >> 5f3bca0db8ac01a7 >> ("ARM: shmobile: apmu: Add APMU DT support via Enable method") and >> dc378795156d980c ("ARM: dts: r8a7790: Add APMU nodes"). >> >> When debug mode is enabled (SW8-4 = OFF), trying to boot secondary CPUs >> may >> lock up the system after a cold boot, depending of boot load version. >> Hence >> we explicitly prohibit that. BTW, this has been the case on Koelsch since >> commit >> 277efd30cfc72ec2 ("ARM: shmobile: Check r8a7791 MD21 at SMP boot"). >> >> Now, does series "[PATCH/RFT 0/4] ARM: shmobile: R-Car Gen2: Allow booting >> secondary CPU cores in debug mode" (included in all renesas-drivers >> releases >> during September) fix it for you? > > Thanks for your series patches. > I have had some problems of receiving post mail from Linux-Renesas Mailing > List recently. > > Could you please forward these series patches to me? They're included in renesas-drivers. You can also get them from gitweb at https://git.kernel.org/cgit/linux/kernel/git/geert/renesas-drivers.git/log/?h=topic/rcar-secondary-booting-in-debug-mode-v1 Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
Re: GEN2:Lager: Only 1 core works when turning off the SW8-PIN4
Hi Geert, Thanks for your reply! On 09/29/2016 10:40 PM, Geert Uytterhoeven wrote: Hi Hiep, On Thu, Sep 29, 2016 at 1:03 PM, Hiep Cao Minh wrote: I'd like to report the issue of the CPU operation. We tested and found it on Lager board at linux-v4.8-rcx. The test procedure is the following: Confirmation command: # cat /proc/interrupts CPU0 19: 2509 GIC-0 27 Level arch_timer 21: 0 GIC-0 36 Level e605.gpio 22: 0 GIC-0 37 Level e6051000.gpio 23: 0 GIC-0 38 Level e6052000.gpio 24: 0 GIC-0 39 Level e6053000.gpio 25: 0 GIC-0 40 Level e6054000.gpio 26: 0 GIC-0 41 Level e6055000.gpio 27: 23 GIC-0 101 Level e61f.thermal …” This issue appears when we changed the SW8-PIN4 of Lager board. SW8-PIN4: ON + At linux-v4.7: OK (4 cores work together normally). + At linux-v4.8-rc2: OK (4 cores work together normally). SW8-PIN4: OFF + At linux-v4.7: -> OK(4 cores work together normally). + At linux-v4.8-rc2: -> NG(Only 1 core works). And the kernel prints "Unable to boot CPU%u when MD21 is set", right? We tried to find out the issued patch and we realize that it happens from the following patch: " 043248c Merge tag 'armsoc-dt' of git://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc" The issue you're seeing is due to a combination of commits 5f3bca0db8ac01a7 ("ARM: shmobile: apmu: Add APMU DT support via Enable method") and dc378795156d980c ("ARM: dts: r8a7790: Add APMU nodes"). When debug mode is enabled (SW8-4 = OFF), trying to boot secondary CPUs may lock up the system after a cold boot, depending of boot load version. Hence we explicitly prohibit that. BTW, this has been the case on Koelsch since commit 277efd30cfc72ec2 ("ARM: shmobile: Check r8a7791 MD21 at SMP boot"). Now, does series "[PATCH/RFT 0/4] ARM: shmobile: R-Car Gen2: Allow booting secondary CPU cores in debug mode" (included in all renesas-drivers releases during September) fix it for you? Thanks for your series patches. I have had some problems of receiving post mail from Linux-Renesas Mailing List recently. Could you please forward these series patches to me? Thank you!. Jinso/Linux-Team Hiep.
Re: GEN2:Lager: Only 1 core works when turning off the SW8-PIN4
Hi Hiep, On Thu, Sep 29, 2016 at 1:03 PM, Hiep Cao Minh wrote: > I'd like to report the issue of the CPU operation. > We tested and found it on Lager board at linux-v4.8-rcx. > > The test procedure is the following: > > Confirmation command: > > # cat /proc/interrupts > CPU0 > 19: 2509 GIC-0 27 Level arch_timer > 21: 0 GIC-0 36 Level e605.gpio > 22: 0 GIC-0 37 Level e6051000.gpio > 23: 0 GIC-0 38 Level e6052000.gpio > 24: 0 GIC-0 39 Level e6053000.gpio > 25: 0 GIC-0 40 Level e6054000.gpio > 26: 0 GIC-0 41 Level e6055000.gpio > 27: 23 GIC-0 101 Level e61f.thermal > …” > > This issue appears when we changed the SW8-PIN4 of Lager board. > > SW8-PIN4: ON > > + At linux-v4.7: OK (4 cores work together normally). > + At linux-v4.8-rc2: OK (4 cores work together normally). > > SW8-PIN4: OFF > > + At linux-v4.7: -> OK(4 cores work together normally). > + At linux-v4.8-rc2: -> NG(Only 1 core works). And the kernel prints "Unable to boot CPU%u when MD21 is set", right? > We tried to find out the issued patch and we realize that it happens from > the following patch: > " 043248c Merge tag 'armsoc-dt' of > git://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc" The issue you're seeing is due to a combination of commits 5f3bca0db8ac01a7 ("ARM: shmobile: apmu: Add APMU DT support via Enable method") and dc378795156d980c ("ARM: dts: r8a7790: Add APMU nodes"). When debug mode is enabled (SW8-4 = OFF), trying to boot secondary CPUs may lock up the system after a cold boot, depending of boot load version. Hence we explicitly prohibit that. BTW, this has been the case on Koelsch since commit 277efd30cfc72ec2 ("ARM: shmobile: Check r8a7791 MD21 at SMP boot"). Now, does series "[PATCH/RFT 0/4] ARM: shmobile: R-Car Gen2: Allow booting secondary CPU cores in debug mode" (included in all renesas-drivers releases during September) fix it for you? Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds