]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
mm/mmap: make vma_wants_writenotify return bool
authorHao Ge <gehao@kylinos.cn>
Sun, 7 Apr 2024 06:26:53 +0000 (14:26 +0800)
committerAndrew Morton <akpm@linux-foundation.org>
Mon, 6 May 2024 00:53:26 +0000 (17:53 -0700)
vma_wants_writenotify() should return bool, so change it.

Link: https://lkml.kernel.org/r/20240407062653.803142-1-gehao@kylinos.cn
Signed-off-by: Hao Ge <gehao@kylinos.cn>
Cc: Liam R. Howlett <Liam.Howlett@oracle.com>
Cc: Lorenzo Stoakes <lstoakes@gmail.com>
Cc: Matthew Wilcox (Oracle) <willy@infradead.org>
Cc: Vlastimil Babka <vbabka@suse.cz>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
include/linux/mm.h
mm/mmap.c

index cc9410d772ddf0c29b56871c974fc686d78e5e36..0b933f9f486438db87600ebfb84be9b953131e51 100644 (file)
@@ -2575,7 +2575,7 @@ extern unsigned long move_page_tables(struct vm_area_struct *vma,
                                            MM_CP_UFFD_WP_RESOLVE)
 
 bool vma_needs_dirty_tracking(struct vm_area_struct *vma);
-int vma_wants_writenotify(struct vm_area_struct *vma, pgprot_t vm_page_prot);
+bool vma_wants_writenotify(struct vm_area_struct *vma, pgprot_t vm_page_prot);
 static inline bool vma_wants_manual_pte_write_upgrade(struct vm_area_struct *vma)
 {
        /*
index d3c2ca8efa534c4d6b50f9dff50dd7e254005f34..c9367454e19f9b5f6096903c8c7640d480504850 100644 (file)
--- a/mm/mmap.c
+++ b/mm/mmap.c
@@ -1514,32 +1514,32 @@ bool vma_needs_dirty_tracking(struct vm_area_struct *vma)
  * to the private version (using protection_map[] without the
  * VM_SHARED bit).
  */
-int vma_wants_writenotify(struct vm_area_struct *vma, pgprot_t vm_page_prot)
+bool vma_wants_writenotify(struct vm_area_struct *vma, pgprot_t vm_page_prot)
 {
        /* If it was private or non-writable, the write bit is already clear */
        if (!vma_is_shared_writable(vma))
-               return 0;
+               return false;
 
        /* The backer wishes to know when pages are first written to? */
        if (vm_ops_needs_writenotify(vma->vm_ops))
-               return 1;
+               return true;
 
        /* The open routine did something to the protections that pgprot_modify
         * won't preserve? */
        if (pgprot_val(vm_page_prot) !=
            pgprot_val(vm_pgprot_modify(vm_page_prot, vma->vm_flags)))
-               return 0;
+               return false;
 
        /*
         * Do we need to track softdirty? hugetlb does not support softdirty
         * tracking yet.
         */
        if (vma_soft_dirty_enabled(vma) && !is_vm_hugetlb_page(vma))
-               return 1;
+               return true;
 
        /* Do we need write faults for uffd-wp tracking? */
        if (userfaultfd_wp(vma))
-               return 1;
+               return true;
 
        /* Can the mapping track the dirty pages? */
        return vma_fs_can_writeback(vma);