Change it to inline function to make callers use the proper argument.
And no need for it to be macro per Andrew's comment [1].

[1] 
https://lore.kernel.org/lkml/20200518221235.1fa32c38e5766113f78e3...@linux-foundation.org/

Signed-off-by: Guoqing Jiang <guoqing.ji...@cloud.ionos.com>
---
 include/linux/mm_types.h | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/include/linux/mm_types.h b/include/linux/mm_types.h
index e88dc5c65c01..64ede5f150dc 100644
--- a/include/linux/mm_types.h
+++ b/include/linux/mm_types.h
@@ -240,7 +240,11 @@ static inline atomic_t *compound_pincount_ptr(struct page 
*page)
 #define PAGE_FRAG_CACHE_MAX_ORDER      get_order(PAGE_FRAG_CACHE_MAX_SIZE)
 
 #define page_private(page)             ((page)->private)
-#define set_page_private(page, v)      ((page)->private = (v))
+
+static inline void set_page_private(struct page *page, unsigned long private)
+{
+       page->private = private;
+}
 
 struct page_frag_cache {
        void * va;
-- 
2.17.1

Reply via email to