[U-Boot] [PATCH] ubi: fastmap: Implement produce_free_peb()

[Heiko Schocher]

Hello Patrice,

Am 22.05.2018 um 10:10 schrieb Patrice Chotard:
> Since 'commit f82290afc847 ("mtd: ubi: Fix worker handling")',
> when booting from NAND, on a fresh NAND just after being flashed (and
> only in this case), we got the following log:
> 
> ubi0: default fastmap pool size: 200
> ubi0: default fastmap WL pool size: 100
> ubi0: attaching mtd2
> ubi0: scanning is finished
> ubi0 error: ubi_update_fastmap: could not find any anchor PEB
> ubi0 error: ubi_update_fastmap: could not find any anchor PEB
> ubi0 error: ubi_wl_get_peb: Unable to get a free PEB from user WL pool
> ubi0 error: autoresize: cannot auto-resize volume 1
> UBI error: cannot attach mtd2UBI error: cannot initialize UBI, error
> -28UBI init error 28
> 
> After analysis, in ubi_wl_init(), when performing schedule_erase(),
> thread_enabled flag is not yet set to 1, which forbids ubi_do_worker()
> to execute pending works.
> 
> This has to effect to not populate ubi->free with free physical
> eraseblocks.
> 
> Following Richard Weinberger's advice, this patch has been
> backported from kernel tree :
> 'commit 1cb8f9776c7d ("ubi: fastmap: Implement produce_free_peb()")'
> 
> Tested-by: Patrice Chotard <patrice.chotard at st.com>
> Signed-off-by: Richard Weinberger <richard at nod.at>
> Signed-off-by: Patrice Chotard <patrice.chotard at st.com>
> ---
> 
>   drivers/mtd/ubi/fastmap-wl.c | 29 +++++++++++++++++++++++++++++
>   1 file changed, 29 insertions(+)

Thanks!

Acked-by: Heiko Schocher <hs at denx.de>

I do some tests and a travis build for it ...

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs at denx.de