[Skiboot] [PATCH] external/gard: Fix displaying 'cleared' gard records
Stewart Smith
stewart at linux.vnet.ibm.com
Thu Jan 7 16:08:09 AEDT 2016
Cyril Bur <cyril.bur at au1.ibm.com> writes:
> When a garded component is replaced hostboot detects this and updates the
> gard partition.
>
> What hostboot does is set the record_id field to 0xFFFFFFFF but leaves the
> rest of the flash untouched, this has caused issues with the gard tool the
> thinking was that an entire record of all 0xFF bytes would signal not a
> valid record.
>
> This patch add rectifies this issue and `gard list` will no longer show any
> record with an id of 0xFFFFFFFF.
>
> Signed-off-by: Cyril Bur <cyril.bur at au1.ibm.com>
> ---
> Stewart: Fixes BZ 131707
I gather this should also head towards stable ?
> diff --git a/external/gard/gard.c b/external/gard/gard.c
> index 0b7a68b..53f31d9 100644
> --- a/external/gard/gard.c
> +++ b/external/gard/gard.c
> @@ -41,6 +41,8 @@
>
> #include "gard.h"
>
> +#define CLEARED_RECORD_ID 0xFFFFFFFF
Prior to this patch, what was the check? We were just going until we saw
a all 0xFF gard record, right? It looks like that functionality isn't
removed, is it still correct?
--
Stewart Smith
OPAL Architect, IBM.
More information about the Skiboot
mailing list