From: "Liam R. Howlett" <Liam.Howlett@oracle.com>
To: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Cc: Andrew Morton <akpm@linux-foundation.org>,
Vlastimil Babka <vbabka@suse.cz>, Jann Horn <jannh@google.com>,
Pedro Falcato <pfalcato@suse.de>,
WangYuli <wangyuli@uniontech.com>,
linux-mm@kvack.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] tools/testing/vma: Clean up stubs in vma_internal.h
Date: Wed, 27 Aug 2025 20:55:09 -0400 [thread overview]
Message-ID: <tldxsryzvi46gpcnbn4zusudc3pd65hbgvnhybtkkdusyfxj6w@6qy6k4gwruuf> (raw)
In-Reply-To: <20250826102824.22730-1-lorenzo.stoakes@oracle.com>
* Lorenzo Stoakes <lorenzo.stoakes@oracle.com> [250826 06:28]:
> We do not need to references arguments just to avoid compiler warnings, the
> warning in question does not arise here, so remove all of the instances of
> '(void)xxx' introduced purely to avoid this warning.
>
> As reported by WagYuli in the referenced mail, GCC 8.3 and before will have
> issues compiling this file if parameter names are not provided, so ensure
> these are always provided.
>
> Finally, perform a trivial fix up of kmem_cache_alloc() which technically
> has parameters in the incorrect order (as reported by Vlastimil Babka
> off-list).
>
> Reported-by: WangYuli <wangyuli@uniontech.com>
> Closes: https://lore.kernel.org/linux-mm/EFCEBE7E301589DE+20250729084700.208767-1-wangyuli@uniontech.com/
> Reported-by: Vlastimil Babka <vbabka@suse.cz>
> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
Reviewed-by: Liam R. Howlett <Liam.Howlett@oracle.com>
> ---
> tools/testing/vma/vma_internal.h | 167 +++++++++++--------------------
> 1 file changed, 57 insertions(+), 110 deletions(-)
>
> diff --git a/tools/testing/vma/vma_internal.h b/tools/testing/vma/vma_internal.h
> index f13354bf0a1e..4b9394d75b34 100644
> --- a/tools/testing/vma/vma_internal.h
> +++ b/tools/testing/vma/vma_internal.h
> @@ -692,9 +692,7 @@ static inline struct kmem_cache *__kmem_cache_create(const char *name,
>
> static inline void *kmem_cache_alloc(struct kmem_cache *s, gfp_t gfpflags)
> {
> - (void)gfpflags;
> -
> - return calloc(s->object_size, 1);
> + return calloc(1, s->object_size);
> }
>
> static inline void kmem_cache_free(struct kmem_cache *s, void *x)
> @@ -858,11 +856,11 @@ static inline unsigned long vma_pages(struct vm_area_struct *vma)
> return (vma->vm_end - vma->vm_start) >> PAGE_SHIFT;
> }
>
> -static inline void fput(struct file *)
> +static inline void fput(struct file *file)
> {
> }
>
> -static inline void mpol_put(struct mempolicy *)
> +static inline void mpol_put(struct mempolicy *pol)
> {
> }
>
> @@ -870,15 +868,15 @@ static inline void lru_add_drain(void)
> {
> }
>
> -static inline void tlb_gather_mmu(struct mmu_gather *, struct mm_struct *)
> +static inline void tlb_gather_mmu(struct mmu_gather *tlb, struct mm_struct *mm)
> {
> }
>
> -static inline void update_hiwater_rss(struct mm_struct *)
> +static inline void update_hiwater_rss(struct mm_struct *mm)
> {
> }
>
> -static inline void update_hiwater_vm(struct mm_struct *)
> +static inline void update_hiwater_vm(struct mm_struct *mm)
> {
> }
>
> @@ -887,36 +885,23 @@ static inline void unmap_vmas(struct mmu_gather *tlb, struct ma_state *mas,
> unsigned long end_addr, unsigned long tree_end,
> bool mm_wr_locked)
> {
> - (void)tlb;
> - (void)mas;
> - (void)vma;
> - (void)start_addr;
> - (void)end_addr;
> - (void)tree_end;
> - (void)mm_wr_locked;
> }
>
> static inline void free_pgtables(struct mmu_gather *tlb, struct ma_state *mas,
> struct vm_area_struct *vma, unsigned long floor,
> unsigned long ceiling, bool mm_wr_locked)
> {
> - (void)tlb;
> - (void)mas;
> - (void)vma;
> - (void)floor;
> - (void)ceiling;
> - (void)mm_wr_locked;
> }
>
> -static inline void mapping_unmap_writable(struct address_space *)
> +static inline void mapping_unmap_writable(struct address_space *mapping)
> {
> }
>
> -static inline void flush_dcache_mmap_lock(struct address_space *)
> +static inline void flush_dcache_mmap_lock(struct address_space *mapping)
> {
> }
>
> -static inline void tlb_finish_mmu(struct mmu_gather *)
> +static inline void tlb_finish_mmu(struct mmu_gather *tlb)
> {
> }
>
> @@ -925,7 +910,7 @@ static inline struct file *get_file(struct file *f)
> return f;
> }
>
> -static inline int vma_dup_policy(struct vm_area_struct *, struct vm_area_struct *)
> +static inline int vma_dup_policy(struct vm_area_struct *src, struct vm_area_struct *dst)
> {
> return 0;
> }
> @@ -952,10 +937,6 @@ static inline void vma_adjust_trans_huge(struct vm_area_struct *vma,
> unsigned long end,
> struct vm_area_struct *next)
> {
> - (void)vma;
> - (void)start;
> - (void)end;
> - (void)next;
> }
>
> static inline void hugetlb_split(struct vm_area_struct *, unsigned long) {}
> @@ -975,51 +956,48 @@ static inline void vm_acct_memory(long pages)
> {
> }
>
> -static inline void vma_interval_tree_insert(struct vm_area_struct *,
> - struct rb_root_cached *)
> +static inline void vma_interval_tree_insert(struct vm_area_struct *vma,
> + struct rb_root_cached *rb)
> {
> }
>
> -static inline void vma_interval_tree_remove(struct vm_area_struct *,
> - struct rb_root_cached *)
> +static inline void vma_interval_tree_remove(struct vm_area_struct *vma,
> + struct rb_root_cached *rb)
> {
> }
>
> -static inline void flush_dcache_mmap_unlock(struct address_space *)
> +static inline void flush_dcache_mmap_unlock(struct address_space *mapping)
> {
> }
>
> -static inline void anon_vma_interval_tree_insert(struct anon_vma_chain*,
> - struct rb_root_cached *)
> +static inline void anon_vma_interval_tree_insert(struct anon_vma_chain *avc,
> + struct rb_root_cached *rb)
> {
> }
>
> -static inline void anon_vma_interval_tree_remove(struct anon_vma_chain*,
> - struct rb_root_cached *)
> +static inline void anon_vma_interval_tree_remove(struct anon_vma_chain *avc,
> + struct rb_root_cached *rb)
> {
> }
>
> -static inline void uprobe_mmap(struct vm_area_struct *)
> +static inline void uprobe_mmap(struct vm_area_struct *vma)
> {
> }
>
> static inline void uprobe_munmap(struct vm_area_struct *vma,
> unsigned long start, unsigned long end)
> {
> - (void)vma;
> - (void)start;
> - (void)end;
> }
>
> -static inline void i_mmap_lock_write(struct address_space *)
> +static inline void i_mmap_lock_write(struct address_space *mapping)
> {
> }
>
> -static inline void anon_vma_lock_write(struct anon_vma *)
> +static inline void anon_vma_lock_write(struct anon_vma *anon_vma)
> {
> }
>
> -static inline void vma_assert_write_locked(struct vm_area_struct *)
> +static inline void vma_assert_write_locked(struct vm_area_struct *vma)
> {
> }
>
> @@ -1029,16 +1007,16 @@ static inline void unlink_anon_vmas(struct vm_area_struct *vma)
> vma->anon_vma->was_unlinked = true;
> }
>
> -static inline void anon_vma_unlock_write(struct anon_vma *)
> +static inline void anon_vma_unlock_write(struct anon_vma *anon_vma)
> {
> }
>
> -static inline void i_mmap_unlock_write(struct address_space *)
> +static inline void i_mmap_unlock_write(struct address_space *mapping)
> {
> }
>
> -static inline void anon_vma_merge(struct vm_area_struct *,
> - struct vm_area_struct *)
> +static inline void anon_vma_merge(struct vm_area_struct *vma,
> + struct vm_area_struct *next)
> {
> }
>
> @@ -1047,27 +1025,22 @@ static inline int userfaultfd_unmap_prep(struct vm_area_struct *vma,
> unsigned long end,
> struct list_head *unmaps)
> {
> - (void)vma;
> - (void)start;
> - (void)end;
> - (void)unmaps;
> -
> return 0;
> }
>
> -static inline void mmap_write_downgrade(struct mm_struct *)
> +static inline void mmap_write_downgrade(struct mm_struct *mm)
> {
> }
>
> -static inline void mmap_read_unlock(struct mm_struct *)
> +static inline void mmap_read_unlock(struct mm_struct *mm)
> {
> }
>
> -static inline void mmap_write_unlock(struct mm_struct *)
> +static inline void mmap_write_unlock(struct mm_struct *mm)
> {
> }
>
> -static inline int mmap_write_lock_killable(struct mm_struct *)
> +static inline int mmap_write_lock_killable(struct mm_struct *mm)
> {
> return 0;
> }
> @@ -1076,10 +1049,6 @@ static inline bool can_modify_mm(struct mm_struct *mm,
> unsigned long start,
> unsigned long end)
> {
> - (void)mm;
> - (void)start;
> - (void)end;
> -
> return true;
> }
>
> @@ -1087,16 +1056,13 @@ static inline void arch_unmap(struct mm_struct *mm,
> unsigned long start,
> unsigned long end)
> {
> - (void)mm;
> - (void)start;
> - (void)end;
> }
>
> -static inline void mmap_assert_locked(struct mm_struct *)
> +static inline void mmap_assert_locked(struct mm_struct *mm)
> {
> }
>
> -static inline bool mpol_equal(struct mempolicy *, struct mempolicy *)
> +static inline bool mpol_equal(struct mempolicy *a, struct mempolicy *b)
> {
> return true;
> }
> @@ -1104,63 +1070,62 @@ static inline bool mpol_equal(struct mempolicy *, struct mempolicy *)
> static inline void khugepaged_enter_vma(struct vm_area_struct *vma,
> vm_flags_t vm_flags)
> {
> - (void)vma;
> - (void)vm_flags;
> }
>
> -static inline bool mapping_can_writeback(struct address_space *)
> +static inline bool mapping_can_writeback(struct address_space *mapping)
> {
> return true;
> }
>
> -static inline bool is_vm_hugetlb_page(struct vm_area_struct *)
> +static inline bool is_vm_hugetlb_page(struct vm_area_struct *vma)
> {
> return false;
> }
>
> -static inline bool vma_soft_dirty_enabled(struct vm_area_struct *)
> +static inline bool vma_soft_dirty_enabled(struct vm_area_struct *vma)
> {
> return false;
> }
>
> -static inline bool userfaultfd_wp(struct vm_area_struct *)
> +static inline bool userfaultfd_wp(struct vm_area_struct *vma)
> {
> return false;
> }
>
> -static inline void mmap_assert_write_locked(struct mm_struct *)
> +static inline void mmap_assert_write_locked(struct mm_struct *mm)
> {
> }
>
> -static inline void mutex_lock(struct mutex *)
> +static inline void mutex_lock(struct mutex *lock)
> {
> }
>
> -static inline void mutex_unlock(struct mutex *)
> +static inline void mutex_unlock(struct mutex *lock)
> {
> }
>
> -static inline bool mutex_is_locked(struct mutex *)
> +static inline bool mutex_is_locked(struct mutex *lock)
> {
> return true;
> }
>
> -static inline bool signal_pending(void *)
> +static inline bool signal_pending(void *p)
> {
> return false;
> }
>
> -static inline bool is_file_hugepages(struct file *)
> +static inline bool is_file_hugepages(struct file *file)
> {
> return false;
> }
>
> -static inline int security_vm_enough_memory_mm(struct mm_struct *, long)
> +static inline int security_vm_enough_memory_mm(struct mm_struct *mm, long pages)
> {
> return 0;
> }
>
> -static inline bool may_expand_vm(struct mm_struct *, vm_flags_t, unsigned long)
> +static inline bool may_expand_vm(struct mm_struct *mm, vm_flags_t flags,
> + unsigned long npages)
> {
> return true;
> }
> @@ -1185,7 +1150,7 @@ static inline void vm_flags_clear(struct vm_area_struct *vma,
> vma->__vm_flags &= ~flags;
> }
>
> -static inline int shmem_zero_setup(struct vm_area_struct *)
> +static inline int shmem_zero_setup(struct vm_area_struct *vma)
> {
> return 0;
> }
> @@ -1195,20 +1160,20 @@ static inline void vma_set_anonymous(struct vm_area_struct *vma)
> vma->vm_ops = NULL;
> }
>
> -static inline void ksm_add_vma(struct vm_area_struct *)
> +static inline void ksm_add_vma(struct vm_area_struct *vma)
> {
> }
>
> -static inline void perf_event_mmap(struct vm_area_struct *)
> +static inline void perf_event_mmap(struct vm_area_struct *vma)
> {
> }
>
> -static inline bool vma_is_dax(struct vm_area_struct *)
> +static inline bool vma_is_dax(struct vm_area_struct *vma)
> {
> return false;
> }
>
> -static inline struct vm_area_struct *get_gate_vma(struct mm_struct *)
> +static inline struct vm_area_struct *get_gate_vma(struct mm_struct *mm)
> {
> return NULL;
> }
> @@ -1233,16 +1198,16 @@ static inline void vma_set_page_prot(struct vm_area_struct *vma)
> WRITE_ONCE(vma->vm_page_prot, vm_page_prot);
> }
>
> -static inline bool arch_validate_flags(vm_flags_t)
> +static inline bool arch_validate_flags(vm_flags_t flags)
> {
> return true;
> }
>
> -static inline void vma_close(struct vm_area_struct *)
> +static inline void vma_close(struct vm_area_struct *vma)
> {
> }
>
> -static inline int mmap_file(struct file *, struct vm_area_struct *)
> +static inline int mmap_file(struct file *file, struct vm_area_struct *vma)
> {
> return 0;
> }
> @@ -1411,8 +1376,6 @@ static inline int mapping_map_writable(struct address_space *mapping)
>
> static inline unsigned long move_page_tables(struct pagetable_move_control *pmc)
> {
> - (void)pmc;
> -
> return 0;
> }
>
> @@ -1420,51 +1383,36 @@ static inline void free_pgd_range(struct mmu_gather *tlb,
> unsigned long addr, unsigned long end,
> unsigned long floor, unsigned long ceiling)
> {
> - (void)tlb;
> - (void)addr;
> - (void)end;
> - (void)floor;
> - (void)ceiling;
> }
>
> static inline int ksm_execve(struct mm_struct *mm)
> {
> - (void)mm;
> -
> return 0;
> }
>
> static inline void ksm_exit(struct mm_struct *mm)
> {
> - (void)mm;
> }
>
> static inline void vma_lock_init(struct vm_area_struct *vma, bool reset_refcnt)
> {
> - (void)vma;
> - (void)reset_refcnt;
> }
>
> static inline void vma_numab_state_init(struct vm_area_struct *vma)
> {
> - (void)vma;
> }
>
> static inline void vma_numab_state_free(struct vm_area_struct *vma)
> {
> - (void)vma;
> }
>
> static inline void dup_anon_vma_name(struct vm_area_struct *orig_vma,
> struct vm_area_struct *new_vma)
> {
> - (void)orig_vma;
> - (void)new_vma;
> }
>
> static inline void free_anon_vma_name(struct vm_area_struct *vma)
> {
> - (void)vma;
> }
>
> /* Declared in vma.h. */
> @@ -1518,7 +1466,6 @@ static inline int vfs_mmap_prepare(struct file *file, struct vm_area_desc *desc)
>
> static inline void fixup_hugetlb_reservations(struct vm_area_struct *vma)
> {
> - (void)vma;
> }
>
> static inline void vma_set_file(struct vm_area_struct *vma, struct file *file)
> @@ -1529,13 +1476,13 @@ static inline void vma_set_file(struct vm_area_struct *vma, struct file *file)
> fput(file);
> }
>
> -static inline bool shmem_file(struct file *)
> +static inline bool shmem_file(struct file *file)
> {
> return false;
> }
>
> -static inline vm_flags_t ksm_vma_flags(const struct mm_struct *, const struct file *,
> - vm_flags_t vm_flags)
> +static inline vm_flags_t ksm_vma_flags(const struct mm_struct *mm,
> + const struct file *file, vm_flags_t vm_flags)
> {
> return vm_flags;
> }
> --
> 2.50.1
>
prev parent reply other threads:[~2025-08-28 0:55 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-08-26 10:28 Lorenzo Stoakes
2025-08-26 19:45 ` Vlastimil Babka
2025-08-28 0:55 ` Liam R. Howlett [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=tldxsryzvi46gpcnbn4zusudc3pd65hbgvnhybtkkdusyfxj6w@6qy6k4gwruuf \
--to=liam.howlett@oracle.com \
--cc=akpm@linux-foundation.org \
--cc=jannh@google.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=lorenzo.stoakes@oracle.com \
--cc=pfalcato@suse.de \
--cc=vbabka@suse.cz \
--cc=wangyuli@uniontech.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox