From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Pinchart Date: Mon, 12 May 2014 20:00:08 +0000 Subject: Re: [PATCH 00/03] ARM: shmobile: Use shmobile_init_delay() on r8a7791 Message-Id: <1509906.x9u0HJ6AbJ@avalon> List-Id: References: <20140511232508.20177.11556.sendpatchset@w520> In-Reply-To: <20140511232508.20177.11556.sendpatchset@w520> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: linux-arm-kernel@lists.infradead.org Hi Magnus, On Monday 12 May 2014 08:25:08 Magnus Damm wrote: > ARM: shmobile: Use shmobile_init_delay() on r8a7791 > > [PATCH 01/03] ARM: shmobile: Use r8a7791 DT CPU Frequency in common case > [PATCH 02/03] ARM: shmobile: Use r8a7791 DT CPU Frequency for Koelsch > [PATCH 03/03] ARM: shmobile: Remove unused r8a7791_init_early() > > Convert r8a7791 to rely on shmobile_init_delay() instead of using a per-SoC > delay setup function. I'm afraid this patch set causes a boot time and CPU load regression on Koelsch. With Simon's latest devel branch which has the patches applied, the kernel boot time (using koelsch_defconfig) up to the point where the kernel tries to connect to the DHCP server is 17.898437s. The CPU load (as reported by top) on an idle system is then around 33%, spent in [kworker/0:1]. If I revert the patches the same time goes down to 1.921875s and the CPU load on an idle system is close to 0. > Signed-off-by: Magnus Damm > --- > > Written against renesas-devel-v3.15-rc5-20140511 > > arch/arm/mach-shmobile/board-koelsch-reference.c | 2 +- > arch/arm/mach-shmobile/board-koelsch.c | 2 +- > arch/arm/mach-shmobile/include/mach/r8a7791.h | 1 - > arch/arm/mach-shmobile/setup-r8a7791.c | 9 +-------- > 4 files changed, 3 insertions(+), 11 deletions(-) -- Regards, Laurent Pinchart