Re: [PATCH] tools/testing/vma: add missing function stub

[Andrew Morton <akpm@linux-foundation.org>]

On Wed, 28 May 2025 15:15:39 +0100 Lorenzo Stoakes via B4 Relay <devnull+lorenzo.stoakes.oracle.com@kernel.org> wrote:

> From: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>
> 
> The hugetlb fix introduced in commit ee40c9920ac2 ("mm: fix copy_vma()
> error handling for hugetlb mappings") mistakenly did not provide a stub for
> the VMA userland testing, which results in a compile error when trying to
> build this.

Thanks, I'll add the Fixes: and the cc:stable (because ee40c9920ac2 had
cc:stable).

> Provide this stub to resolve the issue.
> 
> ---
> Signed-off-by: Lorenzo Stoakes <lorenzo.stoakes@oracle.com>

SOB goes above the "^---$", please ;)

> --- a/tools/testing/vma/vma_internal.h
> +++ b/tools/testing/vma/vma_internal.h
> @@ -1461,4 +1461,9 @@ static inline int __call_mmap_prepare(struct file *file,
>  	return file->f_op->mmap_prepare(desc);
>  }
>  
> +static inline void fixup_hugetlb_reservations(struct vm_area_struct *vma)
> +{
> +	(void)vma;

<Curiousity> Was this statement found to be needed?  Normally we just
use an empty function body in such cases.

> +}
> +
>  #endif	/* __MM_VMA_INTERNAL_H */