On Thu, Sep 7, 2017 at 12:23 AM, Ard Biesheuvel
<[email protected]> wrote:
> On 6 September 2017 at 21:57, Arnd Bergmann <[email protected]> wrote:
>> On Mon, Sep 4, 2017 at 6:19 PM, Romain Izard <[email protected]>
>> wrote:
>
> HAVE_EFFICIENT_UNALIGNED_ACCESS only affects explicit unaligned
> accesses, and selects between fixups in hardware or in software.
> AFAICT the issue here is implicit unaligned accesses, where char
> pointers are passed as u32 * arguments.
The problem with include/linux/unaligned/access_ok.h is that it
converts pointers
that are known by the caller to be potentially unaligned and accesses them as if
they were aligned. This means we require a software fixup through the
trap handler
on ARM in cases that the compiler already knows how to handle correctly when
using linux/unaligned/le_struct.h. On ARMv7 this means it ends up using normal
load/store instructures but not the ldm/stm or ldrd/stdr instructions
that are not
allowed on unaligned pointers.
Doing that solves the problem that Romain ran into and also makes other
code much more efficient on ARMv7.
Arnd
