• K
    arm64: sysreg: Make mrs_s and msr_s macros work with Clang and LTO · be604c61
    Kees Cook 提交于
    Clang's integrated assembler does not allow assembly macros defined
    in one inline asm block using the .macro directive to be used across
    separate asm blocks. LLVM developers consider this a feature and not a
    bug, recommending code refactoring:
    
      https://bugs.llvm.org/show_bug.cgi?id=19749
    
    As binutils doesn't allow macros to be redefined, this change uses
    UNDEFINE_MRS_S and UNDEFINE_MSR_S to define corresponding macros
    in-place and workaround gcc and clang limitations on redefining macros
    across different assembler blocks.
    
    Specifically, the current state after preprocessing looks like this:
    
    asm volatile(".macro mXX_s ... .endm");
    void f()
    {
    	asm volatile("mXX_s a, b");
    }
    
    With GCC, it gives macro redefinition error because sysreg.h is included
    in multiple source files, and assembler code for all of them is later
    combined for LTO (I've seen an intermediate file with hundreds of
    identical definitions).
    
    With clang, it gives macro undefined error because clang doesn't allow
    sharing macros between inline asm statements.
    
    I also seem to remember catching another sort of undefined error with
    GCC due to reordering of macro definition asm statement and generated
    asm code for function that uses the macro.
    
    The solution with defining and undefining for each use, while certainly
    not elegant, satisfies both GCC and clang, LTO and non-LTO.
    Co-developed-by: NAlex Matveev <alxmtvv@gmail.com>
    Co-developed-by: NYury Norov <ynorov@caviumnetworks.com>
    Co-developed-by: NSami Tolvanen <samitolvanen@google.com>
    Reviewed-by: NNick Desaulniers <ndesaulniers@google.com>
    Reviewed-by: NMark Rutland <mark.rutland@arm.com>
    Signed-off-by: NKees Cook <keescook@chromium.org>
    Signed-off-by: NWill Deacon <will.deacon@arm.com>
    be604c61
kvm_hyp.h 6.4 KB