[RFC PATCH 0/2] ARM: assembler: Add uniform assembler framework

Dave Martin dave.martin at linaro.org
Thu Mar 1 08:20:41 EST 2012


On Thu, Mar 01, 2012 at 01:00:51PM +0000, Russell King - ARM Linux wrote:
> On Thu, Mar 01, 2012 at 11:49:42AM +0000, Dave Martin wrote:
> > Are you still interested in this series?
> > 
> > I have a workaround for the above now, but it's not pretty -- I seem to
> > have opened a can of worms here.
> > 
> > 
> > We would need something like
> > 
> > #define EXTENDED_ASM(stuff) asm ( ".include \"macros.inc\"\n\t" stuff )
> > #define EXTENDED_VASM(stuff) asm volatile ( ".include \"macros.inc\"\n\t" stuff )
> > 
> > ...where macros.inc includes some logic to protect against multiple
> > inclusion.
> > 
> > This would avoid problems caused by reordering done by the compiler.
> > 
> > I think this can work, but it may be a step too far...
> 
> I'm fairly tempted to say: "if it ain't broke, don't fix".  AFAIK no one
> has reported problems here, so lets not hastily try to fix something that
> will error out if it does go wrong.

OK -- I'll shelve this series for now and we'll see if we can come up
with something more localised for the kvm/Xen case.

unified.h may work for now, but I don't want to add anything more along
those lines for unless it's definitely correct to do so...

Cheers
---Dave



More information about the linux-arm-kernel mailing list