[U-Boot] [PATCH] Make the generic unaligned access code safe for unaligned access

[Stefan Roese]

Hi Wolfgang,

On Monday 23 November 2009 23:46:38 Wolfgang Denk wrote:
> Dear Remy Bohmer,
> 
> In message <1256764421-27799-4-git-send-email-linux at bohmer.net> you wrote:
> > The current generic code for handling unaligned access assumes that
> > the processor can properly handle unaligned accesses itself.
> > This is at least not the case for ARM, which results in runtime
> > errors.
> >
> > Rewrite it such that it works for ARM as well.
> >
> > Signed-off-by: Remy Bohmer <linux at bohmer.net>
> > ---
> >  include/linux/unaligned/access_ok.h |   48
> > ++++++++++++++++++++++++---------- 1 files changed, 34 insertions(+), 14
> > deletions(-)
> 
> Applied, thanks.

This shouldn't have been applied. Remy did send a different version of this 
patch (according to my suggestion), which is already included:


commit 25793f76bf9a7be59c9415ef0f78d034e8d53dae
Author: Remy Bohmer <linux at bohmer.net>
Date:   Thu Oct 29 12:29:37 2009 +0100

    ARM: Use Linux version for unaligned access code

    The asm-arm/unaligned.h includes linux/unaligned/access_ok.h
    This file is unsafe to be used on ARM, since it does an unaligned memory
    accesses which fails on ARM.

    Lookin at Linux the basic difference seems to be the header
    "include/asm-arm/unaligned.h". The Linux version of "unaligned.h"
    does *not* include "access_ok.h" at all. It includes "le_byteshift.h"
    and "be_byteshift.h" instead.

    Signed-off-by: Remy Bohmer <linux at bohmer.net>
    Signed-off-by: Stefan Roese <sr at denx.de>



Remy, please correct me if I'm wrong here.

Cheers,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de