Re: [PATCH -v6 1/2] Massive code cleanup of sys_msync()

2008-01-18 Thread Anton Salikhmetov
2008/1/18, Miklos Szeredi <[EMAIL PROTECTED]>: > > unsigned long end; > > - struct mm_struct *mm = current->mm; > > + int error, unmapped_error; > > struct vm_area_struct *vma; > > - int unmapped_error = 0; > > - int error = -EINVAL; > > + struct mm_struct *mm; > >

Re: [PATCH -v6 1/2] Massive code cleanup of sys_msync()

2008-01-18 Thread Miklos Szeredi
> unsigned long end; > - struct mm_struct *mm = current->mm; > + int error, unmapped_error; > struct vm_area_struct *vma; > - int unmapped_error = 0; > - int error = -EINVAL; > + struct mm_struct *mm; > > + error = -EINVAL; I think you may have misunderstood

Re: [PATCH -v6 1/2] Massive code cleanup of sys_msync()

2008-01-18 Thread Miklos Szeredi
unsigned long end; - struct mm_struct *mm = current-mm; + int error, unmapped_error; struct vm_area_struct *vma; - int unmapped_error = 0; - int error = -EINVAL; + struct mm_struct *mm; + error = -EINVAL; I think you may have misunderstood my last

Re: [PATCH -v6 1/2] Massive code cleanup of sys_msync()

2008-01-18 Thread Anton Salikhmetov
2008/1/18, Miklos Szeredi [EMAIL PROTECTED]: unsigned long end; - struct mm_struct *mm = current-mm; + int error, unmapped_error; struct vm_area_struct *vma; - int unmapped_error = 0; - int error = -EINVAL; + struct mm_struct *mm; + error =

[PATCH -v6 1/2] Massive code cleanup of sys_msync()

2008-01-17 Thread Anton Salikhmetov
Using the PAGE_ALIGN() macro instead of "manual" alignment and improved readability of the loop traversing the process memory regions. Signed-off-by: Anton Salikhmetov <[EMAIL PROTECTED]> --- mm/msync.c | 77 1 files changed, 36

[PATCH -v6 1/2] Massive code cleanup of sys_msync()

2008-01-17 Thread Anton Salikhmetov
Using the PAGE_ALIGN() macro instead of manual alignment and improved readability of the loop traversing the process memory regions. Signed-off-by: Anton Salikhmetov [EMAIL PROTECTED] --- mm/msync.c | 77 1 files changed, 36