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
| ||
|
Date: Wed, 30 Mar 2016 11:36:52 +0100 From: Lorenzo Pieralisi <lorenzo.pieralisi@....com> To: Daniel Lezcano <daniel.lezcano@...aro.org> Cc: Jisheng Zhang <jszhang@...vell.com>, linux@....linux.org.uk, linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, Catalin Marinas <Catalin.Marinas@....com> Subject: Re: [PATCH 1/2] ARM: cpuidle: fix !cpuidle_ops[cpu].init case during init On Wed, Mar 30, 2016 at 10:09:12AM +0200, Daniel Lezcano wrote: > On 03/30/2016 09:16 AM, Jisheng Zhang wrote: > >Hi Daniel, > > [ ... ] > > Added Lorenzo and Catalin. > > >>Hi Jisheng, > >> > >>this should be handled in the arm_cpuidle_read_ops function. > >> > > > >Thanks for reviewing. After some consideration, I think this patch isn't correct > >There may be platforms which doesn't need the init member at all, although > >currently I don't see such platforms in mainline, So I'll drop this patch > >and send out one v2 only does the optimization. > > There is an inconsistency between ARM and ARM64. The 'cpu_get_ops', > the arm_cpuidle_read_ops from the ARM64 side, returns -EOPNOTSUPP > when the init function is not there for cpuidle. > > I don't think it is a problem, but as ARM/ARM64 are sharing the same > cpuidle-arm.c driver it would make sense to unify the behavior > between both archs. I agree and I think it makes sense to have an arm back-end that fails if there is no cpuidle_ops.init function registered, I doubt any usage of the cpuidle_ops.suspend is reasonable if it was not initialized by a corresponding cpuidle_ops.init at boot, at least that's how I see it working, I am open to other point of views. Thanks, Lorenzo
Powered by blists - more mailing lists