From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Fri, 16 Jan 2015 06:14:08 +0100
Subject: [U-Boot] [PATCHv1 02/22] arm: socfpga: enable saveenv to mmc
	partition
In-Reply-To: <54B8988A.1060102@denx.de>
References: <1421253662-27222-1-git-send-email-dinguyen@opensource.altera.com>
	<201501152308.35178.marex@denx.de> <54B8988A.1060102@denx.de>
Message-ID: <201501160614.08481.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Friday, January 16, 2015 at 05:50:18 AM, Stefan Roese wrote:

Hi!

> >> Would it make sense to make environment bigger (64K?) at this point,
> >> and enabling the redundant environment option, so that one bad block
> >> does not bring the whole device down?
> > 
> > Do you think you'll ever have more than a 4KiB of text data in the env ?
> 
> I have used bigger environment in some cases.
> 
> > I can see a reason for some 16 KiB environment, but that's the ceiling.
> 
> Then lets switch to at least 8KiB please.
> 
> Pavel, would be 8KiB enough for your project?
> 
> > Usually, the env only has to be a few KiB and it's size has to be
> > multiple of the underlying device's erase block. But in case of SD card,
> > there are no explicit erase blocks exported to the user, all is hidden
> > by the controller in the SD card and gives an illusion of a device which
> > has 512b blocks and the user doesn't care for bad block management.
> > Since the controller in the card does the bad block management and wear
> > leveling/relocation of blocks, it also means that in case the controller
> > did fail to provide the block which contains env, the whole card would
> > already be in a very bad state.
> 
> Right. Nevertheless it makes sense to support the redundant envrionment
> (at least from my point of view), regardless of the device the
> evironment is stored in. Thinking for example about power-supply failure
> (power-cut) while writing into the one envrionment version.

That's a fine idea, you're right :)

Best regards,
Marek Vasut