linux-kernel - Re: [PATCH 0/4] ARM: cpuidle: use const and __initconst for cpuidle

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: <20160420050141.GE5862@linaro.org>
Date:	Wed, 20 Apr 2016 07:01:41 +0200
From:	Daniel Lezcano <daniel.lezcano@...aro.org>
To:	Lorenzo Pieralisi <lorenzo.pieralisi@....com>
Cc:	Jisheng Zhang <jszhang@...vell.com>, linux@....linux.org.uk,
	mark.rutland@....com, andy.gross@...aro.org,
	david.brown@...aro.org, linux-arm-kernel@...ts.infradead.org,
	linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org,
	linux-soc@...r.kernel.org
Subject: Re: [PATCH 0/4] ARM: cpuidle: use const and __initconst for
 cpuidle_ops

On Tue, Apr 19, 2016 at 06:31:14PM +0100, Lorenzo Pieralisi wrote:
> Hi Daniel,
> 
> On Fri, Mar 25, 2016 at 12:26:27PM +0100, Daniel Lezcano wrote:
> > On 03/22/2016 03:42 PM, Jisheng Zhang wrote:
> > >These trivial patches are similar as Masahiro posted in[1]. The main
> > >purpose is let cpuidle_ops structure be constified and replace
> > >"__initdata" with "__initconst".
> > >
> > >[1] http://lists.infradead.org/pipermail/linux-arm-kernel/2015-August/365826.html
> > >
> > >Jisheng Zhang (4):
> > >   ARM: cpuidle: add const qualifier to cpuidle_ops member in structures
> > >   ARM: cpuidle: constify return value of arm_cpuidle_get_ops()
> > >   soc: qcom: spm: use const and __initconst for qcom_cpuidle_ops
> > >   drivers: firmware: psci: use const and __initconst for
> > >     psci_cpuidle_ops
> > >
> > >  arch/arm/include/asm/cpuidle.h | 2 +-
> > >  arch/arm/kernel/cpuidle.c      | 6 +++---
> > >  drivers/firmware/psci.c        | 2 +-
> > >  drivers/soc/qcom/spm.c         | 2 +-
> > >  4 files changed, 6 insertions(+), 6 deletions(-)
> > 
> > Sounds reasonable.
> 
> Are you taking them ? I could send the last one but it would
> make more sense for them to be part of the same series, I will
> check they do not conflict with patches queued for PSCI.

Ok, let me know via an acked-by.

Thanks.

  -- Daniel