Re: [PATCH] xen/efi: Reduce variable scope to void compiler warning

[Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>]

On Mon, Jul 21, 2025 at 09:37:06AM +0100, Frediano Ziglio wrote:
> This change removes some pieve of code working around with
> some compiler warnings.
> No functional change.
> 
> Signed-off-by: Frediano Ziglio <frediano.ziglio@xxxxxxxxx>

Acked-by: Marek Marczykowski-Górecki <marmarek@xxxxxxxxxxxxxxxxxxxxxx>

> ---
>  xen/common/efi/boot.c | 15 ++++-----------
>  1 file changed, 4 insertions(+), 11 deletions(-)
> 
> diff --git a/xen/common/efi/boot.c b/xen/common/efi/boot.c
> index 6c2ef13bc5..80d60ed54f 100644
> --- a/xen/common/efi/boot.c
> +++ b/xen/common/efi/boot.c
> @@ -1320,8 +1320,8 @@ void EFIAPI __init noreturn efi_start(EFI_HANDLE 
> ImageHandle,
>      static EFI_GUID __initdata shim_lock_guid = SHIM_LOCK_PROTOCOL_GUID;
>      EFI_LOADED_IMAGE *loaded_image;
>      EFI_STATUS status;
> -    unsigned int i, argc;
> -    CHAR16 **argv, *file_name, *cfg_file_name = NULL, *options = NULL;
> +    unsigned int i;
> +    CHAR16 *file_name, *cfg_file_name = NULL, *options = NULL;
>      UINTN gop_mode = ~0;
>      EFI_SHIM_LOCK_PROTOCOL *shim_lock;
>      EFI_GRAPHICS_OUTPUT_PROTOCOL *gop = NULL;
> @@ -1352,6 +1352,8 @@ void EFIAPI __init noreturn efi_start(EFI_HANDLE 
> ImageHandle,
>  
>      if ( use_cfg_file )
>      {
> +        unsigned int argc;
> +        CHAR16 **argv;
>          UINTN offset = 0;
>  
>          argc = get_argv(0, NULL, loaded_image->LoadOptions,
> @@ -1405,15 +1407,6 @@ void EFIAPI __init noreturn efi_start(EFI_HANDLE 
> ImageHandle,
>          if ( !base_video )
>              efi_console_set_mode();
>      }
> -    else
> -    {
> -        /*
> -         * Some compilers may emit a false "uninitialized use" warning for 
> argc,
> -         * so initialize argc/argv here to avoid the warning.
> -         */
> -        argc = 0;
> -        argv = NULL;
> -    }
>  
>      PrintStr(L"Xen " XEN_VERSION_STRING XEN_EXTRAVERSION
>            " (c/s " XEN_CHANGESET ") EFI loader\r\n");
> -- 
> 2.43.0
> 

-- 
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab

Attachment: signature.asc
Description: PGP signature