mm/zsmalloc: get rid of PAGE_MASK
authorAlexey Romanov <avromanov@sberdevices.ru>
Tue, 16 May 2023 09:50:29 +0000 (12:50 +0300)
committerAndrew Morton <akpm@linux-foundation.org>
Fri, 9 Jun 2023 23:25:21 +0000 (16:25 -0700)
Use offset_in_page() macro instead of 'val & ~PAGE_MASK'

Link: https://lkml.kernel.org/r/20230516095029.49036-2-avromanov@sberdevices.ru
Signed-off-by: Alexey Romanov <avromanov@sberdevices.ru>
Reviewed-by: Sergey Senozhatsky <senozhatsky@chromium.org>
Cc: Minchan Kim <minchan@kernel.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
mm/zsmalloc.c

index 02f7f41..c0d4335 100644 (file)
@@ -1341,7 +1341,7 @@ void *zs_map_object(struct zs_pool *pool, unsigned long handle,
        spin_unlock(&pool->lock);
 
        class = zspage_class(pool, zspage);
-       off = (class->size * obj_idx) & ~PAGE_MASK;
+       off = offset_in_page(class->size * obj_idx);
 
        local_lock(&zs_map_area.lock);
        area = this_cpu_ptr(&zs_map_area);
@@ -1381,7 +1381,7 @@ void zs_unmap_object(struct zs_pool *pool, unsigned long handle)
        obj_to_location(obj, &page, &obj_idx);
        zspage = get_zspage(page);
        class = zspage_class(pool, zspage);
-       off = (class->size * obj_idx) & ~PAGE_MASK;
+       off = offset_in_page(class->size * obj_idx);
 
        area = this_cpu_ptr(&zs_map_area);
        if (off + class->size <= PAGE_SIZE)
@@ -1438,7 +1438,7 @@ static unsigned long obj_malloc(struct zs_pool *pool,
 
        offset = obj * class->size;
        nr_page = offset >> PAGE_SHIFT;
-       m_offset = offset & ~PAGE_MASK;
+       m_offset = offset_in_page(offset);
        m_page = get_first_page(zspage);
 
        for (i = 0; i < nr_page; i++)
@@ -1548,7 +1548,7 @@ static void obj_free(int class_size, unsigned long obj, unsigned long *handle)
        void *vaddr;
 
        obj_to_location(obj, &f_page, &f_objidx);
-       f_offset = (class_size * f_objidx) & ~PAGE_MASK;
+       f_offset = offset_in_page(class_size * f_objidx);
        zspage = get_zspage(f_page);
 
        vaddr = kmap_atomic(f_page);
@@ -1640,8 +1640,8 @@ static void zs_object_copy(struct size_class *class, unsigned long dst,
        obj_to_location(src, &s_page, &s_objidx);
        obj_to_location(dst, &d_page, &d_objidx);
 
-       s_off = (class->size * s_objidx) & ~PAGE_MASK;
-       d_off = (class->size * d_objidx) & ~PAGE_MASK;
+       s_off = offset_in_page(class->size * s_objidx);
+       d_off = offset_in_page(class->size * d_objidx);
 
        if (s_off + class->size > PAGE_SIZE)
                s_size = PAGE_SIZE - s_off;