zlogger: Fix int-conversion error with gcc-14 75/312275/1 accepted/tizen/unified/20240611.123615 accepted/tizen/unified/dev/20240620.010552 accepted/tizen/unified/toolchain/20240610.173225 accepted/tizen/unified/x/20240610.223759 accepted/tizen/unified/x/asan/20240625.092051
authorJaehoon Chung <jh80.chung@samsung.com>
Wed, 5 Jun 2024 11:18:57 +0000 (20:18 +0900)
committerJaehoon Chung <jh80.chung@samsung.com>
Wed, 5 Jun 2024 11:28:43 +0000 (20:28 +0900)
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 <jh80.chung@samsung.com>
kernel/zlogger/zlogger.c

index d78498bbb39a546f5d967e77fc87bb9856dc2709..f5073a1ad69b951a4faa956d267b594246118dc9 100644 (file)
@@ -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;