linux-kernel - Re: [PATCH RT 04/15] posixtimer: init timer only with CONFIG_POSIX

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] [day] [month] [year] [list]

Message-ID: <20171201123528.07eb8091@gandalf.local.home>
Date:   Fri, 1 Dec 2017 12:35:28 -0500
From:   Steven Rostedt <rostedt@...dmis.org>
To:     linux-kernel@...r.kernel.org,
        linux-rt-users <linux-rt-users@...r.kernel.org>
Cc:     Thomas Gleixner <tglx@...utronix.de>,
        Carsten Emde <C.Emde@...dl.org>,
        Sebastian Andrzej Siewior <bigeasy@...utronix.de>,
        John Kacur <jkacur@...hat.com>,
        Paul Gortmaker <paul.gortmaker@...driver.com>,
        Julia Cartwright <julia@...com>,
        Daniel Wagner <daniel.wagner@...mens.com>,
        tom.zanussi@...ux.intel.com, Alex Shi <alex.shi@...aro.org>
Subject: Re: [PATCH RT 04/15] posixtimer: init timer only with
 CONFIG_POSIX_TIMERS enabled

On Fri, 01 Dec 2017 10:50:03 -0500
Steven Rostedt <rostedt@...dmis.org> wrote:

> 4.9.65-rt57-rc1 stable review patch.
> If anyone has any objections, please let me know.
> 
> ------------------
> 
> From: Sebastian Andrzej Siewior <bigeasy@...utronix.de>
> 
> In v4.11 it is possible to disable the posix timers and so we must not
> attempt to initialize the task_struct on RT with !POSIX_TIMERS.
> This patch does so.

Hmm, I may have been too greedy in pulling in this patch. I'm going to
remove it from the list, and post a -rc2.

-- Steve

> 
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@...utronix.de>
> ---
>  include/linux/init_task.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/linux/init_task.h b/include/linux/init_task.h
> index a56e263f5005..526ecfc58909 100644
> --- a/include/linux/init_task.h
> +++ b/include/linux/init_task.h
> @@ -150,7 +150,7 @@ extern struct task_group root_task_group;
>  # define INIT_PERF_EVENTS(tsk)
>  #endif
>  
> -#ifdef CONFIG_PREEMPT_RT_BASE
> +#if defined(CONFIG_POSIX_TIMERS) && defined(CONFIG_PREEMPT_RT_BASE)
>  # define INIT_TIMER_LIST		.posix_timer_list = NULL,
>  #else
>  # define INIT_TIMER_LIST