On 28 May 2015 at 12:00, Sergey Fedorov <serge.f...@gmail.com> wrote:
> ---
> target-arm/helper.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/target-arm/helper.c b/target-arm/helper.c
> index 5d0f011..fc3f192 100644
> --- a/target-arm/helper.c
> +++ b/target-arm/helper.c
> @@ -5333,6 +5333,8 @@ static int get_phys_addr_v6(CPUARMState *env, uint32_t
> address, int access_type,
> if (desc & (1 << 18)) {
> /* Supersection. */
> phys_addr = (desc & 0xff000000) | (address & 0x00ffffff);
> + phys_addr |= ((uint64_t)extract32(desc, 20, 4) << 32);
> + phys_addr |= ((uint64_t)extract32(desc, 5, 4) << 36);
> *page_size = 0x1000000;
> } else {
> /* Section. */
This looks OK codewise, but I can't apply it because you've
forgotten the signed-off-by line.
Can you also provide a useful commit message, please?
Patches should pretty much always have a long-form
commit message, except possibly when they're part of
a large patch series, and rarely even then. (For instance
in this case it's worth mentioning that supporting these
extra bits is IMPDEF in v6 and in v7-without-LPAE but
required in LPAE.)
thanks
-- PMM
