Re: [MTD] UBI: Per volume update marker

Frank Haverkamp
2007-01-26T08:52:24Z

Hi Artem,

On Thu, 2007-01-25 at 20:16 +0200, Artem Bityutskiy wrote:

> The patch in general (did not look closely to the details) looks good
> for me. But are you sure you want this patch? It was a requirement from
> your team to implement the update marker.

The requirement was that we are able to avoid using an UBI volume where
the update was interrupted, and _not_ the update marker. The
update-marker was just one idea how this feature could be implemented.
Alexanders proposal is another idea, which we think helps to reduce
complexity.

> Note, your patch means that
> the boot-loader needs to read and interpret the volume table which
> increases its size.
>

The simplest form of our boot-loader does not know anything about the
volume table and update maker block at all, but just deals with static
volumes. If all blocks of a static volumes are available and have good
CRC the boot-code will use it.

There is also a boot-code enhancement done by Josh which exploits the
volume table and possibly the update marker. If we decide to use the new
mechansim, I think this code needs a little adjustement too, but I think
we can handle that. If that becomes neccessary we also hope that some
complexity will go away there.

Alexander will update the patch according to the comments you gave in
your last mail and resend the patch.

Frank