[RFC PATCH] ARM: new architecture for Energy Micro's EFM32 Cortex-M3 SoCs
Russell King - ARM Linux
linux at arm.linux.org.uk
Mon Jan 16 14:06:37 EST 2012
On Mon, Jan 16, 2012 at 06:10:02PM +0000, Catalin Marinas wrote:
> On Mon, Jan 16, 2012 at 05:40:39PM +0000, Russell King - ARM Linux wrote:
> > The VFP stuff - adding 'clean' which is kernel state to the _user_
> > _exported_ VFP hardware structure is a bad idea. So this needlessly
> > causes a variation in the kernels userspace API. Please find somewhere
> > else to keep kernel internal state. (As that patch comes from Catalin,
> > then that comment is directed to Catalin.)
>
> Are you sure we export vfp_hard_struct to user? That's a kernel-only
> structure (and it's not by any means stable, given the number of
> #ifdef's it has). I would also argue that 'clean' is a hardware state
> (inferred from the exception return value).
Actually, looking at arch/arm/kernel/ptrace.c, we only export the
fpregs and fpscr, so this should be fine.
Still, I don't see why we need this 'clean' state, when normal VFP
doesn't need it. Maybe you could explain why it's necessary?
More information about the linux-arm-kernel
mailing list