On 01.06.2015 20:02, Peter Maydell wrote:
> 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
Thanks, Peter. Your comments are very useful, as always :) Sorry for
forgetting sign-off-by. Will resend the patch soon!
Best,
Sergey
