Re: [PATCH v2 0/5] UBIFS: fix recovery on CFI NOR

Artem Bityutskiy
2011-02-11T13:46:14Z

On Sun, 2011-02-06 at 15:17 +0200, Artem Bityutskiy wrote:
> Hi,
>
> here is a better patch for recovery fix. Comparing to the previous
> patch-set now we make sure we keep write-buffer offset aligned to
> @c->max_write_size (64 in case of CFI NOR) as much as possible.
>
> Also, I've merged the "Add comments" patch with the patch which adds
> the code.
>
> You can find these patches also in the UBIFS git tree, 'cfi-nor-fix-v2'
> branch:
> git://git.infradead.org/ubifs-2.6.git cfi-nor-fix-v2
>
> Please, test. These patches may break NAND setups as well, so anyone
> who is interested in having stable UBIFS in the next release, please,
> also test.

Just wanted to let everyone know that I'll need to work on a bit better
solution.

The reason is that LEB does not have to start at the max_write_size
boundary, and my v2 does not take that into account. Although for CFI
NOR case this should not matter, because LEB starts at offset 128 (UBI
headers take 64 bytes each). However, this might matter on some other
flash.

--
Best Regards,
Artem Bityutskiy (Артём Битюцкий)