[Skiboot] [PATCH] libflash/blocklevel: Allow double ecc protecting a region

Stewart Smith stewart at linux.vnet.ibm.com
Tue Jul 12 17:47:29 AEST 2016


Stewart Smith <stewart at linux.vnet.ibm.com> writes:
> Cyril Bur <cyril.bur at au1.ibm.com> writes:
>> Currently the policy for calling ECC protecting a range at the
>> blocklevel layer is that the requested region be completely
>> unprotected otherwise the call will return an error. It turns out that
>> duplicate calls to ffs_init() with true as the last parameter (for the
>> same blocklevel structure) will cause duplicate attempts to
>> ecc_protect() ranges.
>>
>> Change the policy within blocklevel to allow duplicate protecting.
>> In fact the new policy almost guarantees no failure (baring something
>> odd like malloc() failing). It will detect that the range is currently
>> already fully protected and do nothing, detect that part of the range
>> is or is not and extend the existing range or detect that a range fits
>> perfectly between two ranges in which case it will merge the ranges.
>>
>> Signed-off-by: Cyril Bur <cyril.bur at au1.ibm.com>
>> ---
>> Fixes issue exposed by Alistairs patch: 74ba834 ("flash: Use
>> blocklevel to do ECC reads")
>
> Thanks, 74ba834 reapplied and this patch also applied to master as of
> 0a602c35e5f42aa18ee769cbd6b356ea480eb0df

Actually, no.

Just noticed make check failing just before I pushed:

make -j20 libflash/test/test-blocklevel-check
        [ HOSTCC ]  libflash/test/test-blocklevel.c
        [ RUN-TEST ]  libflash/test/test-blocklevel
Shouldn't have succeeded blocklevel_ecc_protect(0x100, 0x1000)
/home/stewart/skiboot/libflash/test/Makefile.check:17: recipe for target
        'libflash/test/test-blocklevel-check' failed


So I haven't pushed that merge and reapply, and instead will go and make
'make check' output a bit saner.


-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Skiboot mailing list