From: Todd Poynor <toddpoy...@google.com>

gasket_mmap use PAGE_MASK, instead of performing math on PAGE_SIZE, for
simplicity and clarity.

Reported-by: Guenter Roeck <gro...@chromium.org>
Signed-off-by: Simon Que <s...@chromium.org>
Signed-off-by: Todd Poynor <toddpoy...@google.com>
---
 drivers/staging/gasket/gasket_core.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/gasket/gasket_core.c 
b/drivers/staging/gasket/gasket_core.c
index 9a821614b0334..98d5a200aaa5a 100644
--- a/drivers/staging/gasket/gasket_core.c
+++ b/drivers/staging/gasket/gasket_core.c
@@ -1594,7 +1594,7 @@ static int gasket_mmap(struct file *filp, struct 
vm_area_struct *vma)
        }
        driver_desc = gasket_dev->internal_desc->driver_desc;
 
-       if (vma->vm_start & (PAGE_SIZE - 1)) {
+       if (vma->vm_start & ~PAGE_MASK) {
                gasket_log_error(
                        gasket_dev, "Base address not page-aligned: 0x%p\n",
                        (void *)vma->vm_start);
-- 
2.18.0.203.gfac676dfb9-goog

Reply via email to