Re: [PATCH 1/4] msm: scm: Mark inline asm as volatile
From: David Brown
Date: Sun Feb 27 2011 - 12:38:53 EST
On Sun, Feb 27 2011, Will Deacon wrote:
> On Sat, 2011-02-26 at 18:12 +0000, David Brown wrote:
>> On Fri, Feb 25 2011, Will Deacon wrote:
>> > These asm blocks all have sensible looking output constraints. Why
>> > do they need to be marked volatile?
>> Without the volatile, the compiler is free to assume the only side
>> effects of the asm are to modify the output registers. The volatile is
>> needed to indicate to the compiler that the asm has other side effects.
> As far as I know, volatile asm does two things:
> (1) It stops the compiler from reordering the asm block with respect to
> other volatile statements.
> (2) It prevents the compiler from optimising the block away when
> dataflow analysis indicates it's not required.
> If side-effects need to be indicated, won't a memory clobber do the
Per the gcc manual:
If your assembler instructions access memory in an unpredictable
fashion, add `memory' to the list of clobbered registers. This will
cause GCC to not keep memory values cached in registers across the
assembler instruction and not optimize stores or loads to that
memory. You will also want to add the `volatile' keyword if the
memory affected is not listed in the inputs or outputs of the `asm',
as the `memory' clobber does not count as a side-effect of the `asm'.
If you know how large the accessed memory is, you can add it as input
or output but if this is not known, you should add `memory'. As an
example, if you access ten bytes of a string, you can use a memory
The smc instruction is similar to a syscall. When in the secure world,
the processor is making state changes. It's not quite correct to
declare this as memory, because the memory used when secure isn't even
accessible to us. As far as I can tell, the volatile is the only way to
tell the compiler this.
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/