Re: [RFC][PATCH] UBI: Make MTD_UBI_FASTMAP non-experimental

2017-05-09 Thread Richard Weinberger
Jesper, Am 09.05.2017 um 09:46 schrieb Jesper Nilsson: > Hi Richard, > > I'm still worried about this failure case, do we really > believe that the flash could fail in such a way that the > fastmap is corrupted in an undetectable way? > > If we do detect corruption we should be no worse off > th

Re: [RFC][PATCH] UBI: Make MTD_UBI_FASTMAP non-experimental

2017-05-09 Thread Jesper Nilsson
Hi Richard, I'm still worried about this failure case, do we really believe that the flash could fail in such a way that the fastmap is corrupted in an undetectable way? If we do detect corruption we should be no worse off than earlier since we should ignore the fastmap, IIRC. Could you please e

Re: [RFC][PATCH] UBI: Make MTD_UBI_FASTMAP non-experimental

2017-04-03 Thread Jesper Nilsson
Hi Richard, On Thu, Mar 30, 2017 at 11:29:15PM +0200, Richard Weinberger wrote: > Jesper, > > Am 30.03.2017 um 19:39 schrieb Jesper Nilsson: > >> So we should document this with a big fat warning and set fastmap to > >> default=n ? > > > > Does this sound reasonable? > > > > Note that this feat

Re: [RFC][PATCH] UBI: Make MTD_UBI_FASTMAP non-experimental

2017-03-30 Thread Richard Weinberger
Jesper, Am 30.03.2017 um 19:39 schrieb Jesper Nilsson: >> So we should document this with a big fat warning and set fastmap to >> default=n ? > > Does this sound reasonable? > > Note that this feature makes UBI less robust, since Fastmap does not scan > the full flash, which might lead to proble

Re: [RFC][PATCH] UBI: Make MTD_UBI_FASTMAP non-experimental

2017-03-30 Thread Jesper Nilsson
Hi Richard, Marek, On Thu, Mar 30, 2017 at 12:01:41PM +0200, Marek Vasut wrote: > On 03/29/2017 10:04 PM, Richard Weinberger wrote: > > Jesper, > > > > Am 29.03.2017 um 17:38 schrieb Jesper Nilsson: > >> MTD_UBI_FASTMAP has been set as experimental since it > >> was merged back in 2012. > >> > >

Re: [RFC][PATCH] UBI: Make MTD_UBI_FASTMAP non-experimental

2017-03-30 Thread Marek Vasut
On 03/29/2017 10:04 PM, Richard Weinberger wrote: > Jesper, > > Am 29.03.2017 um 17:38 schrieb Jesper Nilsson: >> MTD_UBI_FASTMAP has been set as experimental since it >> was merged back in 2012. >> >> There hasn't been much change in the format, >> so we can consider the feature stable and start

Re: [RFC][PATCH] UBI: Make MTD_UBI_FASTMAP non-experimental

2017-03-29 Thread Richard Weinberger
Jesper, Am 29.03.2017 um 17:38 schrieb Jesper Nilsson: > MTD_UBI_FASTMAP has been set as experimental since it > was merged back in 2012. > > There hasn't been much change in the format, > so we can consider the feature stable and start > being careful about breaking the format. > (This is somewh

[RFC][PATCH] UBI: Make MTD_UBI_FASTMAP non-experimental

2017-03-29 Thread Jesper Nilsson
MTD_UBI_FASTMAP has been set as experimental since it was merged back in 2012. There hasn't been much change in the format, so we can consider the feature stable and start being careful about breaking the format. (This is somewhat of a pre-requisite for anyone actually using the feature in the rea