[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 3/9] target-arm: Add support for S2 page-tabl
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH v3 3/9] target-arm: Add support for S2 page-table protection bits |
Date: |
Wed, 14 Oct 2015 20:38:54 +0100 |
On 14 October 2015 at 13:45, Alex Bennée <address@hidden> wrote:
> While looking I noticed we probably extract more than we need into attrs
> considering the tableattrs is snarfed earlier. Maybe we should only do:
>
> attrs = extract64(descriptor, 2, 10)
> | (extract64(descriptor, 52, 7) << 10);
>
> Or even:
>
> attrs = extract64(descriptor, 2, 10)
> | (extract64(descriptor, 52, 3) << 10);
It seemed neater to grab all of the attribute bits, even the
reserved ones. Otherwise we'll end up needing to change the
code back later if a new attribute we care about is added.
thanks
-- PMM
[Qemu-devel] [PATCH v3 7/9] target-arm: Add S2 translation to 32bit S1 PTWs, Edgar E. Iglesias, 2015/10/08
[Qemu-devel] [PATCH v3 8/9] target-arm: Route S2 MMU faults to EL2, Edgar E. Iglesias, 2015/10/08
[Qemu-devel] [PATCH v3 1/9] target-arm: Add HPFAR_EL2, Edgar E. Iglesias, 2015/10/08
Re: [Qemu-devel] [PATCH v3 1/9] target-arm: Add HPFAR_EL2, Alex Bennée, 2015/10/08
Re: [Qemu-devel] [PATCH v3 1/9] target-arm: Add HPFAR_EL2, Laurent Desnogues, 2015/10/09