From e05ca2520d4a16ada3d1205b27a1f5774e141413 Mon Sep 17 00:00:00 2001 From: Jaehoon Chung Date: Wed, 5 Jun 2024 20:18:57 +0900 Subject: [PATCH] zlogger: Fix int-conversion error with gcc-14 It's failed because of type casting as unsigned long. p is already pointer type, so it doesn't need to cast. - It can already notice as warning with gcc13. [ 28s] /home/abuild/rpmbuild/BUILD/linux-tizen-modules-7.5.0/kernel/zlogger/zlogger.c:502:29: error: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion] [ 28s] /home/abuild/rpmbuild/BUILD/linux-tizen-modules-7.5.0/kernel/zlogger/zlogger.c:557:29: error: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion] [ 28s] /home/abuild/rpmbuild/BUILD/linux-tizen-modules-7.5.0/kernel/zlogger/zlogger.c:617:29: error: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion] [ 28s] /home/abuild/rpmbuild/BUILD/linux-tizen-modules-7.5.0/kernel/zlogger/zlogger.c:1038:37: error: passing argument 1 of 'virt_to_pfn' makes pointer from integer without a cast [-Wint-conversion] Change-Id: Id54c9cfada32d292998e4be2b88877511e4a5da3 Signed-off-by: Jaehoon Chung --- kernel/zlogger/zlogger.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/kernel/zlogger/zlogger.c b/kernel/zlogger/zlogger.c index d78498b..f5073a1 100644 --- a/kernel/zlogger/zlogger.c +++ b/kernel/zlogger/zlogger.c @@ -499,7 +499,7 @@ static vm_fault_t zlogger_fault(struct vm_fault *vmf) if (!p) goto unlock; - page = virt_to_page((unsigned long)p); + page = virt_to_page(p); ret = vmf_insert_pfn(vma, vma->vm_start, page_to_pfn(page)); unlock: @@ -554,7 +554,7 @@ static int zlogger_mmap(struct file *filp, struct vm_area_struct *vma) #endif vma->vm_ops = &zlogger_vm_ops; - page = virt_to_page((unsigned long)p); + page = virt_to_page(p); ret = remap_pfn_range(vma, vma->vm_start, page_to_pfn(page), ZLOGGER_BLOCK_SIZE, vma->vm_page_prot); unlock: @@ -614,7 +614,7 @@ static int zlogger_realloc_mmap(struct file *filp) goto unlock; } - page = virt_to_page((unsigned long)p); + page = virt_to_page(p); zap_vma_ptes(vma, vma->vm_start, vma->vm_end - vma->vm_start); ret = remap_pfn_range(vma, vma->vm_start, page_to_pfn(page), ZLOGGER_BLOCK_SIZE, vma->vm_page_prot); @@ -1035,7 +1035,7 @@ static int zlogger_dump_mmap(struct file *filep, struct vm_area_struct *vma) p = get_shared_memory(dev_index); if (p) - page = virt_to_page((unsigned long)p); + page = virt_to_page(p); else return -EINVAL; -- 2.34.1