Re: [PATCH v2 5/9] mm: Initialize struct vm_unmapped_area_info

[Kirill A. Shutemov]

On Mon, Feb 26, 2024 at 11:09:47AM -0800, Rick Edgecombe wrote:
> diff --git a/arch/alpha/kernel/osf_sys.c b/arch/alpha/kernel/osf_sys.c
> index 5db88b627439..dd6801bb9240 100644
> --- a/arch/alpha/kernel/osf_sys.c
> +++ b/arch/alpha/kernel/osf_sys.c
> @@ -1218,7 +1218,7 @@ static unsigned long
>  arch_get_unmapped_area_1(unsigned long addr, unsigned long len,
>                        unsigned long limit)
>  {
> -     struct vm_unmapped_area_info info;
> +     struct vm_unmapped_area_info info = {};
>  
>       info.flags = 0;
>       info.length = len;

Can we make a step forward and actually move initialization inside the
initializator? Something like below.

I understand that it is substantially more work, but I think it is useful.

diff --git a/arch/alpha/kernel/osf_sys.c b/arch/alpha/kernel/osf_sys.c
index 5db88b627439..c40ddede3b13 100644
--- a/arch/alpha/kernel/osf_sys.c
+++ b/arch/alpha/kernel/osf_sys.c
@@ -1218,14 +1218,12 @@ static unsigned long
 arch_get_unmapped_area_1(unsigned long addr, unsigned long len,
                         unsigned long limit)
 {
-       struct vm_unmapped_area_info info;
+       struct vm_unmapped_area_info info = {
+               .length = len;
+               .low_limit = addr,
+               .high_limit = limit,
+       };

-       info.flags = 0;
-       info.length = len;
-       info.low_limit = addr;
-       info.high_limit = limit;
-       info.align_mask = 0;
-       info.align_offset = 0;
        return vm_unmapped_area(&info);
 }

-- 
  Kiryl Shutsemau / Kirill A. Shutemov