[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] target-arm: crypto: fix BE host support
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] target-arm: crypto: fix BE host support |
Date: |
Mon, 5 Jan 2015 13:58:56 +0000 |
On 5 January 2015 at 13:47, Ard Biesheuvel <address@hidden> wrote:
> On 5 January 2015 at 12:34, Peter Maydell <address@hidden> wrote:
>> On 2 January 2015 at 19:21, Ard Biesheuvel <address@hidden> wrote:
>>> I think the union type may have been a mistake to
>>> begin with, because it introduces endianness dependencies that don't
>>> actually exist in the code. It probably would have been cleaner if I
>>> had defined the relation between VFP D-regs, words and bytes in terms
>>> of shifts and masks instead.
>>
>> Mmm, I think in retrospect this is right. Still, this patch does
>> cause the kernel's self-tests on boot on a BE ppc64 host to pass,
>> and looking through the code we haven't missed any accesses to
>> .bytes or .words, so it looks good to me.
>>
>> Reviewed-by: Peter Maydell <address@hidden>
>>
>
> OK, good, thanks for testing.
>
> I propose we leave the patch as-is then. Are you ok to add the tags
> and merge it?
Yep; I've put it into target-arm.next.
-- PMM