Re: [PATCH 0/3] timers: Allocate per-cpu tvec_base's statically

From: Viresh Kumar
Date: Mon Mar 30 2015 - 05:55:42 EST


On 30 March 2015 at 13:48, Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> I did the below on top; I'll sit on these patches for a wee while in the
> hope of the blackfin maintainer explaining his reasons for wrecking
> ____cacheline_aligned.

Two things I wanted to mention:

1.) We may need to drop the patch 2/3 as that breaks build for xtensa and tile.

kernel/time/timer.c: In function 'init_timers_cpu':
kernel/time/timer.c:1539:1: error: section attribute cannot be
specified for local variables
kernel/time/timer.c:1539:1: error: section attribute cannot be
specified for local variables
>> kernel/time/timer.c:1539:1: error: declaration of '__pcpu_unique___tvec_bases' with no linkage follows extern declaration
kernel/time/timer.c:1539:1: note: previous declaration of
'__pcpu_unique___tvec_bases' was here
kernel/time/timer.c:1539:9: error: section attribute cannot be
specified for local variables
kernel/time/timer.c:1539:9: error: section attribute cannot be
specified for local variables
kernel/time/timer.c:1539:9: error: weak declaration of
'__tvec_bases' must be public
kernel/time/timer.c:1539:9: error: declaration of '__tvec_bases'
with no linkage follows extern declaration
kernel/time/timer.c:1539:9: note: previous declaration of
'__tvec_bases' was here

Can these be fixed somehow ?


2.) We probably don't need to wait for the blackfin guys as we
haven't broken anything yet. It will be broken only once we start
using an extra bit from base pointer, which will be done by a separate
patch trying to migrate running timers.

> ---
> Subject: timer: Further simplify SMP and HOTPLUG logic
> From: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> Date: Mon Mar 30 10:09:19 CEST 2015
>
> Remove one CONFIG_HOTPLUG_CPU #ifdef in trade for introducing one
> CONFIG_SMP #ifdef.
>
> The CONFIG_SMP ifdef avoids declaring the per-cpu __tvec_bases storage
> on UP systems since they already have boot_tvec_bases.
>
> Also (re)add a runtime check on the base alignment -- for the paranoid
> amongst us :-)
>
> Signed-off-by: Peter Zijlstra (Intel) <peterz@xxxxxxxxxxxxx>
> ---
> kernel/time/timer.c | 28 +++++++++++++---------------
> 1 file changed, 13 insertions(+), 15 deletions(-)

Reviewed-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/