Re: [PATCH 1/4] efi: Move efi_status_to_err() to drivers/firmware/efi/

From: Ard Biesheuvel
Date: Mon Mar 21 2016 - 05:43:24 EST


On 17 March 2016 at 13:57, Matt Fleming <matt@xxxxxxxxxxxxxxxxxxx> wrote:
> Move efi_status_to_err() to the architecture independent code as it's
> generally useful in all bits of EFI code where there is a need to
> convert an efi_status_t to a kernel error value.
>
> Cc: Kweh Hock Leong <hock.leong.kweh@xxxxxxxxx>
> Cc: Borislav Petkov <bp@xxxxxxxxx>
> Cc: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>
> Signed-off-by: Matt Fleming <matt@xxxxxxxxxxxxxxxxxxx>

Acked-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx>

> ---
> drivers/firmware/efi/efi.c | 33 +++++++++++++++++++++++++++++++++
> drivers/firmware/efi/vars.c | 33 ---------------------------------
> include/linux/efi.h | 2 ++
> 3 files changed, 35 insertions(+), 33 deletions(-)
>
> diff --git a/drivers/firmware/efi/efi.c b/drivers/firmware/efi/efi.c
> index 3a69ed5ecfcb..b73222a52a9d 100644
> --- a/drivers/firmware/efi/efi.c
> +++ b/drivers/firmware/efi/efi.c
> @@ -637,3 +637,36 @@ u64 __weak efi_mem_attributes(unsigned long phys_addr)
> }
> return 0;
> }
> +
> +int efi_status_to_err(efi_status_t status)
> +{
> + int err;
> +
> + switch (status) {
> + case EFI_SUCCESS:
> + err = 0;
> + break;
> + case EFI_INVALID_PARAMETER:
> + err = -EINVAL;
> + break;
> + case EFI_OUT_OF_RESOURCES:
> + err = -ENOSPC;
> + break;
> + case EFI_DEVICE_ERROR:
> + err = -EIO;
> + break;
> + case EFI_WRITE_PROTECTED:
> + err = -EROFS;
> + break;
> + case EFI_SECURITY_VIOLATION:
> + err = -EACCES;
> + break;
> + case EFI_NOT_FOUND:
> + err = -ENOENT;
> + break;
> + default:
> + err = -EINVAL;
> + }
> +
> + return err;
> +}
> diff --git a/drivers/firmware/efi/vars.c b/drivers/firmware/efi/vars.c
> index 0ac594c0a234..4911d5697844 100644
> --- a/drivers/firmware/efi/vars.c
> +++ b/drivers/firmware/efi/vars.c
> @@ -314,39 +314,6 @@ check_var_size_nonblocking(u32 attributes, unsigned long size)
> return fops->query_variable_store(attributes, size, true);
> }
>
> -static int efi_status_to_err(efi_status_t status)
> -{
> - int err;
> -
> - switch (status) {
> - case EFI_SUCCESS:
> - err = 0;
> - break;
> - case EFI_INVALID_PARAMETER:
> - err = -EINVAL;
> - break;
> - case EFI_OUT_OF_RESOURCES:
> - err = -ENOSPC;
> - break;
> - case EFI_DEVICE_ERROR:
> - err = -EIO;
> - break;
> - case EFI_WRITE_PROTECTED:
> - err = -EROFS;
> - break;
> - case EFI_SECURITY_VIOLATION:
> - err = -EACCES;
> - break;
> - case EFI_NOT_FOUND:
> - err = -ENOENT;
> - break;
> - default:
> - err = -EINVAL;
> - }
> -
> - return err;
> -}
> -
> static bool variable_is_present(efi_char16_t *variable_name, efi_guid_t *vendor,
> struct list_head *head)
> {
> diff --git a/include/linux/efi.h b/include/linux/efi.h
> index c55c452b991a..9c8aae0e3b51 100644
> --- a/include/linux/efi.h
> +++ b/include/linux/efi.h
> @@ -1023,6 +1023,8 @@ static inline void
> efi_reboot(enum reboot_mode reboot_mode, const char *__unused) {}
> #endif
>
> +extern int efi_status_to_err(efi_status_t status);
> +
> /*
> * Variable Attributes
> */
> --
> 2.6.2
>