linux-kernel - Re: [PATCH 0/3] timers: Allocate per-cpu tvec

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAKohponL7cUFUR4eM5QyO8dxrbZGahGU9i-C8qwA-Yx_K1MseQ@mail.gmail.com>
Date:	Mon, 30 Mar 2015 15:25:36 +0530
From:	Viresh Kumar <viresh.kumar@...aro.org>
To:	Peter Zijlstra <peterz@...radead.org>
Cc:	Ingo Molnar <mingo@...hat.com>,
	Thomas Gleixner <tglx@...utronix.de>,
	Linaro Kernel Mailman List <linaro-kernel@...ts.linaro.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 0/3] timers: Allocate per-cpu tvec_base's statically

On 30 March 2015 at 13:48, Peter Zijlstra <peterz@...radead.org> 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@...radead.org>
> 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@...radead.org>
> ---
>  kernel/time/timer.c |   28 +++++++++++++---------------
>  1 file changed, 13 insertions(+), 15 deletions(-)

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