[Skiboot] [PATCH v2 0/5] libflash blocklevel ECC corrections
Cyril Bur
cyril.bur at au1.ibm.com
Tue Mar 13 09:10:09 AEDT 2018
On Thu, 2018-03-08 at 15:03 +1100, Samuel Mendoza-Jonas wrote:
> On Wed, 2018-03-07 at 17:04 +1100, Cyril Bur wrote:
> > V1 was submitted almost a year ago:
> > https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.ozlabs.org_pipermail_skiboot_2017-2DApril_006977.html&d=DwICaQ&c=jf_iaSHvJObTbx-siA1ZOg&r=enAa2wJuZFP2PsCNSamrs-fnBSp0sgKE8NVKBN2MoOk&m=oBR8sexxUpHnpWQzH6srRNUq_Z-DXZohtQk6wC5Vz84&s=6XCN_fCIoW1EcRtTwauRMcNJL7IkY5CpHF-xrDcRB7k&e=
> >
> > 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>
>
Thanks!
> 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.
>
Oh and much bigger thanks for remembering this particular example of
the bug. That was a massive pain.
> >
> >
> > 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