Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-11-06 Thread Christian Zigotzky
On 07 November 2018 at 03:11AM, Scott Wood wrote: On Sun, 2018-10-28 at 17:35 +0100, Christian Zigotzky wrote: Hello, SMP doesn't work anymore with the latest Git kernel (28/10/18 11:12AM GMT) on my P5020 board and on virtual e5500 QEMU machines. Board with P5020 dual core CPU: [

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-11-06 Thread Scott Wood
On Sun, 2018-10-28 at 17:35 +0100, Christian Zigotzky wrote: > Hello, > > SMP doesn't work anymore with the latest Git kernel (28/10/18 11:12AM > GMT) on my P5020 board and on virtual e5500 QEMU machines. > > Board with P5020 dual core CPU: > > [0.00]

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-31 Thread Christian Zigotzky
Hi Michael, Many thanks for this good explanation. I will try to learn more about bisecting. Sometimes the problem is the time. I usually work for a Linux first level support for our AmigaOne machines. That means my main work is end user support. Therefore I have to learn more about second

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-31 Thread Michael Ellerman
Christian Zigotzky writes: > Little progress ... > > I reverted the following two OF files of the commit 'Merge tag > devicetree-for-4.20' and SMP works! The problematic code is somewhere in > these two files. > > a/include/linux/of.h > a/drivers/of/base.c Hi Christian, Trying to debug

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-30 Thread Christian Zigotzky
On 30 October 2018 at 08:27AM, Christian Zigotzky wrote: On 30 October 2018 at 02:59AM, Benjamin Herrenschmidt wrote: On Tue, 2018-10-30 at 02:42 +0100, Christian Zigotzky wrote: OF patch for the latest Git kernel: http://www.xenosoft.de/of_v2.patch This just seems to revert a whole bunch of

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-30 Thread Christian Zigotzky
On 30 October 2018 at 02:59AM, Benjamin Herrenschmidt wrote: On Tue, 2018-10-30 at 02:42 +0100, Christian Zigotzky wrote: OF patch for the latest Git kernel: http://www.xenosoft.de/of_v2.patch This just seems to revert a whole bunch of stuff, not really the right way to go. Why is

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread Benjamin Herrenschmidt
On Tue, 2018-10-30 at 02:42 +0100, Christian Zigotzky wrote: > OF patch for the latest Git kernel: http://www.xenosoft.de/of_v2.patch This just seems to revert a whole bunch of stuff, not really the right way to go. Why is of_get_next_cpu_node() not finding your CPUs ? There must be something

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread Christian Zigotzky
OF patch for the latest Git kernel: http://www.xenosoft.de/of_v2.patch - of_v2.patch - diff -rupN a/drivers/of/base.c b/drivers/of/base.c --- a/drivers/of/base.c    2018-10-30 02:19:30.827089495 +0100 +++ b/drivers/of/base.c    2018-10-30

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread Christian Zigotzky
On 30 October 2018 at 00:04AM, Scott Wood wrote: On Mon, 2018-10-29 at 21:12 +0100, Christian Zigotzky wrote: Hello Christophe, Bisecting isn’t necessary because the problem is in the following files. drivers/of/base.c drivers/of/device.c drivers/of/of_mdio.c drivers/of/of_numa.c

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread Scott Wood
On Mon, 2018-10-29 at 21:12 +0100, Christian Zigotzky wrote: > Hello Christophe, > > Bisecting isn’t necessary because the problem is in the following files. > > drivers/of/base.c > drivers/of/device.c > drivers/of/of_mdio.c > drivers/of/of_numa.c > drivers/of/of_private.h > drivers/of/overlay.c

NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread Christian Zigotzky
Little progress ... I reverted the following two OF files of the commit 'Merge tag devicetree-for-4.20' and SMP works! The problematic code is somewhere in these two files. a/include/linux/of.h a/drivers/of/base.c -- Christian On 29 October 2018 at 6:00PM, Christian Zigotzky wrote:

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread Christian Zigotzky
Hello Christophe, Bisecting isn’t necessary because the problem is in the following files. drivers/of/base.c drivers/of/device.c drivers/of/of_mdio.c drivers/of/of_numa.c drivers/of/of_private.h drivers/of/overlay.c drivers/of/platform.c drivers/of/unittest-data/overlay_15.dts

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread LEROY Christophe
Christian Zigotzky a écrit : Hello, I figured out that the problem is in the OF source code of the commit: Merge tag devicetree-for-4.20. [1] That's a merge commit. Can you bisect the branch and identify the faulting commit ? Christophe I reverted the following OF files and SMP

Re: NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread Christian Zigotzky
Hello, I figured out that the problem is in the OF source code of the commit: Merge tag devicetree-for-4.20. [1] I reverted the following OF files and SMP works! drivers/of/base.c drivers/of/device.c drivers/of/of_mdio.c drivers/of/of_numa.c drivers/of/of_private.h drivers/of/overlay.c

NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-29 Thread Christian Zigotzky
Hello, I have figured out that the commit 'devicetree-for-4.20' [1] is responsible for the SMP problem. I was able to revert this commit with 'git revert b27186abb37b7bd19e0ca434f4f425c807dbd708 -m 1' today. [master ec81438] Revert "Merge tag 'devicetree-for-4.20' of

NXP P50XX/e5500: SMP doesn't work anymore with the latest Git kernel

2018-10-28 Thread Christian Zigotzky
Hello, SMP doesn't work anymore with the latest Git kernel (28/10/18 11:12AM GMT) on my P5020 board and on virtual e5500 QEMU machines. Board with P5020 dual core CPU: [    0.00] - [    0.00] phys_mem_size = 0x2 [