[Skiboot] [PATCH v2 0/5] libflash blocklevel ECC corrections

Samuel Mendoza-Jonas sam at mendozajonas.com
Thu Mar 8 15:03:33 AEDT 2018


On Wed, 2018-03-07 at 17:04 +1100, Cyril Bur wrote:
> V1 was submitted almost a year ago:
> https://lists.ozlabs.org/pipermail/skiboot/2017-April/006977.html
> 
> The core of the problem still exists, this problem does still need to
> get addressed in blocklevel rather than individually in each tool.
> 
> I've rebased this onto master and double checked that everything
> still appears to work.
> 
> Both the gard tool and pflash built with this series have are:
> Tested-by: Pridhiviraj Paidipeddi <ppaidipe at linux.vnet.ibm.com>
> 
> Stewart, I'm not sure if this means we should put the tag on all
> patches or just the last one. I've put it on the last one because
> that makes the most sense to me.

Reviewed-by: Samuel Mendoza-Jonas <sam at mendozajonas.com>

And heavily endorsed since bugs due to this were a huge pain to debug :)
Essentially we can never turn ECC back on for NVRAM without these changes
since the Linux kernel will try to write the lnx,oops-log partition at an
unaligned address.

> 
> 
> Cyril Bur (5):
>   gitignore: Add stb test kernel files
>   libflash/ecc: Add functions to deal with unaligned ECC memcpy
>   libflash/ecc: Add helpers to align a position within an ecc buffer
>   libflash/blocklevel: Return region start from ecc_protected()
>   libflash/blocklevel: Make read/write be ECC agnostic for callers
> 
>  .gitignore                      |   3 +
>  external/gard/gard.c            |  49 +++---
>  libflash/blocklevel.c           | 137 ++++++++++++++---
>  libflash/ecc.c                  | 247 +++++++++++++++++++++++++++---
>  libflash/ecc.h                  |   9 ++
>  libflash/test/test-blocklevel.c | 330 +++++++++++++++++++++++++++++++++++-----
>  libflash/test/test-ecc.c        |  31 ++++
>  7 files changed, 701 insertions(+), 105 deletions(-)
> 



More information about the Skiboot mailing list