[PATCH 5/6] ARM: kvm: Use phys_addr_t instead of unsigned long in mm code
Santosh Shilimkar
santosh.shilimkar at ti.com
Thu Nov 14 19:10:56 EST 2013
On Thursday 14 November 2013 07:09 PM, Christoffer Dall wrote:
> On Thu, Nov 14, 2013 at 02:37:45PM -0500, Santosh Shilimkar wrote:
>> The unsigned long datatype is not sufficient for mapping physical addresses
>> greater than 4GB.
>>
>> So fix the KVM mm code accordingly. Special thanks to Christopher for debug
>> help to figure out the bug.
>>
>> Cc: Christoffer Dall <christoffer.dall at linaro.org>
>> Cc: Marc Zyngier <marc.zyngier at arm.com>
>>
>> Signed-off-by: Santosh Shilimkar <santosh.shilimkar at ti.com>
>> ---
>> arch/arm/kvm/mmu.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm/kvm/mmu.c b/arch/arm/kvm/mmu.c
>> index 3bd652f..657f15e 100644
>> --- a/arch/arm/kvm/mmu.c
>> +++ b/arch/arm/kvm/mmu.c
>> @@ -318,7 +318,7 @@ out:
>> */
>> int create_hyp_mappings(void *from, void *to)
>> {
>> - unsigned long phys_addr = virt_to_phys(from);
>> + phys_addr_t phys_addr = virt_to_phys(from);
>> unsigned long start = KERN_TO_HYP((unsigned long)from);
>> unsigned long end = KERN_TO_HYP((unsigned long)to);
>>
>> --
>> 1.7.9.5
>>
>
> Acked-by: Christoffer Dall <christoffer.dall at linaro.org>
>
Thanks
> I can just apply this to kvm-arm-next immediately, or we can wait and
> pull it in with your series if you prefer. Let me know.
>
Feel free to pick this up since this one can go as bug
fix as well.
Regards,
Santosh
More information about the linux-arm-kernel
mailing list