Re: [PATCH v5 07/13] ARM: smp: Add initialization of CNTVOFF

2018-04-11 Thread Mylène Josserand
Hello, On Mon, 9 Apr 2018 10:24:41 +0200 Geert Uytterhoeven wrote: > On Wed, Apr 4, 2018 at 4:30 PM, Marc Zyngier wrote: > > On Wed, 04 Apr 2018 14:59:09 +0100, > > Mylčne Josserand wrote: > > [Marc: stuck in ISO-8859-1? ;-] > > >> > It'd be good to take this opportunity to refactor the shm

Re: [PATCH v5 07/13] ARM: smp: Add initialization of CNTVOFF

2018-04-09 Thread Marc Zyngier
On 09/04/18 09:24, Geert Uytterhoeven wrote: > On Wed, Apr 4, 2018 at 4:30 PM, Marc Zyngier wrote: >> On Wed, 04 Apr 2018 14:59:09 +0100, >> Mylčne Josserand wrote: > > [Marc: stuck in ISO-8859-1? ;-] I have no idea what Wanderlust does (that's what I use on my laptop). But Thunderbird definitel

Re: [PATCH v5 07/13] ARM: smp: Add initialization of CNTVOFF

2018-04-09 Thread Geert Uytterhoeven
On Wed, Apr 4, 2018 at 4:30 PM, Marc Zyngier wrote: > On Wed, 04 Apr 2018 14:59:09 +0100, > Mylčne Josserand wrote: [Marc: stuck in ISO-8859-1? ;-] >> > It'd be good to take this opportunity to refactor the shmobile code. >> >> I can do it in this series but I do not have any shmobile platforms

Re: [PATCH v5 07/13] ARM: smp: Add initialization of CNTVOFF

2018-04-04 Thread Marc Zyngier
On Wed, 04 Apr 2018 14:59:09 +0100, Mylène Josserand wrote: > > It'd be good to take this opportunity to refactor the shmobile code. > > I can do it in this series but I do not have any shmobile platforms so > I will not be able to test my modifications (only compilation). > > If someone can tes

Re: [PATCH v5 07/13] ARM: smp: Add initialization of CNTVOFF

2018-04-04 Thread Mylène Josserand
Hi Marc, Thank you for the review. On Wed, 4 Apr 2018 14:01:48 +0100 Marc Zyngier wrote: > Hi Mylène, > > On 03/04/18 07:18, Mylène Josserand wrote: > > The CNTVOFF register from arch timer is uninitialized. > > It should be done by the bootloader but it is currently not the case, > > even for

Re: [PATCH v5 07/13] ARM: smp: Add initialization of CNTVOFF

2018-04-04 Thread Marc Zyngier
Hi Mylène, On 03/04/18 07:18, Mylène Josserand wrote: > The CNTVOFF register from arch timer is uninitialized. > It should be done by the bootloader but it is currently not the case, > even for boot CPU because this SoC is booting in secure mode. > It leads to an random offset value meaning that e

[PATCH v5 07/13] ARM: smp: Add initialization of CNTVOFF

2018-04-02 Thread Mylène Josserand
The CNTVOFF register from arch timer is uninitialized. It should be done by the bootloader but it is currently not the case, even for boot CPU because this SoC is booting in secure mode. It leads to an random offset value meaning that each CPU will have a different time, which isn't working very we