[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