[U-Boot] [PATCH v2 2/2] arm: Make arch specific memcpy thumb-safe.
Christoph Müllner
christoph.muellner at theobroma-systems.com
Sat Apr 21 16:45:41 UTC 2018
> On 21 Apr 2018, at 15:24, Måns Rullgård <mans at mansr.com> wrote:
>
> Klaus Goger <klaus.goger at theobroma-systems.com> writes:
>
>> The current arch implementation of memcpy cannot be called
>> from thumb code, because it does not use bx instructions on return.
>> This patch addresses that. Note, that this patch does not touch
>> the hot loop of memcpy, so performance is not affected.
>>
>> Tested on MXS (arm926ejs) with and without thumb-mode enabled.
>>
>> Signed-off-by: Klaus Goger <klaus.goger at theobroma-systems.com>
>> Signed-off-by: Christoph Muellner <christoph.muellner at theobroma-systems.com>
>
> There are many more instances of mov to pc that ought to be fixed too.
> Why not do them all at once rather than picking them off one by one as
> they happen to break things?
I could not find exit points within memcpy other than those which we fixed.
The many other mov pc, $lr instructions are just branches within memcpy.
Am I overseeing anything here?
Thanks,
Christoph
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 874 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180421/b00e3e48/attachment.sig>
More information about the U-Boot
mailing list