[PATCH 2/4] kernel: oops: also check for WARN_ON messages found

[Keng-Yu Lin]

On Thu, Nov 15, 2012 at 8:05 PM, Colin King <colin.king at canonical.com> wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> The oops test may as well test for any WARN_ON messages we find while
> scanning for oops messages since these need to be checked too.
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/kernel/oops/oops.c |    9 ++++++++-
>  1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/src/kernel/oops/oops.c b/src/kernel/oops/oops.c
> index 33ae5ae..81691c0 100644
> --- a/src/kernel/oops/oops.c
> +++ b/src/kernel/oops/oops.c
> @@ -52,8 +52,9 @@ static int oops_deinit(fwts_framework *fw)
>  static int oops_test1(fwts_framework *fw)
>  {
>         int oopses = 0;
> +       int warn_ons = 0;
>
> -       if (fwts_oops_check(fw, klog, &oopses) != FWTS_OK) {
> +       if (fwts_oops_check(fw, klog, &oopses, &warn_ons) != FWTS_OK) {
>                 fwts_log_error(fw, "Error parsing kernel log.");
>                 return FWTS_ERROR;
>         }
> @@ -64,6 +65,12 @@ static int oops_test1(fwts_framework *fw)
>         else
>                 fwts_passed(fw, "Found no oopses in kernel log.");
>
> +       if (warn_ons > 0)
> +               fwts_failed(fw, LOG_LEVEL_HIGH,
> +                       "KernelWarnOns", "Found %d WARN_ON warnings in kernel log.", warn_ons);
> +       else
> +               fwts_passed(fw, "Found no WARN_ON warnings in kernel log.");
> +
>         return FWTS_OK;
>  }
>
> --
> 1.7.10.4
>

This will be interesting data to know.

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