From: Jaehoon Chung Date: Wed, 5 Jun 2024 11:18:57 +0000 (+0900) Subject: zlogger: Fix int-conversion error with gcc-14 X-Git-Tag: accepted/tizen/unified/toolchain/20240610.173225^0 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F75%2F312275%2F1;p=platform%2Fkernel%2Flinux-tizen-modules-source.git 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 --- 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;