public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: Richard Titmuss <richard_titmuss@logitech.com>
To: dedekind@infradead.org
Cc: linux-mtd@lists.infradead.org
Subject: Re: Bootloader support for UBI images
Date: Wed, 02 Jul 2008 08:57:13 +0100	[thread overview]
Message-ID: <486B34D9.2070009@logitech.com> (raw)
In-Reply-To: <1214915164.6573.75.camel@sauron>

Artem Bityutskiy wrote:

>>  Also I need support for 
>> renaming multiple volumes atomically when switching images after the 
>> upgrade. If these sound sensible and useful changes I can look at 
>> creating patches to add these features.
>>     
> Also sounds as a good and useful idea. And implementation should not
> difficult - it is just about changing the volume table. You may count on
> my assistance.
>
> I'm just not sure about the interface. Renaming one volume is fine. But
> you need to rename 2, so the interface should be more complex. And if 2,
> why not 3, or 5? Or 16? Or any amount from 1 to UBI_MAX_VOLUMES?
>   
Right, I was thinking that it would allow any number of volumes for 
renaming. I am also not sure about the interface, and was hoping to get 
some suggestions here. I will think about it and propose something in a 
few days.

> So you use the sub-page read patch BTW?
No, I'd missed that. Where is it?

> Well, surely the boot-loader may create a per-PEB (physical eraseblock)
> array and put all information about each PEB there (mapped/unmapped, LEB
> number, erase-counter, etc)? Then in UBI it will be easy to scan this
> array instead of the flash media. I am just not aware how such kind of
> bootloader->kernel data passing is done in Linux. Can you come up with a
> nice and mainline-acceptible way?
>   
Ok, that is possible. This would also have to include the bad block and 
bit flip status for each PEB. At the moment the Redboot flash api does 
not make this available, so it would mean extending Redboot or writing 
custom flash drivers to access this information.

> I did not actually catch the second part (LER and PER) tables - this is
> something I am not aware of. UBI has only volume table on flash, the
> other things are in-memory and build by scanning.
>   
It was a proposal to _add_ an addition table, that provided a LER to PER 
mapping for static volumes. If this table was stored near the beginning 
of the flash, then it could be accessed without a full scan. This could 
only be used if the UBI volume was mounted read-only, any write 
operations would require scanning all the flash blocks. This is an 
alternative approach to passing data from the bootloader to the kernel.

Richard

  parent reply	other threads:[~2008-07-02  7:57 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-07-01 11:43 Bootloader support for UBI images Richard Titmuss
2008-07-01 12:26 ` Artem Bityutskiy
2008-07-01 12:34   ` Artem Bityutskiy
2008-07-02  7:57   ` Richard Titmuss [this message]
2008-07-02  9:02     ` Artem Bityutskiy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=486B34D9.2070009@logitech.com \
    --to=richard_titmuss@logitech.com \
    --cc=dedekind@infradead.org \
    --cc=linux-mtd@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox