From: Tolunay Orkun <listmember@orkun.us>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Re: Redundant environment
Date: Fri, 05 May 2006 11:42:46 -0500 [thread overview]
Message-ID: <445B8086.9000404@orkun.us> (raw)
In-Reply-To: <20060501231345.89B643525C5@atlas.denx.de>
Wolfgang Denk wrote:
> Dear Tolunay,
>
> in message <445691EF.1000401@orkun.us> you wrote:
>
>> Yes, I can do it in saveenv code to cycle twice but I would rather avoid
>> doing unlock/re-lock/over flag byte stuff twice.
>>
>> Whichever way Wolfgang favors I am ready to work on a patch.
>>
>
> I think adding another set of N #ifdef's to implement this feature is
> not a good idea, when a single one (to duplicate the call to the C
> function) does basicly the same.
>
OK. That makes the patch simpler.
> Ummm... sorry for being stubborn, but before you start can you please
> re-try to explain to me in which specific situations you expect this
> patch to actually improve the reliability of operation of the device?
>
This patch would solve the issue that exists today that when the
"active" environment is lost/corrupted for some reason the "redundant"
environment would contain an exact copy of the primary to have the board
come up without requiring the need to redo the changes that was lost on
last save. Sometimes these changes could be critical enough not to allow
the system boot the OS properly anymore (like changes to bootcmd,
bootargs etc).
Among the things that can cause one environment to go corrupt would be
charge decays in memory cells in aging flash, supply variations/noise
during erase/write and random memory corruption when power is
interrupted while another section of flash memory is being written/erased.
Sure these could cause other problems as well like if this issue happens
for U-Boot code the system might become un-bootable. But at least we
have full recovery for the case when it happens within U-Boot environment.
> I am aware that some people interpreted the term "redundand environ-
> ment" that two identical copies of the environment were stored. This
> was obviously an unlucky choice of the name for this feature. Please
> let's exclude this "I expected to see this, now change the code to
> match my expectations" aspect for a moment. However, I still fail to
> see any improvements in the suggested change; actually I only see
> disadvantages like doubling the number of flash erase cycles for the
> environment sectors.
>
I understand you concern. In our application the environment would not
be updated occasionally so that is not a big concern for us.
Best regards,
Tolunay
next prev parent reply other threads:[~2006-05-05 16:42 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-04-28 16:32 [U-Boot-Users] Redundant environment Tolunay Orkun
2006-04-28 19:31 ` [U-Boot-Users] " Wolfgang Denk
2006-05-01 6:19 ` Stefan Roese
2006-05-01 22:55 ` Tolunay Orkun
2006-05-01 23:13 ` Wolfgang Denk
2006-05-05 16:42 ` Tolunay Orkun [this message]
2006-05-05 16:45 ` Tolunay Orkun
2006-05-05 23:28 ` Wolfgang Denk
2006-05-08 13:09 ` Jerry Van Baren
2006-05-22 21:11 ` Tolunay Orkun
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=445B8086.9000404@orkun.us \
--to=listmember@orkun.us \
--cc=u-boot@lists.denx.de \
/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