From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx119.postini.com [74.125.245.119]) by kanga.kvack.org (Postfix) with SMTP id 6517B6B0044 for ; Fri, 23 Mar 2012 02:54:55 -0400 (EDT) Message-ID: <4F6C1C9B.8010806@snapgear.com> Date: Fri, 23 Mar 2012 16:47:55 +1000 From: Greg Ungerer MIME-Version: 1.0 Subject: Re: [PATCH v2 04/16] mm/nommu: use vm_flags_t for vma flags References: <20120321065629.13852.5630.stgit@zurg> <20120321115632.4571.77859.stgit@zurg> In-Reply-To: <20120321115632.4571.77859.stgit@zurg> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Konstantin Khlebnikov Cc: Andrew Morton , David Howells , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Greg Ungerer Hi Konstantin, On 21/03/12 22:01, Konstantin Khlebnikov wrote: > v2: fix compilation > > Signed-off-by: Konstantin Khlebnikov > Cc: David Howells > Cc: Greg Ungerer > > --- > > This time, I honestly checked the compiling for arm/at91x40_defconfig > last linux-next is broken, but my parts are ok. This looks ok to me. Acked-by: Greg Ungerer Regards Greg > --- > fs/proc/nommu.c | 14 ++++++++------ > fs/proc/task_nommu.c | 14 ++++++++------ > mm/nommu.c | 19 ++++++++++--------- > 3 files changed, 26 insertions(+), 21 deletions(-) > > diff --git a/fs/proc/nommu.c b/fs/proc/nommu.c > index b1822dd..6046ddb 100644 > --- a/fs/proc/nommu.c > +++ b/fs/proc/nommu.c > @@ -39,9 +39,10 @@ static int nommu_region_show(struct seq_file *m, struct vm_region *region) > unsigned long ino = 0; > struct file *file; > dev_t dev = 0; > - int flags, len; > + int len; > + vm_flags_t vm_flags; > > - flags = region->vm_flags; > + vm_flags = region->vm_flags; > file = region->vm_file; > > if (file) { > @@ -54,10 +55,11 @@ static int nommu_region_show(struct seq_file *m, struct vm_region *region) > "%08lx-%08lx %c%c%c%c %08llx %02x:%02x %lu %n", > region->vm_start, > region->vm_end, > - flags& VM_READ ? 'r' : '-', > - flags& VM_WRITE ? 'w' : '-', > - flags& VM_EXEC ? 'x' : '-', > - flags& VM_MAYSHARE ? flags& VM_SHARED ? 'S' : 's' : 'p', > + vm_flags& VM_READ ? 'r' : '-', > + vm_flags& VM_WRITE ? 'w' : '-', > + vm_flags& VM_EXEC ? 'x' : '-', > + vm_flags& VM_MAYSHARE ? > + vm_flags& VM_SHARED ? 'S' : 's' : 'p', > ((loff_t)region->vm_pgoff)<< PAGE_SHIFT, > MAJOR(dev), MINOR(dev), ino,&len); > > diff --git a/fs/proc/task_nommu.c b/fs/proc/task_nommu.c > index 74fe164..9447caa 100644 > --- a/fs/proc/task_nommu.c > +++ b/fs/proc/task_nommu.c > @@ -142,10 +142,11 @@ static int nommu_vma_show(struct seq_file *m, struct vm_area_struct *vma, > unsigned long ino = 0; > struct file *file; > dev_t dev = 0; > - int flags, len; > + int len; > + vm_flags_t vm_flags; > unsigned long long pgoff = 0; > > - flags = vma->vm_flags; > + vm_flags = vma->vm_flags; > file = vma->vm_file; > > if (file) { > @@ -159,10 +160,11 @@ static int nommu_vma_show(struct seq_file *m, struct vm_area_struct *vma, > "%08lx-%08lx %c%c%c%c %08llx %02x:%02x %lu %n", > vma->vm_start, > vma->vm_end, > - flags& VM_READ ? 'r' : '-', > - flags& VM_WRITE ? 'w' : '-', > - flags& VM_EXEC ? 'x' : '-', > - flags& VM_MAYSHARE ? flags& VM_SHARED ? 'S' : 's' : 'p', > + vm_flags& VM_READ ? 'r' : '-', > + vm_flags& VM_WRITE ? 'w' : '-', > + vm_flags& VM_EXEC ? 'x' : '-', > + vm_flags& VM_MAYSHARE ? > + vm_flags& VM_SHARED ? 'S' : 's' : 'p', > pgoff, > MAJOR(dev), MINOR(dev), ino,&len); > > diff --git a/mm/nommu.c b/mm/nommu.c > index f59e170..33d0ab7 100644 > --- a/mm/nommu.c > +++ b/mm/nommu.c > @@ -130,7 +130,7 @@ int __get_user_pages(struct task_struct *tsk, struct mm_struct *mm, > int *retry) > { > struct vm_area_struct *vma; > - unsigned long vm_flags; > + vm_flags_t vm_flags; > int i; > > /* calculate required read or write permissions. > @@ -658,13 +658,13 @@ static void put_nommu_region(struct vm_region *region) > /* > * update protection on a vma > */ > -static void protect_vma(struct vm_area_struct *vma, unsigned long flags) > +static void protect_vma(struct vm_area_struct *vma, vm_flags_t vm_flags) > { > #ifdef CONFIG_MPU > struct mm_struct *mm = vma->vm_mm; > long start = vma->vm_start& PAGE_MASK; > while (start< vma->vm_end) { > - protect_page(mm, start, flags); > + protect_page(mm, start, vm_flags); > start += PAGE_SIZE; > } > update_protections(mm); > @@ -1060,12 +1060,12 @@ static int validate_mmap_request(struct file *file, > * we've determined that we can make the mapping, now translate what we > * now know into VMA flags > */ > -static unsigned long determine_vm_flags(struct file *file, > - unsigned long prot, > - unsigned long flags, > - unsigned long capabilities) > +static vm_flags_t determine_vm_flags(struct file *file, > + unsigned long prot, > + unsigned long flags, > + unsigned long capabilities) > { > - unsigned long vm_flags; > + vm_flags_t vm_flags; > > vm_flags = calc_vm_prot_bits(prot) | calc_vm_flag_bits(flags); > /* vm_flags |= mm->def_flags; */ > @@ -1243,7 +1243,8 @@ unsigned long do_mmap_pgoff(struct file *file, > struct vm_area_struct *vma; > struct vm_region *region; > struct rb_node *rb; > - unsigned long capabilities, vm_flags, result; > + unsigned long capabilities, result; > + vm_flags_t vm_flags; > int ret; > > kenter(",%lx,%lx,%lx,%lx,%lx", addr, len, prot, flags, pgoff); > > > > -- ------------------------------------------------------------------------ Greg Ungerer -- Principal Engineer EMAIL: gerg@snapgear.com SnapGear Group, McAfee PHONE: +61 7 3435 2888 8 Gardner Close FAX: +61 7 3217 5323 Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org