[PATCH 1/3] uefi: uefidump: Add support for HwErrRecSupport

[Keng-Yu Lin]

On Thu, Sep 5, 2013 at 4:27 PM, Ivan Hu <ivan.hu at canonical.com> wrote:
> HwErrRecSupport identifies the level of hardware error record persistence
> support implemented by the platform.
>
> Signed-off-by: Ivan Hu <ivan.hu at canonical.com>
> ---
>  src/uefi/uefidump/uefidump.c |   25 +++++++++++++++++++++++++
>  1 file changed, 25 insertions(+)
>
> diff --git a/src/uefi/uefidump/uefidump.c b/src/uefi/uefidump/uefidump.c
> index dd58a3d..dc576c7 100644
> --- a/src/uefi/uefidump/uefidump.c
> +++ b/src/uefi/uefidump/uefidump.c
> @@ -656,6 +656,30 @@ static void uefidump_info_signature_support(fwts_framework *fw, fwts_uefi_var *v
>         }
>  }
>
> +static void uefidump_info_hwerrrec_support(fwts_framework *fw, fwts_uefi_var *var)
> +{
> +       if (var->datalen != 2) {
> +               /* Should be 2 byte, of not, dump it out as a hex dump */
> +               uefidump_var_hexdump(fw, var);
> +       } else {
> +               char *support;
> +               uint16_t *value = (uint16_t *)var->data;
> +
> +               switch (*value) {
> +               case 0:
> +                       support = " (Not support for Hardware Error Record Persistence)";
> +                       break;
> +               case 1:
> +                       support = " (Support for Hardware Error Record Persistence)";
> +                       break;
> +               default:
> +                       support = " (reserved value)";
> +                       break;
> +               }
> +               fwts_log_info_verbatum(fw, "  Value: 0x%4.4" PRIx16 "%s.", *value, support);
> +       }
> +}
> +
>  static uefidump_info uefidump_info_table[] = {
>         { "PlatformLangCodes",  uefidump_info_platform_langcodes },
>         { "PlatformLang",       uefidump_info_platform_lang },
> @@ -680,6 +704,7 @@ static uefidump_info uefidump_info_table[] = {
>         { "MemoryOverwriteRequestControl",      uefidump_info_morc },
>         { "AcpiGlobalVariable", uefidump_info_acpi_global_variable },
>         { "SignatureSupport",   uefidump_info_signature_support },
> +       { "HwErrRecSupport",    uefidump_info_hwerrrec_support },
>         { NULL, NULL }
>  };
>
> --
> 1.7.9.5
>

Acked-by: Keng-Yu Lin <kengyu at canonical.com>