Re: [PATCH] unicore32: mm: Add missing parameter to arch_vma_access_permitted

[Xuetao Guan]

> ping ... still not upstream.
I'm sorry.
I'll do it in next week.

Xuetao

>
> On 03/21/2016 04:20 AM, Guenter Roeck wrote:
>> unicore32 fails to compile with the following errors.
>>
>> mm/memory.c: In function ??_handle_mm_fault?�:
>> mm/memory.c:3381: error:
>> 	too many arguments to function ??rch_vma_access_permitted?�
>> mm/gup.c: In function ??heck_vma_flags?�:
>> mm/gup.c:456: error:
>> 	too many arguments to function ??rch_vma_access_permitted?�
>> mm/gup.c: In function ??ma_permits_fault?�:
>> mm/gup.c:640: error:
>> 	too many arguments to function ??rch_vma_access_permitted?�
>>
>> Fixes: d61172b4b695b ("mm/core, x86/mm/pkeys: Differentiate instruction
>> fetches")
>> Cc: Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>
>> Cc: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
>> Cc: Ingo Molnar <mingo@xxxxxxxxxx>
>> Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
>> ---
>>   arch/unicore32/include/asm/mmu_context.h | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/unicore32/include/asm/mmu_context.h
>> b/arch/unicore32/include/asm/mmu_context.h
>> index e35632ef23c7..62dfc644c908 100644
>> --- a/arch/unicore32/include/asm/mmu_context.h
>> +++ b/arch/unicore32/include/asm/mmu_context.h
>> @@ -98,7 +98,7 @@ static inline void arch_bprm_mm_init(struct mm_struct
>> *mm,
>>   }
>>
>>   static inline bool arch_vma_access_permitted(struct vm_area_struct
>> *vma,
>> -		bool write, bool foreign)
>> +		bool write, bool execute, bool foreign)
>>   {
>>   	/* by default, allow everything */
>>   	return true;
>>
>