diff options
author | Alexander Kuleshov <kuleshovmail@gmail.com> | 2015-11-05 21:46:35 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-11-05 22:34:48 -0500 |
commit | 1824cb753354e026ab898cd472bddd540b50b00b (patch) | |
tree | 391aa46b768e22a3987adae91b7a4104291d34d2 | |
parent | b0d61c7e56815b0b881c81f6779a65f4fdae4bc0 (diff) |
mm/nommu: use offset_in_page macro
linux/mm.h provides offset_in_page() macro. Let's use already predefined
macro instead of (addr & ~PAGE_MASK).
Signed-off-by: Alexander Kuleshov <kuleshovmail@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | mm/nommu.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/mm/nommu.c b/mm/nommu.c index ab14a2014dea..1e0f1688d9a4 100644 --- a/mm/nommu.c +++ b/mm/nommu.c | |||
@@ -1497,7 +1497,7 @@ SYSCALL_DEFINE1(old_mmap, struct mmap_arg_struct __user *, arg) | |||
1497 | 1497 | ||
1498 | if (copy_from_user(&a, arg, sizeof(a))) | 1498 | if (copy_from_user(&a, arg, sizeof(a))) |
1499 | return -EFAULT; | 1499 | return -EFAULT; |
1500 | if (a.offset & ~PAGE_MASK) | 1500 | if (offset_in_page(a.offset)) |
1501 | return -EINVAL; | 1501 | return -EINVAL; |
1502 | 1502 | ||
1503 | return sys_mmap_pgoff(a.addr, a.len, a.prot, a.flags, a.fd, | 1503 | return sys_mmap_pgoff(a.addr, a.len, a.prot, a.flags, a.fd, |
@@ -1653,9 +1653,9 @@ int do_munmap(struct mm_struct *mm, unsigned long start, size_t len) | |||
1653 | goto erase_whole_vma; | 1653 | goto erase_whole_vma; |
1654 | if (start < vma->vm_start || end > vma->vm_end) | 1654 | if (start < vma->vm_start || end > vma->vm_end) |
1655 | return -EINVAL; | 1655 | return -EINVAL; |
1656 | if (start & ~PAGE_MASK) | 1656 | if (offset_in_page(start)) |
1657 | return -EINVAL; | 1657 | return -EINVAL; |
1658 | if (end != vma->vm_end && end & ~PAGE_MASK) | 1658 | if (end != vma->vm_end && offset_in_page(end)) |
1659 | return -EINVAL; | 1659 | return -EINVAL; |
1660 | if (start != vma->vm_start && end != vma->vm_end) { | 1660 | if (start != vma->vm_start && end != vma->vm_end) { |
1661 | ret = split_vma(mm, vma, start, 1); | 1661 | ret = split_vma(mm, vma, start, 1); |
@@ -1736,7 +1736,7 @@ static unsigned long do_mremap(unsigned long addr, | |||
1736 | if (old_len == 0 || new_len == 0) | 1736 | if (old_len == 0 || new_len == 0) |
1737 | return (unsigned long) -EINVAL; | 1737 | return (unsigned long) -EINVAL; |
1738 | 1738 | ||
1739 | if (addr & ~PAGE_MASK) | 1739 | if (offset_in_page(addr)) |
1740 | return -EINVAL; | 1740 | return -EINVAL; |
1741 | 1741 | ||
1742 | if (flags & MREMAP_FIXED && new_addr != addr) | 1742 | if (flags & MREMAP_FIXED && new_addr != addr) |