Re: Modernize arm assembly in the kernel

2016-09-21 Thread Mark Kettenis
> Date: Wed, 21 Sep 2016 11:00:49 +0200 > From: Joerg Sonnenberger > > On Wed, Sep 21, 2016 at 10:10:43AM +0200, Mark Kettenis wrote: > > 1. In GNU as, .align 0 is equivalent to .align 2, but with clang's > >internal assembler .align 0 means "no alignment". > > It might be even better to use

Re: Modernize arm assembly in the kernel

2016-09-21 Thread Joerg Sonnenberger
On Wed, Sep 21, 2016 at 10:10:43AM +0200, Mark Kettenis wrote: > 1. In GNU as, .align 0 is equivalent to .align 2, but with clang's >internal assembler .align 0 means "no alignment". It might be even better to use .balign or .p2align. > 2. Using "ldr" to load a constant into a register is str

Modernize arm assembly in the kernel

2016-09-21 Thread Mark Kettenis
This fixes two isses. 1. In GNU as, .align 0 is equivalent to .align 2, but with clang's internal assembler .align 0 means "no alignment". 2. Using "ldr" to load a constant into a register is strange. It works with GNU as, but not with clang. No binary change in a kernel compiled with gcc